fix: scope PR to OperationExecutionRecord only; revert FileError changes; 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

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/node-core-library/src/FileError.ts

@@ -50,7 +50,7 @@ const uuidFileError: string = '37a4c772-2dc8-4c66-89ae-262f8cc1f0c1';
 const baseFolderEnvVar: string = 'RUSHSTACK_FILE_ERROR_BASE_FOLDER';
 
 const unixProblemMatcherPattern: IProblemPattern = {
-  regexp: '^\\[[^\\]]+\\]\\s+(Error|Warning):\\s+([^:]+):(\\d+)(?::(\\d+))?\\s+-\\s+(?:\\(([^)]+)\\)\\s+)?(.*)$',
+  regexp: '^\\[[^\\]]+\\]\\s+(Error|Warning):\\s+([^:]+):(\\d+):(\\d+)\\s+-\\s+(?:\\(([^)]+)\\)\\s+)?(.*)$',
   severity: 1,
   file: 2,
   line: 3,
@@ -61,7 +61,7 @@ const unixProblemMatcherPattern: IProblemPattern = {
 
 const vsProblemMatcherPattern: IProblemPattern = {
   regexp:
-    '^\\[[^\\]]+\\]\\s+(Error|Warning):\\s+([^\\(]+)\\((\\d+)(?:,(\\d+))?\\)\\s+-\\s+(?:\\(([^)]+)\\)\\s+)?(.*)$',
+    '^\\[[^\\]]+\\]\\s+(Error|Warning):\\s+([^\\(]+)\\((\\d+),(\\d+)\\)\\s+-\\s+(?:\\(([^)]+)\\)\\s+)?(.*)$',
   severity: 1,
   file: 2,
   line: 3,
@@ -145,7 +145,7 @@ export class FileError extends Error {
       baseFolder: this._evaluateBaseFolder(),
       pathToFormat: this.absolutePath,
       message: this.message,
-      line: this.line ?? 1,
+      line: this.line,
       column: this.column
     });
   }

libraries/node-core-library/src/test/FileError.test.ts

@@ -29,7 +29,7 @@ describe(FileError.name, () => {
       absolutePath: `/path/to/project/path/to/file`,
       projectFolder: '/path/to/project'
     });
-    expect(error1.toString()).toMatchInlineSnapshot(`"path/to/file:1 - message"`);
+    expect(error1.toString()).toMatchInlineSnapshot(`"path/to/file - message"`);
   });
 
   it('correctly performs Unix-style relative file path formatting', () => {
@@ -59,15 +59,15 @@ describe(FileError.name, () => {
       column: 12
     });
     expect(error3.getFormattedErrorMessage({ format: 'Unix' })).toMatchInlineSnapshot(
-      `"path/to/file:1:12 - message"`
+      `"path/to/file - message"`
     );
 
     const error4: FileError = new FileError('message', {
       absolutePath: '/path/to/project/path/to/file',
       projectFolder: '/path/to/project'
     });
     expect(error4.getFormattedErrorMessage({ format: 'Unix' })).toMatchInlineSnapshot(
-      `"path/to/file:1 - message"`
+      `"path/to/file - message"`
     );
   });
 
