Skip to content

ci: Add missing top-level permissions to publish-release workflow#290

Merged
Mrtenz merged 1 commit into
mainfrom
mrtenz/add-top-level-permissions-publish-release
Jun 8, 2026
Merged

ci: Add missing top-level permissions to publish-release workflow#290
Mrtenz merged 1 commit into
mainfrom
mrtenz/add-top-level-permissions-publish-release

Conversation

@Mrtenz

@Mrtenz Mrtenz commented Jun 8, 2026

Copy link
Copy Markdown
Member

Summary

  • Adds top-level permissions: contents: read to the publish-release workflow, following the principle of least privilege.

Note

Low Risk
Workflow-only permission tightening with no application or release logic changes; job overrides preserve existing publish behavior.

Overview
Adds workflow-level permissions: contents: read to publish-release.yml so the reusable workflow defaults to least privilege instead of relying on the repository default token scope.

Job-level overrides are unchanged: publish-release still requests contents: write, and publish-npm still sets contents: read plus id-token: write for OIDC publishing.

Reviewed by Cursor Bugbot for commit 1743f09. Bugbot is set up for automated code reviews on this repo. Configure here.

@Mrtenz Mrtenz marked this pull request as ready for review June 8, 2026 10:44
@Mrtenz Mrtenz requested a review from a team as a code owner June 8, 2026 10:44
@Mrtenz Mrtenz enabled auto-merge (squash) June 8, 2026 10:44
@Mrtenz Mrtenz merged commit a187829 into main Jun 8, 2026
26 checks passed
@Mrtenz Mrtenz deleted the mrtenz/add-top-level-permissions-publish-release branch June 8, 2026 10:47
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