Skip to content

chore: upgrade Node.js 18 (EOL) → 24 LTS#651

Merged
islandbitcoin merged 1 commit into
mainfrom
chore/upgrade-node-24
Jun 26, 2026
Merged

chore: upgrade Node.js 18 (EOL) → 24 LTS#651
islandbitcoin merged 1 commit into
mainfrom
chore/upgrade-node-24

Conversation

@islandbitcoin

Copy link
Copy Markdown
Contributor

Summary

Node 18 reached end-of-life in April 2025. This bumps the project to Node 24 (LTS), supported through ~April 2028.

Changes (7 files)

  • .node-version: 1824
  • package.json: engines.node >=18>=24 (no @types/node entry present)
  • .github/workflows/test.yml: node-version 1824, setup-node@v3@v4
  • .github/workflows/check-code.yml: node-version 1824, setup-node@v3@v4
  • .github/workflows/i18n-drift.yml: node-version 2024 (was a stray partial upgrade — now aligned)
  • .circleci/config.yml: all 3 nodesource setup_18.xsetup_24.x
  • ci/image/Dockerfile: node:18-alpinenode:24-alpine

Validation

Validated locally under Node v24.18.0 (nvm) with Yarn 1.22.22:

  • yarn install — passed, no native-module / node-gyp failures. yarn.lock unchanged.
  • engines: >=24 enforcement confirmed (yarn refuses to run under Node 20).
  • ⚠️ yarn tsc:check fails with a pre-existing TS5095 tsconfig conflict (module: Node16 vs inherited moduleResolution: bundler) — reproduced identically on Node 20, not a regression from this PR, and untouched here. Should be fixed separately.

⚠️ Needs CI verification

Android/iOS native builds and the Metro bundler were not validated locally (out of scope). Please confirm green native builds in CI before merge.

🤖 Generated with Claude Code

Node 18 reached end-of-life in April 2025. Bump .node-version, engines,
GitHub Actions, CircleCI, and the CI Docker image to Node 24 (LTS,
supported through ~April 2028). Aligns i18n-drift workflow (was on 20)
with the rest.

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

Copy link
Copy Markdown
Contributor Author

Android/iOS native builds and the Metro bundler successfully validated locally.

@islandbitcoin islandbitcoin merged commit 86fd979 into main Jun 26, 2026
4 of 7 checks passed
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