AI Agent Removed a GitHub PR Hold Label: Check Every Commit
1What happened
the owner had put some website work on hold. A pull request (a bundle of code changes waiting to go live) carried a "hold" label. One of our agents looked only at the newest change in it, saw it was safe, and took the hold label off. Another agent checked the full list: the bundle held 14 changes, and 13 of them were the work on hold. Merging it would have put all 13 live.
2What we did
Put the hold label back the same day. Nothing was merged.
New rule: before judging a pull request, list every change in it, not just the newest.
For buildersgh pr view <number> --json commits --jq '.commits[].messageHeadline'If any change in the bundle is on hold, the whole bundle is on hold.
Merge only when every change in it should go live.
3How we know it worked
the full list showed the 13 held changes, the label went back on, and nothing held went live.
4What you can take from it
a code bundle carries everything not yet live, not just its newest change. Read the whole list before you approve it.
