devflow-kit 0.5.0 → 0.6.0

package/src/claude/commands/devflow/code-review.md

@@ -5,378 +5,427 @@ allowed-tools: Task, Bash, Read, Write, Grep, Glob
 
 ## Your Task
 
-Perform a comprehensive review of this entire feature branch by orchestrating multiple specialized sub-agents in parallel. This is designed for thorough analysis before creating pull requests or merging branches.
+Orchestrate multiple specialized audit sub-agents to review the current branch, then synthesize their findings into an actionable summary.
 
-### Step 1: Analyze Branch Changes
+---
+
+## Step 1: Determine Review Scope
 
-First, determine the branch and base for comparison:
+Get the current branch and base branch:
 
 ```bash
 # Get current branch
 CURRENT_BRANCH=$(git branch --show-current)
 if [ -z "$CURRENT_BRANCH" ]; then
-    echo "❌ Not on a branch (detached HEAD). Checkout a feature branch first."
+    echo "❌ Not on a branch (detached HEAD)"
     exit 1
 fi
 
-# Determine base branch (main, master, or develop)
+# Find base branch
 BASE_BRANCH=""
 for branch in main master develop; do
-    if git show-ref --verify --quiet refs/heads/$branch; then
-        BASE_BRANCH=$branch
-        break
-    fi
+  if git show-ref --verify --quiet refs/heads/$branch; then
+    BASE_BRANCH=$branch
+    break
+  fi
 done
 
 if [ -z "$BASE_BRANCH" ]; then
-    echo "❌ Could not find base branch (main/master/develop). Specify manually."
+    echo "❌ Could not find base branch (main/master/develop)"
     exit 1
 fi
 
-echo "=== BRANCH REVIEW SCOPE ==="
-echo "Current branch: $CURRENT_BRANCH"
-echo "Base branch: $BASE_BRANCH"
-echo ""
-
-# Check if there are changes to review
+# Check for changes
 if git diff --quiet $BASE_BRANCH...HEAD; then
-    echo "No changes between $BASE_BRANCH and $CURRENT_BRANCH"
+    echo "ℹ️ No changes between $BASE_BRANCH and $CURRENT_BRANCH"
     exit 0
 fi
 
-# Show comprehensive change summary
-echo "=== CHANGES TO REVIEW ==="
+# Show change summary
+echo "=== CODE REVIEW SCOPE ==="
+echo "Branch: $CURRENT_BRANCH"
+echo "Base: $BASE_BRANCH"
+echo ""
 git diff --stat $BASE_BRANCH...HEAD
 echo ""
-echo "=== COMMIT HISTORY ==="
-git log --oneline $BASE_BRANCH..HEAD
+git log --oneline $BASE_BRANCH..HEAD | head -5
 echo ""
 ```
 
-### Step 2: Detect Change Categories
+---
 
-Analyze what types of changes are in this branch to determine which specialized agents are needed:
+## Step 2: Set Up Audit Structure
 
-```bash
-# Check if database-related files changed
-DB_CHANGES=$(git diff --name-only $BASE_BRANCH...HEAD | grep -E '\.(sql|prisma|migration|knex|sequelize|db)' || true)
-DB_CHANGES+=$(git diff --name-only $BASE_BRANCH...HEAD | grep -iE '(migration|schema|database|models/)' || true)
-
-if [ -n "$DB_CHANGES" ]; then
-    echo "🗄️  Database changes detected - will run database audit"
-    INCLUDE_DB_AUDIT=true
-else
-    echo "ℹ️  No database changes detected - skipping database audit"
-    INCLUDE_DB_AUDIT=false
-fi
-echo ""
+Create directory for audit reports:
 
-# Set up audit directory structure
+```bash
 TIMESTAMP=$(date +%Y-%m-%d_%H%M)
 AUDIT_BASE_DIR=".docs/audits/${CURRENT_BRANCH}"
 mkdir -p "$AUDIT_BASE_DIR"
 
