cleanup: five citations a reader cannot follow, and both of the repo's line numbers were wrong (#462)
Second PR of the cleanup pass tracked at #460, one class: a pointer that does not land
where it says. Nothing executable changes.
docs/models.md:464 said a bad `page` extraction "moves five counters a deployment
already records" and named pages_verify_failed, pages_unreadable, reextracts,
corrections_refused and model_mismatch. Iris records none of the five — all five are the
BENCH harness's own field names, where model_mismatch is derived from models_called — so
the load-bearing half of the argument for swapping `page` and not `reader` pointed an
operator at nothing they could look up. Replaced with five a deployment does record and
that a bad extraction moves: the run log's `page_verify_failed`, and /v1/quality's
`structural_defect_rate`, `markup_unbalanced_rate`, `table_no_body_rate` and
`unresolved_rate`.
docs/API.md:451 pointed at `agents/copy-editor.md`, which does not exist: the Copy
Editor's prompt is `EDITOR_SYSTEM` in src/pipeline/review.ts.
The repo cited a `path:line` in exactly two places and both were wrong, which is the
argument for citing by name: `agents/page.md:131` was sent as the rule for deciding a
hyphen and is the illegible-marks rule 90 lines above it, and
`src/pipeline/anchors.ts:611-615` was sent for a claim about `namespaceAnchors` and sits
inside `idAudit`. Both now quote the rule they mean.
docs/verifier-calibration.md:385's "26 places" was never derivable — 44 lines name the
field today, 32 did when the sentence was written — and its likely source is the "26
replies" two lines below. Now 44, with the grep that produces it.
Two rounds. Round 1 found the `EDITOR_SYSTEM` replacement unglossed on its first use in
that document, and fixing it turned up a second lesson: the control for a capitalization
claim is the shorter string. Grepping "Copy Editor Agent" found none and 7 lowercase
"copy editor", so lowercase looked like the file's idiom; "Copy Editor" appears 23 times.
Round 1 also caught "counters" standing over a list that is one event and four rates —
and that repair broke the parallel sentence beside it, #461's shape inside #462.
Left alone, with the reason: docs/models.md's `src/report.mjs` and `src/pricing.mjs` are
bench-relative and correct, and test/config-agents.test.ts:32's `src/pipeline/triage.ts`
is past tense about a deleted file.
Gates on every commit: tsc clean, `npm test` 1694/1694, `./test/e2e.sh` exit 0 with
150 ✓ and 0 ✗.
Refs #460