@deeeed/metamask-harness 0.47.4 → 0.49.0

package/library/actions/extension/perps/mutation-receipt.mjs

@@ -1,3 +1,6 @@
+// Persist trusted mutation intent, accepted order identities and observed outcomes.
+// Resume observation without redispatching an already recorded operation.
+
 import { createHash } from 'node:crypto';
 import { constants as fsConstants } from 'node:fs';
 import {
@@ -461,7 +464,8 @@ function validateIdentity(input, identity) {
     || !/^(?:market|limit|close|margin|tpsl)$/u.test(value.orderType)
     || !/^(?:new|modify|close)$/u.test(value.positionMode)
     || !/^(?:0|[1-9][0-9]{0,29})$/u.test(value.roundingToleranceBaseUnits)
-    || (value.limitPrice !== null && !/^[1-9][0-9]*(?:\.[0-9]{1,6})?$/u.test(value.limitPrice))
+    || (value.limitPrice !== null && (!/^(?:0|[1-9][0-9]*)(?:\.[0-9]{1,6})?$/u.test(value.limitPrice)
+      || !(Number(value.limitPrice) > 0)))
     || (value.offsetPct !== null && !/^-?[0-9]+(?:\.[0-9]{1,6})?$/u.test(value.offsetPct))
     || (value.partialCloseAmount !== null && !/^[1-9][0-9]*(?:\.[0-9]{1,6})?$/u.test(value.partialCloseAmount))) {
     throw new Error('External mutation receipt operation parameters are invalid.');
@@ -738,8 +742,54 @@ export async function recordExternalMutationCleanupCandidate(input, receipt, can
   });
 }
 
