@ledgerhq/live-common 21.25.1 → 21.26.2

package/src/families/bitcoin/wallet-btc/crypto/decred.ts

@@ -74,8 +74,14 @@ class Decred extends Base {
     account: number,
     index: number
   ): string {
-    if (Base.addressCache[`${derivationMode}-${xpub}-${account}-${index}`]) {
-      return Base.addressCache[`${derivationMode}-${xpub}-${account}-${index}`];
+    if (
+      Base.addressCache[
+        `${this.network.name}-${derivationMode}-${xpub}-${account}-${index}`
+      ]
+    ) {
+      return Base.addressCache[
+        `${this.network.name}-${derivationMode}-${xpub}-${account}-${index}`
+      ];
     }
     let buffer: Buffer;
     try {
@@ -90,8 +96,9 @@ class Decred extends Base {
     const hd = new BIP32(X, chainCode, this.network, depth, i);
     const publicKey = hd.derive(account).derive(index).publicKey;
     const address = Decred.getAddressFromPk(publicKey);
-    Base.addressCache[`${derivationMode}-${xpub}-${account}-${index}`] =
-      address;
+    Base.addressCache[
+      `${this.network.name}-${derivationMode}-${xpub}-${account}-${index}`
+    ] = address;
     return address;
   }
 }

package/src/families/bitcoin/wallet-btc/crypto/litecoin.ts

@@ -59,8 +59,14 @@ class Litecoin extends Base implements ICrypto {
     account: number,
     index: number
   ): string {
-    if (Base.addressCache[`${derivationMode}-${xpub}-${account}-${index}`]) {
-      return Base.addressCache[`${derivationMode}-${xpub}-${account}-${index}`];
+    if (
+      Base.addressCache[
+        `${this.network.name}-${derivationMode}-${xpub}-${account}-${index}`
+      ]
+    ) {
+      return Base.addressCache[
+        `${this.network.name}-${derivationMode}-${xpub}-${account}-${index}`
+      ];
     }
     let address: string;
     switch (derivationMode) {
@@ -76,8 +82,9 @@ class Litecoin extends Base implements ICrypto {
       default:
         throw new Error("Should not be reachable");
     }
-    Base.addressCache[`${derivationMode}-${xpub}-${account}-${index}`] =
-      address;
+    Base.addressCache[
+      `${this.network.name}-${derivationMode}-${xpub}-${account}-${index}`
+    ] = address;
     return address;
   }
 

package/src/families/bitcoin/wallet-btc/crypto/zec.ts

@@ -53,12 +53,19 @@ class ZCash implements ICrypto {
     account: number,
     index: number
   ): string {
-    if (Base.addressCache[`${derivationMode}-${xpub}-${account}-${index}`]) {
-      return Base.addressCache[`${derivationMode}-${xpub}-${account}-${index}`];
+    if (
+      Base.addressCache[
+        `${this.network.name}-${derivationMode}-${xpub}-${account}-${index}`
+      ]
+    ) {
+      return Base.addressCache[
+        `${this.network.name}-${derivationMode}-${xpub}-${account}-${index}`
+      ];
     }
     const address = this.getLegacyAddress(xpub, account, index);
-    Base.addressCache[`${derivationMode}-${xpub}-${account}-${index}`] =
-      address;
+    Base.addressCache[
+      `${this.network.name}-${derivationMode}-${xpub}-${account}-${index}`
+    ] = address;
     return address;
   }
 

package/src/families/bitcoin/wallet-btc/crypto/zen.ts

@@ -73,12 +73,19 @@ class Zen implements ICrypto {
     account: number,
     index: number
   ): string {
-    if (Base.addressCache[`${derivationMode}-${xpub}-${account}-${index}`]) {
-      return Base.addressCache[`${derivationMode}-${xpub}-${account}-${index}`];
+    if (
+      Base.addressCache[
+        `${this.network.name}-${derivationMode}-${xpub}-${account}-${index}`
+      ]
+    ) {
+      return Base.addressCache[
+        `${this.network.name}-${derivationMode}-${xpub}-${account}-${index}`
+      ];
     }
     const address = this.getLegacyAddress(xpub, account, index);
-    Base.addressCache[`${derivationMode}-${xpub}-${account}-${index}`] =
-      address;
+    Base.addressCache[
+      `${this.network.name}-${derivationMode}-${xpub}-${account}-${index}`
+    ] = address;
     return address;
   }
 

package/src/families/ethereum/modules/send.ts

@@ -189,6 +189,9 @@ const send: ModeModule = {
       ];
     }
   },
+
+  // This is resolution config is necessary for plugins like Lido and stuff cause they use the send mode
+  getResolutionConfig: () => ({ erc20: true, externalPlugins: true }),
 };
 
 function serializeTransactionData(

package/src/families/tezos/bridge/js.ts

@@ -30,6 +30,7 @@ import { getEnv } from "../../../env";
 import { signOperation } from "../signOperation";
 import { patchOperationWithHash } from "../../../operation";
 import { log } from "@ledgerhq/logs";
+import { InvalidAddressBecauseAlreadyDelegated } from "../../../errors";
 
 const receive = makeAccountBridgeReceive();
 
@@ -70,54 +71,52 @@ const getTransactionStatus = async (
   const { tezosResources } = account;
   if (!tezosResources) throw new Error("tezosResources is missing");
 
-  if (!t.taquitoError) {
-    if (t.mode !== "undelegate") {
-      if (account.freshAddress === t.recipient) {
-        errors.recipient = new InvalidAddressBecauseDestinationIsAlsoSource();
-      } else {
-        const { recipientError, recipientWarning } = await validateRecipient(
-          account.currency,
-          t.recipient
-        );
+  if (t.mode !== "undelegate") {
+    if (account.freshAddress === t.recipient) {
+      errors.recipient = new InvalidAddressBecauseDestinationIsAlsoSource();
+    } else {
+      const { recipientError, recipientWarning } = await validateRecipient(
+        account.currency,
+        t.recipient
+      );
 
-        if (recipientError) {
-          errors.recipient = recipientError;
-        }
+      if (recipientError) {
+        errors.recipient = recipientError;
+      }
 
-        if (recipientWarning) {
-          warnings.recipient = recipientWarning;
-        }
+      if (recipientWarning) {
+        warnings.recipient = recipientWarning;
       }
     }
+  }
 
-    if (t.mode === "send") {
-      if (!errors.amount && t.amount.eq(0)) {
-        errors.amount = new AmountRequired();
-      } else if (!errors.amount && t.amount.lt(0)) {
-        errors.amount = new NotEnoughBalance();
-      } else if (t.amount.gt(0) && estimatedFees.times(10).gt(t.amount)) {
-        warnings.feeTooHigh = new FeeTooHigh();
-      }
+  if (t.mode === "send") {
+    if (!errors.amount && t.amount.eq(0)) {
+      errors.amount = new AmountRequired();
+    } else if (!errors.amount && t.amount.lt(0)) {
+      errors.amount = new NotEnoughBalance();
+    } else if (t.amount.gt(0) && estimatedFees.times(10).gt(t.amount)) {
+      warnings.feeTooHigh = new FeeTooHigh();
+    }
 
-      const thresholdWarning = 0.5 * 10 ** account.currency.units[0].magnitude;
-
-      if (
-        !errors.amount &&
-        account.balance
-          .minus(t.amount)
-          .minus(estimatedFees)
-          .lt(thresholdWarning)
-      ) {
-        if (isAccountDelegating(account)) {
-          if (t.useAllAmount) {
-            errors.amount = new RecommendUndelegation();
-          } else {
-            warnings.amount = new RecommendUndelegation();
-          }
+    const thresholdWarning = 0.5 * 10 ** account.currency.units[0].magnitude;
+
+    if (
+      !errors.amount &&
+      account.balance.minus(t.amount).minus(estimatedFees).lt(thresholdWarning)
+    ) {
+      if (isAccountDelegating(account)) {
+        if (t.useAllAmount) {
+          errors.amount = new RecommendUndelegation();
+        } else {
+          warnings.amount = new RecommendUndelegation();
         }
       }
     }
-  } else {
+  }
+
+  // if we also have taquitoError, we interprete them and they override the previously inferred errors
+  if (t.taquitoError) {
     log("taquitoerror", String(t.taquitoError));
 
     // remap taquito errors
@@ -127,7 +126,9 @@ const getTransactionStatus = async (
       } else {
         errors.amount = new NotEnoughBalanceToDelegate();
       }
-    } else {
+    } else if (t.taquitoError.endsWith("delegate.unchanged")) {
+      errors.recipient = new InvalidAddressBecauseAlreadyDelegated();
+    } else if (!errors.amount) {
       // unidentified error case
       errors.amount = new Error(t.taquitoError);
     }

package/src/families/tezos/hw-getAddress.ts

@@ -3,9 +3,7 @@ import type { Resolver } from "../../hw/getAddress/types";
 
 const resolver: Resolver = async (transport, { path, verify }) => {
   const xtz = new Xtz(transport);
-  const r = await xtz.getAddress(path, {
-    verify,
-  });
+  const r = await xtz.getAddress(path, { verify });
   return { ...r, path };
 };
 

package/src/families/tezos/synchronisation.ts

@@ -48,9 +48,10 @@ export const getAccountShape: GetAccountShape = async (infoInput) => {
     derivationMode,
   });
 
-  const initialStableOperations = initialAccount
-    ? initialAccount.operations
-    : [];
+  const initialStableOperations =
+    initialAccount && initialAccount.id === accountId
+      ? initialAccount.operations
+      : [];
 
   // fetch transactions, incrementally if possible
   const mostRecentStableOperation = initialStableOperations[0];