@juspay/neurolink 10.12.8 → 11.0.0

package/dist/core/modules/GenerationHandler.js

@@ -25,7 +25,7 @@ import { calculateCacheSavingsPercent, extractCacheCreationTokens, extractCacheR
 import { DEFAULT_MAX_STEPS, DEFAULT_WRAPUP_TIME_LEAD_MS, } from "../constants.js";
 import { createStepBudgetGuard, estimateFixedOverheadTokens, } from "../../context/stepBudgetGuard.js";
 import { isTemperatureDeprecatedError, isSchemaComplexityError, isToolsSchemaConflictError, isToolsSchemaExclusionInForce, } from "./structuredOutputPolicy.js";
-import { coerceJsonToSchema } from "../../utils/json/coerce.js";
+import { coerceJsonToSchema, recoverScalarRoot, schemaAccepts, } from "../../utils/json/coerce.js";
 import { convertZodToJsonSchema } from "../../utils/schemaConversion.js";
 import { NoObjectGeneratedError } from "../../utils/generationErrors.js";
 import { Output, stepCountIs } from "../../utils/tool.js";
@@ -247,8 +247,8 @@ export class GenerationHandler {
         // JSON Schema to the provider instead: it appends an additive
         // `final_result` tool and returns the answer as that tool's arguments,
         // keeping the real tools callable. Bedrock is deliberately excluded — it
-        // runs on the third-party @ai-sdk/amazon-bedrock model, which has no such
-        // handling.
+        // talks to the raw AWS SDK directly, not an ai-sdk provider package, and
+        // has no such handling.
         const finalResultSchema = this.providerName === "anthropic" &&
             !!options.schema &&
             shouldUseTools &&
@@ -800,26 +800,35 @@ export class GenerationHandler {
                 }
                 return coerced.content;
             }
-            try {
-                const scalar = JSON.parse(strippedText);
-                if (scalar === "") {
+            const scalar = recoverScalarRoot(strippedText, options.schema);
+            switch (scalar.kind) {
+                case "empty":
                     // A JSON-encoded empty string is an EMPTY completion, not a
                     // recovered scalar — normalize to a true empty ('' content, no
                     // structuredData) so callers' empty-response handling fires
                     // instead of a literal '""' reaching the user.
                     logger.warn("[GenerationHandler] schema requested but the model returned an empty JSON string; normalizing to empty content", { provider: this.providerName, model: this.modelName });
                     return "";
-                }
-                if (scalar !== null && scalar !== undefined) {
-                    structuredData = scalar;
+                case "accepted":
+                    // A JSON scalar root is only real structured data when the caller's
+                    // schema actually accepts it. Under an OBJECT schema a recovered
+                    // string/number is the raw completion in disguise (the shape a
+                    // truncated response degrades to) — publishing it would hand the
+                    // caller a `structuredData` that violates the schema they passed.
+                    structuredData = scalar.value;
+                    return strippedText;
+                case "rejected":
+                    logger.warn("[GenerationHandler] recovered a JSON scalar the requested schema rejects; leaving structuredData unset", {
+                        provider: this.providerName,
+                        model: this.modelName,
+                        scalarType: typeof scalar.value,
+                    });
+                    return strippedText;
+                case "nullish":
+                case "not-json":
+                    logger.warn("[GenerationHandler] schema requested but no JSON could be recovered from model text; returning raw text", { provider: this.providerName, model: this.modelName });
                     return strippedText;
-                }
-            }
-            catch {
-                // not JSON at all — fall through to raw text + WARN
             }
-            logger.warn("[GenerationHandler] schema requested but no JSON could be recovered from model text; returning raw text", { provider: this.providerName, model: this.modelName });
-            return strippedText;
         };
         if (useStructuredOutput) {
             try {
@@ -834,7 +843,20 @@ export class GenerationHandler {
                 // (a string identical to the step text) and coerce it instead.
                 const rawTextEcho = typeof experimentalOutput === "string" &&
                     experimentalOutput === (generateResult.text ?? "");
-                if (experimentalOutput !== undefined && !rawTextEcho) {
+                // The equality check above only catches an EXACT echo. On a multi-step
+                // or truncated turn the echo can differ from `text` (a different step's
+                // text, a fence, trailing whitespace), and a raw string would then be
+                // published as `structuredData` under an object schema — the "returned
+                // a string instead of the schema object" failure. A string is trusted
+                // as structured output ONLY when the caller's schema accepts it
+                // (string-root schemas keep working); otherwise it is coerced like any
+                // other raw model text.
+                const untrustedStringOutput = typeof experimentalOutput === "string" &&
+                    !!options.schema &&
+                    !schemaAccepts(options.schema, experimentalOutput);
+                if (experimentalOutput !== undefined &&
+                    !rawTextEcho &&
+                    !untrustedStringOutput) {
                     // AI-SDK already parsed + schema-validated the object. Expose it
                     // directly and serialise canonically — no hand-parsing needed.
                     structuredData = experimentalOutput;

package/dist/core/modules/structuredOutputPolicy.d.ts

@@ -29,8 +29,8 @@ export declare function isGeminiProvider(providerName: string, modelName: string
  * `providerOptions.anthropic.finalResultSchema`, and the provider appends an
  * additive `final_result` tool (see providers/anthropic/structuredOutput.ts) —
  * schema enforcement without giving up tool calling. "bedrock" has no such
- * handling (it runs on the third-party @ai-sdk/amazon-bedrock model) and still
- * falls back to text-mode coercion.
+ * handling (it talks to the raw AWS SDK directly, not an ai-sdk provider
+ * package) and still falls back to text-mode coercion.
  */
 export declare function isNativeAnthropicProvider(providerName: string): boolean;
 /**

package/dist/core/modules/structuredOutputPolicy.js

@@ -39,8 +39,8 @@ export function isGeminiProvider(providerName, modelName) {
  * `providerOptions.anthropic.finalResultSchema`, and the provider appends an
  * additive `final_result` tool (see providers/anthropic/structuredOutput.ts) —
  * schema enforcement without giving up tool calling. "bedrock" has no such
- * handling (it runs on the third-party @ai-sdk/amazon-bedrock model) and still
- * falls back to text-mode coercion.
+ * handling (it talks to the raw AWS SDK directly, not an ai-sdk provider
+ * package) and still falls back to text-mode coercion.
  */
 export function isNativeAnthropicProvider(providerName) {
     return providerName === "anthropic" || providerName === "bedrock";

package/dist/factories/providerFactory.d.ts

@@ -1,6 +1,21 @@
 import type { NeuroLink } from "../neurolink.js";
 import type { AIProviderName } from "../constants/enums.js";
 import type { AIProvider, NeurolinkCredentials, ProviderConstructor, ProviderRegistration } from "../types/index.js";
+/**
+ * Maps registered provider names to NeurolinkCredentials keys. Most names
+ * match (openai, anthropic, vertex, bedrock, etc.) but every kebab-case
+ * provider whose canonical credentials key is camelCase — or otherwise
+ * spelled differently — MUST be mapped here, otherwise per-call/instance
+ * credential overrides silently get dropped (createProvider() looks up
+ * credentials["lm-studio"], which is undefined, while the caller wrote
+ * credentials.lmStudio).
+ */
+export declare const CREDENTIAL_KEY_MAP: Record<string, string>;
+/**
+ * Resolve a registered provider name to its NeurolinkCredentials key.
+ * Falls back to the provider name itself when no remapping is needed.
+ */
+export declare function resolveCredentialKey(providerName: string): string;
 /**
  * True Factory Pattern implementation for AI Providers
  * Uses registration-based approach to eliminate switch statements

package/dist/factories/providerFactory.js

@@ -1,6 +1,30 @@
 import { logger } from "../utils/logger.js";
 // Pure factory pattern with no hardcoded imports
 // All providers loaded dynamically via registry to avoid circular dependencies
+/**
+ * Maps registered provider names to NeurolinkCredentials keys. Most names
+ * match (openai, anthropic, vertex, bedrock, etc.) but every kebab-case
+ * provider whose canonical credentials key is camelCase — or otherwise
+ * spelled differently — MUST be mapped here, otherwise per-call/instance
+ * credential overrides silently get dropped (createProvider() looks up
+ * credentials["lm-studio"], which is undefined, while the caller wrote
+ * credentials.lmStudio).
+ */
+export const CREDENTIAL_KEY_MAP = {
+    "google-ai": "googleAiStudio",
+    "openai-compatible": "openaiCompatible",
+    huggingface: "huggingFace",
+    "lm-studio": "lmStudio",
+    "nvidia-nim": "nvidiaNim",
+    "together-ai": "together",
+};
+/**
+ * Resolve a registered provider name to its NeurolinkCredentials key.
+ * Falls back to the provider name itself when no remapping is needed.
+ */
+export function resolveCredentialKey(providerName) {
+    return (CREDENTIAL_KEY_MAP[providerName.toLowerCase()] ?? providerName.toLowerCase());
+}
 /**
  * True Factory Pattern implementation for AI Providers
  * Uses registration-based approach to eliminate switch statements
@@ -58,21 +82,7 @@ export class ProviderFactory {
             // Fallback to registry default if no env var
             model = model || registration.defaultModel;
         }
-        // Map registered provider names to NeurolinkCredentials keys.
-        // Most names match (openai, anthropic, vertex, bedrock, etc.)
-        // but every kebab-case provider whose canonical credentials key is
-        // camelCase MUST be mapped here — otherwise per-call credential
-        // overrides silently get dropped (the factory looks up
-        // credentials["lm-studio"] which is undefined while the user wrote
-        // credentials.lmStudio).
-        const credentialKeyMap = {
-            "google-ai": "googleAiStudio",
-            "openai-compatible": "openaiCompatible",
-            huggingface: "huggingFace",
-            "lm-studio": "lmStudio",
-            "nvidia-nim": "nvidiaNim",
-        };
-        const credKey = credentialKeyMap[normalizedName] ?? normalizedName;
+        const credKey = resolveCredentialKey(normalizedName);
         // Extract provider-scoped credential slice (e.g. credentials.openai for OpenAI)
         const scopedCredentials = credentials
             ? credentials[credKey]
@@ -81,30 +91,8 @@ export class ProviderFactory {
             if (typeof registration.constructor !== "function") {
                 throw new Error(`Invalid constructor for provider ${providerName}: not a function`);
             }
-            let result;
-            try {
-                const factoryResult = registration.constructor(model, resolvedProviderName, sdk, region, scopedCredentials);
-                // Handle both sync and async results
-                result =
-                    factoryResult instanceof Promise
-                        ? await factoryResult
-                        : factoryResult;
-            }
-            catch (factoryError) {
-                if (registration.constructor.prototype &&
-                    registration.constructor.prototype.constructor ===
-                        registration.constructor) {
-                    try {
-                        result = new registration.constructor(model, resolvedProviderName, sdk, region, scopedCredentials);
-                    }
-                    catch (constructorError) {
-                        throw new Error(`Both factory function and constructor failed. Factory error: ${factoryError}. Constructor error: ${constructorError}`, { cause: constructorError });
-                    }
-                }
-                else {
-                    throw factoryError;
-                }
-            }
+            const factoryResult = registration.constructor(model, resolvedProviderName, sdk, region, scopedCredentials);
+            const result = factoryResult instanceof Promise ? await factoryResult : factoryResult;
             return result;
         }
         catch (error) {

package/dist/factories/providerRegistry.js

@@ -104,10 +104,10 @@ export class ProviderRegistry {
                 return new GoogleVertexProvider(modelName, providerName, sdk, region, vertexCreds);
             }, VertexModels.CLAUDE_4_6_SONNET, ["vertex", "googleVertex"]);
             // Register Hugging Face provider (Unified Router implementation)
-            ProviderFactory.registerProvider(AIProviderName.HUGGINGFACE, async (modelName, _providerName, _sdk, _region, credentials) => {
+            ProviderFactory.registerProvider(AIProviderName.HUGGINGFACE, async (modelName, _providerName, sdk, region, credentials) => {
                 const hfCreds = credentials;
                 const { HuggingFaceProvider } = await import("../providers/huggingFace/index.js");
-                return new HuggingFaceProvider(modelName, undefined, hfCreds);
+                return new HuggingFaceProvider(modelName, sdk, region, hfCreds);
             }, process.env.HUGGINGFACE_MODEL ||
                 HuggingFaceModels.QWEN_2_5_72B_INSTRUCT, ["huggingface", "hf"]);
             // Register Mistral AI provider

package/dist/lib/core/baseProvider.js

@@ -1,6 +1,6 @@
 import { context, SpanKind, SpanStatusCode, trace } from "@opentelemetry/api";
 import { directAgentTools } from "../agent/directTools.js";
-import { IMAGE_GENERATION_MODELS } from "../core/constants.js";
+import { isImageGenerationModel } from "../core/constants.js";
 import { MiddlewareFactory } from "../middleware/factory.js";
 import { modelSupports } from "../models/modelRegistry.js";
 import { ATTR, tracers } from "../telemetry/index.js";
@@ -195,7 +195,7 @@ export class BaseProvider {
         // Skip this path when the caller explicitly requests non-image output (e.g.
         // JSON analysis) so dual-mode models like gemini-3.1-flash-image-preview
         // can still perform text/structured generation.
-        const isImageModel = IMAGE_GENERATION_MODELS.some((m) => this.modelName.includes(m));
+        const isImageModel = isImageGenerationModel(this.modelName);
         const requestsNonImageOutput = options.output?.format === "json" ||
             options.output?.format === "structured" ||
             options.output?.format === "text";
@@ -966,7 +966,7 @@ export class BaseProvider {
             if (options.output?.mode === "video") {
                 return await this.handleVideoGeneration(options, startTime);
             }
-            const isImageModel = IMAGE_GENERATION_MODELS.some((m) => this.modelName.includes(m));
+            const isImageModel = isImageGenerationModel(this.modelName);
             const requestsNonImageOutput = options.output?.format === "json" ||
                 options.output?.format === "structured" ||
                 options.output?.format === "text";

package/dist/lib/core/dynamicModels.d.ts

@@ -1,4 +1,4 @@
-import type { DynamicModelConfig as ModelConfig } from "../types/index.js";
+import { type DynamicModelConfig as ModelConfig } from "../types/index.js";
 /**
  * Dynamic Model Provider
  * Loads and manages model configurations from external sources

package/dist/lib/core/dynamicModels.js

@@ -1,28 +1,6 @@
-import { z } from "zod";
 import { isAbortError } from "../utils/errorHandling.js";
 import { logger } from "../utils/logger.js";
-/**
- * Model configuration schema for validation
- */
-const ModelConfigSchema = z.object({
-    id: z.string(),
-    displayName: z.string(),
-    capabilities: z.array(z.string()),
-    deprecated: z.boolean(),
-    pricing: z.object({
-        input: z.number(),
-        output: z.number(),
-    }),
-    contextWindow: z.number(),
-    releaseDate: z.string(),
-});
-const ModelRegistrySchema = z.object({
-    version: z.string(),
-    lastUpdated: z.string(),
-    models: z.record(z.string(), z.record(z.string(), ModelConfigSchema)),
-    aliases: z.record(z.string(), z.string()).optional(),
-    defaults: z.record(z.string(), z.string()).optional(),
-});
+import { ModelRegistrySchema, } from "../types/index.js";
 /**
  * Dynamic Model Provider
  * Loads and manages model configurations from external sources

package/dist/lib/core/modelConfiguration.d.ts

@@ -4,7 +4,7 @@
  * Replaces hardcoded model-specific logic with configurable, runtime-updateable configurations.
  * This addresses GitHub Copilot review comment about making model-specific logic configuration-based.
  */
-import type { ModelTier, ProviderConfiguration, ConfigSource } from "../types/index.js";
+import type { ModelTier, ProviderConfiguration } from "../types/index.js";
 /**
  * Model name constants - extracted from hardcoded values for better maintainability
  * These constants can be overridden by environment variables
@@ -153,53 +153,6 @@ export declare class ModelConfigurationManager {
      * Get all provider configurations
      */
     getAllConfigurations(): Map<string, ProviderConfiguration>;
-    /**
-     * Update provider configuration (runtime updates)
-     */
-    updateProviderConfiguration(provider: string, config: ProviderConfiguration): void;
-    /**
-     * Parse YAML content with basic support
-     * For full YAML support, would need a proper YAML library
-     */
-    private parseYAMLContent;
-    /**
-     * Convert simple YAML to JSON object
-     * Handles basic key-value pairs only
-     */
-    private convertSimpleYAMLToJSON;
-    /**
-     * Process a single YAML line
-     * Returns key-value pair if valid, null otherwise
-     */
-    private processYAMLLine;
-    /**
-     * Convert YAML value to appropriate JavaScript type
-     */
-    private convertYAMLValue;
-    /**
-     * Load configurations from external source
-     */
-    loadConfigurationsFromFile(configPath: string): void;
-    /**
-     * Validate provider configuration structure
-     */
-    private isValidProviderConfiguration;
-    /**
-     * Apply model name overrides from configuration file
-     */
-    private applyModelNameOverrides;
-    /**
-     * Apply global default configurations
-     */
-    private applyGlobalDefaults;
-    /**
-     * Get configuration metadata
-     */
-    getConfigurationMeta(): {
-        source: ConfigSource;
-        lastUpdated: number;
-        providerCount: number;
-    };
     /**
      * Get model for specific tier and provider
      */
@@ -224,25 +177,3 @@ export declare class ModelConfigurationManager {
  * Global instance accessor
  */
 export declare const modelConfig: ModelConfigurationManager;
-/**
- * Convenience functions for common operations
- */
-/**
- * Get provider configuration (backwards compatible)
- */
-export declare function getProviderConfiguration(provider: string): ProviderConfiguration | null;
-/**
- * Get model for tier (backwards compatible)
- */
-export declare function getModelForTier(provider: string, tier: ModelTier): string | null;
-/**
- * Get cost information (backwards compatible)
- */
-export declare function getCostInfo(provider: string, model?: string): {
-    input: number;
-    output: number;
-} | null;
-/**
- * Check provider availability (backwards compatible)
- */
-export declare function isProviderAvailable(provider: string): boolean;