@bsv/sdk 1.2.15 → 1.2.18

package/src/transaction/BeefTx.ts

@@ -1,7 +1,7 @@
 import { hash256 } from '../primitives/Hash.js'
 import { Reader, Writer, toHex, toArray } from '../primitives/utils.js'
 import Transaction from './Transaction.js'
-import { BEEF_MAGIC, BEEF_MAGIC_TXID_ONLY_EXTENSION } from './Beef.js'
+import { BEEF_V2, TX_DATA_FORMAT } from './Beef.js'
 
 /**
  * A single bitcoin transaction associated with a `Beef` validity proof set.
@@ -42,20 +42,32 @@ export default class BeefTx {
 
   get txid () {
     if (this._txid) return this._txid
-    if (this._tx) return this._txid = this._tx.id('hex')
-    if (this._rawTx) return this._txid = toHex(hash256(this._rawTx))
+    if (this._tx) {
+      this._txid = this._tx.id('hex')
+      return this._txid
+    }
+    if (this._rawTx) {
+      this._txid = toHex(hash256(this._rawTx))
+      return this._txid
+    }
     throw new Error('Internal')
   }
 
   get tx () {
     if (this._tx) return this._tx
-    if (this._rawTx) return this._tx = Transaction.fromBinary(this._rawTx)
+    if (this._rawTx) {
+      this._tx = Transaction.fromBinary(this._rawTx)
+      return this._tx
+    }
     return undefined
   }
 
   get rawTx () {
     if (this._rawTx) return this._rawTx
-    if (this._tx) return this._rawTx = this._tx.toBinary()
+    if (this._tx) {
+      this._rawTx = this._tx.toBinary()
+      return this._rawTx
+    }
     return undefined
   }
 
@@ -66,87 +78,107 @@ export default class BeefTx {
   constructor (tx: Transaction | number[] | string, bumpIndex?: number) {
     if (typeof tx === 'string') {
       this._txid = tx
+    } else if (Array.isArray(tx)) {
+      this._rawTx = tx
     } else {
-      if (Array.isArray(tx)) {
-        this._rawTx = tx
-      } else {
-        this._tx = tx
-      }
+      this._tx = tx
     }
     this.bumpIndex = bumpIndex
     this.updateInputTxids()
   }
 
+  static fromTx (tx: Transaction, bumpIndex?: number): BeefTx {
+    return new BeefTx(tx, bumpIndex)
+  }
+
+  static fromRawTx (rawTx: number[], bumpIndex?: number): BeefTx {
+    return new BeefTx(rawTx, bumpIndex)
+  }
+
+  static fromTxid (txid: string, bumpIndex?: number): BeefTx {
+    return new BeefTx(txid, bumpIndex)
+  }
+
   private updateInputTxids () {
-    if (this.hasProof || !this.tx)
-    // If we have a proof, or don't have a parsed transaction
-    { this.inputTxids = [] } else {
+    if (this.hasProof || !this.tx) {
+      // If we have a proof, or don't have a parsed transaction
+      this.inputTxids = []
+    } else {
       const inputTxids = {}
       for (const input of this.tx.inputs) { inputTxids[input.sourceTXID] = true }
       this.inputTxids = Object.keys(inputTxids)
     }
   }
 
-  toWriter (writer: Writer, magic: number): void {
-    if (magic === BEEF_MAGIC) {
-      // V1
-      if (this.isTxidOnly) {
-        // Encode just the txid of a known transaction using the txid
-        writer.writeUInt32LE(BEEF_MAGIC_TXID_ONLY_EXTENSION)
-        writer.writeReverse(toArray(this._txid, 'hex'))
-      } else if (this._rawTx) { writer.write(this._rawTx) } else if (this._tx) { writer.write(this._tx.toBinary()) } else { throw new Error('a valid serialized Transaction is expected') }
+  toWriter (writer: Writer, version: number): void {
+    const writeByte = (bb: number) => {
+      writer.writeUInt8(bb)
+    }
+
+    const writeTxid = () => {
+      writer.writeReverse(toArray(this._txid, 'hex'))
+    }
+
+    const writeTx = () => {
+      if (this._rawTx) {
+        writer.write(this._rawTx)
+      } else if (this._tx) {
+        writer.write(this._tx.toBinary())
+      } else {
+        throw new Error('a valid serialized Transaction is expected')
+      }
+    }
+
+    const writeBumpIndex = () => {
       if (this.bumpIndex === undefined) {
-        writer.writeUInt8(0)
+        writeByte(TX_DATA_FORMAT.RAWTX) // 0
       } else {
-        writer.writeUInt8(1)
-        writer.writeVarIntNum(this.bumpIndex)
+        writeByte(TX_DATA_FORMAT.RAWTX_AND_BUMP_INDEX) // 1
+        writer.writeVarIntNum(this.bumpIndex) // the index of the associated bump
       }
-    } else {
-      // V2
+    }
+
+    if (version === BEEF_V2) {
       if (this.isTxidOnly) {
-        // Encode just the txid of a known transaction using the txid
-        writer.writeUInt8(2)
-        writer.writeReverse(toArray(this._txid, 'hex'))
+        writeByte(TX_DATA_FORMAT.TXID_ONLY)
+        writeTxid()
+      } else if (this.bumpIndex !== undefined) {
+        writeByte(TX_DATA_FORMAT.RAWTX_AND_BUMP_INDEX)
+        writer.writeVarIntNum(this.bumpIndex)
+        writeTx()
       } else {
-        if (this.bumpIndex === undefined) {
-          writer.writeUInt8(0)
-        } else {
-          writer.writeUInt8(1)
-          writer.writeVarIntNum(this.bumpIndex)
-        }
-        if (this._rawTx) { writer.write(this._rawTx) } else if (this._tx) { writer.write(this._tx.toBinary()) } else { throw new Error('a valid serialized Transaction is expected') }
+        writeByte(TX_DATA_FORMAT.RAWTX)
+        writeTx()
       }
+    } else {
+      writeTx()
+      writeBumpIndex()
     }
   }
 
-  static fromReader (br: Reader, magic: number): BeefTx {
-    let tx: Transaction | number[] | string | undefined
+  static fromReader (br: Reader, version: number): BeefTx {
+    let data: Transaction | number[] | string | undefined
     let bumpIndex: number | undefined
-
-    if (magic === BEEF_MAGIC) {
-      // V1
-      const version = br.readUInt32LE()
-      if (version === BEEF_MAGIC_TXID_ONLY_EXTENSION) {
-        // This is the extension to support known transactions
-        tx = toHex(br.readReverse(32))
-      } else {
-        br.pos -= 4 // Unread the version...
-        tx = Transaction.fromReader(br)
-      }
-      bumpIndex = br.readUInt8() ? br.readVarIntNum() : undefined
-    } else {
+    let beefTx: BeefTx | undefined
+    if (version === BEEF_V2) {
       // V2
       const format = br.readUInt8()
-      if (format === 2) {
-        // txid only
-        tx = toHex(br.readReverse(32))
+      if (format === TX_DATA_FORMAT.TXID_ONLY) {
+        beefTx = BeefTx.fromTxid(toHex(br.readReverse(32)))
       } else {
-        if (format === 1) { bumpIndex = br.readVarIntNum() }
-        tx = Transaction.fromReader(br)
+        if (format === TX_DATA_FORMAT.RAWTX_AND_BUMP_INDEX) {
+          bumpIndex = br.readVarIntNum()
+        }
+        data = Transaction.fromReader(br)
+        beefTx = BeefTx.fromTx(data, bumpIndex)
       }
+    } else {
+      // V1
+      data = Transaction.fromReader(br)
+      bumpIndex = br.readUInt8() ? br.readVarIntNum() : undefined
+      beefTx = BeefTx.fromTx(data, bumpIndex)
     }
 
-    const beefTx = new BeefTx(tx, bumpIndex)
     return beefTx
   }
 }

package/src/transaction/MerklePath.ts

@@ -50,7 +50,7 @@ export default class MerklePath {
     return MerklePath.fromBinary(toArray(hex, 'hex'))
   }
 
-  static fromReader (reader: Reader): MerklePath {
+  static fromReader (reader: Reader, legalOffsetsOnly: boolean = true): MerklePath {
     const blockHeight = reader.readVarIntNum()
     const treeHeight = reader.readUInt8()
     const path = Array(treeHeight).fill(0).map(() => ([]))
@@ -79,7 +79,7 @@ export default class MerklePath {
       }
       path[level].sort((a, b) => a.offset - b.offset)
     }
-    return new MerklePath(blockHeight, path)
+    return new MerklePath(blockHeight, path, legalOffsetsOnly)
   }
 
   /**
@@ -114,7 +114,7 @@ export default class MerklePath {
     hash?: string
     txid?: boolean
     duplicate?: boolean
-  }>>) {
+  }>>, legalOffsetsOnly: boolean = true) {
     this.blockHeight = blockHeight
     this.path = path
 
@@ -135,7 +135,7 @@ export default class MerklePath {
             }
           }
         } else {
-          if (!legalOffsets[height].has(leaf.offset)) {
+          if (legalOffsetsOnly && !legalOffsets[height].has(leaf.offset)) {
             throw new Error(`Invalid offset: ${leaf.offset}, at height: ${height}, with legal offsets: ${Array.from(legalOffsets[height]).join(', ')}`)
           }
         }

package/src/transaction/Transaction.ts

@@ -12,7 +12,7 @@ import Spend from '../script/Spend.js'
 import ChainTracker from './ChainTracker.js'
 import { defaultBroadcaster } from './broadcasters/DefaultBroadcaster.js'
 import { defaultChainTracker } from './chaintrackers/DefaultChainTracker.js'
-import { ATOMIC_BEEF, BEEF_MAGIC } from './Beef.js'
+import { ATOMIC_BEEF, BEEF_V1 } from './Beef.js'
 import P2PKH from '../script/templates/P2PKH.js'
 
 /**
@@ -59,6 +59,29 @@ export default class Transaction {
   merklePath?: MerklePath
   private cachedHash?: number[]
 
+  // Recursive function for adding merkle proofs or input transactions
+  private static addPathOrInputs (obj: { pathIndex?: number, tx: Transaction }, transactions: Record<string, {
+    pathIndex?: number
+    tx: Transaction
+  }>, BUMPs: MerklePath[]): void {
+    if (typeof obj.pathIndex === 'number') {
+      const path = BUMPs[obj.pathIndex]
+      if (typeof path !== 'object') {
+        throw new Error('Invalid merkle path index found in BEEF!')
+      }
+      obj.tx.merklePath = path
+    } else {
+      for (const input of obj.tx.inputs) {
+        const sourceObj = transactions[input.sourceTXID]
+        if (typeof sourceObj !== 'object') {
+          throw new Error(`Reference to unknown TXID in BEEF: ${input.sourceTXID}`)
+        }
+        input.sourceTransaction = sourceObj.tx
+        this.addPathOrInputs(sourceObj, transactions, BUMPs)
+      }
+    }
+  }
+
   /**
    * Creates a new transaction, linked to its inputs and their associated merkle paths, from a BEEF (BRC-62) structure.
    * Optionally, you can provide a specific TXID to retrieve a particular transaction from the BEEF data.
@@ -81,28 +104,7 @@ export default class Transaction {
       throw new Error(`Transaction with TXID ${targetTXID} not found in BEEF data.`)
     }
 
-    // Recursive function for adding merkle proofs or input transactions
-    const addPathOrInputs = (obj: { pathIndex?: number, tx: Transaction }): void => {
-      if (typeof obj.pathIndex === 'number') {
-        const path = BUMPs[obj.pathIndex]
-        if (typeof path !== 'object') {
-          throw new Error('Invalid merkle path index found in BEEF!')
-        }
-        obj.tx.merklePath = path
-      } else {
-        for (let i = 0; i < obj.tx.inputs.length; i++) {
-          const input = obj.tx.inputs[i]
-          const sourceObj = transactions[input.sourceTXID]
-          if (typeof sourceObj !== 'object') {
-            throw new Error(`Reference to unknown TXID in BEEF: ${input.sourceTXID}`)
-          }
-          input.sourceTransaction = sourceObj.tx
-          addPathOrInputs(sourceObj)
-        }
-      }
-    }
-
-    addPathOrInputs(transactions[targetTXID])
+    this.addPathOrInputs(transactions[targetTXID], transactions, BUMPs)
 
     return transactions[targetTXID].tx
   }
@@ -172,28 +174,7 @@ export default class Transaction {
       throw new Error(`Unrelated transaction with TXID ${txid} found in Atomic BEEF data.`)
     }
 
-    // Build the transaction by linking inputs and merkle paths
-    const addPathOrInputs = (obj: { pathIndex?: number, tx: Transaction }): void => {
-      if (typeof obj.pathIndex === 'number') {
-        const path = BUMPs[obj.pathIndex]
-        if (typeof path !== 'object') {
-          throw new Error('Invalid merkle path index found in BEEF!')
-        }
-        obj.tx.merklePath = path
-      } else {
-        for (let i = 0; i < obj.tx.inputs.length; i++) {
-          const input = obj.tx.inputs[i]
-          const sourceObj = transactions[input.sourceTXID]
-          if (typeof sourceObj !== 'object') {
-            throw new Error(`Reference to unknown TXID in BEEF: ${input.sourceTXID}`)
-          }
-          input.sourceTransaction = sourceObj.tx
-          addPathOrInputs(sourceObj)
-        }
-      }
-    }
-
-    addPathOrInputs(transactions[subjectTXID])
+    this.addPathOrInputs(transactions[subjectTXID], transactions, BUMPs)
 
     return transactions[subjectTXID].tx
   }
@@ -207,8 +188,8 @@ export default class Transaction {
   private static parseBEEFData (reader: Reader): { transactions: Record<string, { pathIndex?: number, tx: Transaction }>, BUMPs: MerklePath[] } {
     // Read the version
     const version = reader.readUInt32LE()
-    if (version !== BEEF_MAGIC) {
-      throw new Error(`Invalid BEEF version. Expected ${BEEF_MAGIC}, received ${version}.`)
+    if (version !== BEEF_V1) {
+      throw new Error(`Invalid BEEF version. Expected ${BEEF_V1}, received ${version}.`)
     }
 
     // Read the BUMPs
@@ -511,7 +492,15 @@ export default class Transaction {
       }
     }
     const fee = await modelOrFee.computeFee(this)
-    // change = inputs - fee - non-change outputs
+    const change = this.calculateChange(fee)
+    if (change <= 0) {
+      this.outputs = this.outputs.filter(output => !output.change)
+      return
+    }
+    this.distributeChange(change, changeDistribution)
+  }
+
+  private calculateChange (fee: number): number {
     let change = 0
     for (const input of this.inputs) {
       if (typeof input.sourceTransaction !== 'object') {
@@ -520,71 +509,60 @@ export default class Transaction {
       change += input.sourceTransaction.outputs[input.sourceOutputIndex].satoshis
     }
     change -= fee
-    let changeCount = 0
     for (const out of this.outputs) {
       if (!out.change) {
         change -= out.satoshis
-      } else {
-        changeCount++
-      }
-    }
-
-    if (change <= changeCount) {
-      // There is not enough change to distribute among the change outputs.
-      // We'll remove all change outputs and leave the extra for the miners.
-      for (let i = 0; i < this.outputs.length; i++) {
-        if (this.outputs[i].change) {
-          this.outputs.splice(i, 1)
-          i--
-        }
       }
-      return
     }
+    return change
+  }
 
-    // Distribute change among change outputs
+  private distributeChange (change: number, changeDistribution: 'equal' | 'random'): void {
     let distributedChange = 0
+    const changeOutputs = this.outputs.filter(out => out.change)
     if (changeDistribution === 'random') {
-      // Implement Benford's Law distribution for change outputs
-      const changeOutputs = this.outputs.filter(out => out.change)
-      let changeToUse = change
-
-      // Helper function to generate a number approximating Benford's Law
-      const benfordNumber = (min: number, max: number): number => {
-        const d = Math.floor(Math.random() * 9) + 1
-        return Math.floor(min + (max - min) * Math.log10(1 + 1 / d) / Math.log10(10))
-      }
-
-      const benfordNumbers = Array(changeOutputs.length).fill(1)
-      changeToUse -= changeOutputs.length
-      distributedChange += changeOutputs.length
-      for (let i = 0; i < changeOutputs.length - 1; i++) {
-        const portion = benfordNumber(0, changeToUse)
-        benfordNumbers[i] += portion
-        distributedChange += portion
-        changeToUse -= portion
-      }
-
-      for (let i = 0; i < this.outputs.length; i++) {
-        if (this.outputs[i].change) {
-          const t = benfordNumbers.shift()
-          this.outputs[i].satoshis = t
-        }
-      }
+      distributedChange = this.distributeRandomChange(change, changeOutputs)
     } else if (changeDistribution === 'equal') {
-      const perOutput = Math.floor(change / changeCount)
-      for (const out of this.outputs) {
-        if (out.change) {
-          distributedChange += perOutput
-          out.satoshis = perOutput
-        }
-      }
+      distributedChange = this.distributeEqualChange(change, changeOutputs)
     }
-    // if there's any remaining change, add it to the last output
     if (distributedChange < change) {
       this.outputs[this.outputs.length - 1].satoshis += (change - distributedChange)
     }
   }
 
+  private distributeRandomChange (change: number, changeOutputs: TransactionOutput[]): number {
+    let distributedChange = 0
+    let changeToUse = change
+    const benfordNumbers = Array(changeOutputs.length).fill(1)
+    changeToUse -= changeOutputs.length
+    distributedChange += changeOutputs.length
+    for (let i = 0; i < changeOutputs.length - 1; i++) {
+      const portion = this.benfordNumber(0, changeToUse)
+      benfordNumbers[i] += portion
+      distributedChange += portion
+      changeToUse -= portion
+    }
+    for (const output of this.outputs) {
+      if (output.change) output.satoshis = benfordNumbers.shift()
+    }
+    return distributedChange
+  }
+
+  private distributeEqualChange (change: number, changeOutputs: TransactionOutput[]): number {
+    let distributedChange = 0
+    const perOutput = Math.floor(change / changeOutputs.length)
+    for (const out of changeOutputs) {
+      distributedChange += perOutput
+      out.satoshis = perOutput
+    }
+    return distributedChange
+  }
+
+  private benfordNumber (min: number, max: number): number {
+    const d = Math.floor(Math.random() * 9) + 1
+    return Math.floor(min + (max - min) * Math.log10(1 + 1 / d) / Math.log10(10))
+  }
+
   /**
    * Utility method that returns the current fee based on inputs and outputs
    *
@@ -923,9 +901,8 @@ export default class Transaction {
    * @throws Error if there are any missing sourceTransactions unless `allowPartial` is true.
    */
   toBEEF (allowPartial?: boolean): number[] {
-    this.outputs.length
     const writer = new Writer()
-    writer.writeUInt32LE(4022206465)
+    writer.writeUInt32LE(BEEF_V1)
     const BUMPs: MerklePath[] = []
     const txs: Array<{ tx: Transaction, pathIndex?: number }> = []
 

package/src/transaction/__tests/Beef.test.ts

@@ -1,16 +1,19 @@
 // The following imports allow flag a large number type checking errors by the VsCode editor due to missing type information:
 import BeefTx from "../../../dist/cjs/src/transaction/BeefTx"
-import Beef from "../../../dist/cjs/src/transaction/Beef"
 import BeefParty from "../../../dist/cjs/src/transaction/BeefParty"
-import { BEEF_MAGIC, BEEF_MAGIC_V2 } from "../../../dist/cjs/src/transaction/Beef"
+import Beef, { BEEF_V1, BEEF_V2 } from "../../../dist/cjs/src/transaction/Beef"
 import Transaction from "../../../dist/cjs/src/transaction/Transaction"
+import { fromBase58 } from '../../../dist/cjs/src/primitives/utils'
 
 // The following imports allow full type checking by the VsCode editor, but tests will fail to run:
-//import BeefTx from '../BeefTx'
-//import Beef from '../Beef'
-//import BeefParty from "../BeefParty"
-//import { BEEF_MAGIC, BEEF_MAGIC_V2 } from "../Beef"
-//import Transaction from "../Transaction"
+/*
+import BeefTx from '../BeefTx'
+import Beef from '../Beef'
+import BeefParty from "../BeefParty"
+import { BEEF_V1, BEEF_V2 } from "../Beef"
+import Transaction from "../Transaction"
+import { fromBase58 } from "../../primitives/utils"
+*/
 
 describe('Beef tests', () => {
     jest.setTimeout(99999999)
@@ -19,6 +22,7 @@ describe('Beef tests', () => {
       isValidRootForHeight: async (root: string, height: number) => {
         switch (height) {
           case 1631619: return root === "b3975a6b69b5ce7fa200649d879f79a11f4d95c054cfe024570be7d60306ecf6"
+          case 875732: return root === "a19c54129ab996c72cda7721b4555b47d11b21e1fe67aa63c59843edb302b6c2"
           default: throw new Error(`unknown height ${height}`)
         }
       }
@@ -97,7 +101,7 @@ describe('Beef tests', () => {
         {
           const version = 4290641921
           expect(() => Beef.fromString(beefs[1]))
-          .toThrow(`Serialized BEEF must start with ${BEEF_MAGIC} or ${BEEF_MAGIC_V2} but starts with ${version}`)
+          .toThrow(`Serialized BEEF must start with ${BEEF_V1} or ${BEEF_V2} but starts with ${version}`)
         }
     })
 
@@ -258,7 +262,7 @@ describe('Beef tests', () => {
       {
         const beef = Beef.fromString(beefs[0])
         expect(beef.toHex()).toBe(beefs[0])
-        const sr = beef.sortTxs()
+        beef.sortTxs()
         const beefHex = beef.toHex()
         const tx = beef.txs[beef.txs.length - 1].tx!
         expect(tx).toBeTruthy()
@@ -280,7 +284,7 @@ describe('Beef tests', () => {
         const beef = Beef.fromString(beefs[0])
         expect(beef.toHex()).toBe(beefs[0])
         beef.mergeTransaction(Transaction.fromHex(txs[0]))
-        const sr = beef.sortTxs()
+        beef.sortTxs()
         const beefHex = beef.toHex()
         const tx = beef.txs[beef.txs.length - 1].tx!
         expect(tx).toBeTruthy()
@@ -297,8 +301,8 @@ describe('Beef tests', () => {
         const beef = Beef.fromString(beefs[0])
         const tx = Transaction.fromHex(txs[0])
         beef.mergeTransaction(tx)
-        const sr = beef.sortTxs()
-        const log = beef.toLogString()
+        beef.sortTxs()
+        beef.toLogString()
         const atomic = beef.toBinaryAtomic(tx.id('hex'))
         const t2 = Transaction.fromAtomicBEEF(atomic)
         const beef2 = t2.toAtomicBEEF()
@@ -317,14 +321,29 @@ describe('Beef tests', () => {
       {
         const beef = new Beef()
         const atx = beef.mergeTxidOnly('a')
-        const btx = beef.mergeTxidOnly('b')
+        beef.mergeTxidOnly('b')
         atx.inputTxids = ['b']
         beef.sortTxs()
         expect(beef.txs[1].txid).toBe('a')
       }
     })
+
+    test('10_deserialize beef with extra leaves', async () => {
+      const b58Beef = b58Beef_10
+
+      const beef = Beef.fromBinary(fromBase58(b58Beef))
+      expect(beef.verifyValid().valid).toBe(true)
+      const abeef = beef.toBinaryAtomic(beef.txs[beef.txs.length -1].txid)
+      const pbeef = Beef.fromBinary(abeef)
+      pbeef.addComputedLeaves();
+      const pbeefBinary = pbeef.toBinary();
+      expect(pbeefBinary).toBeTruthy()
+      expect(pbeef.verifyValid().valid).toBe(true)
+    })
 })
 
+const b58Beef_10 = 'gno9MC7VXii1KoCkc2nsVyYJpqzN3dhBzYATETJcys62emMKfpBof4R7GozwYEaSapUtnNvqQ57aaYYjm3U2dv9eUJ1sV46boHkQgppYmAz9YH8FdZduV8aJayPViaKcyPmbDhEw6UW8TM5iFZLXNs7HBnJHUKCeTdNK4FUEL7vAugxAV9WUUZ43BZjJk2SmSeps9TCXjt1Ci9fKWp3d9QSoYvTpxwzyUFHjRKtbUgwq55ZfkBp5bV2Bpz9qSuKywKewW7Hh4S1nCUScwwzpKDozb3zic1V9p2k8rQxoPsRxjUJ8bjhNDdsN8d7KukFuc3n47fXzdWttvnxwsujLJRGnQbgJuknQqx3KLf5kJXHzwjG6TzigZk2t24qeB6d3hbYiaDr2fFkUJBL3tukTHhfNkQYRXuz3kucVDzvejHyqJaF51mXG8BjMN5aQj91ZJXCaPVqkMWCzmvyaqmXMdRiJdSAynhXbQK91xf6RwdNhz1tg5f9B6oJJMhsi9UYSVymmax8VLKD9AKzBCBDcfyD83m3jyS1VgKGZn3SkQmr6bsoWq88L3GsMnnmYUGogvdAYarTqg3pzkjCMxHzmJBMN6ofnUk8c1sRTXQue7BbyUaN5uZu3KW6CmFsEfpuqVvnqFW93TU1jrPP2S8yz8AexAnARPCKE8Yz7RfVaT6RCavwQKL3u5iookwRWEZXW1QWmM37yJWHD87SjVynyg327a1CLwcBxmE2CB48QeNVGyQki4CTQMqw2o8TMhDPJej1g68oniAjBcxBLSCs7KGvK3k7AfrHbCMULX9CTibYhCjdFjbsbBoocqJpxxcvkMo1fEEiAzZuiBVZQDYktDdTVbhKHvYkW25HcYX75NJrpNAhm7AjFeKLzEVxqAQkMfvTufpESNRZF4kQqg2Rg8h2ajcKTd5cpEPwXCrZLHm4EaZEmZVbg3QNfGhn7BJu1bHMtLqPD4y8eJxm2uGrW6saf6qKYmmu64F8A667NbD4yskPRQ1S863VzwGpxxmgLc1Ta3R46jEqsAoRDoZVUaCgBBZG3Yg1CTgi1EVBMXU7qvY4n3h8o2FLCEMWY4KadnV3iD4FbcdCmg4yxBosNAZgbPjhgGjCimjh4YsLd9zymGLmivmz2ZBg5m3xaiXT9NN81X9C1JUujd'
+
 const txs: string[] = [
     // 0
     "0100000001074d631cc91344daf964cb8b1f43fd3439b804eb673cbe82d9bde3dcc6394abd020000006b48304502210096e24b07db278344a32c12667bac0d38ea87b28e9c52f3630ff7c11760d4002e02201b9dc879465c82152920f650828bfabba0d0bc8f7e15b568b196f1fa7e2cb4644121034e6dcc60f278b83e3e7b2b6570c13a184f4cf27a279199625c981adfe42de997ffffffff03b706000000000000fdc20421029b09fdddfae493e309d3d97b919f6ab2902a789158f6f78489ad903b7a14baeaac2131546f446f44744b7265457a6248594b466a6d6f42756475466d53585855475a474d3004a1edf497c9db470e7862d8c8319b4312f197a18db529a5839ba7c7e4f4dcee4ff4b4651a2a30040ef0aab1c2a4386620b96cb068c2ad2bc06ed90cebed65643710497f785f48c91482ac86893dd675c3de5d7db2e305fb30a588c75282f062367dee1dd4d930502f64f043c45cd1f220f875ff749f5449c99b29a200a86287a842a4f1df21dafda06a1b39b590b33b15109ee8a700728f1a80016ed370f639b35c9d079dea938d7fe451a52c744605a64a89ac7ebfcf692ef345a1e41fba055c197f7e3f608196001ed150bd56883970f3b8954a63aa70e4fa1c33799d43caaacf7ca30f8e873a6a20a04b1508f851633fcb4f30de75954747e702775b143c668c3b80cef96001980f05a8322726a567a379a2901db030057400bf0484c0852b1ec42cdaae6bf9f258ba0c914d83a96522d63b46a71571087fc4ad4ea0be0f6f354e7f638a7c316ceca3be69698104c1662ce0d5f4334b3327377497f9bb8605388c86a5a3928b22dbafd20a488442862c0ea4cd72778ae3b1bfd1d517299969f553b21173f75b530d1c19e23835dbc423c0ab5d3a894560a064951e237d2956db5d007d2132721a949849df49c5c5ab21f9eacb9ea5aa704c37cd101c17bee37954eae01472096f0e3e52b9f98d46426f35437c76c643c6f5d4d0a5ceb89d03a31c451d7882a527c9e78abf01f6178446cd74a7ad4190f364cc79fccc91583f3521e706bd548dc19633234fe96e40e19ef6424b252691e2183484cbf088e965066d4b93b80de927f3426e71f265152d898537c6112ed4500b8b7f23d033c18bad6c3ae5a2e069b03af5b619a334713f367082d41fdff8d5b17ab90a8aff224fbec3a8ca93828fe18c24b54fdcffdd9ee53c40808a4d233a56ed6f21bda0ebf1f4375785f2427da484ba9cdcd45bd28f8072e885bcb687c575ed00bea520508e795b32b7c882ffdd36062e9f748fddbb99a71e7afaed26c24e631ccce19ceb92f1920fb45ae30b58dde09389062eccf1292c55aca96e7eae53a3b6721a568b1bc8f1bad201c0e3b3fe7caec178e491dc12e97a0d0448136f7fb9970c7b053a85a553927a2273682eb2ae4c41c12f1d893d427f659048bbfa6c4be0682d8226d21e4b522e0a2d575655dea64d36d1e0f5719d4f6d06b47f4b4e32731652f0c12e28a9642e90d049b0fbc2c2c94079106c017ad85a81882344ddb71e3c5de31f5b42df9c269e1f64e36b3179ef5e9581fc57c1ed919962ba11686ef0e0a40694f8733bd1e2232196ca4f35c33df1b27872bd3aa0e258601192e50ec5bb64c6ae1a60f580cb6434048c7a419a76028f95442ce5c628f1324281ca3e8050c7ea8a90c3782a6df2cfee5d735e43f518bc215017cd036f70011312204bf885c615b1d1601626de7a1ff9ff21a44818fa19b7a8ed241b6f16380c9e376c6c7a2f48414269e4a3f130ac5c01a7ca8ad4af044ed4288657f59b675e89287c3811c8e81ba1c75f68b8ec85a35d15b5f1cee0fa25874730450221008d6e0b7d98442477d5637fc12fa871f224fd82194d2c3d8e524a12e389ddc35602206286a3b292ef3bac9f047c842a3e9b418102617546e9dc6f9dcc6653c82e645c6d75c8000000000000001976a9142e1ba3aade0562035e69630cb40024f403c0599e88ac77420000000000001976a914eeea6c76530f46d0f2a20ad522c4a6886bad686d88ac00000000"

package/src/transaction/__tests/Transaction.benchmarks.test.ts

@@ -20,7 +20,7 @@ async function measureTime(fn: () => Promise<void>): Promise<number> {
 describe('Transaction Verification Benchmark', () => {
     const privateKey = new PrivateKey(1)
     const publicKey = privateKey.toPublicKey()
-    const publicKeyHash = hash160(publicKey.toDER())
+    const publicKeyHash = publicKey.toHash()
     const p2pkh = new P2PKH()
 
     it('verifies a transaction with a deep input chain', async () => {

package/src/transaction/__tests/Transaction.test.ts

@@ -11,7 +11,7 @@ import Curve from '../../../dist/cjs/src/primitives/Curve'
 import P2PKH from '../../../dist/cjs/src/script/templates/P2PKH'
 import fromUtxo from '../../../dist/cjs/src/compat/Utxo'
 import MerklePath from '../../../dist/cjs/src/transaction/MerklePath'
-import { BEEF_MAGIC } from '../../../dist/cjs/src/transaction/Beef'
+import { BEEF_V1 } from '../../../dist/cjs/src/transaction/Beef'
 
 import sighashVectors from '../../primitives/__tests/sighash.vectors'
 import invalidTransactions from './tx.invalid.vectors'
@@ -817,7 +817,7 @@ describe('Transaction', () => {
       const fakeTXID = toArray('00'.repeat(32), 'hex')
       writer.write(fakeTXID)
       // Write empty BEEF data
-      writer.writeUInt32LE(BEEF_MAGIC) // BEEF version
+      writer.writeUInt32LE(BEEF_V1) // BEEF version
       writer.writeVarIntNum(0) // No BUMPs
       writer.writeVarIntNum(0) // No transactions
 
@@ -890,7 +890,7 @@ describe('Transaction', () => {
 
       // Build BEEF data
       const beefWriter = new Writer()
-      beefWriter.writeUInt32LE(BEEF_MAGIC) // BEEF version
+      beefWriter.writeUInt32LE(BEEF_V1) // BEEF version
       // BUMPs
       beefWriter.writeVarIntNum(2)
       beefWriter.write(sourceTx1.merklePath.toBinary())

package/src/transaction/broadcasters/__tests/WhatsOnChainBroadcaster.test.ts

@@ -46,8 +46,8 @@ describe('WhatsOnChainBroadcaster', () => {
   it('should broadcast successfully using Node.js https', async () => {
     // Mocking Node.js https module
     mockedHttps(successResponse)
-    delete global.window
-
+    global.window = {} as any
+    
     const broadcaster = new WhatsOnChainBroadcaster(network)
     const response = await broadcaster.broadcast(transaction)