nledford Architecture Review Skill
Use this skill as an architecture-specific review lens for an existing system,
proposal, or diff. Always load
review-verification-protocol
before reporting findings. For repository change reviews, also load
code-review.
Do not use it for simple code changes with no boundary impact. Use the matching Clean, Hexagonal, Onion, or DDD skill when the primary task is designing or implementing an architecture rather than reviewing one.
Workflow
- Read project-local guidance, architecture docs, module structure, and the relevant diff before inferring the intended architecture.
- Map dependency direction, module and bounded-context boundaries, ports, adapters, and framework or persistence edges.
- Identify concrete leakage, cycles, misplaced policy, and boundary bypasses.
- Distinguish intentional hybrid choices from accidental drift; prefer coherent, testable structure over pattern purity.
- Verify findings against imports, call paths, tests, or build rules. Record skipped checks and residual architectural risk.
Output
Return findings ordered by severity with file-level evidence, impact, the smallest practical correction, skipped validation, and residual risk.