-echo "📁 Audit reports will be saved to: $AUDIT_BASE_DIR"
+echo "📁 Audit reports: $AUDIT_BASE_DIR"
 echo ""
 ```
 
-### Step 3: Launch Specialized Sub-Agents in Parallel
+---
 
-Launch these sub-agents in parallel based on change detection.
+## Step 3: Launch Audit Sub-Agents in Parallel
 
-**IMPORTANT**: Pass the following variables to each sub-agent:
-- `CURRENT_BRANCH`: The branch being reviewed
-- `AUDIT_BASE_DIR`: Base directory for audit reports (`.docs/audits/${CURRENT_BRANCH}`)
-- `TIMESTAMP`: Current timestamp for report filenames
+Use the Task tool to launch all audit sub-agents in parallel. Each will analyze the branch and save its report.
+
+**Launch these sub-agents:**
+
+Use Task tool with `subagent_type` for each audit:
 
-Each sub-agent should save its report to:
-```
-${AUDIT_BASE_DIR}/<audit-type>-report.${TIMESTAMP}.md
 ```
+1. Launch audit-security sub-agent:
+   "Analyze branch ${CURRENT_BRANCH} for security issues. Compare against ${BASE_BRANCH}. Save report to ${AUDIT_BASE_DIR}/security-report.${TIMESTAMP}.md"
 
-**Example paths**:
-- `.docs/audits/feature-auth/security-report.2025-10-18_1430.md`
-- `.docs/audits/feature-auth/performance-report.2025-10-18_1430.md`
+2. Launch audit-performance sub-agent:
+   "Analyze branch ${CURRENT_BRANCH} for performance issues. Compare against ${BASE_BRANCH}. Save report to ${AUDIT_BASE_DIR}/performance-report.${TIMESTAMP}.md"
 
-**Core Audits (Always Run)**:
-1. audit-security sub-agent → `${AUDIT_BASE_DIR}/security-report.${TIMESTAMP}.md`
-2. audit-performance sub-agent → `${AUDIT_BASE_DIR}/performance-report.${TIMESTAMP}.md`
-3. audit-architecture sub-agent → `${AUDIT_BASE_DIR}/architecture-report.${TIMESTAMP}.md`
-4. audit-tests sub-agent → `${AUDIT_BASE_DIR}/tests-report.${TIMESTAMP}.md`
-5. audit-complexity sub-agent → `${AUDIT_BASE_DIR}/complexity-report.${TIMESTAMP}.md`
-6. audit-dependencies sub-agent → `${AUDIT_BASE_DIR}/dependencies-report.${TIMESTAMP}.md`
-7. audit-documentation sub-agent → `${AUDIT_BASE_DIR}/documentation-report.${TIMESTAMP}.md`
+3. Launch audit-architecture sub-agent:
+   "Analyze branch ${CURRENT_BRANCH} for architecture issues. Compare against ${BASE_BRANCH}. Save report to ${AUDIT_BASE_DIR}/architecture-report.${TIMESTAMP}.md"
 
-**Conditional Audits** (automatically detect and skip if not applicable):
-8. audit-typescript sub-agent → `${AUDIT_BASE_DIR}/typescript-report.${TIMESTAMP}.md`
-9. audit-database sub-agent → `${AUDIT_BASE_DIR}/database-report.${TIMESTAMP}.md`
+4. Launch audit-tests sub-agent:
+   "Analyze branch ${CURRENT_BRANCH} for test coverage and quality issues. Compare against ${BASE_BRANCH}. Save report to ${AUDIT_BASE_DIR}/tests-report.${TIMESTAMP}.md"
 
-### Step 4: Synthesize Comprehensive Review
+5. Launch audit-complexity sub-agent:
+   "Analyze branch ${CURRENT_BRANCH} for code complexity issues. Compare against ${BASE_BRANCH}. Save report to ${AUDIT_BASE_DIR}/complexity-report.${TIMESTAMP}.md"
 
-After all sub-agents complete their analysis:
+6. Launch audit-dependencies sub-agent:
+   "Analyze branch ${CURRENT_BRANCH} for dependency issues. Compare against ${BASE_BRANCH}. Save report to ${AUDIT_BASE_DIR}/dependencies-report.${TIMESTAMP}.md"
 
-1. **Collect Results**: Gather findings from all 7-8 specialized sub-agents (depending on change types)
-2. **Cross-Reference Issues**: Identify overlapping concerns between domains
-3. **Prioritize for PR**: Focus on merge-blocking vs nice-to-have improvements
-4. **Create PR-Ready Review**: Structure for easy consumption by human reviewers
+7. Launch audit-documentation sub-agent:
+   "Analyze branch ${CURRENT_BRANCH} for documentation issues. Compare against ${BASE_BRANCH}. Save report to ${AUDIT_BASE_DIR}/documentation-report.${TIMESTAMP}.md"
 
-### Step 5: Save Comprehensive Review Document
+8. Launch audit-typescript sub-agent (if TypeScript project):
+   "Analyze branch ${CURRENT_BRANCH} for TypeScript issues. Compare against ${BASE_BRANCH}. Save report to ${AUDIT_BASE_DIR}/typescript-report.${TIMESTAMP}.md"
 
-Create a detailed review document at `${AUDIT_BASE_DIR}/comprehensive-review.${TIMESTAMP}.md`:
+9. Launch audit-database sub-agent (if database changes detected):
+   "Analyze branch ${CURRENT_BRANCH} for database issues. Compare against ${BASE_BRANCH}. Save report to ${AUDIT_BASE_DIR}/database-report.${TIMESTAMP}.md"
+```
+
+**IMPORTANT:** Launch ALL applicable sub-agents in a single message using multiple Task tool calls for parallel execution.
+
+---
+
+## Step 4: Read Audit Reports
+
+After all sub-agents complete, read each generated report:
 
 ```bash
-REVIEW_FILE="${AUDIT_BASE_DIR}/comprehensive-review.${TIMESTAMP}.md"
+# List generated reports
+ls -1 "$AUDIT_BASE_DIR"/*-report.${TIMESTAMP}.md
 ```
 
