get the benchmarks working

Author: dboskovic

.gitignore

@@ -4,3 +4,4 @@ node_modules/
 package-lock.json
 yarn.lock
 dist/
+benchmark-results/

src/ci/performance-benchmark.ts

@@ -162,7 +162,6 @@ export class PerformanceBenchmark {
     testData: TestData,
     parser: any,
   ): Promise<BenchmarkResult> {
-    // Similar to benchmarkLegacy but for modern implementation
     const memBefore = process.memoryUsage();
     let memPeak = memBefore;
 
@@ -176,44 +175,45 @@ export class PerformanceBenchmark {
     const startTime = Date.now();
     let errorCount = 0;
 
-    try {
-      const result = parser.parse(testData.csvContent, {
-        header: true,
-        dynamicTyping: true,
-        complete: (results: any) => {
-          // Results handling
-        },
-        error: () => {
-          errorCount++;
-        },
-      });
-
-      const endTime = Date.now();
-      clearInterval(memoryMonitor);
-      const memAfter = process.memoryUsage();
+    return new Promise((resolve, reject) => {
+      try {
+        const result = parser.parse(testData.csvContent, {
+          header: true,
+          dynamicTyping: true,
+          complete: (results: any) => {
+            const endTime = Date.now();
+            clearInterval(memoryMonitor);
+            const memAfter = process.memoryUsage();
 
-      const totalTime = endTime - startTime;
-      const rowsPerSecond = Math.round(
-        (testData.expectedRows / totalTime) * 1000,
-      );
+            const totalTime = endTime - startTime;
+            const rowsPerSecond = Math.round(
+              (testData.expectedRows / totalTime) * 1000,
+            );
 
-      return {
-        implementation: "modern",
-        testName: testData.name,
-        rowsPerSecond,
-        totalRows: testData.expectedRows,
-        totalTime,
-        memoryUsage: {
-          before: memBefore,
-          after: memAfter,
-          peak: memPeak,
-        },
-        errors: errorCount,
-      };
-    } catch (error) {
-      clearInterval(memoryMonitor);
-      throw error;
-    }
+            resolve({
+              implementation: "modern",
+              testName: testData.name,
+              rowsPerSecond,
+              totalRows: testData.expectedRows,
+              totalTime,
+              memoryUsage: {
+                before: memBefore,
+                after: memAfter,
+                peak: memPeak,
+              },
+              errors: errorCount,
+            });
+          },
+          error: (error: any) => {
+            errorCount++;
+            console.warn(`Modern parsing error: ${error.message}`);
+          },
+        });
+      } catch (error) {
+        clearInterval(memoryMonitor);
+        reject(error);
+      }
+    });
   }
 
   /**
@@ -345,13 +345,19 @@ export async function runCIBenchmark(): Promise<void> {
   try {
     // Import both implementations
     const legacyPapa = require("../../legacy/papaparse.js");
-    const modernPapa = require("../../dist/papaparse.js");
+    const modernPapaModule = require("../../dist/papaparse.js");
+    const modernPapa = modernPapaModule.default || modernPapaModule;
 
-    console.log("📊 Running performance comparison between legacy and V6 implementations...");
+    console.log(
+      "📊 Running performance comparison between legacy and V6 implementations...",
+    );
 
     // Run the benchmark suite
-    const { results, summary } = await benchmark.runBenchmarkSuite(legacyPapa, modernPapa);
-    
+    const { results, summary } = await benchmark.runBenchmarkSuite(
+      legacyPapa,
+      modernPapa,
+    );
+
     // Export results for analysis
     benchmark.exportResults(results, `benchmark-${Date.now()}.json`);
 
@@ -360,11 +366,17 @@ export async function runCIBenchmark(): Promise<void> {
     console.log(`  Total Tests: ${summary.totalTests}`);
     console.log(`  Passed: ${summary.passedTests}`);
     console.log(`  Failed: ${summary.failedTests}`);
-    console.log(`  Average Performance Ratio: ${(summary.avgPerformanceRatio * 100).toFixed(1)}%`);
-    console.log(`  Average Memory Ratio: ${(summary.avgMemoryRatio * 100).toFixed(1)}%`);
+    console.log(
+      `  Average Performance Ratio: ${(summary.avgPerformanceRatio * 100).toFixed(1)}%`,
+    );
+    console.log(
+      `  Average Memory Ratio: ${(summary.avgMemoryRatio * 100).toFixed(1)}%`,
+    );
 
     if (!summary.passed) {
-      console.error("❌ Performance benchmarks failed - some tests did not meet thresholds");
+      console.error(
+        "❌ Performance benchmarks failed - some tests did not meet thresholds",
+      );
       process.exit(1);
     }
 

src/ci/snapshots/comments.json

@@ -35,6 +35,6 @@
     }
   },
   "checksum": "0423b324abe92cf93c11d33acb1bac7b7cfb1bce204b383b323aaa274bea2851",
-  "timestamp": "2025-07-29T07:00:38.688Z",
+  "timestamp": "2025-07-29T07:02:58.279Z",
   "version": "1.0.0"
 }

src/ci/snapshots/custom_delimiter.json

@@ -38,6 +38,6 @@
     }
   },
   "checksum": "0423b324abe92cf93c11d33acb1bac7b7cfb1bce204b383b323aaa274bea2851",
-  "timestamp": "2025-07-29T07:00:38.688Z",
+  "timestamp": "2025-07-29T07:02:58.280Z",
   "version": "1.0.0"
 }

src/ci/snapshots/dynamic_typing.json

@@ -40,6 +40,6 @@
     }
   },
   "checksum": "0423b324abe92cf93c11d33acb1bac7b7cfb1bce204b383b323aaa274bea2851",
-  "timestamp": "2025-07-29T07:00:38.686Z",
+  "timestamp": "2025-07-29T07:02:58.278Z",
   "version": "1.0.0"
 }

src/ci/snapshots/edge_case_empty.json

@@ -26,6 +26,6 @@
     }
   },
   "checksum": "58b959bd26bfc22602568dcd31559a1feed2c1952eead50c83bd45300b258a9a",
-  "timestamp": "2025-07-29T07:00:38.688Z",
+  "timestamp": "2025-07-29T07:02:58.280Z",
   "version": "1.0.0"
 }

src/ci/snapshots/empty_fields.json

@@ -42,6 +42,6 @@
     }
   },
   "checksum": "4fee89ca69e645265256377f3546f0ecbf3e4ccc3e4df5304692bdec340aab0f",
-  "timestamp": "2025-07-29T07:00:38.686Z",
+  "timestamp": "2025-07-29T07:02:58.279Z",
   "version": "1.0.0"
 }

src/ci/snapshots/line_breaks_in_quotes.json

@@ -34,6 +34,6 @@
     }
   },
   "checksum": "0423b324abe92cf93c11d33acb1bac7b7cfb1bce204b383b323aaa274bea2851",
-  "timestamp": "2025-07-29T07:00:38.687Z",
+  "timestamp": "2025-07-29T07:02:58.279Z",
   "version": "1.0.0"
 }

src/ci/snapshots/quoted_fields.json

@@ -34,6 +34,6 @@
     }
   },
   "checksum": "0423b324abe92cf93c11d33acb1bac7b7cfb1bce204b383b323aaa274bea2851",
-  "timestamp": "2025-07-29T07:00:38.686Z",
+  "timestamp": "2025-07-29T07:02:58.279Z",
   "version": "1.0.0"
 }

src/ci/snapshots/simple_csv.json

@@ -37,6 +37,6 @@
     }
   },
   "checksum": "0423b324abe92cf93c11d33acb1bac7b7cfb1bce204b383b323aaa274bea2851",
-  "timestamp": "2025-07-29T07:00:38.685Z",
+  "timestamp": "2025-07-29T07:02:58.278Z",
   "version": "1.0.0"
 }