@pellux/goodvibes-sdk 0.33.38 → 0.34.1

package/dist/platform/utils/error-display.js

@@ -177,6 +177,17 @@ function extractErrorMessage(error) {
     }
     return String(error);
 }
+/**
+ * redactedErrorMessage — extracts and redacts a human-readable message from any thrown value.
+ *
+ * Use for user-facing error output where the raw message should never leak
+ * bearer tokens, API keys, or other sensitive data. Do NOT use for logger
+ * fields, structured result/error fields, or diagnostic context — use
+ * `summarizeError` or the structured `normalizeError` for those.
+ */
+export function redactedErrorMessage(error) {
+    return redactSensitiveData(extractErrorMessage(error));
+}
 export function normalizeError(error, options = {}) {
     // Redact Bearer tokens and API keys before any further processing.
     const rawMessage = redactSensitiveData(extractErrorMessage(error));

package/dist/platform/utils/fetch-with-timeout.d.ts

@@ -15,7 +15,25 @@ export declare function sanitizeUrlForLog(url: string | URL | Request): string;
  */
 export declare function instrumentedFetch(url: string | URL | Request, init?: RequestInit): Promise<Response>;
 /**
- * fetchWithTimeout — wraps the global fetch with an AbortController timeout.
+ * createTimeoutController — creates an AbortController that fires after `timeoutMs`.
+ *
+ * When `parentSignal` is provided the returned signal is merged with it via
+ * AbortSignal.any so whichever fires first wins.
+ *
+ * Uses the faster `AbortSignal.timeout` fast-path when available and no parent
+ * signal needs to be merged.
+ *
+ * @param timeoutMs    - Milliseconds before aborting.
+ * @param parentSignal - Optional caller signal to merge with the timeout.
+ * @returns `{ signal, dispose }` — call `dispose()` in a `finally` block to
+ *   clear the underlying timer and avoid keeping the event loop alive.
+ */
+export declare function createTimeoutController(timeoutMs: number, parentSignal?: AbortSignal): {
+    readonly signal: AbortSignal;
+    dispose(): void;
+};
+/**
+ * fetchWithTimeout — wraps a fetch implementation with an AbortController timeout.
  *
  * If the caller passes a signal that is already aborted, the request is
  * rejected immediately. When both a caller signal and the internal timeout
@@ -29,6 +47,8 @@ export declare function instrumentedFetch(url: string | URL | Request, init?: Re
  * @param url       - The URL or Request to fetch.
  * @param init      - Standard RequestInit (optional).
  * @param timeoutMs - Milliseconds before aborting. Default: 30 000.
+ * @param fetchImpl - Fetch implementation to use. Defaults to global `fetch`.
+ *   Pass `instrumentedFetch` to include OUTBOUND_HTTP logging.
  */
-export declare function fetchWithTimeout(url: string | URL | Request, init?: RequestInit, timeoutMs?: number): Promise<Response>;
+export declare function fetchWithTimeout(url: string | URL | Request, init?: RequestInit, timeoutMs?: number, fetchImpl?: (url: string | URL | Request, init?: RequestInit) => Promise<Response>): Promise<Response>;
 //# sourceMappingURL=fetch-with-timeout.d.ts.map

package/dist/platform/utils/fetch-with-timeout.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"fetch-with-timeout.d.ts","sourceRoot":"","sources":["../../../src/platform/utils/fetch-with-timeout.ts"],"names":[],"mappings":"AAWA;;GAEG;AACH,wBAAgB,iBAAiB,CAAC,GAAG,EAAE,MAAM,GAAG,GAAG,GAAG,OAAO,GAAG,MAAM,CAarE;AAED;;;;;;;;;;GAUG;AACH,wBAAsB,iBAAiB,CACrC,GAAG,EAAE,MAAM,GAAG,GAAG,GAAG,OAAO,EAC3B,IAAI,CAAC,EAAE,WAAW,GACjB,OAAO,CAAC,QAAQ,CAAC,CAkBnB;AAED;;;;;;;;;;;;;;;GAeG;AACH,wBAAsB,gBAAgB,CACpC,GAAG,EAAE,MAAM,GAAG,GAAG,GAAG,OAAO,EAC3B,IAAI,CAAC,EAAE,WAAW,EAClB,SAAS,SAAS,GACjB,OAAO,CAAC,QAAQ,CAAC,CAmBnB"}
+{"version":3,"file":"fetch-with-timeout.d.ts","sourceRoot":"","sources":["../../../src/platform/utils/fetch-with-timeout.ts"],"names":[],"mappings":"AAWA;;GAEG;AACH,wBAAgB,iBAAiB,CAAC,GAAG,EAAE,MAAM,GAAG,GAAG,GAAG,OAAO,GAAG,MAAM,CAarE;AAED;;;;;;;;;;GAUG;AACH,wBAAsB,iBAAiB,CACrC,GAAG,EAAE,MAAM,GAAG,GAAG,GAAG,OAAO,EAC3B,IAAI,CAAC,EAAE,WAAW,GACjB,OAAO,CAAC,QAAQ,CAAC,CAkBnB;AAED;;;;;;;;;;;;;GAaG;AACH,wBAAgB,uBAAuB,CACrC,SAAS,EAAE,MAAM,EACjB,YAAY,CAAC,EAAE,WAAW,GACzB;IAAE,QAAQ,CAAC,MAAM,EAAE,WAAW,CAAC;IAAC,OAAO,IAAI,IAAI,CAAA;CAAE,CAcnD;AAED;;;;;;;;;;;;;;;;;GAiBG;AACH,wBAAsB,gBAAgB,CACpC,GAAG,EAAE,MAAM,GAAG,GAAG,GAAG,OAAO,EAC3B,IAAI,CAAC,EAAE,WAAW,EAClB,SAAS,SAAS,EAClB,SAAS,GAAE,CAAC,GAAG,EAAE,MAAM,GAAG,GAAG,GAAG,OAAO,EAAE,IAAI,CAAC,EAAE,WAAW,KAAK,OAAO,CAAC,QAAQ,CAAS,GACxF,OAAO,CAAC,QAAQ,CAAC,CAQnB"}

