@metamask-previews/transaction-controller 57.3.0-preview-4e3fd4c0 → 58.0.0-preview-08a4995

package/CHANGELOG.md

@@ -9,6 +9,27 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Added
 
+- 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 +39,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 +1704,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) => {
@@ -1704,14 +1699,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, [