@deeeed/metamask-harness 0.92.10 → 0.92.12

package/CHANGELOG.md

@@ -2,6 +2,22 @@
 
 ## Unreleased
 
+### Changed
+
+- Refresh Terminal contract goldens for the shipped console-error window, capture-provider metadata and engine evidence inventories.
+
+## 0.92.12 - 2026-10-16
+
+### Fixed
+
+- Stop checkout-owned idb companions with the selected simulator identity, even when Metro and the simulator are already stopped; preserve other devices, checkouts and reused process IDs.
+
+## 0.92.11 - 2026-10-16
+
+### Changed
+
+- Supply the existing platform-specific recipe quality guide to Farmslot quality help, with canonical recipe commands and unchanged opt-in advice.
+
 ## 0.92.10 - 2026-10-16
 
 ### Changed

package/adapters/mobile/lib/idb-companion.sh

@@ -0,0 +1,68 @@
+#!/usr/bin/env bash
+# Stop only companions bound to this checkout and its selected simulator.
+
+_idb_companion_identity() {
+  ps -p "$1" -o pid=,pgid=,lstart=,comm= 2>/dev/null
+}
+
+# 1 is foreign; 2 is unknown and must not authorize a signal.
+_idb_companion_owned() {
+  local repo="$1" udid="$2" pid="$3" identity="$4" current cwd args
+  current="$(_idb_companion_identity "$pid")"
+  [ -n "$current" ] && [ "$current" = "$identity" ] || return 2
+  case "$current" in *'/idb_companion'|*' idb_companion') ;; *) return 1 ;; esac
+  cwd="$(lsof -a -p "$pid" -d cwd -Fn 2>/dev/null | sed -n 's/^n//p')"
+  [ -n "$cwd" ] || return 2
+  case "$cwd" in "$repo"|"$repo/"*) ;; *) return 1 ;; esac
+  [ -n "$udid" ] || return 2
+  args="$(ps -p "$pid" -o args= 2>/dev/null)" || return 2
+  case " $args " in *" --udid $udid "*|*" --udid=$udid "*) ;; *) return 1 ;; esac
+  [ "$(_idb_companion_identity "$pid")" = "$identity" ] || return 2
+}
+
+stop_checkout_idb_companions() {
+  local repo="$1" udid="${2:-}" pids pid identity status signal current attempt stopped
+  if pids="$(pgrep -u "$(id -u)" -x idb_companion 2>/dev/null)"; then :
+  else
+    status=$?
+    [ "$status" -eq 1 ] && return 0
+    printf 'Could not inspect idb companions. Next: mm-harness status --target "%s"\n' "$repo" >&2
+    return 1
+  fi
+  for pid in $pids; do
+    case "$pid" in ''|*[!0-9]*) return 1 ;; esac
+    [ "$pid" -gt 0 ] || return 1
+    identity="$(_idb_companion_identity "$pid")"
+    if [ -z "$identity" ] && ! kill -0 "$pid" 2>/dev/null; then continue; fi
+    if _idb_companion_owned "$repo" "$udid" "$pid" "$identity"; then :
+    else
+      status=$?
+      [ "$status" -eq 1 ] && continue
+      printf 'Companion ownership is unknown. Next: mm-harness status --target "%s"\n' "$repo" >&2
+      return 1
+    fi
+    stopped=false
+    for signal in TERM KILL; do
+      if ! _idb_companion_owned "$repo" "$udid" "$pid" "$identity"; then
+        printf 'Companion ownership changed. Next: mm-harness status --target "%s"\n' "$repo" >&2
+        return 1
+      fi
+      kill -"$signal" "$pid" 2>/dev/null || true
+      for attempt in $(seq 1 20); do
+        current="$(_idb_companion_identity "$pid")"
+        if [ -z "$current" ] && ! kill -0 "$pid" 2>/dev/null; then stopped=true; break; fi
+        if [ "$current" != "$identity" ]; then
+          printf 'Companion identity changed during stop. Next: mm-harness status --target "%s"\n' "$repo" >&2
+          return 1
+        fi
+        sleep 0.1
+      done
+      [ "$stopped" = true ] && break
+    done
+    if [ "$stopped" != true ]; then
+      printf 'Companion %s did not stop. Next: mm-harness status --target "%s"\n' "$pid" "$repo" >&2
+      return 1
+    fi
+    printf 'Stopped checkout idb companion %s\n' "$pid" >&2
+  done
+}

