move function to snap-utils and add JSDocs

Author: GuillaumeRx

packages/snaps-controllers/src/interface/SnapInterfaceController.ts

@@ -467,14 +467,24 @@ export class SnapInterfaceController extends BaseController<
     );
   }
 
+  /**
+   * Get the selected account in the client.
+   *
+   * @returns The selected account.
+   */
   #getSelectedAccount() {
     return this.messagingSystem.call(
       'AccountsController:getSelectedMultichainAccount',
     );
   }
 
+  /**
+   * Set the selected account in the client.
+   *
+   * @param accountId - The account id.
+   */
   #setSelectedAccount(accountId: string) {
-    return this.messagingSystem.call(
+    this.messagingSystem.call(
       'AccountsController:setSelectedAccount',
       accountId,
     );

packages/snaps-controllers/src/interface/utils.test.tsx

@@ -74,20 +74,6 @@ describe('assertNameIsUnique', () => {
   });
 });
 
-describe('createAddressList', () => {
-  it('creates an address list from an account', () => {
-    const result = createAddressList(
-      '0x1234567890123456789012345678901234567890',
-      ['eip155:1', 'eip155:2'],
-    );
-
-    expect(result).toStrictEqual([
-      'eip155:1:0x1234567890123456789012345678901234567890',
-      'eip155:2:0x1234567890123456789012345678901234567890',
-    ]);
-  });
-});
-
 describe('constructState', () => {
   const elementDataGetters = {
     getAssetsState: jest.fn(),

packages/snaps-controllers/src/interface/utils.ts

@@ -27,6 +27,7 @@ import type {
 import { isJSXElementUnsafe } from '@metamask/snaps-sdk/jsx';
 import type { InternalAccount } from '@metamask/snaps-utils';
 import {
+  createAddressList,
   getJsonSizeUnsafe,
   getJsxChildren,
   getJsxElementFromComponent,

packages/snaps-utils/src/address.test.ts

@@ -0,0 +1,15 @@
+import { createAddressList } from './address';
+
+describe('createAddressList', () => {
+  it('creates an address list from an account', () => {
+    const result = createAddressList(
+      '0x1234567890123456789012345678901234567890',
+      ['eip155:1', 'eip155:2'],
+    );
+
+    expect(result).toStrictEqual([
+      'eip155:1:0x1234567890123456789012345678901234567890',
+      'eip155:2:0x1234567890123456789012345678901234567890',
+    ]);
+  });
+});

packages/snaps-utils/src/address.ts

@@ -0,0 +1,19 @@
+import type { CaipAccountId, CaipChainId } from '@metamask/utils';
+import { parseCaipChainId, toCaipAccountId } from '@metamask/utils';
+
+/**
+ * Create a list of CAIP account IDs from an address and a list of scopes.
+ *
+ * @param address - The address to create the account IDs from.
+ * @param scopes - The scopes to create the account IDs from.
+ * @returns The list of CAIP account IDs.
+ */
+export function createAddressList(
+  address: string,
+  scopes: CaipChainId[],
+): CaipAccountId[] {
+  return scopes.map((scope) => {
+    const { namespace, reference } = parseCaipChainId(scope);
+    return toCaipAccountId(namespace, reference, address);
+  });
+}

packages/snaps-utils/src/index.ts

@@ -1,4 +1,5 @@
 export type * from './account';
+export * from './address';
 export * from './array';
 export * from './auxiliary-files';
 export * from './base64';