evolve-coverage-delta.sh's grep accepted any "coverage: N" prefix and
silently consumed Go's event-coverage informational line — emitted by
packages with no _test.go files when the run uses -coverpkg, e.g. cmd/ao
prints `coverage: 1/12 events (informational)`. The "1" was extracted as
1.0% and folded into the project average, dragging it down by tens of
points.
Tighten the regex to `coverage: [0-9]+(\.[0-9]+)?%` so only true
percent-format lines participate in the average.
Verified by isolated parser test (./tests/scripts/evolve-coverage-delta.bats):
Input combining 1/12 events + 76.5% + 88.2% lines now averages to 82.3,
matching what an operator expects (the event line is excluded entirely).
Closes the post-mortem next-work finding "Fix coverage-ratchet.sh cmd/ao
event format handling" (the report named the wrong script — the bug was
in evolve-coverage-delta.sh's averaging step).
- Extract parseFrontmatterBlock/extractPatternNameAndDescription (inject_patterns)
- Extract runCommitLanding/runSyncPushLanding (supervisor landing)
- Extract recordEventOrWarn/bulkApproveEntry (pool)
- Replace sort.Slice with slices.SortFunc across cmd/ao and pool
- Add slow test package duration warning to CI validate workflow
- Add evolve-coverage-delta.sh, evolve-file-complexity.sh, pre-commit-gate.sh
- Modernize test helpers with t.Helper() and slices patterns
Claude