AI agent skill · by @a5c-ai

Code Review Gate

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

Archived 2026-10-23 · Source updated 2026-09-16 · License: MIT. Check the original repository for the latest version.

SKILL.md instructions

# 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