Skip to content

fix(esbuild): update peerDep range#30402

Merged
FrozenPandaz merged 2 commits intonrwl:masterfrom
hendrickson-tyler:master
May 14, 2025
Merged

fix(esbuild): update peerDep range#30402
FrozenPandaz merged 2 commits intonrwl:masterfrom
hendrickson-tyler:master

Conversation

@hendrickson-tyler
Copy link
Copy Markdown
Contributor

@hendrickson-tyler hendrickson-tyler commented Mar 17, 2025

Current Behavior

esbuild has a security advisory for versions older than 0.25.0. @nx/esbuild does not allow versions greater than 0.19.2 due to the range specified in peer dependencies.

A fix for this was attempted in #30167, but it still does not allow any versions greater than 0.19.x due to the way 0.x.x releases are handled.

Expected Behavior

@nx/esbuild allows any version from the current 0.19.2 onwards, not including a future 1.x.x version. This will allow non-vulnerable versions of esbuild to be used.

Related Issue(s)

Fixes #30009
Fixes #30370

@hendrickson-tyler hendrickson-tyler requested a review from a team as a code owner March 17, 2025 19:36
@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 17, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
nx-dev ✅ Ready (Inspect) Visit Preview May 14, 2025 3:52pm

@cjlawson02
Copy link
Copy Markdown

Hi @FrozenPandaz, can we get a review on this please? Thanks!

@xavier-symbe
Copy link
Copy Markdown

A review would be much appreciated - the security advisory has been out for over 3 months now.

@FrozenPandaz FrozenPandaz added the priority: high High Priority (important issues which affect many people severely) label May 14, 2025
@FrozenPandaz FrozenPandaz enabled auto-merge (squash) May 14, 2025 14:10
@FrozenPandaz
Copy link
Copy Markdown
Contributor

Thank you for your contribution! LGTM 🎉

@nx-cloud
Copy link
Copy Markdown
Contributor

nx-cloud Bot commented May 14, 2025

View your CI Pipeline Execution ↗ for commit 2048198.

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-c... ✅ Succeeded 14m 20s View ↗
nx run-many -t check-imports check-commit check... ✅ Succeeded 16s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 2s View ↗
nx-cloud record -- nx format:check ✅ Succeeded 2s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded 1s View ↗
nx documentation ✅ Succeeded 46s View ↗

☁️ Nx Cloud last updated this comment at 2025-05-14 15:59:07 UTC

@FrozenPandaz FrozenPandaz merged commit 21bdb35 into nrwl:master May 14, 2025
6 checks passed
FrozenPandaz pushed a commit that referenced this pull request May 16, 2025
## Current Behavior
`esbuild` has a [security
advisory](GHSA-67mh-4wv8-2f99) for
versions older than 0.25.0. `@nx/esbuild` does not allow versions
greater than 0.19.2 due to the range specified in peer dependencies.

A fix for this was attempted in #30167, but it still does not allow any
versions greater than 0.19.x due to the way [0.x.x releases are
handled](https://github.com/npm/node-semver#caret-ranges-123-025-004).

## Expected Behavior
`@nx/esbuild` allows any version from the current 0.19.2 onwards, not
including a future 1.x.x version. This will allow non-vulnerable
versions of `esbuild` to be used.

## Related Issue(s)
Fixes #30009
Fixes #30370

(cherry picked from commit 21bdb35)
FrozenPandaz pushed a commit that referenced this pull request May 16, 2025
## Current Behavior
`esbuild` has a [security
advisory](GHSA-67mh-4wv8-2f99) for
versions older than 0.25.0. `@nx/esbuild` does not allow versions
greater than 0.19.2 due to the range specified in peer dependencies.

A fix for this was attempted in #30167, but it still does not allow any
versions greater than 0.19.x due to the way [0.x.x releases are
handled](https://github.com/npm/node-semver#caret-ranges-123-025-004).

## Expected Behavior
`@nx/esbuild` allows any version from the current 0.19.2 onwards, not
including a future 1.x.x version. This will allow non-vulnerable
versions of `esbuild` to be used.

## Related Issue(s)
Fixes #30009
Fixes #30370

(cherry picked from commit 21bdb35)
@github-actions
Copy link
Copy Markdown
Contributor

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators May 20, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

priority: high High Priority (important issues which affect many people severely)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

esbuild vulnerability Update esbuild version

5 participants