revert: remove CacheOutputs test type, keep only DeclCollector fix

Remove the CacheOutputs indirect type that was added to demonstrate
the bug. The recursive DeclCollector fix is retained.

https://claude.ai/code/session_01VQvSgXNnYL8tNWydvtbCvg

Author: claude

crates/vite_task_bin/src/lib.rs

@@ -64,7 +64,6 @@ fn synthesize_node_modules_bin_task(
         cache_config: UserCacheConfig::with_config(EnabledCacheConfig {
             envs: None,
             pass_through_envs: None,
-            outputs: None,
         }),
         envs: Arc::clone(envs),
     })
@@ -126,11 +125,7 @@ impl vite_task::CommandHandler for CommandHandler {
                     program: find_executable(get_path_env(&envs), &command.cwd, "print-env")?,
                     args: [name.clone()].into(),
                     cache_config: UserCacheConfig::with_config({
-                        EnabledCacheConfig {
-                            envs: None,
-                            pass_through_envs: Some(vec![name]),
-                            outputs: None,
-                        }
+                        EnabledCacheConfig { envs: None, pass_through_envs: Some(vec![name]) }
                     }),
                     envs: Arc::new(envs),
                 }))

crates/vite_task_bin/src/main.rs

@@ -35,7 +35,6 @@ async fn run() -> anyhow::Result<ExitStatus> {
                         EnabledCacheConfig {
                             envs: Some(Box::from([Str::from("FOO")])),
                             pass_through_envs: None,
-                            outputs: None,
                         }
                     }),
                     envs: Arc::clone(envs),

crates/vite_task_graph/run-config.ts

@@ -1,14 +1,3 @@
-export type CacheOutputs = {
-  /**
-   * Directories to cache (relative to the package root).
-   */
-  include?: Array<string>;
-  /**
-   * Directories to exclude from the cache.
-   */
-  exclude?: Array<string>;
-};
-
 export type Task = {
   /**
    * The command to run for the task.
@@ -38,10 +27,6 @@ export type Task = {
        * Environment variable names to be passed to the task without fingerprinting.
        */
       passThroughEnvs?: Array<string>;
-      /**
-       * Output artifacts to cache.
-       */
-      outputs?: CacheOutputs;
     }
   | {
       /**

crates/vite_task_graph/src/config/mod.rs

@@ -114,11 +114,7 @@ impl ResolvedTaskConfig {
         let cache_config = if cache_scripts {
             UserCacheConfig::Enabled {
                 cache: None,
-                enabled_cache_config: EnabledCacheConfig {
-                    envs: None,
-                    pass_through_envs: None,
-                    outputs: None,
-                },
+                enabled_cache_config: EnabledCacheConfig { envs: None, pass_through_envs: None },
             }
         } else {
             UserCacheConfig::Disabled { cache: MustBe!(false) }

crates/vite_task_graph/src/config/user.rs

@@ -47,19 +47,6 @@ impl UserCacheConfig {
     }
 }
 
-/// Configuration for cache output artifacts
-#[derive(Debug, Deserialize, PartialEq, Eq)]
-// TS derive macro generates code using std types that clippy disallows; skip derive during linting
-#[cfg_attr(all(test, not(clippy)), derive(TS), ts(optional_fields, rename = "CacheOutputs"))]
-#[serde(rename_all = "camelCase")]
-pub struct CacheOutputs {
-    /// Directories to cache (relative to the package root).
-    pub include: Option<Vec<Str>>,
-
-    /// Directories to exclude from the cache.
-    pub exclude: Option<Vec<Str>>,
-}
-
 /// Cache configuration fields when caching is enabled
 #[derive(Debug, Deserialize, PartialEq, Eq)]
 // TS derive macro generates code using std types that clippy disallows; skip derive during linting
@@ -71,9 +58,6 @@ pub struct EnabledCacheConfig {
 
     /// Environment variable names to be passed to the task without fingerprinting.
     pub pass_through_envs: Option<Vec<Str>>,
-
-    /// Output artifacts to cache.
-    pub outputs: Option<CacheOutputs>,
 }
 
 /// Options for user-defined tasks in `vite.config.*`, excluding the command.
@@ -105,11 +89,7 @@ impl Default for UserTaskOptions {
             // Caching enabled with no fingerprinted envs
             cache_config: UserCacheConfig::Enabled {
                 cache: None,
-                enabled_cache_config: EnabledCacheConfig {
-                    envs: None,
-                    pass_through_envs: None,
-                    outputs: None,
-                },
+                enabled_cache_config: EnabledCacheConfig { envs: None, pass_through_envs: None },
             },
         }
     }
@@ -388,7 +368,6 @@ mod tests {
                 enabled_cache_config: EnabledCacheConfig {
                     envs: Some(std::iter::once("NODE_ENV".into()).collect()),
                     pass_through_envs: Some(std::iter::once("FOO".into()).collect()),
-                    outputs: None,
                 }
             },
         );