Skip to content

ci: pin third-party actions to SHAs, persist-credentials: false, seed zizmor.yml - #38

Merged
twistedmelonman merged 1 commit into
mainfrom
claude/ci-zizmor-019HDRKL
Sep 9, 2026
Merged

ci: pin third-party actions to SHAs, persist-credentials: false, seed zizmor.yml#38
twistedmelonman merged 1 commit into
mainfrom
claude/ci-zizmor-019HDRKL

Conversation

@twistedmelonman

Copy link
Copy Markdown
Member

Wave 2 (zizmor) remediation for smartwatermelon/lock-sync.

  • Pins actions/checkout to v7.0.1 (three steps across ci.yml and claude.yml) and anthropics/claude-code-action to v1.0.219 (unpinned-uses).
  • Adds persist-credentials: false to every checkout step (artipacked). No job here pushes, commits, or tags, so none consumes the persisted credential.
  • Seeds zizmor.yml verbatim from github-workflows.

dependabot.yml already covers github-actions. zizmor and actionlint are clean locally; shellcheck and markdownlint findings remain and are wave 3.

https://claude.ai/code/session_019HDRKLQNv82SEBd4zGpcXf

… zizmor.yml

Clears zizmor's unpinned-uses and artipacked findings on ci.yml and
claude.yml.

Pins:
- actions/checkout: v7.0.1 (3d3c42e5aac5ba805825da76410c181273ba90b1)
- anthropics/claude-code-action: v1.0.219 (5ccc3a35a6367cdb8e6fbd0728287467540ecfe2)

Adds persist-credentials: false to all three checkout steps. No job in
either workflow pushes, commits, or tags, so none consumes the persisted
credential. The claude.yml case matches the canonical pattern in
github-workflows/.github/workflows/claude-assistant.yml.

Seeds zizmor.yml verbatim from github-workflows so local and CI runs
resolve the same config. dependabot.yml already covers github-actions.

shellcheck and markdownlint findings remain and are out of scope for this
wave.

Claude-Session: https://claude.ai/code/session_019HDRKLQNv82SEBd4zGpcXf
@twistedmelonman
twistedmelonman merged commit 20310f7 into main Sep 9, 2026
4 of 5 checks passed
@twistedmelonman
twistedmelonman deleted the claude/ci-zizmor-019HDRKL branch September 9, 2026 01:24
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