Skip to content

Conversation

@Tim020
Copy link
Contributor

@Tim020 Tim020 commented Jan 7, 2026

Summary

Upgrades DigiScript from Node v22.14.0 to Node v24.12.0 (Active LTS "Krypton") to resolve Dependabot dependency update failures.

Closes #817

Changes

Configuration:

  • .nvmrc: v24.12.0
  • package.json: Node >=24.0.0 <25, npm >=11.0.0 <12
  • package-lock.json: Regenerated with npm 11

CI/CD:

  • Dockerfile: node:24-bookworm
  • GitHub Actions workflows updated to Node 24
  • Removed redundant npm install steps

Documentation:

  • Updated README.md and CLAUDE.md

Rationale

Dependabot uses Node v24.12.0 and npm 11.7.0, causing EBADENGINE errors with our strict engine requirements. Node v24 is the current Active LTS.

Test Plan

  • Local npm install with Node 24
  • Frontend build succeeds
  • Linting passes
  • Unit tests run
  • CI workflows pass
  • Docker build succeeds

Breaking Changes

For developers: Node v24.12.0 required (run nvm install 24.12.0)

🤖 Generated with Claude Code

@Tim020 Tim020 added the claude Issues created by Claude label Jan 7, 2026
@github-actions github-actions bot added github GitHub actions related issue or pull request client Pull requests changing front end code xlarge-diff labels Jan 7, 2026
@github-actions
Copy link

github-actions bot commented Jan 7, 2026

Python Test Results

  1 files  ±0    1 suites  ±0   19s ⏱️ ±0s
253 tests ±0  253 ✅ ±0  0 💤 ±0  0 ❌ ±0 
258 runs  ±0  258 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 8ca3a04. ± Comparison against base commit 845fcb1.

♻️ This comment has been updated with latest results.

- Update .nvmrc to v24.12.0
- Update package.json engines to require Node >=24.0.0 <25 and npm >=11.0.0 <12
- Regenerate package-lock.json with npm 11 (lockfileVersion 3)
- Update Dockerfile to use node:24-bookworn base image
- Update GitHub Actions workflows to use Node 24
- Remove redundant npm global install steps (npm 11 bundled with Node 24)
- Update documentation to reflect Node 24.x requirement

This resolves Dependabot EBADENGINE errors by aligning with Dependabot's
Node v24.12.0 / npm 11.7.0 environment.

Tested locally:
- npm install completes successfully
- Build process works (npm run build)
- Linting passes (npm run ci-lint)
- Unit tests run (npm run test:run)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@Tim020 Tim020 linked an issue Jan 7, 2026 that may be closed by this pull request
6 tasks
@github-actions
Copy link

github-actions bot commented Jan 7, 2026

Client Test Results

34 tests  ±0   34 ✅ ±0   0s ⏱️ ±0s
 1 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit 8ca3a04. ± Comparison against base commit 845fcb1.

@Tim020 Tim020 merged commit 531583d into dev Jan 7, 2026
18 checks passed
@Tim020 Tim020 deleted the upgrade-node-24 branch January 7, 2026 00:52
Tim020 added a commit that referenced this pull request Jan 7, 2026
Upgrade Node.js from v22 to v24 (Active LTS) (#818)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

claude Issues created by Claude client Pull requests changing front end code github GitHub actions related issue or pull request xlarge-diff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Upgrade Node.js from v22 to v24 (Active LTS)

2 participants