@juspay/neurolink 10.10.8 → 10.10.9

package/dist/lib/skills/skillsManager.js

@@ -87,7 +87,14 @@ export class SkillsManager {
         // name-find would resolve non-deterministically to the stale deprecated one
         // across store backends. Fall back to any match only when no active skill
         // carries the name.
-        const entry = index.find((item) => item.name === idOrName && (item.status ?? "active") === "active") ?? index.find((item) => item.name === idOrName);
+        //
+        // Matched case-insensitively to agree with assertNameAvailable, which
+        // enforces uniqueness that way: names differing only in case cannot
+        // coexist, so a case-sensitive lookup could only fail to find a skill that
+        // is definitively there (`get("DEPLOY")` returned null for "deploy").
+        const wanted = idOrName.toLowerCase();
+        const entry = index.find((item) => item.name.toLowerCase() === wanted &&
+            (item.status ?? "active") === "active") ?? index.find((item) => item.name.toLowerCase() === wanted);
         return entry ? this.store.get(entry.id) : null;
     }
     /**
@@ -237,11 +244,23 @@ export class SkillsManager {
     }
     async assertNameAvailable(name, excludeId) {
         const index = await this.getIndex(true);
-        const clash = index.find((item) => item.id !== excludeId &&
-            item.name.toLowerCase() === name.toLowerCase() &&
-            (item.status ?? "active") === "active");
+        // Deliberately NOT filtered to active (#1139). Soft-delete only flips
+        // status to "deprecated" — the entry stays in the index, so allowing a new
+        // skill to take the name left two entries sharing it. Every name-based
+        // lookup (CLI `skills show/delete <name>`, the skill_update/skill_delete
+        // tools, the `:id`-or-name REST routes) then had to guess which one the
+        // caller meant, and iteration order differs across store backends.
+        //
+        // A deprecated skill's name therefore stays reserved. Reusing it requires
+        // hard-deleting the old skill first, which is the explicit choice the
+        // ambiguity demands.
+        const clash = index.find((item) => item.id !== excludeId && item.name.toLowerCase() === name.toLowerCase());
         if (clash) {
-            throw new Error(`A skill named "${name}" already exists`);
+            const suffix = (clash.status ?? "active") === "active"
+                ? ""
+                : ` (that name belongs to a deprecated skill, id ${clash.id}; ` +
+                    `remove it before reusing the name)`;
+            throw new Error(`A skill named "${name}" already exists${suffix}`);
         }
     }
 }

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

@@ -474,6 +474,10 @@ export type MultimodalPdfEntry = {
     password?: string;
     /** Per-page pixel ceiling for the image fallback (#260). */
     maxCanvasPixels?: number;
