@deeeed/metamask-harness 0.6.0 → 0.6.2

package/CHANGELOG.md

@@ -1,5 +1,25 @@
 # Changelog
 
+## 0.6.2 - 2026-07-06
+
+### Added
+- **`-v` / `--version`** prints the mm-harness version.
+
+### Fixed
+- **`logs` now streams live and shows the file path** — it captured output via spawnSync (never shown for a `tail -F` that doesn't exit), so `logs`/`logs --full` looked dead. New spawnInherit hands the terminal to the follow; the resolved log path + `tail -f`/`less +F` hints print upfront so you can page it yourself. Honors the test override seam.
+- **`debug` resolves the checkout's own port** (slot context → pool → formula) like launch/stop/doctor — bare `mm-harness debug` no longer 'fetch failed' against the wrong port.
+
+## 0.6.1 - 2026-07-06
+
+Fresh-install fixes found live on a published-0.6.0 install.
+
+### Fixed
+- **Mobile `launch` no longer dies on empty `METAMASK_BUILD_TYPE`** — the quick-launch runs `expo start` directly (bypassing scripts/build.sh, which passes the build type as an arg); the fixture `.js.env` shipped `METAMASK_BUILD_TYPE=""`, which Metro's transform rejects (`Invalid METAMASK_BUILD_TYPE`). start-metro now defaults to the main dev client when unset, so already-installed slots launch without re-syncing fixtures.
+- **`doctor` no longer false-flags orphan Metros** — the detector matched any argv containing "metro" (caught the `tail -F metro.log` viewer) and reported the live, valid bundler as leaked. Now it matches only `expo start` bundlers and excludes the one serving the current port.
+
+### Internal
+- Reap-checkout-metros test hardened against a SIGTERM timing flake (polls for exit).
+
 ## 0.6.0 - 2026-07-06
 
 Minor release: CLI hardening (decomposed, single-parser, dist-only) plus a
@@ -31,6 +51,7 @@ independently cross-reviewed.
   `ensure_unlocked` fails teaching `fixtures set` on an un-onboarded wallet instead
   of vacuously passing (mobile + extension); `provision --json` usage errors emit the
   standard envelope; deps-not-ready teaches a command that works on a bare checkout.
+
 ## 0.5.1 - 2026-07-05
 
 Fresh-install hotfixes found in live validation of 0.5.0.

package/adapters/mobile/start-metro.sh

@@ -117,6 +117,12 @@ printf '(Full log: %s — mm-harness logs | mm-harness logs --full)\n' "$LOG_FIL
   # shellcheck disable=SC1091
   [ ! -f .env.local ] || . ./.env.local
   set +a
+  # The quick-launch runs `expo start` directly, bypassing scripts/build.sh which
+  # passes the build type as an argument (start:ios = build.sh ios main dev). The
+  # fixture .js.env ships METAMASK_BUILD_TYPE="" (empty), which Metro's transform
+  # rejects. Default to the main dev client (what runway installs / launch targets)
+  # so already-installed slots work without re-syncing fixtures.
+  [ -n "${METAMASK_BUILD_TYPE:-}" ] || export METAMASK_BUILD_TYPE=main
   export EXPO_NO_TYPESCRIPT_SETUP=1
   export WATCHER_PORT="${PORT}" METRO_PORT="${PORT}"
   export METRO_MAX_WORKERS="${METRO_WORKERS}"

package/adapters/shared/reap-checkout-metros.sh

@@ -1,32 +1,38 @@
 #!/usr/bin/env bash
-# reap-checkout-metros.sh — kill EVERY Metro/Expo bundler bound to a given
-# checkout, regardless of port. Port-scoped stop leaks a bundler whenever a
-# launch drifts ports or fails to record its pid; this sweeps them by checkout
-# path so `stop` fully cleans up and `launch` starts from zero.
+# reap-checkout-metros.sh — find/kill Metro (Expo) bundlers bound to a checkout.
+# The harness starts Metro exactly one way: `yarn expo start`. Match ONLY that
+# signature — matching bare "metro" also caught the `tail -F …/metro.log` viewer
+# and other noise, producing false orphans.
 #
-#   reap_checkout_metros <checkout-abs-path> [keep-pid]
-# keep-pid (optional): a pid to spare (the bundler just started/handled).
+#   reap_checkout_metros  <checkout-abs-path> [keep-pid]
+#   detect_checkout_metros <checkout-abs-path> [live-port]
+# reap kills every bundler for the checkout (stop wants a clean slate), sparing
+# keep-pid. detect lists orphans for doctor and EXCLUDES the bundler serving
+# live-port (the currently-running valid Metro is not a leak).
+
+_checkout_metro_pids() {
+  # pids of `expo start` bundlers whose argv references this checkout (boundary
+  # anchored so /repos/mm-1 never matches /repos/mm-10).
+  local repo="$1" pid args
+  while IFS= read -r pid; do
+    [ -n "$pid" ] || continue
+    args="$(ps -o args= -p "$pid" 2>/dev/null || true)"
+    case "$args" in *"$repo/"*|*"$repo "*) : ;; *) continue ;; esac
+    case "$args" in *"expo start"*) printf '%s\n' "$pid" ;; esac
+  done <<LIST
+$(pgrep -f "expo start" 2>/dev/null)
+LIST
+}
+
 reap_checkout_metros() {
-  local repo="$1" keep="${2:-}" reaped=""
+  local repo="$1" keep="${2:-}" reaped="" pid
   [ -n "$repo" ] || return 0
-  # Match the bundler's argv: `expo start` / metro under this checkout. ps argv
-  # embeds the cwd-launched command; expo/metro run with the repo on the path.
-  local pid args
   while IFS= read -r pid; do
     [ -n "$pid" ] || continue
     [ "$pid" = "$keep" ] && continue
-    args="$(ps -o args= -p "$pid" 2>/dev/null || true)"
-    # Anchor to a path boundary so /repos/mm-1 does not match /repos/mm-10.
-    case "$args" in
-      *"$repo/"*|*"$repo "*) : ;;  # argv references this checkout
-      *) continue ;;
-    esac
-    case "$args" in
-      *"expo start"*|*"metro"*|*"react-native/cli"*|*"@react-native-community/cli"*)
-        kill "$pid" 2>/dev/null && reaped="$reaped $pid" ;;
-    esac
+    kill "$pid" 2>/dev/null && reaped="$reaped $pid"
   done <<LIST
