feat(reporter): return proper exit codes based on task failures

- Return 0 if all tasks succeed
- Return the task's exit code if exactly one task failed
- Return 1 if more than one task failed

Update vite_task_bin to propagate the exit code to the process.
Add e2e tests to verify exit code behavior.

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

Author: branchseer

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

@@ -1,7 +1,7 @@
 pub mod fingerprint;
 pub mod spawn;
 
-use std::sync::Arc;
+use std::{process::ExitCode, sync::Arc};
 
 use futures_util::FutureExt;
 use petgraph::{algo::toposort, graph::DiGraph};
@@ -328,11 +328,10 @@ impl ExecutionContext<'_> {
 }
 
 impl<'a, CustomSubcommand> Session<'a, CustomSubcommand> {
-    pub async fn execute(
-        &self,
-        plan: ExecutionPlan,
-        mut reporter: Box<dyn Reporter>,
-    ) -> anyhow::Result<()> {
+    /// Execute an execution plan, reporting events to the provided reporter.
+    ///
+    /// Returns the ExitCode from the reporter's post_execution method.
+    pub async fn execute(&self, plan: ExecutionPlan, mut reporter: Box<dyn Reporter>) -> ExitCode {
         let mut execution_context = ExecutionContext {
             event_handler: &mut *reporter,
             current_execution_id: ExecutionId::zero(),

crates/vite_task/src/session/reporter.rs

@@ -3,6 +3,7 @@
 use std::{
     collections::HashSet,
     io::Write,
+    process::ExitCode,
     sync::{Arc, LazyLock},
     time::Duration,
 };
@@ -34,8 +35,8 @@ pub trait Reporter {
     fn handle_event(&mut self, event: ExecutionEvent);
 
     /// Called after execution completes (whether successful or not)
-    /// Returns Err if execution failed due to errors
-    fn post_execution(self: Box<Self>) -> anyhow::Result<()>;
+    /// Returns an ExitCode that is a suggestion for process exit
+    fn post_execution(self: Box<Self>) -> ExitCode;
 }
 
 const COMMAND_STYLE: Style = Style::new().cyan();
@@ -486,7 +487,7 @@ impl<W: Write> Reporter for LabeledReporter<W> {
         }
     }
 
-    fn post_execution(mut self: Box<Self>) -> anyhow::Result<()> {
+    fn post_execution(mut self: Box<Self>) -> ExitCode {
         // Check if execution was aborted due to error
         if let Some(error_msg) = &self.first_error {
             // Print separator
@@ -513,7 +514,7 @@ impl<W: Write> Reporter for LabeledReporter<W> {
                     .style(Style::new().bright_black())
             );
 
-            return Err(anyhow::anyhow!("Execution aborted: {}", error_msg));
+            return ExitCode::FAILURE;
         }
 
         // No errors - print summary if not hidden
@@ -525,6 +526,26 @@ impl<W: Write> Reporter for LabeledReporter<W> {
                 self.print_summary();
             }
         }
-        Ok(())
+
+        // Determine exit code based on failed tasks:
+        // 1. All tasks succeed → return 0
+        // 2. Exactly one task failed → return that task's exit code
+        // 3. More than one task failed → return 1
+        let failed_exit_codes: Vec<i32> = self
+            .executions
+            .iter()
+            .filter_map(|exec| exec.exit_status)
+            .filter(|&status| status != 0)
+            .collect();
+
+        match failed_exit_codes.len() {
+            0 => ExitCode::SUCCESS,
+            1 => {
+                // Return the single failed task's exit code (clamped to u8 range)
+                let code = failed_exit_codes[0];
+                ExitCode::from(code.clamp(1, 255) as u8)
+            }
+            _ => ExitCode::FAILURE,
+        }
     }
 }

crates/vite_task_bin/src/main.rs

@@ -1,11 +1,21 @@
-use std::{env, sync::Arc};
+use std::{env, process::ExitCode, sync::Arc};
 
 use vite_path::{AbsolutePath, current_dir};
 use vite_task::{CLIArgs, Session, session::reporter::LabeledReporter};
 use vite_task_bin::{CustomTaskSubcommand, NonTaskSubcommand, OwnedSessionCallbacks};
 
 #[tokio::main]
-async fn main() -> anyhow::Result<()> {
+async fn main() -> ExitCode {
+    match run().await {
+        Ok(exit_code) => exit_code,
+        Err(err) => {
+            eprintln!("Error: {err}");
+            ExitCode::FAILURE
+        }
+    }
+}
+
+async fn run() -> anyhow::Result<ExitCode> {
     let cwd: Arc<AbsolutePath> = current_dir()?.into();
     // Parse the CLI arguments and see if they are for vite-task or not
     let args = match CLIArgs::<CustomTaskSubcommand, NonTaskSubcommand>::try_parse_from(env::args())
@@ -20,7 +30,7 @@ async fn main() -> anyhow::Result<()> {
         CLIArgs::NonTask(NonTaskSubcommand::Version) => {
             // Non-task subcommands are not handled by vite-task's session.
             println!("{}", env!("CARGO_PKG_VERSION"));
-            return Ok(());
+            return Ok(ExitCode::SUCCESS);
         }
     };
 
@@ -30,7 +40,7 @@ async fn main() -> anyhow::Result<()> {
 
     // Create reporter and execute
     let reporter = LabeledReporter::new(std::io::stdout(), session.workspace_path());
-    session.execute(plan, Box::new(reporter)).await?;
+    let exit_code = session.execute(plan, Box::new(reporter)).await;
 
-    Ok(())
+    Ok(exit_code)
 }

crates/vite_task_bin/tests/test_snapshots/fixtures/exit-codes/package.json

@@ -0,0 +1,4 @@
+{
+  "name": "exit-codes-test",
+  "private": true
+}

crates/vite_task_bin/tests/test_snapshots/fixtures/exit-codes/packages/pkg-a/package.json

@@ -0,0 +1,6 @@
+{
+  "name": "pkg-a",
+  "scripts": {
+    "fail": "node -e \"process.exit(42)\""
+  }
+}

crates/vite_task_bin/tests/test_snapshots/fixtures/exit-codes/packages/pkg-b/package.json

@@ -0,0 +1,6 @@
+{
+  "name": "pkg-b",
+  "scripts": {
+    "fail": "node -e \"process.exit(7)\""
+  }
+}

crates/vite_task_bin/tests/test_snapshots/fixtures/exit-codes/pnpm-workspace.yaml

@@ -0,0 +1,2 @@
+packages:
+  - packages/*

crates/vite_task_bin/tests/test_snapshots/fixtures/exit-codes/snapshots.toml

@@ -0,0 +1,11 @@
+[[e2e]]
+name = "single task failure returns task exit code"
+steps = [
+  "vite run pkg-a#fail",
+]
+
+[[e2e]]
+name = "multiple task failures returns exit code 1"
+steps = [
+  "vite run -r fail",
+]

crates/vite_task_bin/tests/test_snapshots/snapshots/test_snapshots__cycle dependency error@error_cycle_dependency.snap

@@ -1,5 +1,6 @@
 ---
 source: crates/vite_task_bin/tests/test_snapshots/main.rs
+assertion_line: 203
 expression: e2e_outputs
 input_file: crates/vite_task_bin/tests/test_snapshots/fixtures/error_cycle_dependency
 ---
@@ -10,4 +11,3 @@ input_file: crates/vite_task_bin/tests/test_snapshots/fixtures/error_cycle_depen
 ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
 Execution aborted due to error: Cycle dependencies detected: Cycle(NodeIndex(ExecutionIx(1)))
 ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
-Error: Execution aborted: Cycle dependencies detected: Cycle(NodeIndex(ExecutionIx(1)))

crates/vite_task_bin/tests/test_snapshots/snapshots/test_snapshots__multiple task failures returns exit code 1@exit-codes.snap

@@ -0,0 +1,27 @@
+---
+source: crates/vite_task_bin/tests/test_snapshots/main.rs
+assertion_line: 203
+expression: e2e_outputs
+input_file: crates/vite_task_bin/tests/test_snapshots/fixtures/exit-codes
+---
+[1]> vite run -r fail
+~/packages/pkg-b$ node -e "process.exit(7)"
+
+~/packages/pkg-a$ node -e "process.exit(42)"
+
+
+━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+    Vite+ Task Runner • Execution Summary
+━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
+
+Statistics:   2 tasks • 0 cache hits • 2 cache misses • 2 failed 
+Performance:  0% cache hit rate
+
+Task Details:
+────────────────────────────────────────────────
+  [1] pkg-b#fail: ~/packages/pkg-b$ node -e "process.exit(7)" ✗ (exit code: 7)
+      → Cache miss: no previous cache entry found
+  ·······················································
+  [2] pkg-a#fail: ~/packages/pkg-a$ node -e "process.exit(42)" ✗ (exit code: 42)
+      → Cache miss: no previous cache entry found
+━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━