Skip to content

ci: narrow workflow permissions - #21

Merged
omkhar merged 3 commits into
mainfrom
codex/zizmor-ignore-removal
Mar 26, 2026
Merged

ci: narrow workflow permissions#21
omkhar merged 3 commits into
mainfrom
codex/zizmor-ignore-removal

Conversation

@omkhar

@omkhar omkhar commented Mar 26, 2026

Copy link
Copy Markdown
Owner

Summary

  • remove the explicit zizmor excessive-permissions ignores by tightening CI and Security to workflow-level permissions: {} with per-job minima
  • narrow the rest of the workflow fleet away from redundant top-level permissions: read-all
  • keep explicit higher-scope jobs only where they are still required, including release preflight actions: read for zizmor-action

Local validation

  • ./scripts/check-workflows.sh
  • actionlint .github/workflows/*.yml
  • zizmor .github/workflows

Peer review

  • permissions-focused review found one real regression in release.yml (zizmor-action still needed actions: read in private-repo preflight); fixed in follow-up commit
  • final reviewed worktree returned no findings and is merge-ready pending PR CI

@omkhar
omkhar merged commit ba59c39 into main Mar 26, 2026
10 checks passed
@omkhar
omkhar deleted the codex/zizmor-ignore-removal branch March 26, 2026 14:11
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