package/dist/platform/utils/fetch-with-timeout.js

@@ -57,7 +57,33 @@ export async function instrumentedFetch(url, init) {
     }
 }
 /**
- * fetchWithTimeout — wraps the global fetch with an AbortController timeout.
+ * createTimeoutController — creates an AbortController that fires after `timeoutMs`.
+ *
+ * When `parentSignal` is provided the returned signal is merged with it via
+ * AbortSignal.any so whichever fires first wins.
+ *
+ * Uses the faster `AbortSignal.timeout` fast-path when available and no parent
+ * signal needs to be merged.
+ *
+ * @param timeoutMs    - Milliseconds before aborting.
+ * @param parentSignal - Optional caller signal to merge with the timeout.
+ * @returns `{ signal, dispose }` — call `dispose()` in a `finally` block to
+ *   clear the underlying timer and avoid keeping the event loop alive.
+ */
+export function createTimeoutController(timeoutMs, parentSignal) {
+    if (typeof AbortSignal.timeout === 'function' && !parentSignal) {
+        return { signal: AbortSignal.timeout(timeoutMs), dispose: () => { } };
+    }
+    const controller = new AbortController();
+    const timer = setTimeout(() => controller.abort(new DOMException('Request timed out', 'TimeoutError')), timeoutMs);
+    timer.unref?.();
+    const signal = parentSignal
+        ? AbortSignal.any([parentSignal, controller.signal])
+        : controller.signal;
+    return { signal, dispose: () => clearTimeout(timer) };
+}
+/**
+ * fetchWithTimeout — wraps a fetch implementation with an AbortController timeout.
  *
  * If the caller passes a signal that is already aborted, the request is
  * rejected immediately. When both a caller signal and the internal timeout
@@ -71,24 +97,16 @@ export async function instrumentedFetch(url, init) {
  * @param url       - The URL or Request to fetch.
  * @param init      - Standard RequestInit (optional).
  * @param timeoutMs - Milliseconds before aborting. Default: 30 000.
+ * @param fetchImpl - Fetch implementation to use. Defaults to global `fetch`.
+ *   Pass `instrumentedFetch` to include OUTBOUND_HTTP logging.
  */
