test: update snapshots for upm - #1597
Conversation
commit: |
CLI benchmark
Full report
|
| Setting | Value |
|---|---|
| Baseline | ref:eec52f07d01cb8384ffddd2e4492268ce2203b81 (v4.0.0) |
| Head | local packages/nuxt-cli at 6d364dc (v4.0.0) |
| Node | v24.21.0 |
| OS | Linux 6.17.0 (kernel 6.17.0-1022-azure) |
| CPU | AMD EPYC 7763 64-Core Processor x 4 |
| Memory | 15.6 GB |
| Load average at start | 1.90, 0.71, 0.27 |
| Run started | 2026-10-06T12:14:12.048Z |
Cold CLI startup
Median of 15 interleaved runs per command, one warmup discarded.
| Command | baseline v4.0.0 median | head v4.0.0 median | Delta | baseline v4.0.0 min / p95 | head v4.0.0 min / p95 |
|---|---|---|---|---|---|
nuxt --version |
66 ms | 66 ms | +0.4% | 63 ms / 68 ms | 64 ms / 69 ms |
nuxt --version (first output byte) |
61 ms | 62 ms | +0.9% | 59 ms / 64 ms | 59 ms / 65 ms |
nuxt --help |
141 ms | 141 ms | +0.2% | 138 ms / 145 ms | 136 ms / 146 ms |
nuxt --help (first output byte) |
135 ms | 136 ms | +0.4% | 132 ms / 139 ms | 131 ms / 141 ms |
nuxt dev --help |
107 ms | 104 ms | -2.2% | 102 ms / 110 ms | 102 ms / 108 ms |
nuxt dev --help (first output byte) |
102 ms | 99 ms | -2.2% | 97 ms / 105 ms | 97 ms / 103 ms |
nuxt <unknown-command> (no-op) |
150 ms | 151 ms | +0.6% | 146 ms / 154 ms | 148 ms / 154 ms |
nuxt <unknown-command> (no-op) (first output byte) |
144 ms | 145 ms | +0.3% | 140 ms / 148 ms | 142 ms / 148 ms |
Module load cost
Counted with a module.registerHooks load hook, compile cache disabled. Counts every JS module actually evaluated on that code path (native addons excluded). Built-ins loaded after bootstrap are counted separately, including the internal modules they load.
| Command | baseline v4.0.0 modules | head v4.0.0 modules | Delta | baseline v4.0.0 source bytes | head v4.0.0 source bytes | Delta | baseline v4.0.0 built-ins | head v4.0.0 built-ins | Delta |
|---|---|---|---|---|---|---|---|---|---|
nuxt --version |
35 | 35 | 0.0% | 297.8 kB | 297.8 kB | 0.0% | 27 | 27 | 0.0% |
nuxt --help |
135 | 135 | 0.0% | 847.1 kB | 847.1 kB | 0.0% | 87 | 87 | 0.0% |
nuxt dev --help |
64 | 64 | 0.0% | 455.3 kB | 455.3 kB | 0.0% | 87 | 87 | 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 | head v4.0.0 | Delta |
|---|---|---|---|
Direct dependencies of @nuxt/cli |
23 | 23 | 0.0% |
| Packages in the installed tree (unique name@version) | 39 | 39 | 0.0% |
| Unique package names | 39 | 39 | 0.0% |
| Package directories on disk (cross-check) | 32 | 32 | 0.0% |
Installed node_modules on disk |
2.46 MB | 2.46 MB | -0.0% |
| Installed files | 435 | 435 | 0.0% |
| Install wall time (warm npm cache, median of 3) | 1.31 s | 1.31 s | -0.1% |
| Published tarball (packed) | 240.2 kB | 240.2 kB | 0.0% |
| Published tarball (unpacked) | 780.0 kB | 780.0 kB | 0.0% |
| Files in tarball | 100 | 100 | 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.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1597 +/- ##
=======================================
Coverage ? 83.99%
=======================================
Files ? 177
Lines ? 11475
Branches ? 3297
=======================================
Hits ? 9638
Misses ? 1546
Partials ? 291 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 22 minutes. View limit detailsLimit details: You’ve used all 2 included reviews currently available. Review configuration: ⚙️ Run configuration
📒 Files selected for processing (1)
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configuration
⛔ Files ignored due to path filters (4)
📒 Files selected for processing (5)
💤 Files with no reviewable changes (1)
Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe package-manager detector dependency ranges are updated to version 1.9.0. Tests cover detection of an Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to The Architecture SummaryArchitecture risk: 🔵 Low · up to The change affects 4 systems. Changed systems: Architecture concerns Review detailsSystems and components
Before / after behavior
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 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 |
danielroe
left a comment
There was a problem hiding this comment.
there's no need for a PR or tests 🙏
the logic here is entirely in package-manager-detector, by design, and we normally manage dependency bumps through renovate
upm
d51be26 to
955a376
Compare
|
Thanks. 😊 |
package-manager-detector new update has upm.