architecture-reviewlisted

# Architecture Review (System Level) Audit structural health: boundaries, dependency direction, layering, ADR drift. Report findings — never modify code. ## Scope **In scope:** module/context boundaries, dependency cycles, layer violations, namespace cohesion, public-API surface leaks, shared-model pollution, drift from documented ADRs. **Out of scope:** SOLID inside a class, naming of private methods, style, line-level bugs (→ `component-review` or general code review). ## Examples ```bash # Review architecture of a specific PR /architecture-review pr 42 # Review a branch against main /architecture-review branch feat/orders-refactor # Focus on a single namespace/module /architecture-review namespace src/Order # Review the whole project /architecture-review ``` ## Workflow ```mermaid flowchart TD A["Parse target mode"] --> B["Load project context"] B --> C["Collect files in scope"] C --> D["Build dependency map"] D --> E["Run checks"] E --> F["Report findings"] ``` ## Phase 1: Parse Target From `$ARGUMENTS`: - `pr <N>` → `gh pr diff <N>`, `gh pr view <N> --json headRefName,files`, read full file context via `git show <branch>:<file>` - `branch <name>` → `git diff main...<name>`, `git ls-tree -r <name>`, read files via `git show <branch>:<file>` - `namespace <path>` → recursively read files under `<path>` - empty → full-project review (sample entry points + top-level modules) ## Phase 2: Load Project Context Check in order: 1. `.agent-context/layer1-