@metamask-previews/transaction-controller 49.0.0-preview-4a9f2de → 49.0.0-preview-0838a01

package/CHANGELOG.md

@@ -19,6 +19,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - **BREAKING:** Add additional metadata for batch metrics ([#5488](https://github.com/MetaMask/core/pull/5488))
   - Change `error` in `TransactionMetadata` to optional for all statuses.
   - Change `nestedTransactions` in `TransactionMetadata` to array of `NestedTransactionMetadata`.
+- Throw if `addTransactionBatch` called with external origin and size limit exceeded ([#5489](https://github.com/MetaMask/core/pull/5489))
 
 ## [49.0.0]
 

package/dist/TransactionController.cjs

@@ -371,6 +371,10 @@ class TransactionController extends base_controller_1.BaseController {
         if (!__classPrivateFieldGet(this, _TransactionController_multichainTrackingHelper, "f").has(networkClientId)) {
             throw new Error(`Network client not found - ${networkClientId}`);
         }
+        const chainId = __classPrivateFieldGet(this, _TransactionController_instances, "m", _TransactionController_getChainId).call(this, networkClientId);
+        const ethQuery = __classPrivateFieldGet(this, _TransactionController_instances, "m", _TransactionController_getEthQuery).call(this, {
+            networkClientId,
+        });
         const permittedAddresses = origin === undefined
             ? undefined
             : await this.getPermittedAccounts?.(origin);
@@ -386,6 +390,7 @@ class TransactionController extends base_controller_1.BaseController {
             txParams,
             type,
         });
+        const delegationAddressPromise = (0, eip7702_1.getDelegationAddress)(txParams.from, ethQuery).catch(() => undefined);
         const isEIP1559Compatible = await this.getEIP1559Compatibility(networkClientId);
         (0, validation_1.validateTxParams)(txParams, isEIP1559Compatible);
         const isDuplicateBatchId = batchId?.length &&
@@ -394,18 +399,8 @@ class TransactionController extends base_controller_1.BaseController {
             throw rpc_errors_1.rpcErrors.invalidInput('Batch ID already exists');
         }
         const dappSuggestedGasFees = this.generateDappSuggestedGasFees(txParams, origin);
-        const chainId = __classPrivateFieldGet(this, _TransactionController_instances, "m", _TransactionController_getChainId).call(this, networkClientId);
-        const ethQuery = __classPrivateFieldGet(this, _TransactionController_instances, "m", _TransactionController_getEthQuery).call(this, {
-            networkClientId,
-        });
         const transactionType = type ?? (await (0, transaction_type_1.determineTransactionType)(txParams, ethQuery)).type;
-        let delegationAddress;
-        try {
-            delegationAddress = await (0, eip7702_1.getDelegationAddress)(txParams.from, ethQuery);
-        }
-        catch (error) {
-            (0, logger_1.projectLogger)('Error checking if account is upgraded to EIP-7702', error);
-        }
+        const delegationAddress = await delegationAddressPromise;
         const existingTransactionMeta = this.getTransactionWithActionId(actionId);
         // If a request to add a transaction with the same actionId is submitted again, a new transaction will not be created for it.
         let addedTransactionMeta = existingTransactionMeta