What problem does it solve? Codebases drift from their intended architecture over time, introducing circular dependencies, bloated public APIs, and inconsistent patterns that erode maintainability. This Skill systematically reviews code changes against established project conventions to catch architectural violations before they merge. ## Core Features & Use Cases - Pattern Compliance Review: Reads AGENTS.md and project documentation to verify changed code follows established conventions and dependency flow rules. - Module Boundary Enforcement: Detects circular dependencies, checks that public API surfaces stay minimal, and validates layer isolation using frameworks like ArchUnit, import-linter, dependency-cruiser, and arch-go. - Parallel Multi-Module Review: Spawns subagents to review each module boundary in parallel for changes spanning multiple modules, and coordinates findings with performance and security reviewers. - Use Case: During a pull request touching several packages, the reviewer flags a new import that creates a circular dependency between the domain and infrastructure layers, classifying it as a VIOLATION with a suggested fix. ## Quick Start Ask the agent to review the current code changes for architectural compliance and report any violations, drift, or suggestions.
