feat(cache): disable cache if there was user input

# Conflicts:
#	crates/vite_task_bin/tests/e2e_snapshots/fixtures/exit-codes/snapshots/multiple task failures returns exit code 1.snap
#	crates/vite_task_bin/tests/e2e_snapshots/fixtures/exit-codes/snapshots/single task failure returns task exit code.snap

Author: branchseer

crates/vite_task/src/session/cache/display.rs

@@ -8,7 +8,9 @@ use std::collections::HashSet;
 use vite_task_plan::cache_metadata::SpawnFingerprint;
 
 use super::{CacheMiss, FingerprintMismatch};
-use crate::session::event::{CacheDisabledReason, CacheStatus};
+use crate::session::event::{
+    CacheDisabledReason, CacheNotUpdatedReason, CacheStatus, CacheUpdateStatus,
+};
 
 /// Describes a single atomic change between two spawn fingerprints
 enum SpawnFingerprintChange {
@@ -283,3 +285,29 @@ pub fn format_cache_status_summary(cache_status: &CacheStatus) -> String {
         }
     }
 }
+
+/// Format cache update status for summary display (post-execution).
+///
+/// Returns Some(formatted_string) only when the reason is not already clear from CacheStatus.
+/// - Updated: No message needed (success is implied)
+/// - CacheHit: No message needed (already shown in CacheStatus::Hit)
+/// - CacheDisabled: No message needed (already shown in CacheStatus::Disabled)
+/// - BuiltInCommand: No message needed (already shown in CacheStatus::Disabled(InProcessExecution))
+/// - NonZeroExitStatus: Shows message that cache wasn't updated due to failure
+///
+/// Note: Returns plain text without styling. The reporter applies colors.
+pub fn format_cache_update_status(status: &CacheUpdateStatus) -> Option<String> {
+    match status {
+        CacheUpdateStatus::Updated => None,
+        CacheUpdateStatus::NotUpdated(reason) => match reason {
+            // These are already clear from CacheStatus in the Start event
+            CacheNotUpdatedReason::CacheHit => None,
+            CacheNotUpdatedReason::CacheDisabled => None,
+            CacheNotUpdatedReason::BuiltInCommand => None,
+            // This needs to be shown - task failed so cache wasn't updated
+            CacheNotUpdatedReason::NonZeroExitStatus => {
+                Some("→ Cache not updated: task failed".to_string())
+            }
+        },
+    }
+}

crates/vite_task/src/session/cache/mod.rs

@@ -6,7 +6,9 @@ use std::{fmt::Display, fs::File, io::Write, sync::Arc, time::Duration};
 
 use bincode::{Decode, Encode, decode_from_slice, encode_to_vec};
 // Re-export display functions for convenience
-pub use display::{format_cache_status_inline, format_cache_status_summary};
+pub use display::{
+    format_cache_status_inline, format_cache_status_summary, format_cache_update_status,
+};
 use rusqlite::{Connection, OptionalExtension as _, config::DbConfig};
 use serde::{Deserialize, Serialize};
 use tokio::sync::Mutex;

crates/vite_task/src/session/event.rs

@@ -27,6 +27,8 @@ pub enum CacheNotUpdatedReason {
     CacheDisabled,
     /// Execution exited with non-zero status
     NonZeroExitStatus,
+    /// Built-in command doesn't support caching
+    BuiltInCommand,
 }
 
 #[derive(Debug)]

crates/vite_task/src/session/execute/mod.rs

@@ -153,13 +153,13 @@ impl ExecutionContext<'_> {
                     },
                 });
 
-                // Emit Finish with CacheDisabled status (in-process executions don't cache)
+                // Emit Finish with BuiltInCommand status (built-in commands don't cache)
                 self.event_handler.handle_event(ExecutionEvent {
                     execution_id,
                     kind: ExecutionEventKind::Finish {
                         status: Some(0),
                         cache_update_status: CacheUpdateStatus::NotUpdated(
-                            CacheNotUpdatedReason::CacheDisabled,
+                            CacheNotUpdatedReason::BuiltInCommand,
                         ),
                     },
                 });

crates/vite_task/src/session/reporter.rs

@@ -11,8 +11,11 @@ use owo_colors::{Style, Styled};
 use vite_path::AbsolutePath;
 
 use super::{
-    cache::{format_cache_status_inline, format_cache_status_summary},
-    event::{CacheStatus, ExecutionEvent, ExecutionEventKind, ExecutionId, ExecutionItemDisplay},
+    cache::{format_cache_status_inline, format_cache_status_summary, format_cache_update_status},
+    event::{
+        CacheStatus, CacheUpdateStatus, ExecutionEvent, ExecutionEventKind, ExecutionId,
+        ExecutionItemDisplay,
+    },
 };
 
 /// Wrap of `OwoColorize` that ignores style if `NO_COLOR` is set.
@@ -55,6 +58,7 @@ const CACHE_MISS_STYLE: Style = Style::new().purple();
 struct ExecutionInfo {
     display: Option<ExecutionItemDisplay>,
     cache_status: CacheStatus, // Non-optional, determined at Start
+    cache_update_status: Option<CacheUpdateStatus>, // Set at Finish
     exit_status: Option<i32>,
     error_message: Option<String>,
 }
