fix(pm): accept positional package names for vp rebuild

vp rebuild previously only accepted -- pass-through args, so running
vp rebuild better-sqlite3 errored with "Unexpected argument". Combined
with pnpm v10+ only rebuilding packages whose build scripts are approved,
bare vp rebuild was a no-op for native modules like better-sqlite3 (71ms
vs 11.7s for pnpm rebuild better-sqlite3) and the user had no
discoverable way to name a package.

Add a packages: Vec<String> positional to the Rebuild clap variant,
matching what the RFC (rfcs/pm-command-group.md §17, lines 660-680 and
mapping table at 1283-1304) already specifies. Forward the names to
pnpm/npm rebuild before the pass-through args. Yarn/Bun paths are
unchanged (warn-and-skip).

Snap test packages/cli/snap-tests-global/command-rebuild-pnpm11 captures
both the help output and the previously-broken positional form.

Author: fengmk2

crates/vite_install/src/commands/rebuild.rs

@@ -10,8 +10,9 @@ use crate::package_manager::{
 };
 
 /// Options for the rebuild command.
-#[derive(Debug)]
+#[derive(Debug, Default)]
 pub struct RebuildCommandOptions<'a> {
+    pub packages: &'a [String],
     pub pass_through_args: Option<&'a [String]>,
 }
 
