@exodus/bitcoin-api 4.15.8 → 4.15.9

package/CHANGELOG.md

@@ -3,6 +3,16 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [4.15.9](https://github.com/ExodusMovement/assets/compare/@exodus/bitcoin-api@4.15.8...@exodus/bitcoin-api@4.15.9) (2026-06-08)
+
+
+### Bug Fixes
+
+
+* fix(bitcoin-api): prevent dust-only CPFP bump transactions (#8218)
+
+
+
 ## [4.15.8](https://github.com/ExodusMovement/assets/compare/@exodus/bitcoin-api@4.15.7...@exodus/bitcoin-api@4.15.8) (2026-06-05)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@exodus/bitcoin-api",
-  "version": "4.15.8",
+  "version": "4.15.9",
   "description": "Bitcoin transaction and fee monitors, RPC with the blockchain node, other networking code.",
   "type": "module",
   "main": "src/index.js",
@@ -63,5 +63,5 @@
     "type": "git",
     "url": "git+https://github.com/ExodusMovement/assets.git"
   },
-  "gitHead": "0d0738ee6470d53801719742d165849969f70c5e"
+  "gitHead": "35f528942e03aa95179f6f8d6b42f1f2317cec5e"
 }

package/src/fee/utxo-selector.js

@@ -31,6 +31,18 @@ const canReplaceSendAll = ({ isSendAll, usableUtxos, confirmedUtxosArray, replac
   return unconfirmedUtxoCount === replaceableTxUtxoCount
 }
 
+const getCpfpOutputDust = ({ asset, amount, isBumpTx }) => {
+  if (!isBumpTx || !amount?.isZero) return asset.currency.ZERO
+  return getChangeDustValue(asset) ?? asset.currency.ZERO
+}
+
+const hasCpfpOutputAboveDust = ({ asset, selectedUtxos, amount, fee, isBumpTx }) => {
+  const cpfpOutputDust = getCpfpOutputDust({ asset, amount, isBumpTx })
+  if (cpfpOutputDust.isZero) return true
+
+  return selectedUtxos.value.sub(amount).sub(fee).gte(cpfpOutputDust)
+}
+
 export const selectUtxos = ({
   asset,
   utxosDescendingOrder,
@@ -103,14 +115,14 @@ export const selectUtxos = ({
       // as inputs of the new transaction;
       if (tx.data.sent.length === 0) continue
 
-      const changeUtxos = usableUtxos.getTxIdUtxos(tx.txId)
+      const internalUtxos = usableUtxos.getTxIdUtxos(tx.txId)
 
       // Do not replace a tx if any of its UTXOs were already spent by another tx.
       // Replacing would conflict with the child tx that spent the original output.
       // Use changeAddress as a legacy fallback for txs created before internalOutputs were introduced.
       if (
-        (tx.data.internalOutputs && tx.data.internalOutputs.length > changeUtxos.size) ||
-        (tx.data.changeAddress && changeUtxos.size === 0)
+        (tx.data.internalOutputs && tx.data.internalOutputs.length > internalUtxos.size) ||
+        (tx.data.changeAddress && internalUtxos.size === 0)
       ) {
         continue
       }
@@ -132,7 +144,7 @@ export const selectUtxos = ({
 
       let fee
       let additionalUtxos
-      const replaceTxAmount = changeUtxos.value.add(tx.feeAmount)
+      const replaceTxAmount = internalUtxos.value.add(tx.feeAmount)
 
       if (isSendAll) {
         additionalUtxos = UtxoCollection.fromArray(confirmedUtxosArray, { currency })
@@ -175,14 +187,24 @@ export const selectUtxos = ({
 
       if (replaceTxAmount.add(additionalUtxos.value).gte(amount.add(fee))) {
         const chainOutputs = isSendAll ? receiveAddresses : [...receiveAddresses, changeAddressType]
+        const chainUtxos = internalUtxos.union(additionalUtxos)
         const chainFee = feeEstimator({
-          inputs: changeUtxos.union(additionalUtxos),
+          inputs: chainUtxos,
           outputs: chainOutputs,
           taprootInputWitnessSize,
         })
-        // If batching is same or more expensive than chaining, don't replace
-        // Unless we are accelerating a tx with no internal outputs, then we must replace because it can't be chained
+        const hasValidCpfpChild = hasCpfpOutputAboveDust({
+          asset,
+          selectedUtxos: chainUtxos,
+          amount,
+          fee: chainFee,
+          isBumpTx,
+        })
+
+        // Only prefer CPFP when the child can keep a non-dust output and is cheaper than RBF/batching.
+        // If the original tx has no internal output, it cannot be chained, so RBF is required.
         if (
+          hasValidCpfpChild &&
           (!isBumpTx || tx.data.internalOutputs?.length || tx.data.changeAddress) &&
           fee.sub(tx.feeAmount).gte(chainFee)
         ) {
@@ -207,8 +229,8 @@ export const selectUtxos = ({
   if (replaceableTxs) {
     for (const tx of replaceableTxs) {
       if (!tx.data.internalOutputs?.length && !tx.data.changeAddress) continue
-      const changeUtxos = usableUtxos.getTxIdUtxos(tx.txId)
-      ourRbfUtxos = ourRbfUtxos.union(changeUtxos)
+      const internalUtxos = usableUtxos.getTxIdUtxos(tx.txId)
+      ourRbfUtxos = ourRbfUtxos.union(internalUtxos)
     }
   }
 
@@ -258,18 +280,23 @@ export const selectUtxos = ({
 
   // start figuring out fees
   const outputs = amount.isZero ? [changeAddressType] : [...receiveAddresses, changeAddressType]
+  const cpfpOutputDust = getCpfpOutputDust({ asset, amount, isBumpTx })
+  const totalOutputAmount = amount.add(cpfpOutputDust)
 
   let fee = feeEstimator({ inputs: selectedUtxos, outputs, taprootInputWitnessSize })
 
-  while (selectedUtxos.value.lt(amount.add(fee))) {
+  while (selectedUtxos.value.lt(totalOutputAmount.add(fee))) {
     // We ran out of UTXOs, give up now
     if (remainingUtxosArray.length === 0) {
       // Try fee with no change
-      fee = feeEstimator({
-        inputs: selectedUtxos,
-        outputs: receiveAddresses,
-        taprootInputWitnessSize,
-      })
+      if (cpfpOutputDust.isZero) {
+        fee = feeEstimator({
+          inputs: selectedUtxos,
+          outputs: receiveAddresses,
+          taprootInputWitnessSize,
+        })
+      }
+
       break
     }
 
@@ -278,7 +305,7 @@ export const selectUtxos = ({
     fee = feeEstimator({ inputs: selectedUtxos, outputs, taprootInputWitnessSize })
   }
 
-  if (selectedUtxos.value.lt(amount.add(fee))) {
+  if (selectedUtxos.value.lt(totalOutputAmount.add(fee))) {
     return { fee }
   }