@bsv/sdk 2.1.5 → 2.1.6

package/package.json

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

package/src/compat/Utxo.ts

@@ -44,7 +44,7 @@ export default function fromUtxo (
   }
 ): TransactionInput {
   const sourceTransaction = new Transaction(0, [], [], 0)
-  sourceTransaction.outputs = Array(utxo.vout + 1).fill(null)
+  sourceTransaction.outputs = new Array(utxo.vout + 1).fill(null)
   sourceTransaction.outputs[utxo.vout] = {
     satoshis: utxo.satoshis,
     lockingScript: LockingScript.fromHex(utxo.script)

package/src/compat/__tests/Mnemonic.additional.test.ts

@@ -15,7 +15,7 @@ describe('Mnemonic – additional coverage', () => {
     })
 
     it('uses 128 bits when bits is NaN', () => {
-      const m = new Mnemonic().fromRandom(NaN)
+      const m = new Mnemonic().fromRandom(Number.NaN)
       expect(m.mnemonic.split(' ')).toHaveLength(12)
     })
 

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

@@ -41,7 +41,7 @@ describe('Mnemonic', function () {
     m = new Mnemonic().fromRandom(128)
     expect(m.check()).toEqual(true)
 
-    const entropy = Array(32)
+    const entropy = new Array(32)
     entropy.fill(0)
     m = new Mnemonic().fromEntropy(entropy)
     expect(m.check()).toEqual(true)

package/src/overlay-tools/__tests/Historian.test.ts

@@ -34,7 +34,7 @@ function makeMockTx(txid: string, outputs: any[] = [], inputs: any[] = []): MTx
 
 function makeMockOutput(scriptHex?: string): TransactionOutput {
   const hex = scriptHex || '76a914' // Default to P2PKH prefix if no script provided
-  const scriptArray = hex.match(/.{2}/g)?.map(byte => parseInt(byte, 16)) || [0x76, 0xa9, 0x14]
+  const scriptArray = hex.match(/.{2}/g)?.map(byte => Number.parseInt(byte, 16)) || [0x76, 0xa9, 0x14]
 
   return {
     satoshis: 1,

package/src/overlay-tools/__tests/HostReputationTracker.additional.test.ts

@@ -93,7 +93,7 @@ describe('HostReputationTracker – additional coverage', () => {
 
     it('treats NaN latency as DEFAULT_LATENCY_MS (1500)', () => {
       const t = new HostReputationTracker()
-      t.recordSuccess('https://host.com', NaN)
+      t.recordSuccess('https://host.com', Number.NaN)
       const snap = t.snapshot('https://host.com')!
       expect(snap.avgLatencyMs).toBe(1500)
     })

package/src/primitives/Hash.ts

@@ -261,8 +261,10 @@ function appendUtf8CodeUnit (msg: string, i: number, out: number[]): number {
 
 function utf8StringToArray (msg: string): number[] {
   const res: number[] = []
-  for (let i = 0; i < msg.length; i++) {
-    i = appendUtf8CodeUnit(msg, i, res)
+  let i = 0
+  while (i < msg.length) {
+    const lastConsumed = appendUtf8CodeUnit(msg, i, res)
+    i = lastConsumed + 1
   }
   return res
 }

package/src/primitives/__tests/BigNumber.constructor.test.ts

@@ -52,7 +52,7 @@ describe('BN.js/Constructor', () => {
     it('should accept base-16 with spaces', () => {
       const num = 'a89c e5af8724 c0a23e0e 0ff77500'
       expect(new BigNumber(num, 16).toString(16)).toEqual(
-        num.replace(/ /g, '')
+        num.replaceAll(' ', '')
       )
     })
 

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

@@ -34,36 +34,28 @@ describe('DRBG', () => {
       const entropy = new Array(31).fill(0x01)
       const nonce = new Array(32).fill(0x02)
 
-      expect(() => {
-        new DRBG(entropy, nonce)
-      }).toThrow('Entropy must be exactly 32 bytes (256 bits)')
+      expect(() => new DRBG(entropy, nonce)).toThrow('Entropy must be exactly 32 bytes (256 bits)')
     })
 
     it('throws if entropy is longer than 32 bytes', () => {
       const entropy = new Array(33).fill(0x01)
       const nonce = new Array(32).fill(0x02)
 
-      expect(() => {
-        new DRBG(entropy, nonce)
-      }).toThrow('Entropy must be exactly 32 bytes (256 bits)')
+      expect(() => new DRBG(entropy, nonce)).toThrow('Entropy must be exactly 32 bytes (256 bits)')
     })
 
     it('throws if nonce is shorter than 32 bytes', () => {
       const entropy = new Array(32).fill(0x01)
       const nonce = new Array(31).fill(0x02)
 
-      expect(() => {
-        new DRBG(entropy, nonce)
-      }).toThrow('Nonce must be exactly 32 bytes (256 bits)')
+      expect(() => new DRBG(entropy, nonce)).toThrow('Nonce must be exactly 32 bytes (256 bits)')
     })
 
     it('throws if nonce is longer than 32 bytes', () => {
       const entropy = new Array(32).fill(0x01)
       const nonce = new Array(33).fill(0x02)
 
-      expect(() => {
-        new DRBG(entropy, nonce)
-      }).toThrow('Nonce must be exactly 32 bytes (256 bits)')
+      expect(() => new DRBG(entropy, nonce)).toThrow('Nonce must be exactly 32 bytes (256 bits)')
     })
 
     it('accepts both hex strings and number[] inputs equivalently', () => {

package/src/primitives/__tests/Point.additional.test.ts

@@ -118,7 +118,7 @@ describe('Point – additional coverage', () => {
     })
 
     it('throws for unknown format', () => {
-      const der = [0x05, ...Array(32).fill(0x01)]
+      const der = [0x05, ...new Array(32).fill(0x01)]
       expect(() => Point.fromDER(der)).toThrow('Unknown point format')
     })
   })
@@ -147,7 +147,7 @@ describe('Point – additional coverage', () => {
     it('fromX accepts number', () => {
       // Use a valid x value that has a square root mod p
       const g = G.mul(new BigNumber(7))
-      const xNum = parseInt(g.getX().toString(16).slice(-4), 16)
+      const xNum = Number.parseInt(g.getX().toString(16).slice(-4), 16)
       // fromX with a number, may produce a point
       const p = Point.fromX(g.getX(), true)
       expect(p.validate()).toBe(true)

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

@@ -40,7 +40,7 @@ describe('Point.fromJSON / fromDER / fromX curve validation (TOB-24)', () => {
   it('rejects invalid compressed points in fromDER', () => {
     // 0x02 is a valid compressed prefix, but x = 0 gives y^2 = 7,
     // which has no square root mod p on secp256k1 → invalid point.
-    const der = [0x02, ...Array(32).fill(0x00)]
+    const der = [0x02, ...new Array(32).fill(0x00)]
     expect(() => Point.fromDER(der)).toThrow(/Invalid point/)
     })
 

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

@@ -37,7 +37,7 @@ describe('Cross-SDK Compatibility Tests', () => {
         // Convert hex to byte array
         const ciphertext: number[] = []
         for (let i = 0; i < ciphertextHex.length; i += 2) {
-          ciphertext.push(parseInt(ciphertextHex.slice(i, i + 2), 16))
+          ciphertext.push(Number.parseInt(ciphertextHex.slice(i, i + 2), 16))
         }
 
         // Decrypt using TypeScript SDK
@@ -83,7 +83,7 @@ describe('Cross-SDK Compatibility Tests', () => {
         // Convert hex to byte array
         const ciphertext: number[] = []
         for (let i = 0; i < ciphertextHex.length; i += 2) {
-          ciphertext.push(parseInt(ciphertextHex.slice(i, i + 2), 16))
+          ciphertext.push(Number.parseInt(ciphertextHex.slice(i, i + 2), 16))
         }
 
         // Decrypt using TypeScript SDK
@@ -127,7 +127,7 @@ describe('Cross-SDK Compatibility Tests', () => {
         // Test TypeScript decrypting Go ciphertext
         const goCiphertextBytes: number[] = []
         for (let i = 0; i < goCiphertext.length; i += 2) {
-          goCiphertextBytes.push(parseInt(goCiphertext.substr(i, 2), 16))
+          goCiphertextBytes.push(Number.parseInt(goCiphertext.substr(i, 2), 16))
         }
 
         const goDecrypted = symKey.decrypt(goCiphertextBytes, 'utf8')
@@ -136,7 +136,7 @@ describe('Cross-SDK Compatibility Tests', () => {
         // Test TypeScript decrypting TypeScript ciphertext (sanity check)
         const tsCiphertextBytes: number[] = []
         for (let i = 0; i < tsCiphertext.length; i += 2) {
-          tsCiphertextBytes.push(parseInt(tsCiphertext.substr(i, 2), 16))
+          tsCiphertextBytes.push(Number.parseInt(tsCiphertext.substr(i, 2), 16))
         }
 
         const tsDecrypted = symKey.decrypt(tsCiphertextBytes, 'utf8')

package/src/registry/__tests/RegistryClient.additional.test.ts

@@ -77,10 +77,10 @@ jest.mock('../../transaction/index.js', () => ({
 jest.mock('../../primitives/index.js', () => ({
   Utils: {
     toArray: jest.fn().mockImplementation((str: string) =>
-      Array.from(str).map((c) => c.charCodeAt(0))
+      Array.from(str).map((c) => c.codePointAt(0))
     ),
     toUTF8: jest.fn().mockImplementation((arr: number[] | string) => {
-      if (Array.isArray(arr)) return arr.map((n) => String.fromCharCode(n)).join('')
+      if (Array.isArray(arr)) return arr.map((n) => String.fromCodePoint(n)).join('')
       return arr
     })
   }
@@ -628,7 +628,7 @@ describe('RegistryClient.resolve – protocol and certificate parsing', () => {
     // protocol has 7 fields: protocolID, name, iconURL, description, docURL, operator, sig
     ;(PushDrop.decode as jest.Mock).mockReturnValueOnce({
       fields: [
-        Array.from('[1,"proto"]').map((c) => c.charCodeAt(0)), // protocolID JSON
+        Array.from('[1,"proto"]').map((c) => c.codePointAt(0)), // protocolID JSON
         [110, 97, 109, 101],  // 'name'
         [105, 99, 111, 110],  // 'icon'
         [100, 101, 115, 99],  // 'desc'
@@ -658,7 +658,7 @@ describe('RegistryClient.resolve – protocol and certificate parsing', () => {
         [105, 99, 111, 110],   // 'icon'
         [100, 101, 115, 99],   // 'desc'
         [100, 111, 99],        // 'doc'
-        Array.from(fieldsJSON).map((c) => c.charCodeAt(0)), // fieldsJSON
+        Array.from(fieldsJSON).map((c) => c.codePointAt(0)), // fieldsJSON
         [111, 112],            // 'op' - operator
         [115, 105, 103]        // signature
       ]

package/src/registry/__tests/RegistryClient.test.ts

@@ -82,11 +82,11 @@ jest.mock('../../primitives/index.js', () => {
   return {
     Utils: {
       toArray: jest.fn().mockImplementation((str: string) =>
-        Array.from(str).map((c) => c.charCodeAt(0))
+        Array.from(str).map((c) => c.codePointAt(0))
       ),
       toUTF8: jest.fn().mockImplementation((arr: number[] | string) => {
         if (Array.isArray(arr)) {
-          return arr.map((n) => String.fromCharCode(n)).join('')
+          return arr.map((n) => String.fromCodePoint(n)).join('')
         }
         return arr
       })

package/src/storage/__tests/StorageUploader.test.ts

@@ -376,7 +376,7 @@ describe('StorageUploader — multi-provider behavior', () => {
     })
 
     // All three quotes requested (up to 2 * resilienceLevel = 4 allowed, but we only have 3 hosts).
-    expect(quoteCalls.sort()).toEqual([
+    expect(quoteCalls.sort((a, b) => a.localeCompare(b))).toEqual([
       'https://a.example/quote',
       'https://b.example/quote',
       'https://c.example/quote'
@@ -611,7 +611,7 @@ describe('StorageUploader — multi-provider behavior', () => {
     })
 
     // Only the first batch of 4 quote requests should have been issued.
-    expect(quoteCalls.sort()).toEqual([
+    expect(quoteCalls.sort((a, b) => a.localeCompare(b))).toEqual([
       'https://h1.example/quote',
       'https://h2.example/quote',
       'https://h3.example/quote',
@@ -649,7 +649,7 @@ describe('StorageUploader — multi-provider behavior', () => {
       retentionPeriod: 60
     })
 
-    expect(quoteCalls.sort()).toEqual([
+    expect(quoteCalls.sort((a, b) => a.localeCompare(b))).toEqual([
       'https://h1.example/quote',
       'https://h2.example/quote',
       'https://h3.example/quote',
@@ -696,7 +696,7 @@ describe('StorageUploader — multi-provider behavior', () => {
 
     // First batch queried h1-h4 (4 hosts), second batch queried only h5
     // (1 host, the exact remainder). Hosts h6-h8 are never contacted.
-    expect(quoteCalls.sort()).toEqual([
+    expect(quoteCalls.sort((a, b) => a.localeCompare(b))).toEqual([
       'https://h1.example/quote',
       'https://h2.example/quote',
       'https://h3.example/quote',
@@ -763,7 +763,7 @@ describe('StorageUploader — multi-host findFile / listUploads / renewFile', ()
 
     const result = await uploader.findFile('uhrp://x')
 
-    expect(calls.sort()).toEqual([
+    expect(calls.sort((a, b) => a.localeCompare(b))).toEqual([
       'https://a.example/find?uhrpUrl=uhrp%3A%2F%2Fx',
       'https://b.example/find?uhrpUrl=uhrp%3A%2F%2Fx',
       'https://c.example/find?uhrpUrl=uhrp%3A%2F%2Fx'
@@ -843,7 +843,7 @@ describe('StorageUploader — multi-host findFile / listUploads / renewFile', ()
     const listing = await uploader.listUploads()
     const byUrl = Object.fromEntries(listing.map((e: any) => [e.uhrpUrl, e]))
 
-    expect(Object.keys(byUrl).sort()).toEqual(['uhrp://one', 'uhrp://shared', 'uhrp://two'])
+    expect(Object.keys(byUrl).sort((a, b) => a.localeCompare(b))).toEqual(['uhrp://one', 'uhrp://shared', 'uhrp://two'])
     expect(byUrl['uhrp://shared'].expiryTime).toBe(300) // longest wins
     expect(byUrl['uhrp://shared'].hostedBy.sort()).toEqual(['https://a.example', 'https://b.example'])
     expect(byUrl['uhrp://one'].hostedBy).toEqual(['https://a.example'])
@@ -1026,7 +1026,7 @@ describe('StorageUploader — multi-host findFile / listUploads / renewFile', ()
       hostedBy: ['https://a.example', 'https://c.example']
     })
 
-    expect(calls.sort()).toEqual([
+    expect(calls.sort((a, b) => a.localeCompare(b))).toEqual([
       'https://a.example/renew',
       'https://c.example/renew'
     ])

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

@@ -703,7 +703,7 @@ describe('Transaction', () => {
             sequence: 0xffffffff
           }
         ],
-        Array(21)
+        new Array(21)
           .fill(null)
           .map(() => ({
             lockingScript: p2pkh.lock(publicKeyHash),
@@ -1508,7 +1508,7 @@ describe('Transaction', () => {
 
             tx.addInput({
               sourceTXID: txid,
-              sourceOutputIndex: parseInt(voutStr),
+              sourceOutputIndex: Number.parseInt(voutStr),
               unlockingScript
             })
           }
@@ -1553,7 +1553,7 @@ describe('Transaction', () => {
         for (const [index, spend] of Object.entries(args.spends)) {
           tx.addInput({
             sourceTXID: '00'.repeat(32),
-            sourceOutputIndex: parseInt(index),
+            sourceOutputIndex: Number.parseInt(index),
             unlockingScript: Script.fromHex((spend as any).unlockingScript)
           })
         }