Improve build failure detection from XCLogParser

Author: aehlke

__tests__/utils/build-log-parser.vitest.test.ts

@@ -0,0 +1,81 @@
+import { describe, it, expect, beforeEach, vi } from 'vitest';
+import { EventEmitter } from 'events';
+
+const spawnMock = vi.hoisted(() => vi.fn()) as ReturnType<typeof vi.fn>;
+
+vi.mock('child_process', () => ({
+  spawn: spawnMock
+}));
+
+import { BuildLogParser } from '../../src/utils/BuildLogParser.js';
+
+class MockChildProcess extends EventEmitter {
+  public stdout: EventEmitter;
+  public stderr: EventEmitter;
+  public killed = false;
+
+  constructor(private readonly stdoutData: string, private readonly stderrData = '', private readonly exitCode = 0) {
+    super();
+    this.stdout = new EventEmitter();
+    this.stderr = new EventEmitter();
+    this.scheduleEmit();
+  }
+
+  kill(): boolean {
+    this.killed = true;
+    return true;
+  }
+
+  private scheduleEmit(): void {
+    setImmediate(() => {
+      if (this.stdoutData) {
+        this.stdout.emit('data', Buffer.from(this.stdoutData));
+      }
+      if (this.stderrData) {
+        this.stderr.emit('data', Buffer.from(this.stderrData));
+      }
+      this.emit('close', this.exitCode);
+    });
+  }
+}
+
+describe('BuildLogParser.parseBuildLog', () => {
+  beforeEach(() => {
+    spawnMock.mockReset();
+  });
+
+  it('falls back to summary data when issues reporter returns no errors but build failed', async () => {
+    spawnMock.mockImplementation((_command: string, args: string[]) => {
+      const reporterIndex = args.indexOf('--reporter');
+      const reporter = reporterIndex >= 0 ? args[reporterIndex + 1] : 'issues';
+
+      if (reporter === 'summaryJson') {
+        return new MockChildProcess(JSON.stringify({
+          buildStatus: 'failed',
+          errorCount: 1,
+          warnings: [],
+          errors: [],
+          notes: [
+            {
+              title: 'Command PhaseScriptExecution failed with a nonzero exit code',
+              documentURL: 'file:///mock/script.sh'
+            }
+          ]
+        }));
+      }
+
+      return new MockChildProcess(JSON.stringify({
+        errors: [],
+        warnings: []
+      }));
+    });
+
+    const result = await BuildLogParser.parseBuildLog('/mock/path/to/log.xcactivitylog');
+
+    expect(spawnMock).toHaveBeenCalledTimes(2);
+    expect(result.buildStatus).toBe('failed');
+    expect(result.errorCount).toBe(1);
+    expect(result.errors).not.toHaveLength(0);
+    expect(result.errors[0]).toContain("Xcode reported build status 'failed'");
+  });
+});

dist/types/index.d.ts

@@ -14,6 +14,8 @@ export interface ParsedBuildResults {
     errors: string[];
     warnings: string[];
     buildStatus?: string;
+    errorCount?: number;
+    warningCount?: number;
 }
 export interface EnvironmentValidationResult {
     valid: boolean;

dist/types/index.d.ts.map

@@ -6,6 +6,7 @@ export declare class BuildLogParser {
     static getRecentBuildLogs(projectPath: string, sinceTime: number): Promise<BuildLogInfo[]>;
     static getLatestTestLog(projectPath: string): Promise<BuildLogInfo | null>;
     static parseBuildLog(logPath: string, retryCount?: number, maxRetries?: number): Promise<ParsedBuildResults>;
+    private static parseBuildSummary;
     static canParseLog(logPath: string): Promise<boolean>;
     static parseTestResults(_xcresultPath: string): Promise<ParsedBuildResults>;
 }