bitfab 0.34.1 → 0.34.2

package/dist/index.d.cts

@@ -288,9 +288,7 @@ interface ReplayContext {
  *
  * Returns `false` when delivery failed or the deadline expired, so a caller
  * that depends on persistence (replay does) can react instead of assuming a
- * drained queue means the server has the data. When delivery goes through a
- * Collector, `true` means the Collector accepted the spans; it is not proof
- * that Bitfab committed them.
+ * drained queue means the server has the data.
  *
  * @param timeoutMs - Maximum total time to wait in milliseconds (default: 5000)
  */
@@ -399,6 +397,15 @@ declare class HttpClient {
         timeout?: number;
         method?: "POST" | "PATCH" | "PUT";
     }): Promise<T>;
+    /**
+     * POST an already-encoded body. The span transport encodes its own batches,
+     * so routing them back through {@link HttpClient.request} would encode the
+     * same data twice.
+     */
+    sendEncoded<T>(endpoint: string, body: string, options?: {
+        timeout?: number;
+        method?: "POST" | "PATCH" | "PUT";
+    }): Promise<T>;
     /**
      * Look up a function by name.
      * Blocks until complete - needed for function execution.
@@ -2329,7 +2336,7 @@ declare class BitfabFunction {
 /**
  * SDK version from package.json (injected at build time)
  */
-declare const __version__ = "0.34.1";
+declare const __version__ = "0.34.2";
 
 /**
  * Constants for the Bitfab SDK.

package/dist/index.d.ts

@@ -288,9 +288,7 @@ interface ReplayContext {
  *
  * Returns `false` when delivery failed or the deadline expired, so a caller
  * that depends on persistence (replay does) can react instead of assuming a
- * drained queue means the server has the data. When delivery goes through a
- * Collector, `true` means the Collector accepted the spans; it is not proof
- * that Bitfab committed them.
+ * drained queue means the server has the data.
  *
  * @param timeoutMs - Maximum total time to wait in milliseconds (default: 5000)
  */
@@ -399,6 +397,15 @@ declare class HttpClient {
         timeout?: number;
         method?: "POST" | "PATCH" | "PUT";
     }): Promise<T>;
+    /**
+     * POST an already-encoded body. The span transport encodes its own batches,
+     * so routing them back through {@link HttpClient.request} would encode the
+     * same data twice.
+     */
+    sendEncoded<T>(endpoint: string, body: string, options?: {
+        timeout?: number;
+        method?: "POST" | "PATCH" | "PUT";
+    }): Promise<T>;
     /**
      * Look up a function by name.
      * Blocks until complete - needed for function execution.
@@ -2329,7 +2336,7 @@ declare class BitfabFunction {
 /**
  * SDK version from package.json (injected at build time)
  */
