Add back EverythingServer for stdio#950
Add back EverythingServer for stdio#950asklar wants to merge 4 commits intomodelcontextprotocol:mainfrom
Conversation
Co-authored-by: asklar <22989529+asklar@users.noreply.github.com>
Co-authored-by: asklar <22989529+asklar@users.noreply.github.com>
| <Project Path="samples/EverythingServer/EverythingServer.csproj" /> | ||
| <Project Path="samples/EverythingServer.Core/EverythingServer.Core.csproj" /> | ||
| <Project Path="samples/EverythingServer.Http/EverythingServer.Http.csproj" /> | ||
| <Project Path="samples/EverythingServer.Stdio/EverythingServer.Stdio.csproj" /> |
There was a problem hiding this comment.
@mikekistler, I think you were interested in scenarios where a single server could be used either via stdio or http? IIRC, should we do that in this one rather than having different projects?
There was a problem hiding this comment.
I think both patterns are valid and it's useful to illustrate how the core MCP logic can be implemented in a class library and used in either a STDIO or sHTTP server project.
There was a problem hiding this comment.
In that case, can we make the STDIO server pack itself as a tool that can be run with dnx (including it's library dependency)?
There was a problem hiding this comment.
I would like to use this server as a sample MCPB-based server. Totally up to y'all if you also want to make it available via dnx.
There was a problem hiding this comment.
Would that be built here? Demonstrating deployment options seems like a good thing to add to the samples, but I'll defer to @mikekistler as I'm new here.
There was a problem hiding this comment.
Perhaps we should start by adding documentation on what an MCPB-based server is and why you might want one. After that we can talk about adding a sample or updating one to demonstrate that.
Adds back EverythingServer option for stdio as it is still useful.
Motivation and Context
Fixes #889
How Has This Been Tested?
locally tested both in inspector
Breaking Changes
no
Types of changes
Checklist
Additional context