refactor: remove oxfmt dependency from TS type generation

The generated run-config.ts no longer needs to be formatted via oxfmt,
removing the need for `pnpm install` in packages/tools during cargo test.

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

Author: branchseer

crates/vite_task_graph/run-config.ts

@@ -67,51 +67,47 @@ export type Task = {
     }
 );
 
-export type UserGlobalCacheConfig =
-  | boolean
-  | {
-      /**
-       * Enable caching for package.json scripts not defined in the `tasks` map.
-       *
-       * When `false`, package.json scripts will not be cached.
-       * When `true`, package.json scripts will be cached with default settings.
-       *
-       * Default: `false`
-       */
-      scripts?: boolean;
-      /**
-       * Global cache kill switch for task entries.
-       *
-       * When `false`, overrides all tasks to disable caching, even tasks with `cache: true`.
-       * When `true`, respects each task's individual `cache` setting
-       * (each task's `cache` defaults to `true` if omitted).
-       *
-       * Default: `true`
-       */
-      tasks?: boolean;
-    };
+export type UserGlobalCacheConfig = boolean | { 
+/**
+ * Enable caching for package.json scripts not defined in the `tasks` map.
+ *
+ * When `false`, package.json scripts will not be cached.
+ * When `true`, package.json scripts will be cached with default settings.
+ *
+ * Default: `false`
+ */
+scripts?: boolean, 
+/**
+ * Global cache kill switch for task entries.
+ *
+ * When `false`, overrides all tasks to disable caching, even tasks with `cache: true`.
+ * When `true`, respects each task's individual `cache` setting
+ * (each task's `cache` defaults to `true` if omitted).
+ *
+ * Default: `true`
+ */
+tasks?: boolean, };
 
-export type RunConfig = {
-  /**
-   * Root-level cache configuration.
-   *
-   * This option can only be set in the workspace root's config file.
-   * Setting it in a package's config will result in an error.
-   */
-  cache?: UserGlobalCacheConfig;
-  /**
-   * Task definitions
-   */
-  tasks?: { [key in string]?: Task };
-  /**
-   * Whether to automatically run `preX`/`postX` package.json scripts as
-   * lifecycle hooks when script `X` is executed.
-   *
-   * When `true` (the default), running script `test` will automatically
-   * run `pretest` before and `posttest` after, if they exist.
-   *
-   * This option can only be set in the workspace root's config file.
-   * Setting it in a package's config will result in an error.
-   */
-  enablePrePostScripts?: boolean;
-};
+export type RunConfig = { 
+/**
+ * Root-level cache configuration.
+ *
+ * This option can only be set in the workspace root's config file.
+ * Setting it in a package's config will result in an error.
+ */
+cache?: UserGlobalCacheConfig, 
+/**
+ * Task definitions
+ */
+tasks?: { [key in string]?: Task }, 
+/**
+ * Whether to automatically run `preX`/`postX` package.json scripts as
+ * lifecycle hooks when script `X` is executed.
+ *
+ * When `true` (the default), running script `test` will automatically
+ * run `pretest` before and `posttest` after, if they exist.
+ *
+ * This option can only be set in the workspace root's config file.
+ * Setting it in a package's config will result in an error.
+ */
+enablePrePostScripts?: boolean, };

crates/vite_task_graph/src/config/user.rs

@@ -273,22 +273,12 @@ impl UserRunConfig {
     pub const TS_TYPE: &str = include_str!("../../run-config.ts");
 
     /// Generates TypeScript type definitions for user task configuration.
-    ///
-    /// # Panics
-    ///
-    /// Panics if `oxfmt` is not found in `packages/tools`, if the formatter process
-    /// fails to spawn or write, or if the output is not valid UTF-8.
     #[cfg(all(test, not(clippy)))]
     #[must_use]
     // test code: uses std types for convenience
     #[expect(clippy::disallowed_types, reason = "test code uses std types for convenience")]
     pub fn generate_ts_definition() -> String {
-        use std::{
-            any::TypeId,
-            collections::HashSet,
-            io::Write,
-            process::{Command, Stdio},
-        };
+        use std::{any::TypeId, collections::HashSet};
 
         use ts_rs::TypeVisitor;
 
@@ -335,32 +325,7 @@ impl UserRunConfig {
         types.push_str("\n\nexport ");
         types.push_str(&Self::decl());
 
-        // Format using oxfmt from packages/tools
-        let workspace_root =
-            std::path::Path::new(env!("CARGO_MANIFEST_DIR")).parent().unwrap().parent().unwrap();
-        let tools_path = workspace_root.join("packages/tools/node_modules/.bin");
-
-        let oxfmt_path = which::which_in("oxfmt", Some(&tools_path), &tools_path)
-            .expect("oxfmt not found in packages/tools");
-
-        let mut child = Command::new(oxfmt_path)
-            .arg("--stdin-filepath=run-config.ts")
-            .stdin(Stdio::piped())
-            .stdout(Stdio::piped())
-            .spawn()
-            .expect("failed to spawn oxfmt");
-
-        child
-            .stdin
-            .take()
-            .unwrap()
-            .write_all(types.as_bytes())
-            .expect("failed to write to oxfmt stdin");
-
-        let output = child.wait_with_output().expect("failed to read oxfmt output");
-        assert!(output.status.success(), "oxfmt failed");
-
-        String::from_utf8(output.stdout).expect("oxfmt output is not valid UTF-8")
+        types
     }
 }