Fix CI failures: remove unused deps, fix clippy, add missing test fixtures

- Remove unused `rustc-hash` from vite_task_bin dependencies
- Remove unused `which` from vite_task_graph dev-dependencies
- Fix clippy: use `Result::ok` instead of redundant `|e| e.ok()` closure in vtt.rs
- Add SAFETY comment for `libc::dup2` unsafe block in vtt.rs (undocumented-unsafe-blocks)
- Add missing `folder1/` and `folder2/` directories to builtin-different-cwd fixture
  (the cd builtin changes CWD before spawning vtt, which fails with ENOENT if the
  directory doesn't exist)
- Fix direct lint snapshot: update quote style to match actual main.js content

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: branchseer

Cargo.lock

@@ -20,7 +20,6 @@ libc = { workspace = true }
 async-trait = { workspace = true }
 clap = { workspace = true, features = ["derive"] }
 jsonc-parser = { workspace = true }
-rustc-hash = { workspace = true }
 serde_json = { workspace = true }
 tokio = { workspace = true, features = ["full"] }
 vite_path = { workspace = true }

crates/vite_task_bin/Cargo.toml

@@ -93,7 +93,7 @@ fn cmd_barrier(args: &[String]) -> Result<(), Box<dyn std::error::Error>> {
     // Poll until <count> matching files exist.
     loop {
         let matches = std::fs::read_dir(dir)?
-            .filter_map(|e| e.ok())
+            .filter_map(Result::ok)
             .filter(|e| e.file_name().to_string_lossy().starts_with(&std::format!("{prefix}_")))
             .count();
         if matches >= count {
@@ -111,6 +111,8 @@ fn cmd_barrier(args: &[String]) -> Result<(), Box<dyn std::error::Error>> {
             use std::os::unix::io::IntoRawFd;
             let null1 = std::fs::OpenOptions::new().write(true).open("/dev/null").unwrap();
             let null2 = std::fs::OpenOptions::new().write(true).open("/dev/null").unwrap();
+            // SAFETY: fds 1 and 2 are always valid (stdout/stderr), and null1/null2 are
+            // valid open file descriptors. dup2 is safe to call with valid fd arguments.
             unsafe {
                 libc::dup2(null1.into_raw_fd(), 1);
                 libc::dup2(null2.into_raw_fd(), 2);

crates/vite_task_bin/src/vtt.rs

@@ -4,7 +4,7 @@ expression: e2e_outputs
 ---
 > vt run lint # cache miss
 $ vt tool print-file main.js
-console.log("hello");
+console.log('hello');
 > echo modified > main.js # modify tracked file
 
 > vt run lint # cache miss, file changed

crates/vite_task_bin/tests/e2e_snapshots/fixtures/builtin-different-cwd/folder1/.gitkeep

@@ -28,7 +28,6 @@ pretty_assertions = { workspace = true }
 ts-rs = { workspace = true, features = ["no-serde-warnings"] }
 vite_path = { workspace = true, features = ["ts-rs"] }
 vite_str = { workspace = true, features = ["ts-rs"] }
-which = { workspace = true }
 
 [lints]
 workspace = true