sbuilder-mcp 0.76.0 → 0.76.2

package/CHANGELOG.md

@@ -6,6 +6,18 @@ 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.76.2] - 2026-09-28
+
+### Fixed
+- `sb_review`'s `productImages` gap now reads "1 active product has no image" instead of "1 active product have no image" when only a single active product is missing `images`.
+
+## [0.76.1] - 2026-09-28
+
+### Fixed
+- `sb_review`'s `unreachablePage` gap now reads a page's `/source` response under its `source.document` envelope instead of a bare `document`, so a link found in another page's body is actually counted; previously every page appeared to carry no links and every candidate page was reported unreachable.
+- `sb_review`'s `no_h1` finding no longer points at a shared header's brand line when that header is composed onto every page; it now recognizes a shared section by either its stored `globalRef` stamp or its composed `globalId` stamp, and fires only on a page's own heading, staying silent when a page has no heading of its own.
+- `sb_review`'s `pageSeo` gap now reads "1 page carries" instead of "1 page carry" when only a single page is missing `settings.title` or `settings.description`.
+
 ## [0.76.0] - 2026-09-28
 
 ### Added

package/CHANGELOG.vi.md

@@ -6,6 +6,18 @@ 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.76.2] - 2026-09-28
+
+### Fixed
+- Gap `productImages` của `sb_review` giờ hiển thị "1 active product has no image" thay vì "1 active product have no image" khi chỉ có một sản phẩm active thiếu `images`.
+
+## [0.76.1] - 2026-09-28
+
+### Fixed
+- Gap `unreachablePage` của `sb_review` giờ đọc response `/source` của trang theo đúng envelope `source.document` thay vì `document` trần trụi, nên một link tìm thấy trong thân một trang khác giờ được tính đúng; trước đó mọi trang đều bị coi là không mang link nào và mọi trang ứng viên đều bị báo là không thể tới được.
+- Finding `no_h1` của `sb_review` không còn trỏ vào dòng thương hiệu của header dùng chung khi header đó được dựng vào mọi trang; giờ nhận diện một section dùng chung qua cả stamp `globalRef` đã lưu lẫn stamp `globalId` được dựng, và chỉ báo trên heading của chính trang đó, im lặng khi trang không có heading riêng nào.
+- Gap `pageSeo` của `sb_review` giờ hiển thị "1 page carries" thay vì "1 page carry" khi chỉ có một trang thiếu `settings.title` hoặc `settings.description`.
+
 ## [0.76.0] - 2026-09-28
 
 ### Added

package/dist/core/tree.js

@@ -14,6 +14,18 @@ export const SPEC_GLOBAL_KIND = 'globalKind';
 export const SPEC_GLOBAL_REV = 'globalRev';
 export const SPEC_OVERLAY_REV = 'overlayRev';
 export const SPEC_GLOBAL_REF = 'globalRef';
+/**
+ * Is this node a shared section, by EITHER stamp: the stored reference
+ * (`globalRef`, what a client writes) or the composed one (`globalId`, what a
+ * page read back from the platform carries). A check knowing only one misses
+ * the other on every read — measured: a header composed as `globalId` on all
+ * six pages of a store, and a rule keyed on `globalRef` treating it as the
+ * page's own.
+ */
+export function isSharedSection(doc, id) {
+    const s = doc.nodes[id]?.specials ?? {};
+    return typeof s[SPEC_GLOBAL_ID] === 'string' || typeof s[SPEC_GLOBAL_REF] === 'string';
+}
 /** The specials stamps a composed APP BLOCK carries (appblocks.go:97-100). */
 export const SPEC_APP_BLOCK_ID = 'appBlockId';
 export const SPEC_APP_BLOCK_REF = 'appBlockRef';

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

@@ -183,8 +183,12 @@ export async function gatherReadiness(ctx, siteId, pageNodes, openPageId) {
         const others = (input.pages ?? [])
             .filter((p) => p.status === 'published' && typeof p.id === 'string' && p.id !== openPageId)
             .slice(0, 40);
-        const sources = await Promise.all(others.map((p) => get(`/api/sites/${site}/pages/${encodeURIComponent(p.id)}/source`)));
-        const nodes = sources.flatMap((src) => Object.values(src?.document?.nodes ?? {}));
+        const sources = await Promise.all(others.map((p) => 
+        // `{ source: { document } }` — the envelope transport/pages.ts loadSource
+        // unwraps; read bare, this pass saw no link in any page (measured on a
+        // local store, every page's document present and every href missed).
+        get(`/api/sites/${site}/pages/${encodeURIComponent(p.id)}/source`)));
+        const nodes = sources.flatMap((src) => Object.values(src?.source?.document?.nodes ?? {}));
         input.menuLinks.hrefs.push(...hrefsIn(nodes));
     }
     return input;

