Skip to content

fix(server): forward list-changed notification options#1777

Draft
theamodhshetty wants to merge 1 commit intomodelcontextprotocol:mainfrom
theamodhshetty:codex/fix-inline-list-changed-notifications
Draft

fix(server): forward list-changed notification options#1777
theamodhshetty wants to merge 1 commit intomodelcontextprotocol:mainfrom
theamodhshetty:codex/fix-inline-list-changed-notifications

Conversation

@theamodhshetty
Copy link
Copy Markdown

Summary

  • forward NotificationOptions through the send*ListChanged() helpers on Server and McpServer
  • add a streamable HTTP regression proving sendToolListChanged({ relatedRequestId }) is delivered inline on the active POST SSE response
  • add a patch changeset for @modelcontextprotocol/server

Why

sendToolListChanged(), sendResourceListChanged(), and sendPromptListChanged() already route through Protocol.notification(), but their public helpers dropped NotificationOptions before they reached the transport. On streamable HTTP servers without a standalone GET SSE channel, that meant list-changed notifications were silently dropped instead of being attached to the active POST SSE response via relatedRequestId.

Validation

  • pnpm --filter @modelcontextprotocol/server test -- streamableHttp.test.ts
  • pnpm --filter @modelcontextprotocol/server typecheck
  • pnpm --filter @modelcontextprotocol/server lint

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Mar 27, 2026

🦋 Changeset detected

Latest commit: 8256623

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

This PR includes changesets to release 4 packages
Name Type
@modelcontextprotocol/server Patch
@modelcontextprotocol/express Patch
@modelcontextprotocol/hono Patch
@modelcontextprotocol/node 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
Copy link
Copy Markdown

pkg-pr-new bot commented Mar 27, 2026

Open in StackBlitz

@modelcontextprotocol/client

npm i https://pkg.pr.new/modelcontextprotocol/typescript-sdk/@modelcontextprotocol/client@1777

@modelcontextprotocol/server

npm i https://pkg.pr.new/modelcontextprotocol/typescript-sdk/@modelcontextprotocol/server@1777

@modelcontextprotocol/express

npm i https://pkg.pr.new/modelcontextprotocol/typescript-sdk/@modelcontextprotocol/express@1777

@modelcontextprotocol/hono

npm i https://pkg.pr.new/modelcontextprotocol/typescript-sdk/@modelcontextprotocol/hono@1777

@modelcontextprotocol/node

npm i https://pkg.pr.new/modelcontextprotocol/typescript-sdk/@modelcontextprotocol/node@1777

commit: 8256623

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