-declare const __version__ = "0.34.1";
+declare const __version__ = "0.34.2";
 
 /**
  * Constants for the Bitfab SDK.

package/dist/index.js

@@ -20,7 +20,7 @@ import {
   getCurrentReplayBranch,
   getCurrentSpan,
   getCurrentTrace
-} from "./chunk-KO373TDH.js";
+} from "./chunk-KQB42J3S.js";
 import {
   BITFAB_PROGRESS_PREFIX,
   BitfabError,
@@ -29,7 +29,7 @@ import {
   __version__,
   flushTraces,
   reportReplayProgress
-} from "./chunk-5ZMEY5NX.js";
+} from "./chunk-SBQQOFA5.js";
 export {
   BITFAB_PROGRESS_PREFIX,
   Bitfab,

package/dist/node.cjs

@@ -97,7 +97,7 @@ var __version__;
 var init_version_generated = __esm({
   "src/version.generated.ts"() {
     "use strict";
-    __version__ = "0.34.1";
+    __version__ = "0.34.2";
   }
 });
 
@@ -436,32 +436,30 @@ function spanToOtlp(span) {
   }
   return result;
 }
-function buildOtlpRequest(first, spans) {
+function byteLength(value) {
+  return textEncoder ? textEncoder.encode(value).length : value.length;
+}
+function encodeSpan(span) {
+  const json = JSON.stringify(spanToOtlp(span));
+  return { json, size: byteLength(json) };
+}
+function requestEnvelope(first) {
   const scope = first.instrumentationScope;
-  return {
-    resourceSpans: [
-      {
-        resource: {
-          attributes: otlpAttributes(
-            first.resource.attributes
-          )
-        },
-        scopeSpans: [
-          {
-            scope: { name: scope.name, version: scope.version ?? "" },
-            spans
-          }
-        ]
-      }
-    ]
-  };
+  const resource = JSON.stringify({
+    attributes: otlpAttributes(
+      first.resource.attributes
+    )
+  });
+  const scopeJson = JSON.stringify({
+    name: scope.name,
+    version: scope.version ?? ""
+  });
+  const head = `{"resourceSpans":[{"resource":${resource},"scopeSpans":[{"scope":${scopeJson},"spans":[`;
+  const tail = "]}]}]}";
+  return { head, tail, size: byteLength(head) + byteLength(tail) };
 }
-function encodedSize(value) {
-  const json = JSON.stringify(value);
-  if (typeof TextEncoder !== "undefined") {
-    return new TextEncoder().encode(json).length;
-  }
-  return json.length;
+function encodeRequest(envelope, spans) {
+  return envelope.head + spans.map((span) => span.json).join(",") + envelope.tail;
 }
 function delay(ms) {
   return new Promise((resolve) => {
@@ -511,10 +509,6 @@ function isRetryable(error) {
   }
   return RETRYABLE_STATUSES.has(status) || status >= 500;
 }
-function normalizeCollectorEndpoint(endpoint) {
-  const trimmed = endpoint.replace(/\/+$/, "");
-  return trimmed.endsWith("/v1/traces") ? trimmed : `${trimmed}/v1/traces`;
-}
 function endSpan(span, endTime) {
   span.end(endTime);
 }
@@ -551,7 +545,6 @@ function hasError(payload) {
 function createOtelTransport(options) {
   return new OtelBatchTransport({
     ...options,
-    collectorEndpoint: readEnv(COLLECTOR_ENDPOINT_ENV) || void 0,
     exportConcurrency: readBoundedIntEnv(
       EXPORT_CONCURRENCY_ENV,
       MAX_EXPORT_CONCURRENCY,
@@ -586,7 +579,7 @@ function shutdownOtelTransports(timeoutMs = DEFAULT_LIFECYCLE_TIMEOUT_MS) {
     (transport, remaining) => transport.shutdown(remaining)
   );
 }
-var import_api, import_core, import_resources, import_sdk_trace_base, OPERATION_ATTRIBUTE, PAYLOAD_ATTRIBUTE, OTLP_TRACES_ENDPOINT, MAX_EXPORT_REQUEST_BYTES, MAX_REQUEST_BYTES_ENV, EXPORT_CONCURRENCY_ENV, COLLECTOR_ENDPOINT_ENV, MAX_QUEUE_SIZE, DIRECT_MAX_EXPORT_BATCH_SIZE, COLLECTOR_MAX_EXPORT_BATCH_SIZE, DIRECT_MAX_REQUEST_BATCH_SIZE, DEFAULT_EXPORT_CONCURRENCY, MAX_EXPORT_CONCURRENCY, SCHEDULE_DELAY_MILLIS, EXPORT_TIMEOUT_MILLIS, RETRY_DELAY_MILLIS, MAX_SEND_ATTEMPTS, DEFAULT_LIFECYCLE_TIMEOUT_MS, RETRYABLE_STATUSES, liveTransports, traceSubmissionSpanIds, replayTraceSubmissions, submissionCounter, OtlpPayloadTooLargeError, OtlpPartialSuccessError, BitfabSpanExporter, CollectorSpanExporter, DeliveryTrackingExporter, OtelBatchTransport;
+var import_api, import_core, import_resources, import_sdk_trace_base, OPERATION_ATTRIBUTE, PAYLOAD_ATTRIBUTE, OTLP_TRACES_ENDPOINT, MAX_EXPORT_REQUEST_BYTES, MAX_REQUEST_BYTES_ENV, EXPORT_CONCURRENCY_ENV, MAX_QUEUE_SIZE, DIRECT_MAX_EXPORT_BATCH_SIZE, DIRECT_MAX_REQUEST_BATCH_SIZE, DEFAULT_EXPORT_CONCURRENCY, MAX_EXPORT_CONCURRENCY, SCHEDULE_DELAY_MILLIS, EXPORT_TIMEOUT_MILLIS, RETRY_DELAY_MILLIS, MAX_SEND_ATTEMPTS, DEFAULT_LIFECYCLE_TIMEOUT_MS, RETRYABLE_STATUSES, liveTransports, traceSubmissionSpanIds, replayTraceSubmissions, submissionCounter, textEncoder, SPAN_SEPARATOR_BYTES, OtlpPayloadTooLargeError, OtlpPartialSuccessError, BitfabSpanExporter, DeliveryTrackingExporter, OtelBatchTransport;
 var init_otel = __esm({
   "src/otel.ts"() {
     "use strict";
@@ -606,10 +599,8 @@ var init_otel = __esm({
     MAX_EXPORT_REQUEST_BYTES = 3e6;
     MAX_REQUEST_BYTES_ENV = "BITFAB_OTEL_MAX_REQUEST_BYTES";
     EXPORT_CONCURRENCY_ENV = "BITFAB_OTEL_EXPORT_CONCURRENCY";
-    COLLECTOR_ENDPOINT_ENV = "BITFAB_OTEL_EXPORTER_ENDPOINT";
     MAX_QUEUE_SIZE = 8192;
     DIRECT_MAX_EXPORT_BATCH_SIZE = 512;
-    COLLECTOR_MAX_EXPORT_BATCH_SIZE = 32;
     DIRECT_MAX_REQUEST_BATCH_SIZE = 8;
     DEFAULT_EXPORT_CONCURRENCY = 32;
     MAX_EXPORT_CONCURRENCY = 64;
@@ -623,6 +614,8 @@ var init_otel = __esm({
     traceSubmissionSpanIds = /* @__PURE__ */ new Map();
     replayTraceSubmissions = /* @__PURE__ */ new Set();
     submissionCounter = 0;
