@juspay/neurolink 9.88.7 → 9.88.8

package/dist/lib/utils/csvProcessor.js

@@ -439,6 +439,33 @@ function detectHasHeaders(headerValues, dataRows) {
  * - Lines with significantly different delimiter count than line 2
  * - Lines that don't match CSV structure of subsequent lines
  */
+/** Strip a leading UTF-8 BOM (U+FEFF) if present. */
+function stripBom(text) {
+    return text.charCodeAt(0) === 0xfeff ? text.slice(1) : text;
+}
+/**
+ * Count commas that fall OUTSIDE double-quoted fields (RFC 4180, escaped-quote
+ * aware). A quoted comma is field content, not a column separator, so counting
+ * naively would misclassify a quoted-comma header line.
+ */
+function countUnquotedCommas(line) {
+    let count = 0;
+    let inQuotes = false;
+    for (let i = 0; i < line.length; i++) {
+        const ch = line[i];
+        if (ch === '"') {
+            if (inQuotes && line[i + 1] === '"') {
+                i++;
+                continue;
+            }
+            inQuotes = !inQuotes;
+        }
+        else if (ch === "," && !inQuotes) {
+            count++;
+        }
+    }
+    return count;
+}
 function isMetadataLine(lines) {
     if (!lines[0] || lines.length < 2) {
         return false;
@@ -448,8 +475,8 @@ function isMetadataLine(lines) {
     if (firstLine.match(/^sep=/i)) {
         return true;
     }
-    const firstCommaCount = (firstLine.match(/,/g) || []).length;
-    const secondCommaCount = (secondLine.match(/,/g) || []).length;
+    const firstCommaCount = countUnquotedCommas(firstLine);
+    const secondCommaCount = countUnquotedCommas(secondLine);
     if (firstCommaCount === 0 && secondCommaCount > 0) {
         return true;
     }
@@ -690,6 +717,9 @@ export class CSVProcessor {
      * @returns Array of row objects
      */
     static async parseCSVFile(filePath, maxRows = 1000) {
+        if (typeof filePath !== "string" || filePath.trim().length === 0) {
+            throw new Error("CSVProcessor.parseCSVFile: filePath must be a non-empty string");
+        }
         const clampedMaxRows = Math.max(1, Math.min(10000, maxRows));
         const fs = await import("fs");
         logger.debug("[CSVProcessor] Starting file parsing", {
@@ -715,6 +745,8 @@ export class CSVProcessor {
                 }
             });
             lineReader.on("end", () => resolve());
+            // Metadata sniffing is best-effort — a read error here must not crash.
+            lineReader.on("error", () => resolve());
         });
         await fileHandle.close();
         const hasMetadataLine = isMetadataLine(firstLines);
@@ -732,6 +764,12 @@ export class CSVProcessor {
                 source.destroy();
                 parser.destroy();
             };
+            // .pipe() does not forward source errors (disk read/permission failures)
+            // to the parser, so listen on the source directly or it throws unhandled.
+            source.on("error", (error) => {
+                abort();
+                reject(new Error(`[CSVProcessor] CSV file read failed after ${count} row(s) (${filePath}): ${error.message}`));
+            });
             source
                 .pipe(parser)
                 .on("data", (row) => {
@@ -753,7 +791,7 @@ export class CSVProcessor {
             })
                 .on("error", (error) => {
                 logger.error("[CSVProcessor] File parsing failed:", error);
-                reject(error);
+                reject(new Error(`[CSVProcessor] CSV parsing failed after ${count} row(s): ${error.message}`));
             });
         });
     }
@@ -766,13 +804,19 @@ export class CSVProcessor {
      * @returns Array of row objects
      */
     static async parseCSVString(csvString, maxRows = 1000) {
+        if (typeof csvString !== "string" || csvString.trim().length === 0) {
+            throw new Error("CSVProcessor.parseCSVString: csvString must be a non-empty string");
+        }
+        // Strip a leading UTF-8 BOM (common in Excel exports) so it does not glue
+        // onto the first column name and break downstream key lookups.
+        const normalized = stripBom(csvString);
         const clampedMaxRows = Math.max(1, Math.min(10000, maxRows));
         logger.debug("[CSVProcessor] Starting string parsing", {
-            inputLength: csvString.length,
+            inputLength: normalized.length,
             maxRows: clampedMaxRows,
         });
         // Detect and skip metadata line
-        const lines = splitCsvLines(csvString);
+        const lines = splitCsvLines(normalized);
         const hasMetadataLine = isMetadataLine(lines);
         const csvData = hasMetadataLine
             ? lines.slice(1).join("\n")
@@ -789,6 +833,13 @@ export class CSVProcessor {
                 source.destroy();
                 parser.destroy();
             };
+            // .pipe() does not forward source-stream errors to the destination, so a
+            // source failure must be listened for directly or it throws as an
+            // unhandled EventEmitter error and can crash the process.
+            source.on("error", (error) => {
+                abort();
+                reject(new Error(`[CSVProcessor] CSV source stream failed after ${count} row(s): ${error.message}`));
+            });
             source
                 .pipe(parser)
                 .on("data", (row) => {
@@ -806,7 +857,7 @@ export class CSVProcessor {
             })
                 .on("error", (error) => {
                 logger.error("[CSVProcessor] Parsing failed:", error);
-                reject(error);
+                reject(new Error(`[CSVProcessor] CSV parsing failed after ${count} row(s): ${error.message}`));
             });
         });
     }

package/dist/utils/csvProcessor.js

@@ -439,6 +439,33 @@ function detectHasHeaders(headerValues, dataRows) {
  * - Lines with significantly different delimiter count than line 2
  * - Lines that don't match CSV structure of subsequent lines
  */
+/** Strip a leading UTF-8 BOM (U+FEFF) if present. */
+function stripBom(text) {
+    return text.charCodeAt(0) === 0xfeff ? text.slice(1) : text;
+}
+/**
+ * Count commas that fall OUTSIDE double-quoted fields (RFC 4180, escaped-quote
+ * aware). A quoted comma is field content, not a column separator, so counting
+ * naively would misclassify a quoted-comma header line.
+ */
+function countUnquotedCommas(line) {
+    let count = 0;
+    let inQuotes = false;
+    for (let i = 0; i < line.length; i++) {
+        const ch = line[i];
+        if (ch === '"') {
+            if (inQuotes && line[i + 1] === '"') {
+                i++;
+                continue;
+            }
+            inQuotes = !inQuotes;
+        }
+        else if (ch === "," && !inQuotes) {
+            count++;
+        }
+    }
+    return count;
+}
 function isMetadataLine(lines) {
     if (!lines[0] || lines.length < 2) {
         return false;
@@ -448,8 +475,8 @@ function isMetadataLine(lines) {
     if (firstLine.match(/^sep=/i)) {
         return true;
     }
-    const firstCommaCount = (firstLine.match(/,/g) || []).length;
-    const secondCommaCount = (secondLine.match(/,/g) || []).length;
+    const firstCommaCount = countUnquotedCommas(firstLine);
+    const secondCommaCount = countUnquotedCommas(secondLine);
     if (firstCommaCount === 0 && secondCommaCount > 0) {
         return true;
     }
@@ -690,6 +717,9 @@ export class CSVProcessor {
      * @returns Array of row objects
      */
     static async parseCSVFile(filePath, maxRows = 1000) {
+        if (typeof filePath !== "string" || filePath.trim().length === 0) {
+            throw new Error("CSVProcessor.parseCSVFile: filePath must be a non-empty string");
+        }
         const clampedMaxRows = Math.max(1, Math.min(10000, maxRows));
         const fs = await import("fs");
         logger.debug("[CSVProcessor] Starting file parsing", {
@@ -715,6 +745,8 @@ export class CSVProcessor {
                 }
             });
             lineReader.on("end", () => resolve());
+            // Metadata sniffing is best-effort — a read error here must not crash.
+            lineReader.on("error", () => resolve());
         });
         await fileHandle.close();
         const hasMetadataLine = isMetadataLine(firstLines);
@@ -732,6 +764,12 @@ export class CSVProcessor {
                 source.destroy();
                 parser.destroy();
             };
+            // .pipe() does not forward source errors (disk read/permission failures)
+            // to the parser, so listen on the source directly or it throws unhandled.
+            source.on("error", (error) => {
+                abort();
+                reject(new Error(`[CSVProcessor] CSV file read failed after ${count} row(s) (${filePath}): ${error.message}`));
+            });
             source
                 .pipe(parser)
                 .on("data", (row) => {
@@ -753,7 +791,7 @@ export class CSVProcessor {
             })
                 .on("error", (error) => {
                 logger.error("[CSVProcessor] File parsing failed:", error);
-                reject(error);
+                reject(new Error(`[CSVProcessor] CSV parsing failed after ${count} row(s): ${error.message}`));
             });
         });
     }
@@ -766,13 +804,19 @@ export class CSVProcessor {
      * @returns Array of row objects
      */
     static async parseCSVString(csvString, maxRows = 1000) {
+        if (typeof csvString !== "string" || csvString.trim().length === 0) {
+            throw new Error("CSVProcessor.parseCSVString: csvString must be a non-empty string");
+        }
+        // Strip a leading UTF-8 BOM (common in Excel exports) so it does not glue
+        // onto the first column name and break downstream key lookups.
+        const normalized = stripBom(csvString);
         const clampedMaxRows = Math.max(1, Math.min(10000, maxRows));
         logger.debug("[CSVProcessor] Starting string parsing", {
-            inputLength: csvString.length,
+            inputLength: normalized.length,
             maxRows: clampedMaxRows,
         });
         // Detect and skip metadata line
-        const lines = splitCsvLines(csvString);
+        const lines = splitCsvLines(normalized);
         const hasMetadataLine = isMetadataLine(lines);
         const csvData = hasMetadataLine
             ? lines.slice(1).join("\n")
@@ -789,6 +833,13 @@ export class CSVProcessor {
                 source.destroy();
                 parser.destroy();
             };
+            // .pipe() does not forward source-stream errors to the destination, so a
+            // source failure must be listened for directly or it throws as an
+            // unhandled EventEmitter error and can crash the process.
+            source.on("error", (error) => {
+                abort();
+                reject(new Error(`[CSVProcessor] CSV source stream failed after ${count} row(s): ${error.message}`));
+            });
             source
                 .pipe(parser)
                 .on("data", (row) => {
@@ -806,7 +857,7 @@ export class CSVProcessor {
             })
                 .on("error", (error) => {
                 logger.error("[CSVProcessor] Parsing failed:", error);
-                reject(error);
+                reject(new Error(`[CSVProcessor] CSV parsing failed after ${count} row(s): ${error.message}`));
             });
         });
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@juspay/neurolink",
-  "version": "9.88.7",
+  "version": "9.88.8",
   "packageManager": "pnpm@10.15.1",
   "description": "TypeScript AI SDK with 24+ LLM providers behind one consistent API. MCP-native (58+ servers), voice TTS/STT/realtime, RAG, agents, memory, context compaction. OpenAI · Anthropic · Gemini · Bedrock · Azure · Ollama · DeepSeek · NVIDIA NIM and more.",
   "author": {