CCPM Code Review

Structured code review workflow integrated with CCPM's Linear-based project management system. Enforces "no completion claims without verification evidence" principle.

When to Use

This skill auto-activates when:

• User says "done", "complete", "finished", "ready to merge"
• Running /ccpm:verify command (natural workflow - recommended)
• Running /ccpm:done command (includes pre-flight verification)
• Running /ccpm:verify command (advanced)
• Before updating Linear task status to "Done"
• Before syncing Jira status
• Before creating GitHub/BitBucket PR
• Before sending Slack completion notifications

CCPM Verification Workflow

CCPM provides a streamlined 3-step verification process:

Step 1: Quality Checks (Automated)

Run linting, tests, and build checks to ensure technical correctness:

/ccpm:verify [issue-id]  # Auto-detects issue from git branch

What it checks:

• ✅ Linting passes (no style errors)
• ✅ Tests pass (all unit and integration tests)
• ✅ Build succeeds (no compilation errors)
• ✅ Checklist complete (100% of implementation items)

If checks fail: Command automatically suggests /ccpm:verify to debug systematically.

Step 2: Agent Code Review

After quality checks pass, agent review analyzes:

• ✅ Code quality and best practices
• ✅ Security vulnerabilities
• ✅ Performance implications
• ✅ Requirement fulfillment
• ✅ Regression risks

Step 3: Final Confirmation

Four verification gates must pass:

1. Tests Pass ✅ - Zero failures
2. Build Succeeds ✅ - Exit status 0
3. Checklist Complete ✅ - 100% checked
4. No Blockers ✅ - No unresolved blockers in Linear

Only after all gates pass can task proceed to finalization with /ccpm:done.

Core Principles

1. Technical Correctness Over Social Comfort

Forbidden performative agreement:

• ❌ "Great point!"
• ❌ "You're absolutely right!"
• ❌ "That makes total sense!"

Required rigorous verification:

• ✅ "Let me verify that assumption"
• ✅ "I'll test this approach first"
• ✅ "Here's the evidence: [test output]"

2. No Implementation Before Verification

When receiving feedback:

1. First: Verify the feedback is technically correct
2. Then: Implement the change
3. Never: Blindly implement without understanding

3. NO COMPLETION CLAIMS WITHOUT EVIDENCE

Required evidence before any "done" claim:

• ✅ Tests: All passing (screenshot or CI link)
• ✅ Build: Exit status 0 (no errors)
• ✅ Linear checklist: 100% complete
• ✅ No unresolved blockers in Linear comments

Integration with CCPM Commands

With /ccpm:verify (Natural Workflow - Recommended)

This skill enforces verification during the streamlined verify command:

# User runs natural verification command
/ccpm:verify

# Or with explicit issue ID
/ccpm:verify AUTH-123

Flow:

User: "I'm done, let me verify"

Claude: [ccpm-code-review activates]

━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
🔍 Smart Verify Command
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

📋 Issue: AUTH-123 - Add user authentication
📊 Status: In Progress
📋 Checklist: 100% (5/5 items)

Verification Flow:
──────────────────
1. Quality Checks (linting, tests, build)
2. Final Verification (code review, security)

═══════════════════════════════════════
Step 1/2: Running Quality Checks
═══════════════════════════════════════

🔍 Running linting...
✅ All files pass linting

🧪 Running tests...
✅ All tests passed (28/28)

🏗️  Running build...
✅ Build successful

📊 Quality Check Results:
  ✅ Linting
  ✅ Tests
  ✅ Build

═══════════════════════════════════════
Step 2/2: Running Final Verification
═══════════════════════════════════════

[Code reviewer agent analyzes changes...]

✅ All requirements met
✅ Code quality standards met
✅ Security best practices followed
✅ Performance acceptable

━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
✅ All Verification Complete!
━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

All verifications passed! Ready to finalize.

💡 What's Next?
⭐ Recommended: /ccpm:done AUTH-123

If verification fails:

❌ Quality Checks Failed

📊 Quality Check Results:
  ✅ Linting
  ❌ Tests (3 failures)
  ✅ Build

To debug and fix issues:
  /ccpm:verify AUTH-123

Then run verification again:
  /ccpm:verify AUTH-123

With /ccpm:done (Pre-Flight Verification)

This skill ensures quality before finalization:

# User attempts to finalize task
/ccpm:done

# Or with explicit issue ID
/ccpm:done AUTH-123

Pre-flight checks enforced by ccpm-code-review:

User: "/ccpm:done AUTH-123"

Claude: [ccpm-code-review activates for pre-flight]

✅ All pre-flight checks passed!

Checking completion readiness:

□ Tests passing?
  → ✅ All 52 tests passed

□ Build successful?
  → ✅ Build completed (exit 0)

□ Linear checklist complete?
  → ✅ 7/7 items checked

□ No blockers?
  → ✅ No blockers found

VERIFICATION COMPLETE ✅

📝 Creating GitHub Pull R

...