Merge pull request #369 from bsv-blockchain/fix/default-fetch-binding

Fix/default fetch binding

Author: ty-everett

CHANGELOG.md

@@ -5,6 +5,7 @@ All notable changes to this project will be documented in this file. The format
 ## Table of Contents
 
 - [Unreleased](#unreleased)
+- [1.8.9 - 2025-10-27](#189---2025-10-27)
 - [1.8.8 - 2025-10-22](#188---2025-10-22)
 - [1.8.7 - 2025-10-22](#187---2025-10-22)
 - [1.8.5 - 2025-10-21](#185---2025-10-21)
@@ -168,6 +169,14 @@ All notable changes to this project will be documented in this file. The format
 
 ---
 
+### [1.8.9] - 2025-10-27
+
+### Fixed
+
+- Default fetch binding now works in more environments.
+
+---
+
 ### [1.8.8] - 2025-10-22
 
 ### Added

docs/reference/kvstore.md

@@ -248,6 +248,7 @@ export interface KVStoreQuery {
     controller?: PubKeyHex;
     protocolID?: WalletProtocol;
     tags?: string[];
+    tagQueryMode?: "all" | "any";
     limit?: number;
     skip?: number;
     sortOrder?: "asc" | "desc";
@@ -256,6 +257,16 @@ export interface KVStoreQuery {
 
 See also: [PubKeyHex](./wallet.md#type-pubkeyhex), [WalletProtocol](./wallet.md#type-walletprotocol)
 
+#### Property tagQueryMode
+
+Controls tag matching behavior when tags are specified.
+- 'all': Requires all specified tags to be present (default)
+- 'any': Requires at least one of the specified tags to be present
+
+```ts
+tagQueryMode?: "all" | "any"
+```
+
 Links: [API](#api), [Interfaces](#interfaces), [Classes](#classes), [Functions](#functions), [Types](#types), [Enums](#enums), [Variables](#variables)
 
 ---

package-lock.json

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

package.json

@@ -3,8 +3,10 @@
 import { AuthMessage, RequestedCertificateSet, Transport } from '../types.js'
 import * as Utils from '../../primitives/utils.js'
 
-// Only bind window.fetch in the browser
-const defaultFetch = typeof window !== 'undefined' ? fetch.bind(window) : fetch
+const defaultFetch: typeof fetch = 
+  typeof globalThis !== 'undefined' && typeof globalThis.fetch === 'function'
+    ? globalThis.fetch.bind(globalThis)
+    : fetch
 
 /**
  * Implements an HTTP-specific transport for handling Peer mutual authentication messages.
@@ -21,6 +23,12 @@ export class SimplifiedFetchTransport implements Transport {
    * @param fetchClient - A fetch implementation to use for HTTP requests (default: global fetch).
    */
   constructor (baseUrl: string, fetchClient = defaultFetch) {
+    if (typeof fetchClient !== 'function') {
+      throw new Error(
+        'SimplifiedFetchTransport requires a fetch implementation. ' +
+        'In environments without fetch, provide a polyfill or custom implementation.'
+      )
+    }
     this.fetchClient = fetchClient
     this.baseUrl = baseUrl
   }

src/auth/transports/SimplifiedFetchTransport.ts

@@ -2,8 +2,10 @@ import { Transaction } from '../transaction/index.js'
 import OverlayAdminTokenTemplate from './OverlayAdminTokenTemplate.js'
 import * as Utils from '../primitives/utils.js'
 
-// Only bind window.fetch in the browser
-const defaultFetch = typeof window !== 'undefined' ? fetch.bind(window) : fetch
+const defaultFetch: typeof fetch =
+  typeof globalThis !== 'undefined' && typeof globalThis.fetch === 'function'
+    ? globalThis.fetch.bind(globalThis)
+    : fetch
 
 /**
  * The question asked to the Overlay Services Engine when a consumer of state wishes to look up information.
@@ -113,6 +115,12 @@ export class HTTPSOverlayLookupFacilitator implements OverlayLookupFacilitator {
   allowHTTP: boolean
 
   constructor (httpClient = defaultFetch, allowHTTP: boolean = false) {
+    if (typeof httpClient !== 'function') {
+      throw new Error(
+        'HTTPSOverlayLookupFacilitator requires a fetch implementation. ' +
+        'In environments without fetch, provide a polyfill or custom implementation.'
+      )
+    }
     this.fetchClient = httpClient
     this.allowHTTP = allowHTTP
   }