fix(release): publish release before pushing bump commit - #100
Merged
Conversation
The current release workflow pushes the `bump to vX.Y.Z` commit to main *before* the matching GitHub release is published. During the ~80s build window the release is still a draft, so asset URLs return 404 to unauthenticated downloads — the `bin/devkit` wrapper's first-run engine download then fails and CC caches the MCP startup failure until restart. Reorder so the bump commit lands only after binaries are uploaded and the release is un-drafted. The release tag now points to the PR-merge commit (matters only for auto-bumps — manual bumps already have the new plugin.json in the merge commit). Cleanup-on-failure intentionally excludes bump failures: the release is already published by then, so artifacts must stay; the next merge's auto-bump walks forward from the latest tag and self-recovers if a single bump push fails.
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
Closes the ~80-second race window between version bump and asset upload that breaks devkit MCP for any CC session opened immediately after a merge.
Today's flow:
bump to vX.Y.Z→main(CC users start syncing the new plugin.json)If CC syncs the plugin between steps 1 and 4,
bin/devkitreadsplugin.jsonat the new version, tries to downloaddevkit-darwin-arm64from the draft release, gets 404, and CC caches the MCP startup failure until restart. This is what just bit us on v2.1.30 (commit at 15:06:05Z, release published at 15:07:27Z — 82s exposure).New flow:
bump to vX.Y.Zcommit tomainplugin.jsononmainadvertises the new version only after the matching release is fully live. Wrapper's first-run download is guaranteed to find assets.Notes
Test plan
bin/devkitfirst-run download succeeds on a clean install of the new version.