n8n-nodes-pdfbro 0.1.13 → 0.1.15

package/dist/nodes/PdfBro/PdfBro.node.js

@@ -198,34 +198,78 @@ class PdfBro {
             }
             return Array.from(pages).sort((a, b) => a - b);
         };
-        for (let i = 0; i < items.length; i++) {
+        // Handle merge operation separately - it needs all items at once
+        if (operation === 'merge') {
             try {
-                if (operation === 'merge') {
-                    const mergedPdf = await PDFDocument.create();
-                    // Get fixed collection
-                    // @ts-ignore
-                    const binaries = ((_a = this.getNodeParameter('inputBinaries', i)) === null || _a === void 0 ? void 0 : _a.files) || [];
-                    // If user provided no binaries, just skip
-                    if (binaries.length === 0)
+                const mergedPdf = await PDFDocument.create();
+                // Get fixed collection from first item
+                // @ts-ignore
+                const binaries = ((_a = this.getNodeParameter('inputBinaries', 0)) === null || _a === void 0 ? void 0 : _a.files) || [];
+                // Collect all PDFs from all input items
+                let pdfCount = 0;
+                // First, try to get PDFs from the specified binary property names across ALL items
+                for (let i = 0; i < items.length; i++) {
+                    const itemBinary = items[i].binary;
+                    if (!itemBinary)
                         continue;
-                    for (const entry of binaries) {
-                        const propName = entry.binaryPropertyName;
-                        if (this.helpers.assertBinaryData(i, propName)) {
-                            const validBuffer = await this.helpers.getBinaryDataBuffer(i, propName);
-                            const pdf = await PDFDocument.load(validBuffer);
-                            const copiedPages = await mergedPdf.copyPages(pdf, pdf.getPageIndices());
-                            copiedPages.forEach((page) => mergedPdf.addPage(page));
+                    // If binaries array is specified, look for those specific properties
+                    if (binaries.length > 0) {
+                        for (const entry of binaries) {
+                            const propName = entry.binaryPropertyName;
+                            if (itemBinary[propName]) {
+                                try {
+                                    const validBuffer = await this.helpers.getBinaryDataBuffer(i, propName);
+                                    const pdf = await PDFDocument.load(validBuffer);
+                                    const copiedPages = await mergedPdf.copyPages(pdf, pdf.getPageIndices());
+                                    copiedPages.forEach((page) => mergedPdf.addPage(page));
+                                    pdfCount++;
+                                }
+                                catch (e) {
+                                    // Skip invalid PDFs
+                                }
+                            }
                         }
                     }
-                    const mergedPdfBuffer = await mergedPdf.save();
-                    returnData.push({
-                        json: { success: true, pageCount: mergedPdf.getPageCount() },
-                        binary: {
-                            data: await this.helpers.prepareBinaryData(Buffer.from(mergedPdfBuffer), 'merged.pdf', 'application/pdf'),
-                        },
-                    });
+                    else {
+                        // If no specific binaries specified, try all binary properties in the item
+                        for (const propName of Object.keys(itemBinary)) {
+                            try {
+                                const validBuffer = await this.helpers.getBinaryDataBuffer(i, propName);
+                                const pdf = await PDFDocument.load(validBuffer);
+                                const copiedPages = await mergedPdf.copyPages(pdf, pdf.getPageIndices());
+                                copiedPages.forEach((page) => mergedPdf.addPage(page));
+                                pdfCount++;
+                            }
+                            catch (e) {
+                                // Skip invalid PDFs
+                            }
+                        }
+                    }
+                }
+                if (pdfCount === 0) {
+                    throw new Error('No valid PDF files found in input items');
                 }
-                else if (operation === 'split') {
+                const mergedPdfBuffer = await mergedPdf.save();
+                returnData.push({
+                    json: { success: true, pageCount: mergedPdf.getPageCount(), mergedFiles: pdfCount },
+                    binary: {
+                        data: await this.helpers.prepareBinaryData(Buffer.from(mergedPdfBuffer), 'merged.pdf', 'application/pdf'),
+                    },
+                });
+                return [returnData];
+            }
+            catch (error) {
+                if (this.continueOnFail()) {
+                    returnData.push({ json: { error: error.message } });
+                    return [returnData];
+                }
+                throw error;
+            }
+        }
+        // Handle other operations per-item
+        for (let i = 0; i < items.length; i++) {
+            try {
+                if (operation === 'split') {
                     const binaryPropertyName = this.getNodeParameter('binaryPropertyName', i);
                     const rangeStr = this.getNodeParameter('splitRange', i);
                     const validBuffer = await this.helpers.getBinaryDataBuffer(i, binaryPropertyName);
@@ -269,9 +313,22 @@ class PdfBro {
                     const htmlToPdfmake = require('html-to-pdfmake');
                     const pdfMake = require('pdfmake/build/pdfmake');
                     const pdfFonts = require('pdfmake/build/vfs_fonts');
-                    // Initialize fonts if not already done
+                    // Initialize fonts - handle different module structures
                     if (!pdfMake.vfs) {
-                        pdfMake.vfs = pdfFonts.pdfMake.vfs;
+                        // Try different possible structures of vfs_fonts module
+                        if (pdfFonts && pdfFonts.pdfMake && pdfFonts.pdfMake.vfs) {
+                            pdfMake.vfs = pdfFonts.pdfMake.vfs;
+                        }
+                        else if (pdfFonts && pdfFonts.vfs) {
+                            pdfMake.vfs = pdfFonts.vfs;
+                        }
+                        else if (pdfFonts && typeof pdfFonts === 'object') {
+                            // Some versions export vfs directly
+                            pdfMake.vfs = pdfFonts;
+                        }
+                        else {
+                            throw new Error('Could not load PDF fonts. Please ensure pdfmake is properly installed.');
+                        }
                     }
                     // convert html to pdfmake format (requires window)
                     const { window } = new JSDOM('');

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "n8n-nodes-pdfbro",
-    "version": "0.1.13",
+    "version": "0.1.15",
     "description": "Offline PDF utility node for n8n",
     "keywords": [
         "n8n-community-node"