+    /** Render scale for the image fallback (#297). */
+    scale?: number;
+    /** Max pages converted by the image fallback (#297). */
+    maxPages?: number;
 };
 /** Result of PDF to image conversion. */
 export type PDFImageConversionResult = {

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

@@ -133,6 +133,17 @@ export type GenerateOptions = {
         password?: string;
         /** Max rendered-canvas pixels per page (#260 memory guard); oversized pages auto-downscale. */
         maxCanvasPixels?: number;
+        /**
+         * Render scale for the image fallback used by providers without native PDF
+         * support (#297). Higher is sharper but costs roughly the square in memory
+         * and tokens. Range 0.1-10; defaults to PDF_LIMITS.DEFAULT_SCALE (1.5).
+         */
+        scale?: number;
+        /**
+         * Max pages converted by the image fallback (#297). Pages beyond this are
+         * not sent to the model at all. Defaults to PDF_LIMITS.DEFAULT_MAX_PAGES (20).
+         */
+        maxPages?: number;
     };
     videoOptions?: {
         /** Frames to extract. Unset lets VideoProcessor pick from the clip's duration; clamped to 100. */
@@ -1265,6 +1276,10 @@ export type TextGenerationOptions = {
         password?: string;
         /** Max rendered-canvas pixels per page (#260 memory guard); oversized pages auto-downscale. */
         maxCanvasPixels?: number;
+        /** Render scale for the image fallback (#297); defaults to PDF_LIMITS.DEFAULT_SCALE. */
+        scale?: number;
+        /** Max pages converted by the image fallback (#297); defaults to PDF_LIMITS.DEFAULT_MAX_PAGES. */
+        maxPages?: number;
     };
     enableSummarization?: boolean;
     /**

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

@@ -225,6 +225,17 @@ export type StreamOptions = {
         password?: string;
         /** Max rendered-canvas pixels per page (#260 memory guard); oversized pages auto-downscale. */
         maxCanvasPixels?: number;
+        /**
+         * Render scale for the image fallback used by providers without native PDF
+         * support (#297). Higher is sharper but costs roughly the square in memory
+         * and tokens. Range 0.1-10; defaults to PDF_LIMITS.DEFAULT_SCALE (1.5).
+         */
+        scale?: number;
+        /**
+         * Max pages converted by the image fallback (#297). Pages beyond this are
+         * not sent to the model at all. Defaults to PDF_LIMITS.DEFAULT_MAX_PAGES (20).
+         */
+        maxPages?: number;
     };
     videoOptions?: {
         /** Frames to extract. Unset lets VideoProcessor pick from the clip's duration; clamped to 100. */

package/dist/lib/utils/messageBuilder.js

@@ -4,6 +4,7 @@ import { getGlobalDispatcher, interceptors, request } from "undici";
 import { MultimodalLogger, ProviderImageAdapter, } from "../adapters/providerImageAdapter.js";
 import { CONVERSATION_INSTRUCTIONS, STRUCTURED_OUTPUT_INSTRUCTIONS, } from "../config/conversationMemory.js";
 import { getAvailableInputTokens } from "../constants/contextWindows.js";
+import { PDF_LIMITS } from "../core/constants.js";
 import { enforceAggregateFileBudget, FILE_READ_BUDGET_PERCENT, } from "../context/fileTokenBudget.js";
 import { isCSVContent, SIZE_TIER_THRESHOLDS } from "../types/index.js";
 import { tracers, ATTR, withSpan } from "../telemetry/index.js";
@@ -1150,6 +1151,10 @@ async function processExplicitPdfFiles(options, maxSize, provider) {
                     // #260: carry the per-page canvas-pixel ceiling so the caller can
                     // raise (or lower) the memory guard for the image-fallback render.
                     maxCanvasPixels: options.pdfOptions?.maxCanvasPixels,
+                    // #297: render scale / page ceiling, so the lowered default is
+                    // actually reachable and callers can trade sharpness for memory.
+                    scale: options.pdfOptions?.scale,
+                    maxPages: options.pdfOptions?.maxPages,
                 });
                 logger.info(`[PDF] ✅ Queued for multimodal: ${filename} (${result.metadata?.estimatedPages ?? "unknown"} pages)`);
             }
@@ -1458,6 +1463,8 @@ async function convertContentToProviderFormat(content, provider, _model, pdfOpti
         // guard as the `input.pdfFiles` path (see `processExplicitPdfFiles`).
         password: pdfOptions?.password,
         maxCanvasPixels: pdfOptions?.maxCanvasPixels,
+        scale: pdfOptions?.scale,
+        maxPages: pdfOptions?.maxPages,
     }));
     // #309: same aggregate ceiling as `input.pdfFiles`. Without this, moving an
     // over-limit payload from `input.pdfFiles` to `input.content` skipped the
