awesome-slash 2.4.2

package/plugins/project-review/commands/project-review.md

@@ -0,0 +1,273 @@
+---
+description: Multi-agent code review with iterative improvement
+argument-hint: "[scope] [--recent] [--domain AGENT] [--quick] [--create-tech-debt]"
+---
+
+# /project-review - Multi-Agent Code Review
+
+Comprehensive code review using specialized AI agents with iterative improvement.
+
+## Quick Reference
+
+| Phase | Description | Details |
+|-------|-------------|---------|
+| 1 | Context & Agent Selection | This file |
+| 2 | Multi-Agent Review | See `project-review-agents.md` |
+| 3-4 | Tech Debt & Fixes | This file |
+| 5-6 | Verification & Iteration | This file |
+| 7 | Completion Report | This file |
+| 8 | GitHub Issues | See `project-review-github.md` |
+
+## Arguments
+
+Parse from $ARGUMENTS:
+- **Scope**: Path to review (default: `.`) or `--recent` (last 5 commits only)
+- **--domain AGENT**: Review with specific agent only (e.g., `--domain security`)
+- **--quick**: Single pass, no iteration (fast feedback)
+- **--create-tech-debt**: Force create/update TECHNICAL_DEBT.md
+
+## Phase 1: Context Gathering
+
+### Platform Detection
+
+```bash
+PLATFORM=$(node ${CLAUDE_PLUGIN_ROOT}/lib/platform/detect-platform.js)
+TOOLS=$(node ${CLAUDE_PLUGIN_ROOT}/lib/platform/verify-tools.js)
+
+PROJECT_TYPE=$(echo $PLATFORM | jq -r '.projectType')
+PACKAGE_MGR=$(echo $PLATFORM | jq -r '.packageManager')
+
+# Detect framework
+FRAMEWORK="unknown"
+if [ "$PROJECT_TYPE" = "nodejs" ]; then
+  [ -n "$(jq -e '.dependencies.react' package.json 2>/dev/null)" ] && FRAMEWORK="react"
+  [ -n "$(jq -e '.dependencies.express' package.json 2>/dev/null)" ] && FRAMEWORK="express"
+elif [ "$PROJECT_TYPE" = "python" ]; then
+  grep -q "django" requirements.txt 2>/dev/null && FRAMEWORK="django"
+  grep -q "fastapi" requirements.txt 2>/dev/null && FRAMEWORK="fastapi"
+fi
+```
+
+### Project Analysis
+
+```bash
+FILE_COUNT=$(git ls-files | wc -l)
+TEST_FILES=$(git ls-files | grep -E '(test|spec)\.' | wc -l)
+HAS_TESTS=$( [ "$TEST_FILES" -gt 0 ] && echo "true" || echo "false" )
+HAS_DB=$(grep -rq -E "(Sequelize|Prisma|TypeORM)" . 2>/dev/null && echo "true" || echo "false")
+HAS_API=$(grep -rq -E "(express|fastify|@nestjs)" . 2>/dev/null && echo "true" || echo "false")
+HAS_CICD=$([ -d ".github/workflows" ] && echo "true" || echo "false")
+```
+
+### Agent Selection
+
+**Always Active:**
+- `security-expert`: Security vulnerabilities, auth, input validation
+- `performance-engineer`: Performance bottlenecks, algorithms, memory
+
+**Conditional:**
+- `test-quality-guardian`: Test coverage and quality (if `HAS_TESTS=true`)
+- `architecture-reviewer`: Design patterns (if `FILE_COUNT > 50`)
+- `database-specialist`: Query optimization (if `HAS_DB=true`)
+- `api-designer`: REST best practices (if `HAS_API=true`)
+- `frontend-specialist`: Component design (if React/Vue/Angular)
+- `devops-reviewer`: CI/CD config (if `HAS_CICD=true`)
+
+## Phase 2: Multi-Agent Review
+
+See `project-review-agents.md` for detailed agent coordination.
+
+### Finding Format (Required)
+
+Every finding MUST include:
+- **File:Line**: Exact location (e.g., `src/auth/session.ts:42`)
+- **Severity**: critical | high | medium | low
+- **Category**: From agent domain
+- **Description**: What's wrong and why
+- **Code Quote**: 1-3 lines showing issue
+- **Suggested Fix**: Specific remediation
+- **Effort**: small | medium | large
+
+### Example Finding
+
+```markdown
+### Finding: Unsafe SQL Query
+**Agent**: security-expert
+**File**: src/api/users.ts:87
+**Severity**: critical
+**Code**:
+```typescript
+const query = `SELECT * FROM users WHERE id = ${userId}`;
+```
+**Fix**: Use parameterized queries.
+**Effort**: small
+```
+
+## Phase 3: Tech Debt Documentation
+
+If TECHNICAL_DEBT.md exists or `--create-tech-debt`:
+
+```markdown
+# Technical Debt
+
+Last updated: $(date -I)
+
+## Summary
+**Total Issues**: X | Critical: Y | High: Z | Medium: A | Low: B
+
+## Critical Issues
+[Grouped by severity with file:line, description, fix, effort]
+
+## Progress Tracking
+- [ ] Issue 1
+- [ ] Issue 2
+```
+
+## Phase 4: Automated Fixes
+
+### Fix Strategy
+
+1. **Auto-fixable** (lint, formatting): Apply directly
+2. **Manual fix** (code logic): Implement suggested fix
+3. **Design decision required**: Document in tech debt
+4. **False positive**: Remove from list
+
+### Fix Order
+
+1. Critical severity first
+2. Then by effort (small → large)
+3. Then batch by file
+
+## Phase 5: Verification
+
+```bash
+# Run tests
+[ -n "$TEST_CMD" ] && $TEST_CMD
+TEST_STATUS=$?
+
+# Run linter
+[ -n "$LINT_CMD" ] && $LINT_CMD
+LINT_STATUS=$?
+
+# Run build
+[ -n "$BUILD_CMD" ] && $BUILD_CMD
+BUILD_STATUS=$?
+
+# Overall status
+VERIFICATION_PASSED=$([ $TEST_STATUS -eq 0 ] && [ $LINT_STATUS -eq 0 ] && [ $BUILD_STATUS -eq 0 ] && echo "true" || echo "false")
+```
+
+### Handle Failures
+
+If verification fails:
+1. Review recent changes (`git diff`)
+2. Identify breaking fix
+3. Rollback: `git restore <file>`
+4. Document as "fix caused regression"
+
+## Phase 6: Iteration
+
+```javascript
+const MAX_ITERATIONS = 5;
+let iteration = 1;
+
+while (iteration <= MAX_ITERATIONS) {
+  const fixResult = applyFixes(remainingIssues);
+
+  const verifyResult = runVerification();
+  if (!verifyResult.passed) {
+    rollbackFailed(fixResult);
+  }
+
+  const reReviewResult = reReview(fixResult.changedFiles);
+  remainingIssues = reReviewResult.issues;
+
+  if (remainingIssues.length === 0) {
+    console.log("✓ Zero issues remaining!");
+    break;
+  }
+
+  iteration++;
+}
+```
+
+### Quick Mode
+
+If `--quick` flag: Single pass, findings only, no fixes.
+
+## Phase 7: Completion Report
+
+```markdown
+# Project Review Complete
+
+**Scope**: ${SCOPE} | **Framework**: ${FRAMEWORK}
+**Iterations**: ${iteration} | **Duration**: ${duration}
+
+## Summary
+**Issues Found**: ${initialCount}
+**Issues Fixed**: ${fixedCount}
+**Remaining**: ${remainingCount}
+
+## By Severity
+- Critical: ${criticalFound} → ${criticalRemaining}
+- High: ${highFound} → ${highRemaining}
+- Medium: ${mediumFound} → ${mediumRemaining}
+- Low: ${lowFound} → ${lowRemaining}
+
+## Verification
+- Tests: ✓/✗
+- Linter: ✓/✗
+- Build: ✓/✗
+
+## Files Changed
+${FILE_COUNT} files modified
+
+## Remaining Issues
+[List of issues needing attention]
+```
+
+## Phase 8: GitHub Issue Creation
+
+See `project-review-github.md` for:
+- Creating GitHub issues for deferred items
+- Security issue handling (no public issues)
+- TECHNICAL_DEBT.md cleanup
+
+## Error Handling
+
+### No Framework Detected
+```
+Framework detection failed, using generic patterns.
+```
+
+### No Tests Available
+```
+No test suite detected. Skipping test-quality-guardian.
+```
+
+### All Agents Failed
+```
+ERROR: All review agents failed.
+Try: --recent or specific path for smaller scope.
+```
+
+## Usage Examples
+
+```bash
+/project-review                    # Full review
+/project-review --recent           # Last 5 commits only
+/project-review src/api            # Specific path
+/project-review --domain security  # Security audit only
+/project-review --quick            # Fast feedback, no fixes
+/project-review --create-tech-debt # Force tech debt file
+```
+
+## Success Criteria
+
+- ✓ All agents completed review
+- ✓ Evidence-based findings (file:line provided)
+- ✓ Critical issues fixed or documented
+- ✓ Verification passes
+- ✓ TECHNICAL_DEBT.md updated (if enabled)
+
+Begin Phase 1 now.