@exodus/bitcoin-api 2.7.7 → 2.8.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@exodus/bitcoin-api",
-  "version": "2.7.7",
+  "version": "2.8.0",
   "description": "Exodus bitcoin-api",
   "main": "src/index.js",
   "files": [
@@ -32,7 +32,6 @@
     "coininfo": "5.1.0",
     "delay": "4.0.1",
     "ecpair": "2.0.1",
-    "querystring": "0.2.0",
     "socket.io-client": "2.1.1",
     "tiny-secp256k1": "1.1.3",
     "url-join": "4.0.0"
@@ -43,5 +42,5 @@
     "@scure/btc-signer": "^1.1.0",
     "jest-when": "^3.5.1"
   },
-  "gitHead": "4e504c29d0a5e436a5766d42d4664eaf53b06bc2"
+  "gitHead": "2b4d4989b82576d03b90f2eb7e53366a7a485df1"
 }

package/src/fee/can-bump-tx.js

@@ -3,7 +3,7 @@ import assert from 'minimalistic-assert'
 import { findUnconfirmedSentRbfTxs } from '../tx-utils'
 import { getUsableUtxos, getUtxos } from '../utxos-utils'
 
-import { BumpType } from '@exodus/bitcoin-lib/lib/selectors/get-can-bump-tx-factory'
+import { BumpType } from '@exodus/bitcoin-lib'
 
 export const ASSET_NAMES = ['bitcoin', 'bitcoinregtest', 'bitcointestnet']
 

package/src/insight-api-client/index.js

@@ -1,6 +1,5 @@
 /* global fetch */
 import urlJoin from 'url-join'
-import qs from 'querystring'
 import delay from 'delay'
 import { isEmpty } from 'lodash'
 
