Skip to content

fix: MCP 删除云函数工具参数设计不清晰导致调用失败#634

Merged
binggg merged 1 commit intomainfrom
automation/attribution-issue-mo8yl3n0-8qa2vq-mcp
Apr 29, 2026
Merged

fix: MCP 删除云函数工具参数设计不清晰导致调用失败#634
binggg merged 1 commit intomainfrom
automation/attribution-issue-mo8yl3n0-8qa2vq-mcp

Conversation

@binggg
Copy link
Copy Markdown
Member

@binggg binggg commented Apr 28, 2026

Attribution issue

  • issueId: issue_mo8yl3n0_8qa2vq
  • category: tool
  • canonicalTitle: MCP 删除云函数工具参数设计不清晰导致调用失败
  • representativeRun: atomic-js-cloudbase-delete-function/2026-04-21T18-29-44-492zv5

Automation summary

  • root_cause: The MCP manageFunctions tool was missing the deleteFunction action in its MANAGE_FUNCTION_ACTIONS enum, even though the underlying CloudBase Manager SDK (@cloudbase/manager-node) already supported cloudbase.functions.deleteFunction(name). This caused agents to receive "400 invalid parameter value" errors when attempting to delete cloud functions through the MCP tool.
  • changes: Added the deleteFunction action to mcp/src/tools/functions.ts:
  1. Added "deleteFunction" to the MANAGE_FUNCTION_ACTIONS array
  2. Implemented the case "deleteFunction": handler that requires functionName parameter and confirm=true for safety, then calls cloudbase.functions.deleteFunction(input.functionName)
  3. Updated the action schema description to include deleteFunction as an example
  4. Enhanced the confirm parameter description to clarify which destructive actions require explicit confirmation
  • validation: All existing tests pass (functions.test.ts, build-skills-repo.test.js, build-compat-config.test.js, skill-quality-standards.test.js), and TypeScript compilation succeeds with no errors.
  • follow_up: No further action needed. The fix enables proper clo

Changed files

  • mcp/src/tools/functions.ts

@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@binggg
Copy link
Copy Markdown
Member Author

binggg commented Apr 28, 2026

Attribution post-PR evaluation

  • visibility: internal identifiers, run ids, and private links are intentionally omitted
  • attempt: 1
  • eval_scope: primary_only
  • overall: PASSED
  • summary: all planned evaluation cases passed
  • updated_at: 2026-04-28T18:27:43.547Z

Cases

  • [PASSED] — primary — evaluation passed

@binggg binggg merged commit b4d2168 into main Apr 29, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant