@bsv/sdk 1.0.19 → 1.0.21

package/docs/transaction.md

@@ -442,6 +442,8 @@ export default class Transaction {
     toHex(): string 
     toHexBEEF(): string 
     hash(enc?: "hex"): number[] | string 
+    id(): number[];
+    id(enc: "hex"): string;
     id(enc?: "hex"): number[] | string 
     async verify(chainTracker: ChainTracker | "scripts only"): Promise<boolean> 
     toBEEF(): number[] 
@@ -605,6 +607,35 @@ Argument Details
 
 #### Method id
 
+Calculates the transaction's ID in binary array.
+
+```ts
+id(): number[]
+```
+
+Returns
+
+- The ID of the transaction in the binary array format.
+
+#### Method id
+
+Calculates the transaction's ID in hexadecimal format.
+
+```ts
+id(enc: "hex"): string
+```
+
+Returns
+
+- The ID of the transaction in the hex format.
+
+Argument Details
+
++ **enc**
+  + The encoding to use for the ID. If 'hex', returns a hexadecimal string.
+
+#### Method id
+
 Calculates the transaction's ID.
 
 ```ts

package/package.json

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

package/src/compat/HD.ts

@@ -84,6 +84,16 @@ export default class HD {
     return new this().fromRandom()
   }
 
+  /**
+   * Initializes the HD wallet from a given base58 encoded string.
+   * This method decodes a provided string to set up the HD wallet's properties.
+   * @param str - A base58 encoded string representing the wallet.
+   * @returns {HD} The new instance with properties set from the string.
+   */
+  public static fromString (str: string): HD {
+    return new this().fromString(str)
+  }
+
   /**
      * Initializes the HD wallet from a given base58 encoded string.
      * This method decodes a provided string to set up the HD wallet's properties.
@@ -96,13 +106,13 @@ export default class HD {
   }
 
   /**
-     * Converts the HD wallet to a base58 encoded string.
-     * This method provides a string representation of the HD wallet's current state.
-     * @returns {string} A base58 encoded string of the HD wallet.
-     */
-  public toString (): string {
-    const bin = this.toBinary()
-    return toBase58Check(bin, [])
+   * Initializes the HD wallet from a seed.
+   * This method generates keys and other properties from a given seed, conforming to the BIP32 specification.
+   * @param bytes - An array of bytes representing the seed.
+   * @returns {HD} The current instance with properties set from the seed.
+   */
+  public static fromSeed (bytes: number[]): HD {
+    return new this().fromSeed(bytes)
   }
 
   /**
@@ -132,21 +142,21 @@ export default class HD {
   }
 
   /**
-     * Initializes the HD wallet from a seed.
-     * This method generates keys and other properties from a given seed, conforming to the BIP32 specification.
-     * @param bytes - An array of bytes representing the seed.
-     * @returns {HD} The current instance with properties set from the seed.
-     */
-  public static fromSeed (bytes: number[]): HD {
-    return new this().fromSeed(bytes)
+   * Initializes the HD wallet from a binary buffer.
+   * Parses a binary buffer to set up the wallet's properties.
+   * @param buf - A buffer containing the wallet data.
+   * @returns {HD} The new instance with properties set from the buffer.
+   */
+  public static fromBinary (buf: number[]): HD {
+    return new this().fromBinary(buf)
   }
 
   /**
-     * Initializes the HD wallet from a binary buffer.
-     * Parses a binary buffer to set up the wallet's properties.
-     * @param buf - A buffer containing the wallet data.
-     * @returns {HD} The current instance with properties set from the buffer.
-     */
+   * Initializes the HD wallet from a binary buffer.
+   * Parses a binary buffer to set up the wallet's properties.
+   * @param buf - A buffer containing the wallet data.
+   * @returns {HD} The current instance with properties set from the buffer.
+   */
   public fromBinary (buf: number[]): this {
     // Both pub and private extended keys are 78 buf
     if (buf.length !== 78) {
@@ -176,6 +186,16 @@ export default class HD {
     return this
   }
 
+  /**
+   * Converts the HD wallet to a base58 encoded string.
+   * This method provides a string representation of the HD wallet's current state.
+   * @returns {string} A base58 encoded string of the HD wallet.
+   */
+  public toString (): string {
+    const bin = this.toBinary()
+    return toBase58Check(bin, [])
+  }
+
   /**
      * Derives a child HD wallet based on a given path.
      * The path specifies the hierarchy of the child key to be derived.

package/src/compat/__tests/HD.test.ts

@@ -1,5 +1,5 @@
 import HD from '../../../dist/cjs/src/compat/HD'
-import { fromBase58Check, toBase58Check, toArray, toHex } from '../../../dist/cjs/src/primitives/utils'
+import { fromBase58Check, toArray, toHex } from '../../../dist/cjs/src/primitives/utils'
 
 describe('HD', () => {
     it('should satisfy these basic API features', () => {
@@ -64,232 +64,231 @@ describe('HD', () => {
         let bip32
         bip32 = new HD()
         expect(bip32).toBeDefined()
-        expect(new HD().fromString(vector1mPrivate).toString()).toEqual(vector1mPrivate)
-        expect(new HD()
-            .fromString(new HD().fromString(vector1mPrivate).toString())
+        expect(HD.fromString(vector1mPrivate).toString()).toEqual(vector1mPrivate)
+        expect(HD.fromString(HD.fromString(vector1mPrivate).toString())
             .toString())
             .toEqual(vector1mPrivate)
     })
 
     it('should initialize test vector 1 from the extended public key', () => {
-        const bip32 = new HD().fromString(vector1mPublic)
+        const bip32 = HD.fromString(vector1mPublic)
         expect(bip32).toBeDefined()
     })
 
     it('should initialize test vector 1 from the extended private key', () => {
-        const bip32 = new HD().fromString(vector1mPrivate)
+        const bip32 = HD.fromString(vector1mPrivate)
         expect(bip32).toBeDefined()
     })
 
     it('should get the extended public key from the extended private key for test vector 1', () => {
-        const bip32 = new HD().fromString(vector1mPrivate)
+        const bip32 = HD.fromString(vector1mPrivate)
         expect(bip32.toPublic().toString()).toEqual(vector1mPublic)
     })
 
     it("should get m/0' ext. private key from test vector 1", () => {
-        const bip32 = new HD().fromString(vector1mPrivate)
+        const bip32 = HD.fromString(vector1mPrivate)
         const child = bip32.derive("m/0'")
         expect(child).toBeDefined()
         expect(child.toString()).toEqual(vector1m0hPrivate)
     })
 
     it("should get m/0' ext. public key from test vector 1", () => {
-        const bip32 = new HD().fromString(vector1mPrivate)
+        const bip32 = HD.fromString(vector1mPrivate)
         const child = bip32.derive("m/0'")
         expect(child).toBeDefined()
         expect(child.toPublic().toString()).toEqual(vector1m0hPublic)
     })
 
     it("should get m/0'/1 ext. private key from test vector 1", () => {
-        const bip32 = new HD().fromString(vector1mPrivate)
+        const bip32 = HD.fromString(vector1mPrivate)
         const child = bip32.derive("m/0'/1")
         expect(child).toBeDefined()
         expect(child.toString()).toEqual(vector1m0h1Private)
     })
 
     it("should get m/0'/1 ext. public key from test vector 1", () => {
-        const bip32 = new HD().fromString(vector1mPrivate)
+        const bip32 = HD.fromString(vector1mPrivate)
         const child = bip32.derive("m/0'/1")
         expect(child).toBeDefined()
         expect(child.toPublic().toString()).toEqual(vector1m0h1Public)
     })
 
     it("should get m/0'/1 ext. public key from m/0' public key from test vector 1", () => {
-        const bip32 = new HD().fromString(vector1mPrivate)
+        const bip32 = HD.fromString(vector1mPrivate)
         const child = bip32.derive("m/0'")
-        const childPub = new HD().fromString(child.toPublic().toString())
+        const childPub = HD.fromString(child.toPublic().toString())
         const child2 = childPub.derive('m/1')
         expect(child2).toBeDefined()
         expect(child2.toPublic().toString()).toEqual(vector1m0h1Public)
     })
 
     it("should get m/0'/1/2h ext. private key from test vector 1", () => {
-        const bip32 = new HD().fromString(vector1mPrivate)
+        const bip32 = HD.fromString(vector1mPrivate)
         const child = bip32.derive("m/0'/1/2'")
         expect(child).toBeDefined()
         expect(child.toString()).toEqual(vector1m0h12hPrivate)
     })
 
     it("should get m/0'/1/2h ext. public key from test vector 1", () => {
-        const bip32 = new HD().fromString(vector1mPrivate)
+        const bip32 = HD.fromString(vector1mPrivate)
         const child = bip32.derive("m/0'/1/2'")
         expect(child).toBeDefined()
         expect(child.toPublic().toString()).toEqual(vector1m0h12hPublic)
     })
 
     it("should get m/0'/1/2h/2 ext. private key from test vector 1", () => {
-        const bip32 = new HD().fromString(vector1mPrivate)
+        const bip32 = HD.fromString(vector1mPrivate)
         const child = bip32.derive("m/0'/1/2'/2")
         expect(child).toBeDefined()
         expect(child.toString()).toEqual(vector1m0h12h2Private)
     })
 
     it("should get m/0'/1/2'/2 ext. public key from m/0'/1/2' public key from test vector 1", () => {
-        const bip32 = new HD().fromString(vector1mPrivate)
+        const bip32 = HD.fromString(vector1mPrivate)
         const child = bip32.derive("m/0'/1/2'")
-        const childPub = new HD().fromString(child.toPublic().toString())
+        const childPub = HD.fromString(child.toPublic().toString())
         const child2 = childPub.derive('m/2')
         expect(child2).toBeDefined()
         expect(child2.toPublic().toString()).toEqual(vector1m0h12h2Public)
     })
 
     it("should get m/0'/1/2h/2 ext. public key from test vector 1", () => {
-        const bip32 = new HD().fromString(vector1mPrivate)
+        const bip32 = HD.fromString(vector1mPrivate)
         const child = bip32.derive("m/0'/1/2'/2")
         expect(child).toBeDefined()
         expect(child.toPublic().toString()).toEqual(vector1m0h12h2Public)
     })
 
     it("should get m/0'/1/2h/2/1000000000 ext. private key from test vector 1", () => {
-        const bip32 = new HD().fromString(vector1mPrivate)
+        const bip32 = HD.fromString(vector1mPrivate)
         const child = bip32.derive("m/0'/1/2'/2/1000000000")
         expect(child).toBeDefined()
         expect(child.toString()).toEqual(vector1m0h12h21000000000Private)
     })
 
     it("should get m/0'/1/2h/2/1000000000 ext. public key from test vector 1", () => {
-        const bip32 = new HD().fromString(vector1mPrivate)
+        const bip32 = HD.fromString(vector1mPrivate)
         const child = bip32.derive("m/0'/1/2'/2/1000000000")
         expect(child).toBeDefined()
         expect(child.toPublic().toString()).toEqual(vector1m0h12h21000000000Public)
     })
 
     it("should get m/0'/1/2'/2/1000000000 ext. public key from m/0'/1/2'/2 public key from test vector 1", () => {
-        const bip32 = new HD().fromString(vector1mPrivate)
+        const bip32 = HD.fromString(vector1mPrivate)
         const child = bip32.derive("m/0'/1/2'/2")
-        const childPub = new HD().fromString(child.toPublic().toString())
+        const childPub = HD.fromString(child.toPublic().toString())
         const child2 = childPub.derive('m/1000000000')
         expect(child2).toBeDefined()
         expect(child2.toPublic().toString()).toEqual(vector1m0h12h21000000000Public)
     })
 
     it('should initialize test vector 2 from the extended public key', () => {
-        const bip32 = new HD().fromString(vector2mPublic)
+        const bip32 = HD.fromString(vector2mPublic)
         expect(bip32).toBeDefined()
     })
 
     it('should initialize test vector 2 from the extended private key', () => {
-        const bip32 = new HD().fromString(vector2mPrivate)
+        const bip32 = HD.fromString(vector2mPrivate)
         expect(bip32).toBeDefined()
     })
 
     it('should get the extended public key from the extended private key for test vector 2', () => {
-        const bip32 = new HD().fromString(vector2mPrivate)
+        const bip32 = HD.fromString(vector2mPrivate)
         expect(bip32.toPublic().toString()).toEqual(vector2mPublic)
     })
 
     it('should get m/0 ext. private key from test vector 2', () => {
-        const bip32 = new HD().fromString(vector2mPrivate)
+        const bip32 = HD.fromString(vector2mPrivate)
         const child = bip32.derive('m/0')
         expect(child).toBeDefined()
         expect(child.toString()).toEqual(vector2m0Private)
     })
 
     it('should get m/0 ext. public key from test vector 2', () => {
-        const bip32 = new HD().fromString(vector2mPrivate)
+        const bip32 = HD.fromString(vector2mPrivate)
         const child = bip32.derive('m/0')
         expect(child).toBeDefined()
         expect(child.toPublic().toString()).toEqual(vector2m0Public)
     })
 
     it('should get m/0 ext. public key from m public key from test vector 2', () => {
-        const bip32 = new HD().fromString(vector2mPrivate)
+        const bip32 = HD.fromString(vector2mPrivate)
         const child = bip32.derive('m')
-        const childPub = new HD().fromString(child.toPublic().toString())
+        const childPub = HD.fromString(child.toPublic().toString())
         const child2 = childPub.derive('m/0')
         expect(child2).toBeDefined()
         expect(child2.toPublic().toString()).toEqual(vector2m0Public)
     })
 
     it('should get m/0/2147483647h ext. private key from test vector 2', () => {
-        const bip32 = new HD().fromString(vector2mPrivate)
+        const bip32 = HD.fromString(vector2mPrivate)
         const child = bip32.derive("m/0/2147483647'")
         expect(child).toBeDefined()
         expect(child.toString()).toEqual(vector2m02147483647hPrivate)
     })
 
     it('should get m/0/2147483647h ext. public key from test vector 2', () => {
-        const bip32 = new HD().fromString(vector2mPrivate)
+        const bip32 = HD.fromString(vector2mPrivate)
         const child = bip32.derive("m/0/2147483647'")
         expect(child).toBeDefined()
         expect(child.toPublic().toString()).toEqual(vector2m02147483647hPublic)
     })
 
     it('should get m/0/2147483647h/1 ext. private key from test vector 2', () => {
-        const bip32 = new HD().fromString(vector2mPrivate)
+        const bip32 = HD.fromString(vector2mPrivate)
         const child = bip32.derive("m/0/2147483647'/1")
         expect(child).toBeDefined()
         expect(child.toString()).toEqual(vector2m02147483647h1Private)
     })
 
     it('should get m/0/2147483647h/1 ext. public key from test vector 2', () => {
-        const bip32 = new HD().fromString(vector2mPrivate)
+        const bip32 = HD.fromString(vector2mPrivate)
         const child = bip32.derive("m/0/2147483647'/1")
         expect(child).toBeDefined()
         expect(child.toPublic().toString()).toEqual(vector2m02147483647h1Public)
     })
 
     it('should get m/0/2147483647h/1 ext. public key from m/0/2147483647h public key from test vector 2', () => {
-        const bip32 = new HD().fromString(vector2mPrivate)
+        const bip32 = HD.fromString(vector2mPrivate)
         const child = bip32.derive("m/0/2147483647'")
-        const childPub = new HD().fromString(child.toPublic().toString())
+        const childPub = HD.fromString(child.toPublic().toString())
         const child2 = childPub.derive('m/1')
         expect(child2).toBeDefined()
         expect(child2.toPublic().toString()).toEqual(vector2m02147483647h1Public)
     })
 
     it('should get m/0/2147483647h/1/2147483646h ext. private key from test vector 2', () => {
-        const bip32 = new HD().fromString(vector2mPrivate)
+        const bip32 = HD.fromString(vector2mPrivate)
         const child = bip32.derive("m/0/2147483647'/1/2147483646'")
         expect(child).toBeDefined()
         expect(child.toString()).toEqual(vector2m02147483647h12147483646hPrivate)
     })
 
     it('should get m/0/2147483647h/1/2147483646h ext. public key from test vector 2', () => {
-        const bip32 = new HD().fromString(vector2mPrivate)
+        const bip32 = HD.fromString(vector2mPrivate)
         const child = bip32.derive("m/0/2147483647'/1/2147483646'")
         expect(child).toBeDefined()
         expect(child.toPublic().toString()).toEqual(vector2m02147483647h12147483646hPublic)
     })
 
     it('should get m/0/2147483647h/1/2147483646h/2 ext. private key from test vector 2', () => {
-        const bip32 = new HD().fromString(vector2mPrivate)
+        const bip32 = HD.fromString(vector2mPrivate)
         const child = bip32.derive("m/0/2147483647'/1/2147483646'/2")
         expect(child).toBeDefined()
         expect(child.toString()).toEqual(vector2m02147483647h12147483646h2Private)
     })
 
     it('should get m/0/2147483647h/1/2147483646h/2 ext. public key from test vector 2', () => {
-        const bip32 = new HD().fromString(vector2mPrivate)
+        const bip32 = HD.fromString(vector2mPrivate)
         const child = bip32.derive("m/0/2147483647'/1/2147483646'/2")
         expect(child).toBeDefined()
         expect(child.toPublic().toString()).toEqual(vector2m02147483647h12147483646h2Public)
     })
 
     it('should get m/0/2147483647h/1/2147483646h/2 ext. public key from m/0/2147483647h/2147483646h public key from test vector 2', () => {
-        const bip32 = new HD().fromString(vector2mPrivate)
+        const bip32 = HD.fromString(vector2mPrivate)
         const child = bip32.derive("m/0/2147483647'/1/2147483646'")
-        const childPub = new HD().fromString(child.toPublic().toString())
+        const childPub = HD.fromString(child.toPublic().toString())
         const child2 = childPub.derive('m/2')
         expect(child2).toBeDefined()
         expect(child2.toPublic().toString()).toEqual(vector2m02147483647h12147483646h2Public)
@@ -297,8 +296,8 @@ describe('HD', () => {
 
     describe('#fromRandom', () => {
         it('should not return the same one twice', () => {
-            const bip32a = new HD().fromRandom()
-            const bip32b = new HD().fromRandom()
+            const bip32a = HD.fromRandom()
+            const bip32b = HD.fromRandom()
             expect(bip32a.toString()).not.toEqual(bip32b.toString())
         })
     })
@@ -314,7 +313,7 @@ describe('HD', () => {
     describe('#fromSeed', () => {
         it('should initialize a new Bip32 correctly from test vector 1 seed', () => {
             const hex = vector1master
-            const bip32 = new HD().fromSeed(toArray(hex, 'hex'))
+            const bip32 = HD.fromSeed(toArray(hex, 'hex'))
             expect(bip32).toBeDefined()
             expect(bip32.toString()).toEqual(vector1mPrivate)
             expect(bip32.toPublic().toString()).toEqual(vector1mPublic)
@@ -322,7 +321,7 @@ describe('HD', () => {
 
         it('should initialize a new Bip32 correctly from test vector 2 seed', () => {
             const hex = vector2master
-            const bip32 = new HD().fromSeed(toArray(hex, 'hex'))
+            const bip32 = HD.fromSeed(toArray(hex, 'hex'))
             expect(bip32).toBeDefined()
             expect(bip32.toString()).toEqual(vector2mPrivate)
             expect(bip32.toPublic().toString()).toEqual(vector2mPublic)
@@ -352,12 +351,12 @@ describe('HD', () => {
             const str =
                 'xprv9s21ZrQH143K3QTDL4LXw2F7HEK3wJUD2nW2nRk4stbPy6cq3jPPqjiChkVvvNKmPGJxWUtg6LnF5kejMRNNU3TGtRBeJgk33yuGBxrMPHi'
             const buf = fromBase58Check(str)
-            let bip32 = new HD().fromBinary([...buf.prefix, ...buf.data])
+            let bip32 = HD.fromBinary([...buf.prefix, ...buf.data])
             expect(bip32).toBeDefined()
             expect(bip32.toString()).toEqual(str)
             bip32 = bip32.toPublic()
             const xpub = bip32.toString()
-            bip32 = new HD().fromBinary(bip32.toBinary())
+            bip32 = HD.fromBinary(bip32.toBinary())
             expect(bip32.toString()).toEqual(xpub)
         })
     })
@@ -367,7 +366,7 @@ describe('HD', () => {
             const str =
                 'xprv9s21ZrQH143K3QTDL4LXw2F7HEK3wJUD2nW2nRk4stbPy6cq3jPPqjiChkVvvNKmPGJxWUtg6LnF5kejMRNNU3TGtRBeJgk33yuGBxrMPHi'
             const buf = fromBase58Check(str)
-            const bip32 = new HD().fromString(str)
+            const bip32 = HD.fromString(str)
             expect(toHex(bip32.toBinary())).toEqual(toHex([...buf.prefix, ...buf.data]))
         })
     })
@@ -376,7 +375,7 @@ describe('HD', () => {
         it('should make a bip32 from a string', () => {
             const str =
                 'xprv9s21ZrQH143K3QTDL4LXw2F7HEK3wJUD2nW2nRk4stbPy6cq3jPPqjiChkVvvNKmPGJxWUtg6LnF5kejMRNNU3TGtRBeJgk33yuGBxrMPHi'
-            const bip32 = new HD().fromString(str)
+            const bip32 = HD.fromString(str)
             expect(bip32).toBeDefined()
             expect(bip32.toString()).toEqual(str)
         })
@@ -396,10 +395,10 @@ describe('HD', () => {
 
     describe('#isPrivate', () => {
         it('should know if this bip32 is private', () => {
-            const bip32priv = new HD().fromRandom()
+            const bip32priv = HD.fromRandom()
             const bip32pub = bip32priv.toPublic()
             expect(bip32priv.isPrivate()).toEqual(true)
             expect(bip32pub.isPrivate()).toEqual(false)
         })
     })
-})
+})

package/src/primitives/TransactionSignature.ts

@@ -42,7 +42,7 @@ export default class TransactionSignature extends Signature {
         if (typeof input.sourceTransaction === 'undefined') {
           writer.writeReverse(toArray(input.sourceTXID, 'hex'))
         } else {
-          writer.writeReverse(input.sourceTransaction.id() as number[])
+          writer.writeReverse(input.sourceTransaction.id())
         }
         writer.writeUInt32LE(input.sourceOutputIndex)
       }

package/src/script/ScriptTemplate.ts

@@ -28,9 +28,9 @@ export default interface ScriptTemplate {
    * @returns {Object} - An object containing the `sign` and `estimateLength` functions.
    */
   unlock: (...params: any) =>
-    {
-      sign: (tx: Transaction, inputIndex: number) =>
-        Promise<UnlockingScript>
-      estimateLength: (tx: Transaction, inputIndex: number) => Promise<number>
-    }
+  {
+    sign: (tx: Transaction, inputIndex: number) =>
+    Promise<UnlockingScript>
+    estimateLength: (tx: Transaction, inputIndex: number) => Promise<number>
+  }
 }

package/src/script/Spend.ts

@@ -583,6 +583,7 @@ export default class Spend {
           } else {
             this.programCounter = this.lockingScript.chunks.length
           }
+          this.ifStack = []
           break
 
         case OP.OP_TOALTSTACK:

package/src/script/__tests/spend.valid.vectors.ts

@@ -2286,5 +2286,10 @@ export default [
     "51",
     "000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af000000af",
     "test"
+  ],
+  [
+    "51",
+    "63516a68",
+    "Returning within an if statement should succeed"
   ]
 ]

package/src/transaction/Transaction.ts

@@ -83,7 +83,7 @@ export default class Transaction {
     for (let i = 0; i < numberOfTransactions; i++) {
       const tx = Transaction.fromReader(reader)
       const obj: { pathIndex?: number, tx: Transaction } = { tx }
-      const txid = tx.id('hex') as string
+      const txid = tx.id('hex')
       if (i + 1 === numberOfTransactions) { // The last tXID is stored for later
         lastTXID = txid
       }
@@ -461,6 +461,19 @@ export default class Transaction {
     }
   }
 
+  /**
+   * Calculates the transaction's ID in binary array.
+   *
+   * @returns {number[]} - The ID of the transaction in the binary array format.
+   */
+  id (): number[]
+  /**
+   * Calculates the transaction's ID in hexadecimal format.
+   *
+   * @param {'hex'} enc - The encoding to use for the ID. If 'hex', returns a hexadecimal string.
+   * @returns {string} - The ID of the transaction in the hex format.
+   */
+  id (enc: 'hex'): string
   /**
    * Calculates the transaction's ID.
    *
@@ -487,7 +500,7 @@ export default class Transaction {
     // If the transaction has a valid merkle path, verification is complete.
     if (typeof this.merklePath === 'object' && chainTracker !== 'scripts only') {
       const proofValid = await this.merklePath.verify(
-        this.id('hex') as string,
+        this.id('hex'),
         chainTracker
       )
       // Note that if the proof is provided but not valid, the transaction could still be verified by proving all inputs (if they are available) and checking the associated spends.
@@ -502,10 +515,10 @@ export default class Transaction {
     for (let i = 0; i < this.inputs.length; i++) {
       const input = this.inputs[i]
       if (typeof input.sourceTransaction !== 'object') {
-        throw new Error(`Verification failed because the input at index ${i} of transaction ${this.id('hex') as string} is missing an associated source transaction. This source transaction is required for transaction verification because there is no merkle proof for the transaction spending a UTXO it contains.`)
+        throw new Error(`Verification failed because the input at index ${i} of transaction ${this.id('hex')} is missing an associated source transaction. This source transaction is required for transaction verification because there is no merkle proof for the transaction spending a UTXO it contains.`)
       }
       if (typeof input.unlockingScript !== 'object') {
-        throw new Error(`Verification failed because the input at index ${i} of transaction ${this.id('hex') as string} is missing an associated unlocking script. This script is required for transaction verification because there is no merkle proof for the transaction spending the UTXO.`)
+        throw new Error(`Verification failed because the input at index ${i} of transaction ${this.id('hex')} is missing an associated unlocking script. This script is required for transaction verification because there is no merkle proof for the transaction spending the UTXO.`)
       }
       const sourceOutput = input.sourceTransaction.outputs[input.sourceOutputIndex]
       inputTotal += sourceOutput.satoshis
@@ -516,7 +529,7 @@ export default class Transaction {
       const otherInputs = [...this.inputs]
       otherInputs.splice(i, 1)
       const spend = new Spend({
-        sourceTXID: input.sourceTransaction.id('hex') as string,
+        sourceTXID: input.sourceTransaction.id('hex'),
         sourceOutputIndex: input.sourceOutputIndex,
         lockingScript: sourceOutput.lockingScript,
         sourceSatoshis: sourceOutput.satoshis,