← ClaudeAtlas

architecture-reviewlisted

Self-executed sweep of a codebase for module-deepening opportunities — shallow interfaces, scattered locality, untestable seams — presented as a visual report the user picks a candidate from, then explored interactively. Unlike `architect` (routed consultation) and `code-review` (diff review), this skill is run by the active primary on the whole codebase. Use when the user says "architecture review," "find deepening opportunities," "where is this codebase getting muddy," "improve the architecture," or "/agent-collab:architecture-review." Also offer this proactively when repeated friction in a working session traces to shallow modules — bouncing between many files to follow one concept, or tests that cannot reach behavior through the current interfaces.
sumitake/agent-collab · ★ 0 · Code & Development · score 61
Install: claude install-skill sumitake/agent-collab
# Architecture review — find and explore deepening opportunities Surface architectural friction and propose **deepening opportunities** — refactors that turn shallow modules into deep ones. The aim is testability and navigability. This skill is **self-executed by the active primary**: it reads the codebase directly and produces its own report. It composes with, and does not replace, the routed skills: for an independent cross-family read on the top candidates, route them through `architect` afterward. ## Design vocabulary Use these terms exactly in every suggestion — do not drift into "component," "service," "API," or "boundary": - **Module** — anything with an interface and an implementation; deliberately scale-agnostic (a function, class, package, or tier-spanning slice). - **Interface** — everything a caller must know to use the module correctly: types, invariants, ordering constraints, error modes, configuration, performance characteristics. - **Implementation** — what is inside the module. - **Depth** — leverage at the interface: how much behavior a caller or test can exercise per unit of interface learned. **Deep** = small interface, large behavior; **shallow** = interface nearly as complex as the implementation. - **Seam** — a place where behavior can be altered without editing in that place; where the interface lives. Placing the seam is its own decision. - **Adapter** — a concrete thing satisfying an interface at a seam (a role, not a substance). On