@bsv/sdk 2.0.14 → 2.0.16

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