@@ -133,7 +132,7 @@ export default class InsightAPIClient {
     if (!Array.isArray(addrs) || addrs.length === 0) return { items: [], totalItems: 0 }
 
     options = { noScriptSig: 1, noAsm: 1, noSpent: 0, from: 0, to: 10, ...options }
-    const url = `${urlJoin(this._baseURL, `/addrs/txs`)}?${qs.stringify(options)}`
+    const url = `${urlJoin(this._baseURL, `/addrs/txs`)}?${new URLSearchParams(options)}`
 
     const fetchOptions = {
       method: 'post',

package/src/tx-send/index.js

@@ -5,7 +5,7 @@ import doShuffle from 'lodash/shuffle'
 import { UtxoCollection, Address } from '@exodus/models'
 import { retry } from '@exodus/simple-retry'
 import { selectUtxos } from '../fee/utxo-selector'
-import getTxSequence from '@exodus/bitcoin-lib/lib/utils/get-tx-sequence'
+import { getTxSequence } from '@exodus/bitcoin-lib'
 
 import { parseCurrency } from '../fee/fee-utils'
 

package/src/tx-sign/create-sign-with-wallet.js

@@ -33,8 +33,12 @@ export function createSignWithWallet({
       // dApps request to sign only specific transaction inputs.
       if (!inputInfo) continue
       const { address, sigHash } = inputInfo
-      // TODO: we can remove SIGHASH_ALL, it is default.
-      const sigHashTypes = sigHash !== undefined ? [sigHash || Transaction.SIGHASH_ALL] : undefined
+      // The sighash value from the PSBT input itself will be used.
+      // This list just represents possible sighash values the inputs can have.
+      const allowedSigHashTypes =
+        sigHash !== undefined
+          ? [sigHash, Transaction.SIGHASH_ALL] // `SIGHASH_DEFAULT` is a default safe sig hash, always allow it.
+          : undefined
       const { key, purpose, publicKey } = getKeyAndPurpose(address)
 
       const isP2SH = purpose === 49
@@ -67,7 +71,7 @@ export function createSignWithWallet({
 
       // desktop / BE / mobile with bip-schnorr signing
       const signingKey = isTaprootAddress ? tweakSigner({ signer: key, ECPair, network }) : key
-      await psbt.signInputAsync(index, toAsyncSigner({ keyPair: signingKey }), sigHashTypes)
+      await psbt.signInputAsync(index, toAsyncSigner({ keyPair: signingKey }), allowedSigHashTypes)
     }
   }
 }

package/src/utxos-utils.js

@@ -5,18 +5,32 @@ import assert from 'minimalistic-assert'
 
 const MAX_ORDINAL_VALUE_POSTAGE = 10000
 
+export const getInscriptionTxId = (inscriptionId) => {
+  return inscriptionId.split('i')[0]
+}
+
 export function getTransferOrdinalsUtxos({ inscriptionIds, ordinalsUtxos }) {
   const transferOrdinalsUtxos = ordinalsUtxos.filter((utxo) =>
     utxo.inscriptions?.some((i) => inscriptionIds.includes(i.inscriptionId))
   )
+
+  // this is for the micky mouse case. It has 2 inscriptions from the same tx but different output
+  const inscriptionTxs = inscriptionIds.map(getInscriptionTxId)
+
+  // https://ordinals.hiro.so/inscription/862ecd0fe343da32d19ff9277639ff71e10d894f55b6dee82dbfb9c158d5d30ci0
+  // https://ordinals.hiro.so/inscription/862ecd0fe343da32d19ff9277639ff71e10d894f55b6dee82dbfb9c158d5d30ci1
+
   const unsafeInscriptions = transferOrdinalsUtxos.toArray().flatMap(
     (utxo) =>
-      utxo.inscriptions?.filter((i) => {
+      utxo.inscriptions?.filter((inscription) => {
         const validInscription = isValidInscription({
           value: utxo.value.toBaseNumber(),
-          offset: i.offset,
+          offset: inscription.offset,
         })
-        return validInscription && !inscriptionIds.includes(i.inscriptionId)
+        return (
+          validInscription &&
+          !inscriptionTxs.includes(getInscriptionTxId(inscription.inscriptionId))
+        )
       }) || []
   )
   assert(
@@ -69,7 +83,7 @@ export function getValidInscriptions({ utxo }) {
   )
 }
 
-function isOrdinalUtxo({ utxo, ordinalsEnabled, knownBalanceUtxoIds }) {
+function isOrdinalUtxo({ utxo, ordinalsEnabled, knownBalanceUtxoIds, ordinalAddress }) {
   if (!ordinalsEnabled) {
     return false
   }
@@ -85,18 +99,28 @@ function isOrdinalUtxo({ utxo, ordinalsEnabled, knownBalanceUtxoIds }) {
   }
 
   const validInscriptions = getValidInscriptions({ utxo })
-  return validInscriptions.length > 0
+  const hasOrdinals = validInscriptions.length > 0
+
+  if (ordinalAddress?.toString() === utxo.address.toString()) {
+    // if this condition is true, that means this is taproot
+    // so we always return true to ensure we accidentally don't spend an ordinal as btc
+    if (!hasOrdinals) {
+      console.log('Excluding utxo from btc spending:', utxo.address.toString(), utxo)
+    }
+    return true // assume is ordinal just in case
+  }
+  return hasOrdinals
 }
 
-export function partitionUtxos({ allUtxos, ordinalsEnabled, knownBalanceUtxoIds }) {
+export function partitionUtxos({ allUtxos, ordinalsEnabled, knownBalanceUtxoIds, ordinalAddress }) {
   assert(allUtxos, 'allUtxos is required')
-  // assert(ordinalAddress, 'ordinalAddress is required') // not used atm we may need to tune by ordinalAddress when unconfirmed or rubbish inscriptions
+  if (ordinalsEnabled) assert(ordinalAddress, 'ordinalAddress is required')
   return {
     utxos: allUtxos.filter(
-      (utxo) => !isOrdinalUtxo({ utxo, ordinalsEnabled, knownBalanceUtxoIds })
+      (utxo) => !isOrdinalUtxo({ utxo, ordinalsEnabled, knownBalanceUtxoIds, ordinalAddress })
     ),
     ordinalsUtxos: allUtxos.filter((utxo) =>
-      isOrdinalUtxo({ utxo, ordinalsEnabled, knownBalanceUtxoIds })
+      isOrdinalUtxo({ utxo, ordinalsEnabled, knownBalanceUtxoIds, ordinalAddress })
     ),
   }
 }