Skip to content

Feat:add mcp serve support and client demo#127

Merged
Ki-Seki merged 6 commits intoMemTensor:devfrom
fridayL:feat/mcp-dev
Jul 20, 2025
Merged

Feat:add mcp serve support and client demo#127
Ki-Seki merged 6 commits intoMemTensor:devfrom
fridayL:feat/mcp-dev

Conversation

@fridayL
Copy link
Collaborator

@fridayL fridayL commented Jul 18, 2025

Description

Summary: add mcp serve for http serve

Fix: #18

Reviewer: @Ki-Seki

Checklist:

  • I have performed a self-review of my own code | 我已自行检查了自己的代码
  • I have commented my code in hard-to-understand areas | 我已在难以理解的地方对代码进行了注释
  • I have added tests that prove my fix is effective or that my feature works | 我已添加测试以证明我的修复有效或功能正常
  • I have added necessary documentation (if applicable) | 我已添加必要的文档(如果适用)
  • I have linked the issue to this PR (if applicable) | 我已将 issue 链接到此 PR(如果适用)
  • I have mentioned the person who will review this PR | 我已提及将审查此 PR 的人

@fridayL fridayL requested review from J1awei-Yang and Ki-Seki July 18, 2025 12:31
Copy link
Member

@Ki-Seki Ki-Seki left a comment

Choose a reason for hiding this comment

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

You need to deal with conflicts, Mainly re-add dependencies (fastmcp) in pyproject toml

Copy link
Member

@Ki-Seki Ki-Seki left a comment

Choose a reason for hiding this comment

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

fastmcp包本身比较小,且基本算作核心功能,建议将fastmcp放到主依赖里。

@fridayL fridayL requested a review from Ki-Seki July 19, 2025 13:19
Copy link
Member

@Ki-Seki Ki-Seki left a comment

Choose a reason for hiding this comment

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

src/memos/api/mcp_serve_stdio.py seems too similar to src/memos/api/mcp_serve.py, should some codes be reused/reconstructed?

@fridayL
Copy link
Collaborator Author

fridayL commented Jul 20, 2025

reconstructed

yes , mcp_serve_stdio.py support more mcp protocol, I have rewrite mcp_serve_stdio-->mcp_serve and rm mcp_serve_stdio.py

@Ki-Seki
Copy link
Member

Ki-Seki commented Jul 20, 2025

looks good to me

@Ki-Seki Ki-Seki merged commit 56d6fd5 into MemTensor:dev Jul 20, 2025
20 checks passed
tianxing02 pushed a commit to tianxing02/MemOS that referenced this pull request Feb 24, 2026
* feat:add mcp serve

* feat: add mcp serve and http client api

* add: stdio for memos mcp

* fix:remove dup content and reconstructed code

* fix : reorg core dire and add mcp examples
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.

2 participants