-$(pgrep -f "expo start|metro" 2>/dev/null)
+$(_checkout_metro_pids "$repo")
 LIST
   if [ -n "$reaped" ]; then
     printf 'Reaped leaked Metro bundler(s) for this checkout:%s\n' "$reaped" >&2
@@ -34,20 +40,19 @@ LIST
   return 0
 }
 
-# detect_checkout_metros <checkout-abs-path> — print pids of every metro/expo
-# bundler bound to the checkout, one per line, WITHOUT killing. Powers doctor's
-# orphan visibility so leaked bundlers are surfaced without a launch.
 detect_checkout_metros() {
-  local repo="$1" pid args
+  local repo="$1" live_port="${2:-}" pid args
   [ -n "$repo" ] || return 0
   while IFS= read -r pid; do
     [ -n "$pid" ] || continue
-    args="$(ps -o args= -p "$pid" 2>/dev/null || true)"
-    case "$args" in *"$repo/"*|*"$repo "*) : ;; *) continue ;; esac
-    case "$args" in
-      *"expo start"*|*"metro"*|*"react-native/cli"*|*"@react-native-community/cli"*) printf '%s\n' "$pid" ;;
-    esac
+    # A bundler serving the currently-managed port is the LIVE valid Metro, not
+    # an orphan — exclude it so doctor never flags the running server.
+    if [ -n "$live_port" ]; then
+      args="$(ps -o args= -p "$pid" 2>/dev/null || true)"
+      case "$args" in *"--port $live_port"*|*"--port=$live_port"*) continue ;; esac
+    fi
+    printf '%s\n' "$pid"
   done <<LIST
-$(pgrep -f "expo start|metro" 2>/dev/null)
+$(_checkout_metro_pids "$repo")
 LIST
 }

