Skip to content

fix(release): use PAT for the github release so notify.yaml fires - #9

Merged
Synertry merged 1 commit into
mainfrom
fix/release-cascade-notify
Jun 9, 2026
Merged

fix(release): use PAT for the github release so notify.yaml fires#9
Synertry merged 1 commit into
mainfrom
fix/release-cascade-notify

Conversation

@Synertry

@Synertry Synertry commented Jun 9, 2026

Copy link
Copy Markdown
Owner

Summary

The previous v0.0.1 and v0.0.2 releases published cleanly but `notify.yaml` never ran. The repo has zero `release`-event workflow runs in its history (`gh api 'actions/runs?event=release'` returns `total_count: 0`).

Why

GitHub Actions has a documented loop-protection guardrail: events triggered by `GITHUB_TOKEN` do not cascade into other workflows. The `Create GitHub Release` step was using the default `GITHUB_TOKEN`, so the `release.published` event got suppressed downstream. notify.yaml's trigger filter (`release.published` only) saw nothing to react to.

Side effect of the suppression: GitHub emits placeholder failure runs on each push for any workflow file whose trigger doesn't match the current event, which is why `gh run list --workflow=notify.yaml` showed a string of "failed" runs all with `event: push` and zero jobs. Those weren't real failures, just GitHub's way of surfacing that nothing fired.

Changes

  • `release.yaml`: pass `token: ${{ secrets.PAT }}` to `softprops/action-gh-release`. The PAT is already configured for the `integrate` and `review-approval` workflows so no new secret is needed.
  • Inline comment explains the guardrail so the next person reading the file knows why `GITHUB_TOKEN` is the wrong default here.

The Create GitHub Release step has been using the default GITHUB_TOKEN.
That hit a documented GitHub Actions guardrail: events triggered by
GITHUB_TOKEN do not cascade into downstream workflows (loop protection).
Result: v0.0.1 and v0.0.2 both published cleanly but neither fired the
release.published event, so notify.yaml never ran.

Passing secrets.PAT (already configured in this repo for the integrate +
review-approval workflows) attributes the release to a real identity,
which lets the release.published event propagate to notify.yaml.

Verified by inspecting actions/runs?event=release for the repo, which
returned total_count=0 across the entire repo history.

Signed-off-by: Synertry <60792595+Synertry@users.noreply.github.com>
@github-actions github-actions Bot added area/CI GitHub workflows, Dependabot, labelers, scripts type/fix A bug fix size/xs Extra small diff (< 25 lines, 1 file) status/pr-test-passed Lightweight PR test workflow passed (granted by pr-test.yaml) labels Jun 9, 2026
@Synertry
Synertry merged commit 3f186a9 into main Jun 9, 2026
7 checks passed
@github-actions
github-actions Bot deleted the fix/release-cascade-notify branch June 9, 2026 02:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/CI GitHub workflows, Dependabot, labelers, scripts size/xs Extra small diff (< 25 lines, 1 file) status/pr-test-passed Lightweight PR test workflow passed (granted by pr-test.yaml) type/fix A bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant