fix(jmanus): update MCP protocol #1980
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- 修复了StreamableHttpClientTransport中Integer到String的类型转换错误 - 添加了preprocessJsonForDeserialization方法来预处理JSON内容 - 自动将Integer类型的id字段转换为String类型 - 确保method和jsonrpc字段也是String类型 - 添加了全面的单元测试来验证修复 - 包含错误处理和回退机制 解决了错误: java.lang.ClassCastException: class java.lang.Integer cannot be cast to class java.lang.String
- 添加 MCP SDK 0.11.0 依赖 - 删除自定义的 StreamableHttpClientTransport 实现 - 优化 buildStreamingTransport 方法，使用 WebFluxSseClientTransport - 统一代码风格，与 buildSseTransport 保持一致 - 改进 URL 解析和日志记录 - 删除相关的测试文件
… 使用问题 - 移除未使用的导入 - 修正构造函数调用方式 - 使用正确的 Builder 模式创建 transport 实例
- 在 createWebClientBuilder 方法中配置 maxInMemorySize - 将默认缓冲区大小从 256KB 增加到 1MB - 解决大文件传输和流式数据处理时的缓冲区限制问题
shenwujie
changed the title
shenwujie
changed the title
alibaba-oss
approved these changes
Aug 4, 2025
github-actions bot
pushed a commit
to Hipple/spring-ai-alibaba
that referenced
this pull request
Aug 4, 2025
* fix: 修复MCP JSON-RPC消息处理中的ClassCastException错误 - 修复了StreamableHttpClientTransport中Integer到String的类型转换错误 - 添加了preprocessJsonForDeserialization方法来预处理JSON内容 - 自动将Integer类型的id字段转换为String类型 - 确保method和jsonrpc字段也是String类型 - 添加了全面的单元测试来验证修复 - 包含错误处理和回退机制 解决了错误: java.lang.ClassCastException: class java.lang.Integer cannot be cast to class java.lang.String * feat: 升级 MCP SDK 依赖并优化 buildStreamingTransport 方法 - 添加 MCP SDK 0.11.0 依赖 - 删除自定义的 StreamableHttpClientTransport 实现 - 优化 buildStreamingTransport 方法，使用 WebFluxSseClientTransport - 统一代码风格，与 buildSseTransport 保持一致 - 改进 URL 解析和日志记录 - 删除相关的测试文件 * fix: 修正 buildStreamingTransport 方法中的 WebClientStreamableHttpTransport 使用问题 - 移除未使用的导入 - 修正构造函数调用方式 - 使用正确的 Builder 模式创建 transport 实例 * feat: 添加 DataBuffer 大小配置以解决 DataBufferLimitException 异常 - 在 createWebClientBuilder 方法中配置 maxInMemorySize - 将默认缓冲区大小从 256KB 增加到 1MB - 解决大文件传输和流式数据处理时的缓冲区限制问题 * Update McpTransportBuilder and add .cursorindexingignore * Update static assets and McpTransportBuilder * Add .cursorindexingignore to .gitignore * Remove .cursorindexingignore from repository * Translate Chinese comments to English in McpTransportBuilder * Update McpTransportBuilder
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Describe what this PR does / why we need it
升级MCP协议到0.11.0.该版本支持了Streamable HTTP。
Jmanus Streamable HTTP切换到WebClientStreamableHttpTransport
maxInMemorySize传输内容上限到10M。
Does this pull request fix one issue?
Describe how you did it
Describe how to verify it
Special notes for reviews