Review:architecture

Architecture Code Review

Run an architecture-focused review using 4 architecture checklists via the senior-review-specialist agent.

Instructions

Spawn the senior-review-specialist agent to perform this review.

Checklists to Apply

Load and apply these review checklists:

• commands/review/architecture.md - Boundaries, dependencies, layering
• commands/review/performance.md - Algorithmic efficiency, N+1 queries, bottlenecks
• commands/review/scalability.md - Load handling, dataset growth, multi-tenancy
• commands/review/api-contracts.md - Stability, correctness, consumer usability

Agent Instructions

The agent should:

1. Get working tree changes: Run git diff to see all changes
2. Map the architecture:
   • Identify architectural layers (presentation, service, domain, infra)
   • Identify dependency direction
   • Identify coupling points
3. For each changed file:
   • Read the full file content
   • Go through each diff hunk
   • Apply all 4 architecture checklists
   • Look for layer violations and circular dependencies
4. Cross-reference related files: Check import graphs, module boundaries
5. Assess coupling impact: How many modules are affected?

Output Format

Generate an architecture review report with:

• Critical Issues (BLOCKER): Circular dependencies, boundary violations
• High Priority Issues: God objects, coupling problems
• Medium Priority Issues: Missing abstractions, low cohesion
• Architectural Map: Layers, boundaries, dependency direction
• Coupling Metrics: Fan-in, fan-out, affected modules
• File Summary: Architecture issues per file
• Overall Assessment: Architecture health recommendation