Skip to content

feat: MCP 仅保留 Streamable HTTP,移除 SSE#83

Merged
weibaohui merged 1 commit into
mainfrom
feature/mcp-only-streamable
Mar 23, 2026
Merged

feat: MCP 仅保留 Streamable HTTP,移除 SSE#83
weibaohui merged 1 commit into
mainfrom
feature/mcp-only-streamable

Conversation

@weibaohui
Copy link
Copy Markdown
Owner

Summary

  • 后端移除 SSE 端点(/mcp/sse, /mcp/message),仅保留 Streamable HTTP
  • 前端移除 SSE 配置选项,仅显示 Streamable HTTP 配置
  • 更新协议版本显示为 MCP 2025-11-25

Changes

  • backend/cmd/server/main.go: 移除 SSE Server 创建和路由注册
  • frontend/src/pages/MCPPage.tsx: 简化配置页面,仅保留 Streamable HTTP

Test plan

  • 启动后端,确认只有 /mcp/streamable 端点
  • 访问 MCP 页面,确认只显示 Streamable HTTP 配置

- 后端移除 SSE 端点(/mcp/sse, /mcp/message)
- 前端移除 SSE 配置选项,仅保留 Streamable HTTP
- 更新协议版本显示为 MCP 2025-11-25
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Mar 23, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 77368ef1-564d-442e-92a2-b77a5fa8cbb1

📥 Commits

Reviewing files that changed from the base of the PR and between 7acc285 and 2ea1bd2.

📒 Files selected for processing (2)
  • backend/cmd/server/main.go
  • frontend/src/pages/MCPPage.tsx

📝 Walkthrough

Summary by CodeRabbit

  • 功能变更

    • 停用 SSE 传输方式,统一使用 Streamable HTTP 端点连接
    • 更新 MCP 协议版本至 2025-11-25
  • 文档更新

    • 更新 Cursor 和 Windsurf 客户端配置说明,指向 /mcp/streamable 端点
  • 界面优化

    • 简化配置指南页面,移除 SSE 相关配置显示

Walkthrough

此PR从后端移除SSE服务器创建和路由注册,删除通过SSE暴露的MCP端点,仅保留Streamable HTTP传输方式。前端对应移除SSE相关配置模板和UI元素,更新MCP协议版本号。

Changes

Cohort / File(s) Summary
后端MCP传输移除
backend/cmd/server/main.go
移除SSE服务器初始化和路由注册,删除GET /mcp/ssePOST /mcp/message端点,精简端点日志输出。
前端SSE配置移除及UI更新
frontend/src/pages/MCPPage.tsx
删除SSE配置模板和相关JSON配置,移除SSE状态显示标签和"SSE配置文件"卡片,将MCP协议版本从2024-11-05更新至2025-11-25,更新Cursor和Windsurf客户端配置说明指向.../mcp/streamable端点。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

  • PR #6: 与本PR相关,两者均直接修改backend/cmd/server/main.go中的MCP SSE服务器创建和路由注册逻辑。
  • PR #75: 本PR移除了该PR中添加的MCP SSE服务器和SSE特定的前端UI,两者涉及相同的服务器路由和MCPPage组件修改。

Poem

🐰 SSE的故事已落幕,
Streamable独占鳌头;
前后端齐心,
删除繁复,轻装上阵,
MCP之路更清晰! 🚀

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed 标题清晰准确地总结了主要变更:移除SSE,仅保留Streamable HTTP传输方式。
Description check ✅ Passed 描述详细阐述了后端和前端的具体变更,以及协议版本的更新,与代码改动完全相关。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/mcp-only-streamable

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@weibaohui weibaohui merged commit d10281d into main Mar 23, 2026
2 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.

1 participant