review-pr

Fetch a pull request diff, run code-reviewer analysis, translate findings to owner voice, and post a single batched inline GitHub review.

Usage

/review-pr <PR#> [--tier standard|senior|chief] [--task T-NNN]

Arguments:

• <PR#> — pull request number (required)
• --tier — review depth: standard (default), senior, or chief
• --task — optional JSONL task ID to mark completed after the review posts (e.g. --task T-047)

---

Step 1 — Check prerequisites

Before anything else:

1. Read .rea/policy.yaml — confirm autonomy level is L1 or higher
2. Check for .rea/HALT — if present, stop immediately
3. Verify gh CLI is available: gh --version
4. Confirm caller is in a git repo with a GitHub remote: gh repo view --json nameWithOwner

Extract owner/repo for use in API calls.

---

Step 2 — Fetch PR context

Run these in parallel:

gh pr view <PR#> --json title,body,headRefName,baseRefName,state,commits
gh pr diff <PR#>
gh pr view <PR#> --json commits --jq '.commits[-1].oid'

Store:

• COMMIT_SHA — the latest commit OID (use in the review payload)
• PR_TITLE — for context
• PR_DIFF — the full diff text (pass to code-reviewer)

If the PR is merged or closed, warn the user: "PR #N is already closed — review will post but won't block a merge."

If gh pr diff returns an empty diff, stop and report: "Empty diff — nothing to review."

---

Step 3 — Run code-reviewer analysis

Invoke the code-reviewer agent with the diff and the specified tier.

Pass this context to code-reviewer:

Tier: <standard|senior|chief>

Review the following git diff and produce structured findings as a JSON array.
Each finding must include:
  - file: string (file path from diff header)
  - line: number (line number in the NEW file)
  - start_line?: number (for multi-line spans, the start line)
  - severity: "high" | "medium" | "low"
  - issue: string (the specific problem — no hedging)
  - suggestion_code?: string (the corrected code, if applicable)

Output ONLY the JSON array. No prose. No markdown wrapper. Example:
[
  {
    "file": "src/gateway/middleware/chain.ts",
    "line": 42,
    "severity": "high",
    "issue": "Non-null assertion will throw if upstream returns undefined",
    "suggestion_code": "const result = upstream ?? defaultValue;"
  }
]

If there are no findings, output: []

DIFF:
<paste full diff here>