sbuilder-mcp 0.19.0 → 0.20.0

package/CHANGELOG.md

@@ -6,6 +6,12 @@ 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.20.0] - 2026-09-10
+
+### Added
+- sb_review now reports a `siteChrome` gap when a site has two or more pages and no global section, so each page is carrying its own header and footer with no way to change the menu in one place.
+- sb_review now reports a `cartCount` gap when something opens the cart but no `cart-count` element shows what is in it, since a shopper who adds an item otherwise sees a toast fade with no lasting sign their basket is not empty.
+
 ## [0.19.0] - 2026-09-10
 
 ### Added

package/CHANGELOG.vi.md

@@ -6,6 +6,12 @@ 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.20.0] - 2026-09-10
+
+### Added
+- sb_review giờ báo gap `siteChrome` khi một site có từ hai trang trở lên mà không có global section nào, nghĩa là mỗi trang đang tự mang header và footer riêng, không có cách nào đổi menu ở một chỗ duy nhất.
+- sb_review giờ báo gap `cartCount` khi có thứ mở được giỏ hàng nhưng không có element `cart-count` nào cho thấy trong giỏ có gì, vì khách thêm hàng vào giỏ chỉ thấy một toast tắt đi mà không còn dấu hiệu lâu dài nào cho thấy giỏ không rỗng.
+
 ## [0.19.0] - 2026-09-10
 
 ### Added

package/dist/domains/site/readiness-fetch.js

@@ -61,6 +61,12 @@ export async function gatherReadiness(ctx, siteId, pageNodes) {
     const globalNodes = globals?.globalSections
         ? globals.globalSections.flatMap((g) => Object.values(g.document?.nodes ?? {}))
         : null;
+    // HOW MANY SHARED SECTIONS THE SITE HAS, not just what is in them. An empty
+    // list is the answer to a question nothing asked: a site whose pages each
+    // carry their own header is not one site.
+    const globalKinds = globals?.globalSections
+        ? globals.globalSections.map((g) => g.kind ?? '')
+        : null;
     // ACTIVE means a shopper can see it; PURCHASABLE adds a price above zero. A
     // product priced at zero renders, adds to the cart, and totals nothing — which
     // reads as a working store right up to the money.
@@ -78,6 +84,7 @@ export async function gatherReadiness(ctx, siteId, pageNodes) {
         shippingMethods,
         pageNodes,
         globalNodes,
+        globalKinds,
         categories,
         categoryPageLinks,
     };

package/dist/domains/site/readiness.js

@@ -61,12 +61,43 @@ function isStore(input) {
 }
 const published = (pages, type) => pages.some((p) => p.type === type && p.status === 'published');
 const drafted = (pages, type) => pages.some((p) => p.type === type && p.status !== 'published');
-/** What stands between this site and a paid order, most-blocking first. */
+/**
+ * What stands between this site and a paid order, most-blocking first — and,
+ * asked FIRST because it is not about money at all, whether these pages are one
+ * SITE.
+ *
+ * The precedent is `accountPage` and `searchPage`, already here on the same
+ * reasoning: a shop with no account page still takes orders, so they are a
+ * different question asked second. Shared chrome is a different question asked
+ * FIRST, because it is true of every site and not only of a store.
+ */
 export function readinessGaps(input) {
-    if (!isStore(input))
-        return [];
     const gaps = [];
     const pages = input.pages;
+    // A SITE WITH NO SHARED SECTION IS NOT A SITE, IT IS A STACK OF PAGES.
+    //
+    // Nothing asked this. Every tool here authors ONE page, so a build that never
+    // reaches for a global section gives each page its own header and footer —
+    // and then a change to the nav is one edit per page, the copies drift, and a
+    // visitor meets a slightly different site on every click. It is the most
+    // basic thing a website has that a generated one does not, and it is
+    // invisible to `sb_review`, which reads one page and finds it perfect.
+    //
+    // Two pages is the threshold: a one-page site has nothing to share with.
+    if (pages && pages.length >= 2 && input.globalKinds !== null && input.globalKinds?.length === 0) {
+        gaps.push({
+            id: 'siteChrome',
+            draft: false,
+            problem: `This site has ${pages.length} pages and no global section, so each one carries its own ` +
+                'header and footer. Changing the menu is that many edits, the copies drift apart, and a ' +
+                'visitor meets a slightly different site on every page.',
+            fix: 'Make them shared: POST /api/sites/{siteId}/global-sections with { name, kind: "header" | ' +
+                '"footer", document }, then PUT .../{id}/document with { subtree }. A page then carries a ' +
+                'globalRef instead of a copy, and one edit reaches every page.',
+        });
+    }
+    if (!isStore(input))
+        return gaps;
     if (pages && !published(pages, 'checkout')) {
         const draft = drafted(pages, 'checkout');
         gaps.push({
@@ -181,6 +212,28 @@ export function readinessGaps(input) {
             fix: draft ? `Publish the ${type} page that already exists.` : fix,
         });
     }
+    const everywhere = input.globalNodes !== null ? [...input.pageNodes, ...input.globalNodes] : null;
+    // A BASKET WITH NO NUMBER ON IT. The platform shipped `cart-count` to fix
+    // exactly this, and made it OPT-IN: `open_cart` is an ACTION any element can
+    // carry, not an element type, so there is no "cart icon" to give a badge to by
+    // default — minting one unasked would put a number on every social glyph in
+    // every footer. The cost of that decision is that a site authored through
+    // these tools never has one: a shopper adds an item, gets a toast that fades,
+    // and nothing anywhere on the page says their basket holds anything.
+    //
+    // Only asked when something DOES open the cart — otherwise `cartTrigger`
+    // below is the finding, and this would be a second sentence about the same
+    // missing control.
+    if (everywhere && opensCart(everywhere) && !everywhere.some((n) => n.data.type === 'cart-count')) {
+        gaps.push({
+            id: 'cartCount',
+            draft: false,
+            problem: 'Something opens the cart, but nothing shows what is in it. A shopper who adds an item ' +
+                'sees a toast that fades and then no evidence anywhere that their basket is not empty.',
+            fix: 'Give the control that opens the cart a cart-count satellite — sb_set on it with config ' +
+                '{ cartCountId: … } mints one, or add a cart-count beside it.',
+        });
+    }
     if (input.globalNodes !== null && !opensCart([...input.pageNodes, ...input.globalNodes])) {
         gaps.push({
             id: 'cartTrigger',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sbuilder-mcp",
-  "version": "0.19.0",
+  "version": "0.20.0",
   "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",