@juspay/neurolink 8.6.0 → 8.8.0

package/dist/utils/imageProcessor.js

@@ -3,6 +3,57 @@
  * Handles format conversion for different AI providers
  */
 import { logger } from "./logger.js";
+import { withRetry } from "./retryHandler.js";
+import { SYSTEM_LIMITS } from "../core/constants.js";
+/**
+ * Network error codes that should trigger a retry
+ */
+const RETRYABLE_ERROR_CODES = new Set([
+    "ECONNRESET",
+    "ENOTFOUND",
+    "ECONNREFUSED",
+    "ETIMEDOUT",
+    "ERR_NETWORK",
+]);
+/**
+ * Determines if an HTTP error is retryable based on status code
+ * Only network errors and certain HTTP status codes should be retried
+ * 4xx client errors like 404 (Not Found) and 403 (Forbidden) should NOT be retried
+ *
+ * @param error - The error to check
+ * @returns true if the error is retryable, false otherwise
+ */
+function isRetryableDownloadError(error) {
+    // Network-related errors should be retried
+    if (error && typeof error === "object") {
+        const errorCode = error.code;
+        const errorName = error.name;
+        if (RETRYABLE_ERROR_CODES.has(errorCode || "") ||
+            errorName === "AbortError") {
+            return true;
+        }
+    }
+    // Check for HTTP status code in error message for retryable errors
+    // Only retry on 5xx server errors, 429 (Too Many Requests), and 408 (Request Timeout)
+    // Do NOT retry on 4xx client errors like 404 (Not Found) or 403 (Forbidden)
+    if (error instanceof Error) {
+        const message = error.message;
+        // Extract HTTP status from error message like "HTTP 503: Service Unavailable"
+        const statusMatch = message.match(/HTTP (\d{3}):/);
+        if (statusMatch) {
+            const status = parseInt(statusMatch[1], 10);
+            // Retry on 5xx server errors, 429 (rate limit), 408 (timeout)
+            return status >= 500 || status === 429 || status === 408;
+        }
+        // Check for timeout/network-related error messages
+        // Use more precise matching to avoid false positives like "No timeout specified"
+        if (/\b(request timed out|operation timed out|connection timed out|timed out)\b/i.test(message) ||
+            /\bnetwork (error|failure|unreachable|down)\b/i.test(message)) {
+            return true;
+        }
+    }
+    return false;
+}
 /**
  * Image processor class for handling provider-specific image formatting
  */
@@ -16,9 +67,16 @@ export class ImageProcessor {
      * @returns Processed image as data URI
      */
     static async process(content, _options) {
+        // Validate content is non-empty before processing
+        if (content.length === 0) {
+            logger.error("Empty buffer provided");
+            throw new Error("Invalid image processing: buffer is empty");
+        }
         const mediaType = this.detectImageType(content);
         const base64 = content.toString("base64");
         const dataUri = `data:${mediaType};base64,${base64}`;
+        // Validate output before returning
+        this.validateProcessOutput(dataUri, base64, mediaType);
         return {
             type: "image",
             content: dataUri,
@@ -29,6 +87,37 @@ export class ImageProcessor {
             },
         };
     }
+    /**
+     * Validate processed output meets required format
+     * Checks:
+     * - Base64 content is non-empty
+     * - Data URI format is valid (data:{mimeType};base64,{content})
+     * - MIME type is in the allowed list
+     * @param dataUri - The complete data URI string
+     * @param base64 - The base64-encoded content
+     * @param mediaType - The MIME type of the image
+     * @throws Error if any validation fails
+     */
+    static validateProcessOutput(dataUri, base64, mediaType) {
+        // Validate base64 is non-empty (check first for better error message)
+        if (base64.length === 0) {
+            logger.error("Empty base64 content generated");
+            throw new Error("Invalid image processing: base64 content is empty");
+        }
+        // Validate data URI format with proper base64 character validation
+        // Base64 can only have 0, 1, or 2 padding characters at the end
+        const dataUriRegex = /^data:[^;]+;base64,[A-Za-z0-9+/]*={0,2}$/;
+        if (!dataUriRegex.test(dataUri)) {
+            logger.error("Invalid data URI format generated", { dataUri });
+            throw new Error("Invalid data URI format: must be data:{mimeType};base64,{content}");
+        }
+        // Defensive check: ensure detectImageType() returns valid MIME type
+        // This validation protects against future changes to detectImageType()
+        if (!this.validateImageFormat(mediaType)) {
+            logger.error("Invalid MIME type generated", { mediaType });
+            throw new Error(`Invalid MIME type: ${mediaType} is not in allowed list`);
+        }
+    }
     /**
      * Process image for OpenAI (requires data URI format)
      */
@@ -434,14 +523,35 @@ export const imageUtils = {
         }
     },
     /**
-     * Convert URL to base64 data URI by downloading the image
+     * Convert URL to base64 data URI by downloading the image.
+     * Implements retry logic with exponential backoff for network errors.
+     *
+     * Retries are performed for:
+     * - Network errors (ECONNRESET, ENOTFOUND, ECONNREFUSED, ETIMEDOUT, ERR_NETWORK, AbortError)
+     * - Server errors (5xx status codes)
+     * - Rate limiting (429 Too Many Requests)
+     * - Request timeouts (408 Request Timeout)
+     *
+     * Retries are NOT performed for:
+     * - Client errors (4xx status codes except 408, 429)
+     * - Invalid content type
+     * - Content size limit exceeded
+     * - Unsupported protocol
+     *
+     * @param url - The URL of the image to download
+     * @param options - Configuration options
+     * @param options.timeoutMs - Timeout for each download attempt (default: 15000ms)
+     * @param options.maxBytes - Maximum allowed file size (default: 10MB)
+     * @param options.maxAttempts - Maximum number of total attempts including initial attempt (default: 3)
+     * @returns Promise<string> - Base64 data URI of the downloaded image
      */
