@exodus/bitcoin-api 4.1.2 → 4.1.3

package/CHANGELOG.md

@@ -3,6 +3,18 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [4.1.3](https://github.com/ExodusMovement/assets/compare/@exodus/bitcoin-api@4.1.1...@exodus/bitcoin-api@4.1.3) (2025-10-14)
+
+
+### Bug Fixes
+
+
+* fix: remove isBip70 from bitcoin libs (#6660)
+
+* fix: remove unused bitcoin.api.prepareSendTx (#6662)
+
+
+
 ## [4.1.2](https://github.com/ExodusMovement/assets/compare/@exodus/bitcoin-api@4.1.1...@exodus/bitcoin-api@4.1.2) (2025-10-09)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@exodus/bitcoin-api",
-  "version": "4.1.2",
+  "version": "4.1.3",
   "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": "6237b0d98791ed9e6290260947068d2600bf8270"
+  "gitHead": "48e0d08e2fe044fb98d2fb16fc790f642a3c1eb2"
 }

package/src/fee/get-fee-resolver.js

@@ -2,13 +2,7 @@ import assert from 'minimalistic-assert'
 
 import { findUnconfirmedSentRbfTxs } from '../tx-utils.js'
 import { getUnconfirmedTxAncestorMap } from '../unconfirmed-ancestor-data.js'
-import {
-  getInscriptionIds,
-  getOrdinalsUtxos,
-  getTransferOrdinalsUtxos,
-  getUsableUtxos,
-  getUtxos,
-} from '../utxos-utils.js'
+import { getUsableUtxos, getUtxos } from '../utxos-utils.js'
 import { canBumpTx } from './can-bump-tx.js'
 import { getUtxosData } from './utxo-selector.js'
 
@@ -40,17 +34,9 @@ export class GetFeeResolver {
     amount,
     customFee,
     isSendAll,
-    nft, // sending one nft
     receiveAddress,
     taprootInputWitnessSize,
   }) => {
-    if (nft) {
-      assert(!amount, 'amount must not be provided when nft is provided!!!')
-      assert(!isSendAll, 'isSendAll must not be provided when nft is provided!!!')
-    }
-
-    const inscriptionIds = getInscriptionIds({ nft })
-
     const { fee, unspendableFee, extraFeeData } = this.#getUtxosData({
       asset,
       accountState,
@@ -60,7 +46,6 @@ export class GetFeeResolver {
       amount,
       customFee,
       isSendAll,
-      inscriptionIds,
       taprootInputWitnessSize,
     })
     return { fee, unspendableFee, extraFeeData }
@@ -75,7 +60,6 @@ export class GetFeeResolver {
     amount,
     customFee,
     isSendAll,
-    inscriptionIds,
     taprootInputWitnessSize,
   }) => {
     assert(asset, 'asset must be provided')
@@ -87,12 +71,6 @@ export class GetFeeResolver {
     const utxos = getUtxos({ accountState, asset })
     const unconfirmedTxAncestor = getUnconfirmedTxAncestorMap({ accountState })
 
-    const ordinalsUtxos = getOrdinalsUtxos({ accountState, asset })
-
-    const transferOrdinalsUtxos = inscriptionIds
-      ? getTransferOrdinalsUtxos({ inscriptionIds, ordinalsUtxos })
-      : undefined
-
     const usableUtxos = getUsableUtxos({
       asset,
       utxos,
@@ -110,8 +88,6 @@ export class GetFeeResolver {
       amount,
       feeRate: feePerKB,
       receiveAddress,
-      transferOrdinalsUtxos,
-      inscriptionIds,
       isSendAll,
       getFeeEstimator: this.#getFeeEstimator,
       allowUnconfirmedRbfEnabledUtxos: this.#allowUnconfirmedRbfEnabledUtxos,

package/src/fee/utxo-selector.js

@@ -11,11 +11,7 @@ const { sortBy } = lodash
 
 const MIN_RELAY_FEE = 1000
 
-const getBestReceiveAddresses = ({ asset, receiveAddress, inscriptionIds }) => {
-  if (inscriptionIds) {
-    return receiveAddress || 'P2TR'
-  }
-
+const getBestReceiveAddresses = ({ asset, receiveAddress }) => {
   if (receiveAddress === null) {
     return null
   }
@@ -38,20 +34,15 @@ export const selectUtxos = ({
   mustSpendUtxos,
   allowUnconfirmedRbfEnabledUtxos,
   unconfirmedTxAncestor,
-  inscriptionIds, // for each inscription transfer, we need to calculate one more input and one more output
-  transferOrdinalsUtxos, // to calculate the size of the input
   taprootInputWitnessSize,
   changeAddressType = 'P2PKH',
 }) => {
   const resolvedReceiveAddresses = getBestReceiveAddresses({
     asset,
     receiveAddress,
-    inscriptionIds,
   })
 
-  if (inscriptionIds) {
-    receiveAddresses.push(...inscriptionIds.map(() => resolvedReceiveAddresses))
-  } else if (receiveAddresses.length === 0) {
+  if (receiveAddresses.length === 0) {
     receiveAddresses.push(resolvedReceiveAddresses)
   }
 
@@ -66,7 +57,6 @@ export const selectUtxos = ({
   // We can only replace for a sendAll if only 1 replaceable tx and no unconfirmed utxos
   const confirmedUtxosArray = getConfirmedUtxos({ asset, utxos: usableUtxos }).toArray()
   const canReplace =
-    !inscriptionIds &&
     !mustSpendUtxos &&
     !disableReplacement &&
     replaceableTxs &&
@@ -86,7 +76,6 @@ export const selectUtxos = ({
       }
 
       const replaceFeeEstimator = getFeeEstimator(asset, { feePerKB, unconfirmedTxAncestor })
-      // how to avoid replace tx inputs when inputs are ordinals? !!!!
       const inputs = UtxoCollection.fromJSON(tx.data.inputs, { currency })
       const outputs = isSendAll
         ? tx.data.sent.map(({ address }) => address)
@@ -211,15 +200,10 @@ export const selectUtxos = ({
     selectedUtxosValue = selectedUtxosValue.add(newUtxo.value)
   }
 
-  let selectedUtxos = (transferOrdinalsUtxos || UtxoCollection.createEmpty({ currency })).union(
-    UtxoCollection.fromArray(selectedUtxosArray, { currency })
-  ) // extremelly important, orden must be kept!!! ordinals utxos go first!!!
+  let selectedUtxos = UtxoCollection.fromArray(selectedUtxosArray, { currency })
 
   // start figuring out fees
-  const outputs =
-    amount.isZero && !inscriptionIds
-      ? [changeAddressType]
-      : [...receiveAddresses, changeAddressType]
+  const outputs = amount.isZero ? [changeAddressType] : [...receiveAddresses, changeAddressType]
 
   let fee = feeEstimator({ inputs: selectedUtxos, outputs, taprootInputWitnessSize })
 
@@ -259,8 +243,6 @@ export const getUtxosData = ({
   disableReplacement,
   mustSpendUtxos,
   allowUnconfirmedRbfEnabledUtxos,
-  inscriptionIds,
-  transferOrdinalsUtxos,
   unconfirmedTxAncestor,
   utxosDescendingOrder,
   taprootInputWitnessSize,
@@ -283,8 +265,6 @@ export const getUtxosData = ({
     mustSpendUtxos,
     allowUnconfirmedRbfEnabledUtxos,
     unconfirmedTxAncestor,
-    inscriptionIds,
-    transferOrdinalsUtxos,
     utxosDescendingOrder,
     taprootInputWitnessSize,
     changeAddressType,

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

@@ -7,13 +7,7 @@ import { parseCurrency, serializeCurrency } from '../fee/fee-utils.js'
 import { selectUtxos } from '../fee/utxo-selector.js'
 import { findUnconfirmedSentRbfTxs } from '../tx-utils.js'
 import { getUnconfirmedTxAncestorMap } from '../unconfirmed-ancestor-data.js'
-import {
-  getInscriptionIds,
-  getOrdinalsUtxos,
-  getTransferOrdinalsUtxos,
-  getUsableUtxos,
-  getUtxos,
-} from '../utxos-utils.js'
+import { getUsableUtxos, getUtxos } from '../utxos-utils.js'
 import { createInputs, createOutput, getBlockHeight, getNonWitnessTxs } from './tx-create-utils.js'
 
 async function createUnsignedTx({
@@ -64,13 +58,11 @@ const transferHandler = {
     customFee,
     isSendAll,
     bumpTxId,
-    nft,
     isExchange,
     isRbfAllowed,
     taprootInputWitnessSize,
     accountState,
     feeData,
-    ordinalsEnabled,
     getFeeEstimator,
     allowUnconfirmedRbfEnabledUtxos,
     utxosDescendingOrder,
@@ -84,18 +76,11 @@ const transferHandler = {
     const blockHeight = providedBlockHeight || (await getBlockHeight({ assetName, insightClient }))
 
     const rbfEnabled =
-      providedRbfEnabled || (updatedFeeData.rbfEnabled && !isExchange && isRbfAllowed && !nft)
-
-    const inscriptionIds = getInscriptionIds({ nft })
-
-    assert(
-      ordinalsEnabled || !inscriptionIds,
-      'inscriptions cannot be sent when ordinalsEnabled=false '
-    )
+      providedRbfEnabled || (updatedFeeData.rbfEnabled && !isExchange && isRbfAllowed)
 
     const shuffle = (list) => {
       // Using full lodash.shuffle notation so it can be mocked with spyOn in tests
-      return inscriptionIds ? list : lodash.shuffle(list) // don't shuffle when sending ordinal!!!!
+      return lodash.shuffle(list)
     }
 
     assert(
@@ -107,11 +92,6 @@ const transferHandler = {
       'should not be called without either a receiving toAddress or to bump a tx'
     )
 
-    if (inscriptionIds) {
-      assert(!bumpTxId, 'only inscriptionIds or bumpTxId must be provided')
-      assert(toAddress, 'toAddress must be provided when sending ordinals')
-    }
-
     const useCashAddress = asset.address.isCashAddress?.(toAddress)
 
     const changeAddress = multipleAddressesEnabled
@@ -120,11 +100,6 @@ const transferHandler = {
 
     const txSet = await assetClientInterface.getTxLog({ assetName, walletAccount })
 
-    const currentOrdinalsUtxos = getOrdinalsUtxos({ accountState, asset })
-    const transferOrdinalsUtxos = inscriptionIds
-      ? getTransferOrdinalsUtxos({ inscriptionIds, ordinalsUtxos: currentOrdinalsUtxos })
-      : undefined
-
     const unconfirmedTxAncestor = getUnconfirmedTxAncestorMap({ accountState })
     const usableUtxos = getUsableUtxos({
       asset,
@@ -160,14 +135,14 @@ const transferHandler = {
       }
     }
 
-    const sendAmount = bumpTxId || transferOrdinalsUtxos ? asset.currency.ZERO : amount
+    const sendAmount = bumpTxId ? asset.currency.ZERO : amount
     const receiveAddress = bumpTxId
       ? replaceableTxs.length > 0
         ? null
         : changeAddressType
       : processedAddress
     const feeRate = updatedFeeData.feePerKB
-    const resolvedIsSendAll = (!rbfEnabled && feePerKB) || transferOrdinalsUtxos ? false : isSendAll
+    const resolvedIsSendAll = !rbfEnabled && feePerKB ? false : isSendAll
 
     let { selectedUtxos, fee, replaceTx } = selectUtxos({
       asset,
@@ -182,8 +157,6 @@ const transferHandler = {
       mustSpendUtxos: utxosToBump,
       allowUnconfirmedRbfEnabledUtxos,
       unconfirmedTxAncestor,
-      inscriptionIds,
-      transferOrdinalsUtxos,
       utxosDescendingOrder,
       taprootInputWitnessSize,
       changeAddressType,
@@ -207,7 +180,6 @@ const transferHandler = {
         return { ...to, amount: serializeCurrency(to.amount, asset.currency) }
       })
       selectedUtxos = selectedUtxos.union(
-        // how to avoid replace tx inputs when inputs are ordinals? !!!!
         UtxoCollection.fromJSON(replaceTx.data.inputs, { currency: asset.currency })
       )
     }
@@ -225,16 +197,8 @@ const transferHandler = {
     // Send output
     let sendOutput
     if (processedAddress) {
-      if (transferOrdinalsUtxos) {
-        outputs.push(
-          ...transferOrdinalsUtxos
-            .toArray()
-            .map((ordinalUtxo) => createOutput(assetName, processedAddress, ordinalUtxo.value))
-        )
-      } else {
-        sendOutput = createOutput(assetName, processedAddress, sendAmount)
-        outputs.push(sendOutput)
-      }
+      sendOutput = createOutput(assetName, processedAddress, sendAmount)
+      outputs.push(sendOutput)
     }
 
     const totalAmount = replaceTx
@@ -244,10 +208,7 @@ const transferHandler = {
         )
       : sendAmount
 
-    const change = selectedUtxos.value
-      .sub(totalAmount)
-      .sub(transferOrdinalsUtxos?.value || asset.currency.ZERO)
-      .sub(fee)
+    const change = selectedUtxos.value.sub(totalAmount).sub(fee)
     const dust = getChangeDustValue(asset)
     let ourAddress = replaceTx?.data?.changeAddress || changeAddress
     if (asset.address.toLegacyAddress) {
@@ -285,8 +246,6 @@ const transferHandler = {
       amount,
       change,
       totalAmount,
-      currentOrdinalsUtxos,
-      inscriptionIds,
       address: processedAddress,
       ourAddress,
       receiveAddress,
@@ -294,7 +253,6 @@ const transferHandler = {
       fee,
       usableUtxos,
       selectedUtxos,
-      transferOrdinalsUtxos,
       replaceTx,
       sendOutput,
       changeOutput,
@@ -305,7 +263,6 @@ const transferHandler = {
 
 export const createTxFactory =
   ({
-    ordinalsEnabled,
     getFeeEstimator,
     allowUnconfirmedRbfEnabledUtxos,
     utxosDescendingOrder,
@@ -325,7 +282,6 @@ export const createTxFactory =
     customFee,
     isSendAll,
     bumpTxId,
-    nft,
     isExchange,
     isRbfAllowed,
     taprootInputWitnessSize,
@@ -351,13 +307,11 @@ export const createTxFactory =
       customFee,
       isSendAll,
       bumpTxId,
-      nft,
       isExchange,
       isRbfAllowed,
       taprootInputWitnessSize,
       accountState,
       feeData,
-      ordinalsEnabled,
       getFeeEstimator,
       allowUnconfirmedRbfEnabledUtxos,
       utxosDescendingOrder,

package/src/tx-send/index.js

@@ -109,13 +109,11 @@ const getPrepareSendTransaction = async ({
   changeAddressType,
   getFeeEstimator,
   options,
-  ordinalsEnabled,
   rbfEnabled,
   utxosDescendingOrder,
   walletAccount,
 }) => {
   const createTx = createTxFactory({
-    ordinalsEnabled,
     getFeeEstimator,
     allowUnconfirmedRbfEnabledUtxos,
     utxosDescendingOrder,
@@ -142,21 +140,20 @@ export const createAndBroadcastTXFactory =
     getFeeEstimator,
     getSizeAndChangeScript = getSizeAndChangeScriptFactory(), // for decred customizations
     allowUnconfirmedRbfEnabledUtxos,
-    ordinalsEnabled = false,
     utxosDescendingOrder,
     assetClientInterface,
     changeAddressType,
   }) =>
   async ({ asset, walletAccount, address, amount, options }) => {
     // Prepare transaction
-    const { bumpTxId, nft, isExchange, isRbfAllowed = true } = options
+    const { bumpTxId, isExchange, isRbfAllowed = true } = options
 
     const assetName = asset.name
     const feeData = await assetClientInterface.getFeeConfig({ assetName })
     const accountState = await assetClientInterface.getAccountState({ assetName, walletAccount })
     const insightClient = asset.baseAsset.insightClient
 
-    const rbfEnabled = feeData.rbfEnabled && !isExchange && isRbfAllowed && !nft
+    const rbfEnabled = feeData.rbfEnabled && !isExchange && isRbfAllowed
 
     // blockHeight
     const blockHeight = await getBlockHeight({ assetName, insightClient })
@@ -171,7 +168,6 @@ export const createAndBroadcastTXFactory =
       changeAddressType,
       getFeeEstimator,
       options,
-      ordinalsEnabled,
       rbfEnabled,
       utxosDescendingOrder,
       walletAccount,
@@ -179,15 +175,12 @@ export const createAndBroadcastTXFactory =
     const {
       change,
       totalAmount,
-      currentOrdinalsUtxos,
-      inscriptionIds,
       ourAddress,
       receiveAddress,
       sendAmount,
       fee,
       usableUtxos,
       selectedUtxos,
-      transferOrdinalsUtxos,
       replaceTx,
       sendOutput,
       changeOutput,
@@ -265,7 +258,6 @@ export const createAndBroadcastTXFactory =
 
     const { script, size } = getSizeAndChangeScript({ assetName, tx, rawTx, changeUtxoIndex, txId })
 
-    // for ordinals, used to allow users spending change utxos even when unconfirmed and ordinals are unknown
     const knownBalanceUtxoIds = accountState.knownBalanceUtxoIds || []
     let remainingUtxos = usableUtxos.difference(selectedUtxos)
     if (changeUtxoIndex !== -1) {
@@ -288,16 +280,11 @@ export const createAndBroadcastTXFactory =
       remainingUtxos = remainingUtxos.difference(remainingUtxos.getTxIdUtxos(replaceTx.txId))
     }
 
-    const remainingOrdinalsUtxos = transferOrdinalsUtxos
-      ? currentOrdinalsUtxos.difference(transferOrdinalsUtxos)
-      : currentOrdinalsUtxos
-
     await assetClientInterface.updateAccountState({
       assetName,
       walletAccount,
       newData: {
         utxos: remainingUtxos,
-        ordinalsUtxos: remainingOrdinalsUtxos,
         knownBalanceUtxoIds,
       },
     })
@@ -335,10 +322,6 @@ export const createAndBroadcastTXFactory =
         return asset.currency.ZERO
       }
 
-      if (nft) {
-        return transferOrdinalsUtxos.value.abs().negate()
-      }
-
       return totalAmount.abs().negate()
     }
 
@@ -365,17 +348,6 @@ export const createAndBroadcastTXFactory =
             blocksSeen: 0,
             inputs: selectedUtxos.toJSON(),
             replacedTxId: replaceTx ? replaceTx.txId : undefined,
-            nftId: nft ? `${assetName}:${nft.tokenId}` : undefined, // it allows BE to load the nft info while the nft is in transit
-            inscriptionsIndexed: ordinalsEnabled ? true : undefined,
-            sentInscriptions: inscriptionIds
-              ? inscriptionIds.map((inscriptionId) => {
-                  return {
-                    inscriptionId,
-                    offset: 0,
-                    value: 0,
-                  }
-                })
-              : undefined,
           },
         },
       ],