@exodus/bitcoin-api 4.9.4 → 4.9.5

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.9.5](https://github.com/ExodusMovement/assets/compare/@exodus/bitcoin-api@4.9.4...@exodus/bitcoin-api@4.9.5) (2026-02-14)
+
+
+### Bug Fixes
+
+
+* fix: Remove ordinals paths from bitcoin monitor scanner (#7430)
+
+
+
 ## [4.9.4](https://github.com/ExodusMovement/assets/compare/@exodus/bitcoin-api@4.9.3...@exodus/bitcoin-api@4.9.4) (2026-02-10)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@exodus/bitcoin-api",
-  "version": "4.9.4",
+  "version": "4.9.5",
   "description": "Bitcoin transaction and fee monitors, RPC with the blockchain node, other networking code.",
   "type": "module",
   "main": "src/index.js",
@@ -61,5 +61,5 @@
     "type": "git",
     "url": "git+https://github.com/ExodusMovement/assets.git"
   },
-  "gitHead": "b4d4b4bf36591dca5c82bfacfbac80db41805e02"
+  "gitHead": "8c4fcde522beb978df6a5099e0fa9e34bb4a7e04"
 }

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

@@ -5,9 +5,7 @@ import ms from 'ms'
 
 import { isChangeAddress, isReceiveAddress } from '../address-utils.js'
 import { orderTxs } from '../insight-api-client/util.js'
-import { getOrdinalAddress } from '../ordinals-utils.js'
-import { getOrdinalsUtxos, getUtxos, partitionUtxos } from '../utxos-utils.js'
-import { indexOrdinalUnconfirmedTx } from './ordinals-indexer-utils.js'
+import { getUtxos } from '../utxos-utils.js'
 
 const { compact, isEqual, uniq } = lodash
 
