bricks-builder-mcp 3.11.2 → 3.11.4

package/package.json

@@ -1,7 +1,7 @@
 {
   "type": "module",
   "name": "bricks-builder-mcp",
-  "version": "3.11.2",
+  "version": "3.11.4",
   "description": "Serveur MCP pour piloter Bricks Builder (WordPress) depuis Claude — édition de pages, gestion d'éléments, réordonnancement des sections, vérification visuelle (verify_element), upload optimisé WebP. Communauté Discord : https://discord.gg/rX22zHRzH",
   "homepage": "https://discord.gg/rX22zHRzH",
   "main": "server.js",

package/server.js

@@ -2254,13 +2254,17 @@ mcpServer.setRequestHandler(CallToolRequestSchema, async (request) => {
           }
 
           try {
-            await page.goto(pageUrl, { waitUntil: 'domcontentloaded', timeout: 30000 });
-            // Wait : fonts + lazy load (networkidle pour les images lazy)
-            try {
-              await page.waitForLoadState('networkidle', { timeout: 8000 });
-            } catch {} // pas grave si certaines requêtes restent en cours
-            await page.waitForTimeout(1500);
-            // Scroll bottom puis top pour déclencher les lazy-load IntersectionObserver
+            await page.goto(pageUrl, { waitUntil: 'load', timeout: 30000 });
+            await page.waitForTimeout(1000);
+            // v3.11.3 — Forcer toutes les <img loading="lazy"> en eager pour
+            // déclencher leur chargement IMMÉDIATEMENT (sans attendre l'IntersectionObserver).
+            // Sinon Bricks lazy-load les images et mon audit court trop tôt.
+            await page.evaluate(() => {
+              document.querySelectorAll('img[loading="lazy"]').forEach(img => {
+                img.loading = 'eager';
+              });
+            });
+            // Scroll bottom→top pour déclencher les autres lazy-load (background-image, custom JS)
             await page.evaluate(() => {
               return new Promise(resolve => {
                 const totalHeight = document.documentElement.scrollHeight;
@@ -2272,19 +2276,23 @@ mcpServer.setRequestHandler(CallToolRequestSchema, async (request) => {
                   if (scrolled >= totalHeight) {
                     clearInterval(timer);
                     window.scrollTo(0, 0);
-                    setTimeout(resolve, 800);
+                    setTimeout(resolve, 500);
                   }
-                }, 100);
+                }, 80);
               });
             });
-            // v3.11.2 — attendre que TOUTES les <img> soient chargées avant l'audit
-            // sinon les lazy-loaded apparaissent comme "container vide"
+            // networkidle après le scroll pour laisser les images se télécharger
+            try {
+              await page.waitForLoadState('networkidle', { timeout: 12000 });
+            } catch {}
+            // v3.11.3 — vrai check : naturalWidth > 0 (pas juste img.complete qui est trompeur).
+            // img.complete = true pour les imgs qui ne se sont pas chargées du tout.
             try {
               await page.waitForFunction(
-                () => Array.from(document.images).every(i => i.complete),
-                { timeout: 10000 }
+                () => Array.from(document.images).every(i => i.naturalWidth > 0 || !i.src),
+                { timeout: 8000 }
               );
-            } catch {} // si une img reste cassée, on continue (elle sera flaggée en broken_image)
+            } catch {} // si une img reste vraiment cassée, on continue (elle sera flaggée en broken_image)
             await page.waitForTimeout(500);
 
             // 3) Collecte d'issues globales sur toute la page
@@ -2302,26 +2310,42 @@ mcpServer.setRequestHandler(CallToolRequestSchema, async (request) => {
 
               // === Empty containers ===
               if (cfg.empty_containers) {
+                // v3.11.4 — debug : on remonte le outerHTML pour identifier les faux positifs
+                let debugCount = 0;
                 allBrxe.forEach(b => {
                   const rect = b.getBoundingClientRect();
                   if (rect.width < 50 || rect.height < 50) return;
                   const area = rect.width * rect.height;
                   if (area < 2500) return;
-                  // Ignorer les éléments en dehors du viewport rendu (hauteur fullpage)
                   const hasText = (b.textContent || '').trim().length > 0;
-                  const hasMedia = b.querySelector('img, picture, svg, video, iframe') !== null;
+                  const hasMedia = b.querySelector('img, picture, svg, video, iframe, canvas') !== null;
                   const hasInteractive = b.querySelector('a, button, input, select, textarea') !== null;
                   const bcs = getComputedStyle(b);
                   const hasBgImg = bcs.backgroundImage && bcs.backgroundImage !== 'none';
-                  if (!hasText && !hasMedia && !hasInteractive && !hasBgImg) {
-                    issues.push({
+                  // v3.11.4 — détecte AUSSI les background-image sur les descendants directs (Bricks rend souvent
+                  // l'image en CSS bg sur un enfant, pas sur le wrapper)
+                  const hasChildBgImg = !hasBgImg && Array.from(b.children).some(c => {
+                    const ccs = getComputedStyle(c);
+                    return ccs.backgroundImage && ccs.backgroundImage !== 'none';
+                  });
+                  if (!hasText && !hasMedia && !hasInteractive && !hasBgImg && !hasChildBgImg) {
+                    const issue = {
                       type: 'empty_container',
                       severity: 'warning',
                       element: b.id || (Array.from(b.classList).find(c => c.startsWith('brxe-')) || '?'),
                       label: `Container vide ${Math.round(rect.width)}×${Math.round(rect.height)}px`,
                       hint: "Bloc visible sans contenu — souvent un wrapper écrasé par align-items: stretch. Fixer aspect-ratio ou ajouter du contenu.",
                       bbox: absBbox(rect),
-                    });
+                    };
+                    // DEBUG : 3 premiers seulement (sinon trop verbeux)
+                    if (debugCount < 3) {
+                      issue._debugOuterHTML = (b.outerHTML || '').substring(0, 400);
+                      issue._debugTagName = b.tagName;
+                      issue._debugChildrenCount = b.children.length;
+                      issue._debugChildren = Array.from(b.children).slice(0, 3).map(c => `<${c.tagName.toLowerCase()}${c.id ? '#' + c.id : ''}${c.className ? '.' + (typeof c.className === 'string' ? c.className.split(' ').slice(0,2).join('.') : '') : ''}>`);
+                      debugCount++;
+                    }
+                    issues.push(issue);
                   }
                 });
               }