P-2393 Track the trimmed public API surface - #43
Merged
Conversation
- `wallet_profiles` and `project_retention` are no longer in the MCP allow-list; reference `retention` and `search_profile` instead. - Note the MCP server's management tool coverage (alerts, boards/charts, contracts, segments, profile writes, wallet import) so agents don't assume the CLI is the only management surface. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01BWrVEx5C374qkrAQUCgx7y
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
yosriady
changed the title
Follow-up to getformo/formono#2182 review decisions — the alert-test,
contract-recommendations, contract-pipeline, and chart-meta routes are
now internal-only (dashboard utilities), and GET /boards/{b}/charts is
lazy by default with ?include=results:
- Remove `alerts test`, `contracts recommendations`, and
`contracts pipeline` (pipeline inclusion is set via
`contracts update --include-in-pipeline`).
- `charts list` returns summaries by default and gains `--results` to
execute chart queries; `charts meta` now reads the list endpoint's
summary default instead of the removed /charts/meta route.
- Update the formo-analytics skill notes accordingly.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01BWrVEx5C374qkrAQUCgx7y
yosriady
changed the title
The hunk removing the recommendations block also took the get
registration with it. GET /v0/contracts/{chain}/{address} is still a
public route, so the command comes back verbatim from main.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
yosriady
added a commit
that referenced
this pull request
Aug 6, 2026
…face (#45) Minor bump for the surface trim in #43 (alerts test, contracts recommendations, contracts pipeline removed; adoption is near-zero so we skip the major). README and SKILLS drop the removed sections, document charts list's summaries default + --results, and point pipeline toggling at contracts update --include-in-pipeline. Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Syncs the CLI with getformo/formono#2182 (machine-surface parity + surface trim, P-2393). The backend made the alert-test, contract-recommendations, contract-pipeline, and chart-meta routes internal-only, and
GET /boards/{b}/chartslazy by default with?include=results— released CLI versions calling the removed routes will 404 on those commands.alerts test,contracts recommendations, andcontracts pipeline(pipeline inclusion is set viacontracts update --include-in-pipeline).charts listnow returns summaries by default and gains--resultsto execute chart queries;charts metareads the list endpoint's summary default instead of the removed/charts/metaroute.skills/formo-analytics/SKILL.md: fix stale proxied tool names (wallet_profiles/project_retention→retention/search_profile), note MCP's management tool coverage, drop the removed operations.Companion PR: getformo/docs.formo.so#134.
🤖 Generated with Claude Code
https://claude.ai/code/session_01BWrVEx5C374qkrAQUCgx7y
Generated by Claude Code
<img alt="View with [code]smith" src="https://pr-comments-assets.blacksmith.sh/codesmith/view-with-codesmith-dark-v2.svg"><a href="https://backend.blacksmith.sh/track/enable-autofix?expires=1788594740&installation_model_id=21031&pr_number=43&repository=getformo%2Fcli&return_to=https%3A%2F%2Fgithub.com%2Fgetformo%2Fcli%2Fpull%2F43&signature=89b10b8b0275edcd9b050282452f571df2f1edc8eb6370af7ccff83cf249b6b5"rel="nofollow noreferrer noopener" target="_blank"><img alt="Autofix with [code]smith" src="https://pr-comments-assets.blacksmith.sh/codesmith/autofix-with-codesmith-dark.svg">Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.<img alt="Review in cubic" src="https://www.cubic.dev/buttons/review-in-cubic-dark.svg">