feat(client): adds StreamableHttpClientTransport#144
feat(client): adds StreamableHttpClientTransport#144Aliaksie wants to merge 3 commits intomodelcontextprotocol:mainfrom
Conversation
mcp/src/main/java/io/modelcontextprotocol/client/transport/StreamableHttpClientTransport.java
Outdated
Show resolved
Hide resolved
bb851b3 to
8b593e0
Compare
mcp/src/main/java/io/modelcontextprotocol/client/transport/StreamableHttpClientTransport.java
Outdated
Show resolved
Hide resolved
|
Great job @Aliaksie Here are some of my thoughts after reading this PR and the SDK source code. I think just adding the McpClientTransport implementation cannot give full support for Streamable HTTP because how it works is inconsistent with HTTP+SSE in many places, which might require some redesign or refactoring of the current SDK implementation:
|
f07a466 to
67b2881
Compare
Hey @chickenlj Thanks a lot for the thoughtful feedback! 1 I added to Streamable HTTP transport code to manage I hope all this makes the SDK compliant with Streamable HTTP's flexibility while still maintaining support for HTTP+SSE fallback |
a677cb1 to
c1f0a93
Compare
|
Hey @tzolov, hope you're doing well! Just wanted to check in — have you had a chance to review this PR? I’d appreciate any feedback when you get a moment. Looking forward to your thoughts! |
c1f0a93 to
393233f
Compare
|
Hi team, could you please share the timeline to have streamable http for mcp java sdk ? |
393233f to
fdc11db
Compare
|
when this feature could be ready to be parity with typescript and python? |
fdc11db to
028ad6d
Compare
|
Hi @tzolov👋 Just checking in to see if there’s been a chance to review this PR Would love to hear if this makes sense for the SDK or if I should revise or break it down further. |
adds StreamableHttpClientTransport
Motivation and Context
it is enabler for support 2025-03-26 version of protocol
How Has This Been Tested?
Breaking Changes
No
Types of changes
Checklist
Additional context
The server implementation and the 2025-03-26 scheme will be added as separate PRs.