Skip to content

fix mcp streamable#6289

Merged
yuluo-yx merged 3 commits intomasterfrom
fix/fix_mcp_streamable
Feb 7, 2026
Merged

fix mcp streamable#6289
yuluo-yx merged 3 commits intomasterfrom
fix/fix_mcp_streamable

Conversation

@Aias00
Copy link
Contributor

@Aias00 Aias00 commented Feb 5, 2026

Make sure that:

  • You have read the contribution guidelines.
  • You submit test cases (unit or integration tests) that back your changes.
  • Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

Copilot AI review requested due to automatic review settings February 5, 2026 11:26
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes MCP (Model Context Protocol) streamable functionality by upgrading the MCP SDK from version 0.10.0 to 0.17.0 and adapting the codebase to API changes introduced in the new version. The changes replace direct usage of Jackson's ObjectMapper with the MCP SDK's McpJsonMapper abstraction and update CORS headers to support the Mcp-Protocol-Version header.

Changes:

  • Upgraded MCP SDK from 0.10.0 to 0.17.0 and Spring AI from 1.0.0 to 1.1.2
  • Replaced Jackson TypeReference with MCP SDK's TypeRef throughout transport implementations
  • Added support for multiple MCP protocol versions (2025-03-26 and 2025-11-25)
  • Enhanced CORS headers to include Mcp-Protocol-Version for proper protocol negotiation
  • Improved streamable HTTP transport initialization with lazy creation fallback

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
pom.xml Updated MCP SDK version to 0.17.0 and Spring AI to 1.1.2
shenyu-plugin/shenyu-plugin-mcp-server/pom.xml Added mcp-json-jackson2 dependency for JSON serialization support
shenyu-spring-boot-starter/.../pom.xml Added mcp-json-jackson2 dependency to starter module
ShenyuStreamableHttpServerTransportProvider.java Migrated to McpJsonMapper, added multi-version protocol support, updated CORS headers
ShenyuSseServerTransportProvider.java Migrated to McpJsonMapper for consistent JSON handling
McpServerPluginDataHandler.java Added streamable HTTP transport initialization in selector handler
McpServerPlugin.java Added lazy transport creation fallback and updated CORS headers

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@yuluo-yx yuluo-yx merged commit c224816 into master Feb 7, 2026
42 checks passed
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.

3 participants