chore: move JsonUserConfigLoader from vite_task_graph to vite_task_bin

JsonUserConfigLoader is only instantiated in vite_task_bin, so it
belongs there rather than in the core graph crate. This also removes
the tokio dependency from vite_task_graph.

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

Author: branchseer

Cargo.lock

@@ -14,6 +14,8 @@ path = "src/main.rs"
 anyhow = { workspace = true }
 async-trait = { workspace = true }
 clap = { workspace = true, features = ["derive"] }
+jsonc-parser = { workspace = true }
+serde_json = { workspace = true }
 tokio = { workspace = true, features = ["full"] }
 vite_path = { workspace = true }
 vite_str = { workspace = true }

crates/vite_task_bin/Cargo.toml

@@ -135,10 +135,38 @@ impl vite_task::TaskSynthesizer<CustomTaskSubcommand> for TaskSynthesizer {
     }
 }
 
+/// A `UserConfigLoader` implementation that only loads `vite-task.json`.
+///
+/// This is mainly for examples and testing as it does not require Node.js environment.
+#[derive(Default, Debug)]
+pub struct JsonUserConfigLoader(());
+
+#[async_trait::async_trait(?Send)]
+impl vite_task::loader::UserConfigLoader for JsonUserConfigLoader {
+    async fn load_user_config_file(
+        &self,
+        package_path: &AbsolutePath,
+    ) -> anyhow::Result<Option<vite_task::config::UserRunConfig>> {
+        let config_path = package_path.join("vite-task.json");
+        let config_content = match tokio::fs::read_to_string(&config_path).await {
+            Ok(content) => content,
+            Err(err) if err.kind() == std::io::ErrorKind::NotFound => {
+                return Ok(None);
+            }
+            Err(err) => return Err(err.into()),
+        };
+        let json_value =
+            jsonc_parser::parse_to_serde_value(&config_content, &Default::default())?
+                .unwrap_or_default();
+        let user_config: vite_task::config::UserRunConfig = serde_json::from_value(json_value)?;
+        Ok(Some(user_config))
+    }
+}
+
 #[derive(Default)]
 pub struct OwnedSessionCallbacks {
     task_synthesizer: TaskSynthesizer,
-    user_config_loader: vite_task::loader::JsonUserConfigLoader,
+    user_config_loader: JsonUserConfigLoader,
 }
 
 impl OwnedSessionCallbacks {

crates/vite_task_bin/src/lib.rs

@@ -16,7 +16,6 @@ petgraph = { workspace = true }
 serde = { workspace = true, features = ["derive"] }
 serde_json = { workspace = true }
 thiserror = { workspace = true }
-tokio = { workspace = true, features = ["fs"] }
 vec1 = { workspace = true, features = ["smallvec-v1"] }
 vite_graph_ser = { workspace = true }
 vite_path = { workspace = true }

crates/vite_task_graph/Cargo.toml

@@ -12,30 +12,3 @@ pub trait UserConfigLoader: Debug + Send + Sync {
         package_path: &AbsolutePath,
     ) -> anyhow::Result<Option<UserRunConfig>>;
 }
-
-/// A `UserConfigLoader` implementation that only loads `vite-task.json`.
-///
-/// This is mainly for examples and testing as it does not require Node.js environment.
-#[derive(Default, Debug)]
-pub struct JsonUserConfigLoader(());
-
-#[async_trait::async_trait(?Send)]
-impl UserConfigLoader for JsonUserConfigLoader {
-    async fn load_user_config_file(
-        &self,
-        package_path: &AbsolutePath,
-    ) -> anyhow::Result<Option<UserRunConfig>> {
-        let config_path = package_path.join("vite-task.json");
-        let config_content = match tokio::fs::read_to_string(&config_path).await {
-            Ok(content) => content,
-            Err(err) if err.kind() == std::io::ErrorKind::NotFound => {
-                return Ok(None);
-            }
-            Err(err) => return Err(err.into()),
-        };
-        let json_value = jsonc_parser::parse_to_serde_value(&config_content, &Default::default())?
-            .unwrap_or_default();
-        let user_config: UserRunConfig = serde_json::from_value(json_value)?;
-        Ok(Some(user_config))
-    }
-}