@bsv/sdk 2.0.13 → 2.0.15

package/src/storage/index.ts

@@ -1,5 +1,5 @@
 export * as StorageUtils from './StorageUtils.js'
 
-export { StorageUploader } from './StorageUploader.js'
+export { StorageUploader, DEFAULT_UHRP_SERVERS, RenewResiliencyError } from './StorageUploader.js'
 
 export { StorageDownloader } from './StorageDownloader.js'

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

@@ -36,13 +36,33 @@ describe('ARC Broadcaster', () => {
     transaction = new Transaction()
   })
 
+  async function withoutGlobalFetch<T> (callback: () => Promise<T>): Promise<T> {
+    const originalFetchDescriptor = Object.getOwnPropertyDescriptor(globalThis, 'fetch')
+    Object.defineProperty(globalThis, 'fetch', {
+      configurable: true,
+      writable: true,
+      value: undefined
+    })
+    try {
+      return await callback()
+    } finally {
+      if (originalFetchDescriptor !== undefined) {
+        Object.defineProperty(globalThis, 'fetch', originalFetchDescriptor)
+      } else {
+        delete (globalThis as { fetch?: unknown }).fetch
+      }
+    }
+  }
+
   it('should broadcast successfully using window.fetch', async () => {
     // Mocking window.fetch
     const mockFetch = mockedFetch(successResponse)
     global.window = { fetch: mockFetch } as unknown as Window & typeof globalThis
 
-    const broadcaster = new ARC(URL)
-    const response = await broadcaster.broadcast(transaction)
+    const response = await withoutGlobalFetch(async () => {
+      const broadcaster = new ARC(URL)
+      return await broadcaster.broadcast(transaction)
+    })
 
     expect(mockFetch).toHaveBeenCalled()
     expect(response).toEqual({
@@ -59,8 +79,10 @@ describe('ARC Broadcaster', () => {
       delete (globalThis as { window?: unknown }).window // ✅ Explicit property check
     }
 
-    const broadcaster = new ARC(URL)
-    const response = await broadcaster.broadcast(transaction)
+    const response = await withoutGlobalFetch(async () => {
+      const broadcaster = new ARC(URL)
+      return await broadcaster.broadcast(transaction)
+    })
 
     expect(response).toEqual({
       status: 'success',

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

@@ -30,13 +30,33 @@ describe('WhatsOnChainBroadcaster', () => {
     transaction = new Transaction()
   })
 
+  async function withoutGlobalFetch<T> (callback: () => Promise<T>): Promise<T> {
+    const originalFetchDescriptor = Object.getOwnPropertyDescriptor(globalThis, 'fetch')
+    Object.defineProperty(globalThis, 'fetch', {
+      configurable: true,
+      writable: true,
+      value: undefined
+    })
+    try {
+      return await callback()
+    } finally {
+      if (originalFetchDescriptor !== undefined) {
+        Object.defineProperty(globalThis, 'fetch', originalFetchDescriptor)
+      } else {
+        delete (globalThis as { fetch?: unknown }).fetch
+      }
+    }
+  }
+
   it('should broadcast successfully using window.fetch', async () => {
     // Mocking window.fetch
     const mockFetch = mockedFetch(successResponse)
     global.window = { fetch: mockFetch } as any
 
-    const broadcaster = new WhatsOnChainBroadcaster(network)
-    const response = await broadcaster.broadcast(transaction)
+    const response = await withoutGlobalFetch(async () => {
+      const broadcaster = new WhatsOnChainBroadcaster(network)
+      return await broadcaster.broadcast(transaction)
+    })
 
     expect(mockFetch).toHaveBeenCalled()
     expect(response).toEqual({
@@ -51,8 +71,10 @@ describe('WhatsOnChainBroadcaster', () => {
     mockedHttps(successResponse)
     global.window = {} as any
 
-    const broadcaster = new WhatsOnChainBroadcaster(network)
-    const response = await broadcaster.broadcast(transaction)
+    const response = await withoutGlobalFetch(async () => {
+      const broadcaster = new WhatsOnChainBroadcaster(network)
+      return await broadcaster.broadcast(transaction)
+    })
 
     expect(response).toEqual({
       status: 'success',

package/src/transaction/chaintrackers/__tests/WhatsOnChainChainTracker.test.ts

@@ -14,16 +14,36 @@ describe('WhatsOnChain ChainTracker', () => {
     }
   }
 
+  async function withoutGlobalFetch<T> (callback: () => Promise<T>): Promise<T> {
+    const originalFetchDescriptor = Object.getOwnPropertyDescriptor(globalThis, 'fetch')
+    Object.defineProperty(globalThis, 'fetch', {
+      configurable: true,
+      writable: true,
+      value: undefined
+    })
+    try {
+      return await callback()
+    } finally {
+      if (originalFetchDescriptor !== undefined) {
+        Object.defineProperty(globalThis, 'fetch', originalFetchDescriptor)
+      } else {
+        delete (globalThis as { fetch?: unknown }).fetch
+      }
+    }
+  }
+
   it('should verify merkleroot successfully using window.fetch', async () => {
     // Mocking window.fetch
     const mockFetch = mockedFetch(successResponse)
     global.window = { fetch: mockFetch } as any
 
-    const chainTracker = new WhatsOnChain(network)
-    const response = await chainTracker.isValidRootForHeight(
-      merkleroot,
-      height
-    )
+    const response = await withoutGlobalFetch(async () => {
+      const chainTracker = new WhatsOnChain(network)
+      return await chainTracker.isValidRootForHeight(
+        merkleroot,
+        height
+      )
+    })
 
     expect(mockFetch).toHaveBeenCalled()
     expect(response).toEqual(true)
@@ -34,11 +54,13 @@ describe('WhatsOnChain ChainTracker', () => {
     mockedHttps(successResponse)
     if (typeof global.window !== 'undefined') global.window = {} as any
 
-    const chainTracker = new WhatsOnChain(network)
-    const response = await chainTracker.isValidRootForHeight(
-      merkleroot,
-      height
-    )
+    const response = await withoutGlobalFetch(async () => {
+      const chainTracker = new WhatsOnChain(network)
+      return await chainTracker.isValidRootForHeight(
+        merkleroot,
+        height
+      )
+    })
 
     expect(response).toEqual(true)
   })

package/src/transaction/http/BinaryFetchClient.ts

@@ -99,10 +99,13 @@ export function binaryHttpClient(): HttpClient {
   }
 
   if (typeof window !== 'undefined' && typeof window.fetch === 'function') {
-    // Use fetch in a browser environment
+    // Browser tab/page context
     return new BinaryFetchClient(window.fetch.bind(window))
+  } else if (typeof globalThis.fetch === 'function') {
+    // Service workers, Deno, Node 18+ (any environment with global fetch)
+    return new BinaryFetchClient(globalThis.fetch.bind(globalThis))
   } else if (typeof require !== 'undefined') {
-    // Use Node https module
+    // Older Node.js — use https module
     // eslint-disable-next-line
     try {
       // eslint-disable-next-line @typescript-eslint/no-var-requires

package/src/transaction/http/DefaultHttpClient.ts

@@ -4,8 +4,8 @@ import { FetchHttpClient } from './FetchHttpClient.js'
 
 /**
  * Returns a default HttpClient implementation based on the environment that it is run on.
- * This method will attempt to use `window.fetch` if available (in browser environments).
- * If running in a Node environment, it falls back to using the Node `https` module
+ * This method will attempt to use `window.fetch` if available (in browser environments),
+ * then `globalThis.fetch` (service workers, Deno, Node 18+), then the Node `https` module.
  */
 export function defaultHttpClient(): HttpClient {
   const noHttpClient: HttpClient = {
@@ -15,10 +15,13 @@ export function defaultHttpClient(): HttpClient {
   }
 
   if (typeof window !== 'undefined' && typeof window.fetch === 'function') {
-    // Use fetch in a browser environment
+    // Browser tab/page context
     return new FetchHttpClient(window.fetch.bind(window))
+  } else if (typeof globalThis.fetch === 'function') {
+    // Service workers, Deno, Node 18+ (any environment with global fetch)
+    return new FetchHttpClient(globalThis.fetch.bind(globalThis))
   } else if (typeof require !== 'undefined') {
-    // Use Node https module
+    // Older Node.js — use https module
     // eslint-disable-next-line
     try {
       // eslint-disable-next-line @typescript-eslint/no-var-requires

package/src/transaction/http/__tests/DefaultHttpClient.additional.test.ts

@@ -7,10 +7,17 @@
 // allowing us to control whether `https` throws on require.
 
 describe('defaultHttpClient — noHttpClient fallback paths', () => {
+  let savedFetch: typeof globalThis.fetch | undefined
+
+  beforeEach(() => {
+    savedFetch = globalThis.fetch
+  })
+
   afterEach(() => {
     if ('window' in globalThis) {
       delete (globalThis as { window?: unknown }).window
     }
+    globalThis.fetch = savedFetch as typeof globalThis.fetch
     jest.resetModules()
   })
 
@@ -25,10 +32,11 @@ describe('defaultHttpClient — noHttpClient fallback paths', () => {
       throw new Error('https module not available')
     })
 
-    // window must be absent so the fetch branch is skipped
+    // window and globalThis.fetch must be absent so the fetch branches are skipped
     if ('window' in globalThis) {
       delete (globalThis as { window?: unknown }).window
     }
+    delete (globalThis as { fetch?: unknown }).fetch
 
     let defaultHttpClient: any
 
@@ -61,6 +69,7 @@ describe('defaultHttpClient — noHttpClient fallback paths', () => {
     if ('window' in globalThis) {
       delete (globalThis as { window?: unknown }).window
     }
+    delete (globalThis as { fetch?: unknown }).fetch
 
     let client: any
 
@@ -111,10 +120,17 @@ describe('defaultHttpClient — noHttpClient fallback paths', () => {
 // --------------------------------------------------------------------------
 
 describe('binaryHttpClient — noHttpClient fallback paths', () => {
+  let savedFetch: typeof globalThis.fetch | undefined
+
+  beforeEach(() => {
+    savedFetch = globalThis.fetch
+  })
+
   afterEach(() => {
     if ('window' in globalThis) {
       delete (globalThis as { window?: unknown }).window
     }
+    globalThis.fetch = savedFetch as typeof globalThis.fetch
     jest.resetModules()
   })
 
@@ -126,6 +142,7 @@ describe('binaryHttpClient — noHttpClient fallback paths', () => {
     if ('window' in globalThis) {
       delete (globalThis as { window?: unknown }).window
     }
+    delete (globalThis as { fetch?: unknown }).fetch
 
     let binaryHttpClient: any
 
@@ -153,6 +170,7 @@ describe('binaryHttpClient — noHttpClient fallback paths', () => {
     if ('window' in globalThis) {
       delete (globalThis as { window?: unknown }).window
     }
+    delete (globalThis as { fetch?: unknown }).fetch
 
     let client: any
 

package/src/wallet/WERR_REVIEW_ACTIONS.ts

@@ -2,7 +2,7 @@ import { AtomicBEEF, OutpointString, ReviewActionResult, SendWithResult, TXIDHex
 
 /**
  * When a `createAction` or `signAction` is completed in undelayed mode (`acceptDelayedBroadcast`: false),
- * any unsucccessful result will return the results by way of this exception to ensure attention is
+ * any unsuccessful result will return the results by way of this exception to ensure attention is
  * paid to processing errors.
  */
 export class WERR_REVIEW_ACTIONS extends Error {
@@ -10,7 +10,7 @@ export class WERR_REVIEW_ACTIONS extends Error {
   isError: boolean = true
 
   /**
-   * All parameters correspond to their comparable `createAction` or `signSction` results
+   * All parameters correspond to their comparable `createAction` or `signAction` results
    * with the exception of `reviewActionResults`;
    * which contains more details, particularly for double spend results.
    */

package/src/wallet/Wallet.interfaces.ts

@@ -305,10 +305,10 @@ export interface SendWithResult {
 
 /**
  * Indicates status of a new Action following a `createAction` or `signAction` in immediate mode:
- * When `acceptDelayedBroadcast` is falses.
+ * When `acceptDelayedBroadcast` is false.
  *
  * 'success': The action has been broadcast and accepted by the bitcoin processing network.
- * 'doulbeSpend': The action has been confirmed to double spend one or more inputs, and by the "first-seen-rule" is the loosing transaction.
+ * 'doubleSpend': The action has been confirmed to double spend one or more inputs, and by the "first-seen-rule" is the losing transaction.
  * 'invalidTx': The action was rejected by the processing network as an invalid bitcoin transaction.
  * 'serviceError': The broadcast services are currently unable to reach the bitcoin network. The action is now queued for delayed retries.
  */