Skip to content

feat: add upm package manager support - #1597

Open
iiio2 wants to merge 1 commit into
nuxt:mainfrom
iiio2:feat/ump-add-feature
Open

iiio2 wants to merge 1 commit into
nuxt:mainfrom
iiio2:feat/ump-add-feature

Conversation

@iiio2

@iiio2 iiio2 commented Oct 5, 2026 •

Copy link
Copy Markdown

package-manager-detector new update has upm.

@iiio2
iiio2 requested a review from danielroe as a code owner October 5, 2026 16:28
@pkg-pr-new

pkg-pr-new Bot commented Oct 5, 2026

Copy link
Copy Markdown
  • nuxt-cli-playground

    npm i https://pkg.pr.new/create-nuxt@1597
    
    npm i https://pkg.pr.new/nuxi@1597
    
    npm i https://pkg.pr.new/@nuxt/cli@1597
    

commit: dbd3c8f

@socket-security

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedpackage-manager-detector@​1.9.010010010086100

View full report

@github-actions

github-actions Bot commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

CLI benchmark

@nuxt/cli v4.0.0-alpha.1 (baseline) vs v4.0.0-alpha.1 (this PR)

Metric baseline v4.0.0-alpha.1 head v4.0.0-alpha.1 Delta
nuxt --version wall time (median) 67 ms 66 ms -0.3%
nuxt --help wall time (median) 142 ms 142 ms -0.2%
nuxt dev --help wall time (median) 107 ms 105 ms -1.4%
nuxt --version modules loaded 35 35 0.0%
nuxt --version built-ins loaded 27 27 0.0%
nuxt --help modules loaded 135 135 0.0%
nuxt --help built-ins loaded 87 87 0.0%
nuxt dev --help modules loaded 64 64 0.0%
nuxt dev --help built-ins loaded 87 87 0.0%
Installed node_modules 2.46 MB 2.46 MB -0.0%
Published tarball (packed) 240.1 kB 240.1 kB +0.0%
Full report

@nuxt/cli v4.0.0-alpha.1 (baseline) vs v4.0.0-alpha.1 (head)

Setting Value
Baseline ref:df61b3990a91ef0309180423b58ea0c102319081 (v4.0.0-alpha.1)
Head local packages/nuxt-cli at 4a767f0 (v4.0.0-alpha.1)
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 2.13, 0.72, 0.26
Run started 2026-10-05T16:29:50.691Z

Cold CLI startup

Median of 15 interleaved runs per command, one warmup discarded.

Command baseline v4.0.0-alpha.1 median head v4.0.0-alpha.1 median Delta baseline v4.0.0-alpha.1 min / p95 head v4.0.0-alpha.1 min / p95
nuxt --version 67 ms 66 ms -0.3% 63 ms / 70 ms 64 ms / 71 ms
nuxt --version (first output byte) 62 ms 62 ms -0.4% 59 ms / 66 ms 59 ms / 67 ms
nuxt --help 142 ms 142 ms -0.2% 138 ms / 147 ms 137 ms / 150 ms
nuxt --help (first output byte) 137 ms 137 ms -0.0% 133 ms / 141 ms 132 ms / 145 ms
nuxt dev --help 107 ms 105 ms -1.4% 103 ms / 113 ms 101 ms / 110 ms
nuxt dev --help (first output byte) 101 ms 100 ms -1.1% 98 ms / 108 ms 96 ms / 105 ms
nuxt <unknown-command> (no-op) 151 ms 150 ms -1.0% 146 ms / 159 ms 147 ms / 154 ms
nuxt <unknown-command> (no-op) (first output byte) 145 ms 144 ms -0.8% 141 ms / 153 ms 141 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-alpha.1 modules head v4.0.0-alpha.1 modules Delta baseline v4.0.0-alpha.1 source bytes head v4.0.0-alpha.1 source bytes Delta baseline v4.0.0-alpha.1 built-ins head v4.0.0-alpha.1 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.4 kB 847.4 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-alpha.1 head v4.0.0-alpha.1 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.29 s 1.29 s +0.3%
Published tarball (packed) 240.1 kB 240.1 kB +0.0%
Published tarball (unpacked) 780.2 kB 780.2 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-commenter

codecov-commenter commented Oct 5, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (main@df61b39). Learn more about missing BASE report.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1597   +/-   ##
=======================================
  Coverage        ?   83.99%           
=======================================
  Files           ?      177           
  Lines           ?    11478           
  Branches        ?     3299           
=======================================
  Hits            ?     9641           
  Misses          ?     1546           
  Partials        ?      291           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@codspeed

codspeed Bot commented Oct 5, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 2 untouched benchmarks


Comparing iiio2:feat/ump-add-feature (dbd3c8f) with main (df61b39)

Open in CodSpeed

@coderabbitai

coderabbitai Bot commented Oct 5, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 04080b50-51ab-4006-909a-0f28c941ce3b
📥 Commits

Reviewing files that changed from the base of the PR and between df61b39 and dbd3c8f.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (8)
  • docs/add.md
  • docs/init.md
  • docs/module.md
  • packages/create-nuxt/package.json
  • packages/nuxt-cli/package.json
  • packages/nuxt-cli/test/unit/help.spec.ts
  • packages/nuxt-cli/test/unit/utils/package-managers.spec.ts
  • pnpm-workspace.yaml

Included review availability: This review used your included allowance. Your plan provides up to 2 included reviews per hour; 1 remain after this review.


📝 Walkthrough

Walkthrough

The package-manager detector dependency ranges are updated to version 1.9.0. Tests cover detection of an upm.lock file and the lock-file list for upm. CLI help snapshots and documentation list upm as a supported --packageManager value.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to dbd3c

This adds UPM as a recognized package manager by upgrading the detector library and updating documentation and help output. The new detector version is published and locked, and tests cover UPM detection. No merge-blocking risk was identified.

Architecture Summary

Architecture risk: 🔵 Low · up to dbd3c

The change affects 4 systems.

Changed systems: docs, packages/nuxt-cli, packages/create-nuxt, pnpm-workspace.yaml

Architecture concerns
No architecture-level concerns identified.

Review details

Systems and components

  • observed — docs (service) was modified; 3 changed files map to changed impact.
  • observed — packages/nuxt-cli (library) was modified; 3 changed files map to changed impact.
  • observed — packages/create-nuxt (library) was modified; 1 changed file maps to changed impact.
  • observed — pnpm-workspace.yaml (service) was modified; 1 changed file maps to changed impact.

Before / after behavior

  • observed — Modified behavior in docs/add.md: The usage syntax adds upm to the accepted --packageManager values.
  • observed — Modified behavior in docs/add.md: The options table adds upm to the package managers listed for --packageManager.
  • observed — Modified behavior in docs/init.md: The command synopsis adds upm to the listed --packageManager choices.
  • observed — Modified behavior in docs/init.md: The options table adds upm to the --packageManager choices; the other displayed option entries are unchanged.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 2…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly identifies the main change: adding support for the upm package manager.
Description check ✅ Passed The description relates to the changeset by noting that an update to package-manager-detector adds upm support.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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.

2 participants