Repository navigation
Conversation
Note: Test failure is a pre-existing issue in
|
af995d1 to
074d330
Compare
LuuOW
left a comment
There was a problem hiding this comment.
Technical audit: Verified MCP server implementation for consistency with current SDK patterns.
|
Thank you for this pull request, and apologies for the long wait for a response. Since it was opened, the reference servers have been rebuilt for the 2026-07-28 MCP specification (#4857) and released as v2 (2.0.0 on npm for the TypeScript servers, 2026.10.19 on PyPI for the Python servers). The code this PR changes has been replaced, and the behavior it addresses has either been fixed in v2 or no longer exists in the same form. This is not a judgment on the change or on you; it's a consequence of the code it targets having been replaced. If the problem still happens on v2, please open a fresh issue with the version you're running and steps to reproduce. Note that this repository now accepts contributions as issues rather than pull requests from anyone but its maintainers; see This is part of the backlog triage in #4875. |
Fix: Fix move_file outputSchema to return array
Problem
The move_file tool's outputSchema declared
{ content: array }but the handler returnedstructuredContent: { content: string }, causing MCP protocol -32602 validation error.Solution
Update the handler to return
structuredContent: { content: [contentBlock] }(array) matching the outputSchema.Changes
Closes #3093