Code & Architecture Review Skill

Activate this skill when reviewing code changes, evaluating pull requests, or conducting a project-wide health review (/review or tailor review).

Review Directive: Provide actionable, prioritized feedback focusing on architecture, security, code reuse, and long-term maintainability.

1. Multi-Dimensional Review Rubric

Evaluate code changes across 5 core dimensions:

1. Architecture & Boundaries: Does the implementation violate established ADRs in .ai/DECISIONS.md or cross layer boundaries illegally?
2. Reuse & Duplication: Could this code have reused existing components, utilities, or abstractions instead of introducing duplicates?
3. Dependency Governance: Were unnecessary external packages added when standard library primitives or native APIs suffice?
4. Security & Validation: Are user inputs sanitized? Are database queries parameterized? Are secrets kept out of source code?
5. Testing & Quality: Are there targeted tests covering happy paths, error handling, and edge cases?