-**File structure**:
+Use the Read tool to read each report file:
+- `${AUDIT_BASE_DIR}/security-report.${TIMESTAMP}.md`
+- `${AUDIT_BASE_DIR}/performance-report.${TIMESTAMP}.md`
+- `${AUDIT_BASE_DIR}/architecture-report.${TIMESTAMP}.md`
+- `${AUDIT_BASE_DIR}/tests-report.${TIMESTAMP}.md`
+- `${AUDIT_BASE_DIR}/complexity-report.${TIMESTAMP}.md`
+- `${AUDIT_BASE_DIR}/dependencies-report.${TIMESTAMP}.md`
+- `${AUDIT_BASE_DIR}/documentation-report.${TIMESTAMP}.md`
+- (Plus typescript and database reports if generated)
+
+---
+
+## Step 5: Extract Blocking Issues
+
+From each report, extract issues from the **🔴 Issues in Your Changes** section.
+
+These are blocking issues introduced in this branch that must be fixed before merge.
+
+For each report:
+1. Look for the "🔴 Issues in Your Changes (BLOCKING)" section
+2. Extract all CRITICAL and HIGH severity issues
+3. Note the file:line references
+
+Create a consolidated list of all blocking issues across all audits.
+
+---
+
+## Step 6: Create Summary Report
+
+Create a comprehensive summary at `${AUDIT_BASE_DIR}/review-summary.${TIMESTAMP}.md`:
 
 ```markdown
-# Branch Review - {BRANCH_NAME}
-**Date**: {current_date}
-**Time**: {current_time}
-**Type**: Branch Review (PR Readiness Assessment)
-**Branch**: {CURRENT_BRANCH}
-**Base**: {BASE_BRANCH}
-**Reviewer**: AI Sub-Agent Orchestra
+# Code Review Summary - ${CURRENT_BRANCH}
+
+**Date**: $(date +%Y-%m-%d %H:%M:%S)
+**Branch**: ${CURRENT_BRANCH}
+**Base**: ${BASE_BRANCH}
+**Audits Run**: {count} specialized audits
 
 ---
 
-## 📊 Branch Overview
+## 🚦 Merge Recommendation
 
-**Commits**: {commit_count} commits
-**Files Changed**: {files_changed}
-**Lines Added**: {lines_added}
-**Lines Removed**: {lines_removed}
-**Review Duration**: {duration}
+{One of:}
+- ❌ **BLOCK MERGE** - Critical issues in your changes must be fixed
+- ⚠️ **REVIEW REQUIRED** - High priority issues need attention
+- ✅ **APPROVED WITH CONDITIONS** - Minor issues to address
+- ✅ **APPROVED** - No blocking issues found
 
-### Change Categories
-- 🎯 **Features**: {feature_changes}
-- 🐛 **Bug Fixes**: {bug_fixes}
-- 🔧 **Refactoring**: {refactoring}
-- 📚 **Documentation**: {doc_changes}
-- 🧪 **Tests**: {test_changes}
+**Confidence**: {High/Medium/Low}
 
 ---
 
-## 🎯 PR Readiness Assessment
+## 🔴 Blocking Issues (Must Fix Before Merge)
+
+Issues introduced in lines you added or modified:
+
+### Security (CRITICAL: X, HIGH: Y)
+{List critical/high issues from security audit's 🔴 section}
+- **[Issue]** - `file:line` - {description}
+
+### Performance (CRITICAL: X, HIGH: Y)
+{List critical/high issues from performance audit's 🔴 section}
+- **[Issue]** - `file:line` - {description}
 
-### 🚦 MERGE RECOMMENDATION
-**Status**: {✅ READY TO MERGE | ⚠️ ISSUES TO ADDRESS | 🚫 NOT READY}
+### Architecture (HIGH: X)
+{List high issues from architecture audit's 🔴 section}
+- **[Issue]** - `file:line` - {description}
 
-**Confidence Level**: {High/Medium/Low}
+### Tests (HIGH: X)
+{List high issues from tests audit's 🔴 section}
+- **[Issue]** - `file:line` - {description}
 
-### Blocking Issues (Must Fix Before Merge)
-- 🔴 {critical_issue_1} in {file:line}
-- 🔴 {critical_issue_2} in {file:line}
+### Complexity (HIGH: X)
+{List high issues from complexity audit's 🔴 section}
+- **[Issue]** - `file:line` - {description}
 
-### High Priority (Should Fix Before Merge)
-- 🟠 {high_issue_1} in {file:line}
-- 🟠 {high_issue_2} in {file:line}
+### Dependencies (CRITICAL: X, HIGH: Y)
+{List critical/high issues from dependencies audit's 🔴 section}
+- **[Issue]** - `file:line` - {description}
+
+### Documentation (HIGH: X)
+{List high issues from documentation audit's 🔴 section}
+- **[Issue]** - `file:line` - {description}
+
+### TypeScript (HIGH: X)
+{If applicable - list high issues from typescript audit's 🔴 section}
+- **[Issue]** - `file:line` - {description}
+
+### Database (CRITICAL: X, HIGH: Y)
+{If applicable - list critical/high issues from database audit's 🔴 section}
+- **[Issue]** - `file:line` - {description}
 
 ---
 
-## 🔍 Detailed Sub-Agent Analysis
+## ⚠️ Should Fix While You're Here
 
-### 🔒 Security Analysis (audit-security)
-**Risk Level**: {Low/Medium/High/Critical}
+Issues in code you touched (from ⚠️ sections of each audit):
 
-#### Security Issues Found
-{detailed security findings with file:line references}
+{Count of issues by audit - don't list all, just summarize}
+- Security: {count} issues in code you touched
+- Performance: {count} issues in code you touched
+- Architecture: {count} issues in code you touched
+- Tests: {count} issues in code you touched
+- Complexity: {count} issues in code you touched
 
-#### Security Recommendations
-{specific security improvements needed}
+See individual audit reports for details.
 
-### 📘 TypeScript Analysis (audit-typescript)
-**Type Safety**: {Excellent/Good/Acceptable/Poor}
-**Note**: Only included if TypeScript files changed or project uses TypeScript
+---
 
-#### TypeScript Issues Found
-{detailed type safety findings with file:line references}
+## ℹ️ Pre-existing Issues Found
 
-#### TypeScript Recommendations
-{specific type safety improvements needed}
+Issues unrelated to your changes (from ℹ️ sections):
 
-### ⚡ Performance Analysis (audit-performance)
-**Performance Impact**: {Positive/Neutral/Negative}
+{Count by audit}
+- Security: {count} pre-existing issues
+- Performance: {count} pre-existing issues
+- Architecture: {count} pre-existing issues
+- Tests: {count} pre-existing issues
+- Complexity: {count} pre-existing issues
+- Dependencies: {count} pre-existing issues
+- Documentation: {count} pre-existing issues
 
-#### Performance Issues Found
-{detailed performance findings with optimizations}
+Consider fixing in separate PRs.
 
-#### Performance Recommendations
-{specific performance improvements}
+---
 
-### 🏗️ Architecture Analysis (audit-architecture)
-**Architecture Quality**: {Excellent/Good/Acceptable/Poor}
+## 📊 Summary by Category
 
-#### Architectural Issues Found
-{detailed architecture findings and design concerns}
+**Your Changes (🔴 BLOCKING):**
+- CRITICAL: {total_critical}
+- HIGH: {total_high}
+- MEDIUM: {total_medium}
 
-#### Architecture Recommendations
-{specific architectural improvements}
+**Code You Touched (⚠️ SHOULD FIX):**
+- HIGH: {total_high}
+- MEDIUM: {total_medium}
 
-### 🧪 Test Coverage Analysis (audit-tests)
-**Coverage Assessment**: {Excellent/Good/Adequate/Insufficient}
+**Pre-existing (ℹ️ OPTIONAL):**
+- MEDIUM: {total_medium}
+- LOW: {total_low}
 
-#### Testing Issues Found
-{detailed test coverage gaps and quality issues}
+---
 
-#### Testing Recommendations
-{specific tests that should be added}
+## 🎯 Action Plan
 
-### 🧠 Complexity Analysis (audit-complexity)
-**Maintainability Score**: {Excellent/Good/Acceptable/Poor}
+**Before Merge (Priority Order):**
 
-#### Complexity Issues Found
-{detailed complexity and maintainability concerns}
+1. {Highest priority blocking issue from any audit}
+   - File: {file:line}
+   - Fix: {recommended fix}
 
-#### Complexity Recommendations
-{specific refactoring suggestions}
+2. {Second highest priority blocking issue}
+   - File: {file:line}
+   - Fix: {recommended fix}
 
-### 📦 Dependency Analysis (audit-dependencies)
-**Dependency Health**: {Excellent/Good/Acceptable/Poor}
+3. {Third highest priority blocking issue}
+   - File: {file:line}
+   - Fix: {recommended fix}
 
-#### Dependency Issues Found
-{detailed dependency concerns and security issues}
+{Continue for all blocking issues}
 
-#### Dependency Recommendations
-{specific dependency management improvements}
+**While You're Here (Optional):**
+- Review ⚠️ sections in individual audit reports
+- Fix issues in code you modified
 
-### 📚 Documentation Analysis (audit-documentation)
-**Documentation Quality**: {Excellent/Good/Acceptable/Poor}
+**Future Work:**
+- Create issues for pre-existing problems
+- Track in technical debt backlog
 
-#### Documentation Issues Found
-{detailed documentation drift, missing docs, stale examples}
+---
 
-#### Documentation Recommendations
-{specific documentation updates needed}
+## 📁 Individual Audit Reports
 
-### 🗄️ Database Analysis (audit-database)
-**Database Health**: {Excellent/Good/Acceptable/Poor}
-**Note**: Only included if database changes detected
+Detailed analysis available in:
+- [Security Audit](security-report.${TIMESTAMP}.md)
+- [Performance Audit](performance-report.${TIMESTAMP}.md)
+- [Architecture Audit](architecture-report.${TIMESTAMP}.md)
+- [Test Coverage Audit](tests-report.${TIMESTAMP}.md)
+- [Complexity Audit](complexity-report.${TIMESTAMP}.md)
+- [Dependencies Audit](dependencies-report.${TIMESTAMP}.md)
+- [Documentation Audit](documentation-report.${TIMESTAMP}.md)
+{If applicable:}
+- [TypeScript Audit](typescript-report.${TIMESTAMP}.md)
+- [Database Audit](database-report.${TIMESTAMP}.md)
 
-#### Database Issues Found
-{detailed database design, migration, and query issues}
+---
 
-#### Database Recommendations
-{specific database improvements needed}
+## 💡 Next Steps
+
+{If blocking issues exist:}
+**Fix blocking issues then re-run `/code-review` to verify**
+
+{If no blocking issues:}
+**Ready to create PR:**
+1. Run `/commit` to create final commits
+2. Run `/pull-request` to create PR with this review as reference
+
+{If issues in touched code:}
+**Consider fixing ⚠️ issues while you're working in these files**
 
 ---
 
-## 🎯 Action Plan
+*Review generated by DevFlow audit orchestration*
+*{Timestamp}*
+```
+
+Save this summary using Write tool.
+
+---
 