package/dist/commands/debug.js

@@ -11,6 +11,7 @@ async function handleDebug(argv) {
   if (!adapter) return usageOut(json, "debug", `could not detect the MetaMask repo type for ${target}`, ADAPTER_DETECT_NEXT);
   const surface = getAdapterSurface(adapter);
   if (surface.headless) return usageOut(json, "debug", "core is headless; there is no debug console.", surface.hints.relaunch);
+  surface.resolveSlotPorts(target);
   const worker = flag(options, "worker");
   const devMenu = flag(options, "devMenu");
   if (adapter === "mobile" && worker) {

package/dist/commands/doctor.js

@@ -43,7 +43,7 @@ async function handleDoctor({ options }) {
     runtime = await surface.runtimeStatus(target);
   } catch {
   }
-  const orphanMetros = adapter === "mobile" ? detectOrphanMetros(target) : [];
+  const orphanMetros = adapter === "mobile" ? detectOrphanMetros(target, process.env.WATCHER_PORT) : [];
   if (json) console.log(JSON.stringify({ ...result, runtime, orphanMetros }, null, 2));
   else {
     const out = (style, text) => color(style, text, { stream: process.stdout });
@@ -70,10 +70,10 @@ async function handleDoctor({ options }) {
   }
   return result.status === "pass" ? 0 : 1;
 }
-function detectOrphanMetros(target) {
+function detectOrphanMetros(target, livePort) {
   try {
     const script = path.join(runnerDir, "adapters/shared/reap-checkout-metros.sh");
-    const out = execFileSync("bash", ["-c", `. "$1"; detect_checkout_metros "$2"`, "mm-harness-doctor", script, target], {
+    const out = execFileSync("bash", ["-c", `. "$1"; detect_checkout_metros "$2" "$3"`, "mm-harness-doctor", script, target, livePort ?? ""], {
       encoding: "utf8",
       timeout: 5e3
     });

package/dist/commands/logs.js

@@ -2,7 +2,7 @@ import fs from "node:fs";
 import path from "node:path";
 import { runnerDir } from "../paths.js";
 import { getAdapterSurface } from "../adapters/surface.js";
-import { ADAPTER_DETECT_NEXT, EXIT, flag, parseFlags, resolveAdapter, spawnScript, str, targetOf, usageOut } from "./shared.js";
+import { ADAPTER_DETECT_NEXT, EXIT, flag, parseFlags, resolveAdapter, spawnInherit, str, targetOf, usageOut } from "./shared.js";
 const LOGS_BOOLEANS = /* @__PURE__ */ new Set(["full", "json"]);
 async function handleLogs(argv) {
   const { options } = parseFlags(argv, LOGS_BOOLEANS);
@@ -43,30 +43,28 @@ async function handleLogs(argv) {
       surface.hints.launch
     );
   }
+  if (json) {
+    console.log(JSON.stringify({ schemaVersion: 1, command: "logs", adapter, source, logFile, exitCode: EXIT.ok }, null, 2));
+    return EXIT.ok;
+  }
+  process.stderr.write(
+    `${source} log: ${logFile}
+  tail it yourself: tail -f ${logFile}   \xB7   page it: less +F ${logFile}
+  following below (Ctrl-C to stop) \u2026
+`
+  );
   const full = flag(options, "full");
   if (full) {
-    const result2 = spawnScript("tail", ["-n", "+1", "-F", logFile], target, json);
-    return result2.status === 0 ? EXIT.ok : EXIT.runtime;
+    return spawnInherit("tail", ["-n", "+1", "-F", logFile], target) === 0 ? EXIT.ok : EXIT.runtime;
   }
   const logTui = path.join(runnerDir, "adapters/shared/log-tui.mjs");
   const eventCount = process.env.RECIPE_LOG_EVENTS ?? "20";
   const uiMode = process.env.RECIPE_LOG_UI_MODE ?? "compact";
-  const result = spawnScript(
+  return spawnInherit(
     process.execPath,
     [logTui, "tail", "--log", logFile, "--events", eventCount, "--follow", "--mode", uiMode],
-    target,
-    json
-  );
-  if (json) {
-    console.log(
-      JSON.stringify(
-        { schemaVersion: 1, command: "logs", adapter, source, logFile, exitCode: result.status === 0 ? EXIT.ok : EXIT.runtime },
-        null,
-        2
-      )
-    );
-  }
-  return result.status === 0 ? EXIT.ok : EXIT.runtime;
+    target
+  ) === 0 ? EXIT.ok : EXIT.runtime;
 }
 export {
   handleLogs

package/dist/commands/shared.js

@@ -85,6 +85,27 @@ function spawnScript(script, args, cwd, json, env) {
   if (!json && output) process.stderr.write(output);
   return { status: result.status ?? 1, output };
 }
+function spawnInherit(script, args, cwd) {
+  const isNodeScript = script === process.execPath && args.length > 0;
+  const stem = (isNodeScript ? path.basename(args[0]) : path.basename(script)).replace(/[^A-Za-z0-9]/gu, "_").toUpperCase();
+  const override = process.env[`MM_HARNESS_SCRIPT_BIN_${stem}`];
+  const bin = override ?? script;
+  const directArgs = override !== void 0 && isNodeScript ? args.slice(1) : args;
+  const { bin: invokeBin, args: spawnArgs } = resolveLeafInvoke(bin, directArgs);
+  const result = spawnSync(invokeBin, spawnArgs, { cwd, stdio: "inherit", env: process.env });
+  if (result.error) {
+    const leaf = isNodeScript ? path.basename(args[0]) : path.basename(script);
+    const code = result.error.code ?? "ESPAWN";
+    process.stderr.write(
+      `leaf could not start: ${leaf} (${code})
+  Next: reinstall mm-harness (npm i -g @deeeed/metamask-harness) \u2014 the leaf is missing or not executable
+`
+    );
+    return 1;
+  }
+  if (result.status !== null) return result.status;
+  return result.signal === "SIGINT" || result.signal === "SIGTERM" ? 0 : 1;
+}
 function spawnScriptStreaming(script, args, cwd, env) {
   const isNodeScript = script === process.execPath && args.length > 0;
   const stem = isNodeScript ? path.basename(args[0]).replace(/[^A-Za-z0-9]/gu, "_").toUpperCase() : path.basename(script).replace(/[^A-Za-z0-9]/gu, "_").toUpperCase();
@@ -149,6 +170,7 @@ export {
   flag,
   parseFlags,
   resolveAdapter,
+  spawnInherit,
   spawnScript,
   spawnScriptStreaming,
   str,

package/dist/mm-harness-cli.js

@@ -469,8 +469,15 @@ function translateActionsRaw(argv) {
   const withJson = rest.includes("--json") ? rest : [...rest, "--json"];
   return ["manifest", ...withJson];
 }
+const pkgVersion = (() => {
+  try {
+    return JSON.parse(fs.readFileSync(path.join(packageRoot, "package.json"), "utf8")).version ?? "unknown";
+  } catch {
+    return "unknown";
+  }
+})();
 const program = new Command();
-program.name("mm-harness").description("the MetaMask recipe harness: launch the app, prove behavior, manage the runtime overlay").helpOption("-h, --help", "Show grouped help").showHelpAfterError("(run `mm-harness --help` for the full surface)").configureHelp({ formatHelp: () => groupedHelp() });
+program.name("mm-harness").description("the MetaMask recipe harness: launch the app, prove behavior, manage the runtime overlay").version(pkgVersion, "-v, --version", "Print the mm-harness version").helpOption("-h, --help", "Show grouped help").showHelpAfterError("(run `mm-harness --help` for the full surface)").configureHelp({ formatHelp: () => groupedHelp() });
 for (const command of REAL) {
   program.command(command.name).description(command.summary).allowUnknownOption().helpOption("-h, --help", "Show command help").configureHelp({ formatHelp: () => `${command.helpText}
 ` }).argument("[args...]").action(async () => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@deeeed/metamask-harness",
-  "version": "0.6.0",
+  "version": "0.6.2",
   "type": "module",
   "bin": {
     "mm-harness": "bin/mm-harness"