Skip to content

Pin GitHub Actions to SHAs and fix zizmor findings#10

Merged
ewels merged 1 commit into
mainfrom
security/pin-actions-zizmor
May 20, 2026
Merged

Pin GitHub Actions to SHAs and fix zizmor findings#10
ewels merged 1 commit into
mainfrom
security/pin-actions-zizmor

Conversation

@ewels
Copy link
Copy Markdown
Owner

@ewels ewels commented May 20, 2026

Summary

  • Pin every third-party action in .github/workflows/ to a commit SHA (with a readable # vX.Y.Z comment) via npx actions-up --style sha
  • Resolve all findings from zizmor .:
    • artipacked — set persist-credentials: false on every actions/checkout
    • excessive-permissions — add top-level permissions: contents: read to ci.yml and publish.yml (docs.yml already declared its own)
    • unpinned-uses — pin pypa/gh-action-pypi-publish to v1.14.0 by SHA instead of the moving release/v1 branch

zizmor . now reports No findings to report. Good job! (11 suppressed).

Test plan

  • npx actions-up --style sha --yes applied 19 SHA updates
  • zizmor . exits 0 with no findings
  • prek run --files .github/workflows/*.yml passes prettier formatting
  • CI workflow runs green on this PR
  • docs.yml — manually verify GitHub Pages deploy still works once merged
  • publish.yml — exercised on next release tag

🤖 Generated with Claude Code

Pin every third-party action in .github/workflows to a commit SHA
(with a readable version comment) and resolve the zizmor audit findings
that surfaced once the workflows were no longer protected by tag mutability:

- artipacked: set persist-credentials: false on every actions/checkout
- excessive-permissions: add top-level "permissions: contents: read" to
  ci.yml and publish.yml; docs.yml already declares its own block
- unpinned-uses: pin pypa/gh-action-pypi-publish to v1.14.0 by SHA instead
  of the moving release/v1 branch

zizmor . now reports no findings.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@ewels ewels merged commit f8accc8 into main May 20, 2026
8 checks passed
@ewels ewels deleted the security/pin-actions-zizmor branch May 20, 2026 20:58
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