refactor(tests): split and improve snapshot test infrastructure (#99)

## Summary

- Split `test_snapshots` into separate `plan_snapshots` (vite_task_plan) and `e2e_snapshots` (vite_task_bin) test suites
- Move snapshots to per-fixture directories for better organization
- Remove redundant task graph snapshots from e2e tests (now only in plan tests)
- Rename test directories for clarity (`test_snapshots` → `e2e_snapshots`, `test_plan_snapshots` → `plan_snapshots`)
- Disable libtest harness for both test suites (using custom `main()`)
- Add stdin support for e2e test steps (allows `{ cmd = "...", stdin = "..." }` syntax)
- Add CLI filter argument for running specific fixtures (`cargo test --test e2e_snapshots -- stdin`)

## Test plan

- [x] `cargo test -p vite_task_bin --test e2e_snapshots` passes
- [x] `cargo test -p vite_task_plan --test plan_snapshots` passes
- [x] Filter argument works: `cargo test -p vite_task_bin --test e2e_snapshots -- stdin`

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Author: branchseer

Cargo.lock

@@ -125,6 +125,7 @@ vite_path = { path = "crates/vite_path" }
 vite_shell = { path = "crates/vite_shell" }
 vite_str = { path = "crates/vite_str" }
 vite_task = { path = "crates/vite_task" }
+vite_task_bin = { path = "crates/vite_task_bin" }
 vite_task_graph = { path = "crates/vite_task_graph" }
 vite_task_plan = { path = "crates/vite_task_plan" }
 vite_workspace = { path = "crates/vite_workspace" }

Cargo.toml

@@ -5,36 +5,38 @@ use std::{env::vars_os, ffi::OsString};
 use fspy::{AccessMode, PathAccessIterable};
 use test_log::test;
 
-/// Get the test_bins/.bin directory path
-fn test_bins_bin_dir() -> std::path::PathBuf {
+/// Get the packages/tools/.bin directory path
+fn tools_bin_dir() -> std::path::PathBuf {
     std::path::Path::new(env!("CARGO_MANIFEST_DIR"))
         .parent()
         .unwrap()
-        .join("vite_task_bin")
-        .join("test_bins")
+        .parent()
+        .unwrap()
+        .join("packages")
+        .join("tools")
         .join("node_modules")
         .join(".bin")
 }
 
-/// Find the oxlint executable in test_bins
+/// Find the oxlint executable in packages/tools
 fn find_oxlint() -> std::path::PathBuf {
-    let test_bins_dir = test_bins_bin_dir();
-    which::which_in("oxlint", Some(&test_bins_dir), std::env::current_dir().unwrap())
-        .expect("oxlint not found in test_bins/node_modules/.bin")
+    let tools_dir = tools_bin_dir();
+    which::which_in("oxlint", Some(&tools_dir), std::env::current_dir().unwrap())
+        .expect("oxlint not found in packages/tools/node_modules/.bin")
 }
 
 async fn track_oxlint(dir: &std::path::Path, args: &[&str]) -> anyhow::Result<PathAccessIterable> {
     let oxlint_path = find_oxlint();
     let mut command = fspy::Command::new(&oxlint_path);
 
-    // Build PATH with test_bins/.bin prepended so oxlint can find tsgolint
-    let test_bins_dir = test_bins_bin_dir();
+    // Build PATH with packages/tools/.bin prepended so oxlint can find tsgolint
+    let tools_dir = tools_bin_dir();
     let new_path = if let Some(existing_path) = std::env::var_os("PATH") {
-        let mut paths = vec![test_bins_dir.as_os_str().to_owned()];
+        let mut paths = vec![tools_dir.as_os_str().to_owned()];
         paths.extend(std::env::split_paths(&existing_path).map(|p| p.into_os_string()));
         std::env::join_paths(paths)?
     } else {
-        OsString::from(&test_bins_dir)
+        OsString::from(&tools_dir)
     };
 
     command

crates/fspy/tests/oxlint.rs

@@ -27,14 +27,14 @@ cow-utils = { workspace = true }
 insta = { workspace = true, features = ["glob", "json", "redactions", "filters", "ron"] }
 regex = { workspace = true }
 serde = { workspace = true, features = ["derive", "rc"] }
-serde_json = { workspace = true }
 tempfile = { workspace = true }
 toml = { workspace = true }
-vite_graph_ser = { workspace = true }
 vite_path = { workspace = true, features = ["absolute-redaction"] }
-vite_task_graph = { workspace = true }
 vite_workspace = { workspace = true }
-which = { workspace = true }
 
 [lints]
 workspace = true
+
+[[test]]
+name = "e2e_snapshots"
+harness = false