Skip to content

feat!: upgrade minimum Node.js version to 22 - #9026

Merged
danieljbruce merged 20 commits into
mainfrom
node-22-bump-generator-version
Jul 31, 2026
Merged

feat!: upgrade minimum Node.js version to 22#9026
danieljbruce merged 20 commits into
mainfrom
node-22-bump-generator-version

Conversation

@danieljbruce

@danieljbruce danieljbruce commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Description

Upgrade all generated packages to Node 22 along with their dependencies.

For all package.json files inside packages/** folder we set "node": ">=22" for engines. We also make sure that for all of these package.json files the dependencies that match the core libraries are all on the latest version so that all core library dependencies correspond to node v22.

Note that a branch update is likely required to get the zizmor checks to pass.

Also note that upgrading the auth library dependencies causes compiler errors so we opt out of upgrading the auth library.

Impact

Encourages customers to adopt new Node version which avoids issues due to lack of Node 18 support.

Next Steps

Upgrade the auth dependencies that we chose not to upgrade this time due to the compiler error. Tracked here.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request updates the google-gax dependency to ^6.0.0 and the Node.js engine requirement to >=22 across multiple packages and baselines. Additionally, it introduces new baseline configuration files for ESLint and Mocha. I have no feedback to provide.

@danieljbruce
danieljbruce marked this pull request as ready for review July 31, 2026 18:52
@danieljbruce
danieljbruce requested a review from a team as a code owner July 31, 2026 18:52

@westarle westarle left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Can you please share the script or commands used to make this change in the PR description?

@danieljbruce

Copy link
Copy Markdown
Contributor Author

Can you please share the script or commands used to make this change in the PR description?

There was a lot of back and forth in gemini, but I think I landed on script and then gemini applied bdc816a1a9fa41cfa5d625d8fd048d06cf7f0f5a commit to drop the auth changes.

@danieljbruce
danieljbruce requested a review from westarle July 31, 2026 20:23
@danieljbruce
danieljbruce merged commit 1ed703f into main Jul 31, 2026
34 checks passed
@danieljbruce
danieljbruce deleted the node-22-bump-generator-version branch July 31, 2026 20:43
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