Skip to content

ci: Add explicit permissions to workflows#306

Open
Mrtenz wants to merge 1 commit intomainfrom
mrtenz/workflow-permissions
Open

ci: Add explicit permissions to workflows#306
Mrtenz wants to merge 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
Low risk: only GitHub Actions workflow permission scopes are tightened/declared, with no application/runtime code changes. Potential risk is limited to CI failures if any job implicitly relied on broader default GITHUB_TOKEN permissions.

Overview
Adds explicit default permissions: contents: read to reusable CI workflows (build-lint-test.yml, main.yml, publish-release.yml) to satisfy security/CodeQL guidance.

Hardens the aggregator jobs in main.yml by setting permissions: {} for all-jobs-completed and all-jobs-pass, ensuring they run with no token privileges unless explicitly needed.

Reviewed by Cursor Bugbot for commit df29b67. 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
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