feat(status): surface codex MCP registration + close #36 Phase 4#40
Merged
Conversation
Add a `codex-mcp` row to `ak status` (and its PREF slot in the dashboard) showing whether the project-scoped codex MCP backend (mcp__codex__codex) is registered — the dual-host inline Claude→Codex path (ADR-0001 projection #3, PR #39). Surfaces only when the codex host is enabled. Detection is a pure, spawn-free helper `codexMcpStatus(cfg, cwd)` in mcp.mjs: it reads the project `.mcp.json` (the file `claude mcp add -s project` writes — the deterministic equivalent of `claude mcp get codex`) plus the kit.json `providers.codexMcp === 'ak'` ownership marker, so a pre-existing user server is reported but flagged not-ak-managed. Mirrors registrationStatus()'s file-read approach; unit-tested without spawning claude. Also thread the ruvnet/ruflo#2766 tracking reference into dual.mjs's CLAUDE_FLOW_DB_PATH workaround comments (the shared-memory bootstrap bug the pin neutralizes) so the removal condition is unambiguous once upstream ships. Refs #36, #39, ruvnet/ruflo#2766
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Follow-up polish to #39 (dual-host per-activity LLM routing, merged
fd74187), plus the resolution of #36 Phase 4.Task 1 — #36 Phase 4 decision (verified, no code)
Phase 4 proposed a persisted
kit.jsonrole-split preference that scaffoldspackage.jsondual:*scripts. Both halves are obviated by #39 — closing #36 as complete (decision posted as a comment there):providers.dualRouting(src/lib/config.mjs:25, resolved insrc/lib/routing.mjs). It's a per-activity{host, model, escalate?, source}policy — a superset of a per-template role split, with provenance so user edits survive re-seeding.package.json dual:* scaffolding→ superseded byak dual run <template> "<task>"(src/commands/dual.mjs), which reads that same policy and shells toclaude-flow-codex dual run. Strictly better than mutating the user'spackage.json: no file writes, works in any repo (not just Node), and adds--route/--escalate/--dry-run.Building the scaffolding would write opinionated scripts into the user's
package.jsonfor ~no gain, so it was not built (per the issue's own "opt-in only, explicit sign-off" framing).Task 2 — polish (this PR's code)
(a) codex MCP status visibility. New
codex-mcprow inak status(+ PREF slot indashboard-server.mjs) showing whether the project-scoped codex MCP backend (mcp__codex__codex, ADR-0001 projection #3) is registered. Surfaces only when the codex host is enabled.Detection is a pure, spawn-free helper
codexMcpStatus(cfg, cwd)inmcp.mjs: reads the project.mcp.json(whatclaude mcp add -s projectwrites — the deterministic equivalent ofclaude mcp get codex) plus the kit.jsonproviders.codexMcp === 'ak'ownership marker. A pre-existing user-registered server is reported but flagged not-ak-managed. MirrorsregistrationStatus()'s file-read approach; 6 new unit tests, noclaudespawn.(b) ruvnet/ruflo#2766 tracking. Threaded the issue reference into
dual.mjs'sCLAUDE_FLOW_DB_PATHworkaround comments (the@claude-flow/codexshared-memory bootstrap bug the pin neutralizes), so the removal condition is unambiguous. The workaround stays until #2766 ships; a live end-to-end re-verify is deferred to that point.Verification
pnpm run checkgreen — typecheck, eslint, markdownlint, build-check, and full test suite (incl. the 6 newtests/kit/codex-mcp.test.mjscases).Refs #36, #39, ruvnet/ruflo#2766