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