@juspay/neurolink 12.59.0 → 13.0.1

package/dist/mcp/externalServerManager.js

@@ -14,7 +14,6 @@ import { toolRegistry } from "./toolRegistry.js";
 import { HITLUserRejectedError, HITLTimeoutError } from "../hitl/hitlErrors.js";
 import { detectCategory } from "../utils/mcpDefaults.js";
 import { isObject, isNonNullObject } from "../utils/typeUtils.js";
-import { TelemetryService } from "../telemetry/telemetryService.js";
 import { tracers } from "../telemetry/tracers.js";
 import { SpanStatusCode } from "@opentelemetry/api";
 // `@modelcontextprotocol/sdk`'s stdio transport (imported above) does
@@ -1085,7 +1084,7 @@ export class ExternalServerManager extends EventEmitter {
             throw new Error(`Server '${serverId}' not found`);
         }
         const config = instance.config;
-        const span = tracers.mcp.startSpan("neurolink.mcp.server.start", {
+        const span = tracers.mcp.startSpan("mcp.server.start", {
             attributes: {
                 "mcp.server_id": serverId,
                 "mcp.transport": config.transport,
@@ -1281,7 +1280,7 @@ export class ExternalServerManager extends EventEmitter {
         if (!instance) {
             return;
         }
-        const span = tracers.mcp.startSpan("neurolink.mcp.server.stop", {
+        const span = tracers.mcp.startSpan("mcp.server.stop", {
             attributes: {
                 "mcp.server_id": serverId,
             },
@@ -1503,7 +1502,7 @@ export class ExternalServerManager extends EventEmitter {
         }
         instance.restartTimer = setTimeout(async () => {
             this.retryDeadlines.delete(serverId);
-            const restartSpan = tracers.mcp.startSpan("neurolink.mcp.server.restart", {
+            const restartSpan = tracers.mcp.startSpan("mcp.server.restart", {
                 attributes: {
                     "mcp.server_id": serverId,
                     "mcp.restart_attempt": instance.reconnectAttempts,
@@ -2050,10 +2049,6 @@ export class ExternalServerManager extends EventEmitter {
                 // of counting every resolved error as a success.
                 const resolvedError = result.isErrorResult === true;
                 if (resolvedError) {
-                    // Keep this instance's totalErrors (and the getStatistics() /
-                    // getExternalMCPStatistics() it feeds) consistent with the breaker,
-                    // TelemetryService and ToolDiscoveryService's failedCalls, all of
-                    // which already count a resolved isError result as a failure.
                     instance.metrics.totalErrors++;
                 }
                 mcpLogger.debug(resolvedError
@@ -2061,12 +2056,6 @@ export class ExternalServerManager extends EventEmitter {
                     : `[ExternalServerManager] Tool executed successfully: ${toolName} on ${serverId}`, {
                     duration,
                 });
-                try {
-                    TelemetryService.getInstance()?.recordMCPToolCall(toolName, duration, !resolvedError);
-                }
-                catch {
-                    /* telemetry should not break execution */
-                }
                 return result.data;
             }
             else {
@@ -2075,13 +2064,6 @@ export class ExternalServerManager extends EventEmitter {
         }
         catch (error) {
             instance.metrics.totalErrors++;
-            try {
-                const errorDuration = Date.now() - startTime;
-                TelemetryService.getInstance()?.recordMCPToolCall(toolName, errorDuration, false);
-            }
-            catch {
-                /* telemetry should not break execution */
-            }
             mcpLogger.error(`[ExternalServerManager] Tool execution failed: ${toolName} on ${serverId}`, error);
             // The SDK rejects with its connection-closed error when the server is
             // gone. Without this transition the instance stayed "connected" and

package/dist/mcp/toolDiscoveryService.js

@@ -111,7 +111,7 @@ export class ToolDiscoveryService extends EventEmitter {
      */
     async discoverTools(serverId, client, timeout = DEFAULT_TOOL_TIMEOUT, isCurrentDiscovery = () => true, startupDiscovery = false) {
         return withSpan({
-            name: "neurolink.mcp.discoverTools",
+            name: "mcp.discoverTools",
             tracer: tracers.mcp,
             attributes: { "mcp.server_id": serverId },
         }, async (span) => {
@@ -461,21 +461,14 @@ export class ToolDiscoveryService extends EventEmitter {
             // clean resolve and could never open for a tool that fails this way.
             let isErrorResultDetected = false;
             const result = await circuitBreaker.execute(async (recordResolvedFailure) => {
-                return mcpTracer.startActiveSpan("neurolink.mcp.callTool", {
+                return mcpTracer.startActiveSpan("mcp.callTool", {
                     kind: SpanKind.CLIENT,
                     attributes: {
                         "mcp.server_id": serverId,
                         "mcp.tool_name": toolName,
                         "mcp.timeout_ms": effectiveTimeout,
-                        // Curator P1-4: Langfuse observations rely on ai.*/gen_ai.*
-                        // attributes for tool name and I/O previews. Provide them so
-                        // the SPAN observation in Langfuse is legible without
-                        // timestamp-joining against the parent ai.toolCall. Redact
-                        // parameters via the existing secret-stripping helper so
-                        // tokens/credentials/paths don't leave the process.
                         "ai.tool.name": toolName,
-                        "gen_ai.tool.name": toolName,
-                        "gen_ai.request": safeJsonStringify({
+                        "mcp.request": safeJsonStringify({
                             name: toolName,
                             arguments: redactForPreview(effectiveParameters),
                         }, 2048),
@@ -535,12 +528,7 @@ export class ToolDiscoveryService extends EventEmitter {
                             }
                         }
                         // ── end normalization ─────────────────────────────────────────
-                        // Curator P1-4: build gen_ai.response AFTER normalization so
-                        // large payloads use the compact surrogate instead of the raw
-                        // result (avoids redundant stringify + memory hit on payloads
-                        // that were specifically externalized to Redis). Redact via the
-                        // same secret-stripping path used for request parameters.
-                        callSpan.setAttribute("gen_ai.response", safeJsonStringify(redactForPreview(resultForPreview), 2048));
+                        callSpan.setAttribute("mcp.response", safeJsonStringify(redactForPreview(resultForPreview), 2048));
                         return resultForReturn;
                     }
                     catch (err) {

package/dist/mcp/toolRegistry.js

@@ -288,14 +288,11 @@ export class MCPToolRegistry extends MCPRegistry {
             }
         }
         return withSpan({
-            name: "neurolink.tool.registry.execute",
+            name: "tool.registry.execute",
             tracer: tracers.mcp,
             attributes: {
-                [ATTR.GEN_AI_TOOL_NAME]: toolName,
+                "tool.name": toolName,
                 [ATTR.MCP_SERVER_ID]: preResolvedServerId || "builtin",
-                // Curator P1-3: registry-level wrapper — duplicates ai.toolCall in
-                // Langfuse. Retained for OTel/metrics; skipped for Langfuse export.
-                "langfuse.internal": true,
             },
         }, async (span) => {
             try {

package/dist/memory/memoryRetrievalTools.js

@@ -91,7 +91,7 @@ export function createMemoryRetrievalTools(memoryManager, artifactStore) {
                     "search starts there; use `nextSearchOffset` to continue."),
             }),
             execute: async (args) => withSpan({
-                name: "neurolink.memory.retrieve_context",
+                name: "memory.retrieve_context",
                 tracer: tracers.memory,
                 attributes: {
                     "memory.operation": args.artifactId

package/dist/neurolink.d.ts

@@ -229,9 +229,6 @@ export declare class NeuroLink {
     private decisionHookTimeoutMs?;
     private skillsManagerInstance?;
     private skillsConfig?;
-    /**
-     * Extract and set Langfuse context from options with proper async scoping
-     */
     private setLangfuseContextFromOptions;
     private createMetricsTraceContext;
     private enforceSessionBudget;
@@ -369,6 +366,7 @@ export declare class NeuroLink {
      * @throws {Error} When HITL configuration is invalid (if enabled)
      */
     private observabilityConfig?;
+    private telemetryHeld;
     private metricsAggregator;
     private analyticsService;
     /**
@@ -547,10 +545,8 @@ export declare class NeuroLink {
      * Setup event handlers for external server manager
      */
     private setupExternalServerEventHandlers;
-    /**
-     * Initialize Langfuse observability for AI operations tracking
-     */
-    private initializeLangfuse;
+    private holdTelemetry;
+    private releaseTelemetryHold;
     /**
      * Log constructor completion with final state summary
      */
@@ -676,12 +672,12 @@ export declare class NeuroLink {
      */
     getObservabilityConfig(): ObservabilityConfig | undefined;
     /**
-     * Check if Langfuse telemetry is enabled
-     * Centralized utility to avoid duplication across providers
+     * @description Reports whether any tracing backend is active.
+     * @returns {boolean} `true` when OpenTelemetry is initialized or a Langfuse exporter is running.
      */
     isTelemetryEnabled(): boolean;
     /**
-     * Get comprehensive telemetry status including Langfuse, OTel, and exporter health
+     * Get comprehensive telemetry status including Langfuse and OpenTelemetry state
      */
     getTelemetryStatus(): {
         enabled: boolean;
@@ -695,15 +691,6 @@ export declare class NeuroLink {
             endpoint?: string;
             serviceName?: string;
         };
-        exporters?: Array<{
-            name: string;
-            enabled: boolean;
-            healthy: boolean;
-            pendingSpans: number;
-            lastExportTime?: string;
-            latencyMs?: number;
-            errors?: string[];
-        }>;
     };
     /**
      * Get aggregated observability metrics (latency, tokens, cost, success rate)
@@ -773,10 +760,9 @@ export declare class NeuroLink {
      */
     private estimateCostFromUsage;
     /**
-     * Handle generation:end for Pipeline B spans + advanced analytics.
-     * When pipelineAHandled is set, skips Pipeline B spans but still tracks
-     * analytics so successful AI-SDK generate/stream calls are recorded.
-     * Stream analytics are owned here (not stream:complete) to avoid duplicates.
+     * Handle generation:end: records advanced analytics for every generation
+     * and the metrics span for a non-streamed one. A streamed generation gets
+     * its span from stream:complete / stream:error instead.
      */
     private handleGenerationEndMetrics;
     /**
@@ -903,6 +889,7 @@ export declare class NeuroLink {
      * so every log call beneath it is attributable to this instance.
      */
     private generateInInstanceScope;
+    private runGenerateTurn;
     private fireConsumerOnErrorIfNotFired;
     /**
      * Curator P2-3: wraps a generate/stream call with the fallback
@@ -1213,6 +1200,7 @@ export declare class NeuroLink {
      * are covered.
      */
     private streamInInstanceScope;
+    private runStreamTurn;
     /**
      * Curator P2-3 / Reviewer Finding #2: stream-fallback that also covers
      * errors thrown during async iteration (e.g. LiteLLM throwing inside
@@ -1288,6 +1276,9 @@ export declare class NeuroLink {
     private validateStreamRequestOptions;
     private maybeHandleWorkflowStreamRequest;
     private runStandardStreamRequest;
+    private drainStandardStreamOnce;
+    private drainStandardStream;
+    private settleStandardStreamDrain;
     /** Wrap one provider stream with incremental TTS synthesis. */
     private createIncrementalTTSStream;
     /** Prevent provider fallback streams from duplicating outer streaming TTS. */
@@ -3085,10 +3076,8 @@ export declare class NeuroLink {
      *
      * Worker mode is the framework-provided version of the config block every
      * consumer used to copy by hand: conversation memory OFF, orchestration
-     * OFF, observability inherited from this instance with
-     * `autoDetectExternalProvider: true` + `skipLangfuseSpanProcessor: true`
-     * (worker spans join the host's tracer without duplicate Langfuse
-     * exports), credentials inherited, the host's tool registry shared (so
+     * OFF, observability inherited from this instance (worker spans join the
+     * host's tracer provider), credentials inherited, the host's tool registry shared (so
      * worker tool calls reuse the host's connections), and an internal log
      * bridge attached with a caller-supplied tag.
      *