fix(ci): regenerate stale suite-map coverage snapshot - #815
kylerankin wants to merge 1 commit into
Conversation
|
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
The committed coverage snapshot in suite-map/SKILL.md was stale: the CI 'Coverage snapshot fresh' gate (--check) failed because 2 common-suite scenarios moved from active to @pending since the last regeneration. Regenerate the generated table with scripts/update_coverage_snapshot.py (101→99 active, 20→22 pending in common; totals 526/415→526/413). Unit tests for the generator pass (44). No feature-file or hand-maintained Notes changed — only the generated counts. Signed-off-by: kylerankin <kylerankin@users.noreply.github.com>
9370895 to
452c3dc
Compare
castrojo
left a comment
There was a problem hiding this comment.
LGTM. Generated coverage snapshot matches test suite tag counts; all checks pass.
castrojo
left a comment
There was a problem hiding this comment.
LGTM, coverage snapshot is fresh and all checks pass.
|
This is byte-identical to #807 and #824 — all three resolve to tree The content is correct. It matches freshly-generated output from It's purely a question of which copy lands. I'd suggest #807, only because it's the oldest (2026-09-09) — but this one's commit message is arguably better ( Not closing it myself since it's yours to decide — but flagging so it doesn't sit waiting on a review that duplicate work has already made moot. Note that once any one of the three merges, the other two become empty diffs rather than conflicts, so they won't fail loudly; they'll need closing explicitly. No defect implied. Generated by Claude Code |
castrojo
left a comment
There was a problem hiding this comment.
LGTM: regenerate stale suite-map coverage snapshot
Pull request was closed
Problem
PR validation and testsuite main CI fail on the coverage-snapshot gate:
scripts/update_coverage_snapshot.py --check reports the committed
suite-map coverage snapshot in docs/skills/test-authoring/suite-map/SKILL.md
is stale.
Fix
Regenerated the coverage snapshot table from the tags on every scenario in
tests//features/**/.feature. The common suite gained 2 more @pending
scenarios (101 active/20 pending to 99 active/22 pending), which is what
made the committed table out of date. Numbers are computed by the script;
only the hand-maintained Notes column is touched elsewhere.
Verification
— hive: backend=pi model=lemonade/Ornith-1.5-35B-A3B-GGUF-Q6_K
🐝 Hive Agent:
contributor| SHA:9370895a