Skip to content
This repository was archived by the owner on Jul 31, 2026. It is now read-only.

ci: bump node20 actions to node24 runtime - #351

Merged
alx-xo merged 1 commit into
mainfrom
chore/upgrade-actions-node24
Jun 9, 2026
Merged

ci: bump node20 actions to node24 runtime#351
alx-xo merged 1 commit into
mainfrom
chore/upgrade-actions-node24

Conversation

@alx-xo

@alx-xo alx-xo commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator

What

Bump two GitHub Actions still running the deprecated node20 runtime to node24:

Action Before After
pnpm/action-setup @v4 @v6
amannn/action-semantic-pull-request @v5 @v6

Why

CI emits node20 runtime deprecation warnings. These two were the only actions still on node20 — actions/checkout@v6, actions/setup-node@v6, and cycjimmy/semantic-release-action (v6.0.0) already run node24.

Notes

  • pnpm/action-setup v5+ resolves the pnpm version from packageManager in package.json (pnpm@11.4.0, already present) — no version input needed.
  • Floating major tags kept, consistent with existing workflows.

🤖 Generated with Claude Code


Summary by cubic

Upgrade remaining CI actions from the deprecated node20 runtime to node24 to remove deprecation warnings and keep workflows consistent.

  • Dependencies
    • pnpm/action-setup: @v4@v6 (uses packageManager in package.json; no version input needed)
    • amannn/action-semantic-pull-request: @v5@v6

Written for commit dc77521. Summary will update on new commits.

Review in cubic

pnpm/action-setup v4 and amannn/action-semantic-pull-request v5 ship the
deprecated node20 runtime, triggering CI deprecation warnings. Bump both to
v6 (node24). v5+ of action-setup reads the pnpm version from package.json's
packageManager field, already present.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2 issues found across 6 files

Confidence score: 4/5

  • This PR is likely safe to merge, with moderate but non-blocking CI/CD security hardening gaps rather than an immediate functional regression risk.
  • The most significant issue is in .github/workflows/publish.yml, where pnpm/action-setup@v6 is a floating tag; if that tag is retargeted upstream, pipeline behavior could change unexpectedly and weaken supply-chain integrity.
  • A similar mutable-tag exposure exists in .github/workflows/typecheck.yml; pinning third-party actions to immutable commit SHAs would reduce tampering risk and improve build reproducibility.
  • Pay close attention to .github/workflows/publish.yml and .github/workflows/typecheck.yml - both rely on mutable action tags instead of commit SHAs.
Prompt for AI agents (unresolved issues)

Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.


<file name=".github/workflows/publish.yml">

<violation number="1" location=".github/workflows/publish.yml:22">
P2: The workflow uses a floating GitHub Action tag (`pnpm/action-setup@v6`) instead of pinning to a commit SHA, which weakens CI/CD supply-chain integrity.</violation>
</file>

Reply with feedback, questions, or to request a fix.

Fix all with cubic | Re-trigger cubic

Comment thread .github/workflows/publish.yml
Comment thread .github/workflows/typecheck.yml
@alx-xo
alx-xo merged commit c105b2b into main Jun 9, 2026
6 checks passed
@alx-xo
alx-xo deleted the chore/upgrade-actions-node24 branch June 9, 2026 19:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant