Skip to content

ci: Add explicit permissions to workflows#306

Merged
Mrtenz merged 1 commit intomainfrom
mrtenz/workflow-permissions
Apr 20, 2026
Merged

ci: Add explicit permissions to workflows#306
Mrtenz merged 1 commit intomainfrom
mrtenz/workflow-permissions

Conversation

@Mrtenz
Copy link
Copy Markdown
Member

@Mrtenz Mrtenz commented Apr 15, 2026

This fixes a bunch of CodeQL alerts.


Note

Low Risk
CI-only change that narrows workflow token permissions; main risk is unexpected permission-related failures if a step implicitly relied on broader defaults.

Overview
Sets explicit default permissions on the reusable workflows (build-lint-test.yml, publish-release.yml) and the main.yml workflow to contents: read to satisfy security tooling and follow least-privilege defaults.

Further restricts the main.yml aggregation jobs (all-jobs-completed, all-jobs-pass) by explicitly setting permissions: {} so they run with no GITHUB_TOKEN access.

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

@Mrtenz Mrtenz marked this pull request as ready for review April 15, 2026 16:09
@Mrtenz Mrtenz requested a review from a team as a code owner April 15, 2026 16:09
@Mrtenz Mrtenz force-pushed the mrtenz/workflow-permissions branch from df29b67 to 5ef8fa0 Compare April 20, 2026 09:19
@Mrtenz Mrtenz merged commit 2321b89 into main Apr 20, 2026
26 checks passed
@Mrtenz Mrtenz deleted the mrtenz/workflow-permissions branch April 20, 2026 09:48
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.

3 participants