[jasperboerhof]

Ports the 2026-09-12 changes from the personal /shepard-pr into the catalog copy, generalised. Follows #45.

What changed

Reference files, two kinds. references/repos/<repo>.md holds the repo (the template moved to repos/_template.md, with a Reviewer section and an issue-or-report line on Board). references/reviewers/crit.md holds crit's contract once, read from crit's own code: how to recognise it, the finder trace (crit #282), own-thread resolving, the waiver and settle paths, reply wording and timing, open threads, mechanics. Before, the reviewer contract and a repo's facts sat in one file named after a repo.

The finder trace. Step 2b prints the full root body of every open thread; the trace sits there. Step 3 reads every site the reviewer named as one finding about one mechanism, then gives the mechanism one disposition. Who decides the fix is unchanged: one shape is a FIX, two shapes a DESIGN CALL.

Nitpicks in every round. Inside the diff: fix in the same push. Outside the diff: file or note, never fix. After an approval only a real defect inside the diff pushes, because a push then buys a round.

Issue or report is the repo's call. The repo file says which; with no line, ask the developer once per PR. The reply, not the key, takes a finding off the PR.

Reply before the push, then push once. crit reads the threads seconds after a push (crit#211).

Step 8 is reply only. crit resolves its own threads; the resolve mutation, its four bars and the two-camp table are gone.

Stop rules removed. The same-seam and score-falling signals, the second-site rule and the never-list. The 2026-09-11 patching-chain audit replayed them on 144 rounds: 7 to 30 saved, about half the firings false, and the old process had already stopped at the same rounds. The counters stay as a budget.

Trims per the Claude 5 prompting guidance. Description to what it does and when to trigger; the over-verification lines and capitalised emphasis go; the watch line table moves to references/watch.md.

Scripts. ci-failures.sh gains the newest-run-per-workflow read, the unreadable job list, timed_out, --run validation. pr-watch.sh validates its flags and reads crit's issue/nitpick counts. Both suites pass (61 and 24), ShellCheck clean. ci-failures.sh is byte-identical to emmie's copy again.

Added 2026-09-13, from the consumers

Two fixes lokalekeuze had made to pr-watch.sh since its import, both from crit reviews there, now live here: the town-crier bearer token travels through a curl -K config file instead of argv (PR #209), and a check that finished SKIPPED, NEUTRAL or STALE is its own ATTN state with its own line, counted on workflow lanes only, because an app's tracker card is NEUTRAL forever (PR #212, LK-0325 follow-up). Suite now 30.

Three generalisations so a consumer copy needs no body edit: the scripts are named by <skill dir>, step 0 says what happens when a repo ships its own copy, and "replace the construct" hands off to the repo's planning skill instead of naming two skills a consumer may not have. The template's reviewer example no longer claims crit submits COMMENTED: on the last five merged PRs of emmie, kendo and lokalekeuze crit-ai submitted APPROVED and the PR read APPROVED.

Consumers

Propagated on the same day, each with its own repo file under references/repos/: emmie /shepard, kendo /drive-pr (script-development/kendo#2209), lokalekeuze /shepard. Merge this first; the consumer PRs pin 628b6e8.

emmie /shepard, kendo /drive-pr, lokalekeuze /shepard. lokalekeuze's references/crit.md becomes references/reviewers/crit.md plus a repos/lokalekeuze.md.

🤖 Generated with Claude Code

https://claude.ai/code/session_01VTHk249QRKjpw3ZnsBjEBB