sbuilder-mcp 0.11.0 → 0.11.2

package/CHANGELOG.md

@@ -6,6 +6,20 @@ All notable changes to this project are documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.11.2] - 2026-09-09
+
+### Fixed
+- sb_set's routing of a hover write to an element's legacy home (a `button`, for instance) now also clears any leftover values in the unread `states.hover` slot in the same call, instead of leaving them behind for `sb_review` to keep reporting as `hover_dead`; the finding's own message previously named `unset` as the way to remove them, which could not reach the legacy home either, so the documented fix was unreachable.
+
+## [0.11.1] - 2026-09-09
+
+### Added
+- sb_review now reports `hover_dead` for a node whose element keeps its hover state in a legacy home its own renderer reads instead of the universal `states.hover` slot — a `button`, for instance — naming the sb_set call that moves the values where they will actually paint.
+- npm run codegen now refuses to run against a web_builder checkout with uncommitted changes in the directories it reads (schema/src, editor/src, server/render, server/docs), naming the dirty files instead of silently baking a concurrent session's half-finished work into the committed catalog; `--dirty` overrides it for generating against your own in-progress change.
+
+### Fixed
+- The generated element catalog is regenerated from a committed ref, removing a `cart-count` element and its satellite owner that had leaked in from another session's uncommitted platform change; the catalog is back to 108 elements.
+
 ## [0.11.0] - 2026-09-09
 
 ### Added

package/CHANGELOG.vi.md

