@exodus/bitcoin-api 4.8.2 → 4.9.0

package/CHANGELOG.md

@@ -3,6 +3,26 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [4.9.0](https://github.com/ExodusMovement/assets/compare/@exodus/bitcoin-api@4.8.3...@exodus/bitcoin-api@4.9.0) (2025-12-23)
+
+
+### Features
+
+
+* feat: Expose utxo sendTx workflow to accept provided unsigned tx (#6890)
+
+
+
+## [4.8.3](https://github.com/ExodusMovement/assets/compare/@exodus/bitcoin-api@4.8.2...@exodus/bitcoin-api@4.8.3) (2025-12-16)
+
+
+### Bug Fixes
+
+
+* fix: keep pending txs until /tx check fails for utxo coins (#7009)
+
+
+
 ## [4.8.2](https://github.com/ExodusMovement/assets/compare/@exodus/bitcoin-api@4.8.1...@exodus/bitcoin-api@4.8.2) (2025-12-04)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@exodus/bitcoin-api",
-  "version": "4.8.2",
+  "version": "4.9.0",
   "description": "Bitcoin transaction and fee monitors, RPC with the blockchain node, other networking code.",
   "type": "module",
   "main": "src/index.js",
@@ -60,5 +60,5 @@
     "type": "git",
     "url": "git+https://github.com/ExodusMovement/assets.git"
   },
-  "gitHead": "5440ba73907657281aa3c062168f4fda1f3b6581"
+  "gitHead": "39b743562bc1171bf3d145b47382db9023c660dd"
 }

package/src/index.js

@@ -16,6 +16,7 @@ export * from './unconfirmed-ancestor-data.js'
 export * from './parse-unsigned-tx.js'
 export { getCreateBatchTransaction } from './tx-send/batch-tx.js'
 export { createPsbtToUnsignedTx } from './psbt-utils.js'
+export { createTxFactory } from './tx-create/create-tx.js'
 export * from './insight-api-client/util.js'
 export * from './move-funds.js'
 export { createEncodeMultisigContract } from './multisig-address.js'

package/src/psbt-parser.js

@@ -318,7 +318,7 @@ function buildAddressPathsMap(selectedUtxos, changeOutputData) {
 function buildOutputAddressPurposesMap(outputs) {
   const map = Object.create(null)
   for (const output of outputs) {
-    if (output.address?.meta?.purpose) {
+    if (output.address?.meta?.purpose && output.address?.meta?.path) {
       map[output.address.address] = output.address.meta.purpose
     }
   }

package/src/tx-create/create-tx.js

@@ -1,3 +1,4 @@
+import { Psbt as DefaultPsbt, Transaction as DefaultTransaction } from '@exodus/bitcoinjs'
 import { Address, UtxoCollection } from '@exodus/models'
 import lodash from 'lodash'
 import assert from 'minimalistic-assert'
@@ -471,7 +472,8 @@ const transferHandler = {
       changeAddress: context.changeAddress,
     })
 
-    // Create a map of wallet's own output addresses and their purposes.
+    // Track our own outputs and their purposes. Today this is only the change
+    // output; in the future we may add self-send primaries when applicable.
     const outputAddressPurposesMap = Object.create(null)
 
     // Add the keypath of change address to support Trezor detect the change output.
@@ -517,13 +519,14 @@ export const createTxFactory =
     assetClientInterface,
     changeAddressType,
     allowedPurposes,
-    Psbt,
-    Transaction,
+    Psbt = DefaultPsbt,
+    Transaction = DefaultTransaction,
   }) =>
   async ({
     asset,
     walletAccount,
     type,
+    address, // same as toAddress.
     toAddress,
     amount,
     multipleAddressesEnabled,
@@ -532,7 +535,7 @@ export const createTxFactory =
     isSendAll,
     bumpTxId,
     isExchange,
-    isRbfAllowed,
+    isRbfAllowed = true,
     taprootInputWitnessSize,
   }) => {
     const assetName = asset.name
@@ -547,7 +550,7 @@ export const createTxFactory =
     return txHandler.buildTransaction({
       asset,
       walletAccount,
-      toAddress,
+      toAddress: toAddress ?? address,
       amount,
       multipleAddressesEnabled,
       feePerKB,

package/src/tx-log/bitcoin-monitor-scanner.js

@@ -669,7 +669,26 @@ export class BitcoinMonitorScanner {
     // Keep new utxos when they intersect with the stored utxos.
     utxoCol = utxoCol.union(currentStoredUtxos).difference(utxosToRemoveCol)
 
-    for (const tx of Object.values(unconfirmedTxsToCheck)) {
+    const pendingDropCandidates = Object.values(unconfirmedTxsToCheck)
+    const verificationResults = await Promise.all(
+      pendingDropCandidates.map(async (tx) => {
+        try {
+          const txStatus = await insightClient.fetchTx(tx.txId)
+          return { tx, shouldDrop: !txStatus }
+        } catch (err) {
+          console.log(
+            `${assetName}: failed to verify pending tx ${tx.txId} before drop check: ${
+              err?.message || 'unknown'
+            }`
+          )
+          return { tx, shouldDrop: false }
+        }
+      })
+    )
+
+    for (const { tx, shouldDrop } of verificationResults) {
+      if (!shouldDrop) continue
+
       existingTxs.push({ ...tx, dropped: true }) // TODO: this will decrease the chain index, it shouldn't be an issue considering the gap limit
       utxoCol = utxoCol.difference(utxoCol.getTxIdUtxos(tx.txId))
       const utxosToAdd = []

package/src/tx-send/index.js

@@ -3,7 +3,6 @@ import { Psbt as DefaultPsbt, Transaction as DefaultTransaction } from '@exodus/
 import assert from 'minimalistic-assert'
 
 import { extractTransactionContext } from '../psbt-parser.js'
-import { createTxFactory } from '../tx-create/create-tx.js'
 import { broadcastTransaction } from './broadcast-tx.js'
 import { updateAccountState, updateTransactionLog } from './update-state.js'
 
@@ -99,46 +98,6 @@ export async function signTransaction({
   return { rawTx, txId, tx }
 }
 
-function getTransferUnsignedTx(txContext) {
-  const {
-    inputs,
-    outputs,
-    psbtBuffer,
-    useCashAddress,
-    addressPathsMap,
-    outputAddressPurposesMap,
-    blockHeight,
-    rawTxs,
-    txType,
-    rbfEnabled,
-    bumpTxId,
-    changeOutputIndex,
-    sendOutputIndexes,
-  } = txContext
-
-  return {
-    unsignedTx: {
-      txData: {
-        inputs,
-        outputs,
-        psbtBuffer,
-      },
-      txMeta: {
-        useCashAddress,
-        addressPathsMap,
-        outputAddressPurposesMap,
-        blockHeight,
-        rawTxs,
-        txType,
-        rbfEnabled,
-        bumpTxId,
-        changeOutputIndex,
-        sendOutputIndexes,
-      },
-    },
-  }
-}
-
 // Get additional transaction metadata that are not part of the unsignedTx.txMeta
 function getExtendedTxMeta(txContext) {
   const {
@@ -170,64 +129,40 @@ function getExtendedTxMeta(txContext) {
 
 // not ported from Exodus; but this demos signing / broadcasting
 // NOTE: this will be ripped out in the coming weeks
-export const createAndBroadcastTXFactory =
-  ({
-    getFeeEstimator,
-    getSizeAndChangeScript = getSizeAndChangeScriptFactory(), // for decred customizations
-    allowUnconfirmedRbfEnabledUtxos,
-    utxosDescendingOrder,
-    assetClientInterface,
-    changeAddressType,
-    allowedPurposes,
-    Psbt = DefaultPsbt,
-    Transaction = DefaultTransaction,
-  }) =>
-  async ({
-    asset,
-    walletAccount,
-    address,
-    amount,
-    multipleAddressesEnabled,
-    feePerKB,
-    customFee,
-    isSendAll,
-    bumpTxId: bumpTxIdProvided,
-    isExchange,
-    isRbfAllowed = true,
-    taprootInputWitnessSize,
-  }) => {
+export const sendTxFactory = ({
+  createTx,
+  getSizeAndChangeScript = getSizeAndChangeScriptFactory(), // for decred customizations
+  assetClientInterface,
+  allowedPurposes,
+  Psbt = DefaultPsbt,
+  Transaction = DefaultTransaction,
+}) => {
+  assert(assetClientInterface, 'assetClientInterface is required')
+  assert(createTx, 'createTx is required')
+
+  return async ({ asset, walletAccount, unsignedTx: providedUnsignedTx, ...legacyParams }) => {
+    const { address, ...createTxParams } = legacyParams
     const assetName = asset.name
     const accountState = await assetClientInterface.getAccountState({ assetName, walletAccount })
 
-    const createTx = createTxFactory({
-      getFeeEstimator,
-      allowUnconfirmedRbfEnabledUtxos,
-      utxosDescendingOrder,
-      assetClientInterface,
-      changeAddressType,
-      allowedPurposes,
-      Psbt,
-      Transaction,
-    })
+    const resolveUnsignedTx = async () => {
+      if (providedUnsignedTx) {
+        return { unsignedTx: providedUnsignedTx }
+      }
 
-    const { unsignedTx: unsignedTxByCreateTx } = await createTx({
-      asset,
-      walletAccount,
-      type: 'transfer',
-      toAddress: address,
-      amount,
-      isRbfAllowed,
-      multipleAddressesEnabled,
-      feePerKB,
-      customFee,
-      isSendAll,
-      bumpTxId: bumpTxIdProvided,
-      isExchange,
-      taprootInputWitnessSize,
-    })
+      return createTx({
+        asset,
+        walletAccount,
+        type: 'transfer',
+        toAddress: address,
+        ...createTxParams,
+      })
+    }
+
+    const { unsignedTx } = await resolveUnsignedTx()
 
     const txContext = await extractTransactionContext({
-      unsignedTx: unsignedTxByCreateTx,
+      unsignedTx,
       asset,
       assetClientInterface,
       walletAccount,
@@ -236,8 +171,6 @@ export const createAndBroadcastTXFactory =
       Transaction,
     })
 
-    const { unsignedTx } = getTransferUnsignedTx(txContext)
-
     const {
       fee,
       sendAmount,
@@ -330,6 +263,7 @@ export const createAndBroadcastTXFactory =
       replacedTxId: replaceTx?.txId,
     }
   }
+}
 
 // back compatibiliy
 export { getSendDustValue as getDustValue } from '../dust.js'