-### Pre-Merge Checklist (Blocking)
-- [ ] {blocking_action_1} - {estimated_effort}
-- [ ] {blocking_action_2} - {estimated_effort}
-- [ ] {blocking_action_3} - {estimated_effort}
+## Step 7: Present Results to Developer
 
-### Post-Merge Improvements (Non-Blocking)
-- [ ] {improvement_1} - {estimated_effort}
-- [ ] {improvement_2} - {estimated_effort}
-- [ ] {improvement_3} - {estimated_effort}
+Show clear, actionable summary:
 
-### Follow-Up Tasks
-- [ ] {followup_1}
-- [ ] {followup_2}
+```markdown
+🔍 CODE REVIEW COMPLETE
+
+**Branch**: ${CURRENT_BRANCH}
+**Audits**: {count} specialized audits completed
 
 ---
 
-## 📈 Quality Metrics
+## 🚦 Merge Status
+
+{Show the merge recommendation - one of:}
+❌ **BLOCK MERGE** - {count} critical issues in your changes
+⚠️ **REVIEW REQUIRED** - {count} high priority issues
+✅ **APPROVED WITH CONDITIONS** - {count} minor issues
+✅ **APPROVED** - No blocking issues found
+
+---
 
-### Code Quality Score: {score}/10
+## 🔴 Issues You Introduced ({total_count})
 
-**Breakdown**:
-- Security: {score}/10
-- TypeScript: {score}/10 (if applicable)
-- Performance: {score}/10
-- Architecture: {score}/10
-- Test Coverage: {score}/10
-- Maintainability: {score}/10
-- Dependencies: {score}/10
-- Documentation: {score}/10
-- Database: {score}/10 (if applicable)
+{Show top 3-5 most critical blocking issues}
 
-### Comparison to {BASE_BRANCH}
-- Quality Trend: {Improving/Stable/Declining}
-- Technical Debt: {Reduced/Neutral/Increased}
-- Test Coverage: {Increased/Maintained/Decreased}
+**Security:**
+- {Issue 1} - `file:line`
+
+**Performance:**
+- {Issue 1} - `file:line`
+
+**Architecture:**
+- {Issue 1} - `file:line`
+
+{Show total counts}
+Total blocking issues: {count}
+- CRITICAL: {count}
+- HIGH: {count}
+- MEDIUM: {count}
 
 ---
 
-## 🔗 Related Resources
+## ⚠️ Issues in Code You Touched ({total_count})
 
-### Files Requiring Attention
-- {file1} - {reason}
-- {file2} - {reason}
-- {file3} - {reason}
+{Show counts by audit}
+- Security: {count} issues
+- Performance: {count} issues
+- Architecture: {count} issues
+- Tests: {count} issues
+- Complexity: {count} issues
 
-### Similar Issues in Codebase
-- {related_issue_1} in {location}
-- {related_issue_2} in {location}
+See individual reports for details.
 
-### Documentation Updates Needed
-- {doc_update_1}
-- {doc_update_2}
+---
+
+## ℹ️ Pre-existing Issues ({total_count})
+
+{Show count by audit}
+Found {count} legacy issues unrelated to your changes.
+Consider fixing in separate PRs.
 
 ---
 
-## 💡 Reviewer Notes
+## 📁 Reports Saved
 
-### Human Review Focus Areas
-Based on sub-agent analysis, human reviewers should focus on:
-1. {focus_area_1} - {reason}
-2. {focus_area_2} - {reason}
-3. {focus_area_3} - {reason}
+**Summary**: ${AUDIT_BASE_DIR}/review-summary.${TIMESTAMP}.md
 
-### Discussion Points
-- {discussion_point_1}
-- {discussion_point_2}
-- {discussion_point_3}
+**Individual Audits**:
+{List all generated reports}
 
 ---
 
-*Comprehensive review generated by DevFlow sub-agent orchestration*
-*Next: Address blocking issues, then create PR with this review as reference*
-```
+## 🎯 Next Steps
 
