Skip to content

fix(nextjs): multi-version support compliance (NXC-4395)#35870

Merged
leosvelperez merged 2 commits into
masterfrom
nxc-4395-next-multi-version-compliance
Jun 5, 2026
Merged

fix(nextjs): multi-version support compliance (NXC-4395)#35870
leosvelperez merged 2 commits into
masterfrom
nxc-4395-next-multi-version-compliance

Conversation

@jaysoo

@jaysoo jaysoo commented Jun 3, 2026

Copy link
Copy Markdown
Member

Current Behavior

@nx/next has no below-floor guard; generators silently fall back to latest install constants on unsupported Next versions. The base 20.7.1-beta.0 migration bumps eslint-config-next ungated, hitting v14 users.

Expected Behavior

assertSupportedNextVersion (floor Next 14) throws on sub-floor and is the first statement in every generator. The base 20.7.1-beta.0 migration is gated to >=15 so v14 users keep the v14 eslint-config lane. Support window kept at v14+v15+v16.

Related Issue(s)

Fixes NXC-4395


View session information ↗

@netlify

netlify Bot commented Jun 3, 2026

Copy link
Copy Markdown

Deploy Preview for nx-docs ready!

Name Link
🔨 Latest commit f6414c0
🔍 Latest deploy log https://app.netlify.com/projects/nx-docs/deploys/6a218acb961bd00008c563e3
😎 Deploy Preview https://deploy-preview-35870--nx-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify

netlify Bot commented Jun 3, 2026

Copy link
Copy Markdown

Deploy Preview for nx-dev ready!

Name Link
🔨 Latest commit f6414c0
🔍 Latest deploy log https://app.netlify.com/projects/nx-dev/deploys/6a218acb2ab28e0008c176c0
😎 Deploy Preview https://deploy-preview-35870--nx-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@nx-cloud

nx-cloud Bot commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

View your CI Pipeline Execution ↗ for commit 0575995

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-c... ✅ Succeeded 17m 23s View ↗
nx run-many -t check-imports check-lock-files c... ✅ Succeeded 3s View ↗
nx-cloud record -- pnpm nx-cloud conformance:check ✅ Succeeded 1m 2s View ↗
nx build workspace-plugin ✅ Succeeded <1s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded 18s View ↗
nx-cloud record -- nx format:check ✅ Succeeded 6s View ↗

☁️ Nx Cloud last updated this comment at 2026-06-04 14:47:27 UTC

@jaysoo jaysoo force-pushed the nxc-4395-next-multi-version-compliance branch from 25106e3 to ca93ea9 Compare June 3, 2026 20:04
nx-cloud[bot]

This comment was marked as outdated.

nx-cloud[bot]

This comment was marked as outdated.

@jaysoo jaysoo force-pushed the nxc-4395-next-multi-version-compliance branch from 01b9d60 to 4c4698b Compare June 3, 2026 22:18
@jaysoo jaysoo changed the title fix(next): multi-version support compliance (NXC-4395) fix(nextjs): multi-version support compliance (NXC-4395) Jun 3, 2026
@jaysoo jaysoo force-pushed the nxc-4395-next-multi-version-compliance branch from 4c4698b to 14600f8 Compare June 4, 2026 02:47
nx-cloud[bot]

This comment was marked as outdated.

@jaysoo jaysoo marked this pull request as ready for review June 4, 2026 12:06
@jaysoo jaysoo requested a review from a team as a code owner June 4, 2026 12:06
@jaysoo jaysoo requested a review from MaxKless June 4, 2026 12:06
@jaysoo jaysoo force-pushed the nxc-4395-next-multi-version-compliance branch 2 times, most recently from 50ea8d3 to 00302f5 Compare June 4, 2026 12:12
nx-cloud[bot]

This comment was marked as outdated.

## Current Behavior
- @nx/next has no below-floor guard; generators silently fall back to latest install constants on unsupported Next versions.
- Base 20.7.1-beta.0 migration bumps eslint-config-next ungated, hitting v14 users.

## Expected Behavior
- assertSupportedNextVersion (floor Next 14) throws on sub-floor; asserted as first statement in every generator.
- Base 20.7.1-beta.0 migration gated to ^15.0.0 so v14 users keep the v14 eslint-config lane.
- Inferred plugin TODO replaced with factual comment; v14/v15/v16 emit identical inferred target.
- Generator init/style/swc installs default keepExistingVersions true (preserves user-pinned deps).
- Fresh-install pins raised to lowest CVE-free patch per major (next 14.2.35 / 15.5.18 / 16.1.6).

## Related Issue(s)
Fixes NXC-4395
@jaysoo jaysoo force-pushed the nxc-4395-next-multi-version-compliance branch from 2c22b74 to 0575995 Compare June 4, 2026 13:37

@nx-cloud nx-cloud 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.

Nx Cloud has identified a flaky task in your failed CI:

🔂 Since the failure was identified as flaky, we triggered a CI rerun by adding an empty commit to this branch.

Nx Cloud View detailed reasoning in Nx Cloud ↗


🎓 Learn more about Self-Healing CI on nx.dev

@leosvelperez leosvelperez merged commit 925eda1 into master Jun 5, 2026
18 checks passed
@leosvelperez leosvelperez deleted the nxc-4395-next-multi-version-compliance branch June 5, 2026 07:57
vrxj81 pushed a commit to vrxj81/nx that referenced this pull request Jun 7, 2026
## Current Behavior

`@nx/next` has no below-floor guard; generators silently fall back to
latest install constants on unsupported Next versions. The base
`20.7.1-beta.0` migration bumps `eslint-config-next` ungated, hitting
v14 users.

## Expected Behavior

`assertSupportedNextVersion` (floor Next 14) throws on sub-floor and is
the first statement in every generator. The base `20.7.1-beta.0`
migration is gated to `>=15` so v14 users keep the v14 eslint-config
lane. Support window kept at v14+v15+v16.

## Related Issue(s)

Fixes NXC-4395

<!-- polygraph-session-start -->
---
[View session information
↗](https://snapshot.app.trypolygraph.com/orgs/69cdc268b6aa527e4129c2b4/sessions/multi-4395-ae050ce9)
<!-- polygraph-session-end -->

---------

Co-authored-by: nx-cloud[bot] <71083854+nx-cloud[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants