Skip to content

fix(server): return Tool Execution Errors for input validation failures (SEP-1303)#1966

Open
MukundaKatta wants to merge 1 commit intomodelcontextprotocol:v1.xfrom
MukundaKatta:fix/sep-1303-input-validation-tool-error
Open

fix(server): return Tool Execution Errors for input validation failures (SEP-1303)#1966
MukundaKatta wants to merge 1 commit intomodelcontextprotocol:v1.xfrom
MukundaKatta:fix/sep-1303-input-validation-tool-error

Conversation

@MukundaKatta
Copy link
Copy Markdown

…es (SEP-1303)

Per SEP-1303 (spec 2025-11-25), tool input validation failures should be
surfaced as Tool Execution Errors (a `CallToolResult` with `isError: true`)
rather than as JSON-RPC `InvalidParams` protocol errors. Returning a tool
error lets the model see the validation message and self-correct on retry,
which is the motivation behind SEP-1303.

`McpServer.validateToolInput` no longer throws an `McpError` on validation
failure; it returns either `{ data }` on success or `{ errorResult }` with
the Tool Execution Error result. Both call sites (the regular tools/call
path and the automatic task polling path) handle the error result by
returning it directly to the client.

Output validation behavior is unchanged; SEP-1303 targets input validation.

Closes modelcontextprotocol#1956.
@MukundaKatta MukundaKatta requested a review from a team as a code owner April 26, 2026 22:08
@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Apr 26, 2026

🦋 Changeset detected

Latest commit: 6b528e5

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Apr 26, 2026

Open in StackBlitz

npm i https://pkg.pr.new/@modelcontextprotocol/sdk@1966

commit: 6b528e5

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