@metamask-previews/transaction-controller 57.3.0-preview-3636a128 → 58.0.0-preview-c6e5eb7

package/CHANGELOG.md

@@ -9,6 +9,28 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Added
 
+- Support `containerTypes` property in `updateEditableParams` method ([#6014](https://github.com/MetaMask/core/pull/6014))
+- Add specific transaction types to outgoing transactions retrieved from accounts API ([#5987](https://github.com/MetaMask/core/pull/5987))
+  - Add optional `amount` property to `transferInformation` object in `TransactionMeta` type.
+
+### Changed
+
+- Query only latest page of transactions from accounts API ([#5983](https://github.com/MetaMask/core/pull/5983))
+- Remove incoming transactions when calling `wipeTransactions` ([#5986](https://github.com/MetaMask/core/pull/5986))
+- Poll immediately when calling `startIncomingTransactionPolling` ([#5986](https://github.com/MetaMask/core/pull/5986))
+
+## [58.0.0]
+
+### Changed
+
+- **BREAKING:** Bump peer dependency `@metamask/accounts-controller` to `^31.0.0` ([#5999](https://github.com/MetaMask/core/pull/5999))
+- **BREAKING:** Bump peer dependency `@metamask/gas-fee-controller` to `^24.0.0` ([#5999](https://github.com/MetaMask/core/pull/5999))
+- **BREAKING:** Bump peer dependency `@metamask/network-controller` to `^24.0.0` ([#5999](https://github.com/MetaMask/core/pull/5999))
+
+## [57.4.0]
+
+### Added
+
 - Add optional `afterSimulate` and `beforeSign` hooks to constructor ([#5503](https://github.com/MetaMask/core/pull/5503))
   - Add `AfterSimulateHook` type.
   - Add `BeforeSignHook` type.
@@ -18,6 +40,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
   - Add optional `ignoreDelegationSignatures` boolean to `estimateGas` method.
 - Add `gasFeeEstimates` property to `TransactionBatchMeta`, populated using `DefaultGasFeeFlow` ([#5886](https://github.com/MetaMask/core/pull/5886))
 
+### Fixed
+
+- Handle unknown chain IDs on incoming transactions ([#5985](https://github.com/MetaMask/core/pull/5985))
+
 ## [57.3.0]
 
 ### Added
@@ -1679,7 +1705,9 @@ 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@57.3.0...HEAD
+[Unreleased]: https://github.com/MetaMask/core/compare/@metamask/transaction-controller@58.0.0...HEAD
+[58.0.0]: https://github.com/MetaMask/core/compare/@metamask/transaction-controller@57.4.0...@metamask/transaction-controller@58.0.0
+[57.4.0]: https://github.com/MetaMask/core/compare/@metamask/transaction-controller@57.3.0...@metamask/transaction-controller@57.4.0
 [57.3.0]: https://github.com/MetaMask/core/compare/@metamask/transaction-controller@57.2.0...@metamask/transaction-controller@57.3.0
 [57.2.0]: https://github.com/MetaMask/core/compare/@metamask/transaction-controller@57.1.0...@metamask/transaction-controller@57.2.0
 [57.1.0]: https://github.com/MetaMask/core/compare/@metamask/transaction-controller@57.0.0...@metamask/transaction-controller@57.1.0

package/dist/TransactionController.cjs

@@ -263,23 +263,15 @@ class TransactionController extends base_controller_1.BaseController {
                 _state.methodData[fourBytePrefix] = methodData;
             });
         });
-        const updateCache = (fn) => {
-            this.update((_state) => {
-                fn(_state.lastFetchedBlockNumbers);
-            });
-        };
         __classPrivateFieldSet(this, _TransactionController_incomingTransactionHelper, new IncomingTransactionHelper_1.IncomingTransactionHelper({
             client: __classPrivateFieldGet(this, _TransactionController_incomingTransactionOptions, "f").client,
-            getCache: () => this.state.lastFetchedBlockNumbers,
             getCurrentAccount: () => __classPrivateFieldGet(this, _TransactionController_instances, "m", _TransactionController_getSelectedAccount).call(this),
             getLocalTransactions: () => this.state.transactions,
             includeTokenTransfers: __classPrivateFieldGet(this, _TransactionController_incomingTransactionOptions, "f").includeTokenTransfers,
             isEnabled: __classPrivateFieldGet(this, _TransactionController_incomingTransactionOptions, "f").isEnabled,
             messenger: this.messagingSystem,
-            queryEntireHistory: __classPrivateFieldGet(this, _TransactionController_incomingTransactionOptions, "f").queryEntireHistory,
             remoteTransactionSource: new AccountsApiRemoteTransactionSource_1.AccountsApiRemoteTransactionSource(),
             trimTransactions: __classPrivateFieldGet(this, _TransactionController_instances, "m", _TransactionController_trimTransactionsForState).bind(this),
-            updateCache,
             updateTransactions: __classPrivateFieldGet(this, _TransactionController_incomingTransactionOptions, "f").updateTransactions,
         }), "f");
         __classPrivateFieldGet(this, _TransactionController_instances, "m", _TransactionController_addIncomingTransactionHelperListeners).call(this, __classPrivateFieldGet(this, _TransactionController_incomingTransactionHelper, "f"));
@@ -672,12 +664,15 @@ class TransactionController extends base_controller_1.BaseController {
             });
             return;
         }
-        const newTransactions = this.state.transactions.filter(({ chainId: txChainId, txParams }) => {
+        const newTransactions = this.state.transactions.filter(({ chainId: txChainId, txParams, type }) => {
             const isMatchingNetwork = !chainId || chainId === txChainId;
             if (!isMatchingNetwork) {
                 return true;
             }
-            const isMatchingAddress = !address || txParams.from?.toLowerCase() === address.toLowerCase();
+            const isMatchingAddress = !address ||
+                txParams.from?.toLowerCase() === address.toLowerCase() ||
+                (type === types_1.TransactionType.incoming &&
+                    txParams.to?.toLowerCase() === address.toLowerCase());
             return !isMatchingAddress;
         });
         this.update((state) => {
@@ -869,6 +864,7 @@ class TransactionController extends base_controller_1.BaseController {
      *
      * @param txId - The ID of the transaction to update.
      * @param params - The editable parameters to update.
+     * @param params.containerTypes - Container types applied to the parameters.
      * @param params.data - Data to pass with the transaction.
      * @param params.from - Address to send the transaction from.
      * @param params.gas - Maximum number of units of gas to use for the transaction.
@@ -879,7 +875,7 @@ class TransactionController extends base_controller_1.BaseController {
      * @param params.value - Value associated with the transaction.
      * @returns The updated transaction metadata.
      */
-    async updateEditableParams(txId, { data, from, gas, gasPrice, maxFeePerGas, maxPriorityFeePerGas, to, value, }) {
+    async updateEditableParams(txId, { containerTypes, data, from, gas, gasPrice, maxFeePerGas, maxPriorityFeePerGas, to, value, }) {
         const transactionMeta = __classPrivateFieldGet(this, _TransactionController_instances, "m", _TransactionController_getTransaction).call(this, txId);
         if (!transactionMeta) {
             throw new Error(`Cannot update editable params as no transaction metadata found`);
@@ -904,6 +900,9 @@ class TransactionController extends base_controller_1.BaseController {
         const ethQuery = new eth_query_1.default(provider);
         const { type } = await (0, transaction_type_1.determineTransactionType)(updatedTransaction.txParams, ethQuery);
         updatedTransaction.type = type;
+        if (containerTypes) {
+            updatedTransaction.containerTypes = containerTypes;
+        }
         await (0, layer1_gas_fee_flow_1.updateTransactionLayer1GasFee)({
             layer1GasFeeFlows: __classPrivateFieldGet(this, _TransactionController_layer1GasFeeFlows, "f"),
             messenger: this.messagingSystem,
@@ -1704,14 +1703,23 @@ async function _TransactionController_approveTransaction(transactionId, traceCon
     if (!transactions.length) {
         return;
     }
-    const finalTransactions = transactions.map((tx) => {
+    const finalTransactions = [];
+    for (const tx of transactions) {
         const { chainId } = tx;
-        const networkClientId = __classPrivateFieldGet(this, _TransactionController_instances, "m", _TransactionController_getNetworkClientId).call(this, { chainId });
-        return {
-            ...tx,
-            networkClientId,
-        };
-    });
+        try {
+            const networkClientId = __classPrivateFieldGet(this, _TransactionController_instances, "m", _TransactionController_getNetworkClientId).call(this, { chainId });
+            finalTransactions.push({
+                ...tx,
+                networkClientId,
+            });
+        }
+        catch (error) {
+            (0, logger_1.projectLogger)('Failed to get network client ID for incoming transaction', {
+                chainId,
+                error,
+            });
+        }
+    }
     this.update((state) => {
         const { transactions: currentTransactions } = state;
         state.transactions = __classPrivateFieldGet(this, _TransactionController_instances, "m", _TransactionController_trimTransactionsForState).call(this, [