@juspay/neurolink 9.80.2 → 9.80.4

package/dist/lib/types/generate.d.ts

@@ -296,6 +296,19 @@ export type GenerateOptions = {
      * ```
      */
     enabledToolNames?: string[];
+    /**
+     * Request timeout (e.g. 30000, '30s', '2m').
+     *
+     * PER-STEP semantics in agentic loops: on providers that run a native
+     * multi-step tool loop (Vertex Gemini / Vertex Claude), this bounds EACH
+     * model call in the loop, not the whole turn — a tool-heavy turn may run
+     * far longer than this value in total. Size it for the slowest single
+     * step (default 300s), and use `abortSignal` for a total-turn deadline.
+     *
+     * When set explicitly, a step timeout is surfaced immediately instead of
+     * burning internal retries/fallbacks that would re-run the same
+     * provider+model with the same doomed budget.
+     */
     timeout?: number | string;
     /** AbortSignal for external cancellation of the AI call */
     abortSignal?: AbortSignal;

package/dist/lib/types/stream.d.ts

@@ -536,6 +536,7 @@ export type StreamResult = {
         hasToolErrors?: boolean;
         guardrailsBlocked?: boolean;
         error?: string;
+        finishReason?: string;
         thoughtSignature?: string;
         thoughts?: Array<{
             id?: string;

package/dist/lib/utils/conversationMemory.js

@@ -5,6 +5,7 @@
 import { SpanKind, SpanStatusCode } from "@opentelemetry/api";
 import { tracers } from "../telemetry/tracers.js";
 import { withTimeout } from "./errorHandling.js";
+import { safeDebugSerialize, sanitizeRecord } from "./logSanitize.js";
 import { DEFAULT_FALLBACK_THRESHOLD, getConversationMemoryDefaults, MEMORY_THRESHOLD_PERCENTAGE, } from "../config/conversationMemory.js";
 import { getAvailableInputTokens } from "../constants/contextWindows.js";
 import { buildSummarizationPrompt } from "../context/prompts/summarizationPrompt.js";
@@ -106,20 +107,30 @@ export function applyConversationMemoryDefaults(userConfig) {
  * Get conversation history as message array, summarizing if needed.
  */
 export async function getConversationMessages(conversationMemory, options) {
-    logger.debug("[conversationMemoryUtils] getConversationMessages called", {
-        hasMemory: !!conversationMemory,
-        memoryType: conversationMemory?.constructor?.name || "NONE",
-        hasContext: !!options.context,
-        enableSummarization: options.enableSummarization ?? false,
-        options: JSON.stringify(options, null, 2),
-    });
+    // Logger Guard: options carries the full conversation history + tool
+    // outputs; eager JSON.stringify of it can throw RangeError: Invalid string
+    // length and abort the turn. Serialize lazily, bounded, never-throwing,
+    // and redacted (sanitizeRecord strips credential/PII keys).
+    if (logger.shouldLog("debug")) {
+        logger.debug("[conversationMemoryUtils] getConversationMessages called", {
+            hasMemory: !!conversationMemory,
+            memoryType: conversationMemory?.constructor?.name || "NONE",
+            hasContext: !!options.context,
+            enableSummarization: options.enableSummarization ?? false,
+            options: safeDebugSerialize(sanitizeRecord(options)),
+        });
+    }
     if (!conversationMemory || !options.context) {
         logger.warn("[conversationMemoryUtils] No memory or context, returning empty messages", {
             hasMemory: !!conversationMemory,
             memoryType: conversationMemory?.constructor?.name || "NONE",
             hasContext: !!options.context,
             enableSummarization: options.enableSummarization ?? false,
-            options: JSON.stringify(options, null, 2),
+            // The options dump is debug-grade detail — don't pay for (or leak)
+            // the serialization on every memory-disabled call at warn level.
+            ...(logger.shouldLog("debug")
+                ? { options: safeDebugSerialize(sanitizeRecord(options)) }
+                : {}),
         });
         return [];
     }

package/dist/lib/utils/logSanitize.d.ts

@@ -26,6 +26,32 @@
  * @param maxLen - Maximum number of characters to keep (default 500).
  */
 export declare function sanitizeForLog(text: string, maxLen?: number): string;
+/**
+ * Stringify non-string message/tool content without ever throwing.
+ * JSON.stringify on a giant multimodal/tool-result payload can exceed V8's
+ * maximum string length and throw `RangeError: Invalid string length` — a
+ * generation turn must degrade to a placeholder, not abort, when that
+ * happens. Shared by the SDK core and providers (single source of truth).
+ */
+export declare function stringifyContentSafe(content: unknown): string;
+/**
+ * Serialize an arbitrary value for a debug log without ever throwing or
+ * producing an unbounded string.
+ *
+ * `JSON.stringify` on a full options object (conversation history + tool
+ * outputs) can exceed V8's maximum string length and throw
+ * `RangeError: Invalid string length`, which — when evaluated eagerly inside
+ * a logger call — aborts the surrounding generation turn (observed in
+ * production). This helper caps the output and converts any stringify
+ * failure (RangeError, circular structure, BigInt, …) into a placeholder.
+ *
+ * Callers MUST still gate on `logger.shouldLog("debug")` per the Logger
+ * Guard rule — this helper makes serialization safe, not free.
+ *
+ * @param value  - Arbitrary value to serialize.
+ * @param maxLen - Maximum number of characters to keep (default 10 000).
+ */
+export declare function safeDebugSerialize(value: unknown, maxLen?: number): string;
 /**
  * Strip embedded `user:pass@` credentials from a URL's authority component
  * before logging it or surfacing it in a user-facing error.

package/dist/lib/utils/logSanitize.js

@@ -74,6 +74,13 @@ const SENSITIVE_OBJECT_KEYS = [
     "oauth",
     "oauthToken",
     "credentials",
+    "authContext",
+    "authToken",
+    "sessionToken",
+    "serviceAccountKey",
+    "secretAccessKey",
+    // PII, not a secret — but it has no business in debug log dumps.
+    "userEmail",
 ];
 /**
  * Truncate `text` to `maxLen` chars then replace embedded secrets with `***`.
@@ -91,6 +98,55 @@ export function sanitizeForLog(text, maxLen = 500) {
     }
     return text.slice(0, maxLen).replace(SECRET_PATTERN, "***");
 }
+/**
+ * Stringify non-string message/tool content without ever throwing.
+ * JSON.stringify on a giant multimodal/tool-result payload can exceed V8's
+ * maximum string length and throw `RangeError: Invalid string length` — a
+ * generation turn must degrade to a placeholder, not abort, when that
+ * happens. Shared by the SDK core and providers (single source of truth).
+ */
+export function stringifyContentSafe(content) {
+    if (typeof content === "string") {
+        return content;
+    }
+    try {
+        return JSON.stringify(content) ?? String(content);
+    }
+    catch {
+        return "[content too large to serialize]";
+    }
+}
+/**
+ * Serialize an arbitrary value for a debug log without ever throwing or
+ * producing an unbounded string.
+ *
+ * `JSON.stringify` on a full options object (conversation history + tool
+ * outputs) can exceed V8's maximum string length and throw
+ * `RangeError: Invalid string length`, which — when evaluated eagerly inside
+ * a logger call — aborts the surrounding generation turn (observed in
+ * production). This helper caps the output and converts any stringify
+ * failure (RangeError, circular structure, BigInt, …) into a placeholder.
+ *
+ * Callers MUST still gate on `logger.shouldLog("debug")` per the Logger
+ * Guard rule — this helper makes serialization safe, not free.
+ *
+ * @param value  - Arbitrary value to serialize.
+ * @param maxLen - Maximum number of characters to keep (default 10 000).
+ */
+export function safeDebugSerialize(value, maxLen = 10_000) {
+    try {
+        const json = JSON.stringify(value);
+        if (json === undefined) {
+            return String(value);
+        }
+        return json.length > maxLen
+            ? `${json.slice(0, maxLen)}…[truncated ${json.length - maxLen} chars]`
+            : json;
+    }
+    catch (error) {
+        return `[unserializable: ${error instanceof Error ? error.message : String(error)}]`;
+    }
+}
 /**
  * Strip embedded `user:pass@` credentials from a URL's authority component
  * before logging it or surfacing it in a user-facing error.

package/dist/mcp/externalServerManager.js

@@ -348,7 +348,9 @@ export class ExternalServerManager extends EventEmitter {
                 }
                 catch (error) {
                     const errorMsg = `Failed to load MCP server ${serverId}: ${error instanceof Error ? error.message : String(error)}`;
-                    mcpLogger.warn(`[ExternalServerManager] ${errorMsg}`);
+                    // No log here: the result-processing loop below owns the single
+                    // ERROR record for this failure (logging in both places
+                    // double-counted every parallel-load failure).
                     return { serverId, error: errorMsg };
                 }
             });
@@ -366,11 +368,16 @@ export class ExternalServerManager extends EventEmitter {
                     }
                     else if (error) {
                         errors.push(error);
+                        // Config-loaded servers never pass through
+                        // NeuroLink.addExternalMCPServer, so this is the outermost point
+                        // for them — it owns the single ERROR record per failed server.
+                        mcpLogger.error(`[ExternalServerManager] Failed to load server ${serverId}: ${error}`);
                     }
                     else if (serverResult && !serverResult.success) {
                         const errorMsg = `Failed to load server ${serverId}: ${serverResult.error}`;
                         errors.push(errorMsg);
-                        mcpLogger.warn(`[ExternalServerManager] ${errorMsg}`);
+                        // See above: outermost point for config-loaded servers.
+                        mcpLogger.error(`[ExternalServerManager] ${errorMsg}`);
                     }
                 }
                 else {
@@ -482,13 +489,17 @@ export class ExternalServerManager extends EventEmitter {
                     else {
                         const error = `Failed to load server ${serverId}: ${result.error}`;
                         errors.push(error);
-                        mcpLogger.warn(`[ExternalServerManager] ${error}`);
+                        // Config-loaded servers never pass through
+                        // NeuroLink.addExternalMCPServer — this sequential-load branch is
+                        // their outermost point and owns the single ERROR record.
+                        mcpLogger.error(`[ExternalServerManager] ${error}`);
                     }
                 }
                 catch (error) {
                     const errorMsg = `Failed to load MCP server ${serverId}: ${error instanceof Error ? error.message : String(error)}`;
                     errors.push(errorMsg);
-                    mcpLogger.warn(`[ExternalServerManager] ${errorMsg}`);
+                    // See above: outermost point for config-loaded servers.
+                    mcpLogger.error(`[ExternalServerManager] ${errorMsg}`);
                     // Continue with other servers - don't let one failure break everything
                 }
             }
@@ -708,7 +719,9 @@ export class ExternalServerManager extends EventEmitter {
             };
         }
         catch (error) {
-            mcpLogger.error(`[ExternalServerManager] Failed to add server ${serverId}:`, error);
+            // debug, not error: NeuroLink.addExternalMCPServer (the public entry
+            // point) emits the single ERROR record for a failed registration.
+            mcpLogger.debug(`[ExternalServerManager] Failed to add server ${serverId}:`, error);
             // Clean up if instance was created
             this.servers.delete(serverId);
             return {
@@ -851,7 +864,9 @@ export class ExternalServerManager extends EventEmitter {
             mcpLogger.info(`[ExternalServerManager] Server started successfully: ${serverId}`);
         }
         catch (error) {
-            mcpLogger.error(`[ExternalServerManager] Failed to start server ${serverId}:`, error);
+            // debug, not error: the failure is rethrown below and surfaces once at
+            // the NeuroLink.addExternalMCPServer entry point.
+            mcpLogger.debug(`[ExternalServerManager] Failed to start server ${serverId}:`, error);
             this.updateServerStatus(serverId, "failed");
             instance.lastError =
                 error instanceof Error ? error.message : String(error);

package/dist/mcp/mcpClientFactory.js

@@ -142,7 +142,11 @@ export class MCPClientFactory {
                     duration: Date.now() - startTime,
                 };
             }
-            mcpLogger.error(`[MCPClientFactory] Failed to create client for ${config.id}:`, error);
+            // debug, not error: this failure propagates up through
+            // ExternalServerManager to NeuroLink.addExternalMCPServer, which emits
+            // the single ERROR record. Logging at every layer turned one failed
+            // server registration into 5 ERROR lines in production.
+            mcpLogger.debug(`[MCPClientFactory] Failed to create client for ${config.id}:`, error);
             obsSpan.durationMs = Date.now() - startTime;
             const endedObsSpan = SpanSerializer.endSpan(obsSpan, SpanStatus.ERROR);
             endedObsSpan.statusMessage = errorMessage;

package/dist/neurolink.js

@@ -72,6 +72,7 @@ import { coerceJsonToSchema } from "./utils/json/coerce.js";
 // Factory processing imports
 import { createCleanStreamOptions, enhanceTextGenerationOptions, processFactoryOptions, processStreamingFactoryOptions, validateFactoryConfig, } from "./utils/factoryProcessing.js";
 import { logger, mcpLogger } from "./utils/logger.js";
+import { redactUrlCredentials, safeDebugSerialize, sanitizeRecord, stringifyContentSafe, } from "./utils/logSanitize.js";
 import { extractMcpErrorText } from "./utils/mcpErrorText.js";
 import { createCustomToolServerInfo, detectCategory, } from "./utils/mcpDefaults.js";
 import { resolveModel } from "./utils/modelAliasResolver.js";
@@ -2156,9 +2157,7 @@ Current user's request: ${currentInput}`;
         const anyOptions = optionsOrPrompt;
         if (anyOptions.messages && anyOptions.messages.length > 0) {
             const lastMessage = anyOptions.messages[anyOptions.messages.length - 1];
-            return typeof lastMessage.content === "string"
-                ? lastMessage.content
-                : JSON.stringify(lastMessage.content);
+            return stringifyContentSafe(lastMessage.content);
         }
         // Handle input.text format
         return optionsOrPrompt.input?.text || "";
@@ -3879,9 +3878,7 @@ Current user's request: ${currentInput}`;
                 ?.filter((m) => m.role === "user" || m.role === "assistant")
                 .map((m) => ({
                 role: m.role,
-                content: typeof m.content === "string"
-                    ? m.content
-                    : JSON.stringify(m.content),
+                content: stringifyContentSafe(m.content),
             })) ??
                 options.conversationHistory,
             timeout: options.timeout,
@@ -3978,9 +3975,7 @@ Current user's request: ${currentInput}`;
                 ?.filter((m) => m.role === "user" || m.role === "assistant")
                 .map((m) => ({
                 role: m.role,
-                content: typeof m.content === "string"
-                    ? m.content
-                    : JSON.stringify(m.content),
+                content: stringifyContentSafe(m.content),
             })) ??
                 options.conversationHistory,
             timeout: options.timeout,
@@ -4603,6 +4598,11 @@ Current user's request: ${currentInput}`;
      */
     async performMCPGenerationRetries(options, generateInternalId, generateInternalStartTime, generateInternalHrTimeStart, functionTag) {
         const maxMcpRetries = RETRY_ATTEMPTS.QUICK;
+        // Captured BEFORE the first attempt: ensureMCPGenerationBudget (inside
+        // tryMCPGeneration) auto-scales options.timeout for >100K-token contexts
+        // when the caller left it unset, so checking options.timeout inside the
+        // catch below could not tell caller-set apart from auto-scaled.
+        const callerSetTimeout = options.timeout !== undefined;
         // NL-007: Track retry metadata for observability
         const retryErrors = [];
         let retryCount = 0;
@@ -4645,6 +4645,20 @@ Current user's request: ${currentInput}`;
                     logger.debug(`[${functionTag}] AbortError detected on attempt ${attempt}, stopping retries`);
                     throw error;
                 }
+                // A TimeoutError against an explicit caller-set options.timeout is
+                // deterministic-by-construction: every remaining retry AND the
+                // direct-generation fallback would re-run the same provider+model
+                // under the same per-step budget. Observed in production as ~650s
+                // of doomed follow-ups after a 90s step timeout. Surface it now.
+                // Name check (not instanceof): two TimeoutError classes exist
+                // (utils/timeout.ts and utils/async/withTimeout.ts) and both stamp
+                // name = "TimeoutError".
+                if (callerSetTimeout &&
+                    error instanceof Error &&
+                    error.name === "TimeoutError") {
+                    logger.warn(`[${functionTag}] Per-step timeout (${String(options.timeout)}) exhausted on attempt ${attempt} — surfacing instead of retrying with the same budget`);
+                    throw error;
+                }
                 // NL-007: Record retry error for observability
                 retryCount++;
                 const errMsg = error instanceof Error ? error.message : String(error);
@@ -7220,11 +7234,18 @@ Current user's request: ${currentInput}`;
      */
     async storeStreamConversationMemory(params) {
         const { enhancedOptions, providerName, originalPrompt, accumulatedContent, startTime, eventSequence, } = params;
-        logger.debug("[NeuroLink.stream] Preparing to store conversation turn in memory", {
-            options: JSON.stringify(enhancedOptions),
-            sessionId: enhancedOptions.context
-                ?.sessionId,
-        });
+        // Logger Guard: the full options object (history + tool outputs) can be
+        // enormous — an eager JSON.stringify here threw RangeError: Invalid
+        // string length in production and killed the turn. Serialize lazily,
+        // bounded, and only when debug is actually enabled; sanitizeRecord
+        // redacts credentials/PII keys before anything reaches the sink.
+        if (logger.shouldLog("debug")) {
+            logger.debug("[NeuroLink.stream] Preparing to store conversation turn in memory", {
+                options: safeDebugSerialize(sanitizeRecord(enhancedOptions)),
+                sessionId: enhancedOptions.context
+                    ?.sessionId,
+            });
+        }
         // Guard: skip storing if no meaningful content was produced (no text AND no tool activity)
         const hasToolEvents = eventSequence.some((e) => e.type === "tool:start" || e.type === "tool:end");
         if (!accumulatedContent.trim() && !hasToolEvents) {
@@ -7234,12 +7255,16 @@ Current user's request: ${currentInput}`;
             });
             return;
         }
