-
Notifications
You must be signed in to change notification settings - Fork 0
ci: guard against version drift between main and the published skill #19
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
27 commits
Select commit
Hold shift + click to select a range
9d0ffb1
ci: guard against version drift between main and the published skill
BenKalsky 7c8358e
ci: dispatch the ClawHub publish explicitly (Codex round-1 P1)
BenKalsky 04595bd
ci: roll back the release when the ClawHub publish fails (Codex round…
BenKalsky 0c749a7
ci: pin the release and its publish to the released commit; roll back…
BenKalsky 5a53fc0
ci: close the remaining release-existence loopholes (Codex round-4 P1…
BenKalsky 2be563e
ci: abort pre-flight on lookup errors; roll back canceled UI publishe…
BenKalsky ecc50e2
ci: block freestanding real publishes from manual dispatch (Codex rou…
BenKalsky ab8550c
ci: replace release-existence proof with a publish-completion marker …
BenKalsky decbc79
ci: never delete a release once the registry has the version; seriali…
BenKalsky a205e2e
ci: re-run the failed main guard after the release flip (Codex round-…
BenKalsky dc2b13e
ci: fail closed on every gh run listing (Codex round-10 P1 + P2, plus…
BenKalsky 1f44772
ci: decide rollback from the child's Publish step outcome; re-run mai…
BenKalsky 5414d31
ci: fail the child on exhausted marker attempts, correlate the dispat…
BenKalsky 0bc7bcb
ci: UI rollback only on a verified publish failure (Codex round-13 P1)
BenKalsky 4de4997
ci: verified_failure is the only deletion evidence; interrogate befor…
BenKalsky dfd633b
ci: ambiguity is sticky across every attempt (Codex round-15 P1)
BenKalsky 186d1c1
ci: no deletion on a no-show dispatch; UI tag must match the version;…
BenKalsky 9f7d3df
ci: lineage-gate UI releases; clean verifiably-untouched failures (Co…
BenKalsky a387015
ci: published releases delete only on deterministic proof — the two-g…
BenKalsky 70cab35
ci: address the Copilot review — close the draft-sweep two-generals h…
BenKalsky 4840ea0
ci: rerun after rollback finds no release; prior publish failure bloc…
BenKalsky 34ea214
ci: preserve on lost dispatch ack; aggregate every real run before sw…
BenKalsky 4f959c9
ci: default branch everywhere, not hardcoded main (Copilot x3)
BenKalsky f3006f3
ci: outstanding-dispatch breadcrumb closes the hidden-child sweep rac…
BenKalsky 12488fd
ci: breadcrumb resolves by correlation, never by clock; dry_run expre…
BenKalsky c32329c
ci: fetch the run listing before the breadcrumb consults it (Codex P1…
BenKalsky 5f1756f
ci: badge version extraction is color-agnostic (Copilot)
BenKalsky File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After every version-bump merge, the
pushrun evaluates before the release exists and fails as intended, but publishing the release does not emit either of these configured events. The new release workflow only flips the draft in.github/workflows/release.yml:230and never reruns or dispatches CI, so the failed check on the main commit remains red even after ClawHub and the marker are synchronized; explicitly invoke the guard after the flip or add an event path that the token-driven release flow can trigger.Useful? React with 👍 / 👎.