review-triage

Review triage

You are classifying, not judging. Whether the change is correct is the correctness check's question and it is already dispatched. Yours is narrower: which specialist checks does this diff warrant? The usual honest answer is none.

Inputs — three, in this order

1. The check index — review-checks.toml at the commit under review. It declares the checks you may add. Read it first; it is the one place the available checks are declared.
2. The review bead — check_name, anchor_bead, review_branch / review_base or pr_number, and the dispatch-pinned reviewed_oid. The anchor states what the change was for.
3. The diff, at the pinned commit — git diff --stat first, then a skim of the files the stat names. A skim is the method here; reading every hunk is the specialist reviewer's job, not yours.

The index contract

The check index is closed: you may add any check it declares and you may not invent one. Parse it rather than eyeballing it. The parser and the index come from different places and must be resolved separately: the parser is a pack script, the index is the reviewed repo's own. A pack rung on the index ladder would classify this rig against gc-toolkit's checks. The index comes out of the dispatch-pinned commit, $REVIEWED_OID from input 2, not off disk — the tree you stand in is your own worktree, not the commit you are classifying, and a branch may change the index it is judged against:

PARSER=""
for c in "${GC_PACK_DIR:-}" "${GC_RIG_ROOT:-}"; do
  [ -n "$c" ] && [ -x "$c/assets/scripts/review-checks.sh" ] && { PARSER="$c/assets/scripts/review-checks.sh"; break; }
done
INDEX=$(mktemp); FOUND=""
for c in "$(git rev-parse --show-toplevel 2>/dev/null)" "${GC_RIG_ROOT:-}"; do
  [ -n "$c" ] || continue
  git -C "$c" show "${REVIEWED_OID}:review-checks.toml" >"$INDEX" 2>/dev/null && { FOUND=1; break; }
done
[ -n "$FOUND" ] && [ -n "$PARSER" ] && "$PARSER" --file "$INDEX"

Each row gives you the check, its method, one line of purpose, and its phase — the stage transition by which it must read green. When a check applies is a judgment its method states, so read the method; the phase is fixed per check and not triage's to choose, and there is no applies-when column in the index.