@@ -23,8 +21,6 @@ export class BitcoinMonitorScanner {
   #txFetchLimitResolver
   #shouldExcludeVoutUtxo
   #yieldToUI
-  #ordinalsEnabled
-  #ordinalChainIndex
   #extraChainIndexEnabled
   #extraChainIndex
   #gapLimit
@@ -36,8 +32,6 @@ export class BitcoinMonitorScanner {
     yieldToUI = () => {},
     shouldExcludeVoutUtxo = () => false,
     txFetchLimitResolver = ({ refresh }) => (refresh ? 50 : 10),
-    ordinalsEnabled,
-    ordinalChainIndex,
     extraChainIndexEnabled,
     extraChainIndex,
     gapLimit = 10,
@@ -57,8 +51,6 @@ export class BitcoinMonitorScanner {
     this.#assetClientInterface = assetClientInterface
     this.#txFetchLimitResolver = txFetchLimitResolver
     this.#shouldExcludeVoutUtxo = shouldExcludeVoutUtxo
-    this.#ordinalsEnabled = ordinalsEnabled
-    this.#ordinalChainIndex = ordinalChainIndex
     this.#extraChainIndexEnabled = extraChainIndexEnabled
     this.#extraChainIndex = extraChainIndex
     this.#gapLimit = gapLimit
@@ -92,9 +84,6 @@ export class BitcoinMonitorScanner {
     const multiAddressMode = assetConfig.multiAddressMode ?? true
 
     const storedUtxos = getUtxos({ asset, accountState })
-    const storedOrdinalUtxos = getOrdinalsUtxos({ asset, accountState })
-
-    const currentStoredUtxos = storedUtxos.union(storedOrdinalUtxos)
 
     const currentTime = Date.now()
     const unconfirmedTxsToCheck = [...currentTxs].reduce((txs, tx) => {
@@ -116,9 +105,7 @@ export class BitcoinMonitorScanner {
       : (txs) =>
           txs.filter((tx) => {
             const txItem = currentTxs.get(tx.txid)
-            const confirmed = txItem && txItem.confirmed
-            const inscriptionsIndexed = txItem?.data?.inscriptionsIndexed
-            return confirmed && (!this.#ordinalsEnabled || inscriptionsIndexed)
+            return txItem && txItem.confirmed
           }).length >= txs.length
 
     const unusedAddressIndexes = await assetClientInterface.getUnusedAddressIndexes({
@@ -254,28 +241,7 @@ export class BitcoinMonitorScanner {
       }
 
       const listOfTxs = await Promise.all(promises)
-      const insightTxs = listOfTxs.flat()
-      if (!this.#ordinalsEnabled) {
-        return insightTxs
-      }
-
-      return Promise.all(
-        insightTxs.map((tx) => {
-          try {
-            return indexOrdinalUnconfirmedTx({
-              tx,
-              currency: this.#asset.currency,
-              insightClient,
-            })
-          } catch (e) {
-            console.warn(
-              `Could not index ${asset.name} ordinal tx ${tx.txid} for wallet account ${walletAccount}. message: ${e.message}`,
-              e
-            )
-            return tx
-          }
-        })
-      )
+      return listOfTxs.flat()
     }
 
     const gapSearchParameters = newChains.map(({ purpose, chain }) => {
@@ -305,21 +271,6 @@ export class BitcoinMonitorScanner {
         }
       )
 
-      if (
-        fetchCount === 0 &&
-        this.#ordinalsEnabled &&
-        this.#ordinalChainIndex > 1 &&
-        purposes.includes(86)
-      ) {
-        // this is the ordinal address
-        chainObjects.push({
-          purpose: 86,
-          chainIndex: this.#ordinalChainIndex,
-          startAddressIndex: 0,
-          endAddressIndex: 1,
-        })
-      }
-
       if (
         fetchCount === 0 &&
         this.#extraChainIndexEnabled &&
@@ -373,10 +324,7 @@ export class BitcoinMonitorScanner {
             const addressString = String(address)
             const purposeToUpdate = purposeMap[addressString]
 
-            if (
-              metaAddressIndex === undefined ||
-              (metaChainIndex === this.#ordinalChainIndex && this.#ordinalChainIndex > 1)
-            ) {
+            if (metaAddressIndex === undefined) {
               return
             }
 
@@ -461,15 +409,6 @@ export class BitcoinMonitorScanner {
         currencies: { [assetName]: currency },
       }
 
-      if (this.#ordinalsEnabled) {
-        txLogItem.data = {
-          ...txLogItem.data,
-          inscriptionsIndexed: txItem.inscriptionsIndexed,
-          sentInscriptions: [],
-          receivedInscriptions: [],
-        }
-      }
-
       let from = []
 
       // if txItem.vin has an address that matches ours, means we've spent this tx
@@ -495,14 +434,6 @@ export class BitcoinMonitorScanner {
         txLogItem.coinAmount = txLogItem.coinAmount.sub(currency.defaultUnit(vin.value))
         isSent = true
         txLogItem.data.sent = []
-        if (this.#ordinalsEnabled && vin.inscriptions) {
-          txLogItem.data.sentInscriptions.push(
-            ...vin.inscriptions.map((i) => ({
-              ...i,
-              value: currency.defaultUnit(vin.value).toBaseNumber(),
-            }))
-          )
-        }
 
         // this is only used to exclude the utxos in the reducer which is why we don't care about the other fields
         utxosToRemove.push({
@@ -584,15 +515,6 @@ export class BitcoinMonitorScanner {
           amount: currency.defaultUnit(vout.value).toBaseString(),
         })
 
-        if (this.#ordinalsEnabled && vout.inscriptions) {
-          txLogItem.data.receivedInscriptions.push(
-            ...vout.inscriptions.map((i) => ({
-              ...i,
-              value: currency.defaultUnit(vout.value).toBaseNumber(),
-            }))
-          )
-        }
-
         // it was sent to us...
         const val = currency.defaultUnit(vout.value)
         txLogItem.coinAmount = txLogItem.coinAmount.add(val)
@@ -609,11 +531,6 @@ export class BitcoinMonitorScanner {
           ...(isCoinbase ? { isCoinbase: true } : undefined),
         }
 
-        if (this.#ordinalsEnabled) {
-          output.inscriptionsIndexed = txItem.inscriptionsIndexed
-          output.inscriptions = vout.inscriptions || []
-        }
-
         if (this.#shouldExcludeVoutUtxo({ asset, output, txItem, vout })) {
           utxosToRemove.push({
             address,
@@ -682,7 +599,7 @@ export class BitcoinMonitorScanner {
 
     const utxosToRemoveCol = UtxoCollection.fromArray(utxosToRemove, { currency })
     // Keep new utxos when they intersect with the stored utxos.
-    utxoCol = utxoCol.union(currentStoredUtxos).difference(utxosToRemoveCol)
+    utxoCol = utxoCol.union(storedUtxos).difference(utxosToRemoveCol)
 
     const pendingDropCandidates = Object.values(unconfirmedTxsToCheck)
     const verificationResults = await Promise.all(
@@ -727,7 +644,7 @@ export class BitcoinMonitorScanner {
     }
 
     // no changes, ignore
-    if (utxoCol.equals(currentStoredUtxos)) {
+    if (utxoCol.equals(storedUtxos)) {
       utxoCol = null
     }
 
@@ -737,28 +654,10 @@ export class BitcoinMonitorScanner {
       return !isEqual(chain, originalChain.chain)
     })
 
-    const ordinalAddress = await this.getOrdinalAddress({ walletAccount })
-
-    // latest account state, in case knownBalanceUtxoIds or mustAvoidUtxoIds gets updated in on another promise
-    const latestAccountState = await assetClientInterface.getAccountState({
-      assetName,
-      walletAccount,
-    })
-    const utxosData = utxoCol
-      ? partitionUtxos({
-          allUtxos: utxoCol,
-          ordinalsEnabled: this.#ordinalsEnabled,
-          ordinalAddress,
-          knownBalanceUtxoIds: latestAccountState.knownBalanceUtxoIds,
-          mustAvoidUtxoIds: latestAccountState.mustAvoidUtxoIds,
-          additionalInscriptions: latestAccountState.additionalInscriptions,
-        })
-      : {}
-
     return {
       txsToUpdate: existingTxs,
       txsToAdd: newTxs,
-      ...utxosData,
+      ...(utxoCol ? { utxos: utxoCol } : {}),
       changedUnusedAddressIndexes,
     }
   }
@@ -771,8 +670,6 @@ export class BitcoinMonitorScanner {
     const accountState = await aci.getAccountState({ assetName, walletAccount })
 
     const storedUtxos = getUtxos({ accountState, asset })
-    const storedOrdinalsUtxos = getOrdinalsUtxos({ accountState, asset })
-    const allStoredUtxos = storedUtxos.union(storedOrdinalsUtxos)
 
     const currentTxs = [...(await aci.getTxLog({ assetName, walletAccount }))]
 
@@ -819,38 +716,18 @@ export class BitcoinMonitorScanner {
       })
       .filter((tx) => Object.keys(tx).length > 1)
 
-    const txConfirmedUtxos = allStoredUtxos.updateConfirmations(confirmationsList)
-
-    const ordinalAddress = await this.getOrdinalAddress({ walletAccount })
-
-    // latest account state, in case knownBalanceUtxoIds or mustAvoidUtxoIds gets updated in on another promise
-    const latestAccountState = await aci.getAccountState({
-      assetName,
-      walletAccount,
+    // Check before updateConfirmations since it mutates the shared utxo object references,
+    // which would cause storedUtxos.toJSON() to also reflect the updated confirmations.
+    const utxosChanged = storedUtxos.toArray().some((utxo) => {
+      const update = confirmationsList.find(({ txId }) => utxo.txId === txId)
+      return update && utxo.confirmations !== update.confirmations
     })
 
-    const { utxos, ordinalsUtxos } = partitionUtxos({
-      allUtxos: txConfirmedUtxos,
-      ordinalsEnabled: this.#ordinalsEnabled,
-      ordinalAddress,
-      knownBalanceUtxoIds: latestAccountState.knownBalanceUtxoIds,
-      mustAvoidUtxoIds: latestAccountState.mustAvoidUtxoIds,
-      additionalInscriptions: latestAccountState.additionalInscriptions,
-    })
+    const txConfirmedUtxos = storedUtxos.updateConfirmations(confirmationsList)
 
     return {
-      utxos: utxos.equals(storedUtxos) ? null : utxos,
-      ordinalsUtxos: ordinalsUtxos.equals(storedOrdinalsUtxos) ? null : ordinalsUtxos,
+      utxos: utxosChanged ? txConfirmedUtxos : null,
       txsToUpdate: updatedPropertiesTxs,
     }
   }
-
-  getOrdinalAddress({ walletAccount }) {
-    return getOrdinalAddress({
-      asset: this.#asset,
-      assetClientInterface: this.#assetClientInterface,
-      walletAccount,
-      ordinalChainIndex: this.#ordinalChainIndex,
-    })
-  }
 }

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

@@ -166,13 +166,12 @@ export class Monitor extends BaseMonitor {
     const assetName = asset.name
     const walletAccounts = await aci.getWalletAccounts({ assetName })
     for (const walletAccount of walletAccounts) {
-      const { txsToUpdate, utxos, ordinalsUtxos } = await this.#scanner.rescanOnNewBlock({
+      const { txsToUpdate, utxos } = await this.#scanner.rescanOnNewBlock({
         walletAccount,
       })
 
       const newData = {}
       if (utxos) newData.utxos = utxos
-      if (ordinalsUtxos) newData.ordinalsUtxos = ordinalsUtxos
 
       if (txsToUpdate.length > 0) {
         await this.updateTxLog({ assetName, walletAccount, logItems: txsToUpdate })
@@ -237,7 +236,7 @@ export class Monitor extends BaseMonitor {
     // wait for all wallet accounts to load
     await aci.getWalletAccounts({ assetName })
 
-    const { txsToAdd, txsToUpdate, utxos, ordinalsUtxos, changedUnusedAddressIndexes } =
+    const { txsToAdd, txsToUpdate, utxos, changedUnusedAddressIndexes } =
       await this.#scanner.rescanBlockchainInsight({
         walletAccount,
         refresh,
@@ -245,7 +244,6 @@ export class Monitor extends BaseMonitor {
 
     const newData = {}
     if (utxos) newData.utxos = utxos
-    if (ordinalsUtxos) newData.ordinalsUtxos = ordinalsUtxos
 
     if (!isEmpty(changedUnusedAddressIndexes)) {
       // Only for mobile atm, browser and hydra calculates from the latest txLogs