package/dist/domains/site/readiness.js

@@ -333,7 +333,8 @@ export function readinessGaps(input) {
                 id: 'pageSeo',
                 draft: false,
                 problem: pageCount(bare) +
-                    ' carry no settings.title or settings.description. Each serves the SITE NAME as its ' +
+                    (bare.length === 1 ? ' carries' : ' carry') +
+                    ' no settings.title or settings.description. Each serves the SITE NAME as its ' +
                     '<title> and no meta description — a search result cannot tell them apart, and ' +
                     'nothing on the canvas or in a screenshot shows it.',
                 fix: 'PATCH /api/v1/pages/{id} with {settings: {...existing, title, description}} — ' +
@@ -594,7 +595,7 @@ export function readinessGaps(input) {
         gaps.push({
             id: 'productImages',
             draft: false,
-            problem: `${n} of ${input.products.active} active products have no image, so every card, cart ` +
+            problem: `${n} of ${input.products.active} active products ${n === 1 ? 'has' : 'have'} no image, so every card, cart ` +
                 "line and product page shows the platform's grey placeholder where the photo goes.",
             fix: 'Get a picture into the library (sb_media_upload: a file, a url, or query + pick), then ' +
                 'PUT the whole product back with images: [url, …] — first is the primary. POST ' +

package/dist/domains/site/review.js

@@ -1,4 +1,4 @@
-import { childrenOf, childrenWithSatellites, isOverlay, pageChildren, appBlockRoot, SPEC_GLOBAL_REF, SPEC_APP_BLOCK_REF } from '../../core/tree.js';
+import { childrenOf, childrenWithSatellites, isOverlay, isSharedSection, pageChildren, appBlockRoot, SPEC_GLOBAL_REF, SPEC_APP_BLOCK_REF } from '../../core/tree.js';
 import { STUCK_STATE, stickyBlockedBy, stuckHostOf, isPinnedNode } from './sticky.js';
 import { bandOf } from './traps.js';
 import { HOVER_STATE, hoverHome } from './hover.js';
@@ -704,7 +704,28 @@ opts = {}) {
     // level but the one a search engine and a screen reader look for first.
     // Nothing on the canvas shows the tag. Page bands only (`band` excludes the
     // overlays): a drawer heading is not the page's.
+    //
+    // NAMED ON THE PAGE'S OWN HEADING, never a shared section's. The draft
+    // composes the global header in, so its brand line is the first heading on
+    // EVERY page — measured on a local store: one node id reported on six pages,
+    // and the fix would have made the site name the h1 everywhere. A shared h1
+    // still counts as present (that is a site's own decision); a page whose only
+    // headings are shared has none of its own and is left alone.
     {
+        const shared = new Set();
+        for (const sectionId of pageChildren(d)) {
+            if (!isSharedSection(d, sectionId))
+                continue;
+            const stack = [sectionId];
+            while (stack.length) {
+                const id = stack.pop();
+                if (shared.has(id))
+                    continue;
+                shared.add(id);
+                for (const k of childrenWithSatellites(d, id))
+                    stack.push(k);
+            }
+        }
         const level = (id) => {
             const n = d.nodes[id];
             const tag = n.specials?.htmlTag;
@@ -713,8 +734,9 @@ opts = {}) {
             return n.data.type === 'heading' ? 'h2' : null;
         };
         const headings = walkOrder.filter((id) => band.has(id) && level(id) !== null);
-        if (headings.length > 0 && !headings.some((id) => level(id) === 'h1')) {
-            const first = headings[0];
+        const own = headings.filter((id) => !shared.has(id));
+        if (own.length > 0 && !headings.some((id) => level(id) === 'h1')) {
+            const first = own[0];
             out.push({
                 code: 'no_h1',
                 nodeId: first,

package/package.json

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