bricks-builder-mcp 3.11.1 → 3.11.2

package/package.json

@@ -1,7 +1,7 @@
 {
   "type": "module",
   "name": "bricks-builder-mcp",
-  "version": "3.11.1",
+  "version": "3.11.2",
   "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,
@@ -2254,7 +2260,7 @@ mcpServer.setRequestHandler(CallToolRequestSchema, async (request) => {
               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
+            // Scroll bottom puis top pour déclencher les lazy-load IntersectionObserver
             await page.evaluate(() => {
               return new Promise(resolve => {
                 const totalHeight = document.documentElement.scrollHeight;
@@ -2271,6 +2277,15 @@ mcpServer.setRequestHandler(CallToolRequestSchema, async (request) => {
                 }, 100);
               });
             });
+            // v3.11.2 — attendre que TOUTES les <img> soient chargées avant l'audit
+            // sinon les lazy-loaded apparaissent comme "container vide"
+            try {
+              await page.waitForFunction(
+                () => Array.from(document.images).every(i => i.complete),
+                { timeout: 10000 }
+              );
+            } catch {} // si une img reste 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 +2356,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({