fix(analyze,build,dev,preview): server build descriptor with nitro fallback - #1509
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (3)
Included review availability: Your plan provides up to 2 included reviews per hour; 1 remains after this review. 📝 WalkthroughWalkthroughThe CLI now normalizes declared server builder names by removing the Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to The change improves server-build metadata resolution by falling back to Nitro values when descriptor fields are missing while preserving explicit overrides. No actionable merge-blocking risk remains. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
commit: |
CLI benchmark
Full report
|
| Setting | Value |
|---|---|
| Baseline | ref:11fc0abdd3c0f4dcd4a5cd60ec2885fa662335df (v4.0.0-alpha.0) |
| Head | local packages/nuxt-cli at efd69c6 (v4.0.0-alpha.0) |
| Node | v24.19.0 |
| OS | Linux 6.17.0 (kernel 6.17.0-1022-azure) |
| CPU | AMD EPYC 9V74 80-Core Processor x 4 |
| Memory | 15.6 GB |
| Load average at start | 2.11, 0.64, 0.22 |
| Run started | 2026-08-31T07:20:19.619Z |
Cold CLI startup
Median of 15 interleaved runs per command, one warmup discarded.
| Command | baseline v4.0.0-alpha.0 median | head v4.0.0-alpha.0 median | Delta | baseline v4.0.0-alpha.0 min / p95 | head v4.0.0-alpha.0 min / p95 |
|---|---|---|---|---|---|
nuxt --version |
66 ms | 66 ms | -1.0% | 64 ms / 68 ms | 64 ms / 69 ms |
nuxt --version (first output byte) |
62 ms | 61 ms | -1.5% | 60 ms / 64 ms | 60 ms / 65 ms |
nuxt --help |
138 ms | 138 ms | -0.3% | 134 ms / 141 ms | 135 ms / 141 ms |
nuxt --help (first output byte) |
133 ms | 132 ms | -0.4% | 128 ms / 136 ms | 129 ms / 135 ms |
nuxt dev --help |
107 ms | 104 ms | -2.6% | 104 ms / 118 ms | 102 ms / 117 ms |
nuxt dev --help (first output byte) |
102 ms | 99 ms | -2.5% | 99 ms / 111 ms | 97 ms / 112 ms |
nuxt <unknown-command> (no-op) |
147 ms | 147 ms | +0.3% | 144 ms / 150 ms | 145 ms / 150 ms |
nuxt <unknown-command> (no-op) (first output byte) |
140 ms | 142 ms | +0.8% | 138 ms / 145 ms | 139 ms / 144 ms |
Module load cost
Counted with a module.registerHooks load hook, compile cache disabled. Counts every JS module actually evaluated on that code path (built-ins excluded, native addons excluded).
| Command | baseline v4.0.0-alpha.0 modules | head v4.0.0-alpha.0 modules | Delta | baseline v4.0.0-alpha.0 source bytes | head v4.0.0-alpha.0 source bytes | Delta |
|---|---|---|---|---|---|---|
nuxt --version |
38 | 38 | 0.0% | 298.7 kB | 298.7 kB | 0.0% |
nuxt --help |
138 | 138 | 0.0% | 1.01 MB | 1.01 MB | +0.0% |
nuxt dev --help |
81 | 81 | 0.0% | 620.1 kB | 620.3 kB | +0.0% |
Install footprint and published tarball
Each version installed on its own into an empty project with nothing but @nuxt/cli as a dependency, so the tree is exactly the CLI and its transitive dependencies. npm cache is warm and the registry is only consulted for metadata, so install wall time is indicative, not a network benchmark.
| Metric | baseline v4.0.0-alpha.0 | head v4.0.0-alpha.0 | Delta |
|---|---|---|---|
Direct dependencies of @nuxt/cli |
21 | 21 | 0.0% |
| Packages in the installed tree (unique name@version) | 34 | 34 | 0.0% |
| Unique package names | 34 | 34 | 0.0% |
| Package directories on disk (cross-check) | 28 | 28 | 0.0% |
Installed node_modules on disk |
2.38 MB | 2.38 MB | +0.0% |
| Installed files | 419 | 419 | 0.0% |
| Install wall time (warm npm cache, median of 3) | 655 ms | 654 ms | -0.1% |
| Published tarball (packed) | 284.6 kB | 284.9 kB | +0.1% |
| Published tarball (unpacked) | 983.7 kB | 983.9 kB | +0.0% |
| Files in tarball | 131 | 131 | 0.0% |
Interleaved runs on a shared runner: trust the deltas, not the absolute timings. The dev, restart and build suites run locally via pnpm bench:cli.
🔗 Linked issue
📚 Description
follow-up to #1505 - issue is in nuxt v4.6+, we always write a descriptor, meaning it no longer tells us anything except that nuxt is newer...
so this makes the descriptor a preferred source rather than an exclusive one, respecting nitro-reported values unless it's been overridden...