review-triage-phase

1---2name: review-triage-phase3description: Produces canonical review actions from fetched review state and renders action markdown. Use when the user wants only triage/action-planning for the review-framework workflow.4argument-hint: "[pr-url] [output-dir]"5---6 7# Review Triage Phase8 9Run only the triage phase of the review-framework loop:10 11read current review state artifacts, bootstrap canonical `review-actions.json`, update triage fields, validate, then render `review-actions.md`.12 13Run commands from this skill directory. All script paths below are relative to it.14 15## Inputs16 17- Required:18  - PR URL19- Optional:20  - output directory21 22If output directory is omitted, derive:23 24`wip/reviews/<owner>_<repo>_pr-<number>/`25 26## Preconditions27 28Expected inputs in output dir:29 30- `<output-dir>/review-state.json`31- `<output-dir>/review-targets.json` (optional)32- `<output-dir>/review-state.md` (optional)33- `<output-dir>/summary.txt` (optional)34 35If `review-state.json` is missing, instruct user to run:36 37- `/review-fetch-phase <PR_URL> [output-dir]`38 39Note:40- `review-actions.json` scaffolding is generated from `review-state.targets`, taking the thread and review-body targets. Issue-comment targets are captured in `review-state.json` but get no scaffolded action.41- **Compound review bodies**: A single `pull_request_review` target may contain multiple distinct findings (e.g., CodeRabbit bundles "outside diff range" comments, actionable comments, and nitpicks into one review body). The triager must decompose these into individual action items during triage — never blanket-dismiss a review body without reading its content.42 43## Behavior44 451. Compute deterministic paths:46   - `<output-dir>/review-state.json`47   - `<output-dir>/review-actions.json`48   - `<output-dir>/review-actions.md`492. Enforce artifact safety before generation (the artifacts must stay untracked). In a git checkout the guard asks git directly; in a Jujutsu workspace with no git directory it accepts a directory under the workspace root's ignored `wip/` tree:50 51```bash52node ../review-fetch-phase/scripts/guard-review-artifacts-ignored.mjs --dir <output-dir>53```54 553. Bootstrap canonical action scaffold from review state:56 57```bash58node ./scripts/bootstrap-review-actions.mjs --in <output-dir>/review-state.json --out <output-dir>/review-actions.json59```60 614. Delegate triage to:62  - `./agents/review-triager.md`635. Require triager output contract:64   - `review-actions.json` is valid v265   - targets use node IDs only66   - actions remain intentionally ordered67   - every `defer` action includes a Linear issue identifier68   - will-address actions for `review_thread` targets must remain actionable via **thread replies** (not PR reviews)696. Create Linear tracking for deferred work:70   - create one or more Linear tickets (group related deferred comments)71   - attach each deferred action to a Linear issue ID (`linearIssue`)72   - ensure the deferred thread reply includes that Linear ticket URL737. Validate canonical actions JSON and enforce completed triage decisions:74 75```bash76node ./scripts/validate-review-actions.mjs --in <output-dir>/review-actions.json --require-final77```78 798. Render markdown from canonical actions JSON:80 81```bash82node ./scripts/render-review-actions.mjs --in <output-dir>/review-actions.json --out <output-dir>/review-actions.md83```84 85## Schema contract86 87- `review-actions.json` is canonical and must be schema version `2`.88- No backward compatibility is provided for v1 artifacts.89- The triager edits scaffolded actions in place instead of reconstructing the entire file.90- Triage phase is only complete when there are zero `triage_pending` decisions.91- Review artifacts are generated files and must remain untracked in git.92- `defer` decisions require Linear tracking (`linearIssue`) and a thread reply with that link.93 94## Output to user95 96Return artifact paths:97 98- `review-actions.json`99- `review-actions.md`100 101Suggest next step:102 103- `/review-implement-phase <PR_URL> [output-dir]`104