Remove json-patch command; use replace-file-content instead

replace-file-content now errors with exit 1 when searchValue is not
found. All test fixtures converted from json-patch to replace-file-content.
Removes json-patch and serde_json dependencies from vite_task_tools.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: branchseer

Cargo.lock

@@ -81,7 +81,6 @@ fspy_shared_unix = { path = "crates/fspy_shared_unix" }
 futures = "0.3.31"
 futures-util = "0.3.31"
 insta = "1.44.3"
-json-patch = "4.1.0"
 jsonc-parser = { version = "0.29.0", features = ["serde"] }
 libc = "0.2.172"
 memmap2 = "0.9.7"
@@ -147,7 +146,6 @@ 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_task_tools = { path = "crates/vite_task_tools" }
 vite_workspace = { path = "crates/vite_workspace" }
 vt100 = "0.16.2"
 wax = "0.7.0"

Cargo.toml

@@ -25,7 +25,6 @@ tokio = { workspace = true, features = ["full"] }
 vite_path = { workspace = true }
 vite_str = { workspace = true }
 vite_task = { workspace = true }
-vite_task_tools = { workspace = true }
 which = { workspace = true }
 
 [dev-dependencies]

crates/vite_task_bin/Cargo.toml

@@ -1,3 +1,119 @@
+// This is a standalone test utility binary that deliberately uses std types
+// rather than the project's custom types (vite_str, vite_path, etc.).
+#![expect(clippy::disallowed_types, reason = "standalone test utility uses std types")]
+#![expect(clippy::disallowed_macros, reason = "standalone test utility uses std macros")]
+#![expect(clippy::disallowed_methods, reason = "standalone test utility uses std methods")]
+#![expect(clippy::print_stderr, reason = "CLI tool error output")]
+#![expect(clippy::print_stdout, reason = "CLI tool output")]
+
 fn main() {
-    vite_task_tools::main();
+    let args: Vec<String> = std::env::args().collect();
+    if args.len() < 2 {
+        eprintln!("Usage: vtt <subcommand> [args...]");
+        eprintln!(
+            "Subcommands: check-tty, print, print-env, print-file, read-stdin, replace-file-content, touch-file"
+        );
+        std::process::exit(1);
+    }
+
+    let result: Result<(), Box<dyn std::error::Error>> = match args[1].as_str() {
+        "check-tty" => {
+            cmd_check_tty();
+            Ok(())
+        }
+        "print" => {
+            cmd_print(&args[2..]);
+            Ok(())
+        }
+        "print-env" => cmd_print_env(&args[2..]),
+        "print-file" => cmd_print_file(&args[2..]),
+        "read-stdin" => cmd_read_stdin(),
+        "replace-file-content" => cmd_replace_file_content(&args[2..]),
+        "touch-file" => cmd_touch_file(&args[2..]),
+        other => {
+            eprintln!("Unknown subcommand: {other}");
+            std::process::exit(1);
+        }
+    };
+
+    if let Err(err) = result {
+        eprintln!("{err}");
+        std::process::exit(1);
+    }
+}
+
+fn cmd_check_tty() {
+    use std::io::IsTerminal as _;
+    let stdin_tty = if std::io::stdin().is_terminal() { "tty" } else { "not-tty" };
+    let stdout_tty = if std::io::stdout().is_terminal() { "tty" } else { "not-tty" };
+    let stderr_tty = if std::io::stderr().is_terminal() { "tty" } else { "not-tty" };
+    println!("stdin:{stdin_tty}");
+    println!("stdout:{stdout_tty}");
+    println!("stderr:{stderr_tty}");
+}
+
+fn cmd_print(args: &[String]) {
+    println!("{}", args.join(" "));
+}
+
+fn cmd_print_env(args: &[String]) -> Result<(), Box<dyn std::error::Error>> {
+    if args.is_empty() {
+        return Err("Usage: vtt print-env <VAR_NAME>".into());
+    }
+    let value = std::env::var(&args[0]).unwrap_or_else(|_| "(undefined)".to_string());
+    println!("{value}");
+    Ok(())
+}
+
+fn cmd_print_file(args: &[String]) -> Result<(), Box<dyn std::error::Error>> {
+    use std::io::Write as _;
+    let stdout = std::io::stdout();
+    let mut out = stdout.lock();
+    for file in args {
+        match std::fs::read(file) {
+            Ok(content) => out.write_all(&content)?,
+            Err(_) => eprintln!("{file}: not found"),
+        }
+    }
+    Ok(())
+}
+
+fn cmd_read_stdin() -> Result<(), Box<dyn std::error::Error>> {
+    use std::io::{Read as _, Write as _};
+    let mut stdin = std::io::stdin().lock();
+    let mut stdout = std::io::stdout().lock();
+    let mut buf = [0u8; 8192];
+    loop {
+        match stdin.read(&mut buf) {
+            Ok(0) | Err(_) => break,
+            Ok(n) => stdout.write_all(&buf[..n])?,
+        }
+    }
+    Ok(())
+}
+
+fn cmd_replace_file_content(args: &[String]) -> Result<(), Box<dyn std::error::Error>> {
+    if args.len() < 3 {
+        return Err("Usage: vtt replace-file-content <filename> <searchValue> <newValue>".into());
+    }
+    let filename = &args[0];
+    let search_value = &args[1];
+    let new_value = &args[2];
+
+    let filepath = std::path::Path::new(filename).canonicalize()?;
+    let content = std::fs::read_to_string(&filepath)?;
+    if !content.contains(search_value) {
+        return Err(std::format!("searchValue not found in {filename}: {search_value:?}").into());
+    }
+    let new_content = content.replacen(search_value, new_value, 1);
+    std::fs::write(&filepath, new_content)?;
+    Ok(())
+}
+
+fn cmd_touch_file(args: &[String]) -> Result<(), Box<dyn std::error::Error>> {
+    if args.is_empty() {
+        return Err("Usage: vtt touch-file <filename>".into());
+    }
+    let _file = std::fs::OpenOptions::new().read(true).write(true).open(&args[0])?;
+    Ok(())
 }

