fix: use @yarnpkg/shell for cross-platform e2e tests

- Replace platform-specific shell (cmd.exe/sh) with @yarnpkg/shell
  for consistent cross-platform behavior in e2e tests
- Add Node.js experimental warning filtering in redact.rs to handle
  Type Stripping warnings from newer Node.js versions
- Add @yarnpkg/shell 4.1.3 dependency to test_bins
- Use `which` crate to locate shell executable in test_bins

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: branchseer

crates/vite_task_bin/Cargo.toml

@@ -24,6 +24,7 @@ which = { workspace = true }
 [dev-dependencies]
 copy_dir = { workspace = true }
 cow-utils = { workspace = true }
+which = { workspace = true }
 insta = { workspace = true, features = ["glob", "json", "redactions", "filters", "ron"] }
 regex = { workspace = true }
 serde = { workspace = true, features = ["derive", "rc"] }

crates/vite_task_bin/test_bins/package.json

@@ -9,6 +9,7 @@
     "replace-file-content": "./src/replace-file-content.ts"
   },
   "dependencies": {
+    "@yarnpkg/shell": "catalog:",
     "cross-env": "^10.1.0",
     "oxlint": "catalog:",
     "vite-task-test-bins": "link:"

crates/vite_task_bin/tests/test_snapshots/main.rs

@@ -82,6 +82,10 @@ fn run_case(runtime: &Runtime, tmpdir: &AbsolutePath, fixture_path: &Path) {
             .into_os_string(),
     );
 
+    // Find @yarnpkg/shell executable in test_bins
+    let shell_exe = which::which_in("shell", Some(&*test_bin_path), std::env::current_dir().unwrap())
+        .expect("shell executable not found in test_bins/node_modules/.bin");
+
     // Add test_bins to PATH so test programs (such as print-file) in fixtures can be found.
     let plan_envs: HashMap<Arc<OsStr>, Arc<OsStr>> = [
         (Arc::<OsStr>::from(OsStr::new("PATH")), Arc::clone(&test_bin_path)),
@@ -172,21 +176,8 @@ fn run_case(runtime: &Runtime, tmpdir: &AbsolutePath, fixture_path: &Path) {
 
             let mut e2e_outputs = String::new();
             for step in e2e.steps {
-                let mut cmd = if cfg!(windows) {
-                    let mut cmd = Command::new("cmd.exe");
-                    // https://github.com/nodejs/node/blob/dbd24b165128affb7468ca42f69edaf7e0d85a9a/lib/child_process.js#L633
-                    cmd.args(["/d", "/s", "/c"]);
-                    cmd
-                } else {
-                    let mut cmd = Command::new("sh");
-                    cmd.args(["-c"]);
-                    cmd
-                };
-                // On Windows, env_clear() would break cmd.exe, so only set PATH
-                // On Unix, clear all envs for reproducibility
-                if !cfg!(windows) {
-                    cmd.env_clear();
-                }
+                // Use @yarnpkg/shell for cross-platform shell execution
+                let mut cmd = Command::new(&shell_exe);
                 cmd.arg(step.as_str())
                     .env("PATH", &e2e_env_path)
                     .env("NO_COLOR", "1")

crates/vite_task_bin/tests/test_snapshots/redact.rs

@@ -67,6 +67,16 @@ pub fn redact_e2e_output(mut output: String, workspace_root: &str) -> String {
     let thread_regex = regex::Regex::new(r"using \d+ threads").unwrap();
     output = thread_regex.replace_all(&output, "using <n> threads").into_owned();
 
+    // Remove Node.js experimental warnings (e.g., Type Stripping warnings)
+    let node_warning_regex =
+        regex::Regex::new(r"(?m)^\(node:\d+\) ExperimentalWarning:.*\n?").unwrap();
+    output = node_warning_regex.replace_all(&output, "").into_owned();
+    let node_trace_warning_regex = regex::Regex::new(
+        r"(?m)^\(Use `node --trace-warnings \.\.\.` to show where the warning was created\)\n?",
+    )
+    .unwrap();
+    output = node_trace_warning_regex.replace_all(&output, "").into_owned();
+
     output
 }