@juspay/neurolink 8.13.0 → 8.13.2

package/CHANGELOG.md

@@ -1,3 +1,15 @@
+## [8.13.2](https://github.com/juspay/neurolink/compare/v8.13.1...v8.13.2) (2025-12-14)
+
+### Bug Fixes
+
+- **(sdk):** Replace hardcoded timeouts with class constants ([a34c291](https://github.com/juspay/neurolink/commit/a34c29155e82ef4f498714c031401463351171bd))
+
+## [8.13.1](https://github.com/juspay/neurolink/compare/v8.13.0...v8.13.1) (2025-12-13)
+
+### Bug Fixes
+
+- **(provider):** Implement image count limits with validation and warnings ([ff3e27a](https://github.com/juspay/neurolink/commit/ff3e27a5ab3aafffc8312f645e0ebc566600cc63))
+
 ## [8.13.0](https://github.com/juspay/neurolink/compare/v8.12.0...v8.13.0) (2025-12-13)
 
 ### Features

package/dist/adapters/providerImageAdapter.d.ts

@@ -33,6 +33,11 @@ export declare class ProviderImageAdapter {
      * Format content for Vertex AI (model-specific routing)
      */
     private static formatForVertex;
+    /**
+     * Validate image count against provider limits
+     * Warns at 80% threshold, throws error if limit exceeded
+     */
+    private static validateImageCount;
     /**
      * Validate that provider and model support vision
      */

package/dist/adapters/providerImageAdapter.js

@@ -16,6 +16,28 @@ export class MultimodalLogger {
         }
     }
 }
+/**
+ * Image count limits per provider
+ * These limits prevent API rejections when too many images are sent
+ */
+const IMAGE_LIMITS = {
+    openai: 10,
+    azure: 10, // Same as OpenAI
+    "google-ai": 16,
+    google: 16,
+    anthropic: 20,
+    vertex: {
+        // Vertex has model-specific limits
+        claude: 20, // Claude models on Vertex
+        gemini: 16, // Gemini models on Vertex
+        default: 16,
+    },
+    ollama: 10, // Conservative limit for Ollama
+    litellm: 10, // Conservative limit, as it proxies to various providers
+    mistral: 10, // Conservative limit for Mistral
+    // Note: Bedrock limit defined for future use when vision support is added
+    bedrock: 20, // Same as Anthropic for Claude models on Bedrock
+};
 /**
  * Vision capability definitions for each provider
  */
@@ -368,7 +390,9 @@ export class ProviderImageAdapter {
                     break;
                 case "azure":
                 case "azure-openai":
-                    adaptedPayload = this.formatForOpenAI(text, images);
+                    // Azure uses same format as OpenAI but validate with azure provider name
+                    this.validateImageCount(images.length, "azure");
+                    adaptedPayload = this.formatForOpenAI(text, images, true);
                     break;
                 case "google-ai":
                 case "google":
@@ -381,7 +405,9 @@ export class ProviderImageAdapter {
                     adaptedPayload = this.formatForVertex(text, images, model);
                     break;
                 case "ollama":
-                    adaptedPayload = this.formatForOpenAI(text, images);
+                    // Ollama uses same format as OpenAI but validate with ollama provider name
+                    this.validateImageCount(images.length, "ollama");
+                    adaptedPayload = this.formatForOpenAI(text, images, true);
                     break;
                 case "huggingface":
                     adaptedPayload = this.formatForOpenAI(text, images);
@@ -415,7 +441,11 @@ export class ProviderImageAdapter {
     /**
      * Format content for OpenAI (GPT-4o format)
      */
-    static formatForOpenAI(text, images) {
+    static formatForOpenAI(text, images, skipValidation = false) {
+        // Validate image count before processing (unless called from another formatter)
+        if (!skipValidation) {
+            this.validateImageCount(images.length, "openai");
+        }
         const content = [{ type: "text", text }];
         images.forEach((image, index) => {
             try {
@@ -438,7 +468,11 @@ export class ProviderImageAdapter {
     /**
      * Format content for Google AI (Gemini format)
      */
-    static formatForGoogleAI(text, images) {
+    static formatForGoogleAI(text, images, skipValidation = false) {
+        // Validate image count before processing (unless called from another formatter)
+        if (!skipValidation) {
+            this.validateImageCount(images.length, "google-ai");
+        }
         const parts = [{ text }];
         images.forEach((image, index) => {
             try {
@@ -460,7 +494,11 @@ export class ProviderImageAdapter {
     /**
      * Format content for Anthropic (Claude format)
      */
-    static formatForAnthropic(text, images) {
+    static formatForAnthropic(text, images, skipValidation = false) {
+        // Validate image count before processing (unless called from another formatter)
+        if (!skipValidation) {
+            this.validateImageCount(images.length, "anthropic");
+        }
         const content = [{ type: "text", text }];
         images.forEach((image, index) => {
             try {
@@ -488,15 +526,65 @@ export class ProviderImageAdapter {
      * Format content for Vertex AI (model-specific routing)
      */
     static formatForVertex(text, images, model) {
-        // Route based on model type
+        // Validate image count with model-specific limits before processing
+        this.validateImageCount(images.length, "vertex", model);
+        // Route based on model type, skip validation in delegated methods
         if (model.includes("gemini")) {
-            return this.formatForGoogleAI(text, images);
+            return this.formatForGoogleAI(text, images, true);
         }
         else if (model.includes("claude")) {
-            return this.formatForAnthropic(text, images);
+            return this.formatForAnthropic(text, images, true);
+        }
+        else {
+            return this.formatForGoogleAI(text, images, true);
+        }
+    }
+    /**
+     * Validate image count against provider limits
+     * Warns at 80% threshold, throws error if limit exceeded
+     */
+    static validateImageCount(imageCount, provider, model) {
+        const normalizedProvider = provider.toLowerCase();
+        let limit;
+        // Determine the limit based on provider
+        if (normalizedProvider === "vertex" && model) {
+            // Vertex has model-specific limits
+            if (model.includes("claude")) {
+                limit = IMAGE_LIMITS.vertex.claude;
+            }
+            else if (model.includes("gemini")) {
+                limit = IMAGE_LIMITS.vertex.gemini;
+            }
+            else {
+                limit = IMAGE_LIMITS.vertex.default;
+            }
         }
         else {
-            return this.formatForGoogleAI(text, images);
+            // Use provider-specific limit
+            const providerLimit = normalizedProvider in IMAGE_LIMITS
+                ? IMAGE_LIMITS[normalizedProvider]
+                : undefined;
+            // If provider not found in limits map, use a conservative default
+            if (providerLimit === undefined) {
+                // Conservative default for unknown providers
+                limit = 10;
+                logger.warn(`Image count limit not defined for provider ${provider}. Using conservative default of 10 images.`);
+            }
+            else {
+                // providerLimit is always a number when defined (except vertex which is handled separately)
+                limit = providerLimit;
+            }
+        }
+        // Warn only once at 80% threshold to avoid noise in batch processing
+        const warningThreshold = Math.floor(limit * 0.8);
+        if (imageCount === warningThreshold) {
+            logger.warn(`Image count (${imageCount}) is approaching the limit for ${provider}. ` +
+                `Maximum allowed: ${limit}. Please reduce the number of images.`);
+        }
+        // Throw error if limit exceeded
+        if (imageCount > limit) {
+            throw new Error(`Image count (${imageCount}) exceeds the maximum limit for ${provider}. ` +
+                `Maximum allowed: ${limit}. Please reduce the number of images.`);
         }
     }
     /**

package/dist/lib/adapters/providerImageAdapter.d.ts

@@ -33,6 +33,11 @@ export declare class ProviderImageAdapter {
      * Format content for Vertex AI (model-specific routing)
      */
     private static formatForVertex;
+    /**
+     * Validate image count against provider limits
+     * Warns at 80% threshold, throws error if limit exceeded
+     */
+    private static validateImageCount;
     /**
      * Validate that provider and model support vision
      */

package/dist/lib/adapters/providerImageAdapter.js

@@ -16,6 +16,28 @@ export class MultimodalLogger {
         }
     }
 }
+/**
+ * Image count limits per provider
+ * These limits prevent API rejections when too many images are sent
+ */
+const IMAGE_LIMITS = {
+    openai: 10,
+    azure: 10, // Same as OpenAI
+    "google-ai": 16,
+    google: 16,
+    anthropic: 20,
+    vertex: {
+        // Vertex has model-specific limits
+        claude: 20, // Claude models on Vertex
+        gemini: 16, // Gemini models on Vertex
+        default: 16,
+    },
+    ollama: 10, // Conservative limit for Ollama
+    litellm: 10, // Conservative limit, as it proxies to various providers
+    mistral: 10, // Conservative limit for Mistral
+    // Note: Bedrock limit defined for future use when vision support is added
+    bedrock: 20, // Same as Anthropic for Claude models on Bedrock
+};
 /**
  * Vision capability definitions for each provider
  */
@@ -368,7 +390,9 @@ export class ProviderImageAdapter {
                     break;
                 case "azure":
                 case "azure-openai":
-                    adaptedPayload = this.formatForOpenAI(text, images);
+                    // Azure uses same format as OpenAI but validate with azure provider name
+                    this.validateImageCount(images.length, "azure");
+                    adaptedPayload = this.formatForOpenAI(text, images, true);
                     break;
                 case "google-ai":
                 case "google":
@@ -381,7 +405,9 @@ export class ProviderImageAdapter {
                     adaptedPayload = this.formatForVertex(text, images, model);
                     break;
                 case "ollama":
-                    adaptedPayload = this.formatForOpenAI(text, images);
+                    // Ollama uses same format as OpenAI but validate with ollama provider name
+                    this.validateImageCount(images.length, "ollama");
+                    adaptedPayload = this.formatForOpenAI(text, images, true);
                     break;
                 case "huggingface":
                     adaptedPayload = this.formatForOpenAI(text, images);
@@ -415,7 +441,11 @@ export class ProviderImageAdapter {
     /**
      * Format content for OpenAI (GPT-4o format)
      */
-    static formatForOpenAI(text, images) {
+    static formatForOpenAI(text, images, skipValidation = false) {
+        // Validate image count before processing (unless called from another formatter)
+        if (!skipValidation) {
+            this.validateImageCount(images.length, "openai");
+        }
         const content = [{ type: "text", text }];
         images.forEach((image, index) => {
             try {
@@ -438,7 +468,11 @@ export class ProviderImageAdapter {
     /**
      * Format content for Google AI (Gemini format)
      */
-    static formatForGoogleAI(text, images) {
+    static formatForGoogleAI(text, images, skipValidation = false) {
+        // Validate image count before processing (unless called from another formatter)
+        if (!skipValidation) {
+            this.validateImageCount(images.length, "google-ai");
+        }
         const parts = [{ text }];
         images.forEach((image, index) => {
             try {
@@ -460,7 +494,11 @@ export class ProviderImageAdapter {
     /**
      * Format content for Anthropic (Claude format)
      */
-    static formatForAnthropic(text, images) {
+    static formatForAnthropic(text, images, skipValidation = false) {
+        // Validate image count before processing (unless called from another formatter)
+        if (!skipValidation) {
+            this.validateImageCount(images.length, "anthropic");
+        }
         const content = [{ type: "text", text }];
         images.forEach((image, index) => {
             try {
@@ -488,15 +526,65 @@ export class ProviderImageAdapter {
      * Format content for Vertex AI (model-specific routing)
      */
     static formatForVertex(text, images, model) {
-        // Route based on model type
+        // Validate image count with model-specific limits before processing
+        this.validateImageCount(images.length, "vertex", model);
+        // Route based on model type, skip validation in delegated methods
         if (model.includes("gemini")) {
-            return this.formatForGoogleAI(text, images);
+            return this.formatForGoogleAI(text, images, true);
         }
         else if (model.includes("claude")) {
-            return this.formatForAnthropic(text, images);
+            return this.formatForAnthropic(text, images, true);
+        }
+        else {
+            return this.formatForGoogleAI(text, images, true);
+        }
+    }
+    /**
+     * Validate image count against provider limits
+     * Warns at 80% threshold, throws error if limit exceeded
+     */
+    static validateImageCount(imageCount, provider, model) {
+        const normalizedProvider = provider.toLowerCase();
+        let limit;
+        // Determine the limit based on provider
+        if (normalizedProvider === "vertex" && model) {
+            // Vertex has model-specific limits
+            if (model.includes("claude")) {
+                limit = IMAGE_LIMITS.vertex.claude;
+            }
+            else if (model.includes("gemini")) {
+                limit = IMAGE_LIMITS.vertex.gemini;
+            }
+            else {
+                limit = IMAGE_LIMITS.vertex.default;
+            }
         }
         else {
-            return this.formatForGoogleAI(text, images);
+            // Use provider-specific limit
+            const providerLimit = normalizedProvider in IMAGE_LIMITS
+                ? IMAGE_LIMITS[normalizedProvider]
+                : undefined;
+            // If provider not found in limits map, use a conservative default
+            if (providerLimit === undefined) {
+                // Conservative default for unknown providers
+                limit = 10;
+                logger.warn(`Image count limit not defined for provider ${provider}. Using conservative default of 10 images.`);
+            }
+            else {
+                // providerLimit is always a number when defined (except vertex which is handled separately)
+                limit = providerLimit;
+            }
+        }
+        // Warn only once at 80% threshold to avoid noise in batch processing
+        const warningThreshold = Math.floor(limit * 0.8);
+        if (imageCount === warningThreshold) {
+            logger.warn(`Image count (${imageCount}) is approaching the limit for ${provider}. ` +
+                `Maximum allowed: ${limit}. Please reduce the number of images.`);
+        }
+        // Throw error if limit exceeded
+        if (imageCount > limit) {
+            throw new Error(`Image count (${imageCount}) exceeds the maximum limit for ${provider}. ` +
+                `Maximum allowed: ${limit}. Please reduce the number of images.`);
         }
     }
     /**

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

@@ -15,6 +15,8 @@ import type { FileInput, FileProcessingResult, FileDetectorOptions } from "../ty
  * ```
  */
 export declare class FileDetector {
+    static readonly DEFAULT_NETWORK_TIMEOUT = 30000;
+    static readonly DEFAULT_HEAD_TIMEOUT = 5000;
     /**
      * Auto-detect file type and process in one call
      *

package/dist/lib/utils/fileDetector.js

@@ -35,6 +35,10 @@ function formatFileSize(bytes) {
  * ```
  */
 export class FileDetector {
+    // FD-017: Replace hardcoded timeouts with constants.
+    // These default ensure consistent timeout behavior across all file-detection logic.
+    static DEFAULT_NETWORK_TIMEOUT = 30000; // 30 seconds
+    static DEFAULT_HEAD_TIMEOUT = 5000; // 5 seconds
     /**
      * Auto-detect file type and process in one call
      *
@@ -144,7 +148,7 @@ export class FileDetector {
      */
     static async loadFromURL(url, options) {
         const maxSize = options?.maxSize || 10 * 1024 * 1024;
-        const timeout = options?.timeout || 30000;
+        const timeout = options?.timeout || this.DEFAULT_NETWORK_TIMEOUT;
         const response = await request(url, {
             dispatcher: getGlobalDispatcher().compose(interceptors.redirect({ maxRedirections: 5 })),
             method: "GET",
@@ -273,8 +277,8 @@ class MimeTypeStrategy {
             const response = await request(input, {
                 dispatcher: getGlobalDispatcher().compose(interceptors.redirect({ maxRedirections: 5 })),
                 method: "HEAD",
-                headersTimeout: 5000,
-                bodyTimeout: 5000,
+                headersTimeout: FileDetector.DEFAULT_HEAD_TIMEOUT,
+                bodyTimeout: FileDetector.DEFAULT_HEAD_TIMEOUT,
             });
             const contentType = response.headers["content-type"] || "";
             const type = this.mimeToFileType(contentType);

package/dist/utils/fileDetector.d.ts

@@ -15,6 +15,8 @@ import type { FileInput, FileProcessingResult, FileDetectorOptions } from "../ty
  * ```
  */
 export declare class FileDetector {
+    static readonly DEFAULT_NETWORK_TIMEOUT = 30000;
+    static readonly DEFAULT_HEAD_TIMEOUT = 5000;
     /**
      * Auto-detect file type and process in one call
      *

package/dist/utils/fileDetector.js

@@ -35,6 +35,10 @@ function formatFileSize(bytes) {
  * ```
  */
 export class FileDetector {
+    // FD-017: Replace hardcoded timeouts with constants.
+    // These default ensure consistent timeout behavior across all file-detection logic.
+    static DEFAULT_NETWORK_TIMEOUT = 30000; // 30 seconds
+    static DEFAULT_HEAD_TIMEOUT = 5000; // 5 seconds
     /**
      * Auto-detect file type and process in one call
      *
@@ -144,7 +148,7 @@ export class FileDetector {
      */
     static async loadFromURL(url, options) {
         const maxSize = options?.maxSize || 10 * 1024 * 1024;
-        const timeout = options?.timeout || 30000;
+        const timeout = options?.timeout || this.DEFAULT_NETWORK_TIMEOUT;
         const response = await request(url, {
             dispatcher: getGlobalDispatcher().compose(interceptors.redirect({ maxRedirections: 5 })),
             method: "GET",
@@ -273,8 +277,8 @@ class MimeTypeStrategy {
             const response = await request(input, {
                 dispatcher: getGlobalDispatcher().compose(interceptors.redirect({ maxRedirections: 5 })),
                 method: "HEAD",
-                headersTimeout: 5000,
-                bodyTimeout: 5000,
+                headersTimeout: FileDetector.DEFAULT_HEAD_TIMEOUT,
+                bodyTimeout: FileDetector.DEFAULT_HEAD_TIMEOUT,
             });
             const contentType = response.headers["content-type"] || "";
             const type = this.mimeToFileType(contentType);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@juspay/neurolink",
-  "version": "8.13.0",
+  "version": "8.13.2",
   "description": "Universal AI Development Platform with working MCP integration, multi-provider support, and professional CLI. Built-in tools operational, 58+ external MCP servers discoverable. Connect to filesystem, GitHub, database operations, and more. Build, test, and deploy AI applications with 9 major providers: OpenAI, Anthropic, Google AI, AWS Bedrock, Azure, Hugging Face, Ollama, and Mistral AI.",
   "author": {
     "name": "Juspay Technologies",