Skip to content

fix: remove the recommended port button for uploading local images to MCP - #3601

Merged
Dallas98 merged 30 commits into
ModelEngine-Group:developfrom
menglinghan:feat/cross-tenant-repository
Aug 5, 2026
Merged

fix: remove the recommended port button for uploading local images to MCP#3601
Dallas98 merged 30 commits into
ModelEngine-Group:developfrom
menglinghan:feat/cross-tenant-repository

Conversation

@menglinghan

@menglinghan menglinghan commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

恢复“docker/k8s部署nexent时,通过容器化部署添加mcp使用统一端口”,删除本地镜像上传mcp时推荐端口按键,由用户自己输入;
本地镜像删除端口

修复agent配置中删除mcp不能在已选工具列表中马上刷新的问题;

mcp仓库user guide更新
文档
文档英文

多租户市场隔离;
MCP列表缓存与过滤一致性;
本地镜像上传补充;
修复outer-apis不显示的问题
修复英文页面存在中文问题;
修复保存后页面不会自动关闭的问题;
修复上传镜像后的错误提示;
# Conflicts:
#	backend/services/remote_mcp_service.py
#	frontend/app/[locale]/mcp-space/components/McpServiceDetailModal.tsx
#	frontend/app/[locale]/mcp-space/page.tsx
#	frontend/hooks/mcpTools/useMcpServiceDetail.ts
#	frontend/public/locales/en/common.json
#	frontend/public/locales/zh/common.json
#	test/backend/services/test_remote_mcp_service.py
…如 mcp-echarts 的 --port 3033),端口映射改为宿主→容器内部端口,修复此类镜像连不上问题。

 2. 工具扫描防误隐藏(tool_db.py + tool_configuration_service.py):update_tool_table_from_scan_tool_list 新增 enabled_mcp_names,启用中 MCP 的工具在瞬时扫描失败时保留可用状态,不再被误隐藏。
修复api添加mcp显示错误问题,增加前端必选字段要求;
…repository

# Conflicts:
#	sdk/nexent/container/docker_client.py
@menglinghan menglinghan closed this Aug 5, 2026
@menglinghan menglinghan reopened this Aug 5, 2026
合并 develop 后,MCP 端口相关文件被 develop(ModelEngine-Group#3581)的虚拟端口机制覆盖,
导致 readonlyHintKey 未定义却被引用、CI 类型检查失败。恢复本分支版本:
- 移除 mcp_ports_virtual/固定默认端口 5020/跳过端口冲突检查逻辑
- 移除 readonlyHintKey/portUnifiedHint,本地镜像上传无推荐端口按钮
- 保留 develop 在 locale 中的 chatInput 改动
@codecov

codecov Bot commented Aug 5, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@Dallas98
Dallas98 merged commit 031182c into ModelEngine-Group:develop Aug 5, 2026
17 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.

2 participants