@sundaeswap/wallet-lite 0.0.68 → 0.0.69

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sundaeswap/wallet-lite",
-  "version": "0.0.68",
+  "version": "0.0.69",
   "license": "MIT",
   "type": "module",
   "main": "./dist/cjs/index.js",

package/src/@types/events.ts

@@ -1,6 +1,7 @@
 import { IAssetAmountMetadata } from "@sundaeswap/asset";
 
 import { WalletBalanceMap } from "../classes/WalletBalanceMap.class.js";
+import { IWalletObserverSync } from "./observer.js";
 
 /**
  * A list of observer events that a client can hook into.
@@ -24,12 +25,7 @@ export interface IWalletObserverEventValues<
   [EWalletObserverEvents.SYNCING_WALLET_START]: undefined;
   [EWalletObserverEvents.SYNCING_WALLET_END]:
     | undefined
-    | {
-        balanceMap: WalletBalanceMap<T>;
-        network: number;
-        usedAddresses: string[];
-        unusedAddresses: string[];
-      };
+    | IWalletObserverSync<T>;
   [EWalletObserverEvents.CONNECT_WALLET_START]: undefined;
   [EWalletObserverEvents.CONNECT_WALLET_END]: undefined;
   [EWalletObserverEvents.GET_BALANCE_MAP_START]: undefined;

package/src/@types/observer.ts

@@ -93,9 +93,10 @@ export type TGetPeerConnectInstance = () => {
 export interface IWalletObserverSync<
   AssetMetadata extends IAssetAmountMetadata = IAssetAmountMetadata,
 > {
-  balanceMap: WalletBalanceMap<AssetMetadata>;
-  usedAddresses: string[];
-  unusedAddresses: string[];
-  utxos?: TransactionUnspentOutput[];
-  network: number;
+  balanceMap: Error | WalletBalanceMap<AssetMetadata>;
+  usedAddresses: Error | string[];
+  unusedAddresses: Error | string[];
+  utxos?: Error | TransactionUnspentOutput[];
+  collateral?: Error | TransactionUnspentOutput[];
+  network: Error | number;
 }

package/src/classes/WalletObserver.class.ts

@@ -129,18 +129,16 @@ export class WalletObserver<
       this._performingSync = true;
       this.dispatch(EWalletObserverEvents.SYNCING_WALLET_START);
 
-      let newNetwork: number;
-      try {
+      let newNetwork = await this.getNetwork();
+      if (newNetwork instanceof Error) {
+        await this.syncApi();
         newNetwork = await this.getNetwork();
-      } catch (e) {
-        try {
-          await this.syncApi();
-          newNetwork = await this.getNetwork();
-        } catch (e) {
+
+        if (newNetwork instanceof Error) {
           this.dispatch(EWalletObserverEvents.SYNCING_WALLET_END);
           this.dispatch(EWalletObserverEvents.CONNECT_WALLET_END);
           this._performingSync = false;
-          throw e;
+          throw newNetwork;
         }
       }
 
@@ -312,9 +310,16 @@ export class WalletObserver<
 
     this.activeWallet = extension;
     if (this._options.persistence && api) {
+      const addresses = await this.getUsedAddresses();
+      if (addresses instanceof Error) {
+        addresses.cause =
+          "Could not get a list of used addresses from the wallet when trying to save the connection.";
+        throw addresses;
+      }
+
       const seed: IWalletObserverSeed = {
         activeWallet: extension,
-        mainAddress: (await this.getUsedAddresses())[0],
+        mainAddress: addresses[0],
       };
 
       window.localStorage.setItem(
@@ -375,7 +380,9 @@ export class WalletObserver<
    *
    * @returns {Promise<WalletBalanceMap<AssetMetadata>>} - A promise that resolves to a map of asset amounts keyed by asset IDs.
    */
