@ledgerhq/live-common 21.16.1 → 21.16.2

package/src/__tests__/families/bitcoin/wallet-btc/utils.test.ts

@@ -1,6 +1,8 @@
 import { bech32m, bech32, BechLib } from "bech32";
 import * as utils from "../../../../families/bitcoin/wallet-btc/utils";
 import { Currency } from "../../../../families/bitcoin/wallet-btc/crypto/types";
+import cryptoFactory from "../../../../families/bitcoin/wallet-btc/crypto/factory";
+import { DerivationModes } from "../../../../families/bitcoin/wallet-btc";
 
 function validateAddrs(
   addresses: string[],
@@ -208,3 +210,106 @@ describe("Unit tests for various utils functions", () => {
     validateAddrs(["S6NMcEfYbavHrP3Uo1wbEUvKhAbKeMugaa"], "stealthcoin", false);
   });
 });
+
+describe("Unit tests for maxTxSize", () => {
+  const btc = cryptoFactory("bitcoin_testnet");
+  it("Empty tx non-segwit", () => {
+    const s = utils.maxTxSizeCeil(0, [], false, btc, DerivationModes.LEGACY);
+    expect(s).toEqual(10);
+  });
+  it("Empty tx segwit", () => {
+    const s = utils.maxTxSizeCeil(
+      0,
+      [],
+      false,
+      btc,
+      DerivationModes.NATIVE_SEGWIT
+    );
+    expect(s).toEqual(11);
+  });
+  it("1 p2wpkh input 1 p2wpkh output", () => {});
+
+  const tr = "tb1pdedptfps5k6hmzars9ks93ejsujh28kw5dhe06gutt9svap40j8qql2dw2";
+  const sh = "2Mv9bnjkBUr1GfCybJc7XmnJ5VG4SAT3shT";
+  const pkh = "mkQiYQGCkCC7Wus3vfhfTDjgJzQK554evn";
+  const wpkh = "tb1qhff3j7euu6t3lv8s5gsy9t5x82wuhfw5z863gj";
+  const p2tr = DerivationModes.TAPROOT;
+  const p2sh = DerivationModes.SEGWIT;
+  const p2pkh = DerivationModes.LEGACY;
+  const p2wpkh = DerivationModes.NATIVE_SEGWIT;
+
+  test.each([
+    // In 148 out 34
+    [0, [], false, p2pkh, 10],
+    [0, [], true, p2pkh, 44],
+    [0, [pkh], false, p2pkh, 44],
+    [0, [pkh], true, p2pkh, 78],
+    [1, [pkh], true, p2pkh, 226],
+    [2, [pkh], true, p2pkh, 374],
+    [1, [wpkh], true, p2pkh, 223],
+    [1, [sh], true, p2pkh, 224],
+    [1, [tr], true, p2pkh, 235],
+
+    // In 68 out 31
+    [0, [], false, p2wpkh, 11],
+    [0, [], true, p2wpkh, 42],
+    [1, [], false, p2wpkh, 79],
+    [1, [], true, p2wpkh, 110],
+    [2, [], true, p2wpkh, 178],
+
+    // In 90 out 32, scriptSig 22
+    [0, [], false, p2sh, 11],
+    [0, [], true, p2sh, 43],
+    [1, [], false, p2sh, 101],
+    [1, [sh], false, p2sh, 133],
+    [2, [sh], false, p2sh, 223],
+
+    // Fixed 10.5 In 57.75 out 43
+    [0, [], false, p2tr, 11],
+    [0, [], true, p2tr, 54],
+    [1, [], true, p2tr, Math.ceil(53.5 + 57.75 * 1)],
+    [2, [], true, p2tr, Math.ceil(53.5 + 57.75 * 2)],
+    [3, [], true, p2tr, Math.ceil(53.5 + 57.75 * 3)],
+
+    // 0xfc=252 inputs
+    [247, [], true, p2tr, Math.ceil(53.5 + 57.75 * 247)],
+    [248, [], true, p2tr, Math.ceil(53.5 + 57.75 * 248)],
+    [249, [], true, p2tr, Math.ceil(53.5 + 57.75 * 249)],
+    [250, [], true, p2tr, Math.ceil(53.5 + 57.75 * 250)],
+    [251, [], true, p2tr, Math.ceil(53.5 + 57.75 * 251)],
+    [252, [], true, p2tr, Math.ceil(53.5 + 57.75 * 252)],
+    // 0xfd=253 inputs will add 2 bytes for >= 0xfd
+    [253, [], true, p2tr, Math.ceil(53.5 + 57.75 * 253 + 2)],
+    [
+      256 * 256 - 1,
+      [],
+      true,
+      p2tr,
+      Math.ceil(53.5 + 57.75 * (256 * 256 - 1) + 2),
+    ],
+    // 0xffff inputs will add 4 bytes
+    [256 * 256, [], true, p2tr, Math.ceil(53.5 + 57.75 * (256 * 256) + 4)],
+
+    // test address with witness version 16 and witness program [0x01, 0x02] => tb1sqypqyuzpgh
+    [
+      2,
+      [tr, sh, pkh, wpkh, "tb1sqypqyuzpgh"],
+      false,
+      p2wpkh,
+      Math.ceil(10.5 + 68 * 2 + 43 + 32 + 34 + 31 + 13),
+    ],
+  ])(
+    "%i, %s, %s, %s => %i",
+    (inputCount, outputAddrs, useChange, derivationMode, exp) => {
+      const s = utils.maxTxSizeCeil(
+        inputCount,
+        outputAddrs,
+        useChange,
+        btc,
+        derivationMode
+      );
+      //const s = utils.estimateTxSize(inputCount, outputAddrs.length + (useChange ? 1 : 0), btc, derivationMode);
+      expect(s).toEqual(exp);
+    }
+  );
+});

