fix: Revert interactive/stream change.

Author: milesj

.yarn/versions/7ea1a917.yml

@@ -0,0 +1,9 @@
+releases:
+  "@moonrepo/cli": patch
+  "@moonrepo/core-linux-arm64-gnu": patch
+  "@moonrepo/core-linux-arm64-musl": patch
+  "@moonrepo/core-linux-x64-gnu": patch
+  "@moonrepo/core-linux-x64-musl": patch
+  "@moonrepo/core-macos-arm64": patch
+  "@moonrepo/core-macos-x64": patch
+  "@moonrepo/core-windows-x64-msvc": patch

CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## Unreleased
+
+#### 🐞 Fixes
+
+- Reverted "Fixed an issue where task output wasn't captured for the "review" section in a run
+  summary." as it caused a regression.
+
 ## 1.32.8
 
 #### 🐞 Fixes

crates/task-runner/src/command_executor.rs

@@ -113,8 +113,8 @@ impl<'task> CommandExecutor<'task> {
                 interactive: bool,
             ) -> miette::Result<Output> {
                 match (stream, interactive) {
-                    (true, false) | (true, true) => command.exec_stream_and_capture_output().await,
-                    (false, true) => command.exec_stream_output().await,
+                    (true, true) | (false, true) => command.exec_stream_output().await,
+                    (true, false) => command.exec_stream_and_capture_output().await,
                     _ => command.exec_capture_output().await,
                 }
             }
@@ -293,10 +293,11 @@ impl<'task> CommandExecutor<'task> {
 
     fn prepare_state(&mut self, context: &ActionContext, report_item: &mut TaskReportItem) {
         let is_primary = context.is_primary_target(&self.task.target);
+        let is_ci = is_ci_env();
 
         // When a task is configured as local (no caching), or the interactive flag is passed,
         // we don't "capture" stdout/stderr (which breaks stdin) and let it stream natively.
-        if !self.task.options.cache && context.primary_targets.len() == 1 {
+        if !self.task.options.cache && context.primary_targets.len() == 1 && !is_ci {
             self.interactive = true;
         }
 
@@ -305,7 +306,7 @@ impl<'task> CommandExecutor<'task> {
         self.stream = if let Some(output_style) = &self.task.options.output_style {
             matches!(output_style, TaskOutputStyle::Stream)
         } else {
-            is_primary || is_ci_env()
+            is_primary || is_ci
         };
 
         // If only a single persistent task is being ran, we should not prefix the output.