@davidilie/telemetry-node 0.1.3 → 0.3.0

package/README.md

@@ -47,8 +47,10 @@ process.once("SIGTERM", async () => {
 ```
 
 `registerNodeTelemetry` installs one `NodeSDK` per JavaScript global. Repeated
-calls return the first client and ignore later configuration until that client
-successfully shuts down. Do not register this package beside another provider.
+calls return the first client and ignore later configuration. Registration is
+terminal after shutdown: restart the process to register again. OTel retains
+global provider references after shutdown, so another registration would
+otherwise silently use a stopped provider. Do not register this package beside another provider.
 
 The optional `sdk` property accepts `NodeSDKConfiguration` fields other than
 `resource` and `serviceName`. Use it for maintained instrumentations, readers,
@@ -165,10 +167,17 @@ collected into the same backend. Disable one route deliberately.
 
 ## Spans and context
 
-The `NodeTelemetryClient` overrides the generic core `withSpan` so the new span
-is active during the operation. Use it when events/logs need the same trace.
+The Node adapter supplies active-span execution to the shared core `withSpan`.
+The span is active during the operation, including asynchronous work, while
+core applies context, consent, sampling, and shutdown checks. Use it when events/logs need the same trace.
 `client.startSpan()` creates a span but does not make it active.
 
+The callback receives a safe span handle, including `traceContext()`.
+`client.traceWebhook({ destination: "discord.sales" }, send)` tracks one
+outgoing POST under the active parent. See the
+[webhook guide](https://github.com/DavidIlie/davidapps-telemetry/blob/main/docs/webhooks.md)
+for privacy, delivery outcomes, metrics, and Grafana Discord notifications.
+
 Both methods accept `{ kind: "internal" | "server" | "client" | "producer" |
 "consumer" }` after attributes. Manual span handles also expose
 `setStatus("unset" | "ok" | "error", message?)`.
@@ -211,9 +220,11 @@ context; `startSpan` does not.
   apply; set one layer to `1` if multiplied rates are not intentional.
 - State changes affect new work and do not erase an already exported or queued
   provider batch.
-- `flush()` drains core work, but the Node adapter does not own a provider
-  force-flush method. `shutdown()` on the `/register` client shuts down the
-  `NodeSDK` and its processors.
+- `flush()` drains core work, then calls the registered trace, log, and metric
+  providers' public `forceFlush()` methods when available. This flushes all
+  pending work on those providers, not only this client's records. Failures
+  are reported through `onError` without rejecting the client promise.
+  `shutdown()` on the `/register` client shuts down the `NodeSDK` and its processors.
 - Only a client returned by `/register` owns and shuts down the installed
   `NodeSDK`. A client created from the root entrypoint does not own the
   framework provider.

package/THIRD_PARTY_NOTICES.md

@@ -7,5 +7,6 @@ DavidApps Telemetry depends on permissively licensed open-source projects and us
 - Vercel Analytics and Vercel OTEL — MIT License — Copyright Vercel, Inc.
 - PostHog JavaScript SDKs — Apache-2.0/MIT — API reference only; no source is vendored.
 - Honeycomb OpenTelemetry React Native — MIT License — Copyright Honeycomb.io.
+- Stripe Node.js SDK — MIT License — Copyright Ask Bjørn Hansen and Stripe, Inc.; API reference only.
 
-Reference license texts are retained in `licenses/` so future adaptations cannot accidentally lose their attribution obligations. The current source tree does not vendor code from PostHog, Vercel, Honeycomb, or the OpenTelemetry demo.
+Reference license texts are retained in `licenses/` so future adaptations cannot accidentally lose their attribution obligations. The current source tree does not vendor code from PostHog, Vercel, Honeycomb, Stripe, or the OpenTelemetry demo.

package/UPSTREAM.yml

@@ -54,3 +54,8 @@ sources:
     license: MIT
     commit: 07d144459177a58a347e18be762cf8f04e63cb1d
     use: api-reference
+  - name: stripe-node
+    repository: https://github.com/stripe/stripe-node
+    license: MIT
+    commit: 8adba1d35fdb76ba4104e5d7ae1ac999649c7061
+    use: api-reference

package/dist/{client-BzoFCwmQ.d.cts → client-DYrSJX10.d.cts}

@@ -9,6 +9,7 @@ declare class OpenTelemetrySpan implements TelemetrySpan {
   recordException(error: unknown, attributes?: Attributes$1): void;
   setStatus(status: TelemetrySpanStatus, message?: string): this;
   end(): void;
+  traceContext(): TraceContext | undefined;
 }
 interface OpenTelemetryAdapterConfig {
   resource: TelemetryResource;
@@ -31,7 +32,8 @@ declare class OpenTelemetryAdapter implements TelemetryAdapter {
   send(signal: TelemetrySignal): void;
   startSpan(name: string, attributes?: Attributes$1, options?: TelemetrySpanOptions): TelemetrySpan;
   currentTraceContext(): TraceContext | undefined;
-  withActiveSpan<T>(name: string, operation: () => T | Promise<T>, attributes?: Attributes$1, options?: TelemetrySpanOptions): Promise<T>;
+  withActiveSpan<T>(name: string, operation: (span: TelemetrySpan) => T | Promise<T>, attributes?: Attributes$1, options?: TelemetrySpanOptions): Promise<T>;
+  flush(): Promise<void>;
   shutdown(): Promise<void>;
 }
 declare function createOpenTelemetryAdapter(config: OpenTelemetryAdapterConfig): OpenTelemetryAdapter;
@@ -78,7 +80,6 @@ declare class NodeTelemetryClient extends TelemetryClient {
   constructor(config: NodeTelemetryInternalConfig);
   setEnabled(enabled: boolean): void;
   setConsent(consent: "granted" | "denied" | "pending"): void;
-  withSpan<T>(name: string, operation: () => T | Promise<T>, attributes?: Attributes$1, options?: TelemetrySpanOptions): Promise<T>;
 }
 declare function createNodeTelemetry(config: NodeTelemetryConfig): NodeTelemetryClient;
 //#endregion

package/dist/{client-BzoFCwmQ.d.mts → client-DYrSJX10.d.mts}

@@ -9,6 +9,7 @@ declare class OpenTelemetrySpan implements TelemetrySpan {
   recordException(error: unknown, attributes?: Attributes$1): void;
   setStatus(status: TelemetrySpanStatus, message?: string): this;
   end(): void;
+  traceContext(): TraceContext | undefined;
 }
 interface OpenTelemetryAdapterConfig {
   resource: TelemetryResource;
@@ -31,7 +32,8 @@ declare class OpenTelemetryAdapter implements TelemetryAdapter {
   send(signal: TelemetrySignal): void;
   startSpan(name: string, attributes?: Attributes$1, options?: TelemetrySpanOptions): TelemetrySpan;
   currentTraceContext(): TraceContext | undefined;
-  withActiveSpan<T>(name: string, operation: () => T | Promise<T>, attributes?: Attributes$1, options?: TelemetrySpanOptions): Promise<T>;
+  withActiveSpan<T>(name: string, operation: (span: TelemetrySpan) => T | Promise<T>, attributes?: Attributes$1, options?: TelemetrySpanOptions): Promise<T>;
+  flush(): Promise<void>;
   shutdown(): Promise<void>;
 }
 declare function createOpenTelemetryAdapter(config: OpenTelemetryAdapterConfig): OpenTelemetryAdapter;
@@ -78,7 +80,6 @@ declare class NodeTelemetryClient extends TelemetryClient {
   constructor(config: NodeTelemetryInternalConfig);
   setEnabled(enabled: boolean): void;
   setConsent(consent: "granted" | "denied" | "pending"): void;
-  withSpan<T>(name: string, operation: () => T | Promise<T>, attributes?: Attributes$1, options?: TelemetrySpanOptions): Promise<T>;
 }
 declare function createNodeTelemetry(config: NodeTelemetryConfig): NodeTelemetryClient;
 //#endregion

package/dist/index.cjs

@@ -1,5 +1,5 @@
 Object.defineProperty(exports, Symbol.toStringTag, { value: "Module" });
-const require_sampler = require("./sampler-CGBWEZod.cjs");
+const require_sampler = require("./sampler-D5mi13Eh.cjs");
 exports.DynamicTelemetrySampler = require_sampler.DynamicTelemetrySampler;
 exports.NodeTelemetryClient = require_sampler.NodeTelemetryClient;
 exports.OpenTelemetryAdapter = require_sampler.OpenTelemetryAdapter;

package/dist/index.d.cts

@@ -1,4 +1,4 @@
-import { a as OpenTelemetryAdapter, c as createOpenTelemetryAdapter, d as startSpan, f as withSpan, i as MeasurementMode, l as currentTraceContext, n as NodeTelemetryConfig, o as OpenTelemetryAdapterConfig, r as createNodeTelemetry, s as OpenTelemetrySpan, t as NodeTelemetryClient, u as recordException } from "./client-BzoFCwmQ.cjs";
+import { a as OpenTelemetryAdapter, c as createOpenTelemetryAdapter, d as startSpan, f as withSpan, i as MeasurementMode, l as currentTraceContext, n as NodeTelemetryConfig, o as OpenTelemetryAdapterConfig, r as createNodeTelemetry, s as OpenTelemetrySpan, t as NodeTelemetryClient, u as recordException } from "./client-DYrSJX10.cjs";
 import { AttributeValue, Attributes, Context, Link, SpanKind } from "@opentelemetry/api";
 import { AttributeValue as AttributeValue$1, Attributes as Attributes$1, TelemetryResource } from "@davidilie/telemetry-core";
 import { Sampler, SamplingResult } from "@opentelemetry/sdk-trace-base";

package/dist/index.d.mts

@@ -1,4 +1,4 @@
-import { a as OpenTelemetryAdapter, c as createOpenTelemetryAdapter, d as startSpan, f as withSpan, i as MeasurementMode, l as currentTraceContext, n as NodeTelemetryConfig, o as OpenTelemetryAdapterConfig, r as createNodeTelemetry, s as OpenTelemetrySpan, t as NodeTelemetryClient, u as recordException } from "./client-BzoFCwmQ.mjs";
+import { a as OpenTelemetryAdapter, c as createOpenTelemetryAdapter, d as startSpan, f as withSpan, i as MeasurementMode, l as currentTraceContext, n as NodeTelemetryConfig, o as OpenTelemetryAdapterConfig, r as createNodeTelemetry, s as OpenTelemetrySpan, t as NodeTelemetryClient, u as recordException } from "./client-DYrSJX10.mjs";
 import { AttributeValue, Attributes, Context, Link, SpanKind } from "@opentelemetry/api";
 import { AttributeValue as AttributeValue$1, Attributes as Attributes$1, TelemetryResource } from "@davidilie/telemetry-core";
 import { Sampler, SamplingResult } from "@opentelemetry/sdk-trace-base";

package/dist/index.mjs

@@ -1,2 +1,2 @@
-import { a as OpenTelemetrySpan, c as recordException, d as telemetryResourceAttributes, f as toOtelAttributeValue, i as OpenTelemetryAdapter, l as startSpan, n as NodeTelemetryClient, o as createOpenTelemetryAdapter, p as toOtelAttributes, r as createNodeTelemetry, s as currentTraceContext, t as DynamicTelemetrySampler, u as withSpan } from "./sampler-ND4YGh_7.mjs";
+import { a as OpenTelemetrySpan, c as recordException, d as telemetryResourceAttributes, f as toOtelAttributeValue, i as OpenTelemetryAdapter, l as startSpan, n as NodeTelemetryClient, o as createOpenTelemetryAdapter, p as toOtelAttributes, r as createNodeTelemetry, s as currentTraceContext, t as DynamicTelemetrySampler, u as withSpan } from "./sampler-B1PsF--M.mjs";
 export { DynamicTelemetrySampler, NodeTelemetryClient, OpenTelemetryAdapter, OpenTelemetrySpan, createNodeTelemetry, createOpenTelemetryAdapter, currentTraceContext, recordException, startSpan, telemetryResourceAttributes, toOtelAttributeValue, toOtelAttributes, withSpan };

package/dist/register.cjs

@@ -1,5 +1,5 @@
 Object.defineProperty(exports, Symbol.toStringTag, { value: "Module" });
-const require_sampler = require("./sampler-CGBWEZod.cjs");
+const require_sampler = require("./sampler-D5mi13Eh.cjs");
 let _davidilie_telemetry_core = require("@davidilie/telemetry-core");
 let _opentelemetry_sdk_trace_base = require("@opentelemetry/sdk-trace-base");
 let _opentelemetry_resources = require("@opentelemetry/resources");
@@ -13,7 +13,10 @@ function traceSampleRate(config) {
 function registerNodeTelemetry(config) {
 	const sharedGlobal = globalThis;
 	const existing = sharedGlobal[REGISTRATION];
-	if (existing) return existing.client;
+	if (existing) {
+		if (existing.stopped) throw new Error("Node telemetry registration is terminal after shutdown; restart the process to register again");
+		return existing.client;
+	}
 	const resource = (0, _davidilie_telemetry_core.sanitizeResource)(config.resource);
 	const sampler = new require_sampler.DynamicTelemetrySampler(config.sdk?.sampler ?? new _opentelemetry_sdk_trace_base.ParentBasedSampler({ root: new _opentelemetry_sdk_trace_base.TraceIdRatioBasedSampler(traceSampleRate(config)) }), {
 		enabled: config.enabled ?? true,
@@ -37,13 +40,15 @@ function registerNodeTelemetry(config) {
 		},
 		measurementMode: config.measurementMode ?? (config.sdk?.metricReaders?.length ? "metrics" : "spans"),
 		shutdown: async () => {
+			const registration = sharedGlobal[REGISTRATION];
+			if (registration?.client === client) registration.stopped = true;
 			await sdk.shutdown();
-			if (sharedGlobal[REGISTRATION]?.client === client) delete sharedGlobal[REGISTRATION];
 		}
 	});
 	sharedGlobal[REGISTRATION] = {
 		client,
-		sdk
+		sdk,
+		stopped: false
 	};
 	return client;
 }

package/dist/register.d.cts

@@ -1,4 +1,4 @@
-import { n as NodeTelemetryConfig, t as NodeTelemetryClient } from "./client-BzoFCwmQ.cjs";
+import { n as NodeTelemetryConfig, t as NodeTelemetryClient } from "./client-DYrSJX10.cjs";
 import { NodeSDKConfiguration, NodeSDKConfiguration as NodeSDKConfiguration$1 } from "@opentelemetry/sdk-node";
 //#region src/register.d.ts
 interface RegisterNodeTelemetryConfig extends NodeTelemetryConfig {

package/dist/register.d.mts

@@ -1,4 +1,4 @@
-import { n as NodeTelemetryConfig, t as NodeTelemetryClient } from "./client-BzoFCwmQ.mjs";
+import { n as NodeTelemetryConfig, t as NodeTelemetryClient } from "./client-DYrSJX10.mjs";
 import { NodeSDKConfiguration, NodeSDKConfiguration as NodeSDKConfiguration$1 } from "@opentelemetry/sdk-node";
 //#region src/register.d.ts
 interface RegisterNodeTelemetryConfig extends NodeTelemetryConfig {

package/dist/register.mjs

@@ -1,4 +1,4 @@
-import { d as telemetryResourceAttributes, n as NodeTelemetryClient, t as DynamicTelemetrySampler } from "./sampler-ND4YGh_7.mjs";
+import { d as telemetryResourceAttributes, n as NodeTelemetryClient, t as DynamicTelemetrySampler } from "./sampler-B1PsF--M.mjs";
 import { sanitizeResource } from "@davidilie/telemetry-core";
 import { ParentBasedSampler, TraceIdRatioBasedSampler } from "@opentelemetry/sdk-trace-base";
 import { resourceFromAttributes } from "@opentelemetry/resources";
@@ -12,7 +12,10 @@ function traceSampleRate(config) {
 function registerNodeTelemetry(config) {
 	const sharedGlobal = globalThis;
 	const existing = sharedGlobal[REGISTRATION];
-	if (existing) return existing.client;
+	if (existing) {
+		if (existing.stopped) throw new Error("Node telemetry registration is terminal after shutdown; restart the process to register again");
+		return existing.client;
+	}
 	const resource = sanitizeResource(config.resource);
 	const sampler = new DynamicTelemetrySampler(config.sdk?.sampler ?? new ParentBasedSampler({ root: new TraceIdRatioBasedSampler(traceSampleRate(config)) }), {
 		enabled: config.enabled ?? true,
@@ -36,13 +39,15 @@ function registerNodeTelemetry(config) {
 		},
 		measurementMode: config.measurementMode ?? (config.sdk?.metricReaders?.length ? "metrics" : "spans"),
 		shutdown: async () => {
+			const registration = sharedGlobal[REGISTRATION];
+			if (registration?.client === client) registration.stopped = true;
 			await sdk.shutdown();
-			if (sharedGlobal[REGISTRATION]?.client === client) delete sharedGlobal[REGISTRATION];
 		}
 	});
 	sharedGlobal[REGISTRATION] = {
 		client,
-		sdk
+		sdk,
+		stopped: false
 	};
 	return client;
 }

package/dist/{sampler-ND4YGh_7.mjs → sampler-B1PsF--M.mjs}

@@ -1,4 +1,4 @@
-import { SpanKind, SpanStatusCode, context, metrics, trace } from "@opentelemetry/api";
+import { SpanKind, SpanStatusCode, context, isSpanContextValid, metrics, trace } from "@opentelemetry/api";
 import { SeverityNumber, logs } from "@opentelemetry/api-logs";
 import { TelemetryClient, redactText, sanitizeAttributes, sanitizeResource, sanitizeSignal } from "@davidilie/telemetry-core";
 import { SamplingDecision } from "@opentelemetry/sdk-trace-base";
@@ -38,6 +38,22 @@ function telemetryResourceAttributes(resource) {
 	};
 }
 //#endregion
+//#region src/flush.ts
+/** Flush registered SDK providers using their public lifecycle methods. */
+async function flushOpenTelemetryProviders() {
+	let tracerProvider = trace.getTracerProvider();
+	if ("getDelegate" in tracerProvider && typeof tracerProvider.getDelegate === "function") tracerProvider = tracerProvider.getDelegate();
+	const providers = [
+		tracerProvider,
+		logs.getLoggerProvider(),
+		metrics.getMeterProvider()
+	];
+	const errors = (await Promise.allSettled(providers.map(async (provider) => {
+		if ("forceFlush" in provider && typeof provider.forceFlush === "function") await provider.forceFlush();
+	}))).flatMap((result) => result.status === "rejected" ? [result.reason] : []);
+	if (errors.length > 0) throw new AggregateError(errors, "OpenTelemetry provider flush failed");
+}
+//#endregion
 //#region src/adapter.ts
 const severityNumbers = {
 	debug: SeverityNumber.DEBUG,
@@ -100,6 +116,14 @@ var OpenTelemetrySpan = class {
 	end() {
 		this.#span.end();
 	}
+	traceContext() {
+		const value = this.#span.spanContext();
+		return isSpanContextValid(value) ? {
+			traceId: value.traceId,
+			spanId: value.spanId,
+			traceFlags: value.traceFlags
+		} : void 0;
+	}
 };
 var OpenTelemetryAdapter = class {
 	#instrumentationName;
@@ -129,7 +153,7 @@ var OpenTelemetryAdapter = class {
 		switch (signal.type) {
 			case "event": {
 				const activeSpan = trace.getActiveSpan();
-				if (activeSpan) {
+				if (activeSpan?.isRecording()) {
 					activeSpan.addEvent(signal.name, attributes, new Date(signal.timestamp));
 					return;
 				}
@@ -144,22 +168,23 @@ var OpenTelemetryAdapter = class {
 			}
 			case "exception": {
 				const activeSpan = trace.getActiveSpan();
-				const span = activeSpan ?? this.#tracer().startSpan("exception", {
+				const recordingSpan = activeSpan?.isRecording() ? activeSpan : void 0;
+				const span = recordingSpan ?? this.#tracer().startSpan("exception", {
 					kind: SpanKind.INTERNAL,
 					attributes,
 					startTime: new Date(signal.timestamp)
 				});
-				span.setAttributes(attributes);
-				span.recordException({
-					name: signal.exception.name,
-					message: signal.exception.message,
-					...signal.exception.stack ? { stack: signal.exception.stack } : {}
-				});
+				span.addEvent("exception", {
+					...attributes,
+					"exception.type": signal.exception.name,
+					"exception.message": signal.exception.message,
+					...signal.exception.stack ? { "exception.stacktrace": signal.exception.stack } : {}
+				}, new Date(signal.timestamp));
 				span.setStatus({
 					code: SpanStatusCode.ERROR,
 					message: signal.exception.message
 				});
-				if (!activeSpan) span.end(new Date(signal.timestamp));
+				if (!recordingSpan) span.end(new Date(signal.timestamp));
 				return;
 			}
 			case "log":
@@ -239,7 +264,7 @@ var OpenTelemetryAdapter = class {
 			}, async (span) => {
 				operationRan = true;
 				try {
-					operationResult = await operation();
+					operationResult = await operation(new OpenTelemetrySpan(span));
 					operationSucceeded = true;
 				} catch (error) {
 					operationError = error;
@@ -261,7 +286,10 @@ var OpenTelemetryAdapter = class {
 		}
 		if (operationSucceeded) return operationResult;
 		if (operationRan) throw operationError;
-		return operation();
+		throw new Error("OpenTelemetry tracer did not invoke the span operation");
+	}
+	flush() {
+		return flushOpenTelemetryProviders();
 	}
 	async shutdown() {
 		await this.#shutdown?.();
@@ -275,7 +303,7 @@ function createOpenTelemetryAdapter(config) {
 }
 function currentTraceContext() {
 	const spanContext = trace.getSpan(context.active())?.spanContext();
-	if (!spanContext) return void 0;
+	if (!spanContext || !isSpanContextValid(spanContext)) return void 0;
 	return {
 		traceId: spanContext.traceId,
 		spanId: spanContext.spanId,
@@ -325,9 +353,6 @@ function recordException(error, attributes = {}) {
 //#endregion
 //#region src/client.ts
 var NodeTelemetryClient = class extends TelemetryClient {
-	#adapter;
-	#sampleRate;
-	#onError;
 	#onCollectionStateChange;
 	#enabled;
 	#consent;
@@ -353,9 +378,6 @@ var NodeTelemetryClient = class extends TelemetryClient {
 			...config.debug !== void 0 ? { debug: config.debug } : {},
 			...config.onError ? { onError: config.onError } : {}
 		});
-		this.#adapter = adapter;
-		this.#sampleRate = config.providerManagedSampling ? 1 : Number.isFinite(config.sampleRate) ? Math.min(1, Math.max(0, config.sampleRate ?? 1)) : 1;
-		this.#onError = config.onError;
 		this.#onCollectionStateChange = config.onCollectionStateChange;
 		this.#enabled = config.enabled ?? true;
 		this.#consent = config.consent ?? "granted";
@@ -376,26 +398,6 @@ var NodeTelemetryClient = class extends TelemetryClient {
 			consent: this.#consent
 		});
 	}
-	async withSpan(name, operation, attributes = {}, options = {}) {
-		if (!this.#enabled || this.#consent !== "granted" || Math.random() >= this.#sampleRate) return operation();
-		let operationStarted = false;
-		try {
-			return await this.#adapter.withActiveSpan(redactText(name, 256), () => {
-				operationStarted = true;
-				return operation();
-			}, sanitizeAttributes(attributes), options);
-		} catch (error) {
-			if (operationStarted) throw error;
-			this.#reportAdapterError(error, { operation: "startSpan" });
-			return operation();
-		}
-	}
-	#reportAdapterError(error, context) {
-		if (!this.#onError) return;
-		try {
-			Promise.resolve(this.#onError(error, context)).catch(() => void 0);
-		} catch {}
-	}
 };
 function createNodeTelemetry(config) {
 	return new NodeTelemetryClient(config);

package/dist/{sampler-CGBWEZod.cjs → sampler-D5mi13Eh.cjs}

@@ -38,6 +38,22 @@ function telemetryResourceAttributes(resource) {
 	};
 }
 //#endregion
+//#region src/flush.ts
+/** Flush registered SDK providers using their public lifecycle methods. */
+async function flushOpenTelemetryProviders() {
+	let tracerProvider = _opentelemetry_api.trace.getTracerProvider();
+	if ("getDelegate" in tracerProvider && typeof tracerProvider.getDelegate === "function") tracerProvider = tracerProvider.getDelegate();
+	const providers = [
+		tracerProvider,
+		_opentelemetry_api_logs.logs.getLoggerProvider(),
+		_opentelemetry_api.metrics.getMeterProvider()
+	];
+	const errors = (await Promise.allSettled(providers.map(async (provider) => {
+		if ("forceFlush" in provider && typeof provider.forceFlush === "function") await provider.forceFlush();
+	}))).flatMap((result) => result.status === "rejected" ? [result.reason] : []);
+	if (errors.length > 0) throw new AggregateError(errors, "OpenTelemetry provider flush failed");
+}
+//#endregion
 //#region src/adapter.ts
 const severityNumbers = {
 	debug: _opentelemetry_api_logs.SeverityNumber.DEBUG,
@@ -100,6 +116,14 @@ var OpenTelemetrySpan = class {
 	end() {
 		this.#span.end();
 	}
+	traceContext() {
+		const value = this.#span.spanContext();
+		return (0, _opentelemetry_api.isSpanContextValid)(value) ? {
+			traceId: value.traceId,
+			spanId: value.spanId,
+			traceFlags: value.traceFlags
+		} : void 0;
+	}
 };
 var OpenTelemetryAdapter = class {
 	#instrumentationName;
@@ -129,7 +153,7 @@ var OpenTelemetryAdapter = class {
 		switch (signal.type) {
 			case "event": {
 				const activeSpan = _opentelemetry_api.trace.getActiveSpan();
-				if (activeSpan) {
+				if (activeSpan?.isRecording()) {
 					activeSpan.addEvent(signal.name, attributes, new Date(signal.timestamp));
 					return;
 				}
@@ -144,22 +168,23 @@ var OpenTelemetryAdapter = class {
 			}
 			case "exception": {
 				const activeSpan = _opentelemetry_api.trace.getActiveSpan();
-				const span = activeSpan ?? this.#tracer().startSpan("exception", {
+				const recordingSpan = activeSpan?.isRecording() ? activeSpan : void 0;
+				const span = recordingSpan ?? this.#tracer().startSpan("exception", {
 					kind: _opentelemetry_api.SpanKind.INTERNAL,
 					attributes,
 					startTime: new Date(signal.timestamp)
 				});
-				span.setAttributes(attributes);
-				span.recordException({
-					name: signal.exception.name,
-					message: signal.exception.message,
-					...signal.exception.stack ? { stack: signal.exception.stack } : {}
-				});
+				span.addEvent("exception", {
+					...attributes,
+					"exception.type": signal.exception.name,
+					"exception.message": signal.exception.message,
+					...signal.exception.stack ? { "exception.stacktrace": signal.exception.stack } : {}
+				}, new Date(signal.timestamp));
 				span.setStatus({
 					code: _opentelemetry_api.SpanStatusCode.ERROR,
 					message: signal.exception.message
 				});
-				if (!activeSpan) span.end(new Date(signal.timestamp));
+				if (!recordingSpan) span.end(new Date(signal.timestamp));
 				return;
 			}
 			case "log":
@@ -239,7 +264,7 @@ var OpenTelemetryAdapter = class {
 			}, async (span) => {
 				operationRan = true;
 				try {
-					operationResult = await operation();
+					operationResult = await operation(new OpenTelemetrySpan(span));
 					operationSucceeded = true;
 				} catch (error) {
 					operationError = error;
@@ -261,7 +286,10 @@ var OpenTelemetryAdapter = class {
 		}
 		if (operationSucceeded) return operationResult;
 		if (operationRan) throw operationError;
-		return operation();
+		throw new Error("OpenTelemetry tracer did not invoke the span operation");
+	}
+	flush() {
+		return flushOpenTelemetryProviders();
 	}
 	async shutdown() {
 		await this.#shutdown?.();
@@ -275,7 +303,7 @@ function createOpenTelemetryAdapter(config) {
 }
 function currentTraceContext() {
 	const spanContext = _opentelemetry_api.trace.getSpan(_opentelemetry_api.context.active())?.spanContext();
-	if (!spanContext) return void 0;
+	if (!spanContext || !(0, _opentelemetry_api.isSpanContextValid)(spanContext)) return void 0;
 	return {
 		traceId: spanContext.traceId,
 		spanId: spanContext.spanId,
@@ -325,9 +353,6 @@ function recordException(error, attributes = {}) {
 //#endregion
 //#region src/client.ts
 var NodeTelemetryClient = class extends _davidilie_telemetry_core.TelemetryClient {
-	#adapter;
-	#sampleRate;
-	#onError;
 	#onCollectionStateChange;
 	#enabled;
 	#consent;
@@ -353,9 +378,6 @@ var NodeTelemetryClient = class extends _davidilie_telemetry_core.TelemetryClien
 			...config.debug !== void 0 ? { debug: config.debug } : {},
 			...config.onError ? { onError: config.onError } : {}
 		});
-		this.#adapter = adapter;
-		this.#sampleRate = config.providerManagedSampling ? 1 : Number.isFinite(config.sampleRate) ? Math.min(1, Math.max(0, config.sampleRate ?? 1)) : 1;
-		this.#onError = config.onError;
 		this.#onCollectionStateChange = config.onCollectionStateChange;
 		this.#enabled = config.enabled ?? true;
 		this.#consent = config.consent ?? "granted";
@@ -376,26 +398,6 @@ var NodeTelemetryClient = class extends _davidilie_telemetry_core.TelemetryClien
 			consent: this.#consent
 		});
 	}
-	async withSpan(name, operation, attributes = {}, options = {}) {
-		if (!this.#enabled || this.#consent !== "granted" || Math.random() >= this.#sampleRate) return operation();
-		let operationStarted = false;
-		try {
-			return await this.#adapter.withActiveSpan((0, _davidilie_telemetry_core.redactText)(name, 256), () => {
-				operationStarted = true;
-				return operation();
-			}, (0, _davidilie_telemetry_core.sanitizeAttributes)(attributes), options);
-		} catch (error) {
-			if (operationStarted) throw error;
-			this.#reportAdapterError(error, { operation: "startSpan" });
-			return operation();
-		}
-	}
-	#reportAdapterError(error, context) {
-		if (!this.#onError) return;
-		try {
-			Promise.resolve(this.#onError(error, context)).catch(() => void 0);
-		} catch {}
-	}
 };
 function createNodeTelemetry(config) {
 	return new NodeTelemetryClient(config);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@davidilie/telemetry-node",
-  "version": "0.1.3",
+  "version": "0.3.0",
   "description": "OpenTelemetry-backed telemetry for Node.js services.",
   "keywords": [
     "analytics",
@@ -65,7 +65,7 @@
     "clean": "rm -rf dist coverage"
   },
   "dependencies": {
-    "@davidilie/telemetry-core": "0.1.3",
+    "@davidilie/telemetry-core": "0.3.0",
     "@opentelemetry/api-logs": "0.221.0",
     "@opentelemetry/resources": "2.10.0",
     "@opentelemetry/sdk-node": "0.221.0",