Skip to content

fix: upgrade action runtime to node24 - #1

Open
yukicoder0509 wants to merge 1 commit into
mainfrom
upgrade-node24
Open

fix: upgrade action runtime to node24#1
yukicoder0509 wants to merge 1 commit into
mainfrom
upgrade-node24

Conversation

@yukicoder0509

@yukicoder0509 yukicoder0509 commented Aug 29, 2026

Copy link
Copy Markdown
Member

Summary

  • Node 20 is being deprecated by GitHub Actions; hosted runners now support node24.
  • Bump action.yml runtime from node20 to node24. Rebuilt dist/index.js under Node 24 — output was unchanged since this action has no native dependencies.
  • Add .github/workflows/test.yml to exercise the action end-to-end via uses: ./ on push/PR, asserting the extracted outputs match expectations, so this stays verified going forward.
  • This would resolve the issue Dependency "node 20" is deprecated by GitHub Action Ocean1029/extract-pr-body-action#1

Test plan

  • Rebuilt dist/index.js with Node 24 (nvm use 24 && npm run build) — no diff from committed output.
  • Ran node dist/index.js directly under Node 24 with INPUT_PR_BODY/GITHUB_OUTPUT env vars simulating the Actions runtime — outputs (type_block, purpose_block) parsed correctly.
  • New test.yml workflow will run on this PR to confirm on an actual ubuntu-latest runner.

🤖 Generated with Claude Code

https://claude.ai/code/session_019Z4cVCaJBKd9gDJF9vR43e

Node 20 is being deprecated by GitHub Actions and node24 is now
supported on hosted runners. Rebuilt dist under Node 24 (output was
unchanged) and added a workflow that exercises the action end-to-end
via `uses: ./` to verify outputs on push/PR.

Fixes #1

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019Z4cVCaJBKd9gDJF9vR43e
@Ocean1029
Ocean1029 requested review from thomas0217 and removed request for Ocean1029 August 31, 2026 13:04
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.

2 participants