-
Notifications
You must be signed in to change notification settings - Fork 36
Make tool parameter optional in missing_tool safe output #9600
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
🎬 THE END — Smoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨ |
|
💀 Blimey! Changeset Generator failed and walked the plank! No treasure today, matey! ☠️ |
|
📰 DEVELOPING STORY: Smoke Copilot reports failed. Our correspondents are investigating the incident... |
|
✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟 |
Claude Engine Smoke Test ResultsLast 2 Merged PRs:
Test Results: Status: PARTIAL PASS - Core functionality verified, 2 tools missing
|
The
missing_toolsafe output required bothtoolandreasonparameters, preventing agents from reporting general limitations without naming a specific tool. The MCP server validation rejected calls missing thetoolparameter.Changes
Schema updates (
safe_outputs_tools.json,agent-output.json):required: ["tool", "reason"]→required: ["reason"]Handler updates (
missing_tool.cjs):toolfieldnulltool valuesType definitions (
safe-outputs.d.ts):tool: string→tool?: stringUsage
All existing tests pass. Added test coverage for optional tool parameter.
Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.