code-review-gate · diff

git:20260507.56f011d to git:20260507.8dfdcb4

6 added, 4 removed. Audit A to C.

---

name: code-review-gate

description: Architect code review with DRY, YAGNI, abstraction, and test coverage principle enforcement

allowed-tools: [Read, Write, Edit, Bash, Grep, Glob]

- graph:

- domains: [domain:software-engineering]

- ---

+ graph:

+ domains: [domain:software-engineering]

+ skillAreas: [skill-area:agentic-loops, skill-area:orchestration-loop]

+ workflows: [workflow:feature-development]

+ topics: [topic:developer-experience]

+ roles: [role:tech-lead, role:backend-engineer]

# Code Review Gate

## Capabilities

Performs architect-level code review enforcing four core principles: DRY (no unnecessary duplication), YAGNI (no speculative features), proper abstraction (correct encapsulation), and test coverage (adequate automated tests). Provides numeric quality scores and specific file:line feedback.

## Tool Use Instructions

- Use **Read** to examine code changes and test files

- Use **Grep** to search for duplication patterns and anti-patterns

- Use **Glob** to verify test file coverage

- Use **Bash** to run lint, test, and coverage commands

- Use **Write** to generate review reports

## Process Integration

- Used in `maestro-orchestrator.js` Phase 4 (Architect Code Review)

- Used in `maestro-development.js` (PR Review Cycle)

- Used in `maestro-hotfix.js` (Expedited Review)

- Maps to tasks: `maestro-architect-code-review`, `maestro-dev-architect-review`, `maestro-hotfix-review`

- Agents: Architect, Code Reviewer

- Quality convergence loop: rejected code returns to coder for fixes

- Checks are "turned up to 11" by default