-
Notifications
You must be signed in to change notification settings - Fork 2
playbooks: repo-sweep next does not detect a sweep PR that CI has stopped testing #5206
Copy link
Copy link
Closed
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: mediumReal value, no hard deadline; normal backlog flow.Real value, no hard deadline; normal backlog flow.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: mediumReal value, no hard deadline; normal backlog flow.Real value, no hard deadline; normal backlog flow.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.
/playbooks:repo-sweep nextpushes one commit per step but never checks that CI ran on it. In melodic-software/.github#153,mainmoved during the sweep (#158 re-pinned.cursor/install.sh), and the PR becamemergeable: CONFLICTING. GitHub does not runpull_requestworkflows on a conflicting PR, so the last six step commits (5ebec0e..a0b83b9) got no CI run.state.shand every step reported normally; the gap surfaced only during a manual review after the finalverifystep.Fix: have
state.shreadgh pr view --json mergeableand reportmergeable CONFLICTINGas a warning line (or a new exit code). Havenextsection 1 stop on it and tell the user to merge the base before running the next step. That way each step's commit is tested by CI before the next step builds on it.Evidence:
gh run list --branch chore/repo-sweep-hygiene-20260926shows the newest run on head7c794ea, while the PR head isa0b83b9.git merge-treereports a conflict in.cursor/install.sh.🤖 Generated with Claude Code
https://claude.ai/code/session_01S5b57FoPJkJMe1BDwhExiG