Code Review Gate

---
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]
  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