Skip to content

fix(notify): scope run/jobs lookups to github.repository with optional run-token - #23

Merged
gamechanger1s merged 4 commits into
mainfrom
fix/notify-run-token
Aug 11, 2026
Merged

gamechanger1s merged 4 commits into
mainfrom
fix/notify-run-token

Conversation

@gamechanger1s

Copy link
Copy Markdown
Contributor

Bug

The Slack card is missing DURATION and FAILED AT when the caller sets target-repo for cross-org attribution.

notify@v4.0.0 calls the Actions API on target-repo to fill those two rows — but Actions runs only exist in github.repository. So the calls 404, get swallowed by 2>/dev/null, and both rows silently disappear.

Same-repo callers (target-repo == github.repository) never saw it.

Fix

  • Look up run/jobs on github.repository (that's where the run actually is — hard-coded, not overridable).
  • Add optional run-token for those two calls; falls back to github-token, so existing callers are byte-identical.
  • Also fixes a load-time crash from ${{ }} inside the composite action's description: block.

Fully additive — no consumer has to change to keep working.

@gamechanger1s
gamechanger1s merged commit 044aa36 into main Aug 11, 2026
2 checks passed
@gamechanger1s
gamechanger1s deleted the fix/notify-run-token branch August 11, 2026 14:28
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