feat: Add support for specifying NotificationOptions #1486
Open
daamitt wants to merge 4 commits intomodelcontextprotocol:mainfrom
Open
feat: Add support for specifying NotificationOptions #1486daamitt wants to merge 4 commits intomodelcontextprotocol:mainfrom
daamitt wants to merge 4 commits intomodelcontextprotocol:mainfrom
Conversation
…owlevel Server creation
9 tasks
Contributor
Author
Contributor
Author
|
Hello @maxisbey , please let me know how to proceed with this feature PR. Thanks |
Contributor
Author
|
@felixweinberger @maxisbey would really appreciate your thoughts on this. As of today (afaik) there is no clean way for FastMCP based servers to specify MCP capabilities like prompt / tools / resources listChanged notifications. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add support for specifying NotificationOptions in FastMCP and lowlevel Server creation
Motivation and Context
Currently there is no straightforward way to specify FastMCP server notification capabilities. Some clients do not respect notifications if server does not disclose support in the initliazation phase. eg: "notifications/resources/list_changed"
How Has This Been Tested?
uv run pytest:
Results (18.39s):
653 passed
2 skipped
1 xfailed
Breaking Changes
No
Types of changes
Checklist
Additional context
Have updated notification snippet and snippet readme