Skip to content

Switch to modelcontextprotocol/go-sdk#77

Merged
slimslenderslacks merged 13 commits intomainfrom
slim/sdk-refactor
Aug 5, 2025
Merged

Switch to modelcontextprotocol/go-sdk#77
slimslenderslacks merged 13 commits intomainfrom
slim/sdk-refactor

Conversation

@slimslenderslacks
Copy link
Copy Markdown
Collaborator

@slimslenderslacks slimslenderslacks commented Jul 29, 2025

What I did

Refactor gateway to use the official GO SDK

Related issue

Changes

  • switches custom mcp client for official client SDK (major change)
    • this is a major change because our custom client was fairly minimal and tended to not be less strict with tool call payloads
  • server sdk is switched from mark3labs to the official SDK (minor chanage)

@slimslenderslacks slimslenderslacks marked this pull request as ready for review July 31, 2025 18:51
@slimslenderslacks slimslenderslacks requested a review from a team as a code owner July 31, 2025 18:51
* the Client interface is a duplicate of ClientSession
Copy link
Copy Markdown
Contributor

@cmrigney cmrigney left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work! There could be one blocking issue, see my comments. Also, I am still testing the changes out. But code lgtm in general 👍

* add context aware stdio transport (ctrl-c works again)
* add ServersSessions to subscribe/unsubscribe handlers
@slimslenderslacks slimslenderslacks merged commit a57c84e into main Aug 5, 2025
8 checks passed
@slimslenderslacks slimslenderslacks deleted the slim/sdk-refactor branch August 26, 2025 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

MCP server roots/list requests are not being delivered to the client.

2 participants