@metamask-previews/account-tree-controller 1.5.0-preview-cd26f001 → 1.5.0-preview-e9856975

package/CHANGELOG.md

@@ -11,6 +11,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Changed
 
+- **BREAKING:** Use new `Messenger` from `@metamask/messenger` ([#6380](https://github.com/MetaMask/core/pull/6380))
+  - Previously, `AccountTreeController` accepted a `RestrictedMessenger` instance from `@metamask/base-controller`.
 - Use non-EVM account names for group names ([#6831](https://github.com/MetaMask/core/pull/6831))
   - EVM accounts still take precedence over non-EVM accounts.
   - Before accounts get re-aligned, it is possible that a group contains only non-EVM accounts, in which case, the first non-EVM account name will be used for that account group.

package/dist/AccountTreeController.cjs

@@ -14,7 +14,7 @@ var _AccountTreeController_instances, _AccountTreeController_accountIdToContext,
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.AccountTreeController = exports.getDefaultAccountTreeControllerState = exports.controllerName = void 0;
 const account_api_1 = require("@metamask/account-api");
-const base_controller_1 = require("@metamask/base-controller");
+const next_1 = require("@metamask/base-controller/next");
 const keyring_api_1 = require("@metamask/keyring-api");
 const utils_1 = require("@metamask/utils");
 const analytics_1 = require("./backup-and-sync/analytics/index.cjs");
@@ -29,31 +29,31 @@ const accountTreeControllerMetadata = {
     accountTree: {
         includeInStateLogs: true,
         persist: false,
-        anonymous: false,
+        includeInDebugSnapshot: false,
         usedInUi: true,
     },
     isAccountTreeSyncingInProgress: {
         includeInStateLogs: false,
         persist: false,
-        anonymous: false,
+        includeInDebugSnapshot: false,
         usedInUi: true,
     },
     hasAccountTreeSyncingSyncedAtLeastOnce: {
         includeInStateLogs: true,
         persist: true,
-        anonymous: false,
+        includeInDebugSnapshot: false,
         usedInUi: true,
     },
     accountGroupsMetadata: {
         includeInStateLogs: true,
         persist: true,
-        anonymous: false,
+        includeInDebugSnapshot: false,
         usedInUi: true,
     },
     accountWalletsMetadata: {
         includeInStateLogs: true,
         persist: true,
-        anonymous: false,
+        includeInDebugSnapshot: false,
         usedInUi: true,
     },
 };
@@ -75,7 +75,7 @@ function getDefaultAccountTreeControllerState() {
     };
 }
 exports.getDefaultAccountTreeControllerState = getDefaultAccountTreeControllerState;
-class AccountTreeController extends base_controller_1.BaseController {
+class AccountTreeController extends next_1.BaseController {
     /**
      * Constructor for AccountTreeController.
      *
@@ -114,11 +114,11 @@ class AccountTreeController extends base_controller_1.BaseController {
         // Rules to apply to construct the wallets tree.
         __classPrivateFieldSet(this, _AccountTreeController_rules, [
             // 1. We group by entropy-source
-            new entropy_1.EntropyRule(this.messagingSystem),
+            new entropy_1.EntropyRule(this.messenger),
             // 2. We group by Snap ID
-            new snap_1.SnapRule(this.messagingSystem),
+            new snap_1.SnapRule(this.messenger),
             // 3. We group by wallet type (this rule cannot fail and will group all non-matching accounts)
-            new keyring_1.KeyringRule(this.messagingSystem),
+            new keyring_1.KeyringRule(this.messenger),
         ], "f");
         // Initialize trace function
         __classPrivateFieldSet(this, _AccountTreeController_trace, config?.trace ?? analytics_1.traceFallback, "f");
@@ -131,19 +131,19 @@ class AccountTreeController extends base_controller_1.BaseController {
         }, "f");
         // Initialize the backup and sync service
         __classPrivateFieldSet(this, _AccountTreeController_backupAndSyncService, new service_1.BackupAndSyncService(__classPrivateFieldGet(this, _AccountTreeController_instances, "m", _AccountTreeController_createBackupAndSyncContext).call(this)), "f");
-        this.messagingSystem.subscribe('AccountsController:accountAdded', (account) => {
+        this.messenger.subscribe('AccountsController:accountAdded', (account) => {
             __classPrivateFieldGet(this, _AccountTreeController_instances, "m", _AccountTreeController_handleAccountAdded).call(this, account);
         });
-        this.messagingSystem.subscribe('AccountsController:accountRemoved', (accountId) => {
+        this.messenger.subscribe('AccountsController:accountRemoved', (accountId) => {
             __classPrivateFieldGet(this, _AccountTreeController_instances, "m", _AccountTreeController_handleAccountRemoved).call(this, accountId);
         });
-        this.messagingSystem.subscribe('AccountsController:selectedAccountChange', (account) => {
+        this.messenger.subscribe('AccountsController:selectedAccountChange', (account) => {
             __classPrivateFieldGet(this, _AccountTreeController_instances, "m", _AccountTreeController_handleSelectedAccountChange).call(this, account);
         });
-        this.messagingSystem.subscribe('UserStorageController:stateChange', (userStorageControllerState) => {
+        this.messenger.subscribe('UserStorageController:stateChange', (userStorageControllerState) => {
             __classPrivateFieldGet(this, _AccountTreeController_backupAndSyncService, "f").handleUserStorageStateChange(userStorageControllerState);
         });
-        this.messagingSystem.subscribe('MultichainAccountService:walletStatusChange', (walletId, status) => {
+        this.messenger.subscribe('MultichainAccountService:walletStatusChange', (walletId, status) => {
             __classPrivateFieldGet(this, _AccountTreeController_instances, "m", _AccountTreeController_handleMultichainAccountWalletStatusChange).call(this, walletId, status);
         });
         __classPrivateFieldGet(this, _AccountTreeController_instances, "m", _AccountTreeController_registerMessageHandlers).call(this);
@@ -232,7 +232,7 @@ class AccountTreeController extends base_controller_1.BaseController {
         if (previousSelectedAccountGroup !==
             this.state.accountTree.selectedAccountGroup) {
             (0, logger_1.projectLogger)(`Selected (initial) group is: [${this.state.accountTree.selectedAccountGroup}]`);
-            this.messagingSystem.publish(`${exports.controllerName}:selectedAccountGroupChange`, this.state.accountTree.selectedAccountGroup, previousSelectedAccountGroup);
+            this.messenger.publish(`${exports.controllerName}:selectedAccountGroupChange`, this.state.accountTree.selectedAccountGroup, previousSelectedAccountGroup);
         }
         (0, logger_1.projectLogger)('Initialized!');
         __classPrivateFieldSet(this, _AccountTreeController_initialized, true, "f");
@@ -293,7 +293,7 @@ class AccountTreeController extends base_controller_1.BaseController {
         }
         const accounts = [];
         for (const id of group.accounts) {
-            const account = this.messagingSystem.call('AccountsController:getAccount', id);
+            const account = this.messenger.call('AccountsController:getAccount', id);
             // For now, we're filtering undefined account, but I believe
             // throwing would be more appropriate here.
             if (account) {
@@ -345,10 +345,10 @@ class AccountTreeController extends base_controller_1.BaseController {
             state.accountTree.selectedAccountGroup = groupId;
         });
         (0, logger_1.projectLogger)(`Selected group is now: [${this.state.accountTree.selectedAccountGroup}]`);
-        this.messagingSystem.publish(`${exports.controllerName}:selectedAccountGroupChange`, groupId, previousSelectedAccountGroup);
+        this.messenger.publish(`${exports.controllerName}:selectedAccountGroupChange`, groupId, previousSelectedAccountGroup);
         // Update AccountsController - this will trigger selectedAccountChange event,
         // but our handler is idempotent so it won't cause infinite loop
-        this.messagingSystem.call('AccountsController:setSelectedAccount', accountToSelect);
+        this.messenger.call('AccountsController:setSelectedAccount', accountToSelect);
     }
     /**
      * Resolves name conflicts by adding a suffix to make the name unique.
@@ -596,7 +596,7 @@ _AccountTreeController_accountIdToContext = new WeakMap(), _AccountTreeControlle
 }, _AccountTreeController_getComputedAccountGroupName = function _AccountTreeController_getComputedAccountGroupName(wallet, group) {
     let proposedName = ''; // Empty means there's no computed name for this group.
     for (const id of group.accounts) {
-        const account = this.messagingSystem.call('AccountsController:getAccount', id);
+        const account = this.messenger.call('AccountsController:getAccount', id);
         if (!account || !account.metadata.name.length) {
             continue;
         }
@@ -735,7 +735,7 @@ _AccountTreeController_accountIdToContext = new WeakMap(), _AccountTreeControlle
                 }
             }
         });
-        this.messagingSystem.publish(`${exports.controllerName}:accountTreeChange`, this.state.accountTree);
+        this.messenger.publish(`${exports.controllerName}:accountTreeChange`, this.state.accountTree);
     }
 }, _AccountTreeController_handleAccountRemoved = function _AccountTreeController_handleAccountRemoved(accountId) {
     // We wait for the first `init` to be called to actually build up the tree and
@@ -770,10 +770,10 @@ _AccountTreeController_accountIdToContext = new WeakMap(), _AccountTreeControlle
                 }
             }
         });
-        this.messagingSystem.publish(`${exports.controllerName}:accountTreeChange`, this.state.accountTree);
+        this.messenger.publish(`${exports.controllerName}:accountTreeChange`, this.state.accountTree);
         // Emit selectedAccountGroupChange event if the selected group changed
         if (selectedAccountGroupChanged) {
-            this.messagingSystem.publish(`${exports.controllerName}:selectedAccountGroupChange`, this.state.accountTree.selectedAccountGroup, previousSelectedAccountGroup);
+            this.messenger.publish(`${exports.controllerName}:selectedAccountGroupChange`, this.state.accountTree.selectedAccountGroup, previousSelectedAccountGroup);
         }
         // Clear reverse-mapping for that account.
         __classPrivateFieldGet(this, _AccountTreeController_accountIdToContext, "f").delete(accountId);
@@ -868,7 +868,7 @@ _AccountTreeController_accountIdToContext = new WeakMap(), _AccountTreeControlle
         sortOrder,
     });
 }, _AccountTreeController_listAccounts = function _AccountTreeController_listAccounts() {
-    return this.messagingSystem.call('AccountsController:listMultichainAccounts');
+    return this.messenger.call('AccountsController:listMultichainAccounts');
 }, _AccountTreeController_assertAccountGroupExists = function _AccountTreeController_assertAccountGroupExists(groupId) {
     const exists = __classPrivateFieldGet(this, _AccountTreeController_groupIdToWalletId, "f").has(groupId);
     if (!exists) {
@@ -884,7 +884,7 @@ _AccountTreeController_accountIdToContext = new WeakMap(), _AccountTreeControlle
         throw new Error('Account group name already exists');
     }
 }, _AccountTreeController_getDefaultSelectedAccountGroup = function _AccountTreeController_getDefaultSelectedAccountGroup(wallets) {
-    const selectedAccount = this.messagingSystem.call('AccountsController:getSelectedMultichainAccount');
+    const selectedAccount = this.messenger.call('AccountsController:getSelectedMultichainAccount');
     if (selectedAccount && selectedAccount.id) {
         const accountMapping = __classPrivateFieldGet(this, _AccountTreeController_accountIdToContext, "f").get(selectedAccount.id);
         if (accountMapping) {
@@ -910,7 +910,7 @@ _AccountTreeController_accountIdToContext = new WeakMap(), _AccountTreeControlle
     this.update((state) => {
         state.accountTree.selectedAccountGroup = groupId;
     });
-    this.messagingSystem.publish(`${exports.controllerName}:selectedAccountGroupChange`, groupId, previousSelectedAccountGroup);
+    this.messenger.publish(`${exports.controllerName}:selectedAccountGroupChange`, groupId, previousSelectedAccountGroup);
 }, _AccountTreeController_handleMultichainAccountWalletStatusChange = function _AccountTreeController_handleMultichainAccountWalletStatusChange(walletId, walletStatus) {
     this.update((state) => {
         const wallet = state.accountTree.wallets[walletId];
@@ -926,7 +926,7 @@ _AccountTreeController_accountIdToContext = new WeakMap(), _AccountTreeControlle
     if (group) {
         let candidate;
         for (const id of group.accounts) {
-            const account = this.messagingSystem.call('AccountsController:getAccount', id);
+            const account = this.messenger.call('AccountsController:getAccount', id);
             if (!candidate) {
                 candidate = id;
             }
@@ -950,7 +950,7 @@ _AccountTreeController_accountIdToContext = new WeakMap(), _AccountTreeControlle
                 candidate = group.id;
             }
             for (const id of group.accounts) {
-                const account = this.messagingSystem.call('AccountsController:getAccount', id);
+                const account = this.messenger.call('AccountsController:getAccount', id);
                 if (account && (0, keyring_api_1.isEvmAccountType)(account.type)) {
                     // EVM accounts have a higher priority, so if we find any, we just
                     // use that group!
@@ -961,18 +961,18 @@ _AccountTreeController_accountIdToContext = new WeakMap(), _AccountTreeControlle
     }
     return candidate;
 }, _AccountTreeController_registerMessageHandlers = function _AccountTreeController_registerMessageHandlers() {
-    this.messagingSystem.registerActionHandler(`${exports.controllerName}:getSelectedAccountGroup`, this.getSelectedAccountGroup.bind(this));
-    this.messagingSystem.registerActionHandler(`${exports.controllerName}:setSelectedAccountGroup`, this.setSelectedAccountGroup.bind(this));
-    this.messagingSystem.registerActionHandler(`${exports.controllerName}:getAccountsFromSelectedAccountGroup`, this.getAccountsFromSelectedAccountGroup.bind(this));
-    this.messagingSystem.registerActionHandler(`${exports.controllerName}:setAccountWalletName`, this.setAccountWalletName.bind(this));
-    this.messagingSystem.registerActionHandler(`${exports.controllerName}:setAccountGroupName`, this.setAccountGroupName.bind(this));
-    this.messagingSystem.registerActionHandler(`${exports.controllerName}:setAccountGroupPinned`, this.setAccountGroupPinned.bind(this));
-    this.messagingSystem.registerActionHandler(`${exports.controllerName}:setAccountGroupHidden`, this.setAccountGroupHidden.bind(this));
+    this.messenger.registerActionHandler(`${exports.controllerName}:getSelectedAccountGroup`, this.getSelectedAccountGroup.bind(this));
+    this.messenger.registerActionHandler(`${exports.controllerName}:setSelectedAccountGroup`, this.setSelectedAccountGroup.bind(this));
+    this.messenger.registerActionHandler(`${exports.controllerName}:getAccountsFromSelectedAccountGroup`, this.getAccountsFromSelectedAccountGroup.bind(this));
+    this.messenger.registerActionHandler(`${exports.controllerName}:setAccountWalletName`, this.setAccountWalletName.bind(this));
+    this.messenger.registerActionHandler(`${exports.controllerName}:setAccountGroupName`, this.setAccountGroupName.bind(this));
+    this.messenger.registerActionHandler(`${exports.controllerName}:setAccountGroupPinned`, this.setAccountGroupPinned.bind(this));
+    this.messenger.registerActionHandler(`${exports.controllerName}:setAccountGroupHidden`, this.setAccountGroupHidden.bind(this));
 }, _AccountTreeController_createBackupAndSyncContext = function _AccountTreeController_createBackupAndSyncContext() {
     return {
         ...__classPrivateFieldGet(this, _AccountTreeController_backupAndSyncConfig, "f"),
         controller: this,
-        messenger: this.messagingSystem,
+        messenger: this.messenger,
         controllerStateUpdateFn: this.update.bind(this),
         traceFn: __classPrivateFieldGet(this, _AccountTreeController_trace, "f").bind(this),
         groupIdToWalletId: __classPrivateFieldGet(this, _AccountTreeController_groupIdToWalletId, "f"),