Extract duplicated isResultBundlePathValue helper to shared utility

Create src/utils/result-bundle-path.ts with shared result bundle path
parsing utilities to eliminate duplication between test-common.ts and
simulator-test-execution.ts

Author: cursoragent

src/utils/result-bundle-path.ts

@@ -0,0 +1,34 @@
+/**
+ * Shared utilities for parsing and validating -resultBundlePath arguments
+ */
+
+export function isResultBundlePathValue(value: string | undefined): value is string {
+  return value !== undefined && value.length > 0 && !value.startsWith('-');
+}
+
+export function findResultBundlePathArg(extraArgs?: readonly string[]): string | undefined {
+  if (!extraArgs) {
+    return undefined;
+  }
+
+  let resultBundlePath: string | undefined;
+  for (let index = 0; index < extraArgs.length; index += 1) {
+    const argument = extraArgs[index];
+    if (argument === '-resultBundlePath') {
+      const value = extraArgs[index + 1];
+      if (isResultBundlePathValue(value)) {
+        resultBundlePath = value;
+        index += 1;
+      }
+      continue;
+    }
+    if (argument?.startsWith('-resultBundlePath=')) {
+      const value = argument.slice('-resultBundlePath='.length);
+      if (isResultBundlePathValue(value)) {
+        resultBundlePath = value;
+      }
+    }
+  }
+
+  return resultBundlePath;
+}

src/utils/simulator-test-execution.ts

@@ -1,8 +1,5 @@
 import type { TestPreflightResult } from './test-preflight.ts';
-
-function isResultBundlePathValue(value: string | undefined): value is string {
-  return value !== undefined && value.length > 0 && !value.startsWith('-');
-}
+import { isResultBundlePathValue } from './result-bundle-path.ts';
 
 function parseTestSelectorArgs(extraArgs: string[] | undefined): {
   remainingArgs: string[];

src/utils/test-common.ts

@@ -16,6 +16,7 @@ import { getDefaultCommandExecutor } from './command.ts';
 import { type TestPreflightResult } from './test-preflight.ts';
 
 import { createSimulatorTwoPhaseExecutionPlan } from './simulator-test-execution.ts';
+import { findResultBundlePathArg } from './result-bundle-path.ts';
 
 import type {
   BuildTarget,
@@ -59,37 +60,6 @@ function getFallbackErrorMessages(
   return [...streamedLines, ...(responseContent ?? []).map((item) => item.text)];
 }
 
-function isResultBundlePathValue(value: string | undefined): value is string {
-  return value !== undefined && value.length > 0 && !value.startsWith('-');
-}
-
-function findResultBundlePathArg(extraArgs?: readonly string[]): string | undefined {
-  if (!extraArgs) {
-    return undefined;
-  }
-
-  let resultBundlePath: string | undefined;
-  for (let index = 0; index < extraArgs.length; index += 1) {
-    const argument = extraArgs[index];
-    if (argument === '-resultBundlePath') {
-      const value = extraArgs[index + 1];
-      if (isResultBundlePathValue(value)) {
-        resultBundlePath = value;
-        index += 1;
-      }
-      continue;
-    }
-    if (argument?.startsWith('-resultBundlePath=')) {
-      const value = argument.slice('-resultBundlePath='.length);
-      if (isResultBundlePathValue(value)) {
-        resultBundlePath = value;
-      }
-    }
-  }
-
-  return resultBundlePath;
-}
-
 function createXcodebuildTestArtifacts(
   params: Pick<SharedTestExecutorParams, 'deviceId'>,
   started: ReturnType<typeof createDomainStreamingPipeline>,