Skip to content

after_release: fix dead release trigger (winget PR never auto-fired)#66

Merged
packetThrower merged 1 commit into
mainfrom
fix-after-release-trigger
Jul 17, 2026
Merged

after_release: fix dead release trigger (winget PR never auto-fired)#66
packetThrower merged 1 commit into
mainfrom
fix-after-release-trigger

Conversation

@packetThrower

Copy link
Copy Markdown
Owner

Summary

The winget submission never fired automatically: release.yml publishes the Release with the default GITHUB_TOKEN, and GitHub suppresses events created by that token from triggering other workflows (recursion guard). The release: published trigger in after_release.yml was dead code — 4.1.3 and 4.1.4 were both submitted via manual workflow_dispatch.

Fix: trigger on workflow_run (Release, completed) instead — exempt from the suppression. Tag comes from workflow_run.head_branch (the tag name on tag-push runs). Job guard now requires: successful Release conclusion, v-prefixed head_branch (filters out release.yml's own workflow_dispatch runs, which carry a branch name), no hyphen (stable only, as before).

Manual workflow_dispatch path is unchanged as the fallback.

Test plan

  • Merge, then verify on the next stable tag that "After Release" fires without a manual dispatch
  • v4.1.4 winget submission already done via manual dispatch (run 29439023228, success)

🤖 Generated with Claude Code

release.yml publishes the GitHub Release with the default
GITHUB_TOKEN, and GitHub suppresses events created by that token
from triggering other workflows (recursion guard). The
`release: published` trigger here was dead code — every winget
submission to date has been a manual workflow_dispatch.

Switch to `workflow_run` on Release completion, which is exempt
from the suppression. The tag arrives as workflow_run.head_branch
on tag-push runs; the job guard now also requires a successful
conclusion and a `v`-prefixed, hyphen-free (stable) tag.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@packetThrower
packetThrower merged commit c3aa646 into main Jul 17, 2026
4 of 5 checks passed
packetThrower added a commit that referenced this pull request Jul 17, 2026
The 'future' section described a plan superseded by after_release.yml
(and its #66 trigger fix, validated on Zorite v0.9.0).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
packetThrower added a commit that referenced this pull request Jul 17, 2026
The 'future' section described a plan superseded by after_release.yml
(and its #66 trigger fix, validated on Zorite v0.9.0).

Co-Authored-By: Claude Fable 5 <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.

1 participant