Skip to content

build: update dependency undici to v8 (main)#32933

Open
angular-robot wants to merge 1 commit intoangular:mainfrom
angular-robot:ng-renovate/main-undici-8-x
Open

build: update dependency undici to v8 (main)#32933
angular-robot wants to merge 1 commit intoangular:mainfrom
angular-robot:ng-renovate/main-undici-8-x

Conversation

@angular-robot
Copy link
Copy Markdown
Contributor

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
undici (source) 7.24.68.0.0 age adoption passing confidence

  • If you want to rebase/retry this PR, check this box

Release Notes

nodejs/undici (undici)

v8.0.0

Compare Source

What's Changed

Full Changelog: nodejs/undici@v7.24.7...v8.0.0

v7.24.7

Compare Source

What's Changed
New Contributors

Full Changelog: nodejs/undici@v7.24.6...v7.24.7

See associated pull request for more information.
@angular-robot angular-robot added action: merge The PR is ready for merge by the caretaker area: build & ci Related the build and CI infrastructure of the project target: automation This PR is targeted to only merge into the branch defined in Github [bot use only] labels Apr 3, 2026
Copy link
Copy Markdown

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

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 undici dependency from version 7.24.6 to 8.0.0 across the project. Review feedback highlights that this major version upgrade increases the minimum Node.js requirement to 22.19.0, effectively dropping support for Node.js 18 and 20. To maintain compatibility with these LTS versions, it is recommended to use version 7.24.7 instead.

"tslib": "2.8.1",
"typescript": "6.0.2",
"undici": "7.24.6",
"undici": "8.0.0",
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

high

Updating undici to v8.0.0 is a major breaking change. Most notably, it increases the minimum Node.js requirement to >=22.19.0 (as evidenced in the lockfile), which drops support for current LTS versions like Node.js 18 and 20. Additionally, HTTP/2 is now enabled by default, which may cause connectivity issues in environments with incompatible proxies or legacy servers. If maintaining compatibility with older Node.js LTS versions is required, consider updating to v7.24.7 instead.

Suggested change
"undici": "8.0.0",
"undici": "7.24.7",

"browser-sync": "3.0.4",
"ng-packagr": "22.0.0-next.1",
"undici": "7.24.6"
"undici": "8.0.0"
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

high

This update to undici v8.0.0 introduces a minimum Node.js requirement of >=22.19.0. This will break compatibility for users on Node.js 18 or 20. If this package is intended to be used in environments still running these LTS versions, it is recommended to stay on the v7 branch.

Suggested change
"undici": "8.0.0"
"undici": "7.24.7"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: build & ci Related the build and CI infrastructure of the project target: automation This PR is targeted to only merge into the branch defined in Github [bot use only]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant