Skip to content

fix(release): let the release workflow inherit the caller's permissions - #64

Merged
unional merged 1 commit into
mainfrom
fix/release-inherit-caller-permissions
Oct 11, 2026
Merged

unional merged 1 commit into
mainfrom
fix/release-inherit-caller-permissions

Conversation

@unional

@unional unional commented Oct 11, 2026

Copy link
Copy Markdown
Contributor

The CI dispatch step added for the Version Packages PR fails with HTTP 403: Resource not accessible by integration (cyberuni/iso-error, release run 38112276866), even though the caller's release job grants actions: write.

The cause is this reusable workflow's own top-level permissions: block (id-token, contents, pull-requests). It caps the token, so every scope it omits, actions included, becomes none. Adding actions: write to the block would break every caller that does not grant it, because a called workflow may not request more than its caller grants.

This removes the block, so the job inherits exactly what the caller grants. A reusable workflow can never exceed its caller's grant, so existing callers lose nothing: those that grant id-token, contents and pull-requests keep working, and those that also grant actions: write get CI on the Version Packages PR. The header now says what callers must grant and that the workflow declares no permissions of its own.

Verified with actionlint.

🤖 Generated with Claude Code

The top-level permissions block capped GITHUB_TOKEN and set actions to none,
so the Version Packages PR CI dispatch failed with 403 even when the caller
granted actions: write. Without the block the job inherits the caller's grant.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: unional <homawong@gmail.com>
@changeset-bot

changeset-bot Bot commented Oct 11, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: d9a1389

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@mergify

mergify Bot commented Oct 11, 2026 •

Copy link
Copy Markdown

This pull request does not currently match the merge queue conditions, so it cannot be queued from here. The box comes back if it matches again.

@unional
unional merged commit 9e19976 into main Oct 11, 2026
3 checks passed
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