Skip to content

fix(plugins): validate and surface MCP runtime lifecycle - #457

Open
GTC2333 wants to merge 4 commits into
mainfrom
codex/stage2-07-mcp-lifecycle
Open

fix(plugins): validate and surface MCP runtime lifecycle#457
GTC2333 wants to merge 4 commits into
mainfrom
codex/stage2-07-mcp-lifecycle

Conversation

@GTC2333

@GTC2333 GTC2333 commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator

Summary

Stage 2 PR 7/9.

  • validate MCP-backed plugin activation and report startup failures
  • track observed MCP server states in the runtime
  • expose MCP status through the runtime/backend APIs
  • show restart requirements and ready/degraded/failed states in Marketplace
  • refresh composer MCP mentions after a runtime restart
  • harden the bundled Playwright MCP launcher

Workspace-file-link scoping from the original development sequence is intentionally excluded; it belongs to PR 9.

Verification

  • root typecheck
  • 165 targeted runtime/backend tests
  • 61 targeted Web tests
  • Web production build
  • diff whitespace validation

Merge notes

Targets main directly and does not depend on PR 5 or PR 9.

@Fxde42 Fxde42 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Request changes:static runtime 模式下,StaticRuntimeOrchestrator.stopRuntime 是 no-op,但 POST /runtime/restart 仍返回成功,Marketplace 随后关闭提示并把 mentions 当作已刷新,实际 runtime 和 MCP tools 都没有重启。建议 static 模式返回明确的 409 和手动重启容器指引,并保留待重启提示;不要向 UI 报告假成功。请补 StaticRuntimeOrchestrator 路径的 API 测试。

@GTC2333

GTC2333 commented Aug 14, 2026

Copy link
Copy Markdown
Collaborator Author

Addressed the static-runtime restart issue in c6803b3. The Orchestrator interface now exposes externally managed lifecycle ownership; /api/runtime/restart returns 409 with manual container/process restart guidance before stopping, clearing clients, or reporting success. The existing Marketplace error path retains the restart prompt and does not emit the restarted event. Added a real StaticRuntimeOrchestrator endpoint regression test. Validation: root typecheck, 117 backend/runtime tests, 62 Web tests, and Web production build.

@GTC2333
GTC2333 requested a review from Fxde42 August 14, 2026 10:35
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.

3 participants