Skip to content

feat(nuxt): Detect the server API with @nuxt/kit 4.6 - #25138

Open
s1gr1d wants to merge 1 commit into
developfrom
sig/nuxt-server-api-detection
Open

s1gr1d wants to merge 1 commit into
developfrom
sig/nuxt-server-api-detection

Conversation

@s1gr1d

@s1gr1d s1gr1d commented Oct 7, 2026 •

Copy link
Copy Markdown
Member

Bumps @nuxt/kit to ^4.6.0 for resolveServerVariant. The Vite server and other non-Nitro builders need this server-API detection.

Looks like a lot of code changes, but most of it is just moved code. Those are the important parts:

  • resolveServerApi() replaces getNitroMajorVersion and the local-pkg dependency (we can get the nitro version from there). This is actually the most important and biggest change in the PR.
  • On the nuxt server API (no Nitro), the module skips all Nitro setup. With this, Kit prints no NUXT_B8024 warnings
  • addServerPlugin → addNitroPlugin. Kit 4.6 deprecates the old name, and the behavior is the same
  • all nitro-related code moves to nitroServer.ts

Part of #25072

@s1gr1d
s1gr1d requested a review from a team as a code owner October 7, 2026 14:46
@s1gr1d
s1gr1d requested review from mydea and nicohrubec and removed request for a team October 7, 2026 14:46

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This is just copy-pasted code from the module.ts file

@s1gr1d
s1gr1d requested review from chargome and logaretm October 7, 2026 14:47
@s1gr1d

s1gr1d commented Oct 7, 2026

Copy link
Copy Markdown
Member Author

bugbot run

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 51cc215. Configure here.

This branch has not been deployed

No deployments
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.

1 participant