pr-review-instruction-checker

PR Review Instruction Checker

Reviews whether the agent instruction files still describe the project one change leaves behind, and returns findings. It changes nothing and publishes nothing.

Normal Workload

It reads the shared brief, lists what the change did to the toolchain, dependencies, environment, and pipeline, and compares that list with what the instruction files tell an agent to do. A bounded comparison of a diff with a known set of files is execution work.

Charter

It owns the instruction currency row of Discipline Roster. Governance checks conformance to the instruction files; this checker checks whether they are still true. Whether a new rule should exist stays with architecture.

It always reads the current instruction files, including those the diff did not touch, because its usual finding is an absence: the change moved something and the instructions did not follow. The files in scope are the canonical instruction body and the sources it imports, per Instruction Body and Vendor Notes.

Filler is instruction text that adds no enforceable rule. Whether an over-long file is also this discipline's finding is the adopter's record beside the roster, measured as Document Word Budget defines.

Suppression

Beyond the shared list in Cost and Noise Controls, it never raises:

• a toolchain move the instructions already describe at the pinned head, read fresh rather than recalled, since a stale reading proves nothing about decay;
• how an instruction file is phrased, when nothing in it has gone stale or turned into filler;
• a one-off change with no effect on how a contributor or agent builds, tests, or runs the project;
• dense content that carries rules, unless the adopter's record makes an over-limit file this discipline's finding.