@ciq-dev/neoiq-foundation-node 1.1.2-beta.1 → 1.1.2-beta.3

package/dist/index.mjs

@@ -1,7 +1,7 @@
-import { AutoInstrumentationConfigSchema, FeaturesConfigSchema, FoundationConfigSchema, LoggingConfigSchema, OtelConfigSchema, RedactionConfigSchema, RequestLoggingConfigSchema, ShutdownConfigSchema, SpanStatusCode, VaultConfigSchema, context, getActiveSpan, getDefaultOtelEndpoint, getTraceContext, getTracer, isTracingEnabled, parseConfig, propagation, setupTracing, shutdownTracing, trace } from "./tracing-Mxxp4y94.mjs";
+import { AutoInstrumentationConfigSchema, FeaturesConfigSchema, FoundationConfigSchema, LoggingConfigSchema, OtelConfigSchema, RedactionConfigSchema, RequestLoggingConfigSchema, ShutdownConfigSchema, SpanStatusCode, VaultConfigSchema, context, getActiveSpan, getDefaultOtelEndpoint, getTraceContext, getTracer, isTracingEnabled, parseConfig, propagation, setupTracing, shutdownTracing, trace } from "./tracing-BdAbHWu1.mjs";
 import { resourceFromAttributes } from "@opentelemetry/resources";
 import { ATTR_SERVICE_NAME, ATTR_SERVICE_VERSION } from "@opentelemetry/semantic-conventions";
-import { DiagLogLevel, SpanStatusCode as SpanStatusCode$1, context as context$1, diag, metrics, propagation as propagation$1, trace as trace$1 } from "@opentelemetry/api";
+import { SpanStatusCode as SpanStatusCode$1, context as context$1, metrics, propagation as propagation$1, trace as trace$1 } from "@opentelemetry/api";
 import { AsyncLocalStorage } from "async_hooks";
 import pino from "pino";
 import { OTLPMetricExporter } from "@opentelemetry/exporter-metrics-otlp-grpc";
@@ -341,7 +341,34 @@ function getGlobalLogger() {
 //#region src/features/metrics.ts
 let meterProvider = null;
 let isInitialized = false;
-/** Initialize OpenTelemetry metrics */
+/**
+* Build a gRPC OTLP metric reader for the tracing NodeSDK to own.
+*
+* Preferred path: hand this reader to `setupTracing({ metricReader })` so a single
+* NodeSDK owns both traces and metrics — one global MeterProvider, exporting to the
+* same gRPC collector. This avoids the duplicate global-MeterProvider registration
+* that occurs when NodeSDK and a standalone `setupMetrics` both try to register.
+*
+* The MeterProvider, resource, and global registration are owned by NodeSDK; this
+* function only constructs the reader/exporter.
+*/
+function createMetricReader(options = {}) {
+	const { endpoint = getDefaultOtelEndpoint(), intervalMs = 5e3 } = options;
+	const exporter = new OTLPMetricExporter({ url: endpoint });
+	const reader = new PeriodicExportingMetricReader({
+		exporter,
+		exportIntervalMillis: intervalMs
+	});
+	isInitialized = true;
+	return reader;
+}
+/**
+* Initialize a standalone metrics pipeline with its own MeterProvider.
+*
+* Fallback for when tracing (and thus the NodeSDK) is disabled but metrics are
+* enabled. When tracing is on, prefer {@link createMetricReader} + NodeSDK so a
+* single SDK owns both signals.
+*/
 function setupMetrics(options) {
 	if (isInitialized) {
 		console.warn("[neoiq-foundation] Metrics already initialized");
@@ -944,38 +971,16 @@ function createFoundation(input) {
 		};
 		console.error("[neoiq-foundation] Logger setup failed, using console:", loggingError);
 	}
-	const diagLevelName = (process.env.OTEL_DIAG_LOG_LEVEL || otel.diagLogLevel).toLowerCase();
-	if (diagLevelName !== "none") {
-		const diagLevels = {
-			error: DiagLogLevel.ERROR,
-			warn: DiagLogLevel.WARN,
-			info: DiagLogLevel.INFO,
-			debug: DiagLogLevel.DEBUG,
-			verbose: DiagLogLevel.VERBOSE
-		};
-		const diagLogger = {
-			error: (message, ...args) => logger.error({
-				source: "otel",
-				args
-			}, message),
-			warn: (message, ...args) => logger.warn({
-				source: "otel",
-				args
-			}, message),
-			info: (message, ...args) => logger.info({
-				source: "otel",
-				args
-			}, message),
-			debug: (message, ...args) => logger.debug({
-				source: "otel",
-				args
-			}, message),
-			verbose: (message, ...args) => logger.debug({
-				source: "otel",
-				args
-			}, message)
-		};
-		diag.setLogger(diagLogger, diagLevels[diagLevelName] ?? DiagLogLevel.WARN);
+	let metricReader;
+	let metricsError;
+	if (features.metrics && features.tracing) try {
+		metricReader = createMetricReader({
+			endpoint: otel.endpoint,
+			intervalMs: otel.metricsIntervalMs
+		});
+	} catch (err) {
+		metricsError = err.message;
+		logger.error({ error: metricsError }, "Metrics setup failed - continuing without metrics");
 	}
 	let tracerInstance = null;
 	let tracingError;
@@ -985,7 +990,8 @@ function createFoundation(input) {
 			serviceVersion,
 			environment,
 			endpoint: otel.endpoint,
-			autoInstrumentation: featuresConfig.autoInstrumentation
+			autoInstrumentation: featuresConfig.autoInstrumentation,
+			metricReader
 		});
 		tracerInstance = getTracer(serviceName);
 		logger.info({
@@ -996,9 +1002,7 @@ function createFoundation(input) {
 		tracingError = err.message;
 		logger.error({ error: tracingError }, "Tracing setup failed - continuing without tracing");
 	}
-	let meterInstance = null;
-	let metricsError;
-	if (features.metrics) try {
+	if (features.metrics && !features.tracing && !metricsError) try {
 		setupMetrics({
 			serviceName,
 			serviceVersion,
@@ -1006,14 +1010,18 @@ function createFoundation(input) {
 			endpoint: otel.endpoint,
 			intervalMs: otel.metricsIntervalMs
 		});
+	} catch (err) {
+		metricsError = err.message;
+		logger.error({ error: metricsError }, "Metrics setup failed - continuing without metrics");
+	}
+	else if (metricReader && tracingError && !metricsError) metricsError = `metrics SDK (tracing) failed to start: ${tracingError}`;
+	let meterInstance = null;
+	if (features.metrics && !metricsError) {
 		meterInstance = getMeter(serviceName);
 		logger.info({
 			feature: "metrics",
 			interval: `${otel.metricsIntervalMs}ms`
 		}, "Metrics enabled");
-	} catch (err) {
-		metricsError = err.message;
-		logger.error({ error: metricsError }, "Metrics setup failed - continuing without metrics");
 	}
 	logger.info({
 		serviceName,