fix: stop intercepting vite as a vp command in task scripts

`VitePlusCommandHandler` was intercepting both `vp` and `vite` program
names in task scripts. This caused `vp run dev` to fail with "Invalid
vite task command" when a package.json script used bare `vite` (e.g.
`"dev": "vite"`), because `vite` is a separate tool, not a vp alias.

Only intercept `vp` commands now; `vite` runs verbatim as an external
process.

Closes #1176

Author: fengmk2

packages/cli/binding/src/cli.rs

@@ -551,16 +551,12 @@ impl CommandHandler for VitePlusCommandHandler {
         &mut self,
         command: &mut ScriptCommand,
     ) -> anyhow::Result<HandledCommand> {
-        // Intercept both "vp" and "vite" commands in task scripts.
-        // "vp" is the conventional alias used in vite-plus task configs.
-        // "vite" must also be intercepted so that `vite test`, `vite build`, etc.
-        // in task scripts are synthesized in-session rather than spawning a new CLI process.
+        // Only intercept "vp" commands in task scripts.
+        // "vite" is a separate tool and should always run verbatim.
         let program = command.program.as_str();
-        if program != "vp" && program != "vite" {
+        if program != "vp" {
             return Ok(HandledCommand::Verbatim);
         }
-        // Parse "vp <args>" using CLIArgs — always use "vp" as the program name
-        // so clap shows "Usage: vp ..." even if the original command was "vite ..."
         let cli_args = match CLIArgs::try_parse_from(
             iter::once("vp").chain(command.args.iter().map(Str::as_str)),
         ) {

packages/cli/snap-tests-global/command-run-script-vite-program/package.json

@@ -0,0 +1,10 @@
+{
+  "name": "command-run-script-vite-program",
+  "version": "1.0.0",
+  "scripts": {
+    "dev": "vite",
+    "dev-help": "vite -h",
+    "dev-version": "vite --version"
+  },
+  "packageManager": "pnpm@10.19.0"
+}

packages/cli/snap-tests-global/command-run-script-vite-program/setup-bin.js

@@ -0,0 +1,8 @@
+const fs = require('fs');
+fs.mkdirSync('node_modules/.bin', { recursive: true });
+fs.writeFileSync(
+  'node_modules/.bin/vite',
+  '#!/usr/bin/env node\nconst args = process.argv.slice(2);\nconsole.log(args.length ? "vite " + args.join(" ") : "vite");\n',
+  { mode: 0o755 },
+);
+fs.writeFileSync('node_modules/.bin/vite.cmd', '@node "%~dp0\\vite" %*\n');

packages/cli/snap-tests-global/command-run-script-vite-program/snap.txt

@@ -0,0 +1,21 @@
+> node setup-bin.js
+> vp run dev # should run vite binary, not parse as vp subcommand
+VITE+ - The Unified Toolchain for the Web
+
+$ vite ⊘ cache disabled
+vite
+
+
+> vp run dev-help # should run vite -h, not parse as vp subcommand
+VITE+ - The Unified Toolchain for the Web
+
+$ vite -h ⊘ cache disabled
+vite -h
+
+
+> vp run dev-version # should run vite --version, not parse as vp subcommand
+VITE+ - The Unified Toolchain for the Web
+
+$ vite --version ⊘ cache disabled
+vite --version
+

packages/cli/snap-tests-global/command-run-script-vite-program/steps.json

@@ -0,0 +1,8 @@
+{
+  "commands": [
+    { "command": "node setup-bin.js", "ignoreOutput": true },
+    "vp run dev # should run vite binary, not parse as vp subcommand",
+    "vp run dev-help # should run vite -h, not parse as vp subcommand",
+    "vp run dev-version # should run vite --version, not parse as vp subcommand"
+  ]
+}