@deeeed/metamask-harness 0.48.0 → 0.49.0

package/library/actions/core/perps/teardown_state.mjs

@@ -1,21 +1,10 @@
+// Restore selected testnet baselines, canceling orders before closing positions.
+// Omitted domains default to empty; explicit false skips a domain.
+
 import { isDirectRun, resolveNetwork, runAdapter } from './_controller.mjs';
 import { ensureOrders } from './ensure_orders.mjs';
 import { ensurePositions } from './ensure_positions.mjs';
 
-// core Headless composite that restores the POSITION and ORDER baselines for the
-// selected market(s) on HyperLiquid testnet after the proof window. Mirrors the
-// extension perps-domain-dispatcher teardown_state semantics but SKIPS every
-// navigation/page/HUD step — headless core has no UI surface.
-//
-// Params (extension schema, testnet-default): profile, market(s)/symbol(s),
-// positions{state,...}, orders{state,...}, network. The UI-only `page`/`hud`
-// fields are accepted by the schema but intentionally ignored here.
-//
-// Teardown defaults BOTH domains to state=none (leave the shared dev account
-// FLAT) when a baseline block is omitted; pass a block explicitly to override, or
-// `false` to skip a domain. Orders are canceled FIRST, then positions are closed,
-// so resting orders never re-arm against a position mid-teardown.
-
 const BASELINE_KEYS = [
   'market',
   'symbol',

package/library/actions/core/perps/update_position_tpsl.mjs

@@ -1,3 +1,6 @@
+// Manage full or partial TP/SL for a selected testnet position and verify the resulting triggers.
+// Fractional sizes use live venue quantity precision.
+
 import {
   configuredSymbols,
   controllerRejection,
@@ -10,14 +13,6 @@ import {
   selectedItems,
 } from './_controller.mjs';
 
-// core Set or replace the TP/SL attached to an existing Perps POSITION by driving
-// the headless PerpsController.updatePositionTPSL() through the full
-// signing/provider path. This is the position-bound half of the advanced-order
-// contract (TAT-3511): it covers both whole-position TP/SL and PARTIAL
-// (quantity-scoped) TP/SL, which the provider must express as standalone
-// reduce-only trigger orders because a position-bound TP/SL always closes the
-// whole position.
-//
 // Params use the controller's own provider-agnostic names: take_profit_price /
 // take_profit_size / stop_loss_price / stop_loss_size (camelCase aliases
 // accepted). Omitting a price removes that side; omitting a size covers the
@@ -129,9 +124,6 @@ export async function updatePositionTpsl(input) {
   // Prices and sizes are resolved from live mid and the actual position, so a
   // recipe never has to name a number that only holds for one market at one
   // moment. An absolute value still wins when the caller gives one.
-  // Both derivations below need the market, and each used to fetch it
-  // separately. getMarketDataWithPrices reads every market, so doing it twice in
-  // one adapter is what pushed this past the node timeout.
   let market;
   const readMarket = async () => {
     if (market === undefined) {
@@ -181,11 +173,16 @@ export async function updatePositionTpsl(input) {
 
   if (takeProfitFraction !== undefined || stopLossFraction !== undefined) {
     const absolutePositionSize = Math.abs(Number(position.size ?? position.szi ?? 0));
-    // Round to the market's own size precision before submitting. The venue
-    // rounds anyway, and the read-back below matches on size — so an unrounded
-    // fraction asks for a size that can never come back, and the poll spins
-    // until it times out.
-    const szDecimals = Number((await readMarket())?.szDecimals);
+    // MarketInfo carries quantity precision; formatted market prices do not.
+    const markets = await controller.getMarkets({ symbols: [symbol] });
+    const szDecimals = markets.find(
+      (item) => item.name.toUpperCase() === symbol.toUpperCase(),
+    )?.szDecimals;
+    if (!Number.isInteger(szDecimals) || szDecimals < 0) {
+      throw new Error(
+        `core updatePositionTPSL found no valid size precision for ${symbol}.\nNext: run mm-harness verify --adapter core`,
+      );
+    }
     const fromFraction = (raw, name) => {
       const fraction = Number(raw);
       if (!Number.isFinite(fraction) || fraction <= 0 || fraction > 1) {
@@ -194,9 +191,6 @@ export async function updatePositionTpsl(input) {
         );
       }
       const exact = absolutePositionSize * fraction;
-      if (!Number.isFinite(szDecimals)) {
-        return String(exact);
-      }
       const rounded = Number(exact.toFixed(szDecimals));
       if (rounded <= 0) {
         throw new Error(

package/library/actions/core/wallet/list_accounts.mjs

@@ -1,3 +1,6 @@
+// Derive public account identities from the checkout's wallet fixture and apply scope selection.
+// Credential values are not included in the returned account list.
+
 import { readFile, writeFile } from 'node:fs/promises';
 import { pathToFileURL } from 'node:url';
 import { mnemonicToAccount, privateKeyToAccount } from 'viem/accounts';

package/library/actions/extension/analytics/consent.mjs

@@ -1,3 +1,5 @@
+// Builds controller requests to set and verify analytics consent.
+// The fallback reloads the page with a temporary Chrome bridge and removes it afterward.
 export function consentControllerExpression({
   bridgeSymbol,
   participate,

package/library/actions/extension/analytics/set_consent.mjs

@@ -1,3 +1,5 @@
+// Sets MetaMetrics and marketing consent through background controllers and checks settled state.
+// Reloads with a temporary Chrome bridge when the debug request bridge is unavailable.
 import { runAdapter, withExtensionPage } from '../platform/cdp.mjs';
 import { consentParams } from '../../shared/analytics/consent.mjs';
 import {

package/library/actions/extension/assets/home-token-identity.mjs

@@ -1,3 +1,4 @@
+// Opens a Home token to verify its ERC-20 identity from the details route, then returns Home.
 import { openHome } from '../ui/navigate.mjs';
 
 export async function verifyHomeIdentity(page, token, assetId, timeoutMs) {

package/library/actions/extension/assets/import_custom_token.mjs

@@ -1,3 +1,5 @@
+// Imports an ERC-20 through token management and checks its Home row.
+// Reusing a visible token requires a matching details-route identity and an unchanged account.
 import { dataTestId, runAdapter, withExtensionPage } from '../platform/cdp.mjs';
 import { readWalletStateExpression } from '../wallet/state.mjs';
 import { verifyHomeIdentity } from './home-token-identity.mjs';

package/library/actions/extension/assets/open_details.mjs

@@ -1,3 +1,4 @@
+// Opens asset details from a uniquely visible token symbol and waits for the asset route.
 import { runAdapter, withExtensionPage } from '../platform/cdp.mjs';
 
 runAdapter(async (input) => withExtensionPage(input, async (page) => {

package/library/actions/extension/assets/prepare_send.mjs

@@ -1,3 +1,4 @@
+// Fills a token send with a recipient and explicit or Max amount, stopping at enabled Continue.
 import { runAdapter, withExtensionPage } from '../platform/cdp.mjs';
 import { selectors, tokenRow, visible, waitFor } from './send.mjs';
 

package/library/actions/extension/assets/read_details.mjs

@@ -1,3 +1,4 @@
+// Reads asset details from the DOM, rejecting malformed values and waiting for requested chart data.
 import { runAdapter, withExtensionPage } from '../platform/cdp.mjs';
 
 function clean(value) {

package/library/actions/extension/assets/read_visible_state.mjs

@@ -1,3 +1,4 @@
+// Captures token rows within the viewport, including displayed balances and price-change tones.
 import { runAdapter, withExtensionPage } from '../platform/cdp.mjs';
 import {
   normalizeVisibleAssetsState,

package/library/actions/extension/assets/review_send.mjs

@@ -1,3 +1,4 @@
+// Advances a prepared send through alerts to confirmation and checks that a network fee is shown.
 import { runAdapter, withExtensionPage } from '../platform/cdp.mjs';
 import { bodyState, selectors, visible, waitFor } from './send.mjs';
 

package/library/actions/extension/assets/select_network_scope.mjs

@@ -1,3 +1,4 @@
+// Opens Home and selects the all-networks token filter, then checks the label and closed menu.
 import { dataTestId, runAdapter, withExtensionPage } from '../platform/cdp.mjs';
 import { openHome } from '../ui/navigate.mjs';
 import { visible, waitFor } from './send.mjs';

package/library/actions/extension/assets/send.mjs

@@ -1,3 +1,5 @@
+// Shares send controls, UI polling, and transaction-state readers.
+// Token and activity lookup helpers mark matching DOM rows for later clicks.
 import { dataTestId } from '../platform/cdp.mjs';
 
 export async function visible(page, selector) {

package/library/actions/extension/assets/set_custom_gas.mjs

@@ -1,3 +1,4 @@
+// Saves a custom gas limit in send confirmation, supplying a positive priority fee if needed.
 import { dataTestId, runAdapter, withExtensionPage } from '../platform/cdp.mjs';
 import { bodyState, visible, waitFor } from './send.mjs';
 

package/library/actions/extension/assets/set_token_visibility.mjs

@@ -1,3 +1,5 @@
+// Sets a token-management toggle and checks Home visibility, with optional exact ERC-20 identity.
+// Can require a change or an already-matching preference.
 import { dataTestId, runAdapter, withExtensionPage } from '../platform/cdp.mjs';
 import { verifyHomeIdentity } from './home-token-identity.mjs';
 

package/library/actions/extension/assets/verify_sorting.mjs

@@ -1,3 +1,4 @@
+// Checks Home token order under alphabetical and declining-balance sorts, then restores the preference.
 import { dataTestId, runAdapter, withExtensionPage } from '../platform/cdp.mjs';
 
 const TOGGLE = dataTestId('sort-by-popover-toggle');

package/library/actions/extension/deeplink/open.mjs

@@ -1,3 +1,4 @@
+// Navigates the Extension page to an external deeplink URL and returns the navigation result.
 import { pathToFileURL } from 'node:url';
 
 import {

package/library/actions/extension/networks/add_chainlist.mjs

@@ -1,3 +1,5 @@
+// Adds a missing network through ChainList and selects it in the Home network filter.
+// Can reject existing networks so later cleanup does not delete prior configuration.
 import { dataTestId, runAdapter, withExtensionPage } from '../platform/cdp.mjs';
 import {
   assertNetworkAdditionAllowed,

package/library/actions/extension/networks/add_custom.mjs

@@ -1,3 +1,5 @@
+// Adds a missing custom RPC network through the UI and selects it on Home.
+// Can reject existing networks so later cleanup does not delete prior configuration.
 import { dataTestId, runAdapter, withExtensionPage } from '../platform/cdp.mjs';
 import {
   assertNetworkAdditionAllowed,

package/library/actions/extension/networks/custom_network.mjs

@@ -1,3 +1,5 @@
+// Shares network-manager navigation, chain/name matching, and Home network selection.
+// Button lookup marks DOM targets; network additions can reject existing configuration.
 import { dataTestId } from '../platform/cdp.mjs';
 
 async function visible(page, selector) {

package/library/actions/extension/networks/read_visible_state.mjs

@@ -1,3 +1,4 @@
+// Samples the displayed network, token rows, and errors, with an optional settling interval.
 import { runAdapter, withExtensionPage } from '../platform/cdp.mjs';
 import {
   normalizeVisibleNetworkState,

package/library/actions/extension/networks/remove_custom.mjs

@@ -1,3 +1,4 @@
+// Switches Home to Ethereum, deletes the named custom network, and checks its absence in the manager.
 import { dataTestId, runAdapter, withExtensionPage } from '../platform/cdp.mjs';
 import {
   findNetworkItem,

package/library/actions/extension/performance/_navigation-memory.mjs

@@ -1,3 +1,5 @@
+// Runs navigation and idle memory measurements with forced GC, CDP metrics, and heap snapshots.
+// Checks configured growth bounds and writes diagnostic artifacts for failed measurements.
 import { mkdtemp, writeFile } from 'node:fs/promises';
 import path from 'node:path';
 

package/library/actions/extension/performance/compare_idle_navigation_memory.mjs

@@ -1,3 +1,5 @@
+// Compares post-GC listener growth across equal-duration idle and navigation rounds against a configured bound.
+// Writes a diagnostic report when the measurement exceeds that bound.
 import { runAdapter, withExtensionPage } from '../platform/cdp.mjs';
 import {
   compareIdleNavigation,

package/library/actions/extension/performance/measure_detached_dom.mjs

@@ -1,3 +1,5 @@
+// Checks detached-node growth between post-GC heap snapshots before and after repeated screen navigation.
+// Writes a diagnostic report when growth exceeds the configured bound.
 import { runAdapter, withExtensionPage } from '../platform/cdp.mjs';
 import {
   detachedDomBound,

package/library/actions/extension/performance/measure_navigation_memory.mjs

@@ -1,3 +1,5 @@
+// Repeatedly navigates a screen and measures post-GC DOM-node, listener, and heap growth slopes.
+// Enforces node and listener slope bounds and writes a diagnostic report when either fails.
 import { runAdapter, withExtensionPage } from '../platform/cdp.mjs';
 import {
   measureNavigationSlope,

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

@@ -1,3 +1,4 @@
+// Assert that explicitly selected Perps orders are present or absent in live order state.
 import { runAdapter } from '../platform/cdp.mjs';
 import { assertOrders } from './perps.mjs';
 

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

@@ -1,3 +1,4 @@
+// Assert that explicitly selected Perps positions are present or absent in runtime position state.
 import { runAdapter } from '../platform/cdp.mjs';
 import { assertPositions } from './perps.mjs';
 

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

@@ -1,3 +1,5 @@
+// Sample changing Perps UI values or check recent activity containment across filters.
+// Some modes navigate lists, select filters, or open watchlist market details.
 import { pathToFileURL } from 'node:url';
 
 import { runAdapter, withExtensionPage } from '../platform/cdp.mjs';

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

@@ -1,3 +1,4 @@
+// Cancel selected Perps orders through guarded mutation helpers and verify their absence.
 import { runAdapter } from '../platform/cdp.mjs';
 import { assertOrders, closeOrders } from './perps.mjs';
 
@@ -5,5 +6,7 @@ runAdapter(async (input) => {
   const close = await closeOrders(input);
   if (close.proofPath === 'visible-product-controls') return close;
   const assertion = await assertOrders(input, false);
-  return { ...assertion, close };
+  return close.proofPath === 'accepted-order-status'
+    ? { ...assertion, ...close }
+    : { ...assertion, close };
 });

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

@@ -1,3 +1,4 @@
+// Verify selected Perps positions are already absent; reject positions that need live closure.
 import { runAdapter } from '../platform/cdp.mjs';
 import { assertPositions, closePositions } from './perps.mjs';
 

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

@@ -1,3 +1,6 @@
+// Close visible Perps positions with funding checks and receipt-bound identity.
+// Accepted fills require lineage checks; full closes retain their own venue identity.
+
 import { dataTestId, runAdapter, withExtensionPage } from '../platform/cdp.mjs';
 import {
   assertTrustedExternalMutationTarget,
@@ -10,7 +13,7 @@ import {
   trustedExternalMutationPlanIsExact,
   withExternalMutationRecovery,
 } from './mutation-receipt.mjs';
-import { placementReceiptSelection, requireCurrentVisibleFundingBalance, requirePerpsTradingContext } from './perps.mjs';
+import { observeHyperliquidSubmission, placementReceiptSelection, requireCurrentVisibleFundingBalance, requirePerpsTradingContext } from './perps.mjs';
 import {
   absoluteDelta,
   parsePositionDetail,
@@ -58,6 +61,97 @@ async function readVisibleClosePosition(page, market) {
   })()`);
 }
 
+async function readAcceptedFill(page, placement, market, side, visibleSize) {
+  const accepted = placement.record?.acceptance;
+  if (!accepted?.venueOrderId) throw new Error('Position cleanup requires the exact accepted venue order ID.');
+  const deadline = Date.now() + 10_000;
+  let startTime = null;
+  // Explicit end windows read current fills instead of the controller's recent-history cache.
+  while (Date.now() < deadline) {
+    const result = await page.session.call('Runtime.evaluate', {
+      expression: `(async () => {
+        const request = globalThis.stateHooks?.submitRequestToBackground;
+        if (typeof request !== 'function') throw new Error('Read-only Perps fill API is unavailable.');
+        const recent = await request('perpsGetOrderFills', [{ aggregateByTime: false, endTime: ${JSON.stringify(Date.now())} }]);
+        if (!Array.isArray(recent) || recent.some(fill => !fill || typeof fill !== 'object'
+          || Array.isArray(fill) || !['string', 'number'].includes(typeof fill.orderId))) {
+          throw new Error('Read-only Perps fill history is malformed.');
+        }
+        const owned = recent.filter(fill => String(fill?.orderId) === ${JSON.stringify(accepted.venueOrderId)});
+        if (owned.some(fill => !Number.isSafeInteger(fill.timestamp) || fill.timestamp <= 0)) {
+          throw new Error('Accepted order has malformed fill timestamps.');
+        }
+        return owned.length ? Math.min(...owned.map(fill => fill.timestamp)) : null;
+      })()`,
+      awaitPromise: true, returnByValue: true,
+    }, { timeoutMs: Math.max(1, deadline - Date.now()) });
+    if (result.exceptionDetails) {
+      throw new Error(result.exceptionDetails.exception?.description ?? result.exceptionDetails.text ?? 'Perps fill observation failed.');
+    }
+    startTime = result.result?.value;
+    if (startTime !== null) {
+      if (!Number.isSafeInteger(startTime) || startTime <= 0) throw new Error('Accepted fill observation returned an invalid timestamp.');
+      break;
+    }
+    await new Promise((resolve) => setTimeout(resolve, Math.min(250, Math.max(1, deadline - Date.now()))));
+  }
+  if (startTime === null) throw new Error('Accepted order did not reach fill history before the cleanup observation deadline.');
+  const data = await page.evaluate(`(async () => {
+    const request = globalThis.stateHooks?.submitRequestToBackground;
+    if (typeof request !== 'function') throw new Error('Read-only Perps fill API is unavailable.');
+    const startTime = ${JSON.stringify(startTime)};
+    const [fills, orders] = await Promise.all([
+      request('perpsGetOrderFills', [{ aggregateByTime: false, startTime, endTime: ${JSON.stringify(Date.now())} }]),
+      request('perpsGetOpenOrders', []),
+    ]);
+    if (!Array.isArray(fills) || !Array.isArray(orders) || fills.length >= 2000) {
+      throw new Error('Read-only Perps fill coverage is incomplete.');
+    }
+    const market = ${JSON.stringify(market)};
+    return {
+      openOrderCount: orders.filter(order => String(order.symbol ?? order.coin).toUpperCase() === market).length,
+      fills: fills.filter(fill => String(fill.symbol).toUpperCase() === market).map(fill => ({
+        orderId: String(fill.orderId), side: fill.side, size: fill.size,
+        startPosition: fill.startPosition, timestamp: fill.timestamp,
+      })),
+    };
+  })()`);
+  if (data.openOrderCount !== 0) throw new Error('Cancel the exact accepted order remainder before closing its filled position.');
+  const decimal = (value) => {
+    const match = /^(-?)(0|[1-9][0-9]{0,19})(?:\.([0-9]{1,8}))?$/u.exec(String(value));
+    if (!match) throw new Error('Perps fill size identity is invalid.');
+    return BigInt(`${match[1]}${match[2]}${String(match[3] ?? '').padEnd(8, '0')}`);
+  };
+  const owned = data.fills.filter((fill) => fill.orderId === accepted.venueOrderId);
+  if (!owned.length || data.fills.some((fill) => !Number.isSafeInteger(fill.timestamp) || fill.timestamp <= 0)) {
+    throw new Error('Accepted venue order has no independently observed complete fill identity.');
+  }
+  const firstTimestamp = Math.min(...owned.map((fill) => fill.timestamp));
+  if (data.fills.some((fill) => fill.timestamp >= firstTimestamp && fill.orderId !== accepted.venueOrderId)) {
+    throw new Error('Another order changed this market after the accepted fill; refusing shared-position cleanup.');
+  }
+  const sign = side === 'long' ? 1n : -1n;
+  owned.sort((left, right) => {
+    const difference = sign * (decimal(left.startPosition) - decimal(right.startPosition));
+    return difference < 0n ? -1 : difference > 0n ? 1 : 0;
+  });
+  let position = 0n;
+  for (const fill of owned) {
+    const size = decimal(fill.size);
+    if (fill.side !== (side === 'long' ? 'buy' : 'sell') || size <= 0n || decimal(fill.startPosition) !== position) {
+      throw new Error('Accepted order fills do not prove an exclusively owned position starting from zero.');
+    }
+    position += sign * size;
+  }
+  const sizeMatch = /^([0-9.,]+) ([A-Z0-9:_-]{2,20})$/u.exec(visibleSize);
+  if (!sizeMatch || sizeMatch[2] !== market || decimal(sizeMatch[1].replace(/,/gu, '')) !== sign * position
+    || sign * position > decimal(accepted.size)) {
+    throw new Error('Visible position size does not equal the exact accepted-order fills.');
+  }
+  return { venueOrderId: accepted.venueOrderId, fillCount: owned.length, size: visibleSize,
+    firstFillTimestamp: firstTimestamp, lastFillTimestamp: Math.max(...owned.map((fill) => fill.timestamp)) };
+}
+
 async function prepareCloseModal(page, partialAmount) {
   let confirmation;
   for (let attempt = 0; attempt < 3 && !confirmation; attempt += 1) {
@@ -75,6 +169,7 @@ async function prepareCloseModal(page, partialAmount) {
         return {
           amount: String(document.querySelector('[data-testid="close-amount-value"] input')?.value || read('close-amount-value')),
           fullSizeSelected: Boolean(document.querySelector('[data-testid="close-amount-slider-pct-100"]')),
+          marketOrder: !modal.querySelector('[data-testid="limit-price-input"]'),
           margin: read('perps-close-summary-margin-value'),
           fees: read('perps-close-summary-fees-value'),
           receive: read('perps-close-summary-receive-value'),
@@ -129,6 +224,20 @@ async function proveGlobalCleanup(page, balanceBeforeBaseUnits, toleranceBaseUni
   };
 }
 
+function requireCloseAcceptance(receipt, maximumBaseUnits) {
+  const accepted = receipt?.acceptance;
+  const size = receipt?.facts?.closeSizing?.size;
+  if (!accepted?.venueOrderId || accepted.status !== 'filled' || Number(accepted.filledSize) !== Number(size)
+    || accepted.venueOrderId === receipt?.facts?.acceptedFill?.venueOrderId) {
+    throw new Error('Full close has no distinct accepted full-fill identity.');
+  }
+  const notional = Math.ceil(Number(accepted.filledSize) * Number(accepted.averagePrice) * 1_000_000);
+  if (!Number.isSafeInteger(notional) || notional <= 0 || BigInt(notional) > BigInt(maximumBaseUnits)) {
+    throw new Error('Accepted close fill exceeds the trusted operation cap.');
+  }
+  return { venueOrderId: accepted.venueOrderId, filledSize: accepted.filledSize, averagePrice: accepted.averagePrice };
+}
+
 export async function closeVisiblePosition(input) {
   return withExtensionPage(input, async (page) => {
   const market = marketSymbol(input.node?.market);
@@ -176,6 +285,7 @@ export async function closeVisiblePosition(input) {
     roundingToleranceBaseUnits,
     ...(cleanupOnly ? { positionSize: fundingPositionSize, cleanupOnly: true } : {}),
   });
+  const closeBudget = trustedExternalMutationBudget(input, 'perps-close-position');
   const existingReceipt = await readExternalMutationReceipt(input, identity);
   if (existingReceipt) {
     return withExternalMutationRecovery(existingReceipt, async () => {
@@ -187,7 +297,8 @@ export async function closeVisiblePosition(input) {
         dispatch: async () => {
           throw new Error('resumed close must not dispatch');
         },
-        observe: async () => {
+        observe: async ({ receipt }) => {
+          const acceptedClose = partialAmount == null ? requireCloseAcceptance(receipt, closeBudget.maxBaseUnits) : null;
           const deadline = Date.now() + 45_000;
           let finalState;
           let absentSince = null;
@@ -218,6 +329,7 @@ export async function closeVisiblePosition(input) {
                 );
                 return {
                   success: true,
+                  ...acceptedClose,
                   finalState,
                   cleanup,
                 };
@@ -238,6 +350,8 @@ export async function closeVisiblePosition(input) {
         receipt: mutation.receipt,
         resumed: true,
         recovery: mutation.recovery,
+        ...(partialAmount == null ? { venueOrderId: mutation.observation.venueOrderId,
+          filledSize: mutation.observation.filledSize, averagePrice: mutation.observation.averagePrice } : {}),
         finalState: mutation.observation.finalState,
         cleanup: mutation.observation.cleanup ?? null,
         proofPath: 'trusted-pointer-visible-ui',
@@ -275,20 +389,30 @@ export async function closeVisiblePosition(input) {
   const observedNotionalBaseUnits = sizeMatch && sizeMatch[2] === market && priceBaseUnits != null
     ? `${Math.round(Number(sizeMatch[1]) * Number(priceBaseUnits))}`
     : null;
-  const closeBudget = trustedExternalMutationBudget(input, 'perps-close-position');
   if (!/^[1-9][0-9]{0,29}$/u.test(String(observedNotionalBaseUnits ?? ''))
     || BigInt(observedNotionalBaseUnits) > BigInt(closeBudget.maxBaseUnits)) {
     throw new Error('Visible Perps position notional is unavailable or exceeds the authorized close cap.');
   }
-  if (!cleanupOnly) {
-    await readPriorMutationReceipt(input, identity, 'metamask.perps.place_order', placementReceiptSelection(input));
+  const placement = cleanupOnly ? null
+    : await readPriorMutationReceipt(input, identity, 'metamask.perps.place_order', placementReceiptSelection(input));
+  let acceptedFill = null;
+  if (['limit', 'market'].includes(placement?.record?.identity?.orderType)) {
+    if (partialAmount != null) throw new Error('Accepted-fill cleanup requires a full owned-size close.');
+    acceptedFill = await readAcceptedFill(page, placement, market, fundingSide, exactBefore.size);
   }
-  const balanceBeforeBaseUnits = fundingBalance.observedBaseUnits;
+  const balanceBeforeBaseUnits = String(placement?.record?.facts?.balanceBeforeBaseUnits ?? fundingBalance.observedBaseUnits);
   if (!/^(?:0|[1-9][0-9]{0,29})$/u.test(balanceBeforeBaseUnits)) {
     throw new Error('Prior order receipt has no valid visible pre-order balance.');
   }
   const confirmation = await prepareCloseModal(page, partialAmount);
-  const mutation = await runGuardedExternalMutation({
+  if (partialAmount == null && (!confirmation.fullSizeSelected || !confirmation.marketOrder)) {
+    throw new Error('Receipt-bound full close requires the visible 100% market-close form.');
+  }
+  const closeSizing = partialAmount == null ? { size: sizeMatch[1], side: fundingSide === 'long' ? 'short' : 'long' } : null;
+  let submissionObserver = null;
+  let mutation;
+  try {
+    mutation = await runGuardedExternalMutation({
     input,
     identity,
     publicationFacts: {
@@ -298,6 +422,8 @@ export async function closeVisiblePosition(input) {
       cleanupOnly,
       fundingBalance,
       tradingContext,
+      ...(acceptedFill ? { acceptedFill } : {}),
+      ...(closeSizing ? { closeSizing } : {}),
     },
     dispatch: async () => {
       let dispatchTarget;
@@ -312,7 +438,20 @@ export async function closeVisiblePosition(input) {
         }) || currentPosition.size !== exactBefore.size) {
           throw new Error('Visible Perps position changed before close dispatch.');
         }
-        await prepareCloseModal(page, partialAmount);
+        if (acceptedFill && JSON.stringify(await readAcceptedFill(page, placement, market, fundingSide, currentPosition.size))
+          !== JSON.stringify(acceptedFill)) {
+          throw new Error('Accepted-order fill identity changed before close dispatch.');
+        }
+        const currentConfirmation = await prepareCloseModal(page, partialAmount);
+        if (closeSizing) {
+          if (!currentConfirmation.fullSizeSelected || !currentConfirmation.marketOrder) {
+            throw new Error('Full market-close selection changed before dispatch.');
+          }
+          submissionObserver = await observeHyperliquidSubmission(page, { ...tradingContext, market,
+            side: closeSizing.side, size: closeSizing.size, orderType: 'market', reduceOnly: true,
+            leverage, maximumNotionalBaseUnits: closeBudget.maxBaseUnits });
+          await submissionObserver.checkSizing();
+        }
         dispatchTarget = await page.evaluate(`(() => {
           const element = document.querySelector('[data-testid="perps-close-position-modal-submit"]');
           if (!element || element.disabled || element.getAttribute('aria-disabled') === 'true') {
@@ -326,6 +465,7 @@ export async function closeVisiblePosition(input) {
       } catch (error) {
         throw externalMutationNotDispatched(error instanceof Error ? error.message : String(error));
       }
+      submissionObserver?.arm();
       await page.session.call('Input.dispatchMouseEvent', {
         type: 'mousePressed', x: dispatchTarget.x, y: dispatchTarget.y,
         button: 'left', buttons: 1, clickCount: 1,
@@ -334,8 +474,10 @@ export async function closeVisiblePosition(input) {
         type: 'mouseReleased', x: dispatchTarget.x, y: dispatchTarget.y,
         button: 'left', buttons: 0, clickCount: 1,
       });
+      return submissionObserver ? submissionObserver.accepted(timeoutMs) : undefined;
     },
-    observe: async () => {
+    observe: async ({ receipt }) => {
+      const acceptedClose = closeSizing ? requireCloseAcceptance(receipt, closeBudget.maxBaseUnits) : null;
       const closeDeadline = Date.now() + 45_000;
       let finalState;
       while (Date.now() <= closeDeadline) {
@@ -364,6 +506,7 @@ export async function closeVisiblePosition(input) {
             );
             return {
               success: true,
+              ...acceptedClose,
               finalState,
               cleanup,
             };
@@ -373,7 +516,10 @@ export async function closeVisiblePosition(input) {
       }
       throw new Error('matching closed-position postcondition was not visible before timeout');
     },
-  });
+    });
+  } finally {
+    submissionObserver?.close();
+  }
   return {
     action: input.action,
     market,
@@ -387,6 +533,9 @@ export async function closeVisiblePosition(input) {
     receipt: mutation.receipt,
     resumed: mutation.resumed,
     recovery: mutation.recovery,
+    ...(closeSizing ? { venueOrderId: mutation.observation.venueOrderId,
+      filledSize: mutation.observation.filledSize, averagePrice: mutation.observation.averagePrice,
+      closeSizing: submissionObserver?.sizing } : {}),
     proofPath: 'trusted-pointer-visible-ui',
   };
   });

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

@@ -1,3 +1,5 @@
+// Fetch production Hyperliquid data and compare it with the opened market's displayed values.
+// Also check the funding countdown and nonblank chart canvases.
 import { dataTestId, marketSymbol, runAdapter, withExtensionPage } from '../platform/cdp.mjs';
 
 function numeric(value) {

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

@@ -1,3 +1,5 @@
+// Submit live margin changes through visible controls with mutation receipts and position identity checks.
+// Observe the resulting margin and balance; removal requires a prior successful add receipt.
 import { dataTestId, runAdapter, withExtensionPage } from '../platform/cdp.mjs';
 import {
   assertTrustedExternalMutationTarget,

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

@@ -1,3 +1,4 @@
+// Require selected Perps orders to match the requested state without placing or cancelling orders.
 import { runAdapter } from '../platform/cdp.mjs';
 import { ensureOrders } from './perps.mjs';
 

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

@@ -1,3 +1,4 @@
+// Require selected Perps positions to match the requested state without opening or closing positions.
 import { runAdapter } from '../platform/cdp.mjs';
 import { ensurePositions } from './perps.mjs';