@deeeed/metamask-harness 0.66.0 → 0.66.1

package/CHANGELOG.md

@@ -2,6 +2,14 @@
 
 ## Unreleased
 
+## 0.66.1 - 2026-09-25
+
+### Fixed
+
+- Capture market-detail report evidence from the driven document, excluding adjacent Extension views. Require an explicit market and assert no position before proving Long/Short controls. Clarify window versus document captures in discovery.
+
+- Recognize current and legacy Perps Home markers and wait for loading to finish. Navigation reports tutorial gates without dismissing them; dependent navigation and consistency checks require explicit setup.
+
 ## 0.66.0 - 2026-09-25
 
 ### Added

package/docs/RECIPES.md

@@ -213,7 +213,7 @@ mm-harness run perps.release-market-filters --describe --adapter extension --jso
 | Deposit and withdraw previews without submission | `perps.release-funds-flows` |
 | Resting limit placement and exact cancellation | `perps.release-live-order order_type=limit limit_price='"<price>"'` |
 | Filled market placement and exact position close | `perps.release-live-order order_type=market` |
-| Market header and statistics | `perps.release-market-details` |
+| Market header and statistics on an unheld market | `perps.release-market-details market='<unheld symbol>'` |
 | Market categories | `perps.release-market-filters` |
 | Market text search | `perps.release-market-search` |
 | Long/Short and Market/Limit entry controls | `perps.release-order-entry case=configuration` |

package/library/actions/extension/perps/assert_visible_consistency.mjs