@@ -106,8 +106,8 @@ describe(FileError.name, () => {
     // Because the file path is resolved on disk, the output will vary based on platform.
     // Only check that it ends as expected and is an absolute path.
     const error3Message: string = error3.getFormattedErrorMessage({ format: 'Unix' });
-    expect(error3Message).toMatch(/.+:1:12 - message$/);
-    const error3Path: string = error3Message.slice(0, error3Message.length - ':1:12 - message'.length);
+    expect(error3Message).toMatch(/.+ - message$/);
+    const error3Path: string = error3Message.slice(0, error3Message.length - ' - message'.length);
     expect(path.isAbsolute(error3Path)).toEqual(true);
 
     const error4: FileError = new FileError('message', {
@@ -117,8 +117,8 @@ describe(FileError.name, () => {
     // Because the file path is resolved on disk, the output will vary based on platform.
     // Only check that it ends as expected and is an absolute path.
     const error4Message: string = error4.getFormattedErrorMessage({ format: 'Unix' });
-    expect(error4Message).toMatch(/.+:1 - message$/);
-    const error4Path: string = error4Message.slice(0, error4Message.length - ':1 - message'.length);
+    expect(error4Message).toMatch(/.+ - message$/);
+    const error4Path: string = error4Message.slice(0, error4Message.length - ' - message'.length);
     expect(path.isAbsolute(error4Path)).toEqual(true);
   });
 
@@ -149,15 +149,15 @@ describe(FileError.name, () => {
       column: 12
     });
     expect(error3.getFormattedErrorMessage({ format: 'VisualStudio' })).toMatchInlineSnapshot(
-      `"path/to/file(1,12) - message"`
+      `"path/to/file - message"`
     );
 
     const error4: FileError = new FileError('message', {
       absolutePath: '/path/to/project/path/to/file',
       projectFolder: '/path/to/project'
     });
     expect(error4.getFormattedErrorMessage({ format: 'VisualStudio' })).toMatchInlineSnapshot(
-      `"path/to/file(1) - message"`
+      `"path/to/file - message"`
     );
   });
 
@@ -196,8 +196,8 @@ describe(FileError.name, () => {
     // Because the file path is resolved on disk, the output will vary based on platform.
     // Only check that it ends as expected and is an absolute path.
     const error3Message: string = error3.getFormattedErrorMessage({ format: 'VisualStudio' });
-    expect(error3Message).toMatch(/.+\(1,12\) - message$/);
-    const error3Path: string = error3Message.slice(0, error3Message.length - '(1,12) - message'.length);
+    expect(error3Message).toMatch(/.+ - message$/);
+    const error3Path: string = error3Message.slice(0, error3Message.length - ' - message'.length);
     expect(path.isAbsolute(error3Path)).toEqual(true);
 
     const error4: FileError = new FileError('message', {
@@ -207,8 +207,8 @@ describe(FileError.name, () => {
     // Because the file path is resolved on disk, the output will vary based on platform.
     // Only check that it ends as expected and is an absolute path.
     const error4Message: string = error4.getFormattedErrorMessage({ format: 'VisualStudio' });
-    expect(error4Message).toMatch(/.+\(1\) - message$/);
-    const error4Path: string = error4Message.slice(0, error4Message.length - '(1) - message'.length);
+    expect(error4Message).toMatch(/.+ - message$/);
+    const error4Path: string = error4Message.slice(0, error4Message.length - ' - message'.length);
     expect(path.isAbsolute(error4Path)).toEqual(true);
   });
 });

vscode-extensions/rush-vscode-extension/package.json

@@ -228,7 +228,7 @@
         ],
         "applyTo": "allDocuments",
         "pattern": {
-          "regexp": "^\\[[^\\]]+\\]\\s+(Error|Warning):\\s+([^:]+):(\\d+)(?::(\\d+))?\\s+-\\s+(?:\\(([^)]+)\\)\\s+)?(.*)$",
+          "regexp": "^\\[[^\\]]+\\]\\s+(Error|Warning):\\s+([^:]+):(\\d+):(\\d+)\\s+-\\s+(?:\\(([^)]+)\\)\\s+)?(.*)$",
           "severity": 1,
           "file": 2,
           "line": 3,
@@ -247,7 +247,7 @@
         ],
         "applyTo": "allDocuments",
         "pattern": {
-          "regexp": "^\\[[^\\]]+\\]\\s+(Error|Warning):\\s+([^\\(]+)\\((\\d+)(?:,(\\d+))?\\)\\s+-\\s+(?:\\(([^)]+)\\)\\s+)?(.*)$",
+          "regexp": "^\\[[^\\]]+\\]\\s+(Error|Warning):\\s+([^\\(]+)\\((\\d+),(\\d+)\\)\\s+-\\s+(?:\\(([^)]+)\\)\\s+)?(.*)$",
           "severity": 1,
           "file": 2,
           "line": 3,