@metamask-previews/transaction-controller 45.0.0-preview-83e41399 → 45.0.0-preview-224dff25

package/dist/TransactionController.cjs

@@ -239,7 +239,7 @@ class TransactionController extends base_controller_1.BaseController {
             hooks?.beforeCheckPendingTransaction ??
                 /* istanbul ignore next */
                 (() => true);
-        this.beforePublish = hooks?.beforePublish ?? (() => true);
+        this.beforePublish = hooks?.beforePublish ?? (() => Promise.resolve(true));
         this.getAdditionalSignArguments =
             hooks?.getAdditionalSignArguments ?? (() => []);
         this.publish =
@@ -1064,6 +1064,7 @@ class TransactionController extends base_controller_1.BaseController {
     /**
      * Stop the signing process for a specific transaction.
      * Throws an error causing the transaction status to be set to failed.
+     *
      * @param transactionId - The ID of the transaction to stop signing.
      */
     abortTransactionSigning(transactionId) {
@@ -1244,7 +1245,7 @@ class TransactionController extends base_controller_1.BaseController {
             });
             this.onTransactionStatusChange(transactionMeta);
             const rawTx = await __classPrivateFieldGet(this, _TransactionController_trace, "f").call(this, { name: 'Sign', parentContext: traceContext }, () => this.signTransaction(transactionMeta, transactionMeta.txParams));
-            if (!this.beforePublish(transactionMeta, rawTx)) {
+            if (!(await this.beforePublish(transactionMeta))) {
                 (0, logger_1.projectLogger)('Skipping publishing transaction based on hook');
                 this.messagingSystem.publish(`${controllerName}:transactionPublishingSkipped`, transactionMeta);
                 return ApprovalState.SkippedViaBeforePublishHook;
@@ -1336,7 +1337,6 @@ class TransactionController extends base_controller_1.BaseController {
         this.messagingSystem.publish(`${controllerName}:transactionFinished`, updatedTransactionMeta);
         __classPrivateFieldGet(this, _TransactionController_internalEvents, "f").emit(
         // TODO: Either fix this lint violation or explain why it's necessary to ignore.
-        // eslint-disable-next-line @typescript-eslint/restrict-template-expressions
         `${transactionMeta.id}:finished`, updatedTransactionMeta);
         this.messagingSystem.publish(`${controllerName}:transactionRejected`, {
             transactionMeta: updatedTransactionMeta,
@@ -1366,7 +1366,6 @@ class TransactionController extends base_controller_1.BaseController {
             const { chainId, status, txParams, time } = tx;
             if (txParams) {
                 // TODO: Either fix this lint violation or explain why it's necessary to ignore.
-                // eslint-disable-next-line @typescript-eslint/restrict-template-expressions
                 const key = `${String(txParams.nonce)}-${(0, controller_utils_1.convertHexToDecimal)(chainId)}-${new Date(time).toDateString()}`;
                 if (nonceNetworkSet.has(key)) {
                     return true;