fix: return provider info with optional store

Author: arlert-armin

wallets/core/src/hub/provider/provider.test.ts

@@ -54,13 +54,37 @@ describe('check providers', () => {
 
     expect(allNamespaces.size).toBe(2);
   });
-  test("throw an error if store hasn't set and try to access .state() and .info()", () => {
+
+  test('throw error if state() is called before store initialization', () => {
     const provider = new Provider('garbage', namespacesMap, {
       info: garbageWalletInfo,
     });
 
     expect(() => provider.state()).toThrowError();
-    expect(() => provider.info()).toThrowError();
+  });
+
+  test('should return wallet info via info() without store initialization', () => {
+    const provider = new Provider('garbage', namespacesMap, {
+      info: garbageWalletInfo,
+    });
+
+    expect(provider.info()).toBe(garbageWalletInfo);
+  });
+
+  test('should return wallet info via info() after store initialization', () => {
+    const store = createStore();
+    const provider = new Provider(
+      'garbage',
+      namespacesMap,
+      {
+        info: garbageWalletInfo,
+      },
+      { store }
+    );
+
+    expect(provider.info()).toBe(
+      store.getState().providers.list['garbage'].config.info
+    );
   });
 
   test('access state correctly', () => {

wallets/core/src/hub/provider/provider.ts

@@ -42,6 +42,10 @@ export class Provider {
     this.#extendInternalActions = options?.extendInternalActions || {};
     this.#namespaces = namespaces;
 
+    /**
+     * Our assumption is that the store will be optional for the provider.
+     * If a store is provided in the options, it will be initialized and set up.
+     */
     if (options?.store) {
       this.#store = options.store;
       this.#setupStore();
@@ -179,11 +183,8 @@ export class Provider {
   public info(): ProviderConfig['info'] | undefined {
     const store = this.#store;
     if (!store) {
-      throw new Error(
-        'You need to set your store using `.store` method first.'
-      );
+      return this.#configs.info;
     }
-
     return store.getState().providers.list[this.id].config.info;
   }
 

wallets/provider-slush/src/legacy/index.ts

@@ -2,12 +2,7 @@ import type { LegacyProviderInterface } from '@rango-dev/wallets-core/legacy';
 import type { Connect, WalletInfo } from '@rango-dev/wallets-shared';
 
 import { WalletTypes } from '@rango-dev/wallets-shared';
-import {
-  type BlockchainMeta,
-  type SignerFactory,
-  type SuiBlockchainMeta,
-  TransactionType,
-} from 'rango-types';
+import { type BlockchainMeta, type SignerFactory } from 'rango-types';
 
 import { suiWalletInstance } from '../utils.js';
 
@@ -31,40 +26,10 @@ const connect: Connect = async () => {
 export const getSigners: (provider: Provider) => Promise<SignerFactory> =
   signer;
 
-export const getWalletInfo: (allBlockChains: BlockchainMeta[]) => WalletInfo = (
-  allBlockChains
-) => {
-  const sui = allBlockChains.filter(
-    (blockchain) => blockchain.type === TransactionType.SUI
-  );
-
-  return {
-    name: 'Slush',
-    img: 'https://raw.githubusercontent.com/rango-exchange/assets/main/wallets/slush/icon.svg',
-    installLink: {
-      CHROME:
-        'https://chromewebstore.google.com/detail/slush-%E2%80%94-a-sui-wallet/opcgpfmipidbgpenhmajoajpbobppdil',
-      DEFAULT: 'https://slush.app/download',
-    },
-    color: '#4d40c6',
-    // if you are adding a new namespace, don't forget to also update `properties`
-    needsNamespace: {
-      selection: 'multiple',
-      data: [
-        {
-          label: 'Sui',
-          value: 'Sui',
-          id: 'SUI',
-          getSupportedChains: (allBlockchains: BlockchainMeta[]) =>
-            allBlockchains.filter(
-              (chain): chain is SuiBlockchainMeta =>
-                chain.type === TransactionType.SUI
-            ),
-        },
-      ],
-    },
-    supportedChains: sui,
-  };
+export const getWalletInfo: (
+  allBlockChains: BlockchainMeta[]
+) => WalletInfo = () => {
+  throw new Error('not implemented');
 };
 
 const buildLegacyProvider: () => LegacyProviderInterface = () => ({

widget/playground/src/containers/FunctionalLayout/FunctionalLayout.utils.ts

@@ -46,9 +46,9 @@ export function getWalletsList(
       const namespacesProperty = providerProperties?.find(
         (property) => property.name === 'namespaces'
       );
-      const supportedChainsNames = namespacesProperty?.value.data
-        .flatMap((obj) => obj.chains)
-        .map((chain) => chain.name);
+      const supportedChainsNames = namespacesProperty?.value.data.flatMap(
+        (obj) => obj.id
+      );
 
       const supportedChains = blockchains?.filter((blockchain) =>
         supportedChainsNames?.includes(blockchain.name)