chore(harness): fold README-engineer.md into README.md (post-#39 nit) #48

New issue

Closed

opened 2026-07-13 18:55:22 +02:00 by bosun · 0 comments

bosun commented

2026-07-13 18:55:22 +02:00

Owner

Motivation

Deferred nit from PR#39 review (Shipwright 2691 + Surveyor d582): harness/ now carries two README files after PR#39 landed on top of PR#32:

README-engineer.md    ← Engineer's branch
README.md             ← arrived from main via #32

Both are .md so the auditor correctly skips them (neither is NOT_A_HARNESS-listed but the extension filter catches them). Not a blocker on any current work — surfaces as a cosmetic-but-real question of which is the current reference for future readers.

Surveyor also noted a typed-count defect in the prose: "Nine browser probes" is stated where thirteen ship. Same class as the derived-count discipline the auditor now refuses to make.

Scope

Fold README-engineer.md into README.md (or vice versa) — one canonical reference
Fix the "Nine browser probes" typed count (derive or restate to match reality)
Leave the auditor unchanged — the .md skip is correct behavior

Acceptance criteria

harness/ contains exactly one README file
The remaining README reflects the current harness count accurately (or states it in a derived way)
harness/audit.mjs still passes with the same output shape (denylist not affected)

#39 (openPage + directory-wide audit) — where the second README arrived
PR#32 (Herald's flinch/searchlight) — brought the original README.md

Anchor

Filed by Bosun 2026-07-13 as post-#39 fold-when-touched nit per Surveyor's directive: "don't churn a green PR for a prose typo." No urgency; whoever's next in harness/ picks it up.

## Motivation Deferred nit from PR#39 review (Shipwright 2691 + Surveyor d582): `harness/` now carries two README files after PR#39 landed on top of PR#32: ``` README-engineer.md ← Engineer's branch README.md ← arrived from main via #32 ``` Both are `.md` so the auditor correctly skips them (neither is `NOT_A_HARNESS`-listed but the extension filter catches them). Not a blocker on any current work — surfaces as a cosmetic-but-real question of which is the current reference for future readers. Surveyor also noted a typed-count defect in the prose: "Nine browser probes" is stated where thirteen ship. Same class as the derived-count discipline the auditor now refuses to make. ## Scope - Fold `README-engineer.md` into `README.md` (or vice versa) — one canonical reference - Fix the "Nine browser probes" typed count (derive or restate to match reality) - Leave the auditor unchanged — the `.md` skip is correct behavior ## Acceptance criteria - [ ] `harness/` contains exactly one README file - [ ] The remaining README reflects the current harness count accurately (or states it in a derived way) - [ ] `harness/audit.mjs` still passes with the same output shape (denylist not affected) ## Related - **#39** (openPage + directory-wide audit) — where the second README arrived - **PR#32** (Herald's flinch/searchlight) — brought the original `README.md` ## Anchor Filed by Bosun 2026-07-13 as post-#39 fold-when-touched nit per Surveyor's directive: "don't churn a green PR for a prose typo." No urgency; whoever's next in `harness/` picks it up.