fix: remove dead noise patterns and rename durationText to displayDurationText on ParsedTotals

The Resolve Package Graph and Resolved source packages noise patterns
were unreachable dead code -- resolveStageFromLine matches them first
and returns a build-stage event.

Renamed ParsedTotals.durationText to displayDurationText to make clear
it's a display string not parseable by parseDurationMs (the XCTest
totals format is '1.234 (1.235) seconds' which doesn't parse).

Author: cameroncooke

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

@@ -200,7 +200,7 @@ describe('Swift Testing line parsers', () => {
       expect(result).toEqual({
         executed: 6,
         failed: 1,
-        durationText: '0.001s',
+        displayDurationText: '0.001s',
       });
     });
 
@@ -211,7 +211,7 @@ describe('Swift Testing line parsers', () => {
       expect(result).toEqual({
         executed: 5,
         failed: 0,
-        durationText: '0.003s',
+        displayDurationText: '0.003s',
       });
     });
 
@@ -222,7 +222,7 @@ describe('Swift Testing line parsers', () => {
       expect(result).toEqual({
         executed: 5,
         failed: 3,
-        durationText: '0.001s',
+        displayDurationText: '0.001s',
       });
     });
 

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

@@ -140,7 +140,7 @@ export function parseSwiftTestingRunSummary(line: string): ParsedTotals | null {
   }
 
   const total = Number(match[1]);
-  const durationText = `${match[2]}s`;
+  const displayDurationText = `${match[2]}s`;
 
   // Swift Testing reports "issues" not "failed tests" -- a single test can produce
   // multiple issues (e.g. multiple #expect failures). This is the best available
@@ -150,7 +150,7 @@ export function parseSwiftTestingRunSummary(line: string): ParsedTotals | null {
   const issueMatch = line.match(/with (\d+) issues?/u);
   const failed = issueMatch ? Number(issueMatch[1]) : 0;
 
-  return { executed: total, failed, durationText };
+  return { executed: total, failed, displayDurationText };
 }
 
 /**

src/utils/xcodebuild-event-parser.ts

@@ -74,8 +74,6 @@ const IGNORED_NOISE_PATTERNS = [
   /^\d{4}-\d{2}-\d{2}\s+\d{2}:\d{2}:\d{2}\.\d+\s+xcodebuild\[.+\]\s+Writing error result bundle to\s+/u,
   /^Build settings from command line:$/u,
   /^(?:COMPILER_INDEX_STORE_ENABLE|ONLY_ACTIVE_ARCH)\s*=\s*.+$/u,
-  /^Resolve Package Graph$/u,
-  /^Resolved source packages:$/u,
   /^\s*[A-Za-z0-9_.-]+:\s+https?:\/\/.+$/u,
   /^--- xcodebuild: WARNING: Using the first of multiple matching destinations:$/u,
   /^\{\s*platform:.+\}$/u,

src/utils/xcodebuild-line-parsers.ts

@@ -32,7 +32,7 @@ export interface ParsedTestCase {
 export interface ParsedTotals {
   executed: number;
   failed: number;
-  durationText?: string;
+  displayDurationText?: string;
 }
 
 export interface ParsedFailureDiagnostic {
@@ -104,7 +104,7 @@ export function parseTotalsLine(line: string): ParsedTotals | null {
   if (!match) {
     return null;
   }
-  return { executed: Number(match[1]), failed: Number(match[2]), durationText: match[3] };
+  return { executed: Number(match[1]), failed: Number(match[2]), displayDurationText: match[3] };
 }
 
 export function parseFailureDiagnostic(line: string): ParsedFailureDiagnostic | null {