-
Notifications
You must be signed in to change notification settings - Fork 2
playbooks repo-sweep: add a "filed" step outcome for findings fixed in another repo #5174
Copy link
Copy link
Closed
Feature
Copy link
Labels
agent-readyFully specified and briefed; eligible for autonomous pickup from the frontier.Fully specified and briefed; eligible for autonomous pickup from the frontier.priority: lowNice-to-have, cosmetic, or speculative; opportunistic.Nice-to-have, cosmetic, or speculative; opportunistic.status: readyTriaged, unblocked, and fully specified; eligible to pick up.Triaged, unblocked, and fully specified; eligible to pick up.work-class: scopedA briefed fix or small feature; blast radius bounded by the brief, tests exist.A briefed fix or small feature; blast radius bounded by the brief, tests exist.
Description
Activity
Metadata
Metadata
Assignees
Labels
agent-readyFully specified and briefed; eligible for autonomous pickup from the frontier.Fully specified and briefed; eligible for autonomous pickup from the frontier.priority: lowNice-to-have, cosmetic, or speculative; opportunistic.Nice-to-have, cosmetic, or speculative; opportunistic.status: readyTriaged, unblocked, and fully specified; eligible to pick up.Triaged, unblocked, and fully specified; eligible to pick up.work-class: scopedA briefed fix or small feature; blast radius bounded by the brief, tests exist.A briefed fix or small feature; blast radius bounded by the brief, tests exist.
Problem
A repo-sweep step can end with a real finding whose fix belongs in another repository.
tick.shhas only two done outcomes,committed <sha>andno-findings, so such a step has to be recorded asno findings, which is false.Observed in the
melodic-software/.githubhygiene sweep (melodic-software/.github#153), stepprogressive-disclosure: the audit found that.claude/rules/pr-body-contract.mdloads every session but matters only at PR time. The file is sync-managed frommelodic-software/standards, so the fix was filed there (melodic-software/standards#641) and nothing changed locally. The checklist now reads:history.shand any reader of past sweep PRs will treat that step as a clean pass, and the upstream issue is linked nowhere in the sweep record.Proposed fix
tick.sh <id> filed <issue-url> <skill@version>...writing- [x] <id>: <skill@version, ...>, filed <issue-url>., filed <url>as a done suffix wherever, committed <sha>/, no findingsare matched (tick.shline 58,state.sh,history.sh).reference/next.mdsection 4: when the agreed fix lives in another repository, file the issue there after user approval and tick withfiled.history.shshould recommendrerunfor afiledstep once the issue closes.Acceptance
state.shtreats that line as done and moves to the next step.