feat(seedless-onboarding): add dataType support for secret data items

packages/seedless-onboarding-controller/CHANGELOG.md

@@ -7,8 +7,25 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Added
+
+- Add optional `dataType` parameter to `createToprfKeyAndBackupSeedPhrase` method (defaults to `PrimarySrp` for backward compatibility)
+- Add optional `dataType` parameter to `addNewSecretData` method for categorizing secret data on insert
+- Add `updateSecretDataItem` method to update fields for existing items by `itemId`
+- Add `batchUpdateSecretDataItems` method to batch update fields for multiple items
+- Add `SecretDataItemWithMetadata` type for storage-level metadata (`itemId`, `dataType`)
+- Add `SecretMetadata.compareByTimestamp` static method for comparing metadata by timestamp
+- Add `SecretMetadata.matchesType` static method for checking if metadata matches a given type
+- Re-export `EncAccountDataType` from `@metamask/toprf-secure-backup`
+
 ### Changed
 
+- **BREAKING:** Update `fetchAllSecretData` to return `SecretDataItemWithMetadata[]` instead of `SecretMetadata[]`
+  - Consumers must now access secret data via the `secret` property (e.g., `result[0].secret.data` instead of `result[0].data`)
+- **BREAKING:** Remove `parseSecretsFromMetadataStore`, `fromBatch`, and `sort` methods from `SecretMetadata`
+  - Use `SecretMetadata.compareByTimestamp` for sorting
+  - Use `SecretMetadata.matchesType` for filtering
+- Bump `@metamask/toprf-secure-backup` from `^0.10.0` to `^0.11.0`
 - Move peer dependencies for controller and service packages to direct dependencies ([#7209](https://github.com/MetaMask/core/pull/7209))
   - The dependencies moved are:
     - `@metamask/keyring-controller` (^25.0.0)

packages/seedless-onboarding-controller/package.json

@@ -53,7 +53,7 @@
     "@metamask/browser-passworder": "^6.0.0",
     "@metamask/keyring-controller": "^25.0.0",
     "@metamask/messenger": "^0.3.0",
-    "@metamask/toprf-secure-backup": "^0.10.0",
+    "@metamask/toprf-secure-backup": "^0.11.0",
     "@metamask/utils": "^11.8.1",
     "@noble/ciphers": "^1.3.0",
     "@noble/curves": "^1.9.2",

packages/seedless-onboarding-controller/src/SecretMetadata.ts

@@ -68,37 +68,6 @@ export class SecretMetadata<DataType extends SecretDataType = Uint8Array>
     this.#version = options?.version ?? SecretMetadataVersion.V1;
   }
 
-  /**
-   * Create an Array of SecretMetadata instances from an array of secrets.
-   *
-   * To respect the order of the secrets, we add the index to the timestamp
-   * so that the first secret backup will have the oldest timestamp
-   * and the last secret backup will have the newest timestamp.
-   *
-   * @param data - The data to add metadata to.
-   * @param data.value - The SeedPhrase/PrivateKey to add metadata to.
-   * @param data.options - The options for the seed phrase metadata.
-   * @returns The SecretMetadata instances.
-   */
-  static fromBatch<DataType extends SecretDataType = Uint8Array>(
-    data: {
-      value: DataType;
-      options?: Partial<SecretMetadataOptions>;
-    }[],
-  ): SecretMetadata<DataType>[] {
-    const timestamp = Date.now();
-    return data.map((d, index) => {
-      // To respect the order of the seed phrases, we add the index to the timestamp
-      // so that the first seed phrase backup will have the oldest timestamp
-      // and the last seed phrase backup will have the newest timestamp
-      const backupCreatedAt = d.options?.timestamp ?? timestamp + index * 5;
-      return new SecretMetadata(d.value, {
-        timestamp: backupCreatedAt,
-        type: d.options?.type,
-      });
-    });
-  }
-
   /**
    * Assert that the provided value is a valid seed phrase metadata.
    *
@@ -122,34 +91,6 @@ export class SecretMetadata<DataType extends SecretDataType = Uint8Array>
     }
   }
 
-  /**
-   * Parse the SecretMetadata from the metadata store and return the array of SecretMetadata instances.
-   *
-   * This method also sorts the secrets by timestamp in ascending order, i.e. the oldest secret will be the first element in the array.
-   *
-   * @param secretMetadataArr - The array of SecretMetadata from the metadata store.
-   * @param filterType - The type of the secret to filter.
-   * @returns The array of SecretMetadata instances.
-   */
-  static parseSecretsFromMetadataStore<
-    DataType extends SecretDataType = Uint8Array,
-  >(
-    secretMetadataArr: Uint8Array[],
-    filterType?: SecretType,
-  ): SecretMetadata<DataType>[] {
-    const parsedSecertMetadata = secretMetadataArr.map((metadata) =>
-      SecretMetadata.fromRawMetadata<DataType>(metadata),
-    );
-
-    const secrets = SecretMetadata.sort(parsedSecertMetadata);
-
-    if (filterType) {
-      return secrets.filter((secret) => secret.type === filterType);
-    }
-
-    return secrets;
-  }
-
   /**
    * Parse and create the SecretMetadata instance from the raw metadata bytes.
    *
@@ -183,23 +124,35 @@ export class SecretMetadata<DataType extends SecretDataType = Uint8Array>
   }
 
   /**
-   * Sort the seed phrases by timestamp.
-   *
-   * @param data - The secret metadata array to sort.
-   * @param order - The order to sort the seed phrases. Default is `desc`.
+   * Compare two SecretMetadata instances by timestamp.
    *
-   * @returns The sorted secret metadata array.
+   * @param a - The first SecretMetadata instance.
+   * @param b - The second SecretMetadata instance.
+   * @param order - The sort order. Default is 'asc'.
+   * @returns A negative number if a < b, positive if a > b, zero if equal.
    */
-  static sort<DataType extends SecretDataType = Uint8Array>(
-    data: SecretMetadata<DataType>[],
+  static compareByTimestamp<DataType extends SecretDataType = SecretDataType>(
+    a: SecretMetadata<DataType>,
+    b: SecretMetadata<DataType>,
     order: 'asc' | 'desc' = 'asc',
-  ): SecretMetadata<DataType>[] {
-    return data.sort((a, b) => {
-      if (order === 'asc') {
-        return a.timestamp - b.timestamp;
-      }
-      return b.timestamp - a.timestamp;
-    });
+  ): number {
+    return order === 'asc'
+      ? a.timestamp - b.timestamp
+      : b.timestamp - a.timestamp;
+  }
+
+  /**
+   * Check if a SecretMetadata instance matches the given type.
+   *
+   * @param secret - The SecretMetadata instance to check.
+   * @param type - The type to match against.
+   * @returns True if the secret matches the type.
+   */
+  static matchesType<DataType extends SecretDataType = SecretDataType>(
+    secret: SecretMetadata<DataType>,
+    type: SecretType,
+  ): boolean {
+    return secret.type === type;
   }
 
   get data(): DataType {