package/adapters/mobile/stop-metro.sh

@@ -32,11 +32,12 @@ if ! command -v recipe_runtime_dir >/dev/null 2>&1; then
   exit 1
 fi
 
+. "$MOBILE_STOP_DIR/../shared/resolve-slot-ports.sh"
+
 # No explicit port: resolve the checkout's own (slot context first, then the
 # pool/formula fallbacks) so we stop THIS slot's Metro, not the 8081 default.
 if [ -z "$PORT" ]; then
   # shellcheck disable=SC1091
-  . "$MOBILE_STOP_DIR/../shared/resolve-slot-ports.sh"
   resolved_port="$(resolve_mobile_runtime_ports "$TARGET" 2>/dev/null | sed -n 's/^WATCHER_PORT=//p' | head -1 || true)"
   PORT="${resolved_port:-8081}"
 fi
@@ -56,6 +57,15 @@ else
   printf 'Metro not running on port %s — nothing to stop\n' "$PORT" >&2
 fi
 
+# Companion cleanup retains the selected device and checkout boundary.
+. "$MOBILE_STOP_DIR/lib/idb-companion.sh"
+companion_udid="${SIM_UDID:-}"
+if [ -z "$companion_udid" ]; then
+  mobile_identity="$(resolve_mobile_runtime_ports "$TARGET" 2>/dev/null || true)"
+  companion_udid="$(printf '%s\n' "$mobile_identity" | sed -n 's/^SIM_UDID=//p' | head -1)"
+fi
+stop_checkout_idb_companions "$TARGET" "$companion_udid" || exit 1
+
 # The stop decision is made; everything below is best-effort cleanup (pid file,
 # leaked-bundler sweep, tmux window). None of it may fail the command — the
 # idempotent-stop contract is already satisfied. Drop set -e for the tail so an

package/dist/commands/help.js

@@ -49,6 +49,17 @@ function recipeHelpPayload(topic, harnessVersion, adapter, target) {
     safety: topic.safety
   };
 }
