architecture-review
Installation
SKILL.md
Architecture Review (Solution Architect)
This is the architect's loop: does the running system still match the intended architecture, and are the guards (fitness functions, ADRs) still true?
It is not a pull-request nitpick, a quality gate, or an incident review.
| Concern | Owner | Do not do here |
|---|---|---|
| C4 vs code, wrong quantum, missing ADR, dead fitness function | SA (this skill) | — |
| Implementation, tests, PR correctness | SE (merge-request-review) |
Do not review the diff as if you were the engineer |
| SLO burn, on-call, capacity, failover ops | SRE | Do not write runbooks or page anyone |
| Threat model, secrets, vuln class | SecOps (threat-modeling, security-analysis) |
Do not claim a security review |
| Coverage / SAST / merge gate | QC (code-quality-gate) |
Do not "pass" architecture because CI is green |
Diagrams: c4-model and the L1–L3 skills. Fitness functions: evolutionary-architecture (do not reinvent them here). Decisions already made: read architecture-decision-records instead of relitigating. Why the system exists: why-strategic-rationale.