@unrulysystems/rn-playwright-driver-runner 0.1.0 → 0.2.0

package/dist/cli.mjs

@@ -47,6 +47,10 @@ var DEFAULTS = {
   androidTokenFileName: "rn-driver-touch-token"
 };
 var SECRET_PLACEHOLDER = "<token-file>";
+var COMPANION_FAILURE_MARKERS = {
+  ios: ["** BUILD FAILED **", "** TEST FAILED **"],
+  android: ["INSTRUMENTATION_FAILED", "Process crashed"]
+};
 
 // src/plan/env.ts
 function buildIosDriverEnv(resolved, metro, timeoutMs) {
@@ -209,9 +213,10 @@ function planAndroid(input) {
       }
     }
   });
-  const launchCommand = launchCommandFor(android, serial);
-  push(launchStep("android.launch-1", android, serial));
-  push(hermesStep("android.hermes-1", android, metro, resolved, hermesDeviceName, launchCommand));
+  const launch1Command = launchCommandFor(android, resolved, serial, { forceStopBefore: true });
+  const launch2Command = launchCommandFor(android, resolved, serial, { forceStopBefore: false });
+  push(launchStep("android.launch-1", android, launch1Command));
+  push(hermesStep("android.hermes-1", android, metro, resolved, hermesDeviceName, launch1Command));
   push({
     id: "android.forward-clean",
     stage: "companion",
@@ -265,11 +270,14 @@ function planAndroid(input) {
         port: resolved.touchPort,
         tokenFile: resolved.tokenFile,
         timeoutMs: resolved.companionReadyTimeoutMs
-      }
+      },
+      // Abort early if `am instrument` reports the companion failed to start (crash / missing
+      // androidTest target) instead of waiting out the readiness budget.
+      failureMarkers: COMPANION_FAILURE_MARKERS.android
     }
   });
