bricks-builder-mcp 3.11.4 → 3.11.5

package/package.json

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

@@ -2310,42 +2310,36 @@ 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;
+                // v3.11.5 — Bricks 2.3 rend ses images comme des <img class="brxe-image" id="brxe-{id}">
+                // (pas un wrapper). Idem pour vidéos/iframes/svg. On skip ces balises : elles SONT le média.
+                const MEDIA_TAGS = new Set(['IMG', 'PICTURE', 'SVG', 'VIDEO', 'IFRAME', 'CANVAS', 'AUDIO', 'OBJECT', 'EMBED']);
+                const INTERACTIVE_TAGS = new Set(['A', 'BUTTON', 'INPUT', 'SELECT', 'TEXTAREA']);
                 allBrxe.forEach(b => {
+                  // Skip si l'élément EST déjà un média ou un interactif natif
+                  if (MEDIA_TAGS.has(b.tagName) || INTERACTIVE_TAGS.has(b.tagName)) return;
                   const rect = b.getBoundingClientRect();
                   if (rect.width < 50 || rect.height < 50) return;
                   const area = rect.width * rect.height;
                   if (area < 2500) return;
                   const hasText = (b.textContent || '').trim().length > 0;
-                  const hasMedia = b.querySelector('img, picture, svg, video, iframe, canvas') !== null;
+                  const hasMedia = b.querySelector('img, picture, svg, video, iframe, canvas, audio, object, embed') !== null;
                   const hasInteractive = b.querySelector('a, button, input, select, textarea') !== null;
                   const bcs = getComputedStyle(b);
                   const hasBgImg = bcs.backgroundImage && bcs.backgroundImage !== 'none';
-                  // 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)
+                  // Détecte aussi les background-image sur les descendants directs (cas Bricks où le bg est sur un sub-div)
                   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 = {
+                    issues.push({
                       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);
+                    });
                   }
                 });
               }