code-review-gate ยท diff
git:20260325.a6b3a88 to git:20260507.56f011d
31 added, 28 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]
- ---
- # 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
+ ---
+ 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]
+
+ ---
+ # 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