@deeeed/metamask-harness 0.17.1 → 0.17.3

package/CHANGELOG.md

@@ -2,6 +2,37 @@
 
 ## Unreleased
 
+## 0.17.3 - 2026-07-15
+
+### Added
+
+- Added a real-checkout human-outcome gate covering live readiness, runtime verification, debugger target resolution, smoke evidence, and unchanged tracked product source across Extension, Mobile, and Core.
+- Added generic Core runner smoke and fixture-backed wallet smoke recipes so the reusable examples are not tied to Perps.
+- Added nine-slot isolation coverage for platform ports, runtime files, logs, and headless Core behavior.
+
+### Changed
+
+- `doctor` now reports CDP or simulator/ADB screenshot fallback when `capture-helper` is absent or unhealthy, while identifying video as unavailable separately.
+- Action, flow, and recipe discovery/documentation now use one concise scope model; the default recipe-library and adapter-surface references were reduced to current behavior.
+- Fixture discovery now uses one canonical checkout path instead of legacy fallback locations.
+
+### Fixed
+
+- Mobile launch now waits for two consecutive platform-matched agentic bridge snapshots, preventing a transient startup route from being reported ready before an immediate verify.
+- `verify --json` keeps stdout to one machine-readable JSON document even when the platform harness must first be installed.
+- Mobile install metadata safely preserves checkout paths containing shell-significant characters.
+- Cleanup discovery no longer advertises a removed compatibility flag, and Core recovery guidance now points to the canonical wallet fixture.
+- Mobile console forwarding now uses canonical checkout paths, readiness rejects impossible one-poll stability checks, and Extension debug handles malformed WebSocket frames without crashing.
+- Extension watch refreshes remove stale runtime files, delay file replacement until transfer completion, allow slower console attachment, and stop the detached webpack process group.
+
+## 0.17.2 - 2026-07-15
+
+### Fixed
+
+- Passive update discovery now runs in a detached, once-daily registry probe, so fast commands populate the cache without adding latency or contaminating machine output.
+
+## 0.17.1 - 2026-07-15
+
 ### Fixed
 
 - Extension `debug` now uses Chrome's native DevTools command for the active slot target and opens directly on Console; `debug --worker` targets the service worker without creating a frontend browser tab.
@@ -9,6 +40,11 @@
 - Successful Extension watch rebuilds now update the isolated loaded runtime in place, so a normal refresh of the current fullscreen or sidepanel UI shows source edits without another harness build or relaunch.
 - Extension slot titles are stamped into the isolated runtime after snapshots and successful rebuilds, preserving the slot identifier across an operator's normal browser refresh without changing product source.
 - Extension sidepanel opening no longer waits for the harness-owned user-gesture button to pass visual actionability checks; occluded wallet loading surfaces cannot block the trusted click.
+- Human recipe results now print an absolute-path artifact inventory and expand run-scoped application diagnostics inline; single-action calls show the same `CLEAN`/`REVIEW` diagnostics instead of requiring another query.
+- Extension console collection now starts after every successful public launch path, including quick reattach, and persists as one checkout-scoped process until `stop`. MetaMask UI/service-worker events and dapp-page events are routed to separate `extension-console.log` and `dapp-console.log` files; bare `logs` defaults to MetaMask while `--source dapp` and `--source webpack` remain explicit.
+- Mobile CDP-forwarded app events now write to `app-console.log`, so `logs --source app` and run diagnostics are no longer mixed with Metro bundler output; warm and cold launches start a process-group-detached collector only after bridge readiness so capture remains live after the launch command exits.
+- Extension `debug` now opens Chrome's local DevTools frontend inside the exact slot-owned profile, targets the MetaMask extension page rather than the dapp, and verifies the debugging connection before reporting success.
+- Mobile fixture application now fails immediately when the checkout fixture is absent, rejects the misplaced `fixtures set --dev` flag with the correct initialization command, and never prints a success message after failure.
 
 ### Changed
 
@@ -20,16 +56,6 @@
 - Removed compatibility-only runtime-start fields, hidden `live` execution, removed-command handlers, undomained recipe aliases, flat/non-ESM action lookup, old port-resolver filenames, duplicate Extension launcher naming, obsolete fixture search locations, and the alternate Mobile product-source bridge. Mobile install now requires the current product-owned bridge and writes metadata only.
 - Removed the historical CLI migration/spec documents; current behavior is documented in the README, cheatsheet, architecture, and human QA checklist.
 