crates/vite_task_bin/src/vtt.rs

@@ -5,6 +5,6 @@ name = "associate existing cache"
 steps = [
   "vt run script1 # cache miss",
   "vt run script2 # cache hit, same command as script1",
-  "vtt json-patch package.json '[{\"op\":\"add\",\"path\":\"/scripts/script2\",\"value\":\"vtt print world\"}]' # change script2",
+  "vtt replace-file-content package.json '\"script2\": \"vtt print hello\"' '\"script2\": \"vtt print world\"' # change script2",
   "vt run script2 # cache miss",
 ]

crates/vite_task_bin/tests/e2e_snapshots/fixtures/associate-existing-cache/snapshots.toml

@@ -11,7 +11,7 @@ hello
 
 ---
 vt run: cache hit.
-> vtt json-patch package.json '[{"op":"add","path":"/scripts/script2","value":"vtt print world"}]' # change script2
+> vtt replace-file-content package.json '"script2": "vtt print hello"' '"script2": "vtt print world"' # change script2
 
 > vt run script2 # cache miss
 $ vtt print world ○ cache miss: args changed, executing

crates/vite_task_bin/tests/e2e_snapshots/fixtures/associate-existing-cache/snapshots/associate existing cache.snap

@@ -4,8 +4,8 @@
 name = "cache miss command change"
 steps = [
   "vt run task # cache miss",
-  "vtt json-patch vite-task.json '[{\"op\":\"replace\",\"path\":\"/tasks/task/command\",\"value\":\"vtt print baz && vtt print bar\"}]' # change first subtask",
+  "vtt replace-file-content vite-task.json 'vtt print foo && vtt print bar' 'vtt print baz && vtt print bar' # change first subtask",
   "vt run task # first: cache miss, second: cache hit",
-  "vtt json-patch vite-task.json '[{\"op\":\"replace\",\"path\":\"/tasks/task/command\",\"value\":\"vtt print bar\"}]' # remove first subtask",
+  "vtt replace-file-content vite-task.json 'vtt print baz && vtt print bar' 'vtt print bar' # remove first subtask",
   "vt run task # cache hit",
 ]

crates/vite_task_bin/tests/e2e_snapshots/fixtures/cache-miss-command-change/snapshots.toml

@@ -11,7 +11,7 @@ bar
 
 ---
 vt run: 0/2 cache hit (0%). (Run `vt run --last-details` for full details)
