fix(mcp): expose streamable HTTP resumable transport options#2006
fix(mcp): expose streamable HTTP resumable transport options#2006guslegend0510 wants to merge 2 commits into
Conversation
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
|
Do you think it reasonable if we disable SSE by default when streamable is enabled? Currently, user has to set Also, have you checked the latest MCP sdk release? I mean will it bring any behaviour changes if we upgrade to the latest version? |
|
This PR has conflicts with the git fetch origin
git checkout fix/2003-streamable-http-resumable-streams
git rebase origin/main
# resolve conflicts, then:
git push --force-with-leaseThis is a one-time reminder. Feel free to @mention me for a re-review after conflicts are resolved. Automated notification by github-manager-bot |
AgentScopeJavaBot
left a comment
There was a problem hiding this comment.
🤖 AI Review
Exposes streamable HTTP resumable transport options for MCP clients. Adds configuration surface for MCP streamable HTTP transport including resumability settings.
Background
Fixes #2003.
When connecting to a Streamable HTTP MCP server that disables SSE and only accepts POST requests, tool calls can hang because AgentScope-Java does not expose the underlying MCP SDK transport options needed to disable resumable SSE-style startup behavior.
What Changed
resumableStreams(boolean)onMcpClientBuilderfor Streamable HTTP transportopenConnectionOnStartup(boolean)onMcpClientBuilderfor Streamable HTTP transportHttpClientStreamableHttpTransport.BuilderresumableStreamsandopenConnectionOnStartupsupport totools.jsonMCP server configMcpServerRegistrarExample
Java API:
Testing
mvn -pl "agentscope-core,agentscope-harness" spotless:apply
mvn -pl "agentscope-core,agentscope-harness" "-Dtest=McpClientBuilderTest,ToolsConfigLoaderTest" test