code-review-orchestration ยท diff
git:20260507.9f5be95 to git:20260601.da7723a
42 added, 42 removed. Audit A to A.
- ---
- name: code-review-orchestration
- description: 6-agent parallel code review orchestration covering architecture, security, performance, testing, quality, and documentation dimensions with weighted scoring.
- allowed-tools: Read, Bash, Grep, Glob
+ ---
+ name: code-review-orchestration
+ description: 6-agent parallel code review orchestration covering architecture, security, performance, testing, quality, and documentation dimensions with weighted scoring.
+ allowed-tools: Read, 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]
- Module boundaries, dependency direction, design pattern adherence, architectural drift.
-
- ### Security (weight: 25%)
- Injection vulnerabilities, auth, secrets, crypto, dependencies, headers.
-
- ### Performance (weight: 15%)
- Algorithmic complexity, resource leaks, database patterns, caching, async.
-
- ### Testing (weight: 15%)
- Coverage, quality, edge cases, isolation, flakiness, integration, error paths.
-
- ### Quality (weight: 15%)
- Naming, readability, error handling, type safety, DRY, comments.
-
- ### Documentation (weight: 10%)
- JSDoc, README updates, changelog, inline comments, type documentation.
-
+ Module boundaries, dependency direction, design pattern adherence, architectural drift.
+
+ ### Security (weight: 25%)
+ Injection vulnerabilities, auth, secrets, crypto, dependencies, headers.
+
+ ### Performance (weight: 15%)
+ Algorithmic complexity, resource leaks, database patterns, caching, async.
+
+ ### Testing (weight: 15%)
+ Coverage, quality, edge cases, isolation, flakiness, integration, error paths.
+
+ ### Quality (weight: 15%)
+ Naming, readability, error handling, type safety, DRY, comments.
+
+ ### Documentation (weight: 10%)
+ JSDoc, README updates, changelog, inline comments, type documentation.
+
---
- ## Scoring and Recommendation
-
- - **APPROVE**: overall >= 80 AND zero critical issues
- - **REQUEST_CHANGES**: overall >= 60 OR has critical issues
- - **REJECT**: overall < 60
-
- ## When to Use
-
- - `/code-review` slash command
- - Post-implementation review in spec execution
- - Pre-merge quality gate
-
- ## Agents Used
-
- - `code-review-coordinator`, `security-analyst`, `performance-analyst`
- - `testing-specialist`, `architecture-reviewer`
-
- ## Processes Used By
-
- - `claudekit-code-review` (primary consumer)
- - `claudekit-orchestrator` (via command dispatch)
+ ## Scoring and Recommendation
+
+ - **APPROVE**: overall >= 80 AND zero critical issues
+ - **REQUEST_CHANGES**: overall >= 60 OR has critical issues
+ - **REJECT**: overall < 60
+
+ ## When to Use
+
+ - `/code-review` slash command
+ - Post-implementation review in spec execution
+ - Pre-merge quality gate
+
+ ## Agents Used
+
+ - `code-review-coordinator`, `security-analyst`, `performance-analyst`
+ - `testing-specialist`, `architecture-reviewer`
+
+ ## Processes Used By
+
+ - `claudekit-code-review` (primary consumer)
+ - `claudekit-orchestrator` (via command dispatch)