-## 0.17.1 - 2026-07-14
-
-### Fixed
-
-- Human recipe results now print an absolute-path artifact inventory and expand run-scoped application diagnostics inline; single-action calls show the same `CLEAN`/`REVIEW` diagnostics instead of requiring another query.
-- Extension console collection now starts after every successful public launch path, including quick reattach, and persists as one checkout-scoped process until `stop`. MetaMask UI/service-worker events and dapp-page events are routed to separate `extension-console.log` and `dapp-console.log` files; bare `logs` defaults to MetaMask while `--source dapp` and `--source webpack` remain explicit.
-- Mobile CDP-forwarded app events now write to `app-console.log`, so `logs --source app` and run diagnostics are no longer mixed with Metro bundler output; warm and cold launches start a process-group-detached collector only after bridge readiness so capture remains live after the launch command exits.
-- Extension `debug` now opens Chrome's local DevTools frontend inside the exact slot-owned profile, targets the MetaMask extension page rather than the dapp, and verifies the debugging connection before reporting success.
-- Mobile fixture application now fails immediately when the checkout fixture is absent, rejects the misplaced `fixtures set --dev` flag with the correct initialization command, and never prints a success message after failure.
-
 ## 0.17.0 - 2026-07-14
 
 ### Added

package/README.md

@@ -12,6 +12,12 @@ It has two jobs:
 The generic graph engine lives in `@farmslot/recipe-harness`. This package owns
 only MetaMask-specific actions and runtime control.
 
+One mental model:
+
+- **Action** — one typed operation, such as reading wallet state.
+- **Flow** — a reusable, parameterized sequence of actions.
+- **Recipe** — the complete executable proof for a task or regression.
+
 ## Install
 
 ```bash
@@ -70,11 +76,11 @@ mm-harness flows
 
 mm-harness run path/to/recipe.json --plan
 mm-harness run path/to/recipe.json --artifacts-dir temp/recipe-proof
+mm-harness run wallet.smoke --artifacts-dir temp/wallet-proof
 ```
 
-`actions` is the executable vocabulary. `call` exercises one action. `flows`
-lists reusable parameterized behavior from personal, team, and bundled recipe
-libraries. A recipe connects those capabilities into a deterministic proof.
+`call` exercises one action. A recipe composes actions and flows into a
+deterministic proof.
 
 A run prints its findings, diagnostics, and absolute evidence paths. The artifact
 directory contains the executed recipe, summary, trace, diagnostics when
