[rush] fix: wire ProblemCollector correctly into the terminal pipeline (#5707)

* fix: add preventAutoclose to ProblemCollector, inject strip-colors transform, always wire problemCollector

Co-authored-by: dmichon-msft <26827560+dmichon-msft@users.noreply.github.com>
Agent-Logs-Url: https://github.com/microsoft/rushstack/sessions/8b2eb275-c191-44e9-8d69-dd1c166c7985

* fix: add @microsoft/rush patch change file

Co-authored-by: dmichon-msft <26827560+dmichon-msft@users.noreply.github.com>
Agent-Logs-Url: https://github.com/microsoft/rushstack/sessions/1bcedf3e-f266-4978-ad80-c7ccb325056a

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: dmichon-msft <26827560+dmichon-msft@users.noreply.github.com>

Author: Copilot

common/changes/@microsoft/rush/fix-problem-collector-pipeline_2026-03-20-22-38.json

@@ -0,0 +1,10 @@
+{
+  "changes": [
+    {
+      "packageName": "@microsoft/rush",
+      "comment": "Fix ProblemCollector wiring in OperationExecutionRecord: add preventAutoclose, strip ANSI colors before matching, and always include problemCollector in the terminal pipeline",
+      "type": "patch"
+    }
+  ],
+  "packageName": "@microsoft/rush"
+}

libraries/rush-lib/src/logic/operations/OperationExecutionRecord.ts

@@ -114,6 +114,8 @@ export class OperationExecutionRecord implements IOperationRunnerContext, IOpera
     preventAutoclose: true
   });
   public readonly problemCollector: ProblemCollector = new ProblemCollector({
+    // Allow writing to this object after transforms have been closed. We clean it up manually in a finally block.
+    preventAutoclose: true,
     matcherJson: [
       {
         name: 'rushstack-file-error-unix',
@@ -324,14 +326,21 @@ export class OperationExecutionRecord implements IOperationRunnerContext, IOpera
       //
       //                             +--> quietModeTransform? --> collatedWriter
       //                             |
-      // normalizeNewlineTransform --1--> stderrLineTransform --2--> projectLogWritable
+      // normalizeNewlineTransform --1--> stderrLineTransform --2--> projectLogWritable?
       //                                                        |
       //                                                        +--> stdioSummarizer
-      const destination: TerminalWritable = projectLogWritable
-        ? new SplitterTransform({
-            destinations: [projectLogWritable, stdioSummarizer, problemCollector]
-          })
-        : stdioSummarizer;
+      //                                                        |
+      //                                                        +--> removeColorsTransform --> problemCollector
+      const removeColorsTransform: TextRewriterTransform = new TextRewriterTransform({
+        destination: problemCollector,
+        removeColors: true
+      });
+
+      const destination: TerminalWritable = new SplitterTransform({
+        destinations: projectLogWritable
+          ? [projectLogWritable, stdioSummarizer, removeColorsTransform]
+          : [stdioSummarizer, removeColorsTransform]
+      });
 
       const stderrLineTransform: StderrLineTransform = new StderrLineTransform({
         destination,