+function recipeHelpSectionLines(topic, adapter, sectionId) {
+  const paint = (style, text) => color(style, text, { stream: process.stdout });
+  const lines = [];
+  for (const [index, section] of scopedSections(topic, adapter).entries()) {
+    if (sectionId && section.id !== sectionId) continue;
+    lines.push("", paint("label", `${index + 1}. ${section.title}`), `   ${section.instruction}`);
+    for (const detail of section.details) lines.push(`   - ${detail}`);
+    for (const command of section.commands) lines.push(`   $ ${paint("cmd", command)}`);
+  }
+  return lines;
+}
 function renderRecipeHelp(topic, harnessVersion, adapter) {
   const paint = (style, text) => color(style, text, { stream: process.stdout });
   const lines = [
@@ -60,11 +71,7 @@ function renderRecipeHelp(topic, harnessVersion, adapter) {
     `Visual tutorial: ${paint("cmd", "mm-harness tutorial")}`
   ];
   if (adapter === "unscoped") lines.push(paint("warn", `No MetaMask checkout detected. Next: ${unscopedNext()}.`));
-  for (const [index, section] of scopedSections(topic, adapter).entries()) {
-    lines.push("", paint("label", `${index + 1}. ${section.title}`), `   ${section.instruction}`);
-    for (const detail of section.details) lines.push(`   - ${detail}`);
-    for (const command of section.commands) lines.push(`   $ ${paint("cmd", command)}`);
-  }
+  lines.push(...recipeHelpSectionLines(topic, adapter));
   lines.push("", `${paint("warn", "Safety:")} ${topic.safety}`);
   return `${lines.join("\n")}
 `;
@@ -88,5 +95,6 @@ export {
   handleHelp,
   loadRecipeHelp,
   recipeHelpPayload,
+  recipeHelpSectionLines,
   renderRecipeHelp
 };

package/dist/provider.js

@@ -16,6 +16,8 @@ import { abortRunMainnetOrders, cancelRunMainnetOrders } from "./adapters/perps-
 import { adapterRegistry } from "./adapters/registry.js";
 import { getAdapterSurface } from "./adapters/surface.js";
 import { applyDeviceTargeting } from "./commands/device-target.js";
+import { loadRecipeHelp, recipeHelpSectionLines } from "./commands/help.js";
+import { runnerDir } from "./paths.js";
 import { metaMaskEngine } from "./engine.js";
 import { providerHost } from "./host.js";
 import { configureProviderReadiness, doctorOptions } from "./provider-doctor.js";
@@ -70,7 +72,19 @@ async function createProvider(context) {
   return {
     runtime,
     engine: metaMaskEngine,
-    commandOptions: { ...helperCommandOptions, status: statusOptions, stop: stopOptions, doctor: doctorOptions, logs: createLogsOptions(runtime) },
+    commandOptions: {
+      ...helperCommandOptions,
+      recipeQuality: {
+        ...helperCommandOptions.recipeQuality,
+        get guidance() {
+          return recipeHelpSectionLines(loadRecipeHelp(runnerDir), adapter, "quality").join("\n");
+        }
+      },
+      status: statusOptions,
+      stop: stopOptions,
+      doctor: doctorOptions,
+      logs: createLogsOptions(runtime)
+    },
     ...executes ? {
       async cancel() {
         await abortRunMainnetOrders(context.target.value);

package/package.json

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

package/site/assets/help-recipes.json

@@ -216,7 +216,7 @@
             "extension"
           ]
         },
-        "Verdict: pass when every named claim ran an executable path with an assertion and consistent evidence; warn when the supported claims pass but non-blocking gaps remain; fail when a named claim has no executable path or assertion, the graph can pass unconditionally, required evidence is missing or contradictory, or the relevant path did not run. A visual claim cannot pass without a settled capture. Write only the compact {verdict, reasons} to artifacts/recipe-quality-input.json and build the artifact with mm-harness recipe-quality build; never hand-author it.",
+        "Verdict: pass when every named claim ran an executable path with an assertion and consistent evidence; warn when the supported claims pass but non-blocking gaps remain; fail when a named claim has no executable path or assertion, the graph can pass unconditionally, required evidence is missing or contradictory, or the relevant path did not run. A visual claim cannot pass without a settled capture. Write only the compact {verdict, reasons} to artifacts/recipe-quality-input.json and build the artifact with farmslot recipe quality build; never hand-author it.",
         "Independent critique, when one is requested: the verdict with one evidence-backed sentence, uncovered or weak proof targets, graph, composition, adapter and reuse findings, evidence mismatches and flake risks, and the three highest-value concrete edits.",
         {
           "text": "State the simulator or device, build type and app state. Unlock through a fixture or a documented primitive; keep account, balance, network, permissions and feature flags deterministic; use existing page objects, test ids and fixtures; capture only after ui.wait_for, a route assertion or a state assertion; teardown must not leak balances, pending transactions, selected network, permissions or onboarding state into the next run.",
@@ -239,9 +239,9 @@
         "Use fixtures that make the intended and incorrect outcomes differ. For changing geometry or values, declare the observation window and compare the same entity and inputs; do not select only favorable samples. Promote trace, screenshots, video and provenance as one immutable package, and inspect replacement image bytes again."
       ],
       "commands": [
-        "mm-harness run ./proof.recipe.json --artifacts-dir temp/recipe-proof --json",
-        "mm-harness last --json",
-        "mm-harness recipe-quality build --input recipe-quality-input.json --output recipe-quality.json --json"
+        "farmslot recipe run ./proof.recipe.json --artifacts-dir temp/recipe-proof --json",
+        "farmslot recipe status --json",
+        "farmslot recipe quality build --input recipe-quality-input.json --output recipe-quality.json --json"
       ]
     },
     {