fix: snapshot array mutation, overly broad noise regex, and Swift Testing failure duration

- Deep-copy arrays in snapshot() and finalize() so returned state is
  truly immutable and won't be mutated by subsequent push() calls
- Replace overly broad noise regex that matched any 'identifier: content'
  line (swallowing compiler note: diagnostics) with a targeted pattern
  that only matches SPM resolved dependency lines (PackageName: https://...)
- Attach failure duration from Swift Testing result lines by checking
  the result line before flushing the pending issue diagnostic. Previously
  all Swift Testing failure durations were silently dropped.

Author: cameroncooke

src/utils/swift-testing-event-parser.ts

@@ -9,6 +9,7 @@ import {
   parseTestCaseLine,
   parseTotalsLine,
   parseFailureDiagnostic,
+  parseDurationMs,
 } from './xcodebuild-line-parsers.ts';
 
 export interface SwiftTestingEventParser {
@@ -84,6 +85,27 @@ export function createSwiftTestingEventParser(
       return;
     }
 
+    // Check result line BEFORE flushing so we can attach duration to pending issue
+    const stResult = parseSwiftTestingResultLine(line);
+    if (stResult && stResult.status === 'failed' && lastIssueDiagnostic) {
+      const durationMs = parseDurationMs(stResult.durationText);
+      onEvent({
+        type: 'test-failure',
+        timestamp: now(),
+        operation: 'TEST',
+        suite: lastIssueDiagnostic.suiteName,
+        test: lastIssueDiagnostic.testName,
+        message: lastIssueDiagnostic.message,
+        location: lastIssueDiagnostic.location,
+        durationMs,
+      });
+      lastIssueDiagnostic = null;
+      completedCount += 1;
+      failedCount += 1;
+      emitTestProgress();
+      return;
+    }
+
     flushPendingIssue();
 
     // Swift Testing issue line: ✘ Test "Name" recorded an issue at file:line:col: message
@@ -98,13 +120,9 @@ export function createSwiftTestingEventParser(
       return;
     }
 
-    // Swift Testing result line: ✔/✘/◇ Test "Name" passed/failed/skipped
-    const stResult = parseSwiftTestingResultLine(line);
+    // Swift Testing result line: ✔/✘/◇ Test "Name" passed/failed/skipped (non-failure or no pending issue)
     if (stResult) {
       completedCount += 1;
-      if (stResult.status === 'failed') {
-        failedCount += 1;
-      }
       if (stResult.status === 'skipped') {
         skippedCount += 1;
       }

src/utils/xcodebuild-event-parser.ts

@@ -76,7 +76,7 @@ const IGNORED_NOISE_PATTERNS = [
   /^(?:COMPILER_INDEX_STORE_ENABLE|ONLY_ACTIVE_ARCH)\s*=\s*.+$/u,
   /^Resolve Package Graph$/u,
   /^Resolved source packages:$/u,
-  /^\s*[A-Za-z0-9_.-]+:\s+.+$/u,
+  /^\s*[A-Za-z0-9_.-]+:\s+https?:\/\/.+$/u,
   /^--- xcodebuild: WARNING: Using the first of multiple matching destinations:$/u,
   /^\{\s*platform:.+\}$/u,
   /^(?:ComputePackagePrebuildTargetDependencyGraph|Prepare packages|CreateBuildRequest|SendProjectDescription|CreateBuildOperation|ComputeTargetDependencyGraph|GatherProvisioningInputs|CreateBuildDescription)$/u,

src/utils/xcodebuild-run-state.ts

@@ -225,11 +225,25 @@ export function createXcodebuildRunState(options: RunStateOptions): XcodebuildRu
         accept(tailEvent);
       }
 
-      return { ...state };
+      return {
+        ...state,
+        events: [...state.events],
+        milestones: [...state.milestones],
+        warnings: [...state.warnings],
+        errors: [...state.errors],
+        testFailures: [...state.testFailures],
+      };
     },
 
     snapshot(): Readonly<XcodebuildRunState> {
-      return { ...state };
+      return {
+        ...state,
+        events: [...state.events],
+        milestones: [...state.milestones],
+        warnings: [...state.warnings],
+        errors: [...state.errors],
+        testFailures: [...state.testFailures],
+      };
     },
 
     highestStageRank(): number {