You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
playbooks:repo-sweep 0.13.0 records a step as no findings whenever the step changed no file
outside .work/ (reference/next.md section 4, item 3). It never checks what the step's skill
actually covered. In the melodic-software/.github hygiene sweep (PR #153), the dead-code step
ran code-tidying:audit-dead-code. That scan covered 3 of the repo's 5 source files: the 2
standalone .mjs files fell under no lane (#4521). The checklist still reads - [x] dead-code: code-tidying:audit-dead-code@0.23.2, no findings, which a later sweep's history.sh reads as evidence the step ran clean.
Fix
Split the outcome in tick.sh and the checklist format: no findings only when the skill
reports full coverage; no findings, partial coverage: <what was not covered> otherwise.
Summary
playbooks:repo-sweep0.13.0 records a step asno findingswhenever the step changed no fileoutside
.work/(reference/next.mdsection 4, item 3). It never checks what the step's skillactually covered. In the
melodic-software/.githubhygiene sweep (PR #153), thedead-codestepran
code-tidying:audit-dead-code. That scan covered 3 of the repo's 5 source files: the 2standalone
.mjsfiles fell under no lane (#4521). The checklist still reads- [x] dead-code: code-tidying:audit-dead-code@0.23.2, no findings, which a later sweep'shistory.shreads as evidence the step ran clean.Fix
tick.shand the checklist format:no findingsonly when the skillreports full coverage;
no findings, partial coverage: <what was not covered>otherwise.next.mdsection 3, before ticking, have the step read the skill's own coverage statement,for example audit-dead-code's
Lane:lines andSummary coverage:once audit-dead-code: account for every source file; a report is clean only when none is uncovered #4521 ships. Show anyuncovered scope to the user alongside the findings.
history.shtreat a partial-coverage tick asrerun, not as a clean prior run.Verification
the next
planrecommendsrerunfor it.Related
audit-dead-code: detect unreferenced files in any language #4523, audit-dead-code: when coverage is incomplete, offer to file an issue or research and install a detector #4524, audit-dead-code: docs overstate grep-lane and orphaned-file coverage #4525.
melodic-software/.github, PR chore(skill-quality): harden static checker + record terminal retrofit scope #153.