-### Step 6: Provide Executive Summary
+{If blocking issues:}
+1. Fix the {count} blocking issues listed above
+2. Re-run `/code-review` to verify fixes
+3. Then create PR with `/pull-request`
 
-Give the developer a clear, actionable summary:
+{If no blocking issues:}
+1. Review ⚠️ issues (optional improvements)
+2. Create commits: `/commit`
+3. Create PR: `/pull-request`
 
+{Always show:}
+💡 Full details in: ${AUDIT_BASE_DIR}/review-summary.${TIMESTAMP}.md
 ```
-🔍 BRANCH REVIEW COMPLETE: {BRANCH_NAME}
-
-📊 ANALYSIS SUMMARY:
-- Files analyzed: {X} files, {Y} commits
-- Issues found: {Critical} critical, {High} high, {Medium} medium, {Low} low
-- Review confidence: {High/Medium/Low}
-
-🚦 PR READINESS: {✅ READY | ⚠️ ISSUES TO ADDRESS | 🚫 NOT READY}
-
-🎯 CRITICAL ACTIONS BEFORE MERGE:
-1. {Most critical blocking issue}
-2. {Second most critical blocking issue}
-3. {Third most critical blocking issue}
-
-⚡ QUICK WINS:
-- {Easy fix 1} ({estimated time})
-- {Easy fix 2} ({estimated time})
-
-📄 Full review: ${AUDIT_BASE_DIR}/comprehensive-review.${TIMESTAMP}.md
-
-📁 Individual audit reports:
-   ${AUDIT_BASE_DIR}/security-report.${TIMESTAMP}.md
-   ${AUDIT_BASE_DIR}/performance-report.${TIMESTAMP}.md
-   ${AUDIT_BASE_DIR}/architecture-report.${TIMESTAMP}.md
-   ${AUDIT_BASE_DIR}/tests-report.${TIMESTAMP}.md
-   ${AUDIT_BASE_DIR}/complexity-report.${TIMESTAMP}.md
-   ${AUDIT_BASE_DIR}/dependencies-report.${TIMESTAMP}.md
-   ${AUDIT_BASE_DIR}/documentation-report.${TIMESTAMP}.md
-   (+ typescript-report.${TIMESTAMP}.md if applicable)
-   (+ database-report.${TIMESTAMP}.md if applicable)
-
-🔄 NEXT STEPS:
-1. Address blocking issues above
-2. Run `/code-review` after fixes to verify
-3. Create PR using this review as reference
-4. Share review with team for human review focus
-```
+
+---
+
+## Key Principles
+
+1. **Launch sub-agents in parallel** - Use multiple Task calls in one message
+2. **Read all reports** - Don't skip any audit results
+3. **Extract blocking issues** - Focus on 🔴 sections from each report
+4. **Be specific** - File:line references, exact issues, clear fixes
+5. **Prioritize** - Blocking (must fix) vs should fix vs optional
+6. **Be actionable** - Clear next steps based on findings