@bsv/sdk 1.2.12 → 1.2.14

package/docs/transaction.md

@@ -135,7 +135,7 @@ This interface defines a standard method for broadcasting transactions.
 ```ts
 export interface Broadcaster {
     broadcast: (transaction: Transaction) => Promise<BroadcastResponse | BroadcastFailure>;
-    broadcastMany?: (txs: Transaction[]) => Promise<Array<object>>;
+    broadcastMany?: (txs: Transaction[]) => Promise<object[]>;
 }
 ```
 
@@ -1219,6 +1219,7 @@ export default class MerklePath {
     static fromHex(hex: string): MerklePath 
     static fromReader(reader: Reader): MerklePath 
     static fromBinary(bump: number[]): MerklePath 
+    static fromCoinbaseTxidAndHeight(txid: string, height: number): MerklePath 
     constructor(blockHeight: number, path: Array<Array<{
         offset: number;
         hash?: string;
@@ -1309,6 +1310,24 @@ Argument Details
 + **bump**
   + The binary array representation of the Merkle Path.
 
+#### Method fromCoinbaseTxidAndHeight
+
+```ts
+static fromCoinbaseTxidAndHeight(txid: string, height: number): MerklePath 
+```
+See also: [MerklePath](#class-merklepath)
+
+Returns
+
+- A new MerklePath instance which assumes the tx is in a block with no other transactions.
+
+Argument Details
+
++ **txid**
+  + The coinbase txid.
++ **height**
+  + The height of the block.
+
 #### Method fromHex
 
 Creates a MerklePath instance from a hexadecimal string.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bsv/sdk",
-  "version": "1.2.12",
+  "version": "1.2.14",
   "type": "module",
   "description": "BSV Blockchain Software Development Kit",
   "main": "dist/cjs/mod.js",

package/src/primitives/__tests/PrivateKey.test.ts

@@ -11,9 +11,9 @@ describe('PrivateKey', () => {
     BRC42Private.forEach((vector, index) => {
       it(`Passes BRC42 private vector #${index + 1}`, () => {
         const publicKey = PublicKey.fromString(vector.senderPublicKey)
-        const privateKey = PrivateKey.fromString(vector.recipientPrivateKey, 16)
+        const privateKey = PrivateKey.fromString(vector.recipientPrivateKey)
         const derived = privateKey.deriveChild(publicKey, vector.invoiceNumber)
-        expect(derived.toHex(32)).toEqual(vector.privateKey)
+        expect(derived.toHex()).toEqual(vector.privateKey)
       })
     })
   })

package/src/transaction/MerklePath.ts

@@ -94,6 +94,21 @@ export default class MerklePath {
     return MerklePath.fromReader(reader)
   }
 
+  /**
+   *
+   * @static fromCoinbaseTxid
+   *
+   * Creates a MerklePath instance for a coinbase transaction in an empty block.
+   * This edge case is difficult to retrieve from standard APIs.
+   *
+   * @param {string} txid - The coinbase txid.
+   * @param {number} height - The height of the block.
+   * @returns {MerklePath} - A new MerklePath instance which assumes the tx is in a block with no other transactions.
+   */
+  static fromCoinbaseTxidAndHeight (txid: string, height: number): MerklePath {
+    return new MerklePath(height, [[{ offset: 0, hash: txid, txid: true }]])
+  }
+
   constructor (blockHeight: number, path: Array<Array<{
     offset: number
     hash?: string
@@ -127,8 +142,8 @@ export default class MerklePath {
       })
     })
 
-    let root: string
     // every txid must calculate to the same root.
+    let root: string
     this.path[0].map((leaf, idx) => {
       if (idx === 0) root = this.computeRoot(leaf.hash)
       if (root !== this.computeRoot(leaf.hash)) {
@@ -198,6 +213,10 @@ export default class MerklePath {
       hash256(toArray(m, 'hex').reverse())
     ).reverse())
     let workingHash = txid
+
+    // special case for blocks with only one transaction
+    if (this.path.length === 1 && this.path[0].length === 1) return workingHash
+
     for (let height = 0; height < this.path.length; height++) {
       const leaves = this.path[height]
       const offset = index >> height ^ 1

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 { BEEF_MAGIC } from './Beef.js'
+import { ATOMIC_BEEF, BEEF_MAGIC } from './Beef.js'
 
 /**
  * Represents a complete Bitcoin transaction. This class encapsulates all the details
@@ -119,8 +119,8 @@ export default class Transaction {
     const reader = new Reader(beef)
     // Read the Atomic BEEF prefix
     const prefix = reader.readUInt32LE()
-    if (prefix !== 0x01010101) {
-      throw new Error(`Invalid Atomic BEEF prefix. Expected 0x01010101, received ${prefix.toString(16)}.`)
+    if (prefix !== ATOMIC_BEEF) {
+      throw new Error(`Invalid Atomic BEEF prefix. Expected 0x01010101, received 0x${prefix.toString(16)}.`)
     }
 
     // Read the subject TXID

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

@@ -185,4 +185,11 @@ describe('MerklePath', () => {
       expect(() => MerklePath.fromHex(valid.bump)).not.toThrowError()
     }
   })
+  it('Validates a MerklePath for a block which only has 1 tx', () => {
+    const path = MerklePath.fromHex('fdd2040101000202ef57aa9f29c8141ae17935c88434457b2117890f23efba0d0e0cba7a7a37d5')
+    expect(path.computeRoot('d5377a7aba0c0e0dbaef230f8917217b453484c83579e11a14c8299faa57ef02')).toEqual('d5377a7aba0c0e0dbaef230f8917217b453484c83579e11a14c8299faa57ef02')
+  })
+  it('Creates a valid MerklePath from a txid', () => {
+    expect(() => MerklePath.fromCoinbaseTxidAndHeight('d5377a7aba0c0e0dbaef230f8917217b453484c83579e11a14c8299faa57ef02', 1)).not.toThrowError()
+  })
 })

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

@@ -661,6 +661,15 @@ describe('Transaction', () => {
       const verified = await tx.verify("scripts only")
       expect(verified).toBe(true)
     })
+
+    it('Verifies tx scripts only when our input has no MerklePath.', async () => {
+      const sourceTransaction = Transaction.fromHex('01000000013834506e4af751dc3a0d73e9bdd067709ade5248860967fb3eb72854be6011dc020000006b4830450221008abf2ce1afbcc3c4895265b0966ac69e2275fb35a738f9873b5416cd3bb68cc102202c906f9771d2d6868f81a34d6a3af4cd7de070969286db7097c5e0f3a009b638412102f6a81c442dcd903c7808a75abde9217268bb70417ad538c18fb597b4c553c354ffffffff0301000000000000007c21039bf52152665699e24c1018bf5b48e575de316f844962a79db32b93b7e328bf71ac1062656e63686d61726b546f6b656e5f36463044022034ca3cc49163e4d5bc24f41c5ea4631efd4ae7c243f904c90e1db6e6a55e8f0e02206bb031baca2fc6d64da5ad722ce2e07137718e5e58092e5ecf54e52fcdd664ab6dc8000000000000001976a91470763a553d615a8144fa0889a0e1f93d1d8da8b088acaf430000000000001976a9145aa12f9be0121eee54e7331d8f995fe4dded72f588ac00000000')
+      const tx = Transaction.fromHex('01000000015c22e8d34a95a761a0685b8b5a99a0845e9f03d87d5716a7351d08939db5cd92020000006a47304402205101f2bdc40098cbde1a270071a5bfdd5a6fbefc8a38a8b4a1adaca0a6ff55e702200ec5c0a57cb65dfc6aa22e6856bb4c62b8987a7b9fa32cb601c94b579ba67d83412103c8be6b217fc55939851e28239fbbf731f997e026ad344aa9d4b739181fe96743ffffffff0301000000000000007c21039bf52152665699e24c1018bf5b48e575de316f844962a79db32b93b7e328bf71ac1062656e63686d61726b546f6b656e5f374630440220026808e9453f75ca1fc3c7bb0f7be1efaca5cea6b1eda96ab7db56c519b6793202207d1fa5e7d54254d2142256947f93a061781280a6ed2464052470633a7c888d1f6dc8000000000000001976a91428bb6f207ac8f4b7f6c1c5914e143a6165586b2488ace5420000000000001976a914197cbde77abbcb985ca60a85c2988b2ac32c541a88ac00000000')
+      sourceTransaction.merklePath = { assumeValid: true } // can be any object.
+      tx.inputs[0].sourceTransaction = sourceTransaction
+      const verified = await tx.verify('scripts only', undefined)
+      expect(verified).toBe(true)
+    })
   })
 
   describe('vectors: a 1mb transaction', () => {