From b141caf353d8f02146491a426fe99abaf91cf1ca Mon Sep 17 00:00:00 2001
From: Jongsun Suh <jongsun.suh@icloud.com>
Date: Mon, 29 Jul 2024 12:08:02 -0400
Subject: [PATCH] Consolidate comments on ComposableController events allowlist

---
 app/core/Engine.ts | 20 +++-----------------
 1 file changed, 3 insertions(+), 17 deletions(-)

diff --git a/app/core/Engine.ts b/app/core/Engine.ts
index 936b03201275..4d31e936fd30 100644
--- a/app/core/Engine.ts
+++ b/app/core/Engine.ts
@@ -1484,10 +1484,8 @@ class Engine {
       messenger: this.controllerMessenger.getRestricted({
         name: 'ComposableController',
         allowedActions: [],
+        // Exclude non-controllers with empty state and V1 controllers with no messagingSystem.
         allowedEvents: [
-          /**
-           * V1/V2 controllers with correctly defined messengers and `stateChange` events.
-           */
           'AccountsController:stateChange',
           'AccountTrackerController:stateChange',
           'AddressBookController:stateChange',
@@ -1505,29 +1503,17 @@ class Engine {
           'PPOMController:stateChange',
           'PreferencesController:stateChange',
           'SignatureController:stateChange',
+          // 'SmartTransactionsController:stateChange', // TODO: uncomment once controller is migrated to V2.
           'SnapController:stateChange',
           'SnapsRegistry:stateChange',
           'SubjectMetadataController:stateChange',
+          // 'SwapsController:stateChange', // TODO: uncomment once controller is migrated to V2.
           'TokenBalancesController:stateChange',
           'TokenRatesController:stateChange',
           'TokenListController:stateChange',
           'TokensController:stateChange',
           'TransactionController:stateChange',
           'UserStorageController:stateChange',
-
-          /**
-           * V1 controllers that should be excluded from the datamodel's events allowlist for now.
-           * TODO: Each of these events should be added to the allowlist once its controller is migrated to V2.
-           */
-          // 'SmartTransactionsController:stateChange', // StaticIntervalPollingControllerV1, no `messagingSystem`
-          // 'SwapsController:stateChange', // BaseControllerV1, no `messagingSystem`
-
-          /**
-           * Non-controllers that should always be excluded from the datamodel's events allowlist.
-           */
-          // 'AssetsContractController:stateChange', // BaseControllerV1, no `messagingSystem`
-          // 'NftDetectionController:stateChange', // StaticIntervalPollingControllerV1, no `messagingSystem`
-          // 'TokenDetectionController:stateChange', // StaticIntervalPollingController, empty state
         ],
       }),
     });