fixup! fix(@angular/cli): strip terminal escape sequences in modern yarn error parser

Author: clydin

packages/angular/cli/src/package-managers/parsers.ts

@@ -697,21 +697,14 @@ export function parseYarnModernError(output: string, logger?: Logger): ErrorInfo
     }
   }
 
-  const [first, ...rest] = messages;
+  const [first] = messages;
   const code = statusCode ? `E${statusCode}` : first.code;
   logger?.debug(`  Successfully parsed modern yarn error with code '${code}'.`);
 
-  const [summary = '', ...firstDetailLines] = first.data
-    .split('\n')
+  const [summary = `Package manager error: ${code}`, ...detailLines] = messages
+    .flatMap(({ data }) => data.split('\n'))
     .map((line) => line.trim())
     .filter(Boolean);
-  const detailLines = [
-    ...firstDetailLines,
-    ...rest
-      .flatMap(({ data }) => data.split('\n'))
-      .map((line) => line.trim())
-      .filter(Boolean),
-  ];
 
   return {
     code,

packages/angular/cli/src/package-managers/parsers_spec.ts

@@ -732,6 +732,15 @@ project node_modules
       });
     });
 
+    it('should provide a fallback summary when error data is empty or whitespace-only', () => {
+      const stdout = '{"type":"error","name":35,"displayName":"YN0035","indent":"","data":"   "}';
+      expect(parseYarnModernError(stdout)).toEqual({
+        code: 'YN0035',
+        summary: 'Package manager error: YN0035',
+        detail: undefined,
+      });
+    });
+
     it('should return null for successful, empty, or unrecognized output', () => {
       expect(parseYarnModernError('{"name":"@angular/core","version":"22.2.4"}')).toBeNull();
       expect(parseYarnModernError('"@angular/core@npm:22.2.4"\n"rxjs@npm:7.8.2"')).toBeNull();