fix(command): look up PATH case-insensitively in resolve_program

On Windows the canonical key is `Path`, and the TS layer's
`prependToPathToEnvs` preserves whatever casing the process started
with when adding the downloaded package-manager bin dir. Hard-coding
`envs.get("PATH")` missed a `Path`-keyed override and silently fell
back to the process PATH — where the prepended shim is absent —
making `vp create` (and any other flow routed through
`runCommandAndDetectProjectDir` / `run_command_with_fspy`) fail with
`Cannot find binary path...` for `npx`/`pnpm`.

Match the case-insensitive lookup `fspy::Command::resolve_program`
already does, so the previously-correct behavior of the fspy path
holds after vp-side resolution moved up a layer in
1b77707 (refactor(command): extract resolve_program to dedupe
ps1-rewrite plumbing).

Adds a Windows-only regression test (`resolve_program_tests`) that
writes a `.exe` into a tempdir and resolves via a `Path`-keyed env
map; without the fix the test fails because resolution falls back to
the process PATH.

Author: fengmk2

crates/vite_command/src/lib.rs

@@ -53,7 +53,16 @@ fn resolve_program(
     envs: &HashMap<String, String>,
     cwd: &AbsolutePath,
 ) -> Result<(AbsolutePathBuf, Vec<OsString>), Error> {
-    let bin_path = resolve_bin(bin_name, envs.get("PATH").map(|p| OsStr::new(p.as_str())), cwd)?;
+    // Look up PATH case-insensitively: on Windows the canonical key is `Path`
+    // and the TS layer's `prependToPathToEnvs` preserves whatever casing the
+    // process started with. Hard-coding `"PATH"` would miss a `Path`-keyed
+    // override and silently fall back to the process PATH (where the
+    // prepended package-manager bin dir is absent).
+    let path_env = envs
+        .iter()
+        .find(|(k, _)| k.eq_ignore_ascii_case("path"))
+        .map(|(_, v)| OsStr::new(v.as_str()));
+    let bin_path = resolve_bin(bin_name, path_env, cwd)?;
     Ok(match ps1_shim::rewrite_cmd_to_powershell(&bin_path) {
         Some(rewritten) => rewritten,
         None => (bin_path, Vec::new()),
@@ -369,6 +378,50 @@ mod tests {
         }
     }
 
+    #[cfg(windows)]
+    mod resolve_program_tests {
+        use super::*;
+
+        // Regression for voidzero-dev/vite-plus#1489 follow-up: on Windows the
+        // PATH environment variable is conventionally keyed `Path` (and the
+        // TS layer's `prependToPathToEnvs` preserves that casing when adding
+        // the downloaded package-manager bin dir). `resolve_program` must
+        // look up PATH case-insensitively or it falls back to the process
+        // environment and can't find the prepended shim.
+        #[test]
+        fn resolve_program_finds_binary_via_lowercase_path_key() {
+            let temp_dir = create_temp_dir();
+            let temp_dir_path =
+                AbsolutePathBuf::new(temp_dir.path().canonicalize().unwrap().to_path_buf())
+                    .unwrap();
+
+            // Unique name so it can't accidentally resolve via the process PATH.
+            let bin_name = "vp_test_resolve_program_path_casing";
+            let bin_path = temp_dir.path().join(format!("{bin_name}.exe"));
+            std::fs::write(&bin_path, b"").unwrap();
+
+            // `Path`, not `PATH` — the real-world Windows casing.
+            let envs = HashMap::from([(
+                "Path".to_string(),
+                temp_dir_path.as_path().to_string_lossy().into_owned(),
+            )]);
+
+            let result = resolve_program(bin_name, &envs, &temp_dir_path);
+            assert!(
+                result.is_ok(),
+                "resolve_program must find the binary via the `Path` env key on Windows; got: {:?}",
+                result.err()
+            );
+            let (program, prefix_args) = result.unwrap();
+            assert!(prefix_args.is_empty(), "no .ps1 sibling, no PowerShell prefix expected");
+            assert_eq!(
+                program.as_path().file_stem().and_then(|s| s.to_str()),
+                Some(bin_name),
+                "resolved program should be the .exe in the tempdir"
+            );
+        }
+    }
+
     mod run_command_with_fspy_tests {
         use super::*;