Skip to content

fix: normalize paramsSchema in RegisteredTool.update - #2710

Open
VedantMadane wants to merge 8 commits into
modelcontextprotocol:mainfrom
VedantMadane:fix/issue-1960
Open

VedantMadane wants to merge 8 commits into
modelcontextprotocol:mainfrom
VedantMadane:fix/issue-1960

Conversation

@VedantMadane

Copy link
Copy Markdown

Summary

normalize paramsSchema in RegisteredTool.update

Changes

  • RegisteredTool.update routes paramsSchema through normalizeRawShapeSchema

Fixes #1960

@VedantMadane
VedantMadane requested a review from a team as a code owner August 24, 2026 20:31
@changeset-bot

changeset-bot Bot commented Aug 24, 2026 •

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: b6d6cb5

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

This PR includes changesets to release 6 packages
Name Type
@modelcontextprotocol/server Patch
@modelcontextprotocol/client Patch
@modelcontextprotocol/codemod Patch
@modelcontextprotocol/core Patch
@modelcontextprotocol/server-legacy Patch
@modelcontextprotocol/core-internal Patch

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

pkg-pr-new Bot commented Aug 24, 2026 •

Copy link
Copy Markdown

Open in StackBlitz

@modelcontextprotocol/client

npm i https://pkg.pr.new/@modelcontextprotocol/client@2710

@modelcontextprotocol/codemod

npm i https://pkg.pr.new/@modelcontextprotocol/codemod@2710

@modelcontextprotocol/core

npm i https://pkg.pr.new/@modelcontextprotocol/core@2710

@modelcontextprotocol/server

npm i https://pkg.pr.new/@modelcontextprotocol/server@2710

@modelcontextprotocol/server-legacy

npm i https://pkg.pr.new/@modelcontextprotocol/server-legacy@2710

@modelcontextprotocol/express

npm i https://pkg.pr.new/@modelcontextprotocol/express@2710

@modelcontextprotocol/fastify

npm i https://pkg.pr.new/@modelcontextprotocol/fastify@2710

@modelcontextprotocol/hono

npm i https://pkg.pr.new/@modelcontextprotocol/hono@2710

@modelcontextprotocol/node

npm i https://pkg.pr.new/@modelcontextprotocol/node@2710

commit: b6d6cb5

- RegisteredTool.update routes paramsSchema through normalizeRawShapeSchema

Fixes modelcontextprotocol#1960

Signed-off-by: Vedant Madane <6527493+VedantMadane@users.noreply.github.com>
@claude claude Bot added the v2 Ideas, requests and plans for v2 of the SDK which will incorporate major changes and fixes label Sep 23, 2026
@VedantMadane

Copy link
Copy Markdown
Author

Gentle bump: rebased/updated onto latest base (816f1cb). Ready for review when you have a moment. Thanks!

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

v2 Ideas, requests and plans for v2 of the SDK which will incorporate major changes and fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

RegisteredTool.update() crashes with ZodObject inputSchema (passthrough schemas)

1 participant