bitfab 0.52.2 → 0.52.3

package/dist/index.cjs

@@ -51,7 +51,7 @@ var __version__, __packageName__;
 var init_version_generated = __esm({
   "src/version.generated.ts"() {
     "use strict";
-    __version__ = "0.52.2";
+    __version__ = "0.52.3";
     __packageName__ = "bitfab";
   }
 });
@@ -394,15 +394,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({
@@ -544,6 +547,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;
@@ -554,15 +560,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;
@@ -574,35 +617,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();
   }
 });
 
@@ -7467,7 +7506,7 @@ var Bitfab = class {
                 }
               });
               activeTraceStates.delete(traceId);
-              forgetTraceMetadata(traceId);
+              retireTraceMetadata(traceId);
             }
           } catch {
           }
@@ -7660,7 +7699,7 @@ var Bitfab = class {
       } catch (setupError) {
         if (registeredTraceId) {
           activeTraceStates.delete(registeredTraceId);
-          forgetTraceMetadata(registeredTraceId);
+          retireTraceMetadata(registeredTraceId);
         }
         if (setupError instanceof MixedTracingError) {
           throw setupError;
@@ -8059,7 +8098,7 @@ var Bitfab = class {
       });
     } finally {
       activeTraceStates.delete(traceId);
-      forgetTraceMetadata(traceId);
+      retireTraceMetadata(traceId);
     }
     return traceId;
   }
@@ -8105,7 +8144,7 @@ var Bitfab = class {
         await flushTraces2(3e4);
       } finally {
         unrecorded = activeTraceStates.delete(traceId);
-        forgetTraceMetadata(traceId);
+        retireTraceMetadata(traceId);
       }
     }
     if (unrecorded) {