DevFlow: Pull Request Review Flow

Perform complete technical review of Pull Requests, including branch checkout, technical validation, error correction, and quality approval.

When to Use

Use this flow when you need to:

•Review a pull request technically
•Validate PR quality before merge
•Check out and test PR changes
•Approve or request changes on PR

Flow Diagram

mermaid

flowchart TD
    A([BEGIN]) --> B[Fetch PR info from GitHub]
    B --> C[Identify linked issue]
    C --> D[Analyze modified files]
    D --> E[Checkout PR branch]
    E --> F[Install dependencies if needed]
    F --> G[Run technical validations]
    G --> H{Validation passed?}
    H -->|No| I{Auto-fix enabled?}
    I -->|Yes| J[Auto-correct issues]
    J --> G
    I -->|No| K[Report issues to user]
    K --> L[User decides action]
    H -->|Yes| L
    L --> M{Functional tests configured?}
    M -->|Yes| N[Start local server]
    N --> O[Run functional tests]
    O --> P[Capture evidence]
    M -->|No| Q[Skip functional tests]
    P --> R[Consolidate results]
    Q --> R
    R --> S[Classify findings by severity]
    S --> T{Auto-approve & no critical issues?}
    T -->|Yes| U[Auto-approve PR]
    T -->|No| V[Present review to user]
    V --> W{User decision?}
    W -->|Approve| X[Approve PR]
    W -->|Comment| Y[Add review comments]
    W -->|Request changes| Z[List required changes]
    U --> AA([END])
    X --> AA
    Y --> AA
    Z --> AA

Node Details

1. PR Analysis

•Fetch PR details from GitHub
•Identify linked issue for context
•Analyze changed files and impact

2. Environment Setup

•Checkout PR branch (handle forks)
•Install/update dependencies
•Ensure clean test environment

3. Technical Validation

Run complete validation suite:

•Tests
•Linting
•Type checking
•Build

4. Auto-Fix (Optional)

If --fix-issues flag and validation fails:

•Attempt automatic corrections
•Re-run validations
•Report what was fixed

5. Functional Testing (Optional)

If configured in .claude/details/commands/review-pr.md:

•Start local server
•Run automated functional tests
•Capture screenshots/logs as evidence

6. Review Decision

Based on findings, either:

•Auto-approve (if enabled and clean)
•Present findings to user for decision
•Provide approve/comment/request-changes options

Parameters

•<pr-number>: Required - PR to review
•--fix-issues: Automatically correct found problems
•--auto-approve: Auto-approve if no critical issues
•--functional-tests: Enable functional testing

Example Usage

code

/flow:devflow-review-pr 123
/flow:devflow-review-pr 456 --fix-issues
/flow:devflow-review-pr 789 --auto-approve

Output

Complete review report:

code

📋 COMPLETE REVIEW - PR #123

🔗 Implemented issue: #456 ✅
🔧 Technical validations: ✅ 4/4 passing
📊 Coverage: 94% (+2% vs baseline)
🧪 Functional tests: ✅ All passed

✅ Ready for merge