@metamask-previews/keyring-controller 21.0.3-preview-bad0bc3 → 21.0.3-preview-530f9e5

package/CHANGELOG.md

@@ -7,10 +7,6 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
-### Fixed
-
-- Ensure no duplicate accounts are peristed ([#5710](https://github.com/MetaMask/core/pull/5710))
-
 ## [21.0.3]
 
 ### Changed

package/dist/KeyringController.cjs

@@ -36,7 +36,7 @@ var __classPrivateFieldGet = (this && this.__classPrivateFieldGet) || function (
 var __importDefault = (this && this.__importDefault) || function (mod) {
     return (mod && mod.__esModule) ? mod : { "default": mod };
 };
-var _KeyringController_instances, _KeyringController_controllerOperationMutex, _KeyringController_vaultOperationMutex, _KeyringController_keyringBuilders, _KeyringController_encryptor, _KeyringController_cacheEncryptionKey, _KeyringController_keyrings, _KeyringController_unsupportedKeyrings, _KeyringController_keyringsMetadata, _KeyringController_password, _KeyringController_qrKeyringStateListener, _KeyringController_registerMessageHandlers, _KeyringController_getKeyringById, _KeyringController_getKeyringByIdOrDefault, _KeyringController_getKeyringMetadata, _KeyringController_getKeyringBuilderForType, _KeyringController_addQRKeyring, _KeyringController_subscribeToQRKeyringEvents, _KeyringController_unsubscribeFromQRKeyringsEvents, _KeyringController_createNewVaultWithKeyring, _KeyringController_verifySeedPhrase, _KeyringController_getUpdatedKeyrings, _KeyringController_getSerializedKeyrings, _KeyringController_restoreSerializedKeyrings, _KeyringController_unlockKeyrings, _KeyringController_updateVault, _KeyringController_getAccountsFromKeyrings, _KeyringController_createKeyringWithFirstAccount, _KeyringController_newKeyring, _KeyringController_createKeyring, _KeyringController_clearKeyrings, _KeyringController_restoreKeyring, _KeyringController_destroyKeyring, _KeyringController_removeEmptyKeyrings, _KeyringController_assertNoDuplicateAccounts, _KeyringController_setUnlocked, _KeyringController_assertIsUnlocked, _KeyringController_persistOrRollback, _KeyringController_withRollback, _KeyringController_assertControllerMutexIsLocked, _KeyringController_withControllerLock, _KeyringController_withVaultLock;
+var _KeyringController_instances, _KeyringController_controllerOperationMutex, _KeyringController_vaultOperationMutex, _KeyringController_keyringBuilders, _KeyringController_encryptor, _KeyringController_cacheEncryptionKey, _KeyringController_keyrings, _KeyringController_unsupportedKeyrings, _KeyringController_keyringsMetadata, _KeyringController_password, _KeyringController_qrKeyringStateListener, _KeyringController_registerMessageHandlers, _KeyringController_getKeyringById, _KeyringController_getKeyringByIdOrDefault, _KeyringController_getKeyringMetadata, _KeyringController_getKeyringBuilderForType, _KeyringController_addQRKeyring, _KeyringController_subscribeToQRKeyringEvents, _KeyringController_unsubscribeFromQRKeyringsEvents, _KeyringController_createNewVaultWithKeyring, _KeyringController_verifySeedPhrase, _KeyringController_getUpdatedKeyrings, _KeyringController_getSerializedKeyrings, _KeyringController_restoreSerializedKeyrings, _KeyringController_unlockKeyrings, _KeyringController_updateVault, _KeyringController_getAccountsFromKeyrings, _KeyringController_createKeyringWithFirstAccount, _KeyringController_newKeyring, _KeyringController_createKeyring, _KeyringController_clearKeyrings, _KeyringController_restoreKeyring, _KeyringController_destroyKeyring, _KeyringController_removeEmptyKeyrings, _KeyringController_checkForDuplicate, _KeyringController_setUnlocked, _KeyringController_assertIsUnlocked, _KeyringController_persistOrRollback, _KeyringController_withRollback, _KeyringController_assertControllerMutexIsLocked, _KeyringController_withControllerLock, _KeyringController_withVaultLock;
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.KeyringController = exports.getDefaultKeyringState = exports.keyringBuilderFactory = exports.SignTypedDataVersion = exports.AccountImportStrategy = exports.isCustodyKeyring = exports.KeyringTypes = void 0;
 const util_1 = require("@ethereumjs/util");
@@ -1379,8 +1379,6 @@ async function _KeyringController_unlockKeyrings(password, encryptionKey, encryp
     });
 }, _KeyringController_updateVault = function _KeyringController_updateVault() {
     return __classPrivateFieldGet(this, _KeyringController_instances, "m", _KeyringController_withVaultLock).call(this, async () => {
-        // Ensure no duplicate accounts are persisted.
-        await __classPrivateFieldGet(this, _KeyringController_instances, "m", _KeyringController_assertNoDuplicateAccounts).call(this);
         const { encryptionKey, encryptionSalt, vault } = this.state;
         // READ THIS CAREFULLY:
         // We do check if the vault is still considered up-to-date, if not, we would not re-use the
@@ -1530,6 +1528,7 @@ async function _KeyringController_createKeyring(type, data) {
         await keyring.generateRandomMnemonic();
         await keyring.addAccounts(1);
     }
+    await __classPrivateFieldGet(this, _KeyringController_instances, "m", _KeyringController_checkForDuplicate).call(this, type, await keyring.getAccounts());
     if (type === KeyringTypes.qr) {
         // In case of a QR keyring type, we need to subscribe
         // to its events after creating it
@@ -1615,16 +1614,32 @@ async function _KeyringController_removeEmptyKeyrings() {
     }));
     __classPrivateFieldSet(this, _KeyringController_keyrings, validKeyrings, "f");
     __classPrivateFieldSet(this, _KeyringController_keyringsMetadata, validKeyringMetadata, "f");
-}, _KeyringController_assertNoDuplicateAccounts = 
+}, _KeyringController_checkForDuplicate = 
 /**
- * Assert that there are no duplicate accounts in the keyrings.
+ * Checks for duplicate keypairs, using the the first account in the given
+ * array. Rejects if a duplicate is found.
  *
- * @throws If there are duplicate accounts.
+ * Only supports 'Simple Key Pair'.
+ *
+ * @param type - The key pair type to check for.
+ * @param newAccountArray - Array of new accounts.
+ * @returns The account, if no duplicate is found.
  */
-async function _KeyringController_assertNoDuplicateAccounts() {
+async function _KeyringController_checkForDuplicate(type, newAccountArray) {
     const accounts = await __classPrivateFieldGet(this, _KeyringController_instances, "m", _KeyringController_getAccountsFromKeyrings).call(this);
-    if (new Set(accounts).size !== accounts.length) {
-        throw new Error(constants_1.KeyringControllerError.DuplicatedAccount);
+    switch (type) {
+        case KeyringTypes.simple: {
+            const isIncluded = Boolean(accounts.find((key) => newAccountArray[0] &&
+                (key === newAccountArray[0] ||
+                    key === (0, utils_1.remove0x)(newAccountArray[0]))));
+            if (isIncluded) {
+                throw new Error(constants_1.KeyringControllerError.DuplicatedAccount);
+            }
+            return newAccountArray;
+        }
+        default: {
+            return newAccountArray;
+        }
     }
 }, _KeyringController_setUnlocked = function _KeyringController_setUnlocked() {
     __classPrivateFieldGet(this, _KeyringController_instances, "m", _KeyringController_assertControllerMutexIsLocked).call(this);