Skip to content

ci(release): create the release object before asset uploads#20

Merged
yves-vogl merged 1 commit into
mainfrom
ci/fix-release-workflow
Jul 16, 2026
Merged

ci(release): create the release object before asset uploads#20
yves-vogl merged 1 commit into
mainfrom
ci/fix-release-workflow

Conversation

@yves-vogl

Copy link
Copy Markdown
Contributor

Summary

The tag-triggered release workflow was missing the release-creation step: both release-macos and release-windows upload assets via gh release upload, which requires the release object to already exist for the tag — neither job created it, so a fresh tag push failed with "release not found" on both platforms (suite-wide issue discovered tonight).

This copies the fixed pattern already on basilica-audio/crypta main, changing nothing else:

  • New create-release job (ubuntu-latest), idempotent: gh release view "$GITHUB_REF_NAME" || gh release create "$GITHUB_REF_NAME" --title "$GITHUB_REF_NAME" --generate-notes — re-runs after a transient build failure don't error on an already-created release.
  • needs: create-release on both existing build jobs.

Tag-triggered workflows run the workflow file as of the tagged commit, so this must be on main before v0.2.0 is tagged — which is why it's a separate PR after #19 rather than part of it.

Test plan

  • Diff limited to .github/workflows/release.yml (+22 lines, no other changes)
  • Pattern verified against basilica-audio/crypta main (create-release job + needs: on both build jobs at lines 43/142)
  • Real verification happens on the v0.2.0 tag push after this merges

🤖 Generated with Claude Code

Both release jobs upload assets via `gh release upload`, which requires the
release object to already exist for the tag - neither job created it, so a
fresh tag push failed with "release not found" on both platforms. Adds a
create-release job (idempotent: `gh release view` first, so re-runs after a
transient failure don't error on an already-created release) and gates both
build jobs on it via `needs: create-release`, matching the fixed suite-wide
pattern already on basilica-audio/crypta main.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@yves-vogl
yves-vogl merged commit d8ea8ce into main Jul 16, 2026
2 checks passed
@yves-vogl
yves-vogl deleted the ci/fix-release-workflow branch July 16, 2026 03:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant