Skip to content

feat(guards): add workflow-permissions guard and declare top-level pe… - #301

Open
gsebastianlopezillia wants to merge 1 commit into
theam:mainfrom
gsebastianlopezillia:feat/workflow-permissions-guard
Open

feat(guards): add workflow-permissions guard and declare top-level pe…#301
gsebastianlopezillia wants to merge 1 commit into
theam:mainfrom
gsebastianlopezillia:feat/workflow-permissions-guard

Conversation

@gsebastianlopezillia

Copy link
Copy Markdown

…rmissions in ci.yml

Enforce explicit top-level permissions in all GitHub Actions workflows. Without top-level permissions, newly added jobs or jobs omitting per-job permissions inherit the default repository GITHUB_TOKEN scope. Add guards/workflow-permissions.mjs to deterministic guard suite and declare permissions: contents: read in .github/workflows/ci.yml.

What changes

Why

Verification

  • pnpm verify passes locally
  • Behaviour verified beyond the test suite (say how)
  • Documentation updated, or no user-facing change

…rmissions in ci.yml

Enforce explicit top-level permissions in all GitHub Actions workflows. Without top-level permissions, newly added jobs or jobs omitting per-job permissions inherit the default repository GITHUB_TOKEN scope. Add guards/workflow-permissions.mjs to deterministic guard suite and declare permissions: contents: read in .github/workflows/ci.yml.
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