refactor(core): wallets aggregation (#318)

tien · Nov 5, 2024 · ed4e82d · ed4e82d
1 parent 610dc67
commit ed4e82d
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 1 deletion.
diff --git a/.changeset/heavy-ads-pay.md b/.changeset/heavy-ads-pay.md
@@ -0,0 +1,5 @@
+---
+"@reactive-dot/core": patch
+---
+
+Ensure stable reference of wallets coming from wallet provider.
diff --git a/packages/core/src/actions/aggregate-wallets.ts b/packages/core/src/actions/aggregate-wallets.ts
@@ -7,7 +7,7 @@ export function aggregateWallets(
     providersOrWallets.map((walletOrProvider) =>
       walletOrProvider instanceof Wallet
         ? [walletOrProvider]
-        : walletOrProvider.getWallets(),
+        : walletOrProvider.getOrFetchWallets(),
     ),
   ).then((wallets) => wallets.flat());
 }
diff --git a/packages/core/src/wallets/provider.ts b/packages/core/src/wallets/provider.ts
@@ -1,5 +1,14 @@
 import type { Wallet } from "./wallet.js";
 
 export abstract class WalletProvider {
+  #wallets: Wallet[] | undefined;
+
   abstract getWallets(): Wallet[] | Promise<Wallet[]>;
+
+  /**
+   * @internal
+   */
+  async getOrFetchWallets() {
+    return (this.#wallets ??= await this.getWallets());
+  }
 }