-        logger.debug("[NeuroLink.stream] Storing conversation turn in memory", {
-            options: JSON.stringify(enhancedOptions),
-            sessionId: enhancedOptions.context
-                ?.sessionId,
-            conversationMemoryExists: this.conversationMemory ? true : false,
-        });
+        // Logger Guard: see the matching block above — never eagerly stringify
+        // the full options object, and always redact before serializing.
+        if (logger.shouldLog("debug")) {
+            logger.debug("[NeuroLink.stream] Storing conversation turn in memory", {
+                options: safeDebugSerialize(sanitizeRecord(enhancedOptions)),
+                sessionId: enhancedOptions.context
+                    ?.sessionId,
+                conversationMemoryExists: this.conversationMemory ? true : false,
+            });
+        }
         // Store memory after stream consumption is complete
         if (this.conversationMemory && enhancedOptions.context?.sessionId) {
             const sessionId = enhancedOptions.context
@@ -8703,11 +8728,7 @@ Current user's request: ${currentInput}`;
             : this.getCustomTools().has(toolName)
                 ? "custom"
                 : "external";
-        const inputStr = typeof params === "string"
-            ? params
-            : params
-                ? JSON.stringify(params)
-                : "";
+        const inputStr = params ? stringifyContentSafe(params) : "";
         const executionStartTime = Date.now();
         // Per-invocation id so consumers can correlate a tool:start with its matching
         // tool:end even when the same tool runs multiple times concurrently.
@@ -10271,8 +10292,15 @@ Current user's request: ${currentInput}`;
                 });
             }
             else {
+                // Single ERROR record for a failed registration — the inner layers
+                // (client factory, server manager) log at debug so one root cause no
+                // longer fans out into 5 ERROR lines. Carry enough context here to
+                // diagnose without the inner lines.
                 mcpLogger.error(`[NeuroLink] Failed to add external MCP server: ${serverId}`, {
                     error: result.error,
+                    transport: config.transport,
+                    command: config.command,
+                    url: config.url ? redactUrlCredentials(config.url) : undefined,
                 });
             }
             return result;