package/src/__tests__/families/bitcoin/wallet-btc/wallet.estimateMaxSpendable.test.ts

@@ -57,9 +57,10 @@ describe("testing estimateMaxSpendable", () => {
     expect(maxSpendable.toNumber()).toEqual(
       balance -
         feesPerByte *
-          utils.estimateTxSize(
+          utils.maxTxSizeCeil(
             2,
-            1,
+            [],
+            true,
             account.xpub.crypto,
             account.xpub.derivationMode
           )

package/src/__tests__/families/bitcoin/wallet-btc/wallet.integration.test.ts

@@ -63,7 +63,7 @@ describe("testing wallet", () => {
       fromAccount: account,
       txInfo,
     });
-    expect(tx).toEqual("d6154bba4915d07453bb09d4fa02d62e975b4475");
+    expect(Buffer.from(tx, "hex")).toHaveLength(20);
   });
 
   it("should allow to build a transaction splitting outputs", async () => {
@@ -87,7 +87,7 @@ describe("testing wallet", () => {
       fromAccount: account,
       txInfo,
     });
-    expect(tx).toEqual("d922a70d6a66b96cdaba4565270a21f97eb23a6d");
+    expect(Buffer.from(tx, "hex")).toHaveLength(20);
   });
 
   it("should throw during sync if there is an error in explorer", async () => {

package/src/__tests__/families/bitcoin/wallet-btc/xpub.pickingStrategies.integration.test.ts

@@ -3,7 +3,10 @@ import * as bip39 from "bip39";
 import * as bitcoin from "bitcoinjs-lib";
 import coininfo from "coininfo";
 import BigNumber from "bignumber.js";
-import { DerivationModes } from "../../../../families/bitcoin/wallet-btc/types";
+import {
+  DerivationModes,
+  OutputInfo,
+} from "../../../../families/bitcoin/wallet-btc/types";
 import Xpub from "../../../../families/bitcoin/wallet-btc/xpub";
 import BitcoinLikeExplorer from "../../../../families/bitcoin/wallet-btc/explorer";
 import Crypto from "../../../../families/bitcoin/wallet-btc/crypto/bitcoin";
@@ -42,6 +45,17 @@ describe("testing xpub legacy transactions", () => {
     xpub,
   };
 
+  function outputs(amount: number): OutputInfo[] {
+    return [
+      {
+        address: "mwXTtHo8Yy3aNKUUZLkBDrTcKT9qG9TqLb",
+        isChange: false,
+        script: Buffer.from([]),
+        value: new BigNumber(amount),
+      },
+    ];
+  }
+
   it("merge output strategy should be correct", async () => {
     // Initialize the xpub with 2 txs. So that it has 2 utxo
     dataset.xpub.storage.appendTxs([
@@ -134,17 +148,15 @@ describe("testing xpub legacy transactions", () => {
     );
     let res = await utxoPickingStrategy.selectUnspentUtxosToUse(
       dataset.xpub,
-      new BigNumber(10000),
-      0,
-      1
+      outputs(10000),
+      0
     );
     expect(res.unspentUtxos.length).toEqual(1); // only 1 utxo is enough
     expect(Number(res.unspentUtxos[0].value)).toEqual(300000000); // use cheaper utxo first
     res = await utxoPickingStrategy.selectUnspentUtxosToUse(
       dataset.xpub,
-      new BigNumber(500000000),
-      0,
-      1
+      outputs(500000000),
+      0
     );
     expect(res.unspentUtxos.length).toEqual(2); // need 2 utxo
     expect(
@@ -161,17 +173,15 @@ describe("testing xpub legacy transactions", () => {
     );
     let res = await utxoPickingStrategy.selectUnspentUtxosToUse(
       dataset.xpub,
-      new BigNumber(10000),
-      0,
-      1
+      outputs(10000),
+      0
     );
     expect(res.unspentUtxos.length).toEqual(1); // only 1 utxo is enough
     expect(Number(res.unspentUtxos[0].value)).toEqual(5000000000); // use old utxo first
     res = await utxoPickingStrategy.selectUnspentUtxosToUse(
       dataset.xpub,
-      new BigNumber(5200000000),
-      0,
-      1
+      outputs(5200000000),
+      0
     );
     expect(res.unspentUtxos.length).toEqual(2); // need 2 utxo
     expect(
@@ -323,45 +333,40 @@ describe("testing xpub legacy transactions", () => {
     );
     let res = await utxoPickingStrategy.selectUnspentUtxosToUse(
       dataset.xpub,
-      new BigNumber(10000),
-      10,
-      1
+      outputs(10000),
+      10
     );
     expect(res.unspentUtxos.length).toEqual(1);
     expect(Number(res.unspentUtxos[0].value)).toEqual(100000000);
 
     res = await utxoPickingStrategy.selectUnspentUtxosToUse(
       dataset.xpub,
-      new BigNumber(290000000),
-      10,
-      1
+      outputs(290000000),
+      10
     );
     expect(res.unspentUtxos.length).toEqual(1);
     expect(Number(res.unspentUtxos[0].value)).toEqual(300000000);
 
     res = await utxoPickingStrategy.selectUnspentUtxosToUse(
       dataset.xpub,
-      new BigNumber(500000000),
-      10,
-      1
+      outputs(500000000),
+      10
     );
     expect(res.unspentUtxos.length).toEqual(1);
     expect(Number(res.unspentUtxos[0].value)).toEqual(600000000);
 
     res = await utxoPickingStrategy.selectUnspentUtxosToUse(
       dataset.xpub,
-      new BigNumber(800000000),
-      10,
-      1
+      outputs(800000000),
+      10
     );
     expect(res.unspentUtxos.length).toEqual(1);
     expect(Number(res.unspentUtxos[0].value)).toEqual(5000000000);
 
     res = await utxoPickingStrategy.selectUnspentUtxosToUse(
       dataset.xpub,
-      new BigNumber(5000000000),
-      10,
-      1
+      outputs(5000000000),
+      10
     );
     expect(res.unspentUtxos.length).toEqual(2);
     expect(
@@ -370,9 +375,8 @@ describe("testing xpub legacy transactions", () => {
 
     res = await utxoPickingStrategy.selectUnspentUtxosToUse(
       dataset.xpub,
-      new BigNumber(5600000000),
-      10,
-      1
+      outputs(5600000000),
+      10
     );
     expect(res.unspentUtxos.length).toEqual(3);
     expect(

package/src/__tests__/families/bitcoin/wallet-btc/xpub.txs.NP2WPKH.integration.test.ts

@@ -178,9 +178,10 @@ describe.skip("testing xpub segwit transactions", () => {
     await xpubs[1].xpub.sync();
 
     expectedFee1 =
-      utils.estimateTxSize(
+      utils.maxTxSizeCeil(
         inputs.length,
-        outputs.length,
+        outputs.map((o) => o.address),
+        false,
         crypto,
         DerivationModes.SEGWIT
       ) * 100;

package/src/__tests__/families/bitcoin/wallet-btc/xpub.txs.P2WPKH.integration.test.ts

@@ -167,9 +167,10 @@ describe.skip("testing xpub native segwit transactions", () => {
     await xpubs[1].xpub.sync();
 
     expectedFee1 =
-      utils.estimateTxSize(
+      utils.maxTxSizeCeil(
         inputs.length,
-        outputs.length,
+        outputs.map((o) => o.address),
+        false,
         crypto,
         DerivationModes.NATIVE_SEGWIT
       ) * 100;

package/src/__tests__/families/bitcoin/wallet-btc/xpub.txs.integration.test.ts

@@ -148,9 +148,10 @@ describe.skip("testing xpub legacy transactions", () => {
     }
 
     expectedFee1 =
-      utils.estimateTxSize(
+      utils.maxTxSizeCeil(
         inputs.length,
-        outputs.length,
+        outputs.map((o) => o.address),
+        false,
         crypto,
         DerivationModes.LEGACY
       ) * 100;
@@ -280,9 +281,10 @@ describe.skip("testing xpub legacy transactions", () => {
     await xpubs[1].xpub.sync();
 
     expectedFee2 =
-      utils.estimateTxSize(
+      utils.maxTxSizeCeil(
         inputs.length,
-        outputs.length,
+        outputs.map((o) => o.address),
+        false,
         crypto,
         DerivationModes.LEGACY
       ) * 100;

package/src/apps/polyfill.ts

@@ -49,6 +49,9 @@ export const whitelistDependencies = ["Decred", "Decred Testnet"];
   ["Opensea", "Ethereum"],
   ["StakeDAO", "Ethereum"],
   ["Yearn", "Ethereum"],
+  ["RocketPool", "Ethereum"],
+  ["POAP", "Ethereum"],
+  ["OlympusDAO", "Ethereum"],
 ].forEach(([name, dep]) => declareDep(name, dep));
 export const getDependencies = (appName: string): string[] =>
   directDep[appName] || [];

package/src/currencies/__snapshots__/sortByMarketcap.test.ts.snap

@@ -2671,6 +2671,7 @@ Array [
   "ethereum/erc20/ethereum_cell_network",
   "ethereum/erc20/ethereum_dark",
   "ethereum/erc20/ethereum_money",
+  "ethereum/erc20/ethereum_name_service",
   "ethereum/erc20/ethereum_pro",
   "ethereum/erc20/ethereum_push_notification_service",
   "ethereum/erc20/ethereum_stake",
@@ -2994,7 +2995,6 @@ Array [
   "ethereum/erc20/golden_goose",
   "ethereum/erc20/golden_ratio_coin",
   "ethereum/erc20/golden_token",
-  "ethereum/erc20/goldenratio",
   "ethereum/erc20/goldenugget_token",
   "ethereum/erc20/golder_coin",
   "ethereum/erc20/goldextoken",
@@ -3018,6 +3018,7 @@ Array [
   "ethereum/erc20/grandpa_fan",
   "ethereum/erc20/grap",
   "ethereum/erc20/grapefruit",
+  "ethereum/erc20/graph_token",
   "ethereum/erc20/graphlinq",
   "ethereum/erc20/graviton",
   "ethereum/erc20/grearn's_token",
@@ -3889,6 +3890,7 @@ Array [
   "ethereum/erc20/par_stablecoin",
   "ethereum/erc20/paralink_network",
   "ethereum/erc20/parallelchain_token",
+  "ethereum/erc20/paraswap",
   "ethereum/erc20/pareto",
   "ethereum/erc20/paribus",
   "ethereum/erc20/parsec_finance",

package/src/currencies/support.ts

@@ -9,6 +9,7 @@ import {
   hasCryptoCurrencyId,
 } from "@ledgerhq/cryptoassets";
 import { getEnv } from "../env";
+
 // set by user side effect to precise which currencies are considered supported (typically by live)
 let userSupportedCurrencies: CryptoCurrency[] = [];
 let userSupportedFiats: FiatCurrency[] = [];

package/src/families/bitcoin/js-buildTransaction.ts

@@ -44,7 +44,8 @@ export const buildTransaction = async (
     walletAccount,
     transaction.feePerByte.toNumber(), //!\ wallet-btc handles fees as JS number
     transaction.utxoStrategy.excludeUTXOs,
-    transaction.utxoStrategy.pickUnconfirmedRBF
+    transaction.utxoStrategy.pickUnconfirmedRBF,
+    [transaction.recipient]
   );
   log("btcwallet", "building transaction", transaction);
   const txInfo = await wallet.buildAccountTx({

package/src/families/bitcoin/js-estimateMaxSpendable.ts

@@ -30,7 +30,8 @@ const estimateMaxSpendable = async ({
     walletAccount,
     feePerByte.toNumber(), //!\ wallet-btc handles fees as JS number
     transaction?.utxoStrategy?.excludeUTXOs || [],
-    transaction?.utxoStrategy?.pickUnconfirmedRBF || false
+    transaction?.utxoStrategy?.pickUnconfirmedRBF || false,
+    transaction ? [transaction.recipient] : []
   );
   return maxSpendable.lt(0) ? new BigNumber(0) : maxSpendable;
 };

package/src/families/bitcoin/js-synchronisation.ts

@@ -139,8 +139,10 @@ const mapTxToOperations = (
     if (output.address) {
       if (!accountAddresses.includes(output.address)) {
         // The output doesn't belong to this account
-        if (output.output_index < changeOutputIndex) {
-          // The output isn't the change output
+        if (
+          tx.outputs.length === 1 || // The transaction has only 1 output
+          output.output_index < changeOutputIndex // The output isn't the change output
+        ) {
           recipients.push(
             syncReplaceAddress
               ? syncReplaceAddress(output.address)

package/src/families/bitcoin/wallet-btc/pickingstrategies/CoinSelect.ts

@@ -7,14 +7,14 @@ import { PickingStrategy } from "./types";
 import * as utils from "../utils";
 import { DeepFirst } from "./DeepFirst";
 import { log } from "@ledgerhq/logs";
+import { OutputInfo } from "..";
 
 export class CoinSelect extends PickingStrategy {
   // eslint-disable-next-line class-methods-use-this
   async selectUnspentUtxosToUse(
     xpub: Xpub,
-    amount: BigNumber,
-    feePerByte: number,
-    nbOutputsWithoutChange: number
+    outputs: OutputInfo[],
+    feePerByte: number
   ) {
     // get the utxos to use as input
     // from all addresses of the account
@@ -34,18 +34,30 @@ export class CoinSelect extends PickingStrategy {
     const TOTAL_TRIES = 100000;
     log("picking strategy", "utxos", unspentUtxos);
     // Compute cost of change
-    const fixedSize = utils.accurateTxSize(
-      0,
+    const fixedSize = utils.maxTxSize(
       0,
+      [],
+      false,
       this.crypto,
       this.derivationMode
     );
+    const outputAddresses = outputs.map((o) => o.address);
     // Size of only 1 output (without fixed size)
     const oneOutputSize =
-      utils.accurateTxSize(0, 1, this.crypto, this.derivationMode) - fixedSize;
+      outputAddresses.length > 0
+        ? utils.maxTxSize(
+            0,
+            [outputAddresses[0]],
+            false,
+            this.crypto,
+            this.derivationMode
+          ) - fixedSize
+        : utils.maxTxSize(0, [], true, this.crypto, this.derivationMode) -
+          fixedSize;
     // Size 1 signed UTXO (signed input)
     const oneInputSize =
-      utils.accurateTxSize(1, 0, this.crypto, this.derivationMode) - fixedSize;
+      utils.maxTxSize(1, [], false, this.crypto, this.derivationMode) -
+      fixedSize;
 
     // Calculate effective value of outputs
     let currentAvailableValue = 0;
@@ -70,12 +82,16 @@ export class CoinSelect extends PickingStrategy {
     // Get no inputs fees
     // At beginning, there are no outputs in tx, so noInputFees are fixed fees
     const notInputFees =
-      feePerByte * (fixedSize + oneOutputSize * nbOutputsWithoutChange);
+      feePerByte * (fixedSize + oneOutputSize * outputs.length);
 
     // Start coin selection algorithm (according to SelectCoinBnb from Bitcoin Core)
     let currentValue = 0;
     const currentSelection: boolean[] = [];
 
+    const amount = outputs.reduce(
+      (sum, output) => sum.plus(output.value),
+      new BigNumber(0)
+    );
     // Actual amount we are targetting
     const actualTarget = notInputFees + amount.toNumber();
 
@@ -102,9 +118,10 @@ export class CoinSelect extends PickingStrategy {
           currentSelectionNeedChangeoutput = true;
           currentWaste =
             feePerByte *
-            utils.estimateTxSize(
+            utils.maxTxSizeCeil(
               nbInput,
-              nbOutputsWithoutChange + 1,
+              outputAddresses,
+              true,
               this.crypto,
               this.derivationMode
             );
@@ -113,9 +130,10 @@ export class CoinSelect extends PickingStrategy {
           currentSelectionNeedChangeoutput = false;
           currentWaste =
             feePerByte *
-              utils.estimateTxSize(
+              utils.maxTxSizeCeil(
                 nbInput,
-                nbOutputsWithoutChange,
+                outputAddresses,
+                false,
                 this.crypto,
                 this.derivationMode
               ) +
@@ -170,11 +188,10 @@ export class CoinSelect extends PickingStrategy {
       }
       const fee =
         feePerByte *
-        utils.estimateTxSize(
+        utils.maxTxSizeCeil(
           unspentUtxoSelected.length,
-          bestSelectionNeedChangeoutput
-            ? nbOutputsWithoutChange + 1
-            : nbOutputsWithoutChange,
+          outputAddresses,
+          bestSelectionNeedChangeoutput,
           this.crypto,
           this.derivationMode
         );
@@ -191,11 +208,6 @@ export class CoinSelect extends PickingStrategy {
       this.derivationMode,
       this.excludedUTXOs
     );
-    return pickingStrategy.selectUnspentUtxosToUse(
-      xpub,
-      amount,
-      feePerByte,
-      nbOutputsWithoutChange
-    );
+    return pickingStrategy.selectUnspentUtxosToUse(xpub, outputs, feePerByte);
   }
 }

package/src/families/bitcoin/wallet-btc/pickingstrategies/DeepFirst.ts

@@ -6,14 +6,14 @@ import Xpub from "../xpub";
 import { PickingStrategy } from "./types";
 import * as utils from "../utils";
 import { log } from "@ledgerhq/logs";
+import { OutputInfo } from "..";
 
 export class DeepFirst extends PickingStrategy {
   // eslint-disable-next-line class-methods-use-this
   async selectUnspentUtxosToUse(
     xpub: Xpub,
-    amount: BigNumber,
-    feePerByte: number,
-    nbOutputsWithoutChange: number
+    outputs: OutputInfo[],
+    feePerByte: number
   ) {
     // get the utxos to use as input
     // from all addresses of the account
@@ -31,26 +31,47 @@ export class DeepFirst extends PickingStrategy {
         ).length
     );
 
+    const outputAddresses = outputs.map((o) => o.address);
     unspentUtxos = sortBy(unspentUtxos, "block_height");
     // https://metamug.com/article/security/bitcoin-transaction-fee-satoshi-per-byte.html
-    const txSizeNoInput = utils.accurateTxSize(
+    const txSizeNoInput = utils.maxTxSize(
       0,
-      nbOutputsWithoutChange,
+      outputAddresses,
+      false,
       this.crypto,
       this.derivationMode
     );
     let fee = txSizeNoInput * feePerByte;
+    const emptyTxSize = utils.maxTxSizeCeil(
+      0,
+      [],
+      false,
+      this.crypto,
+      this.derivationMode
+    );
     const sizePerInput =
-      utils.accurateTxSize(1, 0, this.crypto, this.derivationMode) -
-      utils.accurateTxSize(0, 0, this.crypto, this.derivationMode);
+      utils.maxTxSize(1, [], false, this.crypto, this.derivationMode) -
+      emptyTxSize;
 
     const sizePerOutput =
-      utils.accurateTxSize(0, 1, this.crypto, this.derivationMode) -
-      utils.accurateTxSize(0, 0, this.crypto, this.derivationMode);
+      (outputAddresses[0]
+        ? utils.maxTxSize(
+            0,
+            [outputAddresses[0]],
+            false,
+            this.crypto,
+            this.derivationMode
+          )
+        : utils.maxTxSize(0, [], true, this.crypto, this.derivationMode)) -
+      emptyTxSize;
 
     let total = new BigNumber(0);
     const unspentUtxoSelected: Output[] = [];
 
+    const amount = outputs.reduce(
+      (sum, output) => sum.plus(output.value),
+      new BigNumber(0)
+    );
     let i = 0;
     while (total.lt(amount.plus(fee))) {
       if (!unspentUtxos[i]) {