Skip to content

fix: MCP 云函数日志查询工具参数设计与错误提示不清晰#616

Open
binggg wants to merge 1 commit intomainfrom
automation/attribution-issue-mo8z1wxq-k14q0z-mcp
Open

fix: MCP 云函数日志查询工具参数设计与错误提示不清晰#616
binggg wants to merge 1 commit intomainfrom
automation/attribution-issue-mo8z1wxq-k14q0z-mcp

Conversation

@binggg
Copy link
Copy Markdown
Member

@binggg binggg commented Apr 28, 2026

Attribution issue

  • issueId: issue_mo8z1wxq_k14q0z
  • category: tool
  • canonicalTitle: MCP 云函数日志查询工具参数设计与错误提示不清晰
  • representativeRun: atomic-js-cloudbase-query-function-logs/2026-04-21T18-42-05-o3qo8r

Automation summary

All tests pass and TypeScript compilation succeeds. Let me provide a summary of the changes. ## SUMMARY - root_cause: The MCP log query tools (queryLogs and queryFunctions) had unclear documentation about when to use each tool for querying cloud function logs. The queryLogs tool's queryString parameter was marked as optional in the schema but required for action=searchLogs, leading to confusing error messages. Additionally, the CLS query syntax was not documented, causing agents to fail when trying to query logs. - changes: 1. mcp/src/tools/logs.ts: - Added comprehensive tool description explaining when to use queryLogs vs queryFunctions - Added detailed parameter descriptions with CLS query syntax examples - Improved error message when queryString is missing, including query examples and guidance to use queryFunctions for function-specific logs 2. mcp/src/tools/functions.ts: - Added clear tool description explaining that this tool should be used for function logs - Added detailed action descriptions listing all available actions - Added parameter descriptions with guidance on which parameters are required for each action 3. **config/source/skills/

Changed files

  • config/.claude/skills/cloud-functions/SKILL.md
  • config/source/skills/cloud-functions/SKILL.md
  • mcp/src/tools/functions.ts
  • mcp/src/tools/logs.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-28T14:59:39.526Z

Cases

  • [PASSED] — primary — evaluation passed

@binggg
Copy link
Copy Markdown
Member Author

binggg commented Apr 29, 2026

  1. config/.claude/skills/cloud-functions/SKILL.md

这个文件存在问题,原文是英文,改成了中文的

  1. 代码冲突了

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