fix: ignore disableCache in client

Author: wan9chi

crates/vite_task_client/src/lib.rs

@@ -84,13 +84,24 @@ impl Client {
         self.send(&Request::IgnoreOutput(&ns))
     }
 
-    /// Fire-and-forget — see [`Self::ignore_input`].
+    /// Temporary no-op.
+    ///
+    /// `disableCache` currently causes too many false opt-outs because tools
+    /// call it during configuration, before they know whether they will start
+    /// an uncached operation such as listening on a port or watching the
+    /// filesystem.
     ///
     /// # Errors
     ///
-    /// Returns an error if the request fails to send.
+    /// This temporary no-op does not currently return errors.
+    #[expect(
+        clippy::missing_const_for_fn,
+        clippy::unnecessary_wraps,
+        clippy::unused_self,
+        reason = "temporary no-op preserves the public API until disableCache semantics are redesigned"
+    )]
     pub fn disable_cache(&self) -> io::Result<()> {
-        self.send(&Request::DisableCache)
+        Ok(())
     }
 
     /// Requests an env value from the runner. Returns `None` if the runner

crates/vite_task_server/tests/integration.rs

@@ -93,6 +93,9 @@ fn single_client_fire_and_forget() {
         let client = connect(&envs);
         client.ignore_input(OsStr::new(in_path)).unwrap();
         client.ignore_output(OsStr::new(out_path)).unwrap();
+        // Temporary workaround: the client currently ignores disableCache so
+        // tools cannot opt out at configuration time before they perform the
+        // operation that actually makes a task uncacheable.
         client.disable_cache().unwrap();
         flush(&client);
     })
@@ -102,6 +105,18 @@ fn single_client_fire_and_forget() {
     let outputs: Vec<_> = reports.ignored_outputs.iter().map(|p| p.as_path().as_os_str()).collect();
     assert_eq!(inputs, vec![OsStr::new(in_path)]);
     assert_eq!(outputs, vec![OsStr::new(out_path)]);
+    assert!(!reports.cache_disabled);
+}
+
+#[test]
+fn raw_disable_cache_request_disables_cache() {
+    let reports = run_with_server(env_map(&[]), |envs| {
+        let name = &envs[0].1;
+        let mut stream = connect_raw(name);
+        send_frame(&mut stream, &Request::DisableCache);
+    })
+    .expect("driver returned error");
+
     assert!(reports.cache_disabled);
 }