Skip to content

fix(mcp): require managed runtime for desktop launch#3510

Merged
t8y2 merged 1 commit into
t8y2:mainfrom
CN-Scars:fix/mcp-require-managed-runtime
Jul 15, 2026
Merged

fix(mcp): require managed runtime for desktop launch#3510
t8y2 merged 1 commit into
t8y2:mainfrom
CN-Scars:fix/mcp-require-managed-runtime

Conversation

@CN-Scars

Copy link
Copy Markdown
Contributor

变更说明

补全 DBX Desktop 的 MCP 受管运行时约束:

  • Codex CLI 启动前必须成功解析兼容的 Node.js 与 MCP package entry 组合。
  • 无法解析时直接返回包含版本要求和恢复方式的错误,不再将 None 传入核心层并重新触发 dbx-mcp-server PATH shim 默认值。
  • MCP 前置检查完成后再注册 AI stream,避免前置校验失败时留下无效任务。
  • 保留 dbx-web 对核心层默认命令的现有行为。

变更类型

  • 新功能
  • Bug 修复
  • 性能优化
  • 代码重构
  • 文档更新
  • CI / 构建

涉及前端

  • 本 PR 涉及前端改动,已附截图/录屏(见下方)

本 PR 不涉及前端 UI 改动。

验证

  • make check 通过
  • make cargo-check-fast 通过
  • 相关测试通过

已执行:

  • cargo fmt --all --check
  • cargo test -p dbx commands::mcp::tests --no-default-features(14 passed)
  • make check
  • make cargo-check-fast
  • git diff --check

关联 Issue

Follow-up to #3455

@CN-Scars
CN-Scars marked this pull request as ready for review July 15, 2026 04:18
@t8y2
t8y2 merged commit b9ac69f into t8y2:main Jul 15, 2026
8 checks passed
@t8y2

t8y2 commented Jul 15, 2026

Copy link
Copy Markdown
Owner

Thanks for the contribution! Merged in b9ac69f, will be released in the next version.

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