Bump action runtime from Node.js 20 to Node.js 24#2
Draft
heyhayes wants to merge 4 commits into
Draft
Conversation
Node.js 20 is deprecated for GitHub Actions runtimes. Update the action to run on node24 and align the CI Node version and package metadata. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XJb45njfgZ9qBnkTQcVcBu
actions/upload-artifact@v2 is auto-failed by GitHub as deprecated, which broke the check-dist job. Bump checkout and upload-artifact to v4. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XJb45njfgZ9qBnkTQcVcBu
The runtime bump to node24 (action.yml) is the deprecation fix; @types/node is a dev-only type package and bumping it forced a package-lock.json change that broke 'npm ci' in check-dist. Revert it to keep the lock in sync. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XJb45njfgZ9qBnkTQcVcBu
The committed dist/index.js.map predated this fork's package rename from @ksivamuthu to @heyhayes, so its embedded webpack source paths were stale. check-dist rebuilds dist/ and compares, so it failed on the mismatch. Rebuild to regenerate the map with the correct package name. Only the sourcemap changes; dist/index.js is unaffected. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01XJb45njfgZ9qBnkTQcVcBu
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Node.js 20 is deprecated as a GitHub Actions runtime, and consumers of
heyhayes/actions-setup-gh-cli@v4are seeing warnings that the action is being force-run on Node.js 24:This bumps the action to target the
node24runtime so it runs natively without the deprecation warning.Changes
action.yml:runs.usingnode20→node24.github/workflows/check-dist.yml: CI build now uses Node.js24.xpackage.json:engines.node>=24.0.0and@types/node^24.0.0No source changes were needed, so the checked-in
dist/index.jsbundle is unchanged — only the runtime declaration moves to Node 24.Releasing v5
Once this merges to
main, cut av5release tag pointing at the merge commit (matching the existingv1–v4major-version tag convention) so consumers can pin@v5.🤖 Generated with Claude Code
Generated by Claude Code