feat: Configure pending transaction polling intervals using remote feature flags

Author: pedronfigueiredo

eslint-warning-thresholds.json

@@ -276,7 +276,6 @@
     "import-x/order": 1
   },
   "packages/multichain/src/adapters/caip-permission-adapter-eth-accounts.ts": {
-    "@typescript-eslint/no-unsafe-enum-comparison": 1,
     "jsdoc/tag-lines": 5
   },
   "packages/multichain/src/adapters/caip-permission-adapter-permittedChains.test.ts": {
@@ -299,7 +298,6 @@
     "jsdoc/require-returns": 1
   },
   "packages/multichain/src/handlers/wallet-invokeMethod.ts": {
-    "@typescript-eslint/no-unsafe-enum-comparison": 1,
     "@typescript-eslint/no-unused-vars": 1,
     "jsdoc/require-returns": 1
   },
@@ -324,9 +322,6 @@
   "packages/multichain/src/scope/assert.test.ts": {
     "@typescript-eslint/no-unused-vars": 3
   },
-  "packages/multichain/src/scope/assert.ts": {
-    "@typescript-eslint/no-unsafe-enum-comparison": 1
-  },
   "packages/multichain/src/scope/authorization.test.ts": {
     "@typescript-eslint/no-unused-vars": 2
   },
@@ -340,9 +335,6 @@
     "@typescript-eslint/no-unused-vars": 1,
     "jsdoc/require-returns": 1
   },
-  "packages/multichain/src/scope/supported.ts": {
-    "@typescript-eslint/no-unsafe-enum-comparison": 6
-  },
   "packages/multichain/src/scope/validation.ts": {
     "jsdoc/tag-lines": 2
   },

packages/transaction-controller/src/TransactionController.ts

@@ -3632,6 +3632,7 @@ export class TransactionController extends BaseController<
           this.beforeCheckPendingTransaction.bind(this),
         beforePublish: this.beforePublish.bind(this),
       },
+      messenger: this.messagingSystem,
     });
 
     this.#addPendingTransactionTrackerListeners(pendingTransactionTracker);

packages/transaction-controller/src/helpers/PendingTransactionTracker.test.ts

@@ -5,6 +5,7 @@ import { freeze } from 'immer';
 
 import { PendingTransactionTracker } from './PendingTransactionTracker';
 import { TransactionPoller } from './TransactionPoller';
+import type { TransactionControllerMessenger } from '../TransactionController';
 import type { TransactionMeta } from '../types';
 import { TransactionStatus } from '../types';
 
@@ -74,11 +75,25 @@ function createTransactionPollerMock(): jest.Mocked<TransactionPoller> {
   } as unknown as jest.Mocked<TransactionPoller>;
 }
 
+/**
+ * Creates a mock messenger instance.
+ *
+ * @returns The mock messenger instance.
+ */
+function createMessengerMock(): jest.Mocked<TransactionControllerMessenger> {
+  return {
+    call: jest.fn().mockReturnValue({
+      remoteFeatureFlags: {},
+    }),
+  } as unknown as jest.Mocked<TransactionControllerMessenger>;
+}
+
 describe('PendingTransactionTracker', () => {
   const queryMock = jest.mocked(query);
   let blockTracker: jest.Mocked<BlockTracker>;
   let pendingTransactionTracker: PendingTransactionTracker;
   let transactionPoller: jest.Mocked<TransactionPoller>;
+  let messenger: jest.Mocked<TransactionControllerMessenger>;
 
   let options: jest.Mocked<
     ConstructorParameters<typeof PendingTransactionTracker>[0]
@@ -112,6 +127,7 @@ describe('PendingTransactionTracker', () => {
   beforeEach(() => {
     blockTracker = createBlockTrackerMock();
     transactionPoller = createTransactionPollerMock();
+    messenger = createMessengerMock();
 
     jest.mocked(TransactionPoller).mockImplementation(() => transactionPoller);
 
@@ -123,6 +139,7 @@ describe('PendingTransactionTracker', () => {
       getTransactions: jest.fn(),
       getGlobalLock: jest.fn(() => Promise.resolve(jest.fn())),
       publishTransaction: jest.fn(),
+      messenger,
     };
   });
 

packages/transaction-controller/src/helpers/PendingTransactionTracker.ts

@@ -11,6 +11,7 @@ import { cloneDeep, merge } from 'lodash';
 
 import { TransactionPoller } from './TransactionPoller';
 import { createModuleLogger, projectLogger } from '../logger';
+import type { TransactionControllerMessenger } from '../TransactionController';
 import type { TransactionMeta, TransactionReceipt } from '../types';
 import { TransactionStatus, TransactionType } from '../types';
 
@@ -107,6 +108,7 @@ export class PendingTransactionTracker {
     getGlobalLock,
     publishTransaction,
     hooks,
+    messenger,
   }: {
     blockTracker: BlockTracker;
     getChainId: () => string;
@@ -125,6 +127,7 @@ export class PendingTransactionTracker {
       ) => boolean;
       beforePublish?: (transactionMeta: TransactionMeta) => boolean;
     };
+    messenger: TransactionControllerMessenger;
   }) {
     this.hub = new EventEmitter() as PendingTransactionTrackerEventEmitter;
 
@@ -138,7 +141,11 @@ export class PendingTransactionTracker {
     this.#getGlobalLock = getGlobalLock;
     this.#publishTransaction = publishTransaction;
     this.#running = false;
-    this.#transactionPoller = new TransactionPoller(blockTracker);
+    this.#transactionPoller = new TransactionPoller(
+      blockTracker,
+      messenger,
+      getChainId(),
+    );
     this.#beforePublish = hooks?.beforePublish ?? (() => true);
     this.#beforeCheckPendingTransaction =
       hooks?.beforeCheckPendingTransaction ?? (() => true);