@deeeed/metamask-harness 0.14.0 → 0.14.2

package/dist/mm-harness-cli.js

@@ -6,6 +6,8 @@ import { Command } from "commander";
 import { color } from "./cli-color.js";
 import { handleUpdate, maybeNudge } from "./commands/update.js";
 import { handleCallHelp } from "./commands/call.js";
+import { getAdapterSurface } from "./adapters/surface.js";
+import { detectAdapter } from "./harness.js";
 const packageRoot = path.resolve(path.dirname(fileURLToPath(import.meta.url)), "..");
 globalThis.__MM_HARNESS_WRAPPER__ = true;
 const { main: recipeMain } = await import("./cli.js");
@@ -24,9 +26,13 @@ const REAL = [
   --target <path>                    Checkout path (default: cwd)
   --json                             Machine-readable envelope { adapter, target, devices[], next }
   --fast                             Skip all live-state probes (instant output, safe for scripts)
+  --all-devices                      Mobile only: show every connected device instead of the slot-scoped target
 
   Human output: static device list prints immediately; live-state lines append
   after the ~2s probe window (screen, wallet, account, fixture per device).
+  By default mobile status shows the configured target only; --device <id>
+  switches the active target for that command. Other devices that answer on the
+  same Metro are reported as additional reachable targets.
 
   The devices[] section carries per device:
     { platform, id, name, state, selected }   always present
@@ -168,6 +174,7 @@ Example:
   --expect-live                      Exit 0 iff the runtime is live (extension: watcher+CDP; mobile: Metro+bridge; core: deps), non-zero + teaching escape otherwise
   --cdp-port <port>                  Extension CDP port for the liveness probe (env: CDP_PORT / RECIPE_CDP_PORT)
   --device <udid|serial|name>        Mobile only: target this device (env: IOS_SIMULATOR / ADB_SERIAL). doctor reports the connected devices; it never gates on ambiguity.
+  --all-devices                      Mobile only: show every connected device instead of the slot-scoped target
   --adapter <mobile|extension|core>  Target adapter (auto-detected inside a checkout)
   --target <path>                    Checkout path (default: cwd)
   --runtime-dir <dir>                Runtime dir containing agentic-runtime.json (relative to target)
@@ -179,6 +186,27 @@ Example:
   mm-harness doctor --adapter extension --target /path/to/checkout --cdp-port 6662 --expect-live
   mm-harness doctor --adapter mobile --target /path/to/checkout`
   },
+  {
+    name: "check",
+    summary: "Run bounded repo-local checks for an active git diff and write validation artifacts.",
+    example: "mm-harness check diff --profile fast",
+    helpText: `mm-harness check diff [flags]
+
+  Run bounded repo-local checks for the active git diff. Fast profile runs changed-file
+  ESLint, Prettier, and changed test files. Full profile also runs repo typecheck
+  when available. Does not launch an app or run a recipe.
+
+  --target <path>                    Checkout path (default: cwd)
+  --adapter <mobile|extension|core>  Adapter label (auto-detected when possible)
+  --base <ref>                       Diff base (default: PR base, then remote HEAD, then repo fallback)
+  --profile <fast|full>              Validation depth (default: fast)
+  --artifacts-dir <dir>              Write validation-summary.json/.md and per-check logs
+  --json                             Machine-readable envelope
+
+Example:
+  mm-harness check diff --profile fast --artifacts-dir artifacts/validation
+  mm-harness check diff --profile full --json`
+  },
   {
     name: "recipe-quality",
     summary: "Build the recipe-quality artifact from a compact verdict JSON (validates before writing).",
@@ -188,7 +216,12 @@ Example:
   Build artifacts/recipe-quality.json from the compact fields a recipe-quality pass
   produces (verdict + reasons + optional guidance/dimensions/findings/delta/training).
   The artifact is validated against the RecipeQualityArtifact schema before it is
-  written \u2014 an invalid input never reaches disk.
+  written \u2014 an invalid input never reaches disk. Shorthand without "build" is
+  also accepted when --input/--output are present.
+
+  This CLI does not run the recipe critique itself; use the fs-recipe-quality
+  skill to judge recipe/evidence quality, then use this command to build the
+  canonical artifact from that compact verdict.
 
   --input <compact.json>   Compact verdict JSON: { "verdict": "pass|warn|fail", "reasons": [..],
                              "betterVersionGuidance"?: [..], "dimensions"?: {..}, "trainingFields"?: {..}, \u2026 }
@@ -199,6 +232,7 @@ Example:
 
 Example:
   mm-harness recipe-quality build --input compact.json --output artifacts/recipe-quality.json
+  mm-harness recipe-quality --input compact.json --output artifacts/recipe-quality.json
   mm-harness recipe-quality build --input compact.json --output artifacts/recipe-quality.json --json`
   },
   {
@@ -297,7 +331,7 @@ Example:
   },
   {
     name: "launch",
-    summary: "Launch the app (Metro/build + boot), auto-ensuring the runtime overlay first. Mobile: ios|android required.",
+    summary: "Launch the app and adapter dev server, auto-ensuring the runtime overlay first. Mobile: ios|android required.",
     example: "mm-harness launch ios",
     helpText: `mm-harness launch [ios|android] [flags]
 
@@ -312,7 +346,7 @@ Example:
   --runway                           Post-launch runway check (mobile only; teaching error elsewhere)
   --watch                            Persistent webpack watcher then relaunch (extension only)
   --sidepanel | --fullscreen         Extension display mode (default --fullscreen)
-  --url <dapp-url>                    Open a dapp in the main tab beside the sidepanel (extension only)
+  --url <dapp-url>                    Open a dapp beside the sidepanel (default: MetaMask test dapp)
   --heal <off|infra-only|auto>       Healing policy (default auto); bounds always enforced
   --device <udid|name>               Target simulator/device (env: IOS_SIMULATOR / ADB_SERIAL)
   --cdp-port <port>                  Extension CDP port (env: CDP_PORT / RECIPE_CDP_PORT)
@@ -424,14 +458,6 @@ Example:
   mm-harness fixtures finalize --fixture wallet-fixture.json --state fixture-state.json --cdp-port 6661 --extension-dir dist/chrome`
   }
 ];
