Skip to content

ci: skip Auto Release when triggered by release App push#53

Merged
aliasunder merged 1 commit into
mainfrom
claude/bootstrap-vault-setup-UP2Ya
May 19, 2026
Merged

ci: skip Auto Release when triggered by release App push#53
aliasunder merged 1 commit into
mainfrom
claude/bootstrap-vault-setup-UP2Ya

Conversation

@aliasunder
Copy link
Copy Markdown
Owner

Summary

  • Adds if: github.actor != 'vault-cortex-release[bot]' to the validate job in auto_release.yml.
  • deploy and release jobs depend on validate via needs:, so they inherit the skip — no need to duplicate the guard.

Why

App installation tokens (unlike GITHUB_TOKEN) do not suppress workflow cascades on push events. After #52, the tag push from Manual Release's bump-and-tag job now triggers auto_release.yml in parallel — observed during the v0.15.2 release (Auto Release fired and was cancelled by the deploy-prod concurrency group).

Manual Release already inlines deploy + release, so the cascaded Auto Release run is fully redundant. Guarding on actor keeps Auto Release useful for the original intent (a future external git push --tags from a developer machine) without firing on every bot release.

Test plan

  • CI green on this PR.
  • Next Manual Release run: confirm only one workflow chain runs end-to-end (no Auto Release sibling).
  • (Optional) Push a tag manually from local: confirm Auto Release does fire in that case.

https://claude.ai/code/session_01K8eXF8Fe3gjCR5e2ndU2Wo


Generated by Claude Code

App installation tokens (unlike GITHUB_TOKEN) do not suppress workflow
cascades, so the tag push from Manual Release's bump-and-tag job now
triggers Auto Release in parallel. Manual Release already inlines deploy
+ release, making the parallel Auto Release run redundant.

Guard the validate job on actor; deploy and release inherit the skip via
needs cascade. Auto Release still fires for any other tag push (e.g.
manual git push --tags from a developer machine).

https://claude.ai/code/session_01K8eXF8Fe3gjCR5e2ndU2Wo
@aliasunder aliasunder merged commit e906e41 into main May 19, 2026
2 checks passed
@aliasunder aliasunder deleted the claude/bootstrap-vault-setup-UP2Ya branch May 19, 2026 14:07
aliasunder added a commit that referenced this pull request May 20, 2026
Every manual release (App-bot tag push) triggered Auto Release, which
failed with "Invalid workflow file" — it calls deploy.yml (needs
id-token: write + packages: write) but only granted contents: write.
GitHub rejects the file at validation time, BEFORE the actor guard
(`if: github.actor != 'vault-cortex-release[bot]'`, PR #53) can skip
the jobs — so it failed loudly on v0.15.2–v0.15.5 instead of skipping.

Added id-token: write + packages: write so the file validates. Now a
bot-pushed tag validates, the actor guard skips validate → deploy →
release, and the run completes as a clean skip (no red X). A genuine
non-bot tag push (local `git push --tags`) now also deploys as intended.

Co-authored-by: Claude <noreply@anthropic.com>
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.

2 participants