fix: correct parser patterns validated against real xcodebuild output

Fixes discovered by auditing synthetic test data against real swift test
and xcodebuild output:

- Swift Testing verbose mode: (aka 'func()') suffix now optionally
  matched in result and issue parsers
- Skipped test format: real output uses arrow symbol and bare function
  name, not diamond with quoted name. Both formats now handled.
- Parameterized tests: 'with N test cases' suffix in results and
  'with N argument value' in issues now matched
- Build errors without line numbers: .xcodeproj path-based errors
  like 'Missing package product' now parsed
- Stage detection: lowercase 'Test suite' from Xcode 26 now matched

Tests updated to use patterns captured from real swift test and
xcodebuild runs.

Author: cameroncooke

src/utils/__tests__/swift-testing-line-parsers.test.ts

@@ -21,6 +21,30 @@ describe('Swift Testing line parsers', () => {
       });
     });
 
+    it('should parse a passed test with verbose aka suffix', () => {
+      const result = parseSwiftTestingResultLine(
+        "✔ Test \"String operations\" (aka 'stringTest()') passed after 0.001 seconds.",
+      );
+      expect(result).toEqual({
+        status: 'passed',
+        rawName: 'String operations',
+        testName: 'String operations',
+        durationText: '0.001s',
+      });
+    });
+
+    it('should parse a passed parameterized test', () => {
+      const result = parseSwiftTestingResultLine(
+        '✔ Test "Parameterized test" with 3 test cases passed after 0.001 seconds.',
+      );
+      expect(result).toEqual({
+        status: 'passed',
+        rawName: 'Parameterized test',
+        testName: 'Parameterized test',
+        durationText: '0.001s',
+      });
+    });
+
     it('should parse a failed test', () => {
       const result = parseSwiftTestingResultLine(
         '✘ Test "Expected failure" failed after 0.001 seconds with 1 issue.',
@@ -33,7 +57,40 @@ describe('Swift Testing line parsers', () => {
       });
     });
 
-    it('should parse a skipped test', () => {
+    it('should parse a failed test with verbose aka suffix', () => {
+      const result = parseSwiftTestingResultLine(
+        "✘ Test \"Expected failure\" (aka 'deliberateFailure()') failed after 0.001 seconds with 1 issue.",
+      );
+      expect(result).toEqual({
+        status: 'failed',
+        rawName: 'Expected failure',
+        testName: 'Expected failure',
+        durationText: '0.001s',
+      });
+    });
+
+    it('should parse a failed parameterized test', () => {
+      const result = parseSwiftTestingResultLine(
+        '✘ Test "Parameterized failure" with 3 test cases failed after 0.001 seconds with 1 issue.',
+      );
+      expect(result).toEqual({
+        status: 'failed',
+        rawName: 'Parameterized failure',
+        testName: 'Parameterized failure',
+        durationText: '0.001s',
+      });
+    });
+
+    it('should parse a skipped test (arrow format)', () => {
+      const result = parseSwiftTestingResultLine('➜ Test disabledTest() skipped: "Not ready yet"');
+      expect(result).toEqual({
+        status: 'skipped',
+        rawName: 'disabledTest',
+        testName: 'disabledTest',
+      });
+    });
+
+    it('should parse a skipped test (legacy diamond format)', () => {
       const result = parseSwiftTestingResultLine('◇ Test "Disabled test" skipped.');
       expect(result).toEqual({
         status: 'skipped',
@@ -42,6 +99,15 @@ describe('Swift Testing line parsers', () => {
       });
     });
 
+    it('should parse a skipped test without reason', () => {
+      const result = parseSwiftTestingResultLine('➜ Test disabledTest skipped');
+      expect(result).toEqual({
+        status: 'skipped',
+        rawName: 'disabledTest',
+        testName: 'disabledTest',
+      });
+    });
+
     it('should return null for non-matching lines', () => {
       expect(parseSwiftTestingResultLine('◇ Test "Foo" started.')).toBeNull();
       expect(parseSwiftTestingResultLine('random text')).toBeNull();
@@ -61,6 +127,30 @@ describe('Swift Testing line parsers', () => {
       });
     });
 
+    it('should parse an issue with verbose aka suffix', () => {
+      const result = parseSwiftTestingIssueLine(
+        "✘ Test \"Expected failure\" (aka 'deliberateFailure()') recorded an issue at AuditTests.swift:5:5: Expectation failed: true == false",
+      );
+      expect(result).toEqual({
+        rawTestName: 'Expected failure',
+        testName: 'Expected failure',
+        location: 'AuditTests.swift:5',
+        message: 'Expectation failed: true == false',
+      });
+    });
+
+    it('should parse a parameterized issue with argument values', () => {
+      const result = parseSwiftTestingIssueLine(
+        '✘ Test "Parameterized failure" recorded an issue with 1 argument value → 0 at ParameterizedTests.swift:10:5: Expectation failed: (value → 0) > 0',
+      );
+      expect(result).toEqual({
+        rawTestName: 'Parameterized failure',
+        testName: 'Parameterized failure',
+        location: 'ParameterizedTests.swift:10',
+        message: 'Expectation failed: (value → 0) > 0',
+      });
+    });
+
     it('should parse an issue without location', () => {
       const result = parseSwiftTestingIssueLine(
         '✘ Test "Some test" recorded an issue: Something went wrong',
@@ -72,6 +162,17 @@ describe('Swift Testing line parsers', () => {
       });
     });
 
+    it('should parse an issue without location with verbose aka suffix', () => {
+      const result = parseSwiftTestingIssueLine(
+        "✘ Test \"Some test\" (aka 'someFunc()') recorded an issue: Something went wrong",
+      );
+      expect(result).toEqual({
+        rawTestName: 'Some test',
+        testName: 'Some test',
+        message: 'Something went wrong',
+      });
+    });
+
     it('should return null for non-matching lines', () => {
       expect(parseSwiftTestingIssueLine('✘ Test "Foo" failed after 0.001 seconds')).toBeNull();
     });
@@ -100,6 +201,17 @@ describe('Swift Testing line parsers', () => {
       });
     });
 
+    it('should parse a summary with singular suite', () => {
+      const result = parseSwiftTestingRunSummary(
+        '✘ Test run with 5 tests in 1 suite failed after 0.001 seconds with 3 issues.',
+      );
+      expect(result).toEqual({
+        executed: 5,
+        failed: 3,
+        durationText: '0.001s',
+      });
+    });
+
     it('should return null for non-matching lines', () => {
       expect(parseSwiftTestingRunSummary('random text')).toBeNull();
     });
@@ -112,6 +224,12 @@ describe('Swift Testing line parsers', () => {
       );
     });
 
+    it('should parse a continuation with version info', () => {
+      expect(parseSwiftTestingContinuationLine('↳ Testing Library Version: 1743')).toBe(
+        'Testing Library Version: 1743',
+      );
+    });
+
     it('should return null for non-continuation lines', () => {
       expect(parseSwiftTestingContinuationLine('regular line')).toBeNull();
     });

src/utils/swift-testing-line-parsers.ts

@@ -5,17 +5,29 @@ import {
   parseRawTestName,
 } from './xcodebuild-line-parsers.ts';
 
+// Optional verbose suffix: (aka 'funcName()')
+// Optional parameterized suffix: with N test cases
+const OPTIONAL_AKA = `(?:\\s*\\(aka '[^']*'\\))?`;
+const OPTIONAL_PARAMETERIZED = `(?:\\s+with \\d+ test cases?)?`;
+
 /**
  * Parse a Swift Testing result line (passed/failed/skipped).
  *
- * Matches:
+ * Matches (non-verbose and verbose):
  *   ✔ Test "Name" passed after 0.001 seconds.
+ *   ✔ Test "Name" (aka 'func()') passed after 0.001 seconds.
+ *   ✔ Test "Name" with 3 test cases passed after 0.001 seconds.
  *   ✘ Test "Name" failed after 0.001 seconds with 1 issue.
- *   ✘ Test "Name" failed after 0.001 seconds with 3 issues.
- *   ◇ Test "Name" skipped.
+ *   ✘ Test "Name" (aka 'func()') failed after 0.001 seconds with 1 issue.
+ *   ➜ Test funcName() skipped: "reason"
+ *   ➜ Test funcName() skipped
  */
 export function parseSwiftTestingResultLine(line: string): ParsedTestCase | null {
-  const passedMatch = line.match(/^[✔] Test "(.+)" passed after ([\d.]+) seconds\.?$/u);
+  const passedRegex = new RegExp(
+    `^[✔] Test "(.+)"${OPTIONAL_AKA}${OPTIONAL_PARAMETERIZED} passed after ([\\d.]+) seconds\\.?$`,
+    'u',
+  );
+  const passedMatch = line.match(passedRegex);
   if (passedMatch) {
     const [, name, duration] = passedMatch;
     const { suiteName, testName } = parseRawTestName(name);
@@ -28,7 +40,11 @@ export function parseSwiftTestingResultLine(line: string): ParsedTestCase | null
     };
   }
 
-  const failedMatch = line.match(/^[✘] Test "(.+)" failed after ([\d.]+) seconds/u);
+  const failedRegex = new RegExp(
+    `^[✘] Test "(.+)"${OPTIONAL_AKA}${OPTIONAL_PARAMETERIZED} failed after ([\\d.]+) seconds`,
+    'u',
+  );
+  const failedMatch = line.match(failedRegex);
   if (failedMatch) {
     const [, name, duration] = failedMatch;
     const { suiteName, testName } = parseRawTestName(name);
@@ -41,7 +57,11 @@ export function parseSwiftTestingResultLine(line: string): ParsedTestCase | null
     };
   }
 
-  const skippedMatch = line.match(/^[◇] Test "(.+)" skipped/u);
+  // Skipped: ➜ Test funcName() skipped: "reason"
+  // Also handle legacy format: ◇ Test "Name" skipped
+  const skippedMatch =
+    line.match(/^[➜] Test (\S+?)(?:\(\))? skipped/u) ??
+    line.match(/^[◇] Test "(.+)" skipped/u);
   if (skippedMatch) {
     const rawName = skippedMatch[1];
     const { suiteName, testName } = parseRawTestName(rawName);
@@ -59,12 +79,19 @@ export function parseSwiftTestingResultLine(line: string): ParsedTestCase | null
 /**
  * Parse a Swift Testing issue line.
  *
- * Matches:
+ * Matches (non-verbose and verbose, including parameterized):
  *   ✘ Test "Name" recorded an issue at File.swift:48:5: Expectation failed: ...
+ *   ✘ Test "Name" (aka 'func()') recorded an issue at File.swift:48:5: msg
+ *   ✘ Test "Name" recorded an issue with 1 argument value → 0 at File.swift:10:5: msg
  *   ✘ Test "Name" recorded an issue: message
  */
 export function parseSwiftTestingIssueLine(line: string): ParsedFailureDiagnostic | null {
-  const locationMatch = line.match(/^[✘] Test "(.+)" recorded an issue at (.+?):(\d+):\d+: (.+)$/u);
+  // Match with location -- handle both aka suffix and parameterized argument values before "at"
+  const locationRegex = new RegExp(
+    `^[✘] Test "(.+)"${OPTIONAL_AKA} recorded an issue(?:\\s+with \\d+ argument values?[^:]*?)? at (.+?):(\\d+):\\d+: (.+)$`,
+    'u',
+  );
+  const locationMatch = line.match(locationRegex);
   if (locationMatch) {
     const [, rawTestName, filePath, lineNumber, message] = locationMatch;
     const { suiteName, testName } = parseRawTestName(rawTestName);
@@ -77,7 +104,12 @@ export function parseSwiftTestingIssueLine(line: string): ParsedFailureDiagnosti
     };
   }
 
-  const simpleMatch = line.match(/^[✘] Test "(.+)" recorded an issue: (.+)$/u);
+  // Match without location
+  const simpleRegex = new RegExp(
+    `^[✘] Test "(.+)"${OPTIONAL_AKA} recorded an issue: (.+)$`,
+    'u',
+  );
+  const simpleMatch = line.match(simpleRegex);
   if (simpleMatch) {
     const [, rawTestName, message] = simpleMatch;
     const { suiteName, testName } = parseRawTestName(rawTestName);

src/utils/xcodebuild-event-parser.ts

@@ -33,7 +33,7 @@ function resolveStageFromLine(line: string): XcodebuildStage | null {
   }
   if (
     /^Testing started$/u.test(line) ||
-    /^Test Suite .+ started/u.test(line) ||
+    /^Test [Ss]uite .+ started/u.test(line) ||
     /^[◇] Test run started/u.test(line)
   ) {
     return 'RUN_TESTS';
@@ -373,7 +373,7 @@ export function createXcodebuildEventParser(options: EventParserOptions): Xcodeb
       return;
     }
 
-    if (/^Test Suite /u.test(line)) {
+    if (/^Test [Ss]uite /u.test(line)) {
       return;
     }
 

src/utils/xcodebuild-line-parsers.ts

@@ -141,6 +141,7 @@ export function parseDurationMs(durationText?: string): number | undefined {
 }
 
 export function parseBuildErrorDiagnostic(line: string): ParsedBuildError | null {
+  // File path with line number: /path/to/File.swift:42:10: error: message
   const locationMatch = line.match(/^(.*?):(\d+)(?::\d+)?: (?:fatal error|error): (.+)$/u);
   if (locationMatch) {
     const [, filePath, lineNumber, message] = locationMatch;
@@ -151,6 +152,18 @@ export function parseBuildErrorDiagnostic(line: string): ParsedBuildError | null
     };
   }
 
+  // Path-based error without line number: /path/to/Project.xcodeproj: error: message
+  const pathErrorMatch = line.match(/^(\/[^:]+): (?:fatal error|error): (.+)$/u);
+  if (pathErrorMatch) {
+    const [, filePath, message] = pathErrorMatch;
+    return {
+      location: filePath,
+      message,
+      renderedLine: line,
+    };
+  }
+
+  // Prefixed error: xcodebuild: error: message / error: message
   const rawMatch = line.match(/^(?:[\w-]+:\s+)?(?:fatal error|error): (.+)$/u);
   if (!rawMatch) {
     return null;