@@ -10304,6 +10332,11 @@ Current user's request: ${currentInput}`;
                     timestamp: Date.now(),
                 });
             }
+            else if (result.error?.includes("not found")) {
+                // Expected no-op: consumers commonly call remove as cleanup after a
+                // failed add, when the server never registered. Not an error.
+                mcpLogger.debug(`[NeuroLink] Remove skipped — external MCP server not registered: ${serverId}`);
+            }
             else {
                 mcpLogger.error(`[NeuroLink] Failed to remove external MCP server: ${serverId}`, {
                     error: result.error,

package/dist/providers/googleVertex.d.ts

@@ -230,6 +230,17 @@ export declare class GoogleVertexProvider extends BaseProvider {
      * callbacks invoked from here.
      */
     private wrapStreamResultWithLifecycle;
+    /**
+     * Re-apply getter-based accessor properties from a source StreamResult onto
+     * a wrapper copy. Wrapper spreads (`{ ...result }`) invoke and SNAPSHOT
+     * enumerable getters at wrap time — for background-loop streams (the native
+     * Anthropic path) that resolve finishReason / structuredOutput / toolCalls
+     * only as the consumer drains, the snapshot is permanently undefined/empty.
+     * Copying the accessor descriptors keeps the wrapped result live. Results
+     * built from plain data properties (the buffered Gemini paths) have no
+     * getters and pass through untouched.
+     */
+    private preserveStreamResultAccessors;
     /**
      * Attach `gen_ai.usage.*` and `neurolink.cost` attributes to a span.
      * Pulled out so the generate / stream / image-gen paths share one