GitHub PR Finding Triage
Use after github-pr-review-comments has produced a commentIndexJsonl file.
This is classification of reviewer claims, not proof that the claims are valid. Verify code before changing behavior.
Do not downgrade refactoring, cleanup, or maintainability findings merely because they are inconvenient or behavior still works; apply code-review-release/subskills/thermo-nuclear-code-quality-review/SKILL.md before dismissing them.
Command
# Build step (one-time or when tools change):
tsc --project $HOME/.agents/skills/github/tools/tsconfig.json
# Then run:
node $HOME/.agents/skills/github/tools/dist/pr-finding-triage.js --index /tmp/...comments.jsonl --repo OWNER/REPO --pr PR_NUMBER
Outputs:
- JSON summary on stdout
- private Markdown report in
/tmp - private full JSON report in
/tmp
Categories
must_fix: reviewer text claims critical/major/security/failing/broken behavior.verify_first: inline or uncertain findings that need local code verification before action.probably_duplicate: same normalized path/line/severity/body preview as an earlier finding.informational: nitpick/trivial/info/style-only comments. Not for structural maintainability findings unless verified against the thermo-nuclear quality bar.needs_human_decision: API/product/UX/tradeoff or low-confidence comments.
Each finding includes:
derivedSeverityconfidenceclassificationBasisrationale- original GitHub node ID, body, URL, path/line metadata
Recommended workflow
- Run
pr-review-commentsand confirm it is complete (ok: true, no pagination gaps). - Run this triage tool on the JSONL index.
- Start with
must_fix, thenverify_first, thenneeds_human_decision. - Do not skip
probably_duplicatewithout checkingduplicateOfIdand context. - Feed the triage JSON into
pr-review-verificationbefore implementing fixes.
Caveats
The classifier is heuristic. It prioritizes actionability and safety over cleverness. A must_fix means “reviewer appears to claim this is important,” not “the code is definitely wrong.”