@metamask-previews/transaction-controller 63.3.1-preview-a560ee403 → 63.3.1-preview-bb9040b29

package/CHANGELOG.md

@@ -10,6 +10,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Changed
 
 - Bump `@metamask/accounts-controller` from `^37.1.0` to `^37.1.1` ([#8325](https://github.com/MetaMask/core/pull/8325))
+- Bump `@metamask/controller-utils` from `^11.19.0` to `^11.20.0` ([#8344](https://github.com/MetaMask/core/pull/8344))
+- Add extra parameters in types and change `addTransaction` behavior ([#8052](https://github.com/MetaMask/core/pull/8052))
+  - Add optional `excludeNativeTokenForFee` in `TransactionMeta` type.
+  - Add optional `actionId` and optional `excludeNativeTokenForFee` in `TransactionBatchRequest` type.
+  - Add optional `excludeNativeTokenForFee` to `AddTransactionOptions` type.
+  - Changed `isGasFeeTokenIgnoredIfBalance` to be false if `excludeNativeTokenForFee` is passed in `addTransaction`.
 
 ### Removed
 

package/dist/TransactionController.cjs

@@ -405,7 +405,7 @@ class TransactionController extends base_controller_1.BaseController {
      */
     async addTransaction(txParams, options) {
         (0, logger_1.projectLogger)('Adding transaction', txParams, options);
-        const { actionId, assetsFiatValues, batchId, deviceConfirmedOn, disableGasBuffer, gasFeeToken, isGasFeeIncluded, isGasFeeSponsored, isStateOnly, method, nestedTransactions, networkClientId, origin, publishHook, requestId, requiredAssets, requireApproval, securityAlertResponse, skipInitialGasEstimate, swaps = {}, traceContext, type, } = options;
+        const { actionId, assetsFiatValues, batchId, deviceConfirmedOn, disableGasBuffer, gasFeeToken, excludeNativeTokenForFee, isGasFeeIncluded, isGasFeeSponsored, isStateOnly, method, nestedTransactions, networkClientId, origin, publishHook, requestId, requiredAssets, requireApproval, securityAlertResponse, skipInitialGasEstimate, swaps = {}, traceContext, type, } = options;
         // eslint-disable-next-line no-param-reassign
         txParams = (0, utils_2.normalizeTransactionParams)(txParams);
         if (!__classPrivateFieldGet(this, _TransactionController_multichainTrackingHelper, "f").has(networkClientId)) {
@@ -443,6 +443,12 @@ class TransactionController extends base_controller_1.BaseController {
                 messenger: this.messenger,
                 networkClientId,
             })).type;
+        /**
+         * Original behavior was that this was set to 'true' whenever a gasFeeToken was passed.
+         * 'excludeNativeTokenForFee' optionally overrides this behavior to prevent native token from
+         * being used when another gasFeeToken is set.
+         */
+        const isGasFeeTokenIgnoredIfBalance = Boolean(gasFeeToken) && !excludeNativeTokenForFee;
         let addedTransactionMeta = {
             actionId,
             assetsFiatValues,
@@ -452,9 +458,13 @@ class TransactionController extends base_controller_1.BaseController {
             deviceConfirmedOn,
             disableGasBuffer,
             id: (0, uuid_1.v1)(),
-            isGasFeeTokenIgnoredIfBalance: Boolean(gasFeeToken),
+            isGasFeeTokenIgnoredIfBalance,
             isGasFeeIncluded,
             isGasFeeSponsored,
+            // To avoid the property to be set as undefined.
+            ...(excludeNativeTokenForFee === undefined
+                ? {}
+                : { excludeNativeTokenForFee }),
             isFirstTimeInteraction: undefined,
             isStateOnly,
             nestedTransactions,