@juspay/neurolink 10.8.13 → 10.8.15

package/dist/processors/media/VideoProcessor.js

@@ -55,39 +55,48 @@ import { SIZE_LIMITS_MB } from "../config/index.js";
 import { FileErrorCode } from "../errors/index.js";
 import { tracers, ATTR, withSpan } from "../../telemetry/index.js";
 import { logger } from "../../utils/logger.js";
+import { tryImport } from "../../utils/tryImport.js";
+/**
+ * Narrow a loaded `fluent-ffmpeg` export to the shape this file actually uses:
+ * a callable carrying the `ffprobe` and `setFfmpegPath` statics.
+ *
+ * `tryImport` proves only that the package RESOLVES. Callers invoke the export
+ * and reach straight for its statics, so a package whose shape changed (ESM
+ * rewrite, major bump, a shim in node_modules) would otherwise surface as
+ * "Cannot read properties of undefined (reading 'ffprobe')" from inside
+ * probeVideo — blaming the call site instead of the package that is wrong.
+ */
+export function assertFluentFfmpegShape(mod) {
+    // Not `Partial<typeof import("fluent-ffmpeg")>`: Partial maps over properties
+    // and drops the call signature, so `typeof x === "function"` would narrow the
+    // result to `never`. Probe the statics structurally instead.
+    const statics = mod;
+    if (typeof mod !== "function" ||
+        typeof statics.ffprobe !== "function" ||
+        typeof statics.setFfmpegPath !== "function") {
+        throw new Error(`The installed "fluent-ffmpeg" package does not export a callable with ` +
+            `ffprobe and setFfmpegPath statics (got ${typeof mod}). ` +
+            `Reinstall a compatible version:\n  pnpm add fluent-ffmpeg`);
+    }
+}
 // fluent-ffmpeg's default export is callable + has static methods — avoid caching
 // the module type (it confuses TS); Node's module cache handles dedup.
 async function loadFluentFfmpeg() {
-    try {
-        const mod = await import(/* @vite-ignore */ "fluent-ffmpeg");
-        return mod.default;
-    }
-    catch (err) {
-        const e = err instanceof Error ? err : null;
-        if (e?.code === "ERR_MODULE_NOT_FOUND" &&
-            e.message.includes("fluent-ffmpeg")) {
-            throw new Error('Video processing requires the "fluent-ffmpeg" package. Install it with:\n  pnpm add fluent-ffmpeg', { cause: err });
-        }
-        throw err;
-    }
+    // fluent-ffmpeg is CJS (`export =`), so `typeof import(...)` describes the
+    // callable itself and carries no `default`. Under Node ESM the namespace
+    // still wraps it, so ask for that shape explicitly.
+    const mod = await tryImport("fluent-ffmpeg", "Video processing");
+    const ffmpeg = mod.default;
+    assertFluentFfmpegShape(ffmpeg);
+    return ffmpeg;
 }
 let _mediabunny = null;
 async function loadMediaBunny() {
     if (_mediabunny) {
         return _mediabunny;
     }
-    try {
-        _mediabunny = await import(/* @vite-ignore */ "mediabunny");
-        return _mediabunny;
-    }
-    catch (err) {
-        const e = err instanceof Error ? err : null;
-        if (e?.code === "ERR_MODULE_NOT_FOUND" &&
-            e.message.includes("mediabunny")) {
-            throw new Error('Video processing requires the "mediabunny" package. Install it with:\n  pnpm add mediabunny', { cause: err });
-        }
-        throw err;
-    }
+    _mediabunny = await tryImport("mediabunny", "Video processing");
+    return _mediabunny;
 }
 // =============================================================================
 // FFMPEG PATH INITIALIZATION

package/dist/providers/sagemaker/client.js

@@ -6,6 +6,7 @@
  */
 import { handleSageMakerError, SageMakerError, isRetryableError, getRetryDelay, } from "./errors.js";
 import { logger } from "../../utils/logger.js";
