Skip to content

mcp: answer 2026-07-28 requests with the spec's status codes (DEX-84) - #39698

Draft
bobbyiliev wants to merge 2 commits into
MaterializeInc:mainfrom
bobbyiliev:mcp-status-codes
Draft

bobbyiliev wants to merge 2 commits into
MaterializeInc:mainfrom
bobbyiliev:mcp-status-codes

Conversation

@bobbyiliev

Copy link
Copy Markdown
Contributor

On 2026-07-28 requests, an unknown method (and ping, which that version removes) now gets HTTP 404 with -32601. A body that parses as JSON but not as a known request (an unknown tool, bad arguments) is still checked against the mirrored headers and params._meta first, then gets -32602 in a 200 response. A notification that fails to parse gets 400, and a body that is not a JSON object gets -32600 with 400. 2025-11-25 requests and the flag-off path keep their current responses, including the 422s. Real clients probing server/discover still fall back to initialize on the 404. Stacked on #39695, so its commits show here until it lands. Closes DEX-84 and DEX-85.

Tests: unit tests for parse failures and the raw-name header check, and new cases in both MCP datadriven files.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant