Skip to content

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

Merged
Mrtenz merged 1 commit into
mainfrom
mrtenz/fix-publish-permissions
Jun 8, 2026
Merged

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

Conversation

@Mrtenz

@Mrtenz Mrtenz commented Jun 8, 2026

Copy link
Copy Markdown
Member

Summary

  • Adds top-level permissions: contents: read to publish-release.yml, restricting default token permissions for all jobs in the workflow.

Note

Low Risk
CI-only least-privilege change; jobs that need write or OIDC permissions still declare them explicitly.

Overview
Adds workflow-level permissions: contents: read to .github/workflows/publish-release.yml so the default GITHUB_TOKEN scope is read-only for every job in this reusable release workflow.

Jobs that still need broader access keep their existing job-level overrides (for example contents: write on release/docs jobs and id-token: write on publish-npm), so release behavior should stay the same while tightening the baseline token permissions.

Reviewed by Cursor Bugbot for commit 6b92d0c. 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:39
@Mrtenz Mrtenz merged commit 8447ac1 into main Jun 8, 2026
37 of 39 checks passed
@Mrtenz Mrtenz deleted the mrtenz/fix-publish-permissions branch June 8, 2026 10:44
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