@metamask-previews/transaction-controller 63.3.1-preview-de55719 → 63.3.1-preview-27fa5d81d

package/CHANGELOG.md

@@ -7,13 +7,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
-### Added
-
-- Add optional `atomic` property to `TransactionBatchRequest` to configure whether EIP-7702 batch calls revert together or can fail independently ([#8320](https://github.com/MetaMask/core/pull/8320))
-
 ### Changed
 
 - Bump `@metamask/accounts-controller` from `^37.1.0` to `^37.1.1` ([#8325](https://github.com/MetaMask/core/pull/8325))
+- 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,