-export async function fetchWithTimeout(url, init, timeoutMs = 30_000) {
-    const controller = new AbortController();
-    const timer = setTimeout(() => controller.abort(new DOMException('Request timed out', 'TimeoutError')), timeoutMs);
-    timer.unref?.();
-    let signal;
+export async function fetchWithTimeout(url, init, timeoutMs = 30_000, fetchImpl = fetch) {
     const callerSignal = init?.signal;
-    if (callerSignal) {
-        // Merge caller signal + our timeout: first one to fire wins
-        signal = AbortSignal.any([callerSignal, controller.signal]);
-    }
-    else {
-        signal = controller.signal;
-    }
+    const { signal, dispose } = createTimeoutController(timeoutMs, callerSignal);
     try {
-        return await fetch(url, { ...init, signal });
+        return await fetchImpl(url, { ...init, signal });
     }
     finally {
-        clearTimeout(timer);
+        dispose();
     }
 }

package/dist/platform/utils/record-coerce.d.ts

@@ -8,4 +8,9 @@
  * satisfy TypeScript's type system at the generic-dispatch boundary.
  */
 export declare function toRecord(value: unknown): Record<string, unknown>;
+/**
+ * Type guard: returns true iff `value` is a plain object (non-null, non-array).
+ * Use this in place of the commonly hand-rolled `typeof x === 'object' && x !== null && !Array.isArray(x)`.
+ */
+export declare function isRecord(value: unknown): value is Record<string, unknown>;
 //# sourceMappingURL=record-coerce.d.ts.map

package/dist/platform/utils/record-coerce.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"record-coerce.d.ts","sourceRoot":"","sources":["../../../src/platform/utils/record-coerce.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AACH,wBAAgB,QAAQ,CAAC,KAAK,EAAE,OAAO,GAAG,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAEhE"}
+{"version":3,"file":"record-coerce.d.ts","sourceRoot":"","sources":["../../../src/platform/utils/record-coerce.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AACH,wBAAgB,QAAQ,CAAC,KAAK,EAAE,OAAO,GAAG,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAEhE;AAED;;;GAGG;AACH,wBAAgB,QAAQ,CAAC,KAAK,EAAE,OAAO,GAAG,KAAK,IAAI,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAEzE"}

package/dist/platform/utils/record-coerce.js

@@ -10,3 +10,10 @@
 export function toRecord(value) {
     return value;
 }
+/**
+ * Type guard: returns true iff `value` is a plain object (non-null, non-array).
+ * Use this in place of the commonly hand-rolled `typeof x === 'object' && x !== null && !Array.isArray(x)`.
+ */
+export function isRecord(value) {
+    return typeof value === 'object' && value !== null && !Array.isArray(value);
+}

package/dist/platform/utils/retry.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"retry.d.ts","sourceRoot":"","sources":["../../../src/platform/utils/retry.ts"],"names":[],"mappings":"AAIA,kEAAkE;AAClE,MAAM,WAAW,WAAW;IAC1B,UAAU,EAAE,MAAM,CAAC;IACnB,cAAc,EAAE,MAAM,CAAC;IACvB,UAAU,EAAE,MAAM,CAAC;CACpB;AAQD,kFAAkF;AAClF,wBAAgB,aAAa,CAAC,GAAG,EAAE,OAAO,GAAG,GAAG,IAAI;IAAE,UAAU,EAAE,MAAM,CAAA;CAAE,CAOzE;AAED,8EAA8E;AAC9E,wBAAgB,SAAS,CAAC,GAAG,EAAE,OAAO,GAAG,GAAG,IAAI;IAAE,MAAM,EAAE,MAAM,CAAA;CAAE,CAOjE;AAED;;;;GAIG;AACH,wBAAgB,gBAAgB,CAAC,KAAK,EAAE,OAAO,GAAG,OAAO,CAaxD;AASD;;;;;;;GAOG;AACH,wBAAsB,SAAS,CAAC,CAAC,EAC/B,EAAE,EAAE,MAAM,OAAO,CAAC,CAAC,CAAC,EACpB,MAAM,CAAC,EAAE,OAAO,CAAC,WAAW,CAAC,EAC7B,OAAO,CAAC,EAAE,CAAC,OAAO,EAAE,MAAM,EAAE,KAAK,EAAE,KAAK,EAAE,OAAO,EAAE,MAAM,KAAK,IAAI,GACjE,OAAO,CAAC,CAAC,CAAC,CA4BZ"}
+{"version":3,"file":"retry.d.ts","sourceRoot":"","sources":["../../../src/platform/utils/retry.ts"],"names":[],"mappings":"AAKA,kEAAkE;AAClE,MAAM,WAAW,WAAW;IAC1B,UAAU,EAAE,MAAM,CAAC;IACnB,cAAc,EAAE,MAAM,CAAC;IACvB,UAAU,EAAE,MAAM,CAAC;CACpB;AAQD,kFAAkF;AAClF,wBAAgB,aAAa,CAAC,GAAG,EAAE,OAAO,GAAG,GAAG,IAAI;IAAE,UAAU,EAAE,MAAM,CAAA;CAAE,CAOzE;AAED,8EAA8E;AAC9E,wBAAgB,SAAS,CAAC,GAAG,EAAE,OAAO,GAAG,GAAG,IAAI;IAAE,MAAM,EAAE,MAAM,CAAA;CAAE,CAOjE;AAED;;;;GAIG;AACH,wBAAgB,gBAAgB,CAAC,KAAK,EAAE,OAAO,GAAG,OAAO,CAaxD;AASD;;;;;;;GAOG;AACH,wBAAsB,SAAS,CAAC,CAAC,EAC/B,EAAE,EAAE,MAAM,OAAO,CAAC,CAAC,CAAC,EACpB,MAAM,CAAC,EAAE,OAAO,CAAC,WAAW,CAAC,EAC7B,OAAO,CAAC,EAAE,CAAC,OAAO,EAAE,MAAM,EAAE,KAAK,EAAE,KAAK,EAAE,OAAO,EAAE,MAAM,KAAK,IAAI,GACjE,OAAO,CAAC,CAAC,CAAC,CAyBZ"}

package/dist/platform/utils/retry.js

@@ -1,6 +1,7 @@
 import { randomInt } from 'node:crypto';
 import { AppError, RETRYABLE_STATUS_CODES } from '../types/errors.js';
 import { summarizeError } from './error-display.js';
+import { sleep } from './concurrency.js';
 const DEFAULT_CONFIG = {
     maxRetries: 3,
     initialDelayMs: 1000,
@@ -70,10 +71,7 @@ export async function withRetry(fn, config, onRetry) {
             }
             const delayMs = computeDelay(attempt, cfg.initialDelayMs, cfg.maxDelayMs);
             onRetry?.(attempt + 1, lastError, delayMs);
-            await new Promise((resolve) => {
-                const timer = setTimeout(resolve, delayMs);
-                timer.unref?.();
-            });
+            await sleep(delayMs);
         }
     }
     throw lastError;

