n8n-nodes-seo-scanner 1.2.1 → 1.2.3

package/dist/SeoScanner.node.js

@@ -32,41 +32,90 @@ var __importStar = (this && this.__importStar) || (function () {
         return result;
     };
 })();
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.SeoScanner = void 0;
 const n8n_workflow_1 = require("n8n-workflow");
 const cheerio = __importStar(require("cheerio"));
 const dns = __importStar(require("dns"));
 const tls = __importStar(require("tls"));
+const puppeteer_1 = __importDefault(require("puppeteer"));
 const reportTemplate_1 = require("./reportTemplate");
 const dnsPromises = dns.promises;
 const USER_AGENT = 'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/120.0.0.0 Safari/537.36';
 const GOOGLEBOT_UA = 'Mozilla/5.0 (compatible; Googlebot/2.1; +http://www.google.com/bot.html)';
 async function fetchPage(url, timeoutMs, opts = {}) {
-    const { userAgent = USER_AGENT, followRedirects = true, acceptLanguage } = opts;
-    const controller = new AbortController();
-    const timeout = setTimeout(() => controller.abort(), timeoutMs);
+    const { userAgent = USER_AGENT, followRedirects = true, acceptLanguage, useJsRendering = false } = opts;
     const start = Date.now();
-    const headers = { 'User-Agent': userAgent };
-    if (acceptLanguage)
-        headers['Accept-Language'] = acceptLanguage;
-    try {
-        const res = await fetch(url, {
-            headers,
-            redirect: followRedirects ? 'follow' : 'manual',
-            signal: controller.signal,
-        });
-        const timeToFirstByteMs = Date.now() - start;
-        const html = await res.text();
-        clearTimeout(timeout);
-        const responseTimeMs = Date.now() - start;
-        const responseHeaders = {};
-        res.headers.forEach((value, key) => { responseHeaders[key.toLowerCase()] = value; });
-        return { html, finalUrl: res.url, statusCode: res.status, responseTimeMs, timeToFirstByteMs, responseHeaders };
+    if (useJsRendering) {
+        let browser;
+        try {
+            browser = await puppeteer_1.default.launch({
+                headless: true,
+                args: ['--no-sandbox', '--disable-setuid-sandbox', '--disable-dev-shm-usage'],
+            });
+            const page = await browser.newPage();
+            if (userAgent)
+                await page.setUserAgent(userAgent);
+            if (acceptLanguage)
+                await page.setExtraHTTPHeaders({ 'Accept-Language': acceptLanguage });
+            let timeToFirstByteMs = 0;
+            page.on('response', (response) => {
+                if (response.url() === page.url()) {
+                    timeToFirstByteMs = Date.now() - start;
+                }
+            });
+            const response = await page.goto(url, {
+                waitUntil: 'networkidle2',
+                timeout: timeoutMs,
+            });
+            if (!response) {
+                throw new Error('No se recibió respuesta al renderizar la página.');
+            }
+            const html = await page.content();
+            const finalUrl = page.url();
+            const statusCode = response.status();
+            const responseTimeMs = Date.now() - start;
+            const headers = response.headers();
+            const responseHeaders = {};
+            for (const key in headers) {
+                responseHeaders[key.toLowerCase()] = headers[key];
+            }
+            await browser.close();
+            return { html, finalUrl, statusCode, responseTimeMs, timeToFirstByteMs: timeToFirstByteMs || responseTimeMs, responseHeaders };
+        }
+        catch (e) {
+            if (browser)
+                await browser.close();
+            throw e;
+        }
     }
-    catch (e) {
-        clearTimeout(timeout);
-        throw e;
+    else {
+        const controller = new AbortController();
+        const timeout = setTimeout(() => controller.abort(), timeoutMs);
+        const headers = { 'User-Agent': userAgent };
+        if (acceptLanguage)
+            headers['Accept-Language'] = acceptLanguage;
+        try {
+            const res = await fetch(url, {
+                headers,
+                redirect: followRedirects ? 'follow' : 'manual',
+                signal: controller.signal,
+            });
+            const timeToFirstByteMs = Date.now() - start;
+            const html = await res.text();
+            clearTimeout(timeout);
+            const responseTimeMs = Date.now() - start;
+            const responseHeaders = {};
+            res.headers.forEach((value, key) => { responseHeaders[key.toLowerCase()] = value; });
+            return { html, finalUrl: res.url, statusCode: res.status, responseTimeMs, timeToFirstByteMs, responseHeaders };
+        }
+        catch (e) {
+            clearTimeout(timeout);
+            throw e;
+        }
     }
 }
 function parseColorToRgb(color) {
@@ -2155,17 +2204,17 @@ ${reportTemplate_1.REPORT_CSS}
             html += `
         <details class="fallos-pagina-details" data-crit="${rcrit.length}" data-imp="${rimp.length}" data-warn="${rwarn.length}" data-info="${rinfo.length}" data-pass="${rpassed.length}">
           <summary class="fallos-pagina-summary">
-            <span class="fallos-pagina-label">${escapeHtml(label)}</span>
-            <div style="display:flex; height:6px; width:60px; background:var(--bg3); border-radius:3px; overflow:hidden; margin-right:12px;">
+            <span class="fallos-pagina-label" style="flex:1;min-width:0;overflow:hidden;text-overflow:ellipsis;white-space:nowrap">${escapeHtml(label)}</span>
+            <div style="display:flex; height:6px; width:60px; background:var(--bg3); border-radius:3px; overflow:hidden; flex-shrink:0">
               <div style="width:${pctCrit}%; background:var(--red);"></div>
               <div style="width:${pctImp}%; background:var(--orange);"></div>
               <div style="width:${pctWarn}%; background:var(--yellow);"></div>
               <div style="width:${pctInfo}%; background:var(--blue);"></div>
               <div style="width:${pctPass}%; background:var(--green);"></div>
             </div>
-			<span class="fallos-pagina-score ${scoreCls}">${sc}</span>
-            <span class="fallos-pagina-count">${totalFallos} fallos</span>
-            <button class="ignore-btn" title="Copiar URL para ignorarla y ocultarla de la lista" onclick="event.preventDefault(); window.ignorePage(this, '${attrEsc(label.split('·')[1]?.trim() || label)}');">Ignorar</button>
+			<span class="fallos-pagina-score ${scoreCls}" style="flex-shrink:0">${sc}</span>
+            <span class="fallos-pagina-count" style="flex-shrink:0">${totalFallos} fallos</span>
+            <button class="ignore-btn" style="margin:0;flex-shrink:0" title="Copiar URL para ignorarla y ocultarla de la lista" onclick="event.preventDefault(); window.ignorePage(this, '${attrEsc(label.split('·')[1]?.trim() || label)}');">Ignorar</button>
           </summary>
           <div class="fallos-pagina-body">`;
             [['Critical', rcrit, 'crit', 'tag-crit', 'CRIT'], ['Important', rimp, 'imp', 'tag-imp', 'IMP'], ['Warnings', rwarn, 'warn', 'tag-warn', 'WARN'], ['Info', rinfo, 'info', 'tag-info', 'INFO']].forEach(([title, arr, rowClass, tagCls, lbl]) => {
@@ -2288,16 +2337,16 @@ ${reportTemplate_1.REPORT_CSS}
                 const srcShort = img.src ? escapeHtml(img.src.replace(/^https?:\/\/[^/]+/, '').slice(0, 60)) : '—';
                 const dims = [img.width, img.height].filter(Boolean).join('×') || '—';
                 html += `
-            <details class="metric-url-row" style="margin:6px 0;padding:8px 10px;background:var(--bg2);border-radius:4px">
-              <summary style="cursor:pointer;font-family:var(--mono);font-size:0.72rem">
-                <span class="metric-url-idx">${String(i + 1).padStart(2, '0')}</span>
-                <span class="metric-v ${img.hasAlt ? 'ok' : ''}" style="margin-left:6px">${img.hasAlt ? 'alt ✓' : 'sin alt'}</span>
-                <span style="color:var(--text3);margin-left:6px">${srcShort}${img.src && img.src.length > 60 ? '…' : ''}</span>
+            <details class="metric-url-row" style="margin:6px 0;padding:8px 10px;background:var(--bg2);border-radius:4px;min-width:0">
+              <summary style="cursor:pointer;font-family:var(--mono);font-size:0.72rem;display:flex;align-items:center;min-width:0">
+                <span class="metric-url-idx" style="flex-shrink:0">${String(i + 1).padStart(2, '0')}</span>
+                <span class="metric-v ${img.hasAlt ? 'ok' : ''}" style="margin-left:6px;flex-shrink:0">${img.hasAlt ? 'alt ✓' : 'sin alt'}</span>
+                <span style="color:var(--text3);margin-left:6px;flex:1;min-width:0;overflow:hidden;text-overflow:ellipsis;white-space:nowrap">${srcShort}${img.src && img.src.length > 60 ? '…' : ''}</span>
               </summary>
-              <div style="margin-top:8px;padding-top:8px;border-top:1px solid var(--line);font-size:0.7rem;color:var(--text2)">
-                <div class="metric-row"><span class="metric-k">src</span><span style="word-break:break-all">${escapeHtml((img.src || '').slice(0, 120))}${(img.src || '').length > 120 ? '…' : ''}</span></div>
-                <div class="metric-row"><span class="metric-k">alt</span><span>${altStr}</span></div>
-                <div class="metric-row"><span class="metric-k">dimensions</span><span>${escapeHtml(dims)}</span></div>
+              <div style="margin-top:8px;padding-top:8px;border-top:1px solid var(--line);font-size:0.7rem;color:var(--text2);min-width:0">
+                <div class="metric-row"><span class="metric-k" style="flex-shrink:0;margin-right:8px">src</span><span style="word-break:break-all;flex:1;min-width:0">${escapeHtml((img.src || '').slice(0, 120))}${(img.src || '').length > 120 ? '…' : ''}</span></div>
+                <div class="metric-row"><span class="metric-k" style="flex-shrink:0;margin-right:8px">alt</span><span style="word-break:break-word;flex:1;min-width:0">${altStr}</span></div>
+                <div class="metric-row"><span class="metric-k" style="flex-shrink:0;margin-right:8px">dimensions</span><span style="flex:1;min-width:0">${escapeHtml(dims)}</span></div>
               </div>
             </details>`;
             });
@@ -2338,22 +2387,22 @@ ${reportTemplate_1.REPORT_CSS}
                 const hasMeta = hasInternalDetails && 'title' in item;
                 if (hasMeta) {
                     const linkItem = item;
-                    html += `<details class="metric-url-row" style="margin:4px 0">
-                  <summary style="cursor:pointer;display:flex;align-items:center;gap:6px">
-                    <span class="metric-url-idx">${String(i + 1).padStart(2, '0')}</span>
+                    html += `<details class="metric-url-row" style="margin:4px 0;min-width:0">
+                  <summary style="cursor:pointer;display:flex;align-items:center;gap:6px;min-width:0">
+                    <span class="metric-url-idx" style="flex-shrink:0">${String(i + 1).padStart(2, '0')}</span>
                     <a class="metric-url" href="${escapeHtml(item.url)}" target="_blank" rel="noopener" onclick="event.stopPropagation()">${escapeHtml(pathPart)}</a>
-                    <span class="metric-url-type">int</span>
-                    ${(linkItem.statusCode ?? 0) >= 400 ? '<span style="color:var(--red)">' + (linkItem.statusCode || '') + '</span>' : ''}
+                    <span class="metric-url-type" style="flex-shrink:0">int</span>
+                    ${(linkItem.statusCode ?? 0) >= 400 ? '<span style="color:var(--red);flex-shrink:0">' + (linkItem.statusCode || '') + '</span>' : ''}
                   </summary>
-                  <div style="margin:6px 0 6px 20px;padding:6px 8px;background:var(--bg2);border-radius:4px;font-size:0.7rem">
-                    ${linkItem.anchorText ? `<div class="metric-row"><span class="metric-k">anchor</span><span>${escapeHtml(linkItem.anchorText.slice(0, 80))}</span></div>` : ''}
-                    <div class="metric-row"><span class="metric-k">title</span><span>${linkItem.title ? escapeHtml(linkItem.title.slice(0, 100)) : '—'}</span></div>
-                    <div class="metric-row"><span class="metric-k">meta</span><span>${linkItem.metaDescription ? escapeHtml(linkItem.metaDescription.slice(0, 120)) + (linkItem.metaDescription.length > 120 ? '…' : '') : '—'}</span></div>
+                  <div style="margin:6px 0 6px 20px;padding:6px 8px;background:var(--bg2);border-radius:4px;font-size:0.7rem;min-width:0">
+                    ${linkItem.anchorText ? `<div class="metric-row"><span class="metric-k" style="flex-shrink:0;margin-right:8px">anchor</span><span style="flex:1;min-width:0;word-break:break-word">${escapeHtml(linkItem.anchorText.slice(0, 80))}</span></div>` : ''}
+                    <div class="metric-row"><span class="metric-k" style="flex-shrink:0;margin-right:8px">title</span><span style="flex:1;min-width:0;word-break:break-word">${linkItem.title ? escapeHtml(linkItem.title.slice(0, 100)) : '—'}</span></div>
+                    <div class="metric-row"><span class="metric-k" style="flex-shrink:0;margin-right:8px">meta</span><span style="flex:1;min-width:0;word-break:break-word">${linkItem.metaDescription ? escapeHtml(linkItem.metaDescription.slice(0, 120)) + (linkItem.metaDescription.length > 120 ? '…' : '') : '—'}</span></div>
                   </div>
                 </details>`;
                 }
                 else {
-                    html += `<div class="metric-url-row"><span class="metric-url-idx">${String(i + 1).padStart(2, '0')}</span><a class="metric-url" href="${escapeHtml(item.url)}" target="_blank" rel="noopener">${escapeHtml(pathPart)}</a><span class="metric-url-type">int</span></div>`;
+                    html += `<div class="metric-url-row" style="min-width:0"><span class="metric-url-idx" style="flex-shrink:0">${String(i + 1).padStart(2, '0')}</span><a class="metric-url" href="${escapeHtml(item.url)}" target="_blank" rel="noopener">${escapeHtml(pathPart)}</a><span class="metric-url-type" style="flex-shrink:0">int</span></div>`;
                 }
             });
             if (pIntLinks.length > maxShow)
@@ -2388,23 +2437,23 @@ ${reportTemplate_1.REPORT_CSS}
                 if (hasMeta) {
                     const linkItem = item;
                     const urlShort = linkItem.url.length > 50 ? linkItem.url.slice(0, 50) + '…' : linkItem.url;
-                    html += `<details class="metric-url-row" style="margin:4px 0">
-                  <summary style="cursor:pointer;display:flex;align-items:center;gap:6px;flex-wrap:wrap">
-                    <span class="metric-url-idx">${String(i + 1).padStart(2, '0')}</span>
+                    html += `<details class="metric-url-row" style="margin:4px 0;min-width:0">
+                  <summary style="cursor:pointer;display:flex;align-items:center;gap:6px;flex-wrap:wrap;min-width:0">
+                    <span class="metric-url-idx" style="flex-shrink:0">${String(i + 1).padStart(2, '0')}</span>
                     <a class="metric-url" href="${escapeHtml(linkItem.url)}" target="_blank" rel="noopener" onclick="event.stopPropagation()">${escapeHtml(urlShort)}</a>
-                    <span class="metric-url-type">ext</span>
-                    ${(linkItem.statusCode ?? 0) >= 400 ? '<span style="color:var(--red)">' + (linkItem.statusCode || '') + '</span>' : ''}
+                    <span class="metric-url-type" style="flex-shrink:0">ext</span>
+                    ${(linkItem.statusCode ?? 0) >= 400 ? '<span style="color:var(--red);flex-shrink:0">' + (linkItem.statusCode || '') + '</span>' : ''}
                   </summary>
-                  <div style="margin:6px 0 6px 20px;padding:6px 8px;background:var(--bg2);border-radius:4px;font-size:0.7rem">
-                    ${linkItem.anchorText ? `<div class="metric-row"><span class="metric-k">anchor</span><span>${escapeHtml(linkItem.anchorText.slice(0, 80))}</span></div>` : ''}
-                    <div class="metric-row"><span class="metric-k">title</span><span>${linkItem.title ? escapeHtml(linkItem.title.slice(0, 100)) : '—'}</span></div>
-                    <div class="metric-row"><span class="metric-k">meta</span><span>${linkItem.metaDescription ? escapeHtml(linkItem.metaDescription.slice(0, 120)) + (linkItem.metaDescription.length > 120 ? '…' : '') : '—'}</span></div>
+                  <div style="margin:6px 0 6px 20px;padding:6px 8px;background:var(--bg2);border-radius:4px;font-size:0.7rem;min-width:0">
+                    ${linkItem.anchorText ? `<div class="metric-row"><span class="metric-k" style="flex-shrink:0;margin-right:8px">anchor</span><span style="flex:1;min-width:0;word-break:break-word">${escapeHtml(linkItem.anchorText.slice(0, 80))}</span></div>` : ''}
+                    <div class="metric-row"><span class="metric-k" style="flex-shrink:0;margin-right:8px">title</span><span style="flex:1;min-width:0;word-break:break-word">${linkItem.title ? escapeHtml(linkItem.title.slice(0, 100)) : '—'}</span></div>
+                    <div class="metric-row"><span class="metric-k" style="flex-shrink:0;margin-right:8px">meta</span><span style="flex:1;min-width:0;word-break:break-word">${linkItem.metaDescription ? escapeHtml(linkItem.metaDescription.slice(0, 120)) + (linkItem.metaDescription.length > 120 ? '…' : '') : '—'}</span></div>
                   </div>
                 </details>`;
                 }
                 else {
                     const u = typeof item === 'object' && item && 'url' in item ? item.url : String(item);
-                    html += `<div class="metric-url-row"><span class="metric-url-idx">${String(i + 1).padStart(2, '0')}</span><a class="metric-url" href="${escapeHtml(u)}" target="_blank" rel="noopener">${escapeHtml(String(u).length > 50 ? String(u).slice(0, 50) + '…' : u)}</a><span class="metric-url-type">ext</span></div>`;
+                    html += `<div class="metric-url-row" style="min-width:0"><span class="metric-url-idx" style="flex-shrink:0">${String(i + 1).padStart(2, '0')}</span><a class="metric-url" href="${escapeHtml(u)}" target="_blank" rel="noopener">${escapeHtml(String(u).length > 50 ? String(u).slice(0, 50) + '…' : u)}</a><span class="metric-url-type" style="flex-shrink:0">ext</span></div>`;
                 }
             });
             if (pExtLinks.length > maxShow)
@@ -3297,6 +3346,13 @@ class SeoScanner {
                     default: false,
                     description: 'Genera un informe HTML (dashboard) con los fallos y recomendaciones. Salida: json.html y json.reportHtml. Para verlo en el navegador: conecta a Respond to Webhook (Respond With = Text, Response Body = {{ $json.html }}) y abre la URL del webhook.',
                 },
+                {
+                    displayName: 'Usar JavaScript Rendering',
+                    name: 'useJsRendering',
+                    type: 'boolean',
+                    default: false,
+                    description: 'Si está activado, usa un navegador invisible (Puppeteer) para cargar la página. Necesario para webs React, Vue o Angular que renderizan meta etiquetas y contenido dinámicamente. Es más lento que la petición HTTP normal.',
+                },
                 {
                     displayName: 'Opciones de detalle',
                     name: 'detailOptions',
@@ -3407,6 +3463,7 @@ class SeoScanner {
         const timeoutMs = timeoutSeconds * 1000;
         const followRedirects = this.getNodeParameter('followRedirects', 0) !== false;
         const acceptLanguage = (this.getNodeParameter('acceptLanguage', 0) || '').trim() || undefined;
+        const useJsRendering = this.getNodeParameter('useJsRendering', 0) ?? false;
         let userAgent = USER_AGENT;
         const simulateGooglebot = this.getNodeParameter('simulateGooglebot', 0) ?? false;
         const customUserAgent = this.getNodeParameter('customUserAgent', 0) ?? false;
@@ -3422,7 +3479,7 @@ class SeoScanner {
             catch {
             }
         }
-        const fetchOpts = { userAgent, followRedirects, acceptLanguage };
+        const fetchOpts = { userAgent, followRedirects, acceptLanguage, useJsRendering };
         const generateHtmlReport = this.getNodeParameter('generateHtmlReport', 0) ?? false;
         const detailOpts = this.getNodeParameter('detailOptions', 0) || {};
         const ignoredIssues = (this.getNodeParameter('ignoredIssues', 0, '') || '').split('\n').map(s => s.trim()).filter(Boolean);