-> vtt json-patch vite-task.json '[{"op":"replace","path":"/tasks/task/command","value":"vtt print baz && vtt print bar"}]' # change first subtask
+> vtt replace-file-content vite-task.json 'vtt print foo && vtt print bar' 'vtt print baz && vtt print bar' # change first subtask
 
 > vt run task # first: cache miss, second: cache hit
 $ vtt print baz ○ cache miss: args changed, executing
@@ -22,7 +22,7 @@ bar
 
 ---
 vt run: 1/2 cache hit (50%). (Run `vt run --last-details` for full details)
-> vtt json-patch vite-task.json '[{"op":"replace","path":"/tasks/task/command","value":"vtt print bar"}]' # remove first subtask
+> vtt replace-file-content vite-task.json 'vtt print baz && vtt print bar' 'vtt print bar' # remove first subtask
 
 > vt run task # cache hit
 $ vtt print bar ◉ cache hit, replaying

crates/vite_task_bin/tests/e2e_snapshots/fixtures/cache-miss-command-change/snapshots/cache miss command change.snap

@@ -19,16 +19,16 @@ steps = ["MY_ENV=1 vt run test # cache miss", "vt run test # cache miss: env rem
 name = "untracked env added"
 steps = [
   "vt run test # cache miss",
-  "vtt json-patch vite-task.json '[{\"op\":\"add\",\"path\":\"/tasks/test/untrackedEnv\",\"value\":[\"MY_UNTRACKED\"]}]' # add untracked env",
+  """vtt replace-file-content vite-task.json '"cache": true' '"cache": true, "untrackedEnv": ["MY_UNTRACKED"]' # add untracked env""",
   "vt run test # cache miss: untracked env added",
 ]
 
 [[e2e]]
 name = "untracked env removed"
 steps = [
-  "vtt json-patch vite-task.json '[{\"op\":\"add\",\"path\":\"/tasks/test/untrackedEnv\",\"value\":[\"MY_UNTRACKED\"]}]' # setup",
+  """vtt replace-file-content vite-task.json '"cache": true' '"cache": true, "untrackedEnv": ["MY_UNTRACKED"]' # setup""",
   "vt run test # cache miss",
-  "vtt json-patch vite-task.json '[{\"op\":\"remove\",\"path\":\"/tasks/test/untrackedEnv\"}]' # remove untracked env",
+  """vtt replace-file-content vite-task.json '"cache": true, "untrackedEnv": ["MY_UNTRACKED"]' '"cache": true' # remove untracked env""",
   "vt run test # cache miss: untracked env removed",
 ]
 
@@ -38,7 +38,7 @@ steps = [
   "vt run test # cache miss",
   "mkdir -p subfolder",
   "cp test.txt subfolder/test.txt",
-  "vtt json-patch vite-task.json '[{\"op\":\"add\",\"path\":\"/tasks/test/cwd\",\"value\":\"subfolder\"}]' # change cwd",
+  """vtt replace-file-content vite-task.json '"cache": true' '"cache": true, "cwd": "subfolder"' # change cwd""",
   "vt run test # cache miss: cwd changed",
 ]
 
@@ -58,7 +58,7 @@ steps = [
 name = "input config changed"
 steps = [
   "vt run test # cache miss",
-  "vtt json-patch vite-task.json '[{\"op\":\"add\",\"path\":\"/tasks/test/input\",\"value\":[\"test.txt\"]}]' # change input config",
+  """vtt replace-file-content vite-task.json '"cache": true' '"cache": true, "input": ["test.txt"]' # change input config""",
   "vt run test # cache miss: configuration changed",
 ]
 

crates/vite_task_bin/tests/e2e_snapshots/fixtures/cache-miss-reasons/snapshots.toml

@@ -9,7 +9,7 @@ initial content
 
 > cp test.txt subfolder/test.txt
 
-> vtt json-patch vite-task.json '[{"op":"add","path":"/tasks/test/cwd","value":"subfolder"}]' # change cwd
+> vtt replace-file-content vite-task.json '"cache": true' '"cache": true, "cwd": "subfolder"' # change cwd
 
 > vt run test # cache miss: cwd changed
 ~/subfolder$ vtt print-file test.txt ○ cache miss: working directory changed, executing