package/dist/platform/version.js

@@ -1,6 +1,6 @@
 import { readFileSync } from 'node:fs';
 import { join } from 'node:path';
-let version = '0.33.38';
+let version = '0.34.1';
 try {
     const pkg = JSON.parse(readFileSync(join(import.meta.dir, '..', '..', 'package.json'), 'utf-8'));
     version = pkg.version ?? version;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pellux/goodvibes-sdk",
-  "version": "0.33.38",
+  "version": "0.34.1",
   "description": "TypeScript SDK for building GoodVibes operator, peer, web, mobile, and daemon-connected apps with typed contracts, auth, realtime events, and transport layers.",
   "keywords": [
     "goodvibes",
@@ -453,14 +453,14 @@
   "sideEffects": false,
   "type": "module",
   "dependencies": {
-    "@pellux/goodvibes-contracts": "0.33.38",
-    "@pellux/goodvibes-daemon-sdk": "0.33.38",
-    "@pellux/goodvibes-errors": "0.33.38",
-    "@pellux/goodvibes-operator-sdk": "0.33.38",
-    "@pellux/goodvibes-peer-sdk": "0.33.38",
-    "@pellux/goodvibes-transport-core": "0.33.38",
-    "@pellux/goodvibes-transport-http": "0.33.38",
-    "@pellux/goodvibes-transport-realtime": "0.33.38"
+    "@pellux/goodvibes-contracts": "0.34.1",
+    "@pellux/goodvibes-daemon-sdk": "0.34.1",
+    "@pellux/goodvibes-errors": "0.34.1",
+    "@pellux/goodvibes-operator-sdk": "0.34.1",
+    "@pellux/goodvibes-peer-sdk": "0.34.1",
+    "@pellux/goodvibes-transport-core": "0.34.1",
+    "@pellux/goodvibes-transport-http": "0.34.1",
+    "@pellux/goodvibes-transport-realtime": "0.34.1"
   },
   "optionalDependencies": {
     "@agentclientprotocol/sdk": "^0.21.0",