+import { tryImport } from "../../utils/tryImport.js";
 /**
  * Lazily load `@aws-sdk/client-sagemaker-runtime`.
  *
@@ -16,17 +17,7 @@ import { logger } from "../../utils/logger.js";
  * error instead of a raw resolution failure.
  */
 async function loadSageMakerRuntime() {
-    try {
-        return await import(/* @vite-ignore */ "@aws-sdk/client-sagemaker-runtime");
-    }
-    catch (err) {
-        const e = err instanceof Error ? err : null;
-        if (e?.code === "ERR_MODULE_NOT_FOUND" &&
-            e.message.includes("client-sagemaker-runtime")) {
-            throw new Error('SageMaker inference requires "@aws-sdk/client-sagemaker-runtime". Install it with:\n  pnpm add @aws-sdk/client-sagemaker-runtime', { cause: err });
-        }
-        throw err;
-    }
+    return tryImport("@aws-sdk/client-sagemaker-runtime", "SageMaker inference");
 }
 /**
  * Enhanced SageMaker Runtime client with retry logic and error handling

package/dist/server/voice/voiceWebSocketHandler.js

@@ -5,20 +5,11 @@ import { timingSafeEqualString } from "./tokenCompare.js";
 import { CartesiaStream } from "../../adapters/tts/cartesiaHandler.js";
 import { NeuroLink } from "../../neurolink.js";
 import { logger } from "../../utils/logger.js";
+import { tryImport } from "../../utils/tryImport.js";
 import { withTimeout } from "../../utils/async/withTimeout.js";
 async function loadCobra(accessKey) {
-    try {
-        const mod = (await import(/* @vite-ignore */ "@picovoice/cobra-node"));
-        return new mod.Cobra(accessKey);
-    }
-    catch (err) {
-        const e = err instanceof Error ? err : null;
-        if (e?.code === "ERR_MODULE_NOT_FOUND" &&
-            e.message.includes("cobra-node")) {
-            throw new Error('Voice activity detection requires "@picovoice/cobra-node". Install it with:\n  pnpm add @picovoice/cobra-node', { cause: err });
-        }
-        throw err;
-    }
+    const mod = await tryImport("@picovoice/cobra-node", "Voice activity detection");
+    return new mod.Cobra(accessKey);
 }
 const SONIOX_URL = process.env.SONIOX_WS_URL ?? "wss://stt-rt.soniox.com/transcribe-websocket";
 function getRequiredEnv(name) {

package/dist/services/server/ai/observability/instrumentation.js

@@ -20,6 +20,7 @@ import { ATTR_SERVICE_NAME, ATTR_SERVICE_VERSION, } from "@opentelemetry/semanti
 import { AsyncLocalStorage } from "async_hooks";
 import { extractMcpErrorText } from "../../../../utils/mcpErrorText.js";
 import { logger } from "../../../../utils/logger.js";
+import { tryImport } from "../../../../utils/tryImport.js";
 import { LANGFUSE_ATTR } from "../../../../telemetry/attributes.js";
 const LOG_PREFIX = "[OpenTelemetry]";
 function createOtelResource(config, serviceName) {
@@ -568,17 +569,7 @@ class ContextEnricher {
     }
 }
 async function createLangfuseProcessor(config) {
-    let mod;
-    try {
-        mod = await import(/* @vite-ignore */ "@langfuse/otel");
-    }
-    catch (err) {
-        const e = err instanceof Error ? err : null;
-        if (e?.code === "ERR_MODULE_NOT_FOUND" && e.message.includes("langfuse")) {
-            throw new Error('Langfuse observability requires "@langfuse/otel". Install it with:\n  pnpm add @langfuse/otel', { cause: err });
-        }
-        throw err;
-    }
+    const mod = await tryImport("@langfuse/otel", "Langfuse observability");
     return new mod.LangfuseSpanProcessor({
         publicKey: config.publicKey,
         secretKey: config.secretKey,

package/dist/tasks/backends/bullmqBackend.js

@@ -7,19 +7,11 @@
  * - Survives process restarts (Redis-persisted)
  */
 import { logger } from "../../utils/logger.js";
+import { tryImport } from "../../utils/tryImport.js";
 import { TaskError } from "../errors.js";
 import { TASK_DEFAULTS, } from "../../types/index.js";
 async function loadBullMQ() {
-    try {
-        return await import(/* @vite-ignore */ "bullmq");
-    }
-    catch (err) {
-        const e = err instanceof Error ? err : null;
-        if (e?.code === "ERR_MODULE_NOT_FOUND" && e.message.includes("bullmq")) {
-            throw new Error('BullMQ task backend requires the "bullmq" package. Install it with:\n  pnpm add bullmq', { cause: err });
-        }
-        throw err;
-    }
+    return tryImport("bullmq", "BullMQ task backend");
 }
 const QUEUE_NAME = "neurolink-tasks";
 export class BullMQBackend {

package/dist/utils/tryImport.d.ts

@@ -0,0 +1,35 @@
+/**
+ * Dynamically import an optional dependency, converting a missing package into
+ * an actionable error.
+ *
+ * NeuroLink declares 42 `optionalDependencies`, and each loader had grown its
+ * own copy of the same try/catch: check `err.code`, substring-match the package
+ * name, rethrow a friendlier Error. The copies had drifted — some said
+ * `pnpm add`, one said `npm install`, and the phrasing varied per call site.
+ *
+ * Only missing-module failures are intercepted. A package that is installed but
+ * throws while evaluating (syntax error, ESM/CJS interop, a broken transitive
+ * dep) rethrows unchanged — turning those into "please install X" would send
+ * the caller after a package they already have. The same applies to specifiers
+ * that are not installable packages at all (relative paths, file/data URLs):
+ * they rethrow untouched rather than producing an unrunnable install command.
+ *
+ * @param pkg - Module specifier. The install hint is only produced for bare
+ *   package names; anything else rethrows the loader's own error.
+ * @param feature - Human-readable capability name, used to open the message.
+ * @returns The imported module namespace, typed as `T` (`unknown` if the
+ *   caller does not supply a type argument, since the shape cannot be known
+ *   for a dynamic specifier).
+ * @throws The loader's original error, or an `Error` naming the package and
+ *   the `pnpm add` command when a bare package is genuinely absent. The
+ *   original failure is always preserved on `cause`.
+ *
+ * @example
+ * ```ts
+ * const ExcelJS = await tryImport<typeof import("exceljs")>(
+ *   "exceljs",
+ *   "Excel file processing",
+ * );
+ * ```
+ */
+export declare function tryImport<T = unknown>(pkg: string, feature: string): Promise<T>;

package/dist/utils/tryImport.js

@@ -0,0 +1,94 @@
+/**
+ * Dynamically import an optional dependency, converting a missing package into
+ * an actionable error.
+ *
+ * NeuroLink declares 42 `optionalDependencies`, and each loader had grown its
+ * own copy of the same try/catch: check `err.code`, substring-match the package
+ * name, rethrow a friendlier Error. The copies had drifted — some said
+ * `pnpm add`, one said `npm install`, and the phrasing varied per call site.
+ *
+ * Only missing-module failures are intercepted. A package that is installed but
+ * throws while evaluating (syntax error, ESM/CJS interop, a broken transitive
+ * dep) rethrows unchanged — turning those into "please install X" would send
+ * the caller after a package they already have. The same applies to specifiers
+ * that are not installable packages at all (relative paths, file/data URLs):
+ * they rethrow untouched rather than producing an unrunnable install command.
+ *
+ * @param pkg - Module specifier. The install hint is only produced for bare
+ *   package names; anything else rethrows the loader's own error.
+ * @param feature - Human-readable capability name, used to open the message.
+ * @returns The imported module namespace, typed as `T` (`unknown` if the
+ *   caller does not supply a type argument, since the shape cannot be known
+ *   for a dynamic specifier).
+ * @throws The loader's original error, or an `Error` naming the package and
+ *   the `pnpm add` command when a bare package is genuinely absent. The
+ *   original failure is always preserved on `cause`.
+ *
+ * @example
+ * ```ts
+ * const ExcelJS = await tryImport<typeof import("exceljs")>(
+ *   "exceljs",
+ *   "Excel file processing",
+ * );
+ * ```
+ */
+export async function tryImport(pkg, feature) {
+    try {
+        return (await import(/* @vite-ignore */ pkg));
+    }
+    catch (err) {
+        if (isMissingModule(err, pkg)) {
+            throw new Error(`${feature} requires the "${pkg}" package. Install it with:\n  pnpm add ${pkg}`, { cause: err });
+        }
+        throw err;
+    }
+}
+/**
+ * True only when `err` is the module loader failing to resolve `pkg` itself.
+ *
+ * The package name is matched inside quotes because that is how both loaders
+ * format it — ESM `Cannot find package 'x' imported from …`, CJS
+ * `Cannot find module 'x'`. Matching the bare substring instead (as the
+ * hand-rolled copies did) also matches a *transitive* dependency whose path
+ * happens to contain the name, which would blame the wrong package.
+ *
+ * The specifier must also be a bare package name. Without that check a missing
+ * relative path or file URL is rewritten into `pnpm add ./fixtures/thing.mjs`,
+ * which is not a command anyone can run — and this helper is already called
+ * with `file:` and `data:` URLs, so that path is reachable rather than
+ * theoretical.
+ */
+function isMissingModule(err, pkg) {
+    if (!(err instanceof Error)) {
+        return false;
+    }
+    if (!isBarePackageSpecifier(pkg)) {
+        return false;
+    }
+    const code = err.code;
+    if (code !== "ERR_MODULE_NOT_FOUND" && code !== "MODULE_NOT_FOUND") {
+        return false;
+    }
+    return err.message.includes(`'${pkg}'`) || err.message.includes(`"${pkg}"`);
+}
+/**
+ * True for specifiers that name an installable package — `exceljs`,
+ * `@scope/pkg`, `pkg/sub/path`.
+ *
+ * False for the two things an install hint cannot help with: filesystem paths
+ * (`./x`, `../x`, `/x`, and Windows `C:\x`, which the scheme test below also
+ * rejects) and any URL (`file:`, `data:`, `http:`), including the `node:`
+ * builtins, which are never installed.
+ */
+function isBarePackageSpecifier(specifier) {
+    if (specifier.length === 0) {
+        return false;
+    }
+    if (specifier.startsWith("/") ||
+        specifier.startsWith("./") ||
+        specifier.startsWith("../")) {
+        return false;
+    }
+    // RFC 3986 scheme: ALPHA *( ALPHA / DIGIT / "+" / "-" / "." ) ":"
+    return !/^[a-zA-Z][a-zA-Z\d+\-.]*:/.test(specifier);
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@juspay/neurolink",
-  "version": "10.8.13",
+  "version": "10.8.15",
   "packageManager": "pnpm@10.15.1",
   "description": "TypeScript AI SDK with 24+ LLM providers behind one consistent API. MCP-native (connect any MCP server), voice TTS/STT/realtime, RAG, agents, memory, context compaction. OpenAI · Anthropic · Gemini · Bedrock · Azure · Ollama · DeepSeek · NVIDIA NIM and more.",
   "author": {
@@ -144,8 +144,8 @@
     "test:tool-dedup": "pnpm run test:tool-dedup:vitest && npx tsx test/continuous-test-suite-tool-dedup.ts",
     "test:model-pool:vitest": "pnpm exec vitest run test/modelPool.test.ts",
     "test:model-pool": "pnpm run test:model-pool:vitest && npx tsx test/continuous-test-suite-model-pool.ts",
-    "test:model-capabilities:vitest": "pnpm exec vitest run test/modelCapabilities.test.ts",
-    "test:agent-runtime:vitest": "pnpm exec vitest run test/agentRuntime.test.ts test/agentDelegation.test.ts test/agentPlumbing.test.ts test/toolExecutionRecorder.test.ts test/samplingParams.test.ts test/structuredRecovery.test.ts",
+    "test:model-capabilities": "npx tsx test/continuous-test-suite-model-capabilities.ts",
+    "test:agent-runtime:vitest": "pnpm exec vitest run test/agentRuntime.test.ts test/agentDelegation.test.ts test/agentPlumbing.test.ts test/toolExecutionRecorder.test.ts",
     "test:retry-after:vitest": "pnpm exec vitest run test/retryAfter.test.ts",
     "test:ci": "pnpm run test && pnpm run test:client && pnpm run test:hitl",
     "// CI tier — fast, no live AI calls, safe for every commit": "",
@@ -155,7 +155,7 @@
     "test:system-messages:vitest": "pnpm exec vitest run test/systemMessages.test.ts",
     "test:tool-routing-semantic:vitest": "pnpm exec vitest run test/toolRoutingSemantic.test.ts",
     "test:tool-routing-semantic": "pnpm run test:tool-routing-semantic:vitest && npx tsx test/continuous-test-suite-tool-routing-semantic.ts",
-    "test:unit": "pnpm run test:envguard && pnpm run test:bugfixes && pnpm run test:file-detector-extension && pnpm run test:file-detector-magic-bytes && pnpm run test:mcp:infra && pnpm run test:mcp:bash && pnpm run test:mcp:limits && pnpm run test:mcp:spans && pnpm run test:autoresearch:redis && pnpm run test:unit:vitest && pnpm run test:tool-routing-cli:vitest && pnpm run test:tool-dedup:vitest && pnpm run test:model-pool:vitest && pnpm run test:litellm-context:vitest && pnpm run test:step-budget-guard:vitest && pnpm run test:system-messages:vitest && pnpm run test:tool-routing-semantic:vitest && pnpm run test:anthropic-tools-policy && pnpm run test:sagemaker-tools && pnpm run test:anthropic-multimodal && pnpm run test:excel-interop && pnpm run test:model-capabilities:vitest && pnpm run test:agent-runtime:vitest && pnpm run test:retry-after:vitest",
+    "test:unit": "pnpm run test:envguard && pnpm run test:bugfixes && pnpm run test:file-detector-extension && pnpm run test:file-detector-magic-bytes && pnpm run test:mcp:infra && pnpm run test:mcp:bash && pnpm run test:mcp:limits && pnpm run test:mcp:spans && pnpm run test:autoresearch:redis && pnpm run test:unit:vitest && pnpm run test:tool-routing-cli:vitest && pnpm run test:tool-dedup:vitest && pnpm run test:model-pool:vitest && pnpm run test:litellm-context:vitest && pnpm run test:step-budget-guard:vitest && pnpm run test:system-messages:vitest && pnpm run test:tool-routing-semantic:vitest && pnpm run test:anthropic-tools-policy && pnpm run test:sagemaker-tools && pnpm run test:anthropic-multimodal && pnpm run test:excel-interop && pnpm run test:model-capabilities && pnpm run test:agent-runtime:vitest && pnpm run test:retry-after:vitest && pnpm run test:sampling-params && pnpm run test:structured-recovery && pnpm run test:prompt-redaction && pnpm run test:mcp-result-cache",
     "// CI tier — live providers, runs only when API keys are present (test:credentials and test:dynamic make real provider calls when keys are set, so they live here, not in test:unit)": "",
     "test:live": "pnpm run test:providers && pnpm run test:mcp:http && pnpm run test:mcp:sdk && pnpm run test:mcp:cli && pnpm run test:observability && pnpm run test:context && pnpm run test:memory && pnpm run test:tool-reliability && pnpm run test:evaluation && pnpm run test:autoresearch && pnpm run test:credentials && pnpm run test:dynamic",
     "// CI tier — product output (image/video/TTS/PPT) — costs $$ per run": "",
@@ -224,7 +224,11 @@
     "test:tts:unit": "npx tsx test/continuous-test-suite-tts-unit.ts",
     "test:video": "npx tsx test/continuous-test-suite-video.ts",
     "test:multimodal": "pnpm run test:audio && pnpm run test:video && pnpm run test:office && pnpm run test:tts:unit && pnpm run test:multimodal:sdk",
-    "test:multimodal:sdk": "npx tsx test/continuous-test-suite-multimodal-sdk.ts"
+    "test:multimodal:sdk": "npx tsx test/continuous-test-suite-multimodal-sdk.ts",
+    "test:sampling-params": "npx tsx test/continuous-test-suite-sampling-params.ts",
+    "test:structured-recovery": "npx tsx test/continuous-test-suite-structured-recovery.ts",
+    "test:prompt-redaction": "npx tsx test/continuous-test-suite-prompt-redaction.ts",
+    "test:mcp-result-cache": "npx tsx test/continuous-test-suite-mcp-result-cache.ts"
   },
   "files": [
     "dist",