feat(mcp): rewrite tool descriptions for agent consumption - #848
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: fdf5a0a392
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Fix inaccurate debug context description, restore lost issue prefix semantics, strengthen delete warning, add side-effect and overwrite warnings, and resolve circular HMR/hot-reload guidance.
vf_get_status: clarify it always reports running=true, not a real health check vf_hot_reload: clarify it is a no-op stub, point to vf_trigger_hmr for actual HMR vf_trigger_hmr: remove misleading cross-reference to no-op vf_hot_reload
Keep our updated descriptions while incorporating title, annotations, and other structural additions from main.
vf_list_routes: include error, loading, and not-found in route types listed vf_list_integrations: cross-reference vf_create_project instead of vf_list_templates
…l descriptions Ensures every in-scope tool description mentions return value shape and has a negative case cross-reference, per acceptance criteria.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7b1d143fe7
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Narrow the "Do not use" guidance to error counts/uptime instead of "current status", since vf_get_status does not probe reachability.
Summary
.describe()parameter annotations (e.g.,Example: 'compile'. Omit to return all types.)vf_get_status(always reports running=true),vf_hot_reload(no-op stub),vf_get_debug_context(actual return fields)issues_delete(irreversible),vf_scaffold(may overwrite),vf_preview_route(API side effects)issues_createprefix semantics (TASK/PLAN/ISSUE guidance)Closes #835
Out of scope
cli/mcp/tools/cicd-tools.tsandcli/mcp/remote-file-tools.tsare not listed in feat(mcp): rewrite tool descriptions for agent consumption #835's file list and were not modifiedTest plan
.describe()with examples and defaultsdeno fmtpassesdeno lintpasses