@metamask/transaction-controller 52.2.0 → 52.3.0

package/CHANGELOG.md

@@ -7,6 +7,21 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [52.3.0]
+
+### Added
+
+- Adds `RandomisedEstimationsGasFeeFlow` to gas fee flows in `TransactionController` ([#5511](https://github.com/MetaMask/core/pull/5511))
+  - Added flow only will be activated if chainId is defined in feature flags.
+- Configure pending transaction polling intervals using remote feature flags ([#5549](https://github.com/MetaMask/core/pull/5549))
+
+### Fixed
+
+- Fix EIP-7702 contract signature validation on chains with odd-length hexadecimal ID ([#5563](https://github.com/MetaMask/core/pull/5563))
+- Fix simulation of type-4 transactions ([#5552](https://github.com/MetaMask/core/pull/5552))
+- Display incoming transactions in active tab ([#5487](https://github.com/MetaMask/core/pull/5487))
+- Fix bug in `updateTransactionGasFees` affecting `txParams` gas updates when `enableTxParamsGasFeeUpdates` is enabled. ([#5539](https://github.com/MetaMask/core/pull/5539))
+
 ## [52.2.0]
 
 ### Added
@@ -1443,7 +1458,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
     All changes listed after this point were applied to this package following the monorepo conversion.
 
-[Unreleased]: https://github.com/MetaMask/core/compare/@metamask/transaction-controller@52.2.0...HEAD
+[Unreleased]: https://github.com/MetaMask/core/compare/@metamask/transaction-controller@52.3.0...HEAD
+[52.3.0]: https://github.com/MetaMask/core/compare/@metamask/transaction-controller@52.2.0...@metamask/transaction-controller@52.3.0
 [52.2.0]: https://github.com/MetaMask/core/compare/@metamask/transaction-controller@52.1.0...@metamask/transaction-controller@52.2.0
 [52.1.0]: https://github.com/MetaMask/core/compare/@metamask/transaction-controller@52.0.0...@metamask/transaction-controller@52.1.0
 [52.0.0]: https://github.com/MetaMask/core/compare/@metamask/transaction-controller@51.0.0...@metamask/transaction-controller@52.0.0

package/dist/TransactionController.cjs

@@ -32,6 +32,7 @@ const accounts_api_1 = require("./api/accounts-api.cjs");
 const DefaultGasFeeFlow_1 = require("./gas-flows/DefaultGasFeeFlow.cjs");
 const LineaGasFeeFlow_1 = require("./gas-flows/LineaGasFeeFlow.cjs");
 const OptimismLayer1GasFeeFlow_1 = require("./gas-flows/OptimismLayer1GasFeeFlow.cjs");
+const RandomisedEstimationsGasFeeFlow_1 = require("./gas-flows/RandomisedEstimationsGasFeeFlow.cjs");
 const ScrollLayer1GasFeeFlow_1 = require("./gas-flows/ScrollLayer1GasFeeFlow.cjs");
 const TestGasFeeFlow_1 = require("./gas-flows/TestGasFeeFlow.cjs");
 const AccountsApiRemoteTransactionSource_1 = require("./helpers/AccountsApiRemoteTransactionSource.cjs");
@@ -248,6 +249,7 @@ class TransactionController extends base_controller_1.BaseController {
             getProvider: (networkClientId) => __classPrivateFieldGet(this, _TransactionController_instances, "m", _TransactionController_getProvider).call(this, { networkClientId }),
             getTransactions: () => this.state.transactions,
             layer1GasFeeFlows: this.layer1GasFeeFlows,
+            messenger: this.messagingSystem,
             onStateChange: (listener) => {
                 this.messagingSystem.subscribe('TransactionController:stateChange', listener);
             },
@@ -398,6 +400,10 @@ class TransactionController extends base_controller_1.BaseController {
         const delegationAddressPromise = (0, eip7702_1.getDelegationAddress)(txParams.from, ethQuery).catch(() => undefined);
         const isEIP1559Compatible = await this.getEIP1559Compatibility(networkClientId);
         (0, validation_1.validateTxParams)(txParams, isEIP1559Compatible);
+        if (!txParams.type) {
+            // Determine transaction type based on transaction parameters and network compatibility
+            (0, utils_2.setEnvelopeType)(txParams, isEIP1559Compatible);
+        }
         const isDuplicateBatchId = batchId?.length &&
             this.state.transactions.some((tx) => tx.batchId?.toLowerCase() === batchId?.toLowerCase());
         if (isDuplicateBatchId && origin && origin !== controller_utils_1.ORIGIN_METAMASK) {
@@ -838,6 +844,7 @@ class TransactionController extends base_controller_1.BaseController {
         updatedTransaction.type = type;
         await (0, layer1_gas_fee_flow_1.updateTransactionLayer1GasFee)({
             layer1GasFeeFlows: this.layer1GasFeeFlows,
+            messenger: this.messagingSystem,
             provider,
             transactionMeta: updatedTransaction,
         });
@@ -1050,7 +1057,7 @@ class TransactionController extends base_controller_1.BaseController {
             networkClientId,
         };
         // Guaranteed as the default gas fee flow matches all transactions.
-        const gasFeeFlow = (0, gas_flow_1.getGasFeeFlow)(transactionMeta, this.gasFeeFlows);
+        const gasFeeFlow = (0, gas_flow_1.getGasFeeFlow)(transactionMeta, this.gasFeeFlows, this.messagingSystem);
         const ethQuery = new eth_query_1.default(provider);
         const gasFeeControllerData = await this.getGasFeeEstimates({
             networkClientId,
@@ -1058,6 +1065,7 @@ class TransactionController extends base_controller_1.BaseController {
         return gasFeeFlow.getGasFees({
             ethQuery,
             gasFeeControllerData,
+            messenger: this.messagingSystem,
             transactionMeta,
         });
     }
@@ -1077,6 +1085,7 @@ class TransactionController extends base_controller_1.BaseController {
         });
         return await (0, layer1_gas_fee_flow_1.getTransactionLayer1GasFee)({
             layer1GasFeeFlows: this.layer1GasFeeFlows,
+            messenger: this.messagingSystem,
             provider,
             transactionMeta: {
                 txParams: transactionParams,
@@ -1235,10 +1244,12 @@ class TransactionController extends base_controller_1.BaseController {
             gasFeeFlows: this.gasFeeFlows,
             getGasFeeEstimates: this.getGasFeeEstimates,
             getSavedGasFees: this.getSavedGasFees.bind(this),
+            messenger: this.messagingSystem,
             txMeta: transactionMeta,
         }));
         await __classPrivateFieldGet(this, _TransactionController_trace, "f").call(this, { name: 'Update Layer 1 Gas Fees', parentContext: traceContext }, async () => await (0, layer1_gas_fee_flow_1.updateTransactionLayer1GasFee)({
             layer1GasFeeFlows: this.layer1GasFeeFlows,
+            messenger: this.messagingSystem,
             provider,
             transactionMeta,
         }));
@@ -1949,6 +1960,7 @@ _TransactionController_internalEvents = new WeakMap(), _TransactionController_me
         getGlobalLock: () => __classPrivateFieldGet(this, _TransactionController_multichainTrackingHelper, "f").acquireNonceLockForChainIdKey({
             chainId,
         }),
+        messenger: this.messagingSystem,
         publishTransaction: (_ethQuery, transactionMeta) => this.publishTransaction(_ethQuery, transactionMeta, {
             skipSubmitHistory: true,
         }),
@@ -1985,7 +1997,11 @@ _TransactionController_internalEvents = new WeakMap(), _TransactionController_me
     if (__classPrivateFieldGet(this, _TransactionController_testGasFeeFlows, "f")) {
         return [new TestGasFeeFlow_1.TestGasFeeFlow()];
     }
-    return [new LineaGasFeeFlow_1.LineaGasFeeFlow(), new DefaultGasFeeFlow_1.DefaultGasFeeFlow()];
+    return [
+        new RandomisedEstimationsGasFeeFlow_1.RandomisedEstimationsGasFeeFlow(),
+        new LineaGasFeeFlow_1.LineaGasFeeFlow(),
+        new DefaultGasFeeFlow_1.DefaultGasFeeFlow(),
+    ];
 }, _TransactionController_getLayer1GasFeeFlows = function _TransactionController_getLayer1GasFeeFlows() {
     return [new OptimismLayer1GasFeeFlow_1.OptimismLayer1GasFeeFlow(), new ScrollLayer1GasFeeFlow_1.ScrollLayer1GasFeeFlow()];
 }, _TransactionController_updateTransactionInternal = function _TransactionController_updateTransactionInternal({ transactionId, note, skipHistory, skipValidation, skipResimulateCheck, }, callback) {
@@ -2075,6 +2091,9 @@ _TransactionController_internalEvents = new WeakMap(), _TransactionController_me
     };
     let gasFeeTokens = [];
     if (__classPrivateFieldGet(this, _TransactionController_isSimulationEnabled, "f").call(this)) {
+        const authorizationAddress = txParams?.authorizationList?.[0]?.address;
+        const senderCode = authorizationAddress &&
+            (eip7702_1.DELEGATION_PREFIX + (0, utils_1.remove0x)(authorizationAddress));
         const result = await __classPrivateFieldGet(this, _TransactionController_trace, "f").call(this, { name: 'Simulate', parentContext: traceContext }, () => (0, simulation_1.getSimulationData)({
             chainId,
             from: from,
@@ -2083,6 +2102,7 @@ _TransactionController_internalEvents = new WeakMap(), _TransactionController_me
             data: data,
         }, {
             blockTime,
+            senderCode,
         }));
         gasFeeTokens = result?.gasFeeTokens;
         simulationData = result?.simulationData;
@@ -2112,12 +2132,10 @@ _TransactionController_internalEvents = new WeakMap(), _TransactionController_me
     (0, logger_1.projectLogger)('Updated simulation data', transactionId, simulationData);
 }, _TransactionController_onGasFeePollerTransactionUpdate = function _TransactionController_onGasFeePollerTransactionUpdate({ transactionId, gasFeeEstimates, gasFeeEstimatesLoaded, layer1GasFee, }) {
     __classPrivateFieldGet(this, _TransactionController_instances, "m", _TransactionController_updateTransactionInternal).call(this, { transactionId, skipHistory: true }, (txMeta) => {
-        // eslint-disable-next-line @typescript-eslint/no-floating-promises
         (0, GasFeePoller_1.updateTransactionGasFees)({
             txMeta,
             gasFeeEstimates,
             gasFeeEstimatesLoaded,
-            getEIP1559Compatibility: this.getEIP1559Compatibility.bind(this),
             isTxParamsGasFeeUpdatesEnabled: this.isTxParamsGasFeeUpdatesEnabled,
             layer1GasFee,
         });