-
Notifications
You must be signed in to change notification settings - Fork 148
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Everything server #151
Everything server #151
Conversation
This will mean we can return multiple AIContent objects from a tool, such as a mixed text/image set Contributes to modelcontextprotocol#68
- Introducing separate classes for Text and Blob ResourceContents - Custom JSON converter to allow those types to be returned, making it easier to follow the type system against the spec - Updated tests
Co-authored-by: Stephen Toub <[email protected]> Co-authored-by: Peder Holdgaard Pedersen <[email protected]>
Co-authored-by: Stephen Toub <[email protected]>
…csharp-sdk into improving-resources
Is the intent to have this kept in sync with the reference It looks (without having dived fully into it yet) to be equivalent right now, but I think it might be worth considering adding things not in the |
Ah sweet, I must've missed that. I'll get it included.
My thought was to keep it in sync, as best as possible since a) it provides a good way to dogfood all the code paths and b) it's useful to have a direct comparison. |
@PederHP - Added logging in a similar design to TestServer. Also added a new method to the |
tests/ModelContextProtocol.Tests/Server/McpServerResourceTests.cs
Outdated
Show resolved
Hide resolved
Co-authored-by: Stephen Halter <[email protected]>
Thank you! |
This is an implementation of https://www.npmjs.com/package/@modelcontextprotocol/server-everything
The objective of this sample is to have all the usage covered using the .NET SDK and ensure that we can implement as much complexity as possible.
Improvements can be done with the completion of #70 and #72.
The only thing I think that is missing from here is logging - I can't find a way to set that.Logging implemented. I added aWithSetLoggingLevelHandler
method to theMcpServerBuilder
and fixed theMcpServerHandlers
that wasn't exposing logging capabilities.Commit history is a bit of a mess as I kept merging with branches as I was updating gaps.