Remove unreachable parseXcodebuildSwiftTestingLine dead code

The updated parseTestCaseLine regex now matches both 'Test Case' and 'Test case'
with an optional 'on' clause, making it strictly more permissive than
parseXcodebuildSwiftTestingLine. Since parseTestCaseLine is checked first in
processLine, parseXcodebuildSwiftTestingLine became unreachable dead code.

Author: cursoragent

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

@@ -4,7 +4,6 @@ import {
   parseSwiftTestingIssueLine,
   parseSwiftTestingRunSummary,
   parseSwiftTestingContinuationLine,
-  parseXcodebuildSwiftTestingLine,
 } from '../swift-testing-line-parsers.ts';
 
 describe('Swift Testing line parsers', () => {
@@ -259,42 +258,4 @@ describe('Swift Testing line parsers', () => {
       expect(parseSwiftTestingContinuationLine('regular line')).toBeNull();
     });
   });
-
-  describe('parseXcodebuildSwiftTestingLine', () => {
-    it('should parse a passed test case', () => {
-      const result = parseXcodebuildSwiftTestingLine(
-        "Test case 'MCPTestTests/appNameIsCorrect()' passed on 'My Mac - MCPTest (78757)' (0.000 seconds)",
-      );
-      expect(result).toEqual({
-        status: 'passed',
-        rawName: 'MCPTestTests/appNameIsCorrect()',
-        suiteName: 'MCPTestTests',
-        testName: 'appNameIsCorrect()',
-        durationText: '0.000s',
-      });
-    });
-
-    it('should parse a failed test case', () => {
-      const result = parseXcodebuildSwiftTestingLine(
-        "Test case 'MCPTestTests/deliberateFailure()' failed on 'My Mac - MCPTest (78757)' (0.000 seconds)",
-      );
-      expect(result).toEqual({
-        status: 'failed',
-        rawName: 'MCPTestTests/deliberateFailure()',
-        suiteName: 'MCPTestTests',
-        testName: 'deliberateFailure()',
-        durationText: '0.000s',
-      });
-    });
-
-    it('should return null for XCTest format lines', () => {
-      expect(
-        parseXcodebuildSwiftTestingLine("Test Case '-[Suite test]' passed (0.001 seconds)."),
-      ).toBeNull();
-    });
-
-    it('should return null for non-matching lines', () => {
-      expect(parseXcodebuildSwiftTestingLine('random text')).toBeNull();
-    });
-  });
 });

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

@@ -176,29 +176,3 @@ export function parseSwiftTestingContinuationLine(line: string): string | null {
   const match = line.match(/^↳ (.+)$/u);
   return match ? match[1] : null;
 }
-
-/**
- * Parse xcodebuild's Swift Testing format.
- *
- * Matches:
- *   Test case 'Suite/testName()' passed on 'My Mac - App (12345)' (0.001 seconds)
- *   Test case 'Suite/testName()' failed on 'My Mac - App (12345)' (0.001 seconds)
- */
-export function parseXcodebuildSwiftTestingLine(line: string): ParsedTestCase | null {
-  const match = line.match(
-    /^Test case '(.+)' (passed|failed|skipped) on '.+' \(([^)]+) seconds?\)$/u,
-  );
-  if (!match) {
-    return null;
-  }
-  const [, rawName, status, duration] = match;
-  const { suiteName, testName } = parseRawTestName(rawName);
-
-  return {
-    status: status as 'passed' | 'failed' | 'skipped',
-    rawName,
-    suiteName,
-    testName,
-    durationText: `${duration}s`,
-  };
-}

src/utils/xcodebuild-event-parser.ts

@@ -13,7 +13,6 @@ import {
   type ParsedTestCase,
 } from './xcodebuild-line-parsers.ts';
 import {
-  parseXcodebuildSwiftTestingLine,
   parseSwiftTestingIssueLine,
   parseSwiftTestingResultLine,
   parseSwiftTestingRunSummary,
@@ -329,12 +328,6 @@ export function createXcodebuildEventParser(options: EventParserOptions): Xcodeb
       return;
     }
 
-    const xcodebuildST = parseXcodebuildSwiftTestingLine(line);
-    if (xcodebuildST) {
-      recordTestCaseResult(xcodebuildST);
-      return;
-    }
-
     const stIssue = parseSwiftTestingIssueLine(line);
     if (stIssue) {
       queueFailureDiagnostic(stIssue);