@@ -126,6 +132,6 @@ Validate changes with:
 yarn check
 ```
 
-Current references: [cheatsheet](docs/CHEATSHEET.md), [architecture](docs/architecture.md),
-[recipe libraries](docs/recipe-libraries.md), [human QA](docs/CLI-ERGONOMICS-HUMAN-QA.md),
-and the checked [video demo validation](docs/VIDEO-DEMO-VALIDATION.md).
+Current references: [cheatsheet](docs/CHEATSHEET.md), [recipe libraries](docs/recipe-libraries.md),
+and [human QA](docs/CLI-ERGONOMICS-HUMAN-QA.md). Maintainers also have the
+[architecture](docs/architecture.md) and [video gate](docs/VIDEO-DEMO-VALIDATION.md).

package/adapters/extension/start-watch.sh

@@ -182,6 +182,7 @@ rm -rf node_modules/.cache/webpack
 killed_harness_pid=""
 if [ -f "$RUNTIME_DIR/recipe-harness-webpack.pid" ]; then
   killed_harness_pid="$(cat "$RUNTIME_DIR/recipe-harness-webpack.pid" 2>/dev/null || true)"
+  [ -z "$killed_harness_pid" ] || kill -- "-$killed_harness_pid" >/dev/null 2>&1 || true
   [ -z "$killed_harness_pid" ] || kill "$killed_harness_pid" >/dev/null 2>&1 || true
   rm -f "$RUNTIME_DIR/recipe-harness-webpack.pid"
 fi

package/adapters/extension/sync-webpack-dist.cjs

@@ -51,7 +51,7 @@ function removeOwnPidFile() {
 
 function syncRuntimeDist() {
   if (!fs.existsSync(path.join(dist, 'manifest.json')) || !fs.existsSync(runtimeDist)) return;
-  const result = spawnSync('rsync', ['-a', '--exclude', '_metadata', `${dist}${path.sep}`, `${runtimeDist}${path.sep}`], {
+  const result = spawnSync('rsync', ['-a', '--delete', '--delay-updates', '--exclude', '_metadata', `${dist}${path.sep}`, `${runtimeDist}${path.sep}`], {
     encoding: 'utf8',
   });
   if (result.status !== 0) {

package/adapters/manifest.json

@@ -87,7 +87,7 @@
       "entry": "adapters/mobile/cleanup.sh",
       "kind": "bash",
       "purpose": "Remove Mobile harness metadata and recorded git excludes without touching product source.",
-      "inputs": "--target --allow-managed-changes; env RECIPE_HARNESS_ROOT",
+      "inputs": "--target; env RECIPE_HARNESS_ROOT",
       "outputs": "removes <harness>/; exit 0/1/2"
     },
     {
@@ -486,6 +486,14 @@
       "purpose": "Open human-facing debugger UIs for extension CDP or mobile Metro runtimes.",
       "inputs": "--adapter extension|mobile [--cdp-port|--port] [--target page|worker] [--json] [--no-open]",
       "outputs": "debug URL/report on stdout; exit 0/1"
+    },
+    {
+      "id": "lib/update-check-worker",
+      "entry": "adapters/shared/update-check-worker.mjs",
+      "kind": "node",
+      "purpose": "Detached, silent registry probe used by the daily passive update notice.",
+      "inputs": "<cache-file> <timeout-ms> [registry-endpoint]",
+      "outputs": "atomic update-cache refresh; always silent and best-effort"
     }
   ]
 }

package/adapters/mobile/bridge-runtime/lib/match-bridge-target.cjs

@@ -50,7 +50,9 @@ function matchesBridgeTarget(target, env) {
 // (an in-app agentic bridge is live), not merely a registered debug target.
 function hasMatchingRoute(value, env) {
   const targets = Array.isArray(value) ? value : [value];
-  return targets.some((t) => matchesBridgeTarget(t, env) && t && t.route);
+  return targets.some(
+    (t) => matchesBridgeTarget(t, env) && t && t.agenticPresent === true && Boolean(t.route),
+  );
 }
 
 // Human summary of the targets that answered (platform / deviceName), for teaching on

package/adapters/mobile/inject.sh

@@ -98,23 +98,25 @@ fi
 RUNNER_REV="$(git -C "$RUNNER_DIR" rev-parse HEAD 2>/dev/null || echo unknown)"
 CLEANUP_Q="$(printf '%q' "$SCRIPT_DIR/cleanup.sh")"
 TARGET_CLEANUP_Q="$(printf '%q' "$TARGET")"
-node - "$HARNESS_DIR/manifest.json" <<NODE
+CLEANUP_COMMAND="RECIPE_HARNESS_ROOT=$(printf '%q' "$HARNESS_ROOT") $CLEANUP_Q --target $TARGET_CLEANUP_Q"
+node - "$HARNESS_DIR/manifest.json" "$TARGET_REV" "$RUNNER_DIR" "$RUNNER_REV" "$TARGET" "$HARNESS_REL" "$HARNESS_ROOT" "$CLEANUP_COMMAND" <<'NODE'
 const fs = require('node:fs');
-fs.writeFileSync(process.argv[2], JSON.stringify({
+const [manifestPath, targetRevision, runnerDir, runnerRevision, target, harnessRel, harnessRoot, cleanupCommand] = process.argv.slice(2);
+fs.writeFileSync(manifestPath, JSON.stringify({
   adapter: 'mobile',
   installMode: 'product-owned',
-  targetRevision: '$TARGET_REV',
+  targetRevision,
   installedAt: new Date().toISOString(),
-  source: { runnerDir: '$RUNNER_DIR', runnerRevision: '$RUNNER_REV' },
-  target: '$TARGET',
+  source: { runnerDir, runnerRevision },
+  target,
   protocolVersion: 'v1',
-  actionManifestPath: '$HARNESS_REL/action-manifest.json',
-  runnerEntrypoint: '$HARNESS_REL/runner/bin/mm-harness',
-  runtimeHelpers: { verify: '$HARNESS_REL/scripts/verify.sh' },
-  harnessInstalledPaths: ['$HARNESS_REL/runner', '$HARNESS_REL/scripts', '$HARNESS_REL/action-manifest.json'],
+  actionManifestPath: `${harnessRel}/action-manifest.json`,
+  runnerEntrypoint: `${harnessRel}/runner/bin/mm-harness`,
+  runtimeHelpers: { verify: `${harnessRel}/scripts/verify.sh` },
+  harnessInstalledPaths: [`${harnessRel}/runner`, `${harnessRel}/scripts`, `${harnessRel}/action-manifest.json`],
   productOwnedPaths: ['app/dev-tools/AgenticService', 'app/core/NavigationService/NavigationService.ts'],
-  cleanupCommand: 'RECIPE_HARNESS_ROOT=$HARNESS_ROOT $CLEANUP_Q --target $TARGET_CLEANUP_Q',
-  productDiffExcludes: [':(exclude)$HARNESS_ROOT'],
+  cleanupCommand,
+  productDiffExcludes: [`:(exclude)${harnessRoot}`],
 }, null, 2) + '\n');
 NODE
 

package/adapters/mobile/start-console-forwarder.sh

@@ -8,8 +8,8 @@ QUIET=false
 
 while [ "$#" -gt 0 ]; do
   case "$1" in
-    --target) TARGET="$2"; shift 2 ;;
-    --port)   PORT="$2"; shift 2 ;;
+    --target) [ "$#" -ge 2 ] || { echo "Missing value for --target" >&2; exit 2; }; TARGET="$2"; shift 2 ;;
+    --port)   [ "$#" -ge 2 ] || { echo "Missing value for --port" >&2; exit 2; }; PORT="$2"; shift 2 ;;
     --quiet)  QUIET=true; shift ;;
     -h|--help)
       printf 'Usage: start-console-forwarder.sh [--target <dir>] [--port <port>] [--quiet]\n'
@@ -20,7 +20,7 @@ while [ "$#" -gt 0 ]; do
 done
 
 [ "${METAMASK_RECIPE_CONSOLE_FORWARD:-1}" != "0" ] || exit 0
-TARGET="$(cd "$TARGET" && pwd)"
+TARGET="$(cd "$TARGET" && pwd -P)"
 SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
 # shellcheck disable=SC1091
 . "$SCRIPT_DIR/../shared/harness-path.sh"

package/adapters/mobile/wait-for-bridge.sh

@@ -24,10 +24,10 @@ REQUIRE_PLATFORM=""
 
 while [ "$#" -gt 0 ]; do
   case "$1" in
-    --target)     TARGET="$2"; shift 2 ;;
-    --port)       PORT="$2"; shift 2 ;;
-    --max-polls)  MAX_POLLS="$2"; shift 2 ;;
-    --platform)   REQUIRE_PLATFORM="$2"; shift 2 ;;
+    --target)     [ "$#" -ge 2 ] || { echo "Missing value for --target" >&2; exit 2; }; TARGET="$2"; shift 2 ;;
+    --port)       [ "$#" -ge 2 ] || { echo "Missing value for --port" >&2; exit 2; }; PORT="$2"; shift 2 ;;
+    --max-polls)  [ "$#" -ge 2 ] || { echo "Missing value for --max-polls" >&2; exit 2; }; MAX_POLLS="$2"; shift 2 ;;
+    --platform)   [ "$#" -ge 2 ] || { echo "Missing value for --platform" >&2; exit 2; }; REQUIRE_PLATFORM="$2"; shift 2 ;;
     -h|--help)
       printf 'Usage: wait-for-bridge.sh [--target <dir>] [--port <n>] [--max-polls <n>] [--platform ios|android]\n'
       exit 0
@@ -36,7 +36,12 @@ while [ "$#" -gt 0 ]; do
   esac
 done
 
-TARGET="$(cd "$TARGET" && pwd)"
+case "$MAX_POLLS" in
+  ''|*[!0-9]*) echo "wait-for-bridge: --max-polls must be an integer of at least 2." >&2; exit 2 ;;
+esac
+[ "$MAX_POLLS" -ge 2 ] || { echo "wait-for-bridge: --max-polls must be at least 2 for stable readiness." >&2; exit 2; }
+
+TARGET="$(cd "$TARGET" && pwd -P)"
 
 # The matcher reads the required platform from the environment so the value survives
 # into the node subprocess that runs the shared matcher against the status output.
@@ -128,15 +133,24 @@ try {
 ' "$MATCH_LIB" "$STATUS_LOG" 2>/dev/null || printf 'none'
 }
 
+READY_STREAK=0
 for ATTEMPT in $(seq 1 "$MAX_POLLS"); do
   if bridge_has_route; then
-    printf 'Mobile bridge ready\n' >&2
-    exit 0
+    READY_STREAK=$((READY_STREAK + 1))
+    # The bridge can appear briefly while React Native is still completing its
+    # startup navigation/reload. Require two consecutive healthy snapshots so a
+    # following command never inherits that transient gap.
+    if [ "$READY_STREAK" -ge 2 ]; then
+      printf 'Mobile bridge ready\n' >&2
+      exit 0
+    fi
+  else
+    READY_STREAK=0
   fi
   if [ "$ATTEMPT" = "1" ] || [ $((ATTEMPT % 5)) -eq 0 ]; then
     printf 'Waiting for Mobile bridge (%s/%s): %s\n' "$ATTEMPT" "$MAX_POLLS" "$(bridge_wait_reason)" >&2
   fi
-  sleep 2
+  [ "$ATTEMPT" -ge "$MAX_POLLS" ] || sleep 2
 done
 
 cat "$STATUS_LOG" >&2 || true

package/adapters/shared/open-debug.mjs

@@ -123,7 +123,13 @@ function sendBrowserCommand(webSocketDebuggerUrl, method, params) {
       }));
     });
     socket.on('message', (raw) => {
-      const message = JSON.parse(String(raw));
+      let message;
+      try {
+        message = JSON.parse(String(raw));
+      } catch {
+        finish({ error: { message: `${method} returned a non-JSON WebSocket frame` } });
+        return;
+      }
       if (message.id !== 1) return;
       finish(message);
     });

package/adapters/shared/update-check-worker.mjs

@@ -0,0 +1,35 @@
+#!/usr/bin/env node
+import fs from 'node:fs';
+import path from 'node:path';
+
+const [cacheFile, timeoutText, endpoint] = process.argv.slice(2);
+const timeoutMs = Number.parseInt(timeoutText ?? '', 10);
+
+async function main() {
+  if (!cacheFile || !endpoint || !Number.isInteger(timeoutMs) || timeoutMs < 1) return;
+  const controller = new AbortController();
+  const timer = setTimeout(() => controller.abort(), timeoutMs);
+  try {
+    const response = await fetch(endpoint, { signal: controller.signal });
+    if (!response.ok) return;
+    const data = await response.json();
+    if (!data || typeof data.latest !== 'string' || data.latest.length === 0) return;
+    let lastCheck = Date.now();
+    try {
+      const cached = JSON.parse(fs.readFileSync(cacheFile, 'utf8'));
+      if (typeof cached.lastCheck === 'number') lastCheck = cached.lastCheck;
+    } catch {
+      // The parent may have exited before creating the cache; recreate it below.
+    }
+    fs.mkdirSync(path.dirname(cacheFile), { recursive: true });
+    const temporary = `${cacheFile}.${process.pid}.tmp`;
+    fs.writeFileSync(temporary, JSON.stringify({ lastCheck, latest: data.latest }));
+    fs.renameSync(temporary, cacheFile);
+  } catch {
+    // Passive checks are intentionally silent; the explicit update command teaches failures.
+  } finally {
+    clearTimeout(timer);
+  }
+}
+
+await main();

package/dist/adapters/extension/console-capture.js

@@ -47,12 +47,21 @@ async function ensureExtensionConsoleCapture(target) {
   child.unref();
   fs.writeFileSync(pidFile, `${child.pid}
 `);
-  for (let attempt = 0; attempt < 30; attempt += 1) {
+  for (let attempt = 0; attempt < 100; attempt += 1) {
+    if (!processAlive(child.pid)) break;
     if (collectorOwnsLog(child.pid, extensionLog) && containsAttachment(extensionLog, dappLog)) return;
     await new Promise((resolve) => setTimeout(resolve, 100));
   }
   throw new Error(`Extension console collector ${child.pid} did not attach to CDP ${cdpPort}.`);
 }
+function processAlive(pid) {
+  try {
+    process.kill(pid, 0);
+    return true;
+  } catch {
+    return false;
+  }
+}
 function readPid(pidFile) {
   try {
     const pid = Number.parseInt(fs.readFileSync(pidFile, "utf8").trim(), 10);

package/dist/adapters/mobile/prepare.js

@@ -56,7 +56,7 @@ async function prepareMobile(target, opts = {}) {
       json,
       preflightMode
     );
-    if (confirm.status === 0) return startMobileConsoleForwarder(target, json);
+    if (confirm.status === 0) return startMobileConsoleForwarder(target, platform, json);
     if (!json) {
       process.stderr.write(
         `launch: runtime looked ready but no ${platform} bridge target answered; launching the app.
