load-pr-review

Files (6)

• SKILL.md
• api-contract.md · references/
• token-budget.md · references/
• verdict-triage-prompt.md · references/
• writeback-guardrails.md · references/
• load-pr-review.js · scripts/

SKILL.md (first lines)

# Load PR Review

## Trigger Keywords

`load pr review`, `pr feedback`, `address review`, `review comments`, `pr comments`

## When NOT to Use

| Need | Use Instead |
|------|-------------|
| Create a code review | `/codex-review-fast` or `/codex-review` |
| Post new review comments | `/pr-comment` |
| Create a PR | `/create-pr` |
| PR status overview | `/pr-summary` |
| Investigate code history | `/git-investigate` |

## Core Principle

```
Load review → independent per-thread triage → analysis report → user decides next step
Default: analysis-only. Fix and writeback require explicit --mode fix / --writeback.
Data plane (JS script) handles fetch/normalize/writeback.
Control plane (this SKILL.md) handles classification, fix orchestration, auto-loop.
```

## Non-Negotiable Rules

| # | Rule | Violation = |
|---|------|-------------|
| 1 | Plan/fix mode MUST invoke `/seek-verdict` via Skill tool per unresolved thread (unless `--no-verdict`) | Report invalid — re-run with verdict |
| 2 | Step 3 (Present) is blocked until Step 2 (Verdict Triage) completes | Report invalid |
| 3 | Each `/seek-verdict` must use fresh Codex context per thread (no batch) | Triage invalid |
| 4 | Plan output MUST include Codex Verdict Threads field (non-empty, from Step 2) | Report rejected |

## Analysis-Only Default ⚠️

This skill is an **analysis tool by default**. It loads PR review comments and produces a triage report. It does NOT auto-fix.

### Prohibited Behaviors

| ❌ Prohibited | ✅ Correct |
|--------------|-----------|
| Auto-fixing code after loading PR reviews | Present analysis report, wait for user to invoke `--mode fix` |
| Editing files in plan mode | Only read and classify; no writes |
| Suggesting "let me fix this" without explicit `--mode fix` | "Use `--mode fix` to start fixing ACTIONABLE threads." |
| Skipping triage and jumping to fixes | Always complete Step 2 verdict triage before any action |
| Skipping `/seek-verdict` and classifying threads with AI judgment alone | Invoke `/seek-verdict` per thread via Skill tool; AI judgment is fallback only for failed calls |
| Outputting plan table without Codex verdict data when `--no-verdict` was not passed | Complete Step 2 before Step 3 |

### Precedence

> **Rule priority**: Plan mode's analysis-only constraint overrides the "Skill analysis-only mode" exception in `@skills/codex-code-review/references/scope-contract.md` § Fix Obligation.
> Issues found in plan mode are recorded in the analysis report (logged as `[ANALYSIS_ONLY_DEFERRED]`), not auto-fixed. User must explicitly invoke `--mode fix` to apply changes.

### Mode Behavior

| Mode | Default? | Reads Code | Edits Code | Writes Back |
|------|----------|------------|------------|-------------|
| `plan` | **Yes** | ✅ | ❌ | ❌ |
| `summary` | No | ❌ | ❌ | ❌ |
| `fix` | No (explicit) | ✅ | ✅ (after AskUserQuestion) | Only with `--writeback` |

## Workflow

```mermaid
sequenceDiagram
    participant U as User
    participant SK as SKILL.md
    participant JS as load-pr-review.js
    participant GH as GitHub (gh CLI)
    participant SV as /seek-verdict (per thread)
    participant AL as Auto-Loop

    U->>SK: /load-pr-review [args]
    SK->>SK: Step 0 — Resolve PR target

    SK->>JS: fetch --pr N --repo owner/repo
    JS->>GH: gh api graphql (reviewThreads)
    alt GraphQL fails
        JS->>GH: gh api REST (fallback)
    end
    JS-->>SK: Normalized JSON

    alt plan/fix mode + verdict enabled
        loop Each unresolved thread (parallel)
            SK->>SV: /seek-verdict (fresh Codex per thread)
            Note over SV: Independent research
            SV-->>SK: Per-thread verdict
        end
    end

    alt summary mode
        SK->>U: Table of threads
    end

    alt plan mode (DEFAULT)
        SK->>SK: Map verdicts to categories
        SK->>U: Analysis report (no edits)
    end

    alt fix mode (explicit --mode fix only)
        SK->>U: AskUserQuestion — select threads
        loop Each selected thread
            SK->>SK: Read file + apply fix
            SK->>AL: auto-loop
        end
    end

    alt --writeback
        SK->>JS: writeback --plan
        JS-->>SK: Dry-run plan
        SK->>U: AskUserQuestion — approve
        U->>SK: Approved
        loop Each thread
            SK->>JS: writeback --execute (one at a time)
            JS->>GH: POST reply + resolve
        end
    end
```

From sd0xdev/sd0x-harness, MIT.

More from sd0xdev/sd0x-harness