@@ -1809,7 +1816,10 @@ async function convertSimpleImagesToProviderFormat(text, images, provider, _mode
 /**
  * Convert multimodal content (images + PDFs) to provider format
  */
-async function convertMultimodalToProviderFormat(text, images, pdfFiles, provider, model) {
+async function convertMultimodalToProviderFormat(text, images, 
+// The canonical entry shape (#309) rather than a fourth copy of it inline —
+// which is what let the render knobs stop short of this function.
+pdfFiles, provider, model) {
     const content = [
         { type: "text", text },
     ];
@@ -1843,14 +1853,41 @@ async function convertMultimodalToProviderFormat(text, images, pdfFiles, provide
         logger.info(`[PDF→Image] Provider ${provider} doesn't support native PDF. Converting ${pdfFiles.length} PDF(s) to images...`);
         for (const pdf of pdfFiles) {
             try {
+                const effectiveMaxPages = pdf.maxPages ?? PDF_LIMITS.DEFAULT_MAX_PAGES;
                 const conversionResult = await PDFImageConverter.convertToImages(pdf.buffer, {
-                    scale: 2.0, // High quality for OCR/analysis
-                    maxPages: 20, // Limit pages to prevent token overflow
+                    // #297: this is the only PDF→image call the product actually makes,
+                    // and it used to hardcode scale 2.0 — silently overriding the
+                    // lowered PDF_LIMITS.DEFAULT_SCALE and keeping the memory cost the
+                    // issue reports (a 100-page render at 2.0 is ~776MB; 1.5 is ~44%
+                    // fewer pixels per page). Callers can raise it back per request.
+                    scale: pdf.scale ?? PDF_LIMITS.DEFAULT_SCALE,
+                    // Page ceiling guards token overflow; also now caller-adjustable
+                    // rather than a constant nothing could reach.
+                    maxPages: effectiveMaxPages,
                     ...(pdf.password ? { password: pdf.password } : {}), // #258
                     ...(pdf.maxCanvasPixels
                         ? { maxCanvasPixels: pdf.maxCanvasPixels }
                         : {}), // #260
                 });
+                // The renderer stops at maxPages, so a longer document is silently
+                // truncated — say so rather than letting the model answer from a
+                // partial document as though it had the whole thing.
+                //
+                // Keyed on the cap being reached, not on pdf.pageCount: that field is
+                // null whenever `input.content` omits `metadata.pages`, which is the
+                // common case, so a page-count comparison would simply never fire
+                // there. Reaching the cap is also unambiguous — a short count caused by
+                // per-page render failures (#294 isolates those into `errors`) would
+                // otherwise be misreported as a maxPages truncation.
+                if (conversionResult.pageCount >= effectiveMaxPages) {
+                    logger.warn(`[PDF→Image] ${safeBasename(pdf.filename)} hit the ${effectiveMaxPages}-page ` +
+                        `conversion limit. Any pages beyond that were not sent — the model may be ` +
+                        `answering from a partial document. Raise pdfOptions.maxPages or split the file.`);
+                }
+                if (conversionResult.errors && conversionResult.errors.length > 0) {
+                    logger.warn(`[PDF→Image] ${safeBasename(pdf.filename)}: ${conversionResult.errors.length} page(s) ` +
+                        `failed to render and were omitted (page ${conversionResult.errors.map((e) => e.page).join(", ")}).`);
+                }
                 logger.info(`[PDF→Image] ✅ Converted ${pdf.filename}: ${conversionResult.pageCount} page(s) → images`);
                 // Add each page as an ImagePart (raw base64, not data: URI — see SSRF note above)
                 conversionResult.images.forEach((base64Image, pageIndex) => {

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

@@ -58,6 +58,8 @@ export declare function buildMultimodalOptions(options: StreamOptions, providerN
     pdfOptions: {
         password?: string;
         maxCanvasPixels?: number;
+        scale?: number;
+        maxPages?: number;
     } | undefined;
     systemPrompt: string | undefined;
     conversationHistory: import("../types/conversation.js").ChatMessage[] | undefined;

package/dist/lib/utils/pdfProcessor.js

@@ -377,6 +377,7 @@ export class PDFProcessor {
             format,
             scale,
             maxCanvasPixels,
+            maxPages,
         });
         logger.debug("[PDF→Image] ✅ PDF validation passed", {
             bufferSize: pdfBuffer.length,
@@ -535,6 +536,14 @@ export class PDFProcessor {
         if (!Number.isFinite(opts.maxCanvasPixels) || opts.maxCanvasPixels <= 0) {
             throw new Error(`Invalid maxCanvasPixels: ${opts.maxCanvasPixels}. Must be a finite number greater than 0.`);
         }
+        // #297: maxPages became caller-controlled, and an unvalidated 0/-1/NaN
+        // silently converts nothing, surfacing later as a misleading
+        // "PDF has 0 pages" from deep inside the renderer. Reject it here where
+        // the message can still name the offending option.
+        if (opts.maxPages !== undefined &&
+            (!Number.isInteger(opts.maxPages) || opts.maxPages < 1)) {
+            throw new Error(`Invalid maxPages: ${opts.maxPages}. Must be a whole number of at least 1.`);
+        }
         if (!pdfBuffer || pdfBuffer.length < 5) {
             throw new Error("Invalid PDF: Buffer is too small or empty. " +
                 "A valid PDF must be at least 5 bytes (PDF header).");
@@ -570,6 +579,7 @@ export class PDFProcessor {
             format,
             scale,
             maxCanvasPixels,
+            maxPages,
         });
         const pdfToImgModule = await import("pdf-to-img");
         const pdf = pdfToImgModule.pdf;

package/dist/skills/skillsManager.js

@@ -87,7 +87,14 @@ export class SkillsManager {
         // name-find would resolve non-deterministically to the stale deprecated one
         // across store backends. Fall back to any match only when no active skill
         // carries the name.
-        const entry = index.find((item) => item.name === idOrName && (item.status ?? "active") === "active") ?? index.find((item) => item.name === idOrName);
+        //
+        // Matched case-insensitively to agree with assertNameAvailable, which
+        // enforces uniqueness that way: names differing only in case cannot
+        // coexist, so a case-sensitive lookup could only fail to find a skill that
+        // is definitively there (`get("DEPLOY")` returned null for "deploy").
+        const wanted = idOrName.toLowerCase();
+        const entry = index.find((item) => item.name.toLowerCase() === wanted &&
+            (item.status ?? "active") === "active") ?? index.find((item) => item.name.toLowerCase() === wanted);
         return entry ? this.store.get(entry.id) : null;
     }
     /**
@@ -237,11 +244,23 @@ export class SkillsManager {
     }
     async assertNameAvailable(name, excludeId) {
         const index = await this.getIndex(true);
-        const clash = index.find((item) => item.id !== excludeId &&
-            item.name.toLowerCase() === name.toLowerCase() &&
-            (item.status ?? "active") === "active");
+        // Deliberately NOT filtered to active (#1139). Soft-delete only flips
+        // status to "deprecated" — the entry stays in the index, so allowing a new
+        // skill to take the name left two entries sharing it. Every name-based
+        // lookup (CLI `skills show/delete <name>`, the skill_update/skill_delete
+        // tools, the `:id`-or-name REST routes) then had to guess which one the
+        // caller meant, and iteration order differs across store backends.
+        //
+        // A deprecated skill's name therefore stays reserved. Reusing it requires
+        // hard-deleting the old skill first, which is the explicit choice the
+        // ambiguity demands.
+        const clash = index.find((item) => item.id !== excludeId && item.name.toLowerCase() === name.toLowerCase());
         if (clash) {
-            throw new Error(`A skill named "${name}" already exists`);
+            const suffix = (clash.status ?? "active") === "active"
+                ? ""
+                : ` (that name belongs to a deprecated skill, id ${clash.id}; ` +
+                    `remove it before reusing the name)`;
+            throw new Error(`A skill named "${name}" already exists${suffix}`);
         }
     }
 }

package/dist/types/file.d.ts

@@ -474,6 +474,10 @@ export type MultimodalPdfEntry = {
     password?: string;
     /** Per-page pixel ceiling for the image fallback (#260). */
     maxCanvasPixels?: number;
+    /** Render scale for the image fallback (#297). */
+    scale?: number;
+    /** Max pages converted by the image fallback (#297). */
+    maxPages?: number;
 };
 /** Result of PDF to image conversion. */
 export type PDFImageConversionResult = {

package/dist/types/generate.d.ts

@@ -133,6 +133,17 @@ export type GenerateOptions = {
         password?: string;
         /** Max rendered-canvas pixels per page (#260 memory guard); oversized pages auto-downscale. */
         maxCanvasPixels?: number;
+        /**
+         * Render scale for the image fallback used by providers without native PDF
+         * support (#297). Higher is sharper but costs roughly the square in memory
+         * and tokens. Range 0.1-10; defaults to PDF_LIMITS.DEFAULT_SCALE (1.5).
+         */
+        scale?: number;
+        /**
+         * Max pages converted by the image fallback (#297). Pages beyond this are
+         * not sent to the model at all. Defaults to PDF_LIMITS.DEFAULT_MAX_PAGES (20).
+         */
+        maxPages?: number;
     };
     videoOptions?: {
         /** Frames to extract. Unset lets VideoProcessor pick from the clip's duration; clamped to 100. */
@@ -1265,6 +1276,10 @@ export type TextGenerationOptions = {
         password?: string;
         /** Max rendered-canvas pixels per page (#260 memory guard); oversized pages auto-downscale. */
         maxCanvasPixels?: number;
+        /** Render scale for the image fallback (#297); defaults to PDF_LIMITS.DEFAULT_SCALE. */
+        scale?: number;
+        /** Max pages converted by the image fallback (#297); defaults to PDF_LIMITS.DEFAULT_MAX_PAGES. */
+        maxPages?: number;
     };
     enableSummarization?: boolean;
     /**

package/dist/types/stream.d.ts

@@ -225,6 +225,17 @@ export type StreamOptions = {
         password?: string;
         /** Max rendered-canvas pixels per page (#260 memory guard); oversized pages auto-downscale. */
         maxCanvasPixels?: number;
+        /**
+         * Render scale for the image fallback used by providers without native PDF
+         * support (#297). Higher is sharper but costs roughly the square in memory
+         * and tokens. Range 0.1-10; defaults to PDF_LIMITS.DEFAULT_SCALE (1.5).
+         */
+        scale?: number;
+        /**
+         * Max pages converted by the image fallback (#297). Pages beyond this are
+         * not sent to the model at all. Defaults to PDF_LIMITS.DEFAULT_MAX_PAGES (20).
+         */
+        maxPages?: number;
     };
     videoOptions?: {
         /** Frames to extract. Unset lets VideoProcessor pick from the clip's duration; clamped to 100. */

package/dist/utils/messageBuilder.js

@@ -4,6 +4,7 @@ import { getGlobalDispatcher, interceptors, request } from "undici";
 import { MultimodalLogger, ProviderImageAdapter, } from "../adapters/providerImageAdapter.js";
 import { CONVERSATION_INSTRUCTIONS, STRUCTURED_OUTPUT_INSTRUCTIONS, } from "../config/conversationMemory.js";
 import { getAvailableInputTokens } from "../constants/contextWindows.js";
+import { PDF_LIMITS } from "../core/constants.js";
 import { enforceAggregateFileBudget, FILE_READ_BUDGET_PERCENT, } from "../context/fileTokenBudget.js";
 import { isCSVContent, SIZE_TIER_THRESHOLDS } from "../types/index.js";
 import { tracers, ATTR, withSpan } from "../telemetry/index.js";
@@ -1150,6 +1151,10 @@ async function processExplicitPdfFiles(options, maxSize, provider) {
                     // #260: carry the per-page canvas-pixel ceiling so the caller can
                     // raise (or lower) the memory guard for the image-fallback render.
                     maxCanvasPixels: options.pdfOptions?.maxCanvasPixels,
+                    // #297: render scale / page ceiling, so the lowered default is
+                    // actually reachable and callers can trade sharpness for memory.
+                    scale: options.pdfOptions?.scale,
+                    maxPages: options.pdfOptions?.maxPages,
                 });
                 logger.info(`[PDF] ✅ Queued for multimodal: ${filename} (${result.metadata?.estimatedPages ?? "unknown"} pages)`);
             }
@@ -1458,6 +1463,8 @@ async function convertContentToProviderFormat(content, provider, _model, pdfOpti
         // guard as the `input.pdfFiles` path (see `processExplicitPdfFiles`).
         password: pdfOptions?.password,
         maxCanvasPixels: pdfOptions?.maxCanvasPixels,
+        scale: pdfOptions?.scale,
+        maxPages: pdfOptions?.maxPages,
     }));
     // #309: same aggregate ceiling as `input.pdfFiles`. Without this, moving an
     // over-limit payload from `input.pdfFiles` to `input.content` skipped the
@@ -1809,7 +1816,10 @@ async function convertSimpleImagesToProviderFormat(text, images, provider, _mode
 /**
  * Convert multimodal content (images + PDFs) to provider format
  */
-async function convertMultimodalToProviderFormat(text, images, pdfFiles, provider, model) {
+async function convertMultimodalToProviderFormat(text, images, 
+// The canonical entry shape (#309) rather than a fourth copy of it inline —
+// which is what let the render knobs stop short of this function.
+pdfFiles, provider, model) {
     const content = [
         { type: "text", text },
     ];
@@ -1843,14 +1853,41 @@ async function convertMultimodalToProviderFormat(text, images, pdfFiles, provide
         logger.info(`[PDF→Image] Provider ${provider} doesn't support native PDF. Converting ${pdfFiles.length} PDF(s) to images...`);
         for (const pdf of pdfFiles) {
             try {
+                const effectiveMaxPages = pdf.maxPages ?? PDF_LIMITS.DEFAULT_MAX_PAGES;
                 const conversionResult = await PDFImageConverter.convertToImages(pdf.buffer, {
-                    scale: 2.0, // High quality for OCR/analysis
-                    maxPages: 20, // Limit pages to prevent token overflow
+                    // #297: this is the only PDF→image call the product actually makes,
+                    // and it used to hardcode scale 2.0 — silently overriding the
+                    // lowered PDF_LIMITS.DEFAULT_SCALE and keeping the memory cost the
+                    // issue reports (a 100-page render at 2.0 is ~776MB; 1.5 is ~44%
+                    // fewer pixels per page). Callers can raise it back per request.
+                    scale: pdf.scale ?? PDF_LIMITS.DEFAULT_SCALE,
+                    // Page ceiling guards token overflow; also now caller-adjustable
+                    // rather than a constant nothing could reach.
+                    maxPages: effectiveMaxPages,
                     ...(pdf.password ? { password: pdf.password } : {}), // #258
                     ...(pdf.maxCanvasPixels
                         ? { maxCanvasPixels: pdf.maxCanvasPixels }
                         : {}), // #260
                 });
+                // The renderer stops at maxPages, so a longer document is silently
+                // truncated — say so rather than letting the model answer from a
+                // partial document as though it had the whole thing.
+                //
+                // Keyed on the cap being reached, not on pdf.pageCount: that field is
+                // null whenever `input.content` omits `metadata.pages`, which is the
+                // common case, so a page-count comparison would simply never fire
+                // there. Reaching the cap is also unambiguous — a short count caused by
+                // per-page render failures (#294 isolates those into `errors`) would
+                // otherwise be misreported as a maxPages truncation.
+                if (conversionResult.pageCount >= effectiveMaxPages) {
+                    logger.warn(`[PDF→Image] ${safeBasename(pdf.filename)} hit the ${effectiveMaxPages}-page ` +
+                        `conversion limit. Any pages beyond that were not sent — the model may be ` +
+                        `answering from a partial document. Raise pdfOptions.maxPages or split the file.`);
+                }
+                if (conversionResult.errors && conversionResult.errors.length > 0) {
+                    logger.warn(`[PDF→Image] ${safeBasename(pdf.filename)}: ${conversionResult.errors.length} page(s) ` +
+                        `failed to render and were omitted (page ${conversionResult.errors.map((e) => e.page).join(", ")}).`);
+                }
                 logger.info(`[PDF→Image] ✅ Converted ${pdf.filename}: ${conversionResult.pageCount} page(s) → images`);
                 // Add each page as an ImagePart (raw base64, not data: URI — see SSRF note above)
                 conversionResult.images.forEach((base64Image, pageIndex) => {

package/dist/utils/multimodalOptionsBuilder.d.ts

@@ -58,6 +58,8 @@ export declare function buildMultimodalOptions(options: StreamOptions, providerN
     pdfOptions: {
         password?: string;
         maxCanvasPixels?: number;
+        scale?: number;
+        maxPages?: number;
     } | undefined;
     systemPrompt: string | undefined;
     conversationHistory: import("../index.js").ChatMessage[] | undefined;

package/dist/utils/pdfProcessor.js

@@ -377,6 +377,7 @@ export class PDFProcessor {
             format,
             scale,
             maxCanvasPixels,
+            maxPages,
         });
         logger.debug("[PDF→Image] ✅ PDF validation passed", {
             bufferSize: pdfBuffer.length,
@@ -535,6 +536,14 @@ export class PDFProcessor {
         if (!Number.isFinite(opts.maxCanvasPixels) || opts.maxCanvasPixels <= 0) {
             throw new Error(`Invalid maxCanvasPixels: ${opts.maxCanvasPixels}. Must be a finite number greater than 0.`);
         }
+        // #297: maxPages became caller-controlled, and an unvalidated 0/-1/NaN
+        // silently converts nothing, surfacing later as a misleading
+        // "PDF has 0 pages" from deep inside the renderer. Reject it here where
+        // the message can still name the offending option.
+        if (opts.maxPages !== undefined &&
+            (!Number.isInteger(opts.maxPages) || opts.maxPages < 1)) {
+            throw new Error(`Invalid maxPages: ${opts.maxPages}. Must be a whole number of at least 1.`);
+        }
         if (!pdfBuffer || pdfBuffer.length < 5) {
             throw new Error("Invalid PDF: Buffer is too small or empty. " +
                 "A valid PDF must be at least 5 bytes (PDF header).");
@@ -570,6 +579,7 @@ export class PDFProcessor {
             format,
             scale,
             maxCanvasPixels,
+            maxPages,
         });
         const pdfToImgModule = await import("pdf-to-img");
         const pdf = pdfToImgModule.pdf;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@juspay/neurolink",
-  "version": "10.10.8",
+  "version": "10.10.9",
   "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": {