+async function recordSubmissionAcceptance(input, receipt, acceptance) {
+  const { venueOrderId, status, requestId, serviceWorkerTargetId, asset, size, limitPriceBaseUnits, filledSize, averagePrice } = acceptance;
+  const closing = receipt.record.identity.operation === 'perps-close-position';
+  const market = closing || receipt.record.identity.operation === 'perps-market-order';
+  const sizing = closing ? receipt.record.facts?.closeSizing
+    : market ? receipt.record.facts?.marketSizing : receipt.record.facts?.limitSizing;
+  if (!['perps-limit-order', 'perps-market-order', 'perps-close-position'].includes(receipt.record.identity.operation)
+    || receipt.record.identity.provider !== 'hyperliquid'
+    || Number(sizing?.size) !== Number(size)
+    || (closing && sizing?.side !== (receipt.record.identity.side === 'long' ? 'short' : 'long'))
+    || !/^[1-9][0-9]{0,19}$/u.test(String(venueOrderId)) || !['resting', 'filled'].includes(status)
+    || !/^[A-Za-z0-9._:-]{1,128}$/u.test(String(requestId))
+    || !/^[A-Za-z0-9._:-]{1,128}$/u.test(String(serviceWorkerTargetId))
+    || !Number.isInteger(asset) || asset < 0 || asset > 100_000
+    || !/^(?:0|[1-9][0-9]{0,19})(?:\.[0-9]{1,8})?$/u.test(String(size)) || !(Number(size) > 0)
+    || !BASE_UNITS.test(String(limitPriceBaseUnits))) {
+    throw recoveryError(receipt, 'submission acceptance has invalid order identity');
+  }
+  if (market && status === 'filled' && (!/^(?:0|[1-9][0-9]{0,19})(?:\.[0-9]{1,8})?$/u.test(String(filledSize))
+    || !(Number(filledSize) > 0) || Number(filledSize) > Number(size)
+    || !/^(?:0|[1-9][0-9]{0,19})(?:\.[0-9]{1,18})?$/u.test(String(averagePrice)) || !(Number(averagePrice) > 0))) {
+    throw recoveryError(receipt, 'market acceptance has invalid filled size or price');
+  }
+  const destination = await receiptDestination(input);
+  const current = await readPublished(destination);
+  if (!current || current.sha256 !== receipt.sha256 || current.record?.state !== 'outcome-unknown') {
+    throw recoveryError(receipt, 'the receipt changed before submission acceptance was recorded');
+  }
+  const record = { ...current.record, acceptance: {
+    venueOrderId, status, requestId, serviceWorkerTargetId, asset, size, limitPriceBaseUnits,
+    ...(market && status === 'filled' ? { filledSize, averagePrice } : {}),
+  } };
+  const staged = await stagedRecord(destination.directory, record);
+  try {
+    await rename(staged.stagedPath, destination.absolute);
+    await durableDirectorySync(destination.directory);
+  } finally {
+    await rm(staged.stagingDirectory, { recursive: true, force: true });
+  }
+  const published = await readPublished(destination);
+  if (!published || canonical(published.record) !== canonical(record)) {
+    throw recoveryError(receipt, 'submission acceptance could not be reread');
+  }
+  return { ...published, resumed: receipt.resumed };
+}
+
 export async function runGuardedExternalMutation({ input, identity, dispatch, observe, publicationFacts, receiptIo }) {
-  const receipt = await beginExternalMutation(input, identity, receiptIo, publicationFacts);
+  let receipt = await beginExternalMutation(input, identity, receiptIo, publicationFacts);
   if (receipt.record?.state === 'observed-policy-violation') {
     throw new Error('External mutation receipt records an observed policy violation and is cleanup-only.');
   }
@@ -748,7 +798,8 @@ export async function runGuardedExternalMutation({ input, identity, dispatch, ob
   }
   if (!receipt.resumed) {
     try {
-      await dispatch();
+      const acceptance = await dispatch();
+      if (acceptance?.venueOrderId) receipt = await recordSubmissionAcceptance(input, receipt, acceptance);
     } catch (error) {
       if (error instanceof ExternalMutationNotDispatchedError) {
         await markExternalMutationNotDispatched(input, receipt, error, receiptIo);

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

@@ -1,3 +1,4 @@
+// Open the Perps deposit or withdrawal flow through balance controls and report the displayed page.
 import { dataTestId, runAdapter, withExtensionPage } from '../platform/cdp.mjs';
 
 const ACTIONS = new Map([
@@ -11,13 +12,22 @@ runAdapter(async (input) => withExtensionPage(input, async (page) => {
   if (!actionTestId) {
     throw new Error(`metamask.perps.open_balance_action received unsupported balance_action ${JSON.stringify(balanceAction)}.`);
   }
-  await page.click(dataTestId('perps-balance-dropdown-balance'));
-  const actionSelector = dataTestId(actionTestId);
-  await page.waitForSelector(actionSelector, { timeoutMs: 10_000 });
+  const directSelector = dataTestId(`perps-balance-actions-${balanceAction}`);
+  const directVisible = await page.evaluate(`(() => {
+    const element = document.querySelector(${JSON.stringify(directSelector)});
+    if (!element) return false;
+    const style = getComputedStyle(element), rect = element.getBoundingClientRect();
+    return style.display !== 'none' && style.visibility !== 'hidden' && rect.width > 0 && rect.height > 0;
+  })()`);
+  const actionSelector = directVisible ? directSelector : dataTestId(actionTestId);
+  if (!directVisible) {
+    await page.click(dataTestId('perps-balance-dropdown-balance'));
+    await page.waitForSelector(actionSelector, { timeoutMs: 10_000 });
+  }
   await page.click(actionSelector);
   const expected = balanceAction === 'withdraw'
-    ? ['perps-withdraw-page', 'perps-withdraw-container', 'perps-withdraw-view']
-    : ['confirm-page-container-content', 'confirm-page-container', 'transaction-confirmation'];
+    ? ['parent-selector-perps-withdraw', 'parent-selector-confirmation-page', 'perps-withdraw-page', 'perps-withdraw-container', 'perps-withdraw-view']
+    : ['parent-selector-confirmation-page', 'confirm-page-container-content', 'confirm-page-container', 'transaction-confirmation'];
   const resultDeadline = Date.now() + 15_000;
   let observed;
   while (Date.now() <= resultDeadline) {
@@ -38,10 +48,10 @@ runAdapter(async (input) => withExtensionPage(input, async (page) => {
         text: String(document.body?.innerText || '').replace(/\\s+/gu, ' ').trim().slice(0, 10_000),
       };
     })()`);
-    if (observed.matched.length > 0 || observed.route !== '#/perps-home') break;
+    if (observed.matched.length > 0) break;
     await new Promise((resolve) => setTimeout(resolve, 100));
   }
-  if (!observed || (observed.matched.length === 0 && observed.route === '#/perps-home')) {
+  if (!observed || observed.matched.length === 0) {
     throw new Error(`Perps ${balanceAction} did not open: ${JSON.stringify(observed)}.`);
   }
   return {

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

@@ -1,3 +1,4 @@
+// Navigate to a market position and open its requested modification form without submitting it.
 import { dataTestId, marketSymbol, runAdapter, withExtensionPage } from '../platform/cdp.mjs';
 
 const ACTIONS = new Map([