-const RETIRED_INTERNAL = [
-  "runtime-health",
-  "runtime-decision",
-  "runtime-launch",
-  "resolve-extension",
-  "ensure-ready",
-  "self-test"
-];
 const RETIRED = [
   {
     name: "live",
@@ -445,14 +471,7 @@ Replacement: mm-harness launch --verify (install overlay \u2192 launch \u2192 CD
 
 Replacement: mm-harness actions --raw   (works now \u2014 dumps the raw action registry JSON,
 identical to the old \`manifest --json\`). Manifest validation moved into doctor / run --plan.`
-  },
-  ...RETIRED_INTERNAL.map((name) => ({
-    name,
-    message: `mm-harness ${name} is retired (exit 2).
-
-It is internal now \u2014 its logic lives inside doctor / launch / verify self-healing.
-Use: mm-harness doctor`
-  }))
+  }
 ];
 const HELP_GROUPS = [
   {
@@ -468,7 +487,7 @@ const HELP_GROUPS = [
   {
     title: "PROVE",
     blurb: "run recipes and inspect readiness",
-    commands: ["run", "doctor", "recipe-quality"]
+    commands: ["run", "doctor", "check", "recipe-quality"]
   },
   {
     title: "RUNTIME OVERLAY",
@@ -486,6 +505,14 @@ function commandMeta(name) {
   if (real) return { summary: real.summary, example: real.example, planned: false };
   return { summary: "", example: "", planned: false };
 }
+function adapterFromRuntimeContext(ctx) {
+  const raw = typeof ctx.platform === "string" ? ctx.platform : void 0;
+  if (!raw) return void 0;
+  if (raw === "mobile" || raw === "ios" || raw === "android") return "mobile";
+  if (raw === "extension" || raw === "chrome-extension") return "extension";
+  if (raw === "core") return "core";
+  return void 0;
+}
 function detectedSlotLine(out) {
   const ctxPath = path.join(
     process.cwd(),
@@ -494,10 +521,14 @@ function detectedSlotLine(out) {
   );
   try {
     const ctx = JSON.parse(fs.readFileSync(ctxPath, "utf8"));
+    const adapter = adapterFromRuntimeContext(ctx) ?? detectAdapter(process.cwd());
     const parts = [];
     if (ctx.slotId) parts.push(`slot ${out("ok", String(ctx.slotId))}`);
     if (ctx.simulator) parts.push(`device ${out("ok", String(ctx.simulator))}`);
-    if (ctx.metroPort) parts.push(`metro :${out("ok", String(ctx.metroPort))}`);
+    const devServerPort = ctx.watcherPort ?? ctx.devServerPort ?? ctx.metroPort;
+    if (adapter && adapter !== "core" && devServerPort) {
+      parts.push(`${getAdapterSurface(adapter).devServer.label} :${out("ok", String(devServerPort))}`);
+    }
     if (ctx.gitBranch) parts.push(`branch ${out("info", String(ctx.gitBranch))}`);
     if (parts.length === 0) return null;
     return `${out("label", "SLOT")} \u2014 this checkout is a prepared slot: ${parts.join(" \xB7 ")}`;
@@ -582,7 +613,15 @@ for (const command of REAL) {
     process.exit(await delegate(argv));
   });
 }
-const HIDDEN = ["completion-candidates"];
+const HIDDEN = [
+  "completion-candidates",
+  "runtime-health",
+  "runtime-decision",
+  "runtime-launch",
+  "resolve-extension",
+  "ensure-ready",
+  "self-test"
+];
 for (const name of HIDDEN) {
   program.command(name, { hidden: true }).allowUnknownOption().helpOption(false).argument("[args...]").action(async () => {
     process.exit(await delegate(rawArgv));

package/docs/CLI-SPEC.md

@@ -54,6 +54,7 @@ Name `metamask-recipe` → **REMOVE** (becomes `mm-harness`; no alias). Capabili
 | `resolve-extension` | Resolve the extension id | extension | advanced | **KEEP-INTERNAL** — internal plumbing; called inside `launch`/`launch --verify` |
 | `ensure-ready` | Ensure extension ready (close home tabs) | extension | advanced | **KEEP-INTERNAL** — becomes internal function inside `launch`/`verify` self-healing path |
 | `run` | Run a recipe, write evidence | all | PROVE | ROUTES-NOW |
+| `check diff` | Run bounded repo-local validation for the active git diff | all | PROVE | ROUTES-NOW |
 | `self-test` | Package-wiring smoke (no device) | mobile+extension | advanced | **KEEP-INTERNAL** — CI smoke test; callable via `package.json` scripts; hidden from `--help` |
 | `flows` | List/promote library flows | all | DISCOVER | ROUTES-NOW |
 | `harness install\|verify\|cleanup\|live` | Runtime overlay lifecycle | all (core: no live) | OVERLAY | ROUTES-NOW (`harness` prefix → top-level); `live` **REMOVE** — dissolved into `launch --verify` |
@@ -437,6 +438,30 @@ Readiness check for a checkout without launching the app. Doctor is the single p
 
 ---
 
+## `check diff` (REAL → bounded local validation)
+
+**Synopsis:** `mm-harness check diff [flags]`
+
+Runs bounded validation against the target checkout's active git diff. This is
+for local PR readiness loops and agent templates; it does not launch an app or
+run a recipe.
+
+| Flag | Values | Default | Purpose |
+| --- | --- | --- | --- |
+| `--target <repo>` | path | cwd | MetaMask checkout to inspect |
+| `--adapter <adapter>` / `--platform <adapter>` | mobile\|extension\|core | auto-detect | Adapter label for the report |
+| `--base <ref>` | git ref | GitHub PR base, then remote HEAD, then `origin/develop`/`origin/main`/local fallback | Diff base |
+| `--profile <profile>` | `fast`\|`full` | `fast` | `fast` runs changed-file ESLint/Prettier/Jest; `full` also runs repo typecheck when available |
+| `--artifacts-dir <dir>` | path | `temp/recipe/check-diff/<timestamp>` | Writes summary and per-check logs |
+| `--json` | bool | false | Print the summary envelope to stdout |
+
+**Artifacts:** `validation-summary.json`, `validation-summary.md`, and one log
+per executed check.
+
+**Exit:** 0 pass · 2 bad args / not a git checkout · 5 validation failure.
+
+---
+
 ## `--device <id>` — first-class mobile device targeting (REAL)
 
 `--device` is the uniform mobile device selector on `run`, `call`, `doctor`, and `fixtures`. Pass the **adb serial** for Android (from `adb devices`) or the **UDID / simulator name** for iOS. The harness resolves the adb serial to the Metro CDP target identity internally — users never need to know or set `ANDROID_DEVICE='Pixel 6 - 16 - API 36'`.

package/library/actions/extension/platform/cdp.mjs

@@ -46,14 +46,17 @@ function cdpPort(input) {
   return port;
 }
 
-function extensionTarget(targets, extensionId = null) {
-  return (Array.isArray(targets) ? targets : []).find((target) => {
+export function extensionTarget(targets, extensionId = null) {
+  const candidates = (Array.isArray(targets) ? targets : []).filter((target) => {
     if (target?.type !== 'page') return false;
     if (!String(target?.url ?? '').startsWith('chrome-extension://')) return false;
-    if (!String(target.url).includes('/home.html')) return false;
+    if (!String(target.url).includes('/home.html') && !String(target.url).includes('/sidepanel.html')) return false;
     if (extensionId && extensionIdFromTarget(target) !== extensionId) return false;
     return Boolean(target.webSocketDebuggerUrl);
-  }) ?? null;
+  });
+  return candidates.find((target) => String(target.url).includes('/home.html')) ??
+    candidates.find((target) => String(target.url).includes('/sidepanel.html')) ??
+    null;
 }
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@deeeed/metamask-harness",
-  "version": "0.14.0",
+  "version": "0.14.2",
   "type": "module",
   "bin": {
     "mm-harness": "bin/mm-harness"
@@ -16,9 +16,9 @@
     "check:syntax": "find . -name '*.mjs' -print0 | xargs -0 -n1 node --check"
   },
   "dependencies": {
-    "@farmslot/agent-runtime": "^0.1.0",
-    "@farmslot/protocol": "^0.7.3",
-    "@farmslot/recipe-harness": "^0.4.0",
+    "@farmslot/agent-runtime": "^0.1.1",
+    "@farmslot/protocol": "^0.7.6",
+    "@farmslot/recipe-harness": "^0.4.3",
     "commander": "^12.0.0",
     "viem": "^2.54.3"
   },