Bump minimum Node.js version requirement to 22.0.0#6
Conversation
There was a problem hiding this comment.
Pull request overview
Bumps the minimum Node.js version from 20.0.0 to 24.0.0 across all packages in the monorepo.
Changes:
- Updated
engines.nodefrom>=20.0.0to>=24.0.0in root,e2e-core, andtinytsdipackage.json files
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| package.json | Root engine requirement bumped to >=24.0.0 |
| packages/e2e-core/package.json | Engine requirement bumped to >=24.0.0 |
| packages/tinytsdi/package.json | Engine requirement bumped to >=24.0.0 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
Root package.json is sufficient for internal packages. Only the published tinytsdi package needs its own engines field. https://claude.ai/code/session_012DCTbfxRHR4E359DuKbkzi
Node 22 is the current Active LTS release. Node 24 would exclude users on Node 22 LTS without any specific Node 24 feature requirement. https://claude.ai/code/session_012DCTbfxRHR4E359DuKbkzi
There was a problem hiding this comment.
Pull request overview
This PR updates the minimum Node.js version requirement across the monorepo. However, the actual changes are inconsistent with the PR description.
Changes:
- Root and
tinytsdipackage.jsonengine requirement changed from>=20.0.0to>=22.0.0(not>=24.0.0as described) e2e-corepackage.jsonhas itsenginesfield removed entirely instead of updated
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| package.json | Root engine bumped from >=20.0.0 to >=22.0.0 |
| packages/tinytsdi/package.json | Engine bumped from >=20.0.0 to >=22.0.0 |
| packages/e2e-core/package.json | engines field removed entirely |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
drdxk
changed the title
There was a problem hiding this comment.
Pull request overview
Bumps the minimum Node.js version from 20.0.0 to 22.0.0 (current Active LTS) and removes a redundant engines field from an internal package.
Changes:
- Updated root and
tinytsdiengines.nodeto>=22.0.0 - Removed redundant
enginesfrom the privatee2e-corepackage
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| package.json | Bump root engine requirement to >=22.0.0 |
| packages/tinytsdi/package.json | Bump published package engine requirement to >=22.0.0 |
| packages/e2e-core/package.json | Remove redundant engines field from private package |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
Summary
Updated the minimum required Node.js version from 20.0.0 to 22.0.0 across relevant packages.
Changes
package.jsonengine requirement to>=22.0.0packages/tinytsdi/package.jsonengine requirement to>=22.0.0enginesfield frompackages/e2e-core/package.json(internal/unpublished package — root coverage is sufficient)Details
Node.js 22 is the current Active LTS release (since October 2024). Setting the minimum to 22 follows the standard LTS progression and avoids excluding users still on Node.js 22 LTS. The
enginesfield is only needed in the root (dev guard) and the publishedtinytsdipackage (for npm consumers).