-  push(launchStep("android.launch-2", android, serial));
-  push(hermesStep("android.hermes-2", android, metro, resolved, hermesDeviceName, launchCommand));
+  push(launchStep("android.launch-2", android, launch2Command));
+  push(hermesStep("android.hermes-2", android, metro, resolved, hermesDeviceName, launch2Command));
   const cleanup = [
     {
       type: "kill-process",
@@ -324,22 +332,34 @@ function planAndroid(input) {
     playwright: playwrightCommand(playwright, specs, passthrough)
   };
 }
-function launchCommandFor(android, serial) {
-  return adb(serial, [
-    "shell",
-    "am",
-    "start",
-    "-W",
-    "-n",
-    `${android.packageName}/${android.activity}`
-  ]);
-}
-function launchStep(id, android, serial) {
+function launchCommandFor(android, resolved, serial, opts) {
+  if (android.launch.kind === "plain") {
+    return adb(serial, [
+      "shell",
+      "am",
+      "start",
+      "-W",
+      "-n",
+      `${android.packageName}/${android.activity}`
+    ]);
+  }
+  if (!android.scheme) {
+    throw new Error('android.scheme is required when android.launch.kind is "expo-dev-client"');
+  }
+  const launchScript = `am start -a android.intent.action.VIEW -d ${shellSingleQuote(
+    devClientUrl(android.scheme, resolved.initialUrl)
+  )}`;
+  return adbShellScript(
+    serial,
+    opts.forceStopBefore ? `am force-stop ${android.packageName} && ${launchScript}` : launchScript
+  );
+}
+function launchStep(id, android, command) {
   return {
     id,
     stage: "app-launch",
     description: `Launch ${android.packageName}/${android.activity}`,
-    action: { type: "command", command: launchCommandFor(android, serial) }
+    action: { type: "command", command }
   };
 }
 function hermesStep(id, android, metro, resolved, deviceNameMatch, launchCommand) {
@@ -373,6 +393,12 @@ function debugHostXml(host) {
 function adb(serial, args) {
   return { command: "adb", args: ["-s", serial, ...args] };
 }
+function devClientUrl(scheme, initialUrl) {
+  return `${scheme}://expo-development-client/?url=${initialUrl}`;
+}
+function shellSingleQuote(value) {
+  return `'${value.replaceAll("'", "'\\''")}'`;
+}
 function adbShellScript(serial, remote) {
   return { command: "adb", args: ["-s", serial, "shell", remote] };
 }
@@ -418,8 +444,7 @@ function planIos(input) {
       // Pass the resolved UI-test scheme so a custom `ios.uitestScheme` scaffolds
       // the SAME target that companion startup later builds (default is
       // `${appScheme}UITests`).
-      command: npx([
-        "rn-driver-xctest-scaffold",
+      command: cmd("node_modules/.bin/rn-driver-xctest-scaffold", [
         "--ios-dir",
         "ios",
         "--project-name",
@@ -579,7 +604,10 @@ function planIos(input) {
         port: resolved.touchPort,
         tokenFile: resolved.tokenFile,
         timeoutMs: resolved.companionReadyTimeoutMs
-      }
+      },
+      // Abort early if `xcodebuild test` reports a build/test failure (it lingers "alive" after, so
+      // the 300s readiness budget would otherwise be burnt waiting for a companion that cannot bind).
+      failureMarkers: COMPANION_FAILURE_MARKERS.ios
     }
   });
   if (isDevClient) {
@@ -713,7 +741,7 @@ function placeholderIos(ios, metro) {
     initialUrl: ios.launch.initialUrl ?? metro.url
   };
 }
-function placeholderAndroid(android, _metro) {
+function placeholderAndroid(android, metro) {
   return {
     serial: "<android-serial>",
     touchPort: android.companion?.port ?? DEFAULTS.companionPort,
@@ -721,7 +749,8 @@ function placeholderAndroid(android, _metro) {
     hermesTimeoutMs: DEFAULTS.hermesTargetTimeoutMs,
     tokenFile: SECRET_PLACEHOLDER,
     deviceTokenFileName: DEFAULTS.androidTokenFileName,
-    instrumentationTarget: instrumentationTarget(android)
+    instrumentationTarget: instrumentationTarget(android),
+    initialUrl: android.launch.initialUrl ?? metro.url
   };
 }
 
@@ -748,7 +777,7 @@ function buildDryRunPlan(config, platform, opts = {}) {
   return planAndroid({
     android,
     metro,
-    resolved: placeholderAndroid(android),
+    resolved: placeholderAndroid(android, metro),
     playwright: config.playwright,
     timeoutMs: config.timeoutMs,
     specs,
@@ -834,8 +863,10 @@ function renderAction(action) {
       return `write ${action.path}${action.mode ? ` (mode ${action.mode.toString(8)})` : ""}`;
     case "free-port":
       return `free-port ${action.port}`;
-    case "probe":
-      return `probe ${renderProbe(action.probe)}`;
+    case "probe": {
+      const fastFail = action.failureMarkers?.length ? ` [fast-fail on: ${action.failureMarkers.join(", ")}]` : "";
+      return `probe ${renderProbe(action.probe)}${fastFail}`;
+    }
     default: {
       const _exhaustive = action;
       throw new Error(`unhandled action: ${JSON.stringify(_exhaustive)}`);
@@ -881,6 +912,25 @@ function renderProbe(probe) {
   }
 }
 
+// src/runner/probe-failure.ts
+var ProbeFailure = class extends Error {
+  marker;
+  constructor(marker, detail) {
+    super(
+      `companion process reported a terminal failure ("${marker}") \u2014 aborting the readiness wait. The build/test failed; it will not become ready.${detail ? `
+${detail}` : ""}`
+    );
+    this.name = "ProbeFailure";
+    this.marker = marker;
+  }
+};
+function findFailureMarker(log, markers) {
+  for (const marker of markers) {
+    if (log.includes(marker)) return marker;
+  }
+  return null;
+}
+
 // src/runner/execute.ts
 var StageError = class extends Error {
   stage;
@@ -948,22 +998,30 @@ async function runStep(step, runner, opts, processes, isAlive) {
     case "probe": {
       const key = processKeyForProbe(action.probe);
       const aliveFn = key === null ? () => true : () => isAlive(key);
-      let ready = await runner.probe(action.probe, aliveFn);
-      let remaining = action.retry?.max ?? 0;
-      while (!ready && remaining > 0) {
-        remaining -= 1;
-        if (action.retry) {
-          runner.log(`retry ${step.id}: re-running launch (${remaining} attempt(s) left)`);
-          await runner.exec(action.retry.command);
+      const watch = key !== null && action.failureMarkers && action.failureMarkers.length > 0 ? { logPath: logPathFor(opts.logDir, key), failureMarkers: action.failureMarkers } : void 0;
+      try {
+        let ready = await runner.probe(action.probe, aliveFn, watch);
+        let remaining = action.retry?.max ?? 0;
+        while (!ready && remaining > 0) {
+          remaining -= 1;
+          if (action.retry) {
+            runner.log(`retry ${step.id}: re-running launch (${remaining} attempt(s) left)`);
+            await runner.exec(action.retry.command);
+          }
+          ready = await runner.probe(action.probe, aliveFn, watch);
         }
-        ready = await runner.probe(action.probe, aliveFn);
-      }
-      if (!ready) {
-        throw new StageError(
-          step.stage,
-          step.id,
-          `readiness timed out after ${action.probe.timeoutMs}ms`
-        );
+        if (!ready) {
+          throw new StageError(
+            step.stage,
+            step.id,
+            `readiness timed out after ${action.probe.timeoutMs}ms`
+          );
+        }
+      } catch (error) {
+        if (error instanceof ProbeFailure) {
+          throw new StageError(step.stage, step.id, error.message);
+        }
+        throw error;
       }
       return;
     }
@@ -1102,10 +1160,15 @@ var NodeProcessRunner = class {
     }
     if (pids.length > 0) await delay(1e3);
   }
-  probe(probe, isAlive) {
+  probe(probe, isAlive, watch) {
     const deadline = Date.now() + probe.timeoutMs;
     const attempt = async () => {
       for (; ; ) {
+        if (watch) {
+          const log = await readWatchedLog(watch.logPath);
+          const marker = findFailureMarker(log, watch.failureMarkers);
+          if (marker) throw new ProbeFailure(marker, lastLines(log));
+        }
         if (!isAlive()) return false;
         if (await probeOnce(probe)) return true;
         if (Date.now() >= deadline) return false;
@@ -1150,6 +1213,19 @@ function delay(ms) {
     setTimeout(resolve, ms);
   });
 }
+async function readWatchedLog(path4) {
+  try {
+    return await readFile(path4, "utf8");
+  } catch (error) {
+    throw new Error(
+      `cannot read companion log for fast-fail marker detection (${path4}): ${String(error)}`,
+      { cause: error }
+    );
+  }
+}
+function lastLines(text, n) {
+  return text.split("\n").filter((line) => line.trim().length > 0).slice(-12).join("\n");
+}
 async function probeOnce(probe) {
   switch (probe.kind) {
     case "metro-status":
@@ -1288,7 +1364,7 @@ async function resolveIosTarget(ios, metro, opts) {
     initialUrl: ios.launch.initialUrl ?? metro.url
   };
 }
-async function resolveAndroidTarget(android, _metro, opts) {
+async function resolveAndroidTarget(android, metro, opts) {
   const serial = await selectSerial(opts.device);
   await requireBooted(serial);
   const deviceName = (await capture("adb", ["-s", serial, "shell", "getprop", "ro.product.model"])).trim();
@@ -1301,7 +1377,8 @@ async function resolveAndroidTarget(android, _metro, opts) {
       hermesTimeoutMs: DEFAULTS.hermesTargetTimeoutMs,
       tokenFile,
       deviceTokenFileName: DEFAULTS.androidTokenFileName,
-      instrumentationTarget: instrumentationTarget(android)
+      instrumentationTarget: instrumentationTarget(android),
+      initialUrl: android.launch.initialUrl ?? metro.url
     },
     deviceName
   };
@@ -1406,6 +1483,7 @@ var IOS_KEYS = /* @__PURE__ */ new Set([
 var ANDROID_KEYS = /* @__PURE__ */ new Set([
   "packageName",
   "activity",
+  "scheme",
   "gradleTasks",
   "appApkPath",
   "testApkPath",
@@ -1507,6 +1585,8 @@ function validateAndroid(android, errors) {
   reportUnknownKeys("config.android", android, ANDROID_KEYS, errors);
   requireAndroidPackage("config.android.packageName", android.packageName, errors);
   requireAndroidActivity("config.android.activity", android.activity, errors);
+  if (android.scheme !== void 0)
+    requireAppScheme("config.android.scheme", android.scheme, errors);
   optionalString("config.android.appApkPath", android.appApkPath, errors);
   optionalString("config.android.testApkPath", android.testApkPath, errors);
   if (android.instrumentationTarget !== void 0)
@@ -1519,7 +1599,10 @@ function validateAndroid(android, errors) {
     errors.push("config.android.gradleTasks: expected an array of strings");
   }
   validateCompanion("config.android.companion", android.companion, errors);
-  validateLaunch("config.android.launch", android.launch, errors);
+  const launch = validateLaunch("config.android.launch", android.launch, errors);
+  if (launch?.kind === "expo-dev-client" && android.scheme === void 0) {
+    errors.push('config.android.scheme: required when android.launch.kind is "expo-dev-client"');
+  }
 }
 function validateCompanion(path4, companion, errors) {
   if (companion === void 0) return;
@@ -1565,6 +1648,7 @@ function requireString(path4, value, errors) {
 }
 var ANDROID_PACKAGE_RE = /^[A-Za-z][A-Za-z0-9_]*(\.[A-Za-z][A-Za-z0-9_]*)+$/;
 var ANDROID_ACTIVITY_RE = /^\.?[A-Za-z][A-Za-z0-9_]*(\.[A-Za-z][A-Za-z0-9_]*)*$/;
+var APP_SCHEME_RE = /^[a-z][a-z0-9+.-]*$/;
 function requireAndroidPackage(path4, value, errors) {
   if (typeof value !== "string" || value.trim() === "") {
     errors.push(`${path4}: required non-empty string`);
@@ -1581,6 +1665,17 @@ function requireAndroidActivity(path4, value, errors) {
   if (!ANDROID_ACTIVITY_RE.test(value))
     errors.push(`${path4}: expected an activity name (e.g. .MainActivity)`);
 }
+function requireAppScheme(path4, value, errors) {
+  if (typeof value !== "string" || value.trim() === "") {
+    errors.push(`${path4}: required non-empty string`);
+    return;
+  }
+  if (!APP_SCHEME_RE.test(value)) {
+    errors.push(
+      `${path4}: expected a valid URL scheme (lowercase letter, then lowercase letters, digits, +, ., or -)`
+    );
+  }
+}
 var ANDROID_INSTRUMENTATION_RE = /^[A-Za-z][A-Za-z0-9_]*(\.[A-Za-z][A-Za-z0-9_]*)*\/[A-Za-z][A-Za-z0-9_]*(\.[A-Za-z][A-Za-z0-9_]*)*$/;
 function requireInstrumentationTarget(path4, value, errors) {
   if (typeof value !== "string" || value.trim() === "") {