bricks-builder-mcp 3.6.4 → 3.6.5

package/package.json

@@ -1,7 +1,7 @@
 {
   "type": "module",
   "name": "bricks-builder-mcp",
-  "version": "3.6.4",
+  "version": "3.6.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 + technique (verify_element).",
   "main": "server.js",
   "bin": {

package/server.js

@@ -109,8 +109,8 @@ function normaliseColor(s) {
   return s;
 }
 
-// Normaliser les valeurs CSS pour comparaison (rgba/spaces/units/hex)
-function normaliseCssValue(val) {
+// Normaliser les valeurs CSS pour comparaison (rgba/spaces/units/hex/vh)
+function normaliseCssValue(val, viewportContext) {
   if (val == null) return '';
   let s = String(val).trim().toLowerCase();
   s = s.replace(/\s+/g, '');
@@ -126,6 +126,23 @@ function normaliseCssValue(val) {
     return normaliseColor(s);
   }
 
+  // vh / vw → px en fonction du viewport actuel (getComputedStyle renvoie en px)
+  if (viewportContext) {
+    const vhMatch = s.match(/^(\d+(?:\.\d+)?)vh$/);
+    if (vhMatch && viewportContext.height) {
+      return Math.round(parseFloat(vhMatch[1]) * viewportContext.height / 100) + 'px';
+    }
+    const vwMatch = s.match(/^(\d+(?:\.\d+)?)vw$/);
+    if (vwMatch && viewportContext.width) {
+      return Math.round(parseFloat(vwMatch[1]) * viewportContext.width / 100) + 'px';
+    }
+    // svh, dvh, lvh : on assimile à vh pour l'instant (approximatif mais OK pour la plupart des cas)
+    const svhMatch = s.match(/^(\d+(?:\.\d+)?)(?:svh|dvh|lvh)$/);
+    if (svhMatch && viewportContext.height) {
+      return Math.round(parseFloat(svhMatch[1]) * viewportContext.height / 100) + 'px';
+    }
+  }
+
   // Normalise "0px" et "0" et "0%"
   if (s === '0' || s === '0px' || s === '0%') return '0';
   // Si valeur sans unité fournie (ex "32") et getComputedStyle renvoie "32px"
@@ -1436,6 +1453,13 @@ mcpServer.setRequestHandler(CallToolRequestSchema, async (request) => {
         const pageId = args.pageId;
         const elementId = args.elementId;
         const viewport = args.viewport || "desktop";
+        const viewportSizes = {
+          desktop: { width: 1920, height: 1080 },
+          tablet: { width: 991, height: 1200 },
+          mobile_landscape: { width: 767, height: 600 },
+          mobile_portrait: { width: 478, height: 800 },
+        };
+        const viewportContext = viewportSizes[viewport] || viewportSizes.desktop;
 
         // 1) Récupérer infos plugin (URL, sélecteur, expected styles)
         const info = await callWordPressAPI("/verify-element-info", "POST", { pageId, elementId });
@@ -1581,12 +1605,12 @@ mcpServer.setRequestHandler(CallToolRequestSchema, async (request) => {
             });
           }
 
-          // Compare chaque expected
+          // Compare chaque expected (viewport-aware pour vh/vw)
           const expected = info.expected || {};
           for (const [prop, expectedVal] of Object.entries(expected)) {
             const got = computed[prop];
             if (got === undefined) continue;
-            const ok = normaliseCssValue(got) === normaliseCssValue(expectedVal);
+            const ok = normaliseCssValue(got, viewportContext) === normaliseCssValue(expectedVal, viewportContext);
             const check = { ok, label: `${prop} = ${expectedVal}` };
             if (!ok) {
               check.expected = expectedVal;
@@ -1597,16 +1621,34 @@ mcpServer.setRequestHandler(CallToolRequestSchema, async (request) => {
             checks.push(check);
           }
 
-          // Console errors
-          if (consoleErrors.length > 0) {
+          // Console errors : séparer les vrais bugs JS des erreurs réseau (429/timeout)
+          // qui dépendent du serveur, pas du code de l'élément.
+          const realErrors = consoleErrors.filter(e =>
+            !e.includes('429') &&
+            !e.includes('net::ERR_') &&
+            !e.includes('Failed to load resource')
+          );
+          const networkErrors = consoleErrors.filter(e =>
+            e.includes('429') || e.includes('net::ERR_') || e.includes('Failed to load resource')
+          );
+
+          if (realErrors.length > 0) {
             checks.push({
               ok: false,
-              label: `${consoleErrors.length} erreur(s) console`,
-              got: consoleErrors.slice(0, 3),
-              hint: "Vérifie les ressources 404 (font, image, script)",
+              label: `${realErrors.length} erreur(s) JS dans la console`,
+              got: realErrors.slice(0, 3),
+              hint: "Erreur JavaScript détectée — pas lié au serveur",
             });
           } else {
-            checks.push({ ok: true, label: "Aucune erreur console" });
+            checks.push({ ok: true, label: "Aucune erreur JS console" });
+          }
+          // Erreurs réseau : info seulement, pas un échec
+          if (networkErrors.length > 0) {
+            checks.push({
+              ok: true,
+              label: `${networkErrors.length} erreur(s) réseau (429/load) — non bloquant`,
+              got: networkErrors.slice(0, 2),
+            });
           }
 
           // Overflow horizontal