sbuilder-mcp 0.75.0 → 0.76.1

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.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
+- `sb_review` gains a `siteFooter` readiness gap: since `sb_store` action `chrome` builds the shared header and treats the footer as an opt-in flag on a separate call, a site built this way can end every page at its last section with no policy links, no contact, and no copyright line; it fires once the site already has at least one global section and two or more pages but no footer among them, pointing at `sb_store` action `chrome` with `footer:true`.
+
 ## [0.75.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.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
+- `sb_review` có thêm gap readiness `siteFooter`: vì `sb_store` action `chrome` dựng header dùng chung còn footer là một cờ tùy chọn ở một lệnh riêng, một site dựng theo cách này có thể khiến mọi trang kết thúc ở section cuối cùng, không có link chính sách, không có liên hệ, không có dòng bản quyền; gap báo khi site đã có ít nhất một global section và từ hai trang trở lên nhưng không trang nào có footer, trỏ tới `sb_store` action `chrome` với `footer:true`.
+
 ## [0.75.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

@@ -217,6 +217,29 @@ export function readinessGaps(input) {
                 'globalRef instead of a copy, and one edit reaches every page.',
         });
     }
+    // A HEADER AND NO FOOTER.
+    //
+    // `sb_store action:"chrome"` builds the header; the footer is an opt-in flag
+    // on the same call, so a site built here has a shared header and, on every
+    // page, either its own footer or none — no policy links, no contact, no
+    // copyright line, and nothing below the last section but the end of the
+    // document. The same threshold as siteChrome, asked only once the site HAS
+    // shared chrome (an empty list is siteChrome's finding, not this one).
+    if (pages &&
+        pages.length >= 2 &&
+        input.globalKinds &&
+        input.globalKinds.length > 0 &&
+        !input.globalKinds.includes('footer')) {
+        gaps.push({
+            id: 'siteFooter',
+            draft: false,
+            problem: 'The site shares a header and no footer, so every page ends at its last section — ' +
+                'no policy links, no contact, no copyright line, and each page that has a footer of ' +
+                'its own drifts from the next.',
+            fix: 'sb_store action:"chrome" footer:true builds one shared footer on a real site menu and ' +
+                'attaches it to every page. Put the policy, contact and about pages in it.',
+        });
+    }
     // THE SITE IS DARK RIGHT NOW AND SAYING NOTHING IN ITS OWN VOICE.
     //
     // Reported FIRST, because it is the only finding here that is not about what
@@ -310,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}} — ' +

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.75.0",
+  "version": "0.76.1",
   "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",