| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
|
@claude can you resolve the conflicts here? |
Sorry, something went wrong.
|
I resolved the conflicts. The failing Claude Code Review appears to be due to a permissions issue in this repository's GitHub Actions workflow configuration. |
Sorry, something went wrong.
Resolve conflicts from upstream's fastmcp→mcpserver rename and McpError→MCPError migration, adapting protocol error handling to use the new MCPError(code=, message=) constructor.
|
I've once again resolved conflicts. It appears that the failing test on Windows is unrelated to these changes. @Kludex, do you think we could review this soon? |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
While developing my Swift SDK for MCP, I encountered the following MCP spec compliance issue in the Python SDK:
When an unknown tool is called, it throws a ToolError, which is caught by the server's exception handler and converted to a response with isError: true.
The spec distinguishes between protocol errors and tool execution errors:
Protocol Errors:
Tool Execution Errors:
Protocol error example for unknown tool:
{ "jsonrpc": "2.0", "id": 3, "error": { "code": -32602, "message": "Unknown tool: invalid_tool_name" } }The same issue applies to prompts and resources. Per the spec:
Changes
How Has This Been Tested?
I updated the tests to check for the correct error handling.
Breaking Changes
Users who expected the incorrect error handling may need to change their code.
Types of changes
Checklist