@@ -6,6 +6,20 @@ Mọi thay đổi đáng chú ý của dự án được ghi lại trong file n
 Định dạng dựa trên [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 và dự án tuân theo [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.11.2] - 2026-09-09
+
+### Fixed
+- Khi sb_set định tuyến một lần ghi hover tới nơi lưu cũ của element (ví dụ `button`), giờ nó cũng xoá luôn mọi giá trị còn sót trong slot `states.hover` không ai đọc, ngay trong cùng một lệnh — thay vì để lại chúng cho sb_review tiếp tục báo là `hover_dead`; thông báo của chính finding này trước đây nêu `unset` là cách xoá, nhưng `unset` cũng không thể chạm tới nơi lưu cũ đó, khiến cách sửa được ghi ra là không thể thực hiện được.
+
+## [0.11.1] - 2026-09-09
+
+### Added
+- sb_review giờ báo cáo `hover_dead` cho một node mà element của nó lưu hover state ở nơi cũ mà chính renderer của nó đọc, thay vì slot `states.hover` phổ quát — ví dụ như `button` — và nêu rõ lệnh sb_set để chuyển giá trị tới đúng nơi sẽ thực sự hiển thị.
+- npm run codegen giờ từ chối chạy trên một checkout web_builder có thay đổi chưa commit trong các thư mục mà nó đọc (schema/src, editor/src, server/render, server/docs), nêu tên các file bẩn thay vì âm thầm đưa công việc dở dang của một phiên làm việc đồng thời vào catalog đã commit; `--dirty` cho phép ghi đè khi bạn tự tạo ra thay đổi đang dở dang đó.
+
+### Fixed
+- Catalog element được sinh lại từ một ref đã commit, loại bỏ một element `cart-count` và chủ sở hữu satellite của nó vốn đã lọt vào từ thay đổi nền tảng chưa commit của một phiên làm việc khác; catalog trở lại 108 element.
+
 ## [0.11.0] - 2026-09-09
 
 ### Added

package/dist/catalog/elements.generated.js

@@ -1,5 +1,5 @@
 export const ELEMENT_SOURCE = {
-    "count": 109,
+    "count": 108,
     "docSchemaVersion": 2
 };
 export const ELEMENTS = {
@@ -1803,8 +1803,7 @@ export const ELEMENTS = {
                         "key": "action",
                         "label": "Action",
                         "controls": [
-                            "action",
-                            "icon_cart_badge"
+                            "action"
                         ]
                     },
                     {
@@ -1877,7 +1876,6 @@ export const ELEMENTS = {
             "icon_color",
             "icon_size",
             "action",
-            "icon_cart_badge",
             "padding",
             "margin",
             "align_self",
@@ -13844,166 +13842,6 @@ export const ELEMENTS = {
             "dynamic"
         ]
     },
-    "cart-count": {
-        "type": "cart-count",
-        "label": "Cart count",
-        "category": "store",
-        "isContainer": false,
-        "isRootOnly": false,
-        "locked": false,
-        "hideInLayer": false,
-        "childAllows": [],
-        "defaults": {
-            "style": {
-                "display": "inline-flex",
-                "alignItems": "center",
-                "justifyContent": "center",
-                "width": "fit-content",
-                "minWidth": "20px",
-                "height": "20px",
-                "paddingLeft": "6px",
-                "paddingRight": "6px",
-                "borderRadius": "999px",
-                "backgroundColor": "#171717",
-                "color": "#ffffff",
-                "fontSize": "12px",
-                "fontWeight": "600",
-                "lineHeight": "1"
-            },
-            "specials": {
-                "placeholder": "0",
-                "hideWhenEmpty": true
-            }
-        },
-        "inspector": [
-            {
-                "tab": "general",
-                "groups": [
-                    {
-                        "key": "content",
-                        "label": "Content",
-                        "controls": [
-                            "cart_count_placeholder",
-                            "cart_count_hide_empty"
-                        ]
-                    },
-                    {
-                        "key": "size",
-                        "label": "Size",
-                        "controls": [
-                            "width_select",
-                            "height_select",
-                            "size_bounds"
-                        ]
-                    },
-                    {
-                        "key": "layout",
-                        "label": "Layout",
-                        "controls": [
-                            "padding",
-                            "margin",
-                            "vertical",
-                            "horizontal"
-                        ]
-                    },
-                    {
-                        "key": "typography",
-                        "label": "Typography",
-                        "controls": [
-                            "font_family",
-                            "font_size",
-                            "text_style",
-                            "text_color"
-                        ]
-                    },
-                    {
-                        "key": "background",
-                        "label": "Background",
-                        "controls": [
-                            "bg_color"
-                        ]
-                    },
-                    {
-                        "key": "shape",
-                        "label": "Shape",
-                        "controls": [
-                            "border",
-                            "corner"
-                        ]
-                    }
-                ]
-            },
-            {
-                "tab": "advanced",
-                "groups": [
-                    {
-                        "key": "spacing",
-                        "label": "Spacing",
-                        "controls": [
-                            "padding_margin"
-                        ]
-                    },
-                    {
-                        "key": "display",
-                        "label": "Display",
-                        "controls": [
-                            "display"
-                        ]
-                    },
-                    {
-                        "key": "class_css",
-                        "label": "Class",
-                        "controls": [
-                            "class_css"
-                        ]
-                    }
-                ]
-            }
-        ],
-        "controls": [
-            "cart_count_placeholder",
-            "cart_count_hide_empty",
-            "width_select",
-            "height_select",
-            "size_bounds",
-            "padding",
-            "margin",
-            "vertical",
-            "horizontal",
-            "font_family",
-            "font_size",
-            "text_style",
-            "text_color",
-            "bg_color",
-            "border",
-            "corner",
-            "padding_margin",
-            "display",
-            "class_css"
-        ],
-        "description": "The number of items in the shopper's cart, as a small badge. The figure comes from the shopper's own cart in their browser, so it changes the moment they add, remove or re-count a line. Its usual home is the cart icon in the site header, where it is the only evidence on the page that the basket holds anything.",
-        "useWhen": [
-            "a header or navigation bar carries a cart icon and a shopper needs to see the basket is not empty",
-            "building a sticky mobile bar or a mini-cart trigger anywhere outside the cart drawer"
-        ],
-        "avoidWhen": [
-            "showing what the cart COSTS — that is a cart-total, which reads a money figure",
-            "showing how many of ONE product a shopper picked — that is the quantity stepper on the product",
-            "showing a past order’s item count; an order is a record, not the live cart"
-        ],
-        "contentTips": [
-            "the placeholder is what the badge reads before the shopper’s cart is known, so write it as a real empty-cart count",
-            "leave “hide when empty” on unless the header’s layout must never shift when the first item lands",
-            "to put it on a cart icon, turn the badge on from the icon’s own inspector rather than placing this element and positioning it by hand"
-        ],
-        "semantics": [
-            "cart",
-            "count",
-            "badge",
-            "header",
-            "commerce"
-        ]
-    },
     "cart-total": {
         "type": "cart-total",
         "label": "Cart total",
@@ -33839,28 +33677,6 @@ export const TRAIT_WRITES = {
             }
         ]
     },
-    "cart_count_placeholder": {
-        "key": "cart_count_placeholder",
-        "label": "Empty count",
-        "writes": [
-            {
-                "target": "specials",
-                "writeKey": "placeholder",
-                "type": "string"
-            }
-        ]
-    },
-    "cart_count_hide_empty": {
-        "key": "cart_count_hide_empty",
-        "label": "Hide when empty",
-        "writes": [
-            {
-                "target": "specials",
-                "writeKey": "hideWhenEmpty",
-                "type": "boolean"
-            }
-        ]
-    },
     "cart_total_placeholder": {
         "key": "cart_total_placeholder",
         "label": "Empty amount",
@@ -34221,13 +34037,6 @@ export const TRAIT_WRITES = {
     }
 };
 export const SATELLITE_RULES = {
-    "icon": [
-        {
-            "type": "cart-count",
-            "configKey": "cartCountId",
-            "optional": true
-        }
-    ],
     "tab": [
         {
             "type": "tab-item",

package/dist/domains/site/builder.js

@@ -264,6 +264,29 @@ export function setKeys(doc, id, keys, opts) {
         // compiler reads. Routed rather than refused: the caller asked for a hover
         // and there is a home that works; `hoverRoutingNote` says where it went.
         if (opts.state === HOVER_STATE && namespace === 'style' && hoverHome(doc.node(id).data.type) === 'legacy') {
+            // …AND CLEAR THE SLOT NOBODY READS, in the same call.
+            //
+            // On an element with this home, `states.hover` has no legitimate use: the
+            // universal compiler stands aside for it and its own renderer looks in the
+            // flat map. Leaving values there is the exact defect `sb_review` reports as
+            // `hover_dead`, so a rewrite that repaired the look and left the dead copy
+            // behind would fix the page and keep the finding — and the fix that
+            // finding names would still be unreachable, which is the shape this repo
+            // has now closed twice.
+            //
+            // Safe against the platform changing its mind: if a button ever keeps its
+            // hover in the node, its meta's `storage` becomes 'node', HOVER_HOMES says
+            // 'state', and this branch does not run at all.
+            const node = doc.node(id);
+            const dead = [];
+            if (node.states?.[HOVER_STATE] !== undefined) {
+                dead.push({ op: 'unset', path: ['nodes', id, 'states', HOVER_STATE] });
+            }
+            for (const [bp, slot] of Object.entries(node.responsive ?? {})) {
+                if (slot?.states?.[HOVER_STATE] !== undefined) {
+                    dead.push({ op: 'unset', path: ['nodes', id, 'responsive', bp, 'states', HOVER_STATE] });
+                }
+            }
             return [
                 ...Object.entries(keys).map(([k, v]) => ({
                     op: 'set',
@@ -271,6 +294,7 @@ export function setKeys(doc, id, keys, opts) {
                     value: v,
                 })),
                 ...(opts.unset ?? []).map((k) => ({ op: 'unset', path: legacyHoverPath(id, k) })),
+                ...dead,
             ];
         }
         // Base state and per-breakpoint state are DIFFERENT PLACES in the document,

package/dist/domains/site/findings.js

@@ -60,6 +60,9 @@ export const FIX = {
         '{ "overflowX": "visible", "overflowY": "visible" } — or move the pinned node outside it. ' +
         'Sticky resolves against its nearest SCROLLING ancestor, so a clipping one becomes that ' +
         'ancestor and the node pins inside a box that never scrolls.',
+    hover_dead: 'Rewrite it with the same values: sb_set id "<id>", namespace style, state "hover", ' +
+        'keys { … } — read them off the node first, because that one call both writes them where ' +
+        'the renderer looks AND clears the slot nobody reads. Nothing else to do afterwards.',
     stuck_no_host: 'Pin something: sb_set id "<id>" (or the section it lives in), namespace style, keys ' +
         '{ "position": "sticky" } — sb_set seeds the offset and the layer order with it. Until ' +
         'then the platform compiles no rule for this state at all, so the override is stored, ' +

package/dist/domains/site/review.js

@@ -1,5 +1,6 @@
 import { childrenOf, childrenWithSatellites, isOverlay, pageChildren, appBlockRoot, SPEC_GLOBAL_REF, SPEC_APP_BLOCK_REF } from '../../core/tree.js';
 import { STUCK_STATE, stickyBlockedBy, stuckHostOf, isPinnedNode } from './sticky.js';
+import { HOVER_STATE, hoverHome } from './hover.js';
 import { ELEMENTS, BINDING_SOURCES, BOUND_SPECIALS, FIRST_CHILD_ONLY, SATELLITE_RULES, ELEMENT_SEEDS } from '../../catalog/elements.generated.js';
 import { fill } from './findings.js';
 /**
@@ -224,6 +225,32 @@ export function reviewDesign(doc) {
             styled.states?.[STUCK_STATE],
             ...Object.values(styled.responsive ?? {}).map((r) => r?.states?.[STUCK_STATE]),
         ];
+        // A HOVER STORED WHERE NOBODY READS IT. The universal hover state is not the
+        // only home: an element whose meta declares a Hover variant keeps it
+        // somewhere its own renderer looks, and for a `button` that is the flat
+        // `config.stateHover` map. Every hover this server wrote onto a button
+        // before it learned that is still sitting in `states.hover`, painting
+        // nothing — on a control that is the single most common thing anyone gives
+        // a hover to.
+        const hoverSlots = [
+            styled.states?.[HOVER_STATE],
+            ...Object.values(styled.responsive ?? {}).map((r) => r?.states?.[HOVER_STATE]),
+        ];
+        const hasHover = hoverSlots.some((slot) => {
+            const h = slot;
+            return !!h && (Object.keys(h.style ?? {}).length > 0 || Object.keys(h.config ?? {}).length > 0);
+        });
+        if (hasHover && hoverHome(type) === 'legacy') {
+            out.push({
+                code: 'hover_dead',
+                nodeId: id,
+                type,
+                problem: `"${type}" keeps its hover in config.stateHover — the flat map its own renderer ` +
+                    'compiles — and the platform\'s universal hover compiler stands aside for it. These ' +
+                    'states.hover values are stored, saved, published and painted by nobody.',
+                fix: fill('hover_dead', { id }),
+            });
+        }
         const hasStuck = stuckSlots.some((slot) => {
             const s = slot;
             return !!s && (Object.keys(s.style ?? {}).length > 0 || Object.keys(s.config ?? {}).length > 0);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sbuilder-mcp",
-  "version": "0.11.0",
+  "version": "0.11.2",
   "description": "MCP server that designs and operates a Store Builder site — pages, data, theme and publish — through the platform's own API and live-edit protocol.",
   "mcpName": "io.github.vuluu2k/sbuilder-mcp",
   "type": "module",