@blaxel/telemetry 0.2.18-dev.141 → 0.2.18-dev.142

package/dist/telemetry.d.ts

@@ -34,7 +34,6 @@ declare class TelemetryManager {
     private configured;
     constructor();
     initialize(): void;
-    setupBasicTracerProvider(): void;
     setConfiguration(): Promise<void>;
     get tracer(): import("@opentelemetry/api").Tracer;
     get enabled(): boolean;
@@ -60,6 +59,7 @@ declare class TelemetryManager {
      */
     getLogExporter(): OTLPLogExporter;
     instrumentApp(): void;
+    setupProviders(): void;
     setExporters(): void;
     shutdownApp(): Promise<void>;
 }

package/dist/telemetry.js

@@ -78,49 +78,13 @@ class TelemetryManager {
         if (!this.enabled || this.initialized) {
             return;
         }
-        // Setup basic TracerProvider first - this is critical for context propagation
-        this.setupBasicTracerProvider();
-        // Now setup instrumentation - it can now extract traceparent headers
         this.instrumentApp();
         this.setupSignalHandler();
         this.initialized = true;
-        // Add exporters later asynchronously
         this.setConfiguration().catch((error) => {
             core_1.logger.error("Error setting configuration:", error);
         });
     }
-    setupBasicTracerProvider() {
-        const resource = new BlaxelResource(this.resourceAttributes);
-        // Create TracerProvider with complete setup including exporters
-        // This ensures we don't need to recreate it later, preserving context
-        const spanProcessors = [
-            new DefaultAttributesSpanProcessor({
-                "workload.id": core_1.settings.name || "",
-                "workload.type": core_1.settings.type ? core_1.settings.type + "s" : "",
-                workspace: core_1.settings.workspace || "",
-            }),
-        ];
-        // Try to add exporters if authentication is available
-        try {
-            if (core_1.settings.authorization) {
-                const traceExporter = this.getTraceExporter();
-                spanProcessors.push(new sdk_trace_node_1.BatchSpanProcessor(traceExporter));
-                spanProcessors.push(new HasBeenProcessedSpanProcessor(traceExporter));
-                console.log("🚀 TracerProvider created with exporters from the start");
-            }
-        }
-        catch {
-            console.log("🚀 TracerProvider created without exporters - will be added later");
-        }
-        this.nodeTracerProvider = new sdk_trace_node_1.NodeTracerProvider({
-            resource,
-            sampler: new sdk_trace_node_1.AlwaysOnSampler(),
-            spanProcessors,
-        });
-        // Register immediately - this enables context propagation
-        this.nodeTracerProvider.register();
-        console.log("🚀 TracerProvider registered BEFORE instrumentation - context propagation enabled");
-    }
     async setConfiguration() {
         if (!this.enabled || this.configured) {
             return;
@@ -230,39 +194,24 @@ class TelemetryManager {
     instrumentApp() {
         // Setup telemetry provider first
         core_1.telemetryRegistry.registerProvider(new telemetry_provider_1.OtelTelemetryProvider());
+        // Setup TracerProvider and other providers BEFORE instrumentation
+        this.setupProviders();
         const httpInstrumentation = new instrumentation_http_1.HttpInstrumentation({
-            requireParentforOutgoingSpans: true, // Maintain parent-child relationships
+            requireParentforOutgoingSpans: false, // Allow spans from incoming headers
             requestHook: (span) => {
                 // Debug incoming trace headers
                 console.log("=== HTTP INSTRUMENTATION REQUEST HOOK ===");
-                // Check if TracerProvider is registered
-                const tracer = api_1.trace.getTracer("debug");
-                console.log("TracerProvider available:", !!tracer);
-                // Check active context
-                const activeContext = api_1.context.active();
-                const spanContext = api_1.trace.getSpanContext(activeContext);
-                console.log("Span context from active context:", spanContext
-                    ? {
-                        traceId: spanContext.traceId,
-                        spanId: spanContext.spanId,
-                        traceFlags: spanContext.traceFlags,
-                    }
-                    : "null");
                 // Check current span context after HTTP instrumentation processes headers
                 const activeSpan = api_1.trace.getActiveSpan();
                 if (activeSpan) {
-                    console.log("✅ Active span found in request hook:", {
+                    console.log("Active span in request hook:", {
                         traceId: activeSpan.spanContext().traceId,
                         spanId: activeSpan.spanContext().spanId,
                         traceFlags: activeSpan.spanContext().traceFlags,
                     });
                 }
                 else {
-                    console.log("❌ No active span found in request hook");
-                    console.log("Possible causes:");
-                    console.log("1. TracerProvider not registered before HTTP instrumentation");
-                    console.log("2. Traceparent header not properly extracted");
-                    console.log("3. Context lost across async boundaries");
+                    console.log("No active span found in request hook");
                 }
                 console.log("Current span from hook parameter:", {
                     traceId: span.spanContext().traceId,
@@ -276,6 +225,24 @@ class TelemetryManager {
             instrumentations: [httpInstrumentation],
         });
     }
+    setupProviders() {
+        const resource = new BlaxelResource(this.resourceAttributes);
+        // Setup TracerProvider first - this is critical for context propagation
+        this.nodeTracerProvider = new sdk_trace_node_1.NodeTracerProvider({
+            resource,
+            sampler: new sdk_trace_node_1.AlwaysOnSampler(),
+            spanProcessors: [
+                new DefaultAttributesSpanProcessor({
+                    "workload.id": core_1.settings.name || "",
+                    "workload.type": core_1.settings.type ? core_1.settings.type + "s" : "",
+                    workspace: core_1.settings.workspace || "",
+                }),
+            ],
+        });
+        // Register the tracer provider BEFORE any instrumentation
+        this.nodeTracerProvider.register();
+        console.log("TracerProvider registered - ready for context propagation");
+    }
     setExporters() {
         const resource = new BlaxelResource(this.resourceAttributes);
         // Setup logging
@@ -285,38 +252,24 @@ class TelemetryManager {
         });
         this.loggerProvider.addLogRecordProcessor(new sdk_logs_1.BatchLogRecordProcessor(logExporter));
         api_logs_1.logs.setGlobalLoggerProvider(this.loggerProvider);
-        // Check if we need to add exporters to existing TracerProvider
-        if (this.nodeTracerProvider && !core_1.settings.authorization) {
-            // If we didn't have authorization before but we do now, recreate with exporters
-            // But only if we didn't already have them
-            try {
-                const traceExporter = this.getTraceExporter();
-                // Unfortunately, we need to recreate the TracerProvider to add exporters
-                // This is a limitation of OpenTelemetry SDK
-                const newTracerProvider = new sdk_trace_node_1.NodeTracerProvider({
-                    resource,
-                    sampler: new sdk_trace_node_1.AlwaysOnSampler(),
-                    spanProcessors: [
-                        new DefaultAttributesSpanProcessor({
-                            "workload.id": core_1.settings.name || "",
-                            "workload.type": core_1.settings.type ? core_1.settings.type + "s" : "",
-                            workspace: core_1.settings.workspace || "",
-                        }),
-                        new sdk_trace_node_1.BatchSpanProcessor(traceExporter),
-                        new HasBeenProcessedSpanProcessor(traceExporter),
-                    ],
-                });
-                this.nodeTracerProvider = newTracerProvider;
-                this.nodeTracerProvider.register();
-                console.log("📡 TracerProvider updated with exporters");
-            }
-            catch {
-                console.log("📡 Could not add exporters to TracerProvider");
-            }
-        }
-        else {
-            console.log("📡 TracerProvider already has exporters or no changes needed");
-        }
+        // Recreate TracerProvider with exporters - this ensures proper initialization
+        const traceExporter = this.getTraceExporter();
+        this.nodeTracerProvider = new sdk_trace_node_1.NodeTracerProvider({
+            resource,
+            sampler: new sdk_trace_node_1.AlwaysOnSampler(),
+            spanProcessors: [
+                new DefaultAttributesSpanProcessor({
+                    "workload.id": core_1.settings.name || "",
+                    "workload.type": core_1.settings.type ? core_1.settings.type + "s" : "",
+                    workspace: core_1.settings.workspace || "",
+                }),
+                new sdk_trace_node_1.BatchSpanProcessor(traceExporter),
+                new HasBeenProcessedSpanProcessor(traceExporter),
+            ],
+        });
+        // Re-register the tracer provider with exporters
+        this.nodeTracerProvider.register();
+        console.log("TracerProvider re-registered with exporters");
         // Setup metrics
         const metricExporter = this.getMetricExporter();
         this.meterProvider = new sdk_metrics_1.MeterProvider({

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@blaxel/telemetry",
-  "version": "0.2.18-dev.141",
+  "version": "0.2.18-dev.142",
   "description": "Blaxel SDK for TypeScript",
   "license": "MIT",
   "author": "Blaxel, INC (https://blaxel.ai)",
@@ -71,7 +71,7 @@
     "@opentelemetry/sdk-trace-base": "^2.0.0",
     "@opentelemetry/sdk-trace-node": "^2.0.0",
     "ai": "^4.3.13",
-    "@blaxel/core": "0.2.18-dev.141"
+    "@blaxel/core": "0.2.18-dev.142"
   },
   "devDependencies": {
     "@eslint/js": "^9.26.0",