chore: rename

rename

rename

wip

Author: micaelae

packages/bridge-status-controller/src/bridge-status-controller.ts

@@ -1082,7 +1082,11 @@ export class BridgeStatusController extends StaticIntervalPollingController<Brid
       action_type: MetricsActionType.SWAPBRIDGE_V1,
       location:
         eventProperties?.location ??
-        (txMetaId ? this.state.txHistory?.[txMetaId]?.location : undefined) ??
+        (txMetaId
+          ? // comseyhil jfildsjfkl jilfd
+            this.state.txHistory?.[txMetaId]?.location
+          : // comseyhil jfildsjfkl jilfd
+            undefined) ??
         MetaMetricsSwapsEventSource.MainView,
       ...(eventProperties ?? {}),
       ...(resolvedAbTests &&

packages/bridge-status-controller/src/strategy/batch-strategy.ts

@@ -1,4 +1,4 @@
-import { isEvmTxData } from '@metamask/bridge-controller';
+import { TxData } from '@metamask/bridge-controller';
 
 import type { SubmitStrategyParams, SubmitStepResult } from './types';
 import {
@@ -13,7 +13,7 @@ import {
  * @yields The approvalMeta and tradeMeta for the batched transaction
  */
 export async function* submitBatchHandler(
-  args: SubmitStrategyParams,
+  args: SubmitStrategyParams<TxData>,
 ): AsyncGenerator<SubmitStepResult, void, void> {
   const {
     requireApproval,
@@ -22,19 +22,11 @@ export async function* submitBatchHandler(
     isBridgeTx,
     addTransactionBatchFn,
   } = args;
-  if (!isEvmTxData(quoteResponse.trade)) {
-    throw new Error(
-      'Failed to submit cross-chain swap transaction: trade is not an EVM transaction',
-    );
-  }
   const transactionParams = await getAddTransactionBatchParams({
     messenger,
     isBridgeTx,
     resetApproval: quoteResponse.resetApproval,
-    approval:
-      quoteResponse.approval && isEvmTxData(quoteResponse.approval)
-        ? quoteResponse.approval
-        : undefined,
+    approval: quoteResponse.approval,
     trade: quoteResponse.trade,
     quoteResponse,
     requireApproval,

packages/bridge-status-controller/src/strategy/index.ts

@@ -1,6 +1,6 @@
 /* eslint-disable @typescript-eslint/prefer-nullish-coalescing */
 /* eslint-disable @typescript-eslint/explicit-function-return-type */
-import { isNonEvmChainId } from '@metamask/bridge-controller';
+import { isEvmTxData, isNonEvmChainId } from '@metamask/bridge-controller';
 
 import { submitBatchHandler } from './batch-strategy';
 import { submitEvmHandler as defaultSubmitHandler } from './evm-strategy';
@@ -25,9 +25,11 @@ const SUBMIT_STRATEGY_REGISTRY: SubmitStrategy[] = [
       const { quoteResponse, isStxEnabledOnClient, isDelegatedAccount } =
         params;
       return (
-        isStxEnabledOnClient ||
-        quoteResponse.quote.gasIncluded7702 ||
-        isDelegatedAccount
+        (isStxEnabledOnClient ||
+          quoteResponse.quote.gasIncluded7702 ||
+          isDelegatedAccount) &&
+        isEvmTxData(quoteResponse.trade) &&
+        (quoteResponse.approval ? isEvmTxData(quoteResponse.approval) : true)
       );
     },
     execute: submitBatchHandler,

packages/bridge-status-controller/src/strategy/types.ts

@@ -3,6 +3,7 @@ import type {
   BridgeClientId,
   QuoteMetadata,
   QuoteResponse,
+  Trade,
 } from '@metamask/bridge-controller';
 import type { TraceCallback } from '@metamask/controller-utils';
 import type {
@@ -55,13 +56,13 @@ export type SubmitStepResult =
 /**
  * The parameters for the submission flow
  */
-export type SubmitStrategyParams = {
+export type SubmitStrategyParams<TradeType extends Trade = Trade> = {
   addTransactionBatchFn: TransactionController['addTransactionBatch'];
   isBridgeTx: boolean;
   isDelegatedAccount: boolean;
   isStxEnabledOnClient: boolean;
   messenger: BridgeStatusControllerMessenger;
-  quoteResponse: QuoteResponse & QuoteMetadata;
+  quoteResponse: QuoteResponse<TradeType, TradeType> & QuoteMetadata;
   requireApproval: boolean;
   selectedAccount: AccountsControllerState['internalAccounts']['accounts'][string];
   traceFn: TraceCallback;