@@ -47,10 +48,12 @@ impl PackageManager {
             PackageManagerType::Npm => {
                 bin_name = "npm".into();
                 args.push("rebuild".into());
+                args.extend_from_slice(options.packages);
             }
             PackageManagerType::Pnpm => {
                 bin_name = "pnpm".into();
                 args.push("rebuild".into());
+                args.extend_from_slice(options.packages);
             }
             PackageManagerType::Yarn => {
                 let is_yarn1 = self.version.starts_with("1.");
@@ -110,7 +113,7 @@ mod tests {
     #[test]
     fn test_npm_rebuild() {
         let pm = create_mock_package_manager(PackageManagerType::Npm, "11.0.0");
-        let result = pm.resolve_rebuild_command(&RebuildCommandOptions { pass_through_args: None });
+        let result = pm.resolve_rebuild_command(&RebuildCommandOptions::default());
         assert!(result.is_some());
         let result = result.unwrap();
         assert_eq!(result.bin_path, "npm");
@@ -120,7 +123,7 @@ mod tests {
     #[test]
     fn test_pnpm_rebuild() {
         let pm = create_mock_package_manager(PackageManagerType::Pnpm, "10.0.0");
-        let result = pm.resolve_rebuild_command(&RebuildCommandOptions { pass_through_args: None });
+        let result = pm.resolve_rebuild_command(&RebuildCommandOptions::default());
         assert!(result.is_some());
         let result = result.unwrap();
         assert_eq!(result.bin_path, "pnpm");
@@ -130,14 +133,54 @@ mod tests {
     #[test]
     fn test_yarn1_rebuild_not_supported() {
         let pm = create_mock_package_manager(PackageManagerType::Yarn, "1.22.0");
-        let result = pm.resolve_rebuild_command(&RebuildCommandOptions { pass_through_args: None });
+        let result = pm.resolve_rebuild_command(&RebuildCommandOptions::default());
         assert!(result.is_none());
     }
 
     #[test]
     fn test_yarn2_rebuild_not_supported() {
         let pm = create_mock_package_manager(PackageManagerType::Yarn, "4.0.0");
-        let result = pm.resolve_rebuild_command(&RebuildCommandOptions { pass_through_args: None });
+        let result = pm.resolve_rebuild_command(&RebuildCommandOptions::default());
         assert!(result.is_none());
     }
+
+    #[test]
+    fn test_npm_rebuild_with_packages() {
+        let pm = create_mock_package_manager(PackageManagerType::Npm, "11.0.0");
+        let packages = vec!["better-sqlite3".to_string(), "sharp".to_string()];
+        let result = pm.resolve_rebuild_command(&RebuildCommandOptions {
+            packages: &packages,
+            ..Default::default()
+        });
+        let result = result.unwrap();
+        assert_eq!(result.bin_path, "npm");
+        assert_eq!(result.args, vec!["rebuild", "better-sqlite3", "sharp"]);
+    }
+
+    #[test]
+    fn test_pnpm_rebuild_with_packages() {
+        let pm = create_mock_package_manager(PackageManagerType::Pnpm, "10.0.0");
+        let packages = vec!["better-sqlite3".to_string()];
+        let result = pm.resolve_rebuild_command(&RebuildCommandOptions {
+            packages: &packages,
+            ..Default::default()
+        });
+        let result = result.unwrap();
+        assert_eq!(result.bin_path, "pnpm");
+        assert_eq!(result.args, vec!["rebuild", "better-sqlite3"]);
+    }
+
+    #[test]
+    fn test_pnpm_rebuild_with_packages_and_pass_through() {
+        let pm = create_mock_package_manager(PackageManagerType::Pnpm, "11.0.6");
+        let packages = vec!["better-sqlite3".to_string()];
+        let pass_through = vec!["--recursive".to_string()];
+        let result = pm.resolve_rebuild_command(&RebuildCommandOptions {
+            packages: &packages,
+            pass_through_args: Some(&pass_through),
+        });
+        let result = result.unwrap();
+        assert_eq!(result.bin_path, "pnpm");
+        assert_eq!(result.args, vec!["rebuild", "better-sqlite3", "--recursive"]);
+    }
 }

crates/vite_pm_cli/src/cli.rs

@@ -860,6 +860,9 @@ pub enum PmCommands {
     /// Rebuild native modules
     #[command(visible_alias = "rb")]
     Rebuild {
+        /// Packages to rebuild (rebuilds all if omitted; with pnpm v10+, bare rebuild only acts on packages whose build scripts are approved)
+        packages: Vec<String>,
+
         /// Additional arguments
         #[arg(last = true, allow_hyphen_values = true)]
         pass_through_args: Option<Vec<String>>,

crates/vite_pm_cli/src/handlers.rs

@@ -452,8 +452,11 @@ pub async fn run_pm_subcommand(
             Ok(pm.run_search_command(&options, cwd).await?)
         }
 
-        PmCommands::Rebuild { pass_through_args } => {
-            let options = RebuildCommandOptions { pass_through_args: pass_through_args.as_deref() };
+        PmCommands::Rebuild { packages, pass_through_args } => {
+            let options = RebuildCommandOptions {
+                packages: &packages,
+                pass_through_args: pass_through_args.as_deref(),
+            };
             Ok(pm.run_rebuild_command(&options, cwd).await?)
         }
 

docs/guide/install.md

@@ -121,15 +121,19 @@ Use these when you need to understand the current state of dependencies.
 Use `vp rebuild` when native modules need to be recompiled, for example after switching Node.js versions or when a C/C++ addon fails to load.
 
 - `vp rebuild` rebuilds all native modules
+- `vp rebuild <package...>` rebuilds the listed packages only
 - `vp rebuild -- <args>` passes extra arguments to the underlying package manager
 
 ```bash
 vp rebuild
+vp rebuild better-sqlite3 sharp
 vp rebuild -- --update-binary
 ```
 
 `vp rebuild` is a shorthand for `vp pm rebuild`.
 
+With pnpm v10+, bare `vp rebuild` only rebuilds packages whose build scripts are listed in `onlyBuiltDependencies` (or approved via `pnpm approve-builds`); name the package explicitly to force a rebuild that bypasses the approval gate.
+
 #### Advanced
 
 Use these when you need lower-level package-manager behavior.

packages/cli/snap-tests-global/command-rebuild-pnpm11/package.json

@@ -0,0 +1,9 @@
+{
+  "name": "command-rebuild-pnpm11",
+  "version": "1.0.0",
+  "private": true,
+  "dependencies": {
+    "testnpm2": "1.0.0"
+  },
+  "packageManager": "pnpm@11.0.6"
+}

packages/cli/snap-tests-global/command-rebuild-pnpm11/snap.txt

@@ -0,0 +1,28 @@
+> vp rebuild --help # should show help with [PACKAGES]... positional
+Usage: vp pm rebuild [PACKAGES]... [-- <PASS_THROUGH_ARGS>...]
+
+Rebuild native modules
+
+Arguments:
+  [PACKAGES]...           Packages to rebuild (rebuilds all if omitted; with pnpm v10+, bare rebuild only acts on packages whose build scripts are approved)
+  [PASS_THROUGH_ARGS]...  Additional arguments
+
+Options:
+  -h, --help  Print help
+
+Documentation: https://viteplus.dev/guide/install
+
+
+> vp install # set up node_modules
+Packages: +<variable>
++
+Progress: resolved <variable>, reused <variable>, downloaded <variable>, added <variable>, done
+
+dependencies:
++ testnpm2 <semver> (1.0.1 is available)
+
+Done in <variable>ms using pnpm v<semver>
+
+> vp rebuild # bare rebuild, no args
+> vp rebuild testnpm2 # should accept positional package name
+> vp rebuild testnpm2 -- --recursive # package name + pass-through args

packages/cli/snap-tests-global/command-rebuild-pnpm11/steps.json

@@ -0,0 +1,10 @@
+{
+  "ignoredPlatforms": ["win32"],
+  "commands": [
+    "vp rebuild --help # should show help with [PACKAGES]... positional",
+    "vp install # set up node_modules",
+    "vp rebuild # bare rebuild, no args",
+    "vp rebuild testnpm2 # should accept positional package name",
+    "vp rebuild testnpm2 -- --recursive # package name + pass-through args"
+  ]
+}