refactor: remove PM fallback from vp run, always delegate to vite-plus (#1179)

The `run_or_delegate` module previously fell back to `<pm> run <script>`
when vite-plus was not a project dependency. This is no longer needed
because the global CLI ships with its own vite-plus installation, and
`delegate::execute()` already resolves local vite-plus first then falls
back to the global one.

Now `vp run` and `vpr` always delegate through the vite-plus task runner,
regardless of whether the project has vite-plus as a dependency.

Closes #1132

Author: fengmk2

crates/vite_global_cli/src/cli.rs

@@ -2041,7 +2041,7 @@ pub async fn run_command_with_options(
                 return Ok(ExitStatus::default());
             }
             print_runtime_header(render_options.show_header);
-            commands::run_or_delegate::execute(cwd, &args).await
+            commands::delegate::execute(cwd, "run", &args).await
         }
 
         Commands::Exec { args } => {

crates/vite_global_cli/src/commands/mod.rs

@@ -182,7 +182,6 @@ pub mod upgrade;
 
 // Category C: Local CLI Delegation
 pub mod delegate;
-pub mod run_or_delegate;
 
 // Re-export command structs for convenient access
 pub use add::AddCommand;

crates/vite_global_cli/src/commands/run_or_delegate.rs

@@ -1,8 +1,7 @@
 //! `vpr` command implementation.
 //!
-//! Standalone shorthand for `vp run`. Executes tasks via the same
-//! run-or-delegate logic: delegates to local vite-plus CLI when
-//! vite-plus is a dependency, otherwise falls back to `<pm> run`.
+//! Standalone shorthand for `vp run`. Delegates to the local or global
+//! vite-plus CLI to execute tasks.
 
 use vite_path::AbsolutePath;
 use vite_shared::output;
@@ -16,7 +15,7 @@ pub async fn execute_vpr(args: &[String], cwd: &AbsolutePath) -> i32 {
     }
 
     let cwd_buf = cwd.to_absolute_path_buf();
-    match super::run_or_delegate::execute(cwd_buf, args).await {
+    match super::delegate::execute(cwd_buf, "run", args).await {
         Ok(status) => status.code().unwrap_or(1),
         Err(e) => {
             output::error(&e.to_string());

crates/vite_global_cli/src/commands/vpr.rs

@@ -20,7 +20,6 @@ pub mod prune;
 pub mod publish;
 pub mod rebuild;
 pub mod remove;
-pub mod run;
 pub mod search;
 pub mod token;
 pub mod unlink;

crates/vite_install/src/commands/mod.rs

@@ -1,24 +1,18 @@
-> vp run hello # should fall back to pnpm run when no vite-plus dependency
+> vp run hello # should execute via global vite-plus task runner
 VITE+ - The Unified Toolchain for the Web
 
-
-> command-run-without-vite-plus@<semver> hello <cwd>
-> echo hello from script
-
+$ echo hello from script ⊘ cache disabled
 hello from script
 
-> vp run greet --arg1 value1 # should pass through args to pnpm run
-VITE+ - The Unified Toolchain for the Web
-
 
-> command-run-without-vite-plus@<semver> greet <cwd>
-> echo greet --arg1 value1
+> vp run greet --arg1 value1 # should pass through args
+VITE+ - The Unified Toolchain for the Web
 
+$ echo greet --arg1 value1 ⊘ cache disabled
 greet --arg1 value1
 
-[1]> vp run nonexistent # should show pnpm missing script error
-VITE+ - The Unified Toolchain for the Web
 
- ERR_PNPM_NO_SCRIPT  Missing script: nonexistent
+[1]> vp run nonexistent # should show task not found error
+VITE+ - The Unified Toolchain for the Web
 
-Command "nonexistent" not found.
+Task "nonexistent" not found.

crates/vite_install/src/commands/run.rs

@@ -1,8 +1,7 @@
 {
-  "ignoredPlatforms": ["win32"],
   "commands": [
-    "vp run hello # should fall back to pnpm run when no vite-plus dependency",
-    "vp run greet --arg1 value1 # should pass through args to pnpm run",
-    "vp run nonexistent # should show pnpm missing script error"
+    "vp run hello # should execute via global vite-plus task runner",
+    "vp run greet --arg1 value1 # should pass through args",
+    "vp run nonexistent # should show task not found error"
   ]
 }

packages/cli/snap-tests-global/command-run-without-vite-plus/snap.txt

@@ -32,24 +32,18 @@ Documentation: https://viteplus.dev/guide/run
 
 
 > vpr hello # should run script via vpr shorthand
-
-> command-vpr@<semver> hello <cwd>
-> node args.mjs hello from script
-
+$ node args.mjs hello from script ⊘ cache disabled
 hello
 from
 script
 
-> vpr greet --arg1 value1 # should pass through additional args
-
-> command-vpr@<semver> greet <cwd>
-> node args.mjs greet --arg1 value1
 
+> vpr greet --arg1 value1 # should pass through additional args
+$ node args.mjs greet --arg1 value1 ⊘ cache disabled
 greet
 --arg1
 value1
 
-[1]> vpr nonexistent # should show pnpm missing script error
- ERR_PNPM_NO_SCRIPT  Missing script: nonexistent
 
-Command "nonexistent" not found.
+[1]> vpr nonexistent # should show pnpm missing script error
+Task "nonexistent" not found.

packages/cli/snap-tests-global/command-run-without-vite-plus/steps.json

@@ -6,12 +6,10 @@ VITE+ - The Unified Toolchain for the Web
 > vp run check-node # Should also use 22.12.0
 VITE+ - The Unified Toolchain for the Web
 
-
-> delegate-respects-default-node-version@<semver> check-node <cwd>
-> node -e "console.log(process.version)"
-
+$ node -e "console.log(process.version)" ⊘ cache disabled
 v<semver>
 
+
 > vp exec node -e "console.log(process.version)" # Should also use 22.12.0
 VITE+ - The Unified Toolchain for the Web