Skip to content

ci: add zizmor security audit for workflow changes#288

Merged
dgilmanuni merged 5 commits into
mainfrom
ci/add-zizmor-audit
Mar 10, 2026
Merged

ci: add zizmor security audit for workflow changes#288
dgilmanuni merged 5 commits into
mainfrom
ci/add-zizmor-audit

Conversation

@msarkor718

Copy link
Copy Markdown
Contributor

Summary

  • Adds a new zizmor.yml workflow that runs the zizmor GitHub Actions security auditor on PRs that modify .github/workflows/** or .github/actions/**
  • Uploads SARIF results to GitHub's Security tab, so findings appear as inline code scanning alerts on PRs
  • Uses SHA-pinned actions, persist-credentials: false, and minimal permissions — consistent with the hardening in fix: harden CI workflows against zizmor security findings #287

Test plan

  • Open a PR that modifies a workflow file and verify zizmor runs
  • Confirm SARIF results appear in the Security tab
  • Open a PR that only modifies hook files and verify zizmor does not run

🤖 Generated with Claude Code

Runs zizmor on PRs that modify .github/workflows/ or .github/actions/
and uploads SARIF results to GitHub Security tab for inline annotations.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
msarkor718 and others added 4 commits March 9, 2026 12:41
The codeql-action/upload-sarif step needs actions:read to access
the workflow-runs API for telemetry and fingerprinting.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Code scanning requires GitHub Advanced Security to be enabled.
Simplify to just run zizmor with default output — the workflow
fails if any findings are reported.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Run zizmor only on workflow/action files changed in the PR
  instead of scanning the entire repo
- Add .zizmor.yml config to ignore secrets-outside-env for
  workflows that intentionally use secrets without environments

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The shallow checkout doesn't have origin/main, so the git diff
to find changed files fails.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@dgilmanuni
dgilmanuni merged commit a3672c4 into main Mar 10, 2026
6 checks passed
@dgilmanuni
dgilmanuni deleted the ci/add-zizmor-audit branch March 10, 2026 01:45
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