@@ -284,7 +284,8 @@ export async function assertVisibleConsistency(page, input, testOptions = {}) {
 
   if (options.mode === 'watchlist-stream') {
     const openHome = testOptions.openHome ?? openPerpsHome;
-    await openHome(page, options.timeoutMs);
+    const home = await openHome(page, options.timeoutMs);
+    if (home?.gate) throw new Error('Perps tutorial blocks navigation; run perps.dismiss-tutorial as explicit setup first.');
     const first = await pollLoaded(
       () => watchlistRows(page, options.maxItems),
       (rows) => rows.length > 0,
@@ -343,7 +344,8 @@ export async function assertVisibleConsistency(page, input, testOptions = {}) {
     { action: 'metamask.perps.select_activity_filter', node: { filter } },
     { timeoutMs },
   ));
-  await openHome(page, options.timeoutMs);
+  const home = await openHome(page, options.timeoutMs);
+  if (home?.gate) throw new Error('Perps tutorial blocks navigation; run perps.dismiss-tutorial as explicit setup first.');
   const recent = await pollLoaded(
     () => recentActivityState(page, options.maxItems),
     (state) => state.ids.length >= options.minimumRecentActivityCount &&

package/library/actions/extension/ui/navigate.mjs

@@ -1,5 +1,5 @@
 // Opens named wallet screens through UI controls or history, and accepts direct URL/hash navigation.
-// Named navigation may dismiss tutorials or drawers and add a missing network before selecting its filter.
+// Named navigation reports tutorial gates, may dismiss drawers, and may add a missing network before selecting its filter.
 import { pathToFileURL } from 'node:url';
 
 import {
@@ -81,12 +81,12 @@ async function goBack(page, timeoutMs) {
 
 async function hasVisibleSelector(page, selector) {
   return page.evaluate(`(() => {
-    const element = document.querySelector(${JSON.stringify(selector)});
-    if (!element) return false;
-    if (element.closest('dialog')?.open === false) return false;
-    const style = getComputedStyle(element);
-    const rect = element.getBoundingClientRect();
-    return style.display !== 'none' && style.visibility !== 'hidden' && rect.width > 0 && rect.height > 0;
+    return [...document.querySelectorAll(${JSON.stringify(selector)})].some((element) => {
+      if (element.closest('dialog')?.open === false) return false;
+      const style = getComputedStyle(element);
+      const rect = element.getBoundingClientRect();
+      return style.display !== 'none' && style.visibility !== 'hidden' && rect.width > 0 && rect.height > 0;
+    });
   })()`);
 }
 
@@ -199,18 +199,25 @@ export async function openHome(page, timeoutMs) {
 export async function openPerpsHome(page, timeoutMs) {
   const bottomNav = dataTestId('bottom-nav-perps');
   const legacyTab = dataTestId('account-overview__perps-tab');
-  const perpsHome = dataTestId('perps-home-page');
+  const perpsHome = ['perps-home-page', 'parent-selector-perps-tab', 'perps-view'].map(dataTestId).join(',');
+  const perpsLoading = dataTestId('perps-view-loading');
   const tutorialSkip = dataTestId('perps-tutorial-skip-button');
   const steps = [];
   const deadline = Date.now() + timeoutMs;
   let pendingActivation;
   let perpsActivation;
+  let perpsControl;
   let perpsHomeVisibleAt;
 
   while (Date.now() < deadline) {
     if (await hasVisibleSelector(page, tutorialSkip)) {
-      await activateSemanticControl(page, tutorialSkip);
-      steps.push(tutorialSkip);
+      return {
+        gate: 'perps-tutorial', gateSelector: tutorialSkip, method: 'visible-ui',
+        ...(perpsActivation ? { navigation: perpsActivation, selector: perpsControl } : {}),
+        href: await currentHref(page), steps,
+      };
+    }
+    if (await hasVisibleSelector(page, perpsLoading)) {
       perpsHomeVisibleAt = undefined;
       await pauseForUi(page);
       continue;
@@ -226,7 +233,7 @@ export async function openPerpsHome(page, timeoutMs) {
         continue;
       }
       return {
-        ...(perpsActivation ? { navigation: perpsActivation, selector: bottomNav } : {}),
+        ...(perpsActivation ? { navigation: perpsActivation, selector: perpsControl } : {}),
         method: 'visible-ui',
         href: await currentHref(page),
         steps,
@@ -236,23 +243,20 @@ export async function openPerpsHome(page, timeoutMs) {
     if (await hasVisibleSelector(page, bottomNav)) {
       if (!perpsActivation) {
         perpsActivation = await activateSemanticControl(page, bottomNav);
+        perpsControl = bottomNav;
         steps.push(bottomNav);
       }
       await pauseForUi(page);
       continue;
     }
     if (await hasVisibleSelector(page, legacyTab)) {
-      const navigation = await activateSemanticControl(page, legacyTab);
-      await page.waitForSelector(dataTestId('perps-view'), {
-        timeoutMs: Math.max(1, deadline - Date.now()),
-      });
-      return {
-        navigation,
-        method: 'visible-ui',
-        href: await currentHref(page),
-        selector: legacyTab,
-        steps,
-      };
+      if (!perpsActivation) {
+        perpsActivation = await activateSemanticControl(page, legacyTab);
+        perpsControl = legacyTab;
+        steps.push(legacyTab);
+      }
+      await pauseForUi(page);
+      continue;
     }
 
     let backControl;
@@ -313,6 +317,7 @@ export async function openPerpsMarket(page, market, timeoutMs) {
     };
   }
   const home = await openPerpsHome(page, Math.max(1, deadline - Date.now()));
+  if (home.gate) throw new Error('Perps tutorial blocks navigation; run perps.dismiss-tutorial as explicit setup first.');
   const selectors = [
     dataTestId(`perps-watchlist-${normalizedMarket}`),
     dataTestId(`explore-markets-${normalizedMarket.replaceAll(':', '-')}`),
@@ -349,6 +354,7 @@ export async function openPerpsActivity(page, timeoutMs) {
   }
   const deadline = Date.now() + timeoutMs;
   const home = await openPerpsHome(page, Math.max(1, deadline - Date.now()));
+  if (home.gate) throw new Error('Perps tutorial blocks navigation; run perps.dismiss-tutorial as explicit setup first.');
   const activity = dataTestId('perps-recent-activity-see-all');
   await page.waitForSelector(activity, { timeoutMs: Math.max(1, deadline - Date.now()) });
   const navigation = await activateSemanticControl(page, activity);
@@ -383,33 +389,17 @@ export async function openPerpsMarketList(page, timeoutMs) {
     };
   }
   const home = await openPerpsHome(page, Math.max(1, deadline - Date.now()));
+  if (home.gate) throw new Error('Perps tutorial blocks navigation; run perps.dismiss-tutorial as explicit setup first.');
   const explore = dataTestId('perps-explore-markets-row');
   const tutorialSkip = dataTestId('perps-tutorial-skip-button');
   if (!(await waitForFirstVisible(page, [explore], deadline))) {
     throw new Error('Perps Explore Markets did not become visible before timeout.');
   }
-  let navigation = await activateSemanticControl(page, explore);
+  const navigation = await activateSemanticControl(page, explore);
   const destination = await waitForFirstVisible(page, [marketList, tutorialSkip], deadline);
   const steps = [...home.steps, explore];
   if (destination === tutorialSkip) {
-    await activateSemanticControl(page, tutorialSkip);
-    steps.push(tutorialSkip);
-    let exploreReady = false;
-    while (Date.now() < deadline) {
-      if (!(await hasVisibleSelector(page, tutorialSkip)) && await hasVisibleSelector(page, explore)) {
-        exploreReady = true;
-        break;
-      }
-      await pauseForUi(page);
-    }
-    if (!exploreReady) {
-      throw new Error('Perps Explore Markets did not become visible after dismissing the tutorial.');
-    }
-    navigation = await activateSemanticControl(page, explore);
-    steps.push(explore);
-    if (!(await waitForFirstVisible(page, [marketList], deadline))) {
-      throw new Error('Perps market list did not become visible after dismissing the tutorial.');
-    }
+    throw new Error('Perps tutorial blocks navigation; run perps.dismiss-tutorial as explicit setup first.');
   } else if (destination !== marketList) {
     throw new Error('Perps market list did not become visible before timeout.');
   }

package/library/manifests/extension.action-manifest.json

@@ -495,7 +495,7 @@
       ]
     },
     "ui.navigate": {
-      "description": "Navigate by semantic page intent when available; the adapter uses the current visible UI or a verified route. Raw url/hash/path values must be verified against the current app first.",
+      "description": "Navigate by semantic page intent when available; the adapter uses the current visible UI or a verified route. Raw url/hash/path values must be verified against the current app first. Perps Home returns gate=perps-tutorial and gateSelector without dismissing a visible tutorial. Complete explicit tutorial setup before dependent market or activity navigation.",
       "schema": {
         "type": "object",
         "properties": {
@@ -1132,7 +1132,7 @@
       ]
     },
     "ui.screenshot": {
-      "description": "Capture the current native browser window on macOS, with bounded Chrome viewport capture as fallback. Always inspect the resulting PNG; artifact metadata records the provider. Failed real capture does not produce a reconstructed DOM image.",
+      "description": "Capture the current native browser window on macOS, with bounded Chrome viewport capture as fallback. A native window can include a separate side panel or a tab other than the driven page. For report evidence of the driven document, use ui.capture_surface. Always inspect the resulting PNG and artifact metadata; failed real capture does not produce a reconstructed DOM image.",
       "examples": [
         {
           "action": "ui.screenshot",

package/library/recipes/shared/perps/release-market-details.recipe.json

@@ -1,15 +1,15 @@
 {
   "$schema": "https://farmslot.io/schemas/recipe-v1.schema.json",
   "title": "Perps release market details",
-  "description": "Opens one live Perps market through visible UI and proves its header price, change, statistics, and Long/Short controls; it does not assert chart data.",
+  "description": "Opens one live Perps market through visible UI and proves its header price, change, statistics, and Long/Short controls; it does not assert chart data. Requires no existing position in the chosen market: held markets show Modify/Close instead. Verify the fixture with a supported read before selecting market; never close a position to satisfy this recipe.",
   "paramsSchema": {
     "type": "object",
     "additionalProperties": false,
+    "required": ["market"],
     "properties": {
       "market": {
         "type": "string",
-        "default": "ETH",
-        "description": "Perps market symbol to open and verify."
+        "description": "Perps market symbol with no existing position, so Long/Short controls are expected."
       },
       "timeout_ms": {
         "type": "integer",
@@ -45,6 +45,13 @@
         ],
         "max_items": 100,
         "intent": "Prove the requested market appears in a populated live market list",
+        "next": "require-flat-market"
+      },
+      "require-flat-market": {
+        "action": "metamask.perps.assert_positions",
+        "market": "{{params.market}}",
+        "state": "none",
+        "intent": "Require the flat-market fixture before proving Long and Short controls; never change existing positions",
         "next": "open-market"
       },
       "open-market": {
@@ -70,7 +77,7 @@
         "next": "verify-directions"
       },
       "capture-market": {
-        "action": "ui.screenshot",
+        "action": "ui.capture_surface",
         "path": "screenshots/perps-market-details.png",
         "label": "Perps release market details",
         "intent": "Preserve reviewer-visible proof of the selected live market",

package/package.json

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