-    urlToBase64DataUri: async (url, { timeoutMs = 15000, maxBytes = 10 * 1024 * 1024 } = {}) => {
-        try {
-            // Basic protocol whitelist
-            if (!/^https?:\/\//i.test(url)) {
-                throw new Error("Unsupported protocol");
-            }
+    urlToBase64DataUri: async (url, { timeoutMs = 15000, maxBytes = 10 * 1024 * 1024, maxAttempts = 3, } = {}) => {
+        // Basic protocol whitelist - fail fast, no retry needed
+        if (!/^https?:\/\//i.test(url)) {
+            throw new Error("Unsupported protocol");
+        }
+        // Perform the actual download with retry logic
+        const performDownload = async () => {
             const controller = new AbortController();
             const t = setTimeout(() => controller.abort(), timeoutMs);
             try {
@@ -467,6 +577,20 @@ export const imageUtils = {
             finally {
                 clearTimeout(t);
             }
+        };
+        try {
+            return await withRetry(performDownload, {
+                maxAttempts,
+                initialDelay: SYSTEM_LIMITS.DEFAULT_INITIAL_DELAY,
+                backoffMultiplier: SYSTEM_LIMITS.DEFAULT_BACKOFF_MULTIPLIER,
+                maxDelay: SYSTEM_LIMITS.DEFAULT_MAX_DELAY,
+                retryCondition: isRetryableDownloadError,
+                onRetry: (attempt, error) => {
+                    const message = error instanceof Error ? error.message : String(error);
+                    const attemptsLeft = maxAttempts - attempt;
+                    logger.warn(`⚠️ Image download attempt ${attempt} failed for ${url}: ${message}. ${attemptsLeft} ${attemptsLeft === 1 ? "attempt" : "attempts"} remaining...`);
+                },
+            });
         }
         catch (error) {
             throw new Error(`Failed to download and convert URL to base64: ${error instanceof Error ? error.message : "Unknown error"}`);

package/dist/utils/pdfProcessor.js

@@ -1,6 +1,6 @@
 import { logger } from "./logger.js";
-import * as pdfjs from "pdfjs-dist/legacy/build/pdf.mjs";
-import { createCanvas } from "canvas";
+// Lazy-load pdfjs-dist to avoid DOMMatrix errors in Node.js server environment
+// import * as pdfjs from "pdfjs-dist/legacy/build/pdf.mjs";
 const PDF_PROVIDER_CONFIGS = {
     anthropic: {
         maxSizeMB: 5,
@@ -196,6 +196,28 @@ export class PDFProcessor {
         }
     }
     static async convertPDFToImages(pdfBuffer, options) {
+        // Dynamic import canvas - only load when actually needed
+        let createCanvas;
+        try {
+            const canvasModule = await import("canvas");
+            createCanvas = canvasModule.createCanvas;
+        }
+        catch {
+            throw new Error("Canvas dependency not available. " +
+                "PDF-to-image conversion requires the 'canvas' package with native bindings. " +
+                "Install with: pnpm install canvas\n" +
+                "Note: This requires native build tools (Python, C++ compiler).");
+        }
+        // Dynamic import pdfjs - only load when actually needed to avoid DOMMatrix errors
+        let pdfjs;
+        try {
+            pdfjs = await import("pdfjs-dist/legacy/build/pdf.mjs");
+        }
+        catch {
+            throw new Error("pdfjs-dist dependency not available. " +
+                "PDF processing requires the 'pdfjs-dist' package. " +
+                "Install with: pnpm install pdfjs-dist");
+        }
         const maxPages = options?.maxPages || 10;
         const scale = options?.scale || 2.0;
         const format = options?.format || "png";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@juspay/neurolink",
-  "version": "8.6.0",
+  "version": "8.8.0",
   "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",
@@ -179,7 +179,6 @@
     "@opentelemetry/sdk-trace-node": "^2.1.0",
     "@opentelemetry/semantic-conventions": "^1.30.1",
     "ai": "4.3.16",
-    "canvas": "^3.2.0",
     "chalk": "^5.6.2",
     "csv-parser": "^3.2.0",
     "dotenv": "^16.6.1",
@@ -201,6 +200,9 @@
     "zod": "^3.22.0",
     "zod-to-json-schema": "^3.24.6"
   },
+  "optionalDependencies": {
+    "canvas": "^3.2.0"
+  },
   "devDependencies": {
     "@biomejs/biome": "^2.2.4",
     "@changesets/changelog-github": "^0.5.1",
@@ -210,7 +212,7 @@
     "@semantic-release/commit-analyzer": "^13.0.1",
     "@semantic-release/git": "^10.0.1",
     "@semantic-release/github": "^11.0.6",
-    "@semantic-release/npm": "^12.0.2",
+    "@semantic-release/npm": "^13.1.2",
     "@semantic-release/release-notes-generator": "^14.1.0",
     "@smithy/types": "^4.5.0",
     "@sveltejs/adapter-auto": "^6.1.0",
@@ -297,7 +299,8 @@
       "@eslint/plugin-kit@<0.3.4": ">=0.3.4",
       "tmp@<=0.2.3": ">=0.2.4",
       "axios@<1.8.2": ">=1.8.2",
-      "glob@>=10.3.7 <=11.0.3": ">=11.1.0"
+      "glob@>=10.3.7 <=11.0.3": ">=11.1.0",
+      "@semantic-release/npm": "^13.1.2"
     }
   },
   "os": [