+    textEncoder = typeof TextEncoder === "undefined" ? void 0 : new TextEncoder();
+    SPAN_SEPARATOR_BYTES = 1;
     OtlpPayloadTooLargeError = class extends Error {
     };
     OtlpPartialSuccessError = class extends Error {
@@ -651,57 +644,55 @@ var init_otel = __esm({
           return true;
         }
         let encoded;
+        let envelope;
         try {
-          encoded = spans.map(spanToOtlp);
+          encoded = spans.map(encodeSpan);
+          envelope = requestEnvelope(spans[0]);
         } catch (error) {
           logError("failed to encode an OpenTelemetry span batch", error);
           return false;
         }
-        const first = spans[0];
-        const batches = this.buildRequestBatches(first, encoded);
+        const batches = this.buildRequestBatches(envelope, encoded);
         const results = await mapWithConcurrency(
           batches,
           this.exportConcurrency,
-          (batch) => this.send(first, batch)
+          (batch) => this.send(envelope, batch)
         );
         return results.every(Boolean);
       }
-      buildRequestBatches(first, spans) {
+      buildRequestBatches(envelope, spans) {
         const batches = [];
         let current = [];
+        let size = envelope.size;
         for (const span of spans) {
-          if (current.length >= this.maxRequestBatchSize) {
-            batches.push(current);
+          const addition = span.size + (current.length > 0 ? SPAN_SEPARATOR_BYTES : 0);
+          if (current.length > 0 && (current.length >= this.maxRequestBatchSize || size + addition > this.maxRequestBytes)) {
+            batches.push({ spans: current, size });
             current = [];
+            size = envelope.size;
           }
-          const candidate = [...current, span];
-          if (current.length > 0 && encodedSize(buildOtlpRequest(first, candidate)) > this.maxRequestBytes) {
-            batches.push(current);
-            current = [span];
-          } else {
-            current = candidate;
-          }
+          current.push(span);
+          size += span.size + (current.length > 1 ? SPAN_SEPARATOR_BYTES : 0);
         }
         if (current.length > 0) {
-          batches.push(current);
+          batches.push({ spans: current, size });
         }
         return batches;
       }
-      async send(first, spans) {
-        const payload = buildOtlpRequest(first, spans);
-        if (encodedSize(payload) > this.maxRequestBytes) {
+      async send(envelope, batch) {
+        if (batch.size > this.maxRequestBytes) {
           logError(
             "a single OpenTelemetry span exceeded the configured request-size target and could not be exported"
           );
           return false;
         }
         try {
-          await this.sendWithRetries(payload);
+          await this.sendWithRetries(encodeRequest(envelope, batch.spans));
           return true;
         } catch (error) {
           if (error instanceof OtlpPayloadTooLargeError) {
             logError(
-              spans.length === 1 ? "a single OpenTelemetry span exceeded the ingestion request limit and could not be exported" : "an OpenTelemetry span batch exceeded the ingestion request limit and could not be exported"
+              batch.spans.length === 1 ? "a single OpenTelemetry span exceeded the ingestion request limit and could not be exported" : "an OpenTelemetry span batch exceeded the ingestion request limit and could not be exported"
             );
             return false;
           }
@@ -725,12 +716,12 @@ var init_otel = __esm({
        * the server does not yet understand. The fix is a client-supplied
        * idempotency key that ingestion dedupes on.
        */
-      async sendWithRetries(payload) {
+      async sendWithRetries(body) {
         for (let attempt = 0; attempt < MAX_SEND_ATTEMPTS; attempt += 1) {
           try {
             const response = await this.directSender(
               OTLP_TRACES_ENDPOINT,
-              payload,
+              body,
               EXPORT_TIMEOUT_MILLIS
             );
             const partialSuccess = asRecord(response?.partialSuccess);
@@ -761,115 +752,6 @@ var init_otel = __esm({
       async forceFlush() {
       }
     };
-    CollectorSpanExporter = class {
-      constructor(endpoint, apiKey, maxRequestBytes) {
-        this.endpoint = endpoint;
-        this.apiKey = apiKey;
-        this.maxRequestBytes = maxRequestBytes;
-      }
-      /**
-       * Loaded through a dynamic import rather than a top-level one so bundlers
-       * code-split it: Collector delivery is opt-in, and a consumer who never sets
-       * an endpoint should not pay for the exporter in their initial bundle. It is
-       * a hard dependency, so this cannot fail for want of the package.
-       */
-      loadExporterModule() {
-        if (!this.pendingModule) {
-          this.pendingModule = import("@opentelemetry/exporter-trace-otlp-proto");
-        }
-        return this.pendingModule;
-      }
-      export(spans, resultCallback) {
-        void this.exportAsync(spans).then(
-          (succeeded) => {
-            resultCallback({
-              code: succeeded ? import_core.ExportResultCode.SUCCESS : import_core.ExportResultCode.FAILED
-            });
-          },
-          (error) => {
-            resultCallback({ code: import_core.ExportResultCode.FAILED, error });
-          }
-        );
-      }
-      async exportAsync(spans) {
-        if (spans.length === 0) {
-          return true;
-        }
-        let delegate;
-        try {
-          delegate = await this.resolveDelegate();
-        } catch (error) {
-          logError("failed to build the OTLP Collector exporter", error);
-          return false;
-        }
-        const results = await Promise.all(
-          this.partition(spans).map(
-            (batch) => new Promise((resolve) => {
-              try {
-                delegate.export(batch, (result) => {
-                  resolve(result.code === import_core.ExportResultCode.SUCCESS);
-                });
-              } catch (error) {
-                logError("Collector export threw", error);
-                resolve(false);
-              }
-            })
-          )
-        );
-        return results.every(Boolean);
-      }
-      /**
-       * Partition by the encoded JSON size of each carrier rather than its encoded
-       * protobuf size. Protobuf is strictly smaller than the equivalent JSON for
-       * these payloads, so the JSON figure is a conservative bound that keeps every
-       * request under the target without pulling `@opentelemetry/otlp-transformer`
-       * into the dependency set purely to measure bytes.
-       */
-      partition(spans) {
-        const batches = [];
-        let current = [];
-        let currentSize = 0;
-        for (const span of spans) {
-          const size = encodedSize(spanToOtlp(span));
-          if (current.length > 0 && currentSize + size > this.maxRequestBytes) {
-            batches.push(current);
-            current = [];
-            currentSize = 0;
-          }
-          current.push(span);
-          currentSize += size;
-        }
-        if (current.length > 0) {
-          batches.push(current);
-        }
-        return batches;
-      }
-      async resolveDelegate() {
-        const apiKey = this.apiKey() ?? "";
-        if (this.delegate && this.delegateApiKey === apiKey) {
-          return this.delegate;
-        }
-        const { OTLPTraceExporter } = await this.loadExporterModule();
-        const previous = this.delegate;
-        this.delegate = new OTLPTraceExporter({
-          url: this.endpoint,
-          headers: { Authorization: `Bearer ${apiKey}` },
-          timeoutMillis: EXPORT_TIMEOUT_MILLIS
-        });
-        this.delegateApiKey = apiKey;
-        if (previous) {
-          void previous.shutdown().catch(() => {
-          });
-        }
-        return this.delegate;
-      }
-      async shutdown() {
-        await this.delegate?.shutdown();
-      }
-      async forceFlush() {
-        await this.delegate?.forceFlush?.();
-      }
-    };
     DeliveryTrackingExporter = class {
       constructor(exporter) {
         this.exporter = exporter;
@@ -912,27 +794,22 @@ var init_otel = __esm({
     OtelBatchTransport = class {
       constructor(options) {
         this.closed = false;
-        const collectorEndpoint = options.collectorEndpoint;
         const maxRequestBytes = options.maxRequestBytes ?? MAX_EXPORT_REQUEST_BYTES;
         const maxRequestBatchSize = options.maxRequestBatchSize ?? DIRECT_MAX_REQUEST_BATCH_SIZE;
         if (maxRequestBatchSize <= 0) {
           throw new BitfabError("maxRequestBatchSize must be a positive integer");
         }
         this.deliveryTracker = new DeliveryTrackingExporter(
-          collectorEndpoint === void 0 ? new BitfabSpanExporter(
+          new BitfabSpanExporter(
             options.directSender,
             maxRequestBytes,
             maxRequestBatchSize,
             options.exportConcurrency ?? DEFAULT_EXPORT_CONCURRENCY
-          ) : new CollectorSpanExporter(
-            normalizeCollectorEndpoint(collectorEndpoint),
-            options.apiKey,
-            maxRequestBytes
           )
         );
         this.processor = new import_sdk_trace_base.BatchSpanProcessor(this.deliveryTracker, {
           maxQueueSize: options.maxQueueSize ?? MAX_QUEUE_SIZE,
-          maxExportBatchSize: options.maxExportBatchSize ?? (collectorEndpoint === void 0 ? DIRECT_MAX_EXPORT_BATCH_SIZE : COLLECTOR_MAX_EXPORT_BATCH_SIZE),
+          maxExportBatchSize: options.maxExportBatchSize ?? DIRECT_MAX_EXPORT_BATCH_SIZE,
           scheduledDelayMillis: SCHEDULE_DELAY_MILLIS,
           exportTimeoutMillis: options.exportTimeoutMillis ?? EXPORT_TIMEOUT_MILLIS
         });
@@ -1149,8 +1026,7 @@ var init_http = __esm({
         }
         if (!this.traceTransport) {
           this.traceTransport = createTraceTransport({
-            apiKey: () => this.resolveApiKey(),
-            directSender: (endpoint, payload, timeoutMs) => this.request(endpoint, payload, {
+            directSender: (endpoint, body, timeoutMs) => this.sendEncoded(endpoint, body, {
               timeout: timeoutMs
             })
           });
@@ -1220,11 +1096,6 @@ var init_http = __esm({
        * @throws {BitfabError} If the request fails
        */
       async request(endpoint, payload, options) {
-        const url = `${this.serviceUrl}${endpoint}`;
-        const timeout = options?.timeout ?? this.timeout;
-        const method = options?.method ?? "POST";
-        const controller = new AbortController();
-        const timeoutId = setTimeout(() => controller.abort(), timeout);
         const { body, dropped } = serializePayloadBody(payload);
         if (dropped.length > 0) {
           try {
@@ -1234,6 +1105,19 @@ var init_http = __esm({
           } catch {
           }
         }
+        return this.sendEncoded(endpoint, body, options);
+      }
+      /**
+       * POST an already-encoded body. The span transport encodes its own batches,
+       * so routing them back through {@link HttpClient.request} would encode the
+       * same data twice.
+       */
+      async sendEncoded(endpoint, body, options) {
+        const url = `${this.serviceUrl}${endpoint}`;
+        const timeout = options?.timeout ?? this.timeout;
+        const method = options?.method ?? "POST";
+        const controller = new AbortController();
+        const timeoutId = setTimeout(() => controller.abort(), timeout);
         try {
           const response = await fetch(url, {
             method,