-  getBalanceMap = async (): Promise<WalletBalanceMap<AssetMetadata>> => {
+  getBalanceMap = async (): Promise<
+    WalletBalanceMap<AssetMetadata> | Error
+  > => {
     if (!this.api) {
       throw new Error("Attempted to query balance without an API instance.");
     }
@@ -383,12 +390,17 @@ export class WalletObserver<
     const start = performance.now();
 
     this.dispatch(EWalletObserverEvents.GET_BALANCE_MAP_START);
-    const [cbor, { Serialization }, typedHex] = await Promise.all([
-      this.api.getBalance(),
+    const [{ Serialization }, typedHex] = await Promise.all([
       getCardanoCore(),
       getCardanoUtil(),
     ]);
 
+    let cbor: string;
+    try {
+      cbor = await this.api.getBalance();
+    } catch (e) {
+      return e as Error;
+    }
     const data = Serialization.Value.fromCbor(typedHex(cbor));
     const multiassetKeys = data.multiasset()?.keys() ?? [];
 
@@ -421,24 +433,30 @@ export class WalletObserver<
     if (this._options.debug) {
       console.log(`getBalanceMap: ${end - start}ms`);
     }
+
     return balanceMap;
   };
 
   /**
    * Gets the current network connection.
    *
-   * @returns {Promise<number>} The network ID.
+   * @returns {Promise<number | Error>} The network ID or an Error from the wallet.
    */
-  getNetwork = async (): Promise<number> => {
+  getNetwork = async (): Promise<number | Error> => {
     if (!this.api) {
       throw new Error("Attempted to query network without an API instance.");
     }
 
     const start = performance.now();
 
-    const val = await this.api.getNetworkId();
-    this.network = val;
+    let val: number;
+    try {
+      val = await this.api.getNetworkId();
+    } catch (e) {
+      return e as Error;
+    }
 
+    this.network = val;
     const end = performance.now();
     if (this._options.debug) {
       console.log(`getNetwork: ${end - start}ms`);
@@ -451,7 +469,7 @@ export class WalletObserver<
    *
    * @returns {Promise<string[]>} The list of addresses.
    */
-  getUsedAddresses = async (): Promise<string[]> => {
+  getUsedAddresses = async (): Promise<string[] | Error> => {
     if (!this.api) {
       throw new Error(
         "Attempted to query used addresses without an API instance.",
@@ -460,12 +478,18 @@ export class WalletObserver<
 
     const start = performance.now();
 
-    const [cbor, { Cardano }, typedHex] = await Promise.all([
-      this.api.getUsedAddresses(),
+    const [{ Cardano }, typedHex] = await Promise.all([
       getCardanoCore(),
       getCardanoUtil(),
     ]);
 
+    let cbor: string[];
+    try {
+      cbor = await this.api.getUsedAddresses();
+    } catch (e) {
+      return e as Error;
+    }
+
     const data = cbor.map((val) =>
       Cardano.Address.fromBytes(typedHex(val)).toBech32(),
     );
@@ -480,9 +504,9 @@ export class WalletObserver<
   /**
    * Gets a list of unused addresses, encoded as Bech32.
    *
-   * @returns {Promise<string[]>} The list of addresses.
+   * @returns {Promise<string[] | Error>} The list of addresses or an Error returned by the wallet.
    */
-  getUnusedAddresses = async (): Promise<string[]> => {
+  getUnusedAddresses = async (): Promise<string[] | Error> => {
     if (!this.api) {
       throw new Error(
         "Attempted to query unused addresses without an API instance.",
@@ -491,12 +515,18 @@ export class WalletObserver<
 
     const start = performance.now();
 
-    const [cbor, { Cardano }, typedHex] = await Promise.all([
-      this.api.getUnusedAddresses(),
+    const [{ Cardano }, typedHex] = await Promise.all([
       getCardanoCore(),
       getCardanoUtil(),
     ]);
 
+    let cbor: string[];
+    try {
+      cbor = await this.api.getUnusedAddresses();
+    } catch (e) {
+      return e as Error;
+    }
+
     const data = cbor.map((val) =>
       Cardano.Address.fromBytes(typedHex(val)).toBech32(),
     );
@@ -513,19 +543,27 @@ export class WalletObserver<
    *
    * @returns {Promise<TransactionUnspentOutput[]>} The list of TransactionUnspentOutputs.
    */
-  getUtxos = async (): Promise<TransactionUnspentOutput[] | undefined> => {
+  getUtxos = async (): Promise<
+    TransactionUnspentOutput[] | Error | undefined
+  > => {
     if (!this.api) {
       throw new Error("Attempted to query UTXOs without an API instance.");
     }
 
     const start = performance.now();
 
-    const [cbor, { Serialization }, typedHex] = await Promise.all([
-      this.api.getUtxos(),
+    const [{ Serialization }, typedHex] = await Promise.all([
       getCardanoCore(),
       getCardanoUtil(),
     ]);
 
+    let cbor: string[] | null;
+    try {
+      cbor = await this.api.getUtxos();
+    } catch (e) {
+      return e as Error;
+    }
+
     const data = cbor?.map((val) => {
       const txOutput = Serialization.TransactionUnspentOutput.fromCbor(
         typedHex(val),
@@ -547,30 +585,36 @@ export class WalletObserver<
   /**
    * Gets a list of wallet UTXOs suitable for collateral.
    *
-   * @returns {Promise<TransactionUnspentOutput[]>} The list of TransactionUnspentOutputs.
+   * @returns {Promise<TransactionUnspentOutput[] | Error | undefined>} The list of TransactionUnspentOutputs, if there are any, or an Error.
    */
-  getCollateral = async (): Promise<TransactionUnspentOutput[] | undefined> => {
+  getCollateral = async (): Promise<
+    TransactionUnspentOutput[] | Error | undefined
+  > => {
     if (!this.api) {
       throw new Error("Attempted to query UTXOs without an API instance.");
     }
 
     const start = performance.now();
 
-    const [cbor, { Serialization }, typedHex] = await Promise.all([
-      (async () => {
-        const funcCall =
-          this.api?.getCollateral ||
-          (this.api?.experimental.getCollateral as GetCollateral);
-        if (typeof funcCall !== "function") {
-          return [];
-        }
-
-        return await funcCall();
-      })(),
+    const [{ Serialization }, typedHex] = await Promise.all([
       getCardanoCore(),
       getCardanoUtil(),
     ]);
 
+    let cbor: string[] | null;
+    try {
+      const funcCall =
+        this.api?.getCollateral ||
+        (this.api?.experimental.getCollateral as GetCollateral);
+      if (typeof funcCall !== "function") {
+        cbor = [];
+      } else {
+        cbor = await funcCall();
+      }
+    } catch (e) {
+      return e as Error;
+    }
+
     const data = cbor?.map((val) => {
       const txOutput = Serialization.TransactionUnspentOutput.fromCbor(
         typedHex(val),

package/src/react-components/WalletObserverProvider/WalletObserverProvider.tsx

@@ -38,6 +38,7 @@ const WalletObserverProvider: FC<
 
   const derivedState = useDerivedState(observerRef.current, {
     usedAddresses: state.usedAddresses,
+    unusedAddresses: state.unusedAddresses,
   });
 
   // Memoize the context value

package/src/react-components/WalletObserverProvider/hooks/effects/useDerivedState.ts

@@ -5,22 +5,25 @@ import { useWalletObserverState } from "../useWalletObserverState.js";
 
 export const useDerivedState = (
   observer: WalletObserver,
-  state: Pick<ReturnType<typeof useWalletObserverState>, "usedAddresses">,
+  state: Pick<
+    ReturnType<typeof useWalletObserverState>,
+    "usedAddresses" | "unusedAddresses"
+  >,
 ) => {
   const [stakeAddress, setStakeAddress] = useState<string>();
 
   useEffect(() => {
-    if (state.usedAddresses.length === 0) {
-      return;
-    }
-
     observer.getUtils().then((utils) => {
-      setStakeAddress(utils.getBech32StakingAddress(state.usedAddresses[0]));
+      setStakeAddress(
+        utils.getBech32StakingAddress(
+          state.usedAddresses[0] || state.unusedAddresses[0],
+        ),
+      );
     });
-  }, [observer, state.usedAddresses[0]]);
+  }, [observer, state.usedAddresses[0], state.unusedAddresses[0]]);
 
   const memoizedDerivedState = useMemo(() => {
-    let mainAddress = state.usedAddresses[0];
+    let mainAddress = state.usedAddresses[0] || state.unusedAddresses[0];
     const persistentCache = window.localStorage.getItem(
       WalletObserver.PERSISTENCE_CACHE_KEY,
     );
@@ -35,7 +38,12 @@ export const useDerivedState = (
       stakeAddress,
       mainAddress,
     };
-  }, [observer, state.usedAddresses[0], stakeAddress]);
+  }, [
+    observer,
+    state.usedAddresses[0],
+    state.unusedAddresses[0],
+    stakeAddress,
+  ]);
 
   return memoizedDerivedState;
 };

package/src/react-components/WalletObserverProvider/hooks/useWalletObserverState.ts

@@ -1,4 +1,4 @@
-import type { TransactionUnspentOutput } from "@cardano-sdk/core/dist/cjs/Serialization/index.js";
+import type { TransactionUnspentOutput } from "@cardano-sdk/core/dist/cjs/Serialization/TransactionUnspentOutput.js";
 import { AssetAmount, IAssetAmountMetadata } from "@sundaeswap/asset";
 import { useCallback, useEffect, useState, useTransition } from "react";
 
@@ -32,6 +32,7 @@ export const useWalletObserverState = <
   const [collateral, setCollateral] = useState<TransactionUnspentOutput[]>();
   const [isCip45, setIsCip45] = useState(false);
   const [switching, setSwitching] = useState(false);
+  const [errorSyncing, setErrorSyncing] = useState(false);
   const [isPending, startTransition] = useTransition();
   const [willAutoConnect, setWillAutoConnect] = useState(
     Boolean(
@@ -68,66 +69,103 @@ export const useWalletObserverState = <
       return;
     }
 
-    const freshData = await observer.sync();
-
-    startTransition(() => {
-      setActiveWallet((prevWallet) =>
-        newWallet === prevWallet ? prevWallet : newWallet,
-      );
+    setIsCip45(newWallet.includes("p2p"));
+    setActiveWallet((prevWallet) =>
+      newWallet === prevWallet ? prevWallet : newWallet,
+    );
+
+    try {
+      const freshData = await observer.sync();
+
+      startTransition(() => {
+        const newBalanceMap = freshData.balanceMap;
+        if (newBalanceMap instanceof WalletBalanceMap) {
+          const newAdaBalance = newBalanceMap.get(ADA_ASSET_ID);
+          if (newAdaBalance) {
+            setAdaBalance((prevBalance) =>
+              prevBalance.amount === newAdaBalance.amount
+                ? prevBalance
+                : newAdaBalance,
+            );
+
+            setBalance((prevBalance) =>
+              areAssetMapsEqual(prevBalance, newBalanceMap)
+                ? prevBalance
+                : newBalanceMap,
+            );
+          }
+        } else {
+          setErrorSyncing(true);
+        }
 
-      const newAdaBalance = freshData.balanceMap.get(ADA_ASSET_ID);
-      if (newAdaBalance) {
-        setAdaBalance((prevBalance) =>
-          prevBalance.amount === newAdaBalance.amount
-            ? prevBalance
-            : newAdaBalance,
-        );
-      }
+        const newUsedAddresses = freshData.usedAddresses;
+        if (newUsedAddresses instanceof Array) {
+          setUsedAddresses((prevValue) => {
+            return JSON.stringify(prevValue) ===
+              JSON.stringify(newUsedAddresses)
+              ? prevValue
+              : newUsedAddresses;
+          });
+        } else {
+          setErrorSyncing(true);
+        }
 
-      setBalance((prevBalance) =>
-        areAssetMapsEqual(prevBalance, freshData.balanceMap)
-          ? prevBalance
-          : freshData.balanceMap,
-      );
-
-      setUsedAddresses((prevValue) =>
-        JSON.stringify(prevValue) === JSON.stringify(freshData.usedAddresses)
-          ? prevValue
-          : freshData.usedAddresses,
-      );
-
-      setUnusedAddresses((prevValue) =>
-        JSON.stringify(prevValue) === JSON.stringify(freshData.unusedAddresses)
-          ? prevValue
-          : freshData.unusedAddresses,
-      );
-
-      setNetwork((prevValue) =>
-        prevValue === freshData.network ? prevValue : freshData.network,
-      );
-
-      setUtxos((prevValue) => {
-        const prevValueRep = prevValue?.map((v) => v.toCbor());
-        const newValueRep = freshData.utxos?.map((v) => v.toCbor());
-        if (prevValueRep !== newValueRep) {
-          return freshData.utxos;
+        const newUnusedAddresses = freshData.unusedAddresses;
+        if (newUnusedAddresses instanceof Array) {
+          setUnusedAddresses((prevValue) =>
+            JSON.stringify(prevValue) === JSON.stringify(newUnusedAddresses)
+              ? prevValue
+              : newUnusedAddresses,
+          );
+        } else {
+          setErrorSyncing(true);
         }
 
-        return prevValue;
-      });
+        const newNetwork = freshData.network;
+        if (typeof newNetwork === "number") {
+          setNetwork((prevValue) =>
+            prevValue === newNetwork ? prevValue : newNetwork,
+          );
+        } else {
+          setErrorSyncing(true);
+        }
 
-      setCollateral((prevValue) => {
-        const prevValueRep = prevValue?.map((v) => v.toCbor());
-        const newValueRep = freshData.utxos?.map((v) => v.toCbor());
-        if (prevValueRep !== newValueRep) {
-          return freshData.utxos;
+        const newUtxos = freshData.utxos;
+        if (newUtxos instanceof Array) {
+          setUtxos((prevValue) => {
+            const prevValueRep = prevValue?.map((v) => v.toCbor());
+            const newValueRep = newUtxos?.map((v) => v.toCbor());
+            if (prevValueRep !== newValueRep) {
+              return newUtxos;
+            }
+
+            return prevValue;
+          });
+        } else {
+          setErrorSyncing(true);
         }
 
-        return prevValue;
+        const newCollateral = freshData.collateral;
+        if (newCollateral instanceof Array) {
+          setCollateral((prevValue) => {
+            const prevValueRep = prevValue?.map((v) => v.toCbor());
+            const newValueRep = newCollateral?.map((v) => v.toCbor());
+            if (prevValueRep !== newValueRep) {
+              return newCollateral;
+            }
+
+            return prevValue;
+          });
+        } else {
+          setErrorSyncing(true);
+        }
       });
-
-      setIsCip45(newWallet.includes("p2p"));
-    });
+    } catch (e) {
+      setErrorSyncing(true);
+      (e as Error).cause =
+        "The wallet through an error while the app was trying to sync with it. Please try again or contact your wallet provider.";
+      throw e;
+    }
   }, [observer, disconnect]);
 
   const connectWallet = useCallback(
@@ -178,6 +216,7 @@ export const useWalletObserverState = <
     disconnect,
     connectWallet,
     switching,
+    errorSyncing,
     setSwitching,
     isPending,
     willAutoConnect,

package/src/react-components/hooks/useWalletObserver.ts

@@ -40,6 +40,7 @@ export const useWalletObserver = <
       disconnect: state.disconnect,
       connectWallet: state.connectWallet,
       switching: state.switching,
+      errorSyncing: state.errorSyncing,
       isPending: state.isPending,
       willAutoConnect: state.willAutoConnect,
     }),
@@ -62,6 +63,7 @@ export const useWalletObserver = <
       state.switching,
       state.isPending,
       state.willAutoConnect,
+      state.errorSyncing,
     ],
   );