@@ -70,7 +70,7 @@ async function prepareMobile(target, opts = {}) {
       json,
       preflightMode
     );
-    return launch.status === 0 ? startMobileConsoleForwarder(target, json) : launch;
+    return launch.status === 0 ? startMobileConsoleForwarder(target, platform, json) : launch;
   }
   if (report.decision === "unknown") {
     const msg = "mobile prepare: runtime state unknown\n  Next: run mm-harness verify --adapter mobile --target <checkout>";
@@ -118,13 +118,13 @@ async function prepareMobile(target, opts = {}) {
       }
     }
   }
-  return startMobileConsoleForwarder(target, json);
+  return startMobileConsoleForwarder(target, platform, json);
 }
-function startMobileConsoleForwarder(target, json) {
+function startMobileConsoleForwarder(target, platform, json) {
   return dispatchAction(
     { id: "start-console-forwarder", cwd: target, argv: json ? ["--quiet"] : [] },
     target,
-    "ios",
+    platform,
     json
   );
 }

package/dist/adapters/slot-ports.js

@@ -113,6 +113,10 @@ function stopExtensionWatcher(target) {
   } catch {
   }
   for (const pid of ownedPids) {
+    try {
+      process.kill(-pid, "SIGTERM");
+    } catch {
+    }
     try {
       process.kill(pid, "SIGTERM");
     } catch {
@@ -123,6 +127,10 @@ function stopExtensionWatcher(target) {
   if (ownedPids.length > 0) {
     Atomics.wait(new Int32Array(new SharedArrayBuffer(4)), 0, 0, 2e3);
     for (const pid of ownedPids) {
+      try {
+        process.kill(-pid, "SIGKILL");
+      } catch {
+      }
       try {
         process.kill(pid, "SIGKILL");
       } catch {

package/dist/cli.js

@@ -104,7 +104,7 @@ async function main(argv) {
         const target = targetPath(options);
         const adapterRaw = (typeof options["adapter"] === "string" ? options["adapter"] : void 0) ?? (typeof options["platform"] === "string" ? options["platform"] : void 0) ?? detectAdapter(target);
         if (adapterRaw === "mobile" || adapterRaw === "extension" || adapterRaw === "core") {
-          await handleHarness(["install", "--platform", adapterRaw, "--target", target]);
+          await handleHarness(["install", "--platform", adapterRaw, "--target", target, "--quiet"]);
         }
       } catch {
       }

package/dist/commands/doctor.js

@@ -140,7 +140,7 @@ async function handleDoctor({ options }) {
     requiredChecks
   };
   const orphanMetros = adapter === "mobile" ? detectOrphanMetros(target, process.env.WATCHER_PORT) : [];
-  const capture = captureHelperHealth();
+  const capture = captureHealth(adapter);
   const deviceView = adapter === "mobile" ? mobileDeviceView(allDevices) : null;
   const liveView = deviceView ? await mobileDeviceLiveView(target, deviceView) : null;
   const devices = liveView?.devicesWithLive ?? deviceView?.devices ?? [];
@@ -192,13 +192,7 @@ async function handleDoctor({ options }) {
       );
       console.log(`  ${out("dim", "Next: mm-harness stop   # reaps every bundler this checkout leaked")}`);
     }
-    if (capture) {
-      console.log(`${out("label", "capture:")} ${out(capture.status === "pass" ? "ok" : "warn", capture.status)} ${out("dim", "(capture-helper: screenshots + --record video)")}`);
-      if (capture.status !== "pass") {
-        if (capture.failing.length > 0) console.log(`  ${out("dim", `failing: ${capture.failing.join(", ")}`)}`);
-        console.log(`  ${out("dim", "Next: grant Screen Recording (System Settings \u2192 Privacy & Security \u2192 Screen Recording), or run: capture-helper doctor --open-permissions")}`);
-      }
-    }
+    if (capture) renderCaptureHealth(capture, out);
     if (adapter === "mobile") {
       if (deviceView) renderMobileDeviceList(deviceView, out);
       if (additionalReachableDevices.length > 0) renderAdditionalReachableDevices(additionalReachableDevices, out);
@@ -280,6 +274,25 @@ function emitExpectLive(adapter, target, runtime, result, orphanMetros, capture,
   emitExpectLiveVerdict(adapter, target, runtime, "not-live");
   return EXIT.runtime;
 }
+function captureHealth(adapter) {
+  if (adapter === "core") return null;
+  const fallback = adapter === "extension" ? ["cdp"] : ["simctl", "adb"];
+  const helper = captureHelperHealth();
+  if (!helper) {
+    return {
+      status: "fallback",
+      failing: [],
+      screenshots: { available: true, providers: fallback },
+      recording: { available: false, provider: null }
+    };
+  }
+  return {
+    status: helper.status,
+    failing: helper.failing,
+    screenshots: { available: true, providers: ["capture-helper", ...fallback] },
+    recording: { available: helper.status === "pass", provider: helper.status === "pass" ? "capture-helper" : null }
+  };
+}
 function captureHelperHealth() {
   if (process.platform !== "darwin") return null;
   const bin = process.env.CAPTURE_HELPER_PATH || "capture-helper";
@@ -292,6 +305,18 @@ function captureHelperHealth() {
     return null;
   }
 }
+function renderCaptureHealth(capture, out) {
+  const video = capture.recording.available ? "capture-helper" : "unavailable";
+  console.log(
+    `${out("label", "capture:")} ${out(capture.status === "pass" ? "ok" : "warn", capture.status)} ${out("dim", `(screenshots: ${capture.screenshots.providers.join(" \u2192 ")}; video: ${video})`)}`
+  );
+  if (capture.status === "warn") {
+    if (capture.failing.length > 0) console.log(`  ${out("dim", `failing: ${capture.failing.join(", ")}`)}`);
+    console.log(`  ${out("dim", "Next: grant Screen Recording (System Settings \u2192 Privacy & Security \u2192 Screen Recording), or run: capture-helper doctor --open-permissions")}`);
+  } else if (capture.status === "fallback") {
+    console.log(`  ${out("dim", "Next: screenshots work without capture-helper; install/configure capture-helper only when recipe video is required")}`);
+  }
+}
 function detectOrphanMetros(target, livePort) {
   try {
     const script = path.join(runnerDir, "adapters/shared/reap-checkout-metros.sh");

package/dist/commands/run-engine.js

@@ -22,8 +22,7 @@ import { listRecipeFiles } from "../recipe-files.js";
 import { startRecipeRecording, stopRecipeRecording } from "../run-recording.js";
 import {
   beginRunDiagnostics,
-  finishRunDiagnostics,
-  stopRunDiagnostics
+  finishRunDiagnostics
 } from "../run-diagnostics.js";
 import { EXIT } from "./shared.js";
 import {
@@ -90,7 +89,6 @@ async function runRecipe(adapter, recipe, artifactsDir, projectRoot, actionManif
       await stopRecipeRecording(recording, result);
       return finishRunDiagnostics(diagnosticBaseline, result);
     } finally {
-      stopRunDiagnostics(diagnosticBaseline);
       await stopRecipeRecording(recording);
     }
   } finally {
@@ -645,7 +643,7 @@ async function executeWithHealBounds(exec, adapter, target, heal, state, recover
   }
 }
 function emitHealViolation(json, command, result, violation, state, adapter) {
-  const userAction = adapter === "core" && violation.code === "WALLET_STATE_REQUIRED" ? 'set MM_TEST_ACCOUNT_ADDRESS=<0x\u2026> in env, or add "account": "<0x\u2026>" to the node block in the recipe' : violation.userAction ?? `inspect ${shellQuoteArg(result.summaryPath)} and ${shellQuoteArg(result.tracePath)}; fix the application or recipe failure before retrying`;
+  const userAction = adapter === "core" && violation.code === "WALLET_STATE_REQUIRED" ? 'create temp/recipe/runtime/wallet-fixture.json, or add "account": "<0x\u2026>" to the node block in the recipe' : violation.userAction ?? `inspect ${shellQuoteArg(result.summaryPath)} and ${shellQuoteArg(result.tracePath)}; fix the application or recipe failure before retrying`;
   if (json) {
     console.log(
       JSON.stringify(