Skip to content

chore(ci): pin actions/setup-node to immutable SHA#15

Merged
Skiipy11 merged 1 commit into
mainfrom
chore/pin-setup-node-v5
Jul 21, 2026
Merged

chore(ci): pin actions/setup-node to immutable SHA#15
Skiipy11 merged 1 commit into
mainfrom
chore/pin-setup-node-v5

Conversation

@Skiipy11

@Skiipy11 Skiipy11 commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Pin actions/setup-node@v5 → full 40-hex SHA in ci.yml and publish.yml.
  • actions/checkout already SHA-pinned.

Why

Public flagship; publish.yml holds NPM_TOKEN + id-token:write — highest supply-chain exposure in the estate.

Test plan

  • CI green on this PR
  • grep uses: shows only 40-hex-pinned action refs

Summary by CodeRabbit

  • Chores
    • Updated automated workflows to use a fixed version of the Node.js setup action, improving build and publishing consistency.

Replace mutable @v5 tags in ci.yml and publish.yml (publish carries
NPM_TOKEN + id-token:write). Renovate helpers:pinGitHubActionDigests
owns future digest updates.
@coderabbitai

coderabbitai Bot commented Jul 21, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: ZenSystemAI/coderabbit/.coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 88a3f631-2ee2-411f-8529-36465f6552af

📥 Commits

Reviewing files that changed from the base of the PR and between a3829f8 and c3149c5.

📒 Files selected for processing (2)
  • .github/workflows/ci.yml
  • .github/workflows/publish.yml

📝 Walkthrough

Walkthrough

The CI and publishing workflows now reference actions/setup-node by a fixed commit SHA instead of the floating v5 tag. No other workflow logic or commands changed.

Changes

Workflow Action Pinning

Layer / File(s) Summary
Pin setup-node in workflow jobs
.github/workflows/ci.yml, .github/workflows/publish.yml
Both workflows replace the actions/setup-node@v5 reference with the specified commit SHA.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and accurately summarizes the main change: pinning actions/setup-node to an immutable SHA in CI.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/pin-setup-node-v5

Comment @coderabbitai help to get the list of available commands.

@Skiipy11
Skiipy11 merged commit 84daba2 into main Jul 21, 2026
5 checks passed
@Skiipy11
Skiipy11 deleted the chore/pin-setup-node-v5 branch July 21, 2026 15:16
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