-
Notifications
You must be signed in to change notification settings - Fork 536
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
Support for "Streamable HTTP" Transport #220
Comments
be aware of modelcontextprotocol/modelcontextprotocol#205 |
Is this as simple as creating a new |
I added support for this to mcp-framework, I assumed this was being worked on already but I'm happy to add this to the sdk. Would that be helpful? |
@QuantGeekDev there is a promising PR for it already. Any review comments you could add there would be great, since you've been through the process. |
@cliffhall the promising PR #230 is just server-side right, additional work would be needed to support clients? |
@ddebrunner Yes, that PR just adds the @jspahrsummers Have you had a look at that PR? Can you weigh in on any missing pieces? |
@cliffhall I'm new to this code, but I tried
|
Yep, you are right. You need both, but I'm not sure that they have to both come in the same PR if the server transport is unit tested and seems viable, another PR could be opened for the client transport. |
Hi, @ddebrunner I'd like to add an Client Transport after |
Should I merge the Client Transport into #230, or would it be better to open a new PR for this? 🤔 @cliffhall |
Hi , @gylove1994 , I think if merge Client Transport into #230 together , it's better than open a new PR.
|
A new PR is better. It's two developers, so if further changes are required, its messy, who will fix them. They'll both end up in the same release, but they can be decoupled for the purposes of contribution. |
My point is that if no one is available to take over the "Client Transport" development, I'd be happy to complete it. However, if I were to open a new PR for this, it might complicate integration testing prior to merging, compared to maintaining it within the current workflow. If you think a new PR is better, I'll fully respect your decision. |
That's a fair take. Are you planning to add the integration tests in the same PR? |
Is your feature request related to a problem? Please describe.
As a developer using the Typescript SDK for the Model Context Protocol (MCP), I need it to support the new "Streamable HTTP" transport introduced in PR modelcontextprotocol/modelcontextprotocol#206 of the specification repository. Without this update, the SDK will be incompatible with servers adopting the new transport, causing integration issues and preventing me from leveraging the specification’s latest improvements.
Describe the solution you'd like
I’d like the Typescript SDK to be updated to implement the "Streamable HTTP" transport as outlined in PR
This includes:
Describe alternatives you've considered
One alternative is manually patching the SDK to support the new transport, but this isn’t a sustainable solution. Another option is waiting for the maintainers to address this independently, but filing this issue ensures the change is tracked and prioritized proactively.
Additional context
The details of the "Streamable HTTP" transport are in PR of the MCP specification repository: modelcontextprotocol/modelcontextprotocol#206. Updating the SDK is essential to keep it aligned with the latest MCP standard and to ensure seamless integration with updated servers.
The text was updated successfully, but these errors were encountered: