@deeeed/metamask-harness 0.92.8 → 0.92.9

package/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 ## Unreleased
 
+## 0.92.9 - 2026-10-16
+
+### Changed
+
+- Use recipe-cli 0.20.0 for shared review commands while preserving MetaMask reference discovery, missing-reference diagnostics and parity rules through the existing resolver and checklist rule formatter.
+
 ## 0.92.8 - 2026-10-16
 
 ### Changed

package/README.md

@@ -411,14 +411,14 @@ into markdown, so fences cannot break.
 
 ```bash
 # Which team library owns the change (declared --domain wins, else owned-paths.json).
-mm-harness domain
+farmslot review domain --project <project>
 
 # The review guide: base review + the library's anti-pattern families and parity rule.
-mm-harness help review --domain perps
+farmslot review checklist --project <project> --domain perps
 
 # A checklist the mark shim can track: Setup, Base review, Domain patterns, Parity, Verdict.
-mm-harness review checklist --domain perps --out temp/tasks/review/TAT-1/CHECKLIST.md
-mm-harness review checklist --domain perps --since <last-reviewed-sha>   # incremental re-review
+farmslot review checklist --project <project> --domain perps --out temp/tasks/review/TAT-1/CHECKLIST.md
+farmslot review checklist --project <project> --domain perps --since <last-reviewed-sha>   # incremental re-review
 
 # Where this machine keeps libraries and the other platform's checkout (outside any repo).
 mm-harness config set references.extension ~/dev/metamask-extension

package/dist/provider-helpers.js

@@ -1,5 +1,14 @@
 import { checkDiffSummaryReady, runCloseout, teachRecipeCompletion } from "./commands/checklist.js";
-import { BASE_REVIEW_STEPS, REVIEW_ADAPTERS } from "./review/knowledge.js";
+import {
+  BASE_REVIEW_STEPS,
+  REVIEW_ADAPTERS,
+  configPath,
+  parityAdapter,
+  parityReviewSteps,
+  readConfig,
+  referenceHint,
+  resolveReference
+} from "./review/knowledge.js";
 const task = {
   stepGates: [{ label: /mm-harness check diff/u, ready: checkDiffSummaryReady }],
   closeout: runCloseout,
@@ -7,8 +16,19 @@ const task = {
 };
 const review = {
   baseRules: BASE_REVIEW_STEPS.join("\n"),
-  runtimeIds: REVIEW_ADAPTERS,
-  parityRuntimeIds: { mobile: "extension", extension: "mobile" }
+  runtimeIds: [...REVIEW_ADAPTERS, "unscoped"],
+  parityRuntimeIds: { mobile: "extension", extension: "mobile" },
+  parity: ({ runtime, target, hasParityGuide }) => {
+    const adapter = REVIEW_ADAPTERS.find((id) => id === runtime) ?? "unscoped";
+    const parity = parityAdapter(adapter);
+    const reference = parity ? resolveReference(parity, { target, config: readConfig(configPath()) }) : void 0;
+    const referenceMissing = parity && !reference ? `no ${parity} checkout found (${referenceHint(parity)})` : void 0;
+    return {
+      ...reference ? { reference } : {},
+      ...referenceMissing ? { referenceMissing } : {},
+      steps: parityReviewSteps({ adapter, reference, referenceMissing, hasParityGuide })
+    };
+  }
 };
 async function recipeAdvice(options, mode) {
   const { handleRecipeAdvice } = await import("./commands/recipe-advice.js");

package/dist/review/knowledge.js

@@ -445,6 +445,18 @@ function parityAdapter(adapter) {
   if (adapter === "extension") return "mobile";
   return void 0;
 }
+function parityReviewSteps(input) {
+  const parity = parityAdapter(input.adapter);
+  if (input.reference && parity) {
+    return [
+      `For each touched screen, hook, or formatter, compare it with its ${parity} counterpart at ${input.reference.root}` + (input.hasParityGuide ? " using review/parity.md" : "") + `; record every divergence, and apply the library's parity rule to decide which platform must change (the reference platform is never changed to match the other).`
+    ];
+  }
+  if (parity) {
+    return [`Parity with ${parity}: not checked \u2014 ${input.referenceMissing ?? "no reference checkout"}. Record this in the verdict.`];
+  }
+  return ["Parity: core changes are consumed by both mobile and extension; confirm both consumers still compile against the public surface."];
+}
 function renderReviewChecklist(input) {
   const diffRange = input.since ? `${input.since}..HEAD` : `${input.base}...HEAD`;
   const lines = [];
@@ -484,16 +496,7 @@ function renderReviewChecklist(input) {
     }
   }
   lines.push("", "## Parity");
-  const parity = parityAdapter(input.adapter);
-  if (input.reference && parity) {
-    item(
-      `For each touched screen, hook, or formatter, compare it with its ${parity} counterpart at ${input.reference.root}` + (input.knowledge?.parity ? " using review/parity.md" : "") + `; record every divergence, and apply the library's parity rule to decide which platform must change (the reference platform is never changed to match the other).`
-    );
-  } else if (parity) {
-    item(`Parity with ${parity}: not checked \u2014 ${input.referenceMissing ?? "no reference checkout"}. Record this in the verdict.`);
-  } else {
-    item("Parity: core changes are consumed by both mobile and extension; confirm both consumers still compile against the public surface.");
-  }
+  for (const step2 of parityReviewSteps({ ...input, hasParityGuide: Boolean(input.knowledge?.parity) })) item(step2);
   lines.push("", "## Verdict");
   item(VERDICT_STEP);
   return `${lines.join("\n")}
@@ -555,6 +558,7 @@ export {
   loadOwnedPaths,
   matchOwnedFiles,
   parityAdapter,
+  parityReviewSteps,
   parseAdvisorSettings,
   parseLibraryPathEnv,
   parseSections,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@deeeed/metamask-harness",
-  "version": "0.92.8",
+  "version": "0.92.9",
   "type": "module",
   "bin": {
     "mm-harness": "bin/mm-harness"
@@ -28,7 +28,7 @@
     "@farmslot/agent-runtime": "0.25.1",
     "@farmslot/handoff": "^0.3.1",
     "@farmslot/protocol": "0.48.0",
-    "@farmslot/recipe-cli": "0.19.1",
+    "@farmslot/recipe-cli": "0.20.0",
     "@farmslot/recipe-runner": "0.37.1",
     "agent-device": "0.19.3",
     "commander": "^12.0.0",