Skip to content

Update @types/node and node versions in package.json#3247

Merged
fellyph merged 11 commits intotrunkfrom
update-node-package
Feb 9, 2026
Merged

Update @types/node and node versions in package.json#3247
fellyph merged 11 commits intotrunkfrom
update-node-package

Conversation

@fellyph
Copy link
Copy Markdown
Collaborator

@fellyph fellyph commented Feb 4, 2026

This pull request updates the project's Node.js compatibility and related type definitions to ensure support for newer Node.js features and APIs.

Dependency updates:

  • Updated the @types/node package from version 20.14.8 to 22.19.8 in package.json, ensuring type compatibility with the latest Node.js releases.

Environment requirements:

  • Increased the minimum required Node.js version from >=20.18.3 to >=22.12.0 and npm from >=10.1.0 to >=10.9.0 in the engines field of package.json, enforcing the use of more recent runtime environments.

@fellyph fellyph requested a review from a team as a code owner February 4, 2026 21:04
@fellyph fellyph requested review from Copilot and zaerl February 4, 2026 21:04
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the project's Node.js runtime requirements and TypeScript type definitions to align with newer versions of Node.js, ensuring compatibility with modern features and APIs.

Changes:

  • Updated @types/node from version 20.14.8 to 22.19.8
  • Increased minimum Node.js version requirement from >=20.18.3 to >=22.12.0
  • Increased minimum npm version requirement from >=10.1.0 to >=10.9.0

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@bgrgicak
Copy link
Copy Markdown
Collaborator

bgrgicak commented Feb 5, 2026

@fellyph there's a bunch of other places like docs and package.json files that should also be updated.

Screenshot 2026-02-05 at 07 28 46

@fellyph
Copy link
Copy Markdown
Collaborator Author

fellyph commented Feb 5, 2026

@bgrgicak, I opened a separate PR for the docs. For the internal packages, I will update them now.

@fellyph
Copy link
Copy Markdown
Collaborator Author

fellyph commented Feb 5, 2026

@bgrgicak I have updated the other packages and updated the git action to run with node 22

@fellyph fellyph requested review from bgrgicak and Copilot February 5, 2026 13:10
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 35 out of 36 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Copy Markdown
Member

@brandonpayton brandonpayton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't see any issues with this, and the changes make sense to me. Thank you!

Note: I didn't check every package.json in the project for Node.js version updates, but if we miss anything, it is easily fixable.

@fellyph fellyph merged commit 015974c into trunk Feb 9, 2026
35 checks passed
@fellyph fellyph deleted the update-node-package branch February 9, 2026 10:31
fellyph added a commit that referenced this pull request Feb 10, 2026
brandonpayton pushed a commit that referenced this pull request Feb 11, 2026
…#3256)

This reverts commit 015974c.

This pull request updates the minimum required Node.js and npm versions
across the codebase, lowering the Node.js requirement from 22.x to 20.x
and adjusting npm requirements accordingly. It also updates the
`@types/node` dependency version and ensures documentation and CI
configurations are consistent with these changes.

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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.

4 participants