@@ -157,6 +161,7 @@ impl<W: Write> LabeledReporter<W> {
             self.executions.push(ExecutionInfo {
                 display: None,
                 cache_status,
+                cache_update_status: None,
                 exit_status: None,
                 error_message: None,
             });
@@ -199,6 +204,7 @@ impl<W: Write> LabeledReporter<W> {
         self.executions.push(ExecutionInfo {
             display: Some(display),
             cache_status,
+            cache_update_status: None,
             exit_status: None,
             error_message: None,
         });
@@ -226,17 +232,23 @@ impl<W: Write> LabeledReporter<W> {
         self.stats.failed += 1;
     }
 
-    fn handle_finish(&mut self, execution_id: ExecutionId, status: Option<i32>) {
+    fn handle_finish(
+        &mut self,
+        execution_id: ExecutionId,
+        status: Option<i32>,
+        cache_update_status: CacheUpdateStatus,
+    ) {
         // Update failure statistics
         if let Some(s) = status {
             if s != 0 {
                 self.stats.failed += 1;
             }
         }
 
-        // Update execution info exit status
+        // Update execution info
         if let Some(exec) = self.executions.last_mut() {
             exec.exit_status = status;
+            exec.cache_update_status = Some(cache_update_status);
         }
 
         // For direct synthetic execution with cache hit, print message at the bottom
@@ -430,6 +442,14 @@ impl<W: Write> LabeledReporter<W> {
             };
             let _ = writeln!(self.writer, "      {}", styled_summary);
 
+            // Cache update status (only shown for NonZeroExitStatus)
+            if let Some(ref cache_update_status) = exec.cache_update_status {
+                if let Some(update_msg) = format_cache_update_status(cache_update_status) {
+                    let _ =
+                        writeln!(self.writer, "      {}", update_msg.style(Style::new().yellow()));
+                }
+            }
+
             // Error message if present
             if let Some(ref error_msg) = exec.error_message {
                 let _ = writeln!(
@@ -486,8 +506,8 @@ impl<W: Write> Reporter for LabeledReporter<W> {
             ExecutionEventKind::Error { message } => {
                 self.handle_error(event.execution_id, message);
             }
-            ExecutionEventKind::Finish { status, cache_update_status: _ } => {
-                self.handle_finish(event.execution_id, status);
+            ExecutionEventKind::Finish { status, cache_update_status } => {
+                self.handle_finish(event.execution_id, status, cache_update_status);
             }
         }
     }

crates/vite_task_bin/tests/e2e_snapshots/fixtures/cache-non-zero-exit/package.json

@@ -0,0 +1,6 @@
+{
+  "name": "cache-non-zero-exit-test",
+  "scripts": {
+    "test": "node -e \"process.exit(1)\""
+  }
+}

crates/vite_task_bin/tests/e2e_snapshots/fixtures/cache-non-zero-exit/snapshots.toml

@@ -0,0 +1,6 @@
+[[e2e]]
+name = "failed task does not update cache"
+steps = [
+  "vite run test",
+  "vite run test",
+]

crates/vite_task_bin/tests/e2e_snapshots/fixtures/cache-non-zero-exit/snapshots/failed task does not update cache.snap

@@ -0,0 +1,40 @@
+---
+source: crates/vite_task_bin/tests/e2e_snapshots/main.rs
+expression: e2e_outputs
+input_file: crates/vite_task_bin/tests/e2e_snapshots/fixtures/cache-non-zero-exit
+---
+[1]> vite run test
+$ node -e "process.exit(1)"
+
+
+━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+    Vite+ Task Runner • Execution Summary
+━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+Statistics:   1 tasks • 0 cache hits • 1 cache misses • 1 failed 
+Performance:  0% cache hit rate
+
+Task Details:
+────────────────────────────────────────────────
+  [1] cache-non-zero-exit-test#test: $ node -e "process.exit(1)" ✗ (exit code: 1)
+      → Cache miss: no previous cache entry found
+      → Cache not updated: task failed
+━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+[1]> vite run test
+$ node -e "process.exit(1)"
+
+
+━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+    Vite+ Task Runner • Execution Summary
+━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+Statistics:   1 tasks • 0 cache hits • 1 cache misses • 1 failed 
+Performance:  0% cache hit rate
+
+Task Details:
+────────────────────────────────────────────────
+  [1] cache-non-zero-exit-test#test: $ node -e "process.exit(1)" ✗ (exit code: 1)
+      → Cache miss: no previous cache entry found
+      → Cache not updated: task failed
+━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

crates/vite_task_bin/tests/e2e_snapshots/fixtures/exit-codes/snapshots/multiple task failures returns exit code 1.snap

@@ -21,7 +21,9 @@ Task Details:
 ────────────────────────────────────────────────
   [1] pkg-b#fail: ~/packages/pkg-b$ node -e "process.exit(7)" ✗ (exit code: 7)
       → Cache miss: no previous cache entry found
+      → Cache not updated: task failed
   ·······················································
   [2] pkg-a#fail: ~/packages/pkg-a$ node -e "process.exit(42)" ✗ (exit code: 42)
       → Cache miss: no previous cache entry found
+      → Cache not updated: task failed
 ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

crates/vite_task_bin/tests/e2e_snapshots/fixtures/exit-codes/snapshots/single task failure returns task exit code.snap

@@ -19,4 +19,5 @@ Task Details:
 ────────────────────────────────────────────────
   [1] pkg-a#fail: ~/packages/pkg-a$ node -e "process.exit(42)" ✗ (exit code: 42)
       → Cache miss: no previous cache entry found
+      → Cache not updated: task failed
 ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━