bricks-builder-mcp 3.11.1 → 3.11.3

package/package.json

@@ -1,7 +1,7 @@
 {
   "type": "module",
   "name": "bricks-builder-mcp",
-  "version": "3.11.1",
+  "version": "3.11.3",
   "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

@@ -229,9 +229,15 @@ function buildSiblingCoherenceChecks(audit) {
   const all = [audit.self, ...siblings];
 
   // 1) text-align mixé entre frères directs
-  // Si un seul frère a un text-align différent, c'est souvent un bug visuel
-  // (ex: H2 left + sous-titre center sans intention design).
-  const aligns = [...new Set(all.map(s => s['text-align']).filter(Boolean))];
+  // v3.11.2 : `start` et `left` sont équivalents en LTR (idem `end`/`right`),
+  // on les normalise avant de comparer pour éviter les faux positifs.
+  // (LTR par défaut, le check est sur des sites WordPress majoritairement français)
+  const normalizeAlign = (val) => {
+    if (val === 'start') return 'left';
+    if (val === 'end') return 'right';
+    return val;
+  };
+  const aligns = [...new Set(all.map(s => normalizeAlign(s['text-align'])).filter(Boolean))];
   if (aligns.length > 1) {
     checks.push({
       ok: false,
@@ -2248,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
+            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;
@@ -2266,11 +2276,24 @@ mcpServer.setRequestHandler(CallToolRequestSchema, async (request) => {
                   if (scrolled >= totalHeight) {
                     clearInterval(timer);
                     window.scrollTo(0, 0);
-                    setTimeout(resolve, 800);
+                    setTimeout(resolve, 500);
                   }
-                }, 100);
+                }, 80);
               });
             });
+            // 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.naturalWidth > 0 || !i.src),
+                { timeout: 8000 }
+              );
+            } 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
             const auditResult = await page.evaluate((cfg) => {
@@ -2341,13 +2364,25 @@ mcpServer.setRequestHandler(CallToolRequestSchema, async (request) => {
 
               // === Sibling coherence ===
               if (cfg.sibling_coherence) {
+                // v3.11.2 : start === left, end === right en LTR (équivalents CSS)
+                const dir = getComputedStyle(document.documentElement).direction || 'ltr';
+                const normalizeAlign = (val) => {
+                  if (dir === 'ltr') {
+                    if (val === 'start') return 'left';
+                    if (val === 'end') return 'right';
+                  } else {
+                    if (val === 'start') return 'right';
+                    if (val === 'end') return 'left';
+                  }
+                  return val;
+                };
                 allBrxe.forEach(parent => {
                   const realChildren = Array.from(parent.children).filter(c =>
                     (c.id && c.id.startsWith('brxe-')) ||
                     Array.from(c.classList).some(cls => cls.startsWith('brxe-'))
                   );
                   if (realChildren.length < 2) return;
-                  const aligns = [...new Set(realChildren.map(c => getComputedStyle(c).textAlign))];
+                  const aligns = [...new Set(realChildren.map(c => normalizeAlign(getComputedStyle(c).textAlign)))];
                   if (aligns.length > 1) {
                     const rect = parent.getBoundingClientRect();
                     issues.push({