code-review-gate ยท diff
git:20260507.9f5be95 to git:20260601.da7723a
27 added, 27 removed. Audit A to A.
- ---
- 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]
+ ---
+ 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
+ # 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