feat(architecture): architecture command + MCP tool (Phase 2/4)#13
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5b2fe8e551
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| typer.echo(json_renderer.render(payload) if want_json else md_renderer.render(payload)) | ||
|
|
||
|
|
||
| @app.command("architecture") |
There was a problem hiding this comment.
When skill/plugin users invoke the supported cbx wrapper instead of the raw codebase-index binary, this new subcommand is still blocked: skill/scripts/cbx, skill/scripts/cbx.ps1, the packaged template copies, and bin/cbx only allow search explain symbol refs impact graph stats doctor update index, so cbx architecture exits 2 before reaching this command. Please add architecture to those synced allowlists so the advertised command works through the bundled wrappers.
Useful? React with 👍 / 👎.
…MCP tool Phase 2 of porting graphify into codebase-index. Surfaces the analytics cached in Phase 1 as a high-level codebase map. - `codebase-index architecture` (+ --json): modules with auto-labels, god nodes, surprising cross-module bridges, suggested questions; reads the cached summary (no recompute). Reports "unavailable" + reindex hint on a pre-analytics index. - MCP tool `architecture_overview` exposes the same payload via the stable envelope; added to server instructions. - service.architecture_payload (shared by CLI + MCP) + render_architecture. - Tests: payload available/unavailable paths; CLI + MCP goldens regenerated. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
denfry
force-pushed
the
feat/architecture-command
branch
from
5b2fe8e to
966e6cd
Compare
1 participant
Phase 2 of 4 — porting safishamsi/graphify into codebase-index
Surfaces the analytics engine from Phase 1 as a user- and agent-facing codebase map.
Added
codebase-index architecture(--jsonfor structured output) — a high-level map read from the analytics cached at index time (no recompute):On a pre-analytics index it reports
available: falsewith a reindex hint instead of crashing.architecture_overviewMCP tool — the same payload over the stable MCP envelope, so an agent can orient before diving in. Added to the server instructions.service.architecture_payload(shared by CLI + MCP, no drift) andrender_architecture.Example (this repo)
Tests / CI
pytest: 381 passed, 14 skipped ·ruff: clean ·mypy: clean🤖 Generated with Claude Code