Adam Holt
Adam Holt
👋🏻 I've been looking into what's required to move https://github.com/github/github-mcp-server and our hosted offering of that off https://github.com/mark3labs/mcp-go and on to this SDK, and this is the main thing stopping...
@joshwlewis whoops, it was private, it's public now. Thanks! I took a look at your PR and implemented it [on a branch in that repo](https://github.com/omgitsads/mcp-go-session-example/tree/generic-session-store-interface) but it suffers from the...
👋🏻 Thanks for the update @findleyr. > How can a distributed server make server->client requests, using the streamable transport? Since the subsequent POST containing the response may be routed to...
https://github.com/modelcontextprotocol/go-sdk/compare/main...omgitsads:go-sdk:notify-tool-list-changed and a calling it with ```go svr := req.GetSession().(*mcp.ServerSession) err := svr.NotifyToolListChanged(ctx, &mcp.ToolListChangedParams{}) ``` seemed to be sufficient to get the notification on the tool stream when @mattdholloway and...
> Do any of the other SDKs make this possible? `mark3labs/mcp-go` supports sending these manually with `SendNotificationToClient` https://github.com/mark3labs/mcp-go/blob/670a95ab91bc3a4a08c3618a56f6555cd17c2307/server/session.go#L311, plus other similar methods, which is what we were using over there....
@findleyr So we're working within the constraints of what was proposed here: https://github.com/modelcontextprotocol/go-sdk/issues/148#issuecomment-3188725198. We run the server with `StreamableHTTPOptions.Stateless` as `false`, and manage our state using `Mcp-Session-Id` outside the scope...
Oops! My apologies, I meant to say that we are running with `Stateless: false` (I was thinking "Stateful" 🙃). I can do a small demo repo showing what we're doing...
🤦 sorry once again, been a busy week getting the Go SDK migration across the finish line. We are running with `Stateless: true`, i.e not having Go SDK manage any...