@bitfab/sdk 0.52.1 → 0.52.3

package/dist/index.d.cts

@@ -1656,12 +1656,16 @@ interface ReplayOptions {
      * Maximum number of traces to replay (1-5,000, default 5). Ignored when
      * `traceIds` is passed (with a warning), or when `datasetId` / `datasetIds`
      * is passed, because either source already determines how many traces replay.
+     * To replay part of a dataset selection, name the members to run in
+     * `traceIds` rather than bounding it here.
      */
     limit?: number;
     attempts?: number;
     /**
-     * Optional list of specific trace IDs to replay (max 100). Mutually exclusive
-     * with `datasetId` and `datasetIds`.
+     * Optional list of specific trace IDs to replay (max 100). Passed alongside
+     * `datasetId` or `datasetIds` it pins which members of that selection replay,
+     * and the server rejects any ID none of those datasets contains. On its own
+     * it selects by ID with no dataset attribution.
      */
     traceIds?: string[];
     /** Optional display name for the resulting experiment/test run. */
@@ -1719,8 +1723,9 @@ interface ReplayOptions {
     /** Group ID to associate this replay with an experiment group for live streaming in Studio. */
     experimentGroupId?: string;
     /**
-     * Dataset this replay runs against. Mutually exclusive with `traceIds` and
-     * with `datasetIds`. When set, the resulting experiment is
+     * Dataset this replay runs against. Mutually exclusive with `datasetIds`.
+     * Alone it replays the dataset's full membership, and with `traceIds` it
+     * replays only those members. When set, the resulting experiment is
      * durably attributed to the dataset (stored on the test run), so it appears
      * under the dataset's experiments even if the trace lineage can't be
      * reconstructed. Validated server-side against the org.
@@ -3288,7 +3293,7 @@ declare class BitfabFunction {
 /**
  * SDK version from package.json (injected at build time)
  */
-declare const __version__ = "0.52.1";
+declare const __version__ = "0.52.3";
 
 /**
  * Constants for the Bitfab SDK.

package/dist/index.d.ts

@@ -1656,12 +1656,16 @@ interface ReplayOptions {
      * Maximum number of traces to replay (1-5,000, default 5). Ignored when
      * `traceIds` is passed (with a warning), or when `datasetId` / `datasetIds`
      * is passed, because either source already determines how many traces replay.
+     * To replay part of a dataset selection, name the members to run in
+     * `traceIds` rather than bounding it here.
      */
     limit?: number;
     attempts?: number;
     /**
-     * Optional list of specific trace IDs to replay (max 100). Mutually exclusive
-     * with `datasetId` and `datasetIds`.
+     * Optional list of specific trace IDs to replay (max 100). Passed alongside
+     * `datasetId` or `datasetIds` it pins which members of that selection replay,
+     * and the server rejects any ID none of those datasets contains. On its own
+     * it selects by ID with no dataset attribution.
      */
     traceIds?: string[];
     /** Optional display name for the resulting experiment/test run. */
@@ -1719,8 +1723,9 @@ interface ReplayOptions {
     /** Group ID to associate this replay with an experiment group for live streaming in Studio. */
     experimentGroupId?: string;
     /**
-     * Dataset this replay runs against. Mutually exclusive with `traceIds` and
-     * with `datasetIds`. When set, the resulting experiment is
+     * Dataset this replay runs against. Mutually exclusive with `datasetIds`.
+     * Alone it replays the dataset's full membership, and with `traceIds` it
+     * replays only those members. When set, the resulting experiment is
      * durably attributed to the dataset (stored on the test run), so it appears
      * under the dataset's experiments even if the trace lineage can't be
      * reconstructed. Validated server-side against the org.
@@ -3288,7 +3293,7 @@ declare class BitfabFunction {
 /**
  * SDK version from package.json (injected at build time)
  */
-declare const __version__ = "0.52.1";
+declare const __version__ = "0.52.3";
 
 /**
  * Constants for the Bitfab SDK.

package/dist/index.js

@@ -18,7 +18,7 @@ import {
   getCurrentSpan,
   getCurrentTrace,
   seedFromRegistry
-} from "./chunk-XAT6WLCS.js";
+} from "./chunk-SECFVEQV.js";
 import "./chunk-ZUD7OFYB.js";
 import {
   BITFAB_PROGRESS_PREFIX,
@@ -27,7 +27,7 @@ import {
   ReplayError,
   reportReplayProgress,
   serializeReplayResult
-} from "./chunk-A5UGSRMK.js";
+} from "./chunk-45SK3FUL.js";
 import {
   BitfabError,
   DEFAULT_SERVICE_URL,
@@ -35,7 +35,7 @@ import {
   MixedTracingError,
   __version__,
   flushTraces
-} from "./chunk-6DIR65JW.js";
+} from "./chunk-ODF4JSYX.js";
 import "./chunk-H6LZRFMN.js";
 export {
   BITFAB_PROGRESS_PREFIX,

package/dist/node.cjs

@@ -88,7 +88,7 @@ var __version__, __packageName__;
 var init_version_generated = __esm({
   "src/version.generated.ts"() {
     "use strict";
-    __version__ = "0.52.1";
+    __version__ = "0.52.3";
     __packageName__ = "@bitfab/sdk";
   }
 });
@@ -392,15 +392,18 @@ var init_payloadBudget = __esm({
 });
 
 // src/warnOnce.ts
+function warn(message) {
+  try {
+    console.warn(`[bitfab] ${message}`);
+  } catch {
+  }
+}
 function warnOnce(key, message) {
   if (warned.has(key)) {
     return;
   }
   warned.add(key);
-  try {
-    console.warn(`[bitfab] ${message}`);
-  } catch {
-  }
+  warn(message);
 }
 var warned;
 var init_warnOnce = __esm({
@@ -542,6 +545,9 @@ var init_serializePayload = __esm({
 });
 
 // src/traceMetadata.ts
+function findRecord(traceId) {
+  return activeRecords.get(traceId) ?? completedRecords.get(traceId);
+}
 function recordCallerTraceMetadata(traceId, metadata) {
   if (typeof traceId !== "string" || traceId === "") {
     return;
@@ -552,15 +558,52 @@ function recordCallerTraceMetadata(traceId, metadata) {
   if (Object.keys(metadata).length === 0) {
     return;
   }
-  callerMetadata.set(traceId, { ...callerMetadata.get(traceId), ...metadata });
+  const existing = findRecord(traceId);
+  if (existing) {
+    Object.assign(existing.caller, metadata);
+    return;
+  }
+  activeRecords.set(traceId, {
+    caller: { ...metadata },
+    derived: {},
+    warnedKeys: /* @__PURE__ */ new Set()
+  });
 }
 function callerTraceMetadata(traceId) {
-  const recorded = callerMetadata.get(traceId);
-  return recorded ? { ...recorded } : void 0;
+  const record = findRecord(traceId);
+  if (!record || Object.keys(record.caller).length === 0) {
+    return void 0;
+  }
+  return { ...record.caller };
 }
-function forgetTraceMetadata(traceId) {
-  callerMetadata.delete(traceId);
-  derivedMetadata.delete(traceId);
+function retireTraceMetadata(traceId) {
+  const record = activeRecords.get(traceId);
+  if (!record) {
+    return;
+  }
+  activeRecords.delete(traceId);
+  completedRecords.set(traceId, record);
+  while (completedRecords.size > RETAINED_COMPLETED_TRACES) {
+    const oldest = completedRecords.keys().next();
+    if (oldest.done) {
+      return;
+    }
+    completedRecords.delete(oldest.value);
+  }
+}
+function warnAboutShadowedKeys(traceId, record) {
+  const shadowed = Object.keys(record.derived).filter(
+    (key) => key in record.caller && record.caller[key] !== record.derived[key] && !record.warnedKeys.has(key)
+  ).sort();
+  if (shadowed.length === 0) {
+    return;
+  }
+  for (const key of shadowed) {
+    record.warnedKeys.add(key);
+  }
+  warn(
+    `trace metadata key(s) ${shadowed.join(", ")} on trace ${traceId} were set both by the caller and by an integration's own trace export; the caller's value is the one kept on the trace.`
+  );
 }
 function mergeCallerMetadataIntoTracePayload(payload) {
   const externalTrace = payload.externalTrace;
@@ -572,35 +615,31 @@ function mergeCallerMetadataIntoTracePayload(payload) {
   if (typeof traceId !== "string" || traceId === "") {
     return payload;
   }
-  const caller = callerMetadata.get(traceId);
-  if (!caller) {
+  const record = findRecord(traceId);
+  if (!record || Object.keys(record.caller).length === 0) {
     return payload;
   }
-  const derived = derivedMetadata.get(traceId) ?? {};
   const exported = external.metadata;
   if (typeof exported === "object" && exported !== null) {
-    Object.assign(derived, exported);
-  }
-  derivedMetadata.set(traceId, derived);
-  const shadowed = Object.keys(derived).filter((key) => key in caller && caller[key] !== derived[key]).sort();
-  if (shadowed.length > 0) {
-    warnOnce(
-      `trace-metadata-shadowed:${shadowed.join(",")}`,
-      `trace metadata key(s) ${shadowed.join(", ")} were set both by the caller and by an integration's own trace export; the caller's value is the one kept on the trace.`
-    );
+    Object.assign(record.derived, exported);
   }
+  warnAboutShadowedKeys(traceId, record);
   return {
     ...payload,
-    externalTrace: { ...external, metadata: { ...derived, ...caller } }
+    externalTrace: {
+      ...external,
+      metadata: { ...record.derived, ...record.caller }
+    }
   };
 }
-var callerMetadata, derivedMetadata;
+var RETAINED_COMPLETED_TRACES, activeRecords, completedRecords;
 var init_traceMetadata = __esm({
   "src/traceMetadata.ts"() {
     "use strict";
     init_warnOnce();
-    callerMetadata = /* @__PURE__ */ new Map();
-    derivedMetadata = /* @__PURE__ */ new Map();
+    RETAINED_COMPLETED_TRACES = 128;
+    activeRecords = /* @__PURE__ */ new Map();
+    completedRecords = /* @__PURE__ */ new Map();
   }
 });
 
@@ -3000,11 +3039,6 @@ async function replay(httpClient, serviceUrl, traceFunctionKey, fn, options, reg
     }
   }
   const datasetIds = resolveDatasetIds(options);
-  if (options?.traceIds !== void 0 && datasetIds !== void 0) {
-    throw new BitfabError(
-      "traceIds and datasetIds select different replay sources and cannot be used together."
-    );
-  }
   if (options?.limit !== void 0 && options?.traceIds !== void 0) {
     try {
       console.warn(
@@ -7477,7 +7511,7 @@ var Bitfab = class {
                 }
               });
               activeTraceStates.delete(traceId);
-              forgetTraceMetadata(traceId);
+              retireTraceMetadata(traceId);
             }
           } catch {
           }
@@ -7670,7 +7704,7 @@ var Bitfab = class {
       } catch (setupError) {
         if (registeredTraceId) {
           activeTraceStates.delete(registeredTraceId);
-          forgetTraceMetadata(registeredTraceId);
+          retireTraceMetadata(registeredTraceId);
         }
         if (setupError instanceof MixedTracingError) {
           throw setupError;
@@ -8069,7 +8103,7 @@ var Bitfab = class {
       });
     } finally {
       activeTraceStates.delete(traceId);
-      forgetTraceMetadata(traceId);
+      retireTraceMetadata(traceId);
     }
     return traceId;
   }
@@ -8115,7 +8149,7 @@ var Bitfab = class {
         await flushTraces2(3e4);
       } finally {
         unrecorded = activeTraceStates.delete(traceId);
-        forgetTraceMetadata(traceId);
+        retireTraceMetadata(traceId);
       }
     }
     if (unrecorded) {