@heripo/pdf-parser 0.1.10 → 0.1.11

package/dist/index.js

@@ -1301,14 +1301,18 @@ var ImageExtractor = class _ImageExtractor {
 // src/processors/page-renderer.ts
 import { existsSync as existsSync2, mkdirSync as mkdirSync2, readdirSync as readdirSync2 } from "fs";
 import { join as join3 } from "path";
-var PROGRESS_POLL_INTERVAL_MS = 2e3;
+var PROGRESS_LOG_PERCENT_STEP = 10;
 var PageRenderer = class {
   constructor(logger) {
     this.logger = logger;
   }
+  lastLoggedPercent = 0;
   /**
    * Render all pages of a PDF to individual PNG files.
    *
+   * Uses per-page rendering (`magick 'input.pdf[N]'`) when page count is known,
+   * limiting peak memory to ~15MB/page instead of loading all pages at once.
+   *
    * @param pdfPath - Absolute path to the source PDF file
    * @param outputDir - Directory where pages/ subdirectory will be created
    * @param options - Rendering options
@@ -1325,50 +1329,54 @@ var PageRenderer = class {
       this.logger.info(
         `[PageRenderer] Rendering ${totalPages} pages at ${dpi} DPI...`
       );
+      this.lastLoggedPercent = 0;
+      for (let i = 0; i < totalPages; i++) {
+        const result = await spawnAsync(
+          "magick",
+          [
+            "-density",
+            dpi.toString(),
+            `${pdfPath}[${i}]`,
+            "-background",
+            "white",
+            "-alpha",
+            "remove",
+            "-alpha",
+            "off",
+            join3(pagesDir, `page_${i}.png`)
+          ],
+          { captureStdout: false }
+        );
+        if (result.code !== 0) {
+          throw new Error(
+            `[PageRenderer] Failed to render page ${i + 1}/${totalPages}: ${result.stderr || "Unknown error"}`
+          );
+        }
+        this.logProgress(i + 1, totalPages);
+      }
     } else {
       this.logger.info(`[PageRenderer] Rendering PDF at ${dpi} DPI...`);
-    }
-    const outputPattern = join3(pagesDir, "page_%d.png");
-    let progressInterval = null;
-    if (totalPages > 0) {
-      let lastLoggedCount = 0;
-      progressInterval = setInterval(() => {
-        try {
-          const rendered = readdirSync2(pagesDir).filter(
-            (f) => f.startsWith("page_") && f.endsWith(".png")
-          ).length;
-          if (rendered > 0 && rendered !== lastLoggedCount) {
-            lastLoggedCount = rendered;
-            this.logger.info(
-              `[PageRenderer] Rendering pages: ${rendered}/${totalPages}`
-            );
-          }
-        } catch {
-        }
-      }, PROGRESS_POLL_INTERVAL_MS);
-    }
-    try {
-      const result = await spawnAsync("magick", [
-        "-density",
-        dpi.toString(),
-        pdfPath,
-        "-background",
-        "white",
-        "-alpha",
-        "remove",
-        "-alpha",
-        "off",
-        outputPattern
-      ]);
+      const result = await spawnAsync(
+        "magick",
+        [
+          "-density",
+          dpi.toString(),
+          pdfPath,
+          "-background",
+          "white",
+          "-alpha",
+          "remove",
+          "-alpha",
+          "off",
+          join3(pagesDir, "page_%d.png")
+        ],
+        { captureStdout: false }
+      );
       if (result.code !== 0) {
         throw new Error(
           `[PageRenderer] Failed to render PDF pages: ${result.stderr || "Unknown error"}`
         );
       }
-    } finally {
-      if (progressInterval) {
-        clearInterval(progressInterval);
-      }
     }
     const pageFiles = readdirSync2(pagesDir).filter((f) => f.startsWith("page_") && f.endsWith(".png")).sort((a, b) => {
       const numA = parseInt(a.replace("page_", "").replace(".png", ""), 10);
@@ -1384,6 +1392,18 @@ var PageRenderer = class {
       pageFiles
     };
   }
+  /**
+   * Log rendering progress at appropriate intervals (every 10%).
+   */
+  logProgress(current, total) {
+    const percent = Math.floor(current / total * 100);
+    if (percent >= this.lastLoggedPercent + PROGRESS_LOG_PERCENT_STEP || current === total) {
+      this.lastLoggedPercent = percent;
+      this.logger.info(
+        `[PageRenderer] Rendering pages: ${current}/${total} (${percent}%)`
+      );
+    }
+  }
   /**
    * Get total page count using pdfinfo.
    * Returns 0 on failure (progress logging will be skipped).