fix(server): support extension subscription notifications - #2593
fix(server): support extension subscription notifications#2593SnowSky1 wants to merge 2 commits into
Conversation
Co-Authored-By: Claude <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
🦋 Changeset detectedLatest commit: 7b63522 The changes in this PR will be included in the next version bump. This PR includes changesets to release 6 packages
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 |
@modelcontextprotocol/client
@modelcontextprotocol/codemod
@modelcontextprotocol/core
@modelcontextprotocol/server
@modelcontextprotocol/server-legacy
@modelcontextprotocol/express
@modelcontextprotocol/fastify
@modelcontextprotocol/hono
@modelcontextprotocol/node
commit: |
|
Closing for the time being: repository permissions prevent me from marking this draft ready for review (the markPullRequestReadyForReview mutation returns FORBIDDEN for my account), and drafts cannot be merged, so I do not want it sitting idle in the review queue. The diff was complete and CI was green at close time; the branch fix/extension-subscription-notifications is kept on my fork. Happy to reopen or resubmit if maintainers want this change or can restore draft-readiness for external contributors. |
Summary
subscriptions/listenfilters and acknowledgementsServerNotifier.extension()event path filtered by the requested extension keyio.modelcontextprotocol/tasks/notifications/tasksthrough schema, event-bus, andcreateMcpHandlertestsFixes #2569.
Validation
Stack note
This draft currently includes #2592 as its first commit because that baseline fix is required for the repository pre-push lint gate. I will rebase this PR onto
mainafter #2592 lands so this diff contains only the extension-subscription commit.