← ClaudeAtlas

architecture-review-criterialisted

Use when reviewing DESIGN artefacts (event models, ADRs, component diagrams, context maps, interface contracts) for quality, DDD compliance, and architectural correctness. Contains gate definitions and scoring rubric for the solution-architect-reviewer lenses.
SebastienDegodez/skraft-plugin · ★ 4 · Code & Development · score 60
Install: claude install-skill SebastienDegodez/skraft-plugin
# Architecture Review Criteria ## Overview 16 gates across 3 lenses, plus 1 cross-cutting escalation gate, applied by the `solution-architect-reviewer` agent to DESIGN artefacts. Gates enforce DDD correctness, Clean Architecture compliance, fitness for the stories in scope, cross-artefact consistency, the prohibition of negative or baseline-restating ADRs, and the integrity of the human-escalation protocol. **Applied by:** `solution-architect-reviewer` **Applied to:** ADRs, event models, component diagrams, context maps, interface contracts, consistency matrices, supersession plans, supersession registry, blocker files **Prior phase required:** DESIGN artefacts from `solution-architect` --- ## Gate Definitions ### Lens 1 — consistency-lens Evaluates: ADRs + supersession registry + diagrams + contracts + consistency matrices + supersession plans + blockers | Gate | Definition | Pass condition | Severity | |---|---|---|---| | G1 | Every structural commitment — whether visible in a diagram OR detected in the existing codebase by Phase 7.0 grep signatures (`ICommandBus\|IQueryBus\|CommandBus\|QueryBus`, `IEventStore\|EventStream`, `Saga\|ProcessManager`, cross-context ACL) — has a traceable `Accepted` ADR justification. Back-fill ADRs are required when production code carries a structural pattern not yet covered by any ADR. | Every structural element in diagrams AND every grep hit from the persona's Step 7.0 detection table references at least one `Accepted` ADR. | BLOCKE