sbuilder-mcp 0.33.0 → 0.34.0

package/CHANGELOG.md

@@ -6,6 +6,14 @@ 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.34.0] - 2026-09-11
+
+### Added
+- sb_store's new `action: "chrome"` builds one shared header (or, with `footer: true`, a shared footer) from the pages a site already has, closing the `siteChrome` gap sb_review has reported since it learned to ask: the menu labels and links come from the site's own pages (home first), the look is read off the home page, and the master is given to every page as a reference instead of copied onto each one. It skips a site that already shares a header or footer, and a site with fewer than two pages. dry_run (the default) returns the planned menu and which pages it would touch without writing anything.
+
+### Fixed
+- The menu built for a shared header or footer (by sb_store's new `action: "chrome"` and by sb_import_site, which already built one) now packs its links against each other at their own width instead of splitting the row into equal-width columns, so three links no longer land spread across the thirds of a wide header with large gaps between them.
+
 ## [0.33.0] - 2026-09-11
 
 ### Added

package/CHANGELOG.vi.md

@@ -6,6 +6,14 @@ 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.34.0] - 2026-09-11
+
+### Added
+- Hành động mới `action: "chrome"` của sb_store dựng một header dùng chung (hoặc, với `footer: true`, một footer dùng chung) từ các trang site đã có sẵn, khép lại khoảng trống `siteChrome` mà sb_review đã báo cáo kể từ khi biết hỏi câu này: nhãn và liên kết của menu lấy từ chính các trang của site (trang chủ trước tiên), giao diện được đọc từ trang chủ, và master được gán cho mọi trang dưới dạng tham chiếu thay vì sao chép riêng vào từng trang. Lệnh này bỏ qua nếu site đã có sẵn header hoặc footer dùng chung, và nếu site có ít hơn hai trang. dry_run (mặc định bật) trả về menu dự kiến và các trang sẽ bị tác động mà không ghi gì cả.
+
+### Fixed
+- Menu được dựng cho header hoặc footer dùng chung (bởi `action: "chrome"` mới của sb_store, và bởi sb_import_site — vốn đã dựng sẵn menu này) giờ xếp các liên kết sát nhau theo đúng chiều rộng nội dung của chúng thay vì chia hàng thành các cột bằng nhau, nên ba liên kết không còn bị dàn trải ra ba phần bằng nhau của một header rộng với khoảng trống lớn giữa chúng.
+
 ## [0.33.0] - 2026-09-11
 
 ### Added

package/dist/domains/site/importmap.js

@@ -423,6 +423,35 @@ function one(c, t) {
             // Mobile gets one column by name. The flex path's `flexDirection: column`
             // says nothing to a grid, and a mobile answer that silently does nothing
             // is rule 3 failing with a value in the document to prove it tried.
+            // A PACKED ROW IS NOT A GRID AND NOT AN EQUAL-SHARE ROW. It wraps, so it
+            // needs no stack breakpoint of its own — a menu that runs out of width
+            // starts a second line, which is what a menu should do.
+            if (c.pack) {
+                return {
+                    type: 'flex-block',
+                    style: {
+                        width: '100%',
+                        display: 'flex',
+                        flexDirection: 'row',
+                        flexWrap: 'wrap',
+                        alignItems: 'center',
+                        gap: '24px',
+                        ...(c.align === 'center' ? { justifyContent: 'center' } : {}),
+                    },
+                    // `width: auto` IS THE LOAD-BEARING HALF, and leaving it out looked
+                    // like the whole idea had failed. `flex-block` seeds `width: 100%`
+                    // from its element defaults, and `flex: 0 0 auto` only says "do not
+                    // grow or shrink from the BASIS" — the basis being `auto`, which
+                    // reads the width. So every packed cell stayed 100% wide and the
+                    // menu came out as a vertical list: measured, three links stacked in
+                    // a 140px header where the equal-share version had been 52.
+                    children: kids.map((k) => ({
+                        type: 'flex-block',
+                        style: { flex: '0 0 auto', width: 'auto', display: 'flex', flexDirection: 'column' },
+                        children: [k],
+                    })),
+                };
+            }
             if (c.wrap) {
                 return {
                     type: 'flex-block',
@@ -654,7 +683,8 @@ export function navSpec(links, t) {
                 {
                     kind: 'group',
                     direction: 'row',
-                    wrap: true,
+                    // PACKED, not an equal share: a menu's links sit against each other.
+                    pack: true,
                     children: links.map((l) => ({
                         kind: 'button',
                         variant: 'link',

package/dist/server.js

@@ -70,7 +70,7 @@ export function createServer(ctx = buildContext()) {
     registerApiTools(server, ctx);
     const pageSession = registerPageTools(server, ctx);
     registerLiveTools(server, ctx, pageSession);
-    registerStoreTools(server, ctx);
+    registerStoreTools(server, ctx, pageSession);
     registerThemeTools(server, ctx);
     registerImportTools(server, ctx, pageSession);
     registerUndoTools(server, ctx);

package/dist/tools/chrome.js

@@ -0,0 +1,93 @@
+import { request } from '../transport/http.js';
+import { siteToken } from './credentialpick.js';
+import { siteFor } from './context.js';
+import { addSubtree } from '../domains/site/builder.js';
+import { menuLabel, navSpec, tokensFromPage } from '../domains/site/importmap.js';
+import { globalDocumentFrom } from './importpage.js';
+/** Does this site already share a section of this kind? */
+export async function hasGlobal(ctx, siteId, kind) {
+    const got = (await request({
+        base: ctx.base,
+        method: 'GET',
+        path: `/api/sites/${encodeURIComponent(siteId)}/global-sections`,
+        token: siteToken(ctx),
+        fetchImpl: ctx.fetchImpl,
+    }));
+    return (got.globalSections ?? []).some((g) => g.kind === kind);
+}
+/**
+ * Create ONE global section from a link list and give every named page a
+ * reference to it.
+ *
+ * NOT ATOMIC, and it must not pretend to be: one page that will not take the
+ * header does not undo the header. The master exists, the others carry it, and
+ * the refusal is reported per page — the same shape `sb_import_site` uses for
+ * its own per-page report.
+ */
+export async function shareChrome(ctx, session, siteId, kind, links, pages, tokens) {
+    const out = { carried: [], failed: [] };
+    const spec = navSpec(links, tokens);
+    if (!spec) {
+        out.skipped = 'no links to put in it';
+        return out;
+    }
+    const made = (await request({
+        base: ctx.base,
+        method: 'POST',
+        path: `/api/sites/${encodeURIComponent(siteId)}/global-sections`,
+        token: siteToken(ctx),
+        body: {
+            name: kind === 'header' ? 'Header' : 'Footer',
+            kind,
+            document: globalDocumentFrom(spec),
+        },
+        fetchImpl: ctx.fetchImpl,
+    }));
+    const gid = made.globalSection?.id;
+    if (typeof gid !== 'string' || !gid) {
+        out.skipped = 'the platform created no global section';
+        return out;
+    }
+    out.created = gid;
+    for (const p of pages) {
+        try {
+            await session.open(siteId, p.id);
+            const doc = session.current();
+            // A HEADER GOES IN FIRST AND A FOOTER LAST. ROOT's children must read
+            // [header*][middle*][footer*] and the platform refuses EVERY save
+            // otherwise — compose turns the reference into a real band, so its index
+            // is the band it becomes.
+            const at = kind === 'header' ? 0 : doc.doc.nodes[doc.doc.root_node_id]?.data?.nodes?.length ?? 0;
+            const { patches } = addSubtree(doc, doc.doc.root_node_id, { type: 'flex-section', specials: { globalRef: gid, globalKind: kind } }, at);
+            await session.applyAndSave(patches);
+            out.carried.push(p.slug);
+        }
+        catch (e) {
+            out.failed.push({ page: p.slug, why: e.message.replace(/^sbuilder:\s*/, '').slice(0, 160) });
+        }
+    }
+    return out;
+}
+/** Every page this site has, as the menu would name them. */
+export async function sitePages(ctx, siteId) {
+    const got = (await request({
+        base: ctx.base,
+        method: 'GET',
+        path: `/api/sites/${encodeURIComponent(siteId)}/pages`,
+        token: siteToken(ctx),
+        fetchImpl: ctx.fetchImpl,
+    }));
+    return (got.pages ?? []).map((p) => ({
+        id: String(p.id ?? ''),
+        slug: String(p.slug ?? ''),
+        name: String(p.name ?? p.title ?? p.slug ?? ''),
+        isHome: p.isHomepage === true || p.slug === '',
+    }));
+}
+/** The link list a menu built from these pages would carry, home first. */
+export function chromeLinks(pages) {
+    return [...pages]
+        .sort((a, b) => Number(b.isHome) - Number(a.isHome))
+        .map((p) => ({ text: menuLabel(p.name), href: p.isHome ? '/' : `/${p.slug}` }));
+}
+export { siteFor, tokensFromPage };

package/dist/tools/store.js

@@ -33,6 +33,8 @@ import { request, redact } from '../transport/http.js';
 import { siteToken } from './credentialpick.js';
 import { siteFor } from './context.js';
 import { genId } from '../domains/site/ids.js';
+import { chromeLinks, hasGlobal, shareChrome, sitePages } from './chrome.js';
+import { tokensFromPage } from '../domains/site/importmap.js';
 import { CHECKOUT_FORM, CHECKOUT_FORM_DOCUMENT, CHECKOUT_PAGE_DOCUMENT, CHECKOUT_TEXT, FORM_ID_SENTINEL, HEADLINE_SENTINEL, FORM_TEMPLATES, } from '../catalog/checkout.generated.js';
 /**
  * The store's live gateways and delivery options.
@@ -274,17 +276,18 @@ async function seedForm(ctx, siteId, key, formName, dryRun) {
             'what makes a form-document edit visible.',
     };
 }
-export function registerStoreTools(server, ctx) {
+export function registerStoreTools(server, ctx, session) {
     server.registerTool('sb_store', {
         description: 'Run a store flow that must happen in a fixed order. action:"checkout" makes the order ' +
             'form, configures it, saves its fields with this store\'s real payment and delivery ' +
             'options, then creates and PUBLISHES the checkout page — /checkout 404s without all ' +
             'four. action:"form" seeds any of the platform\'s other form templates (login, ' +
             'register, forgot, reset, verify, contact, subscribe, booking, review and more) with ' +
-            'its own field document, which is the part that cannot be guessed. Dry run returns ' +
-            'the plan.',
+            'its own field document, which is the part that cannot be guessed. action:"chrome" ' +
+            'gives every page ONE shared header, built from the pages this site already has — the ' +
+            'gap sb_review reports as siteChrome. Dry run returns the plan.',
         inputSchema: {
-            action: z.enum(['checkout', 'form']),
+            action: z.enum(['checkout', 'form', 'chrome']),
             site_id: z.string().optional(),
             language: z.enum(['vi', 'en']).optional().describe('Copy language, default vi'),
             page_name: z.string().optional(),
@@ -294,11 +297,55 @@ export function registerStoreTools(server, ctx) {
                 .optional()
                 .describe('action:"form" — which of the platform\'s own form templates to seed'),
             name: z.string().optional().describe('action:"form" — the form\'s name in the merchant\'s list'),
+            footer: z
+                .boolean()
+                .optional()
+                .describe('action:"chrome" — build a shared FOOTER instead of a header'),
             dry_run: z.boolean().optional(),
         },
         annotations: { readOnlyHint: false, destructiveHint: false, openWorldHint: true },
-    }, async ({ action, site_id: given, language, page_name, headline, template, name, dry_run }) => {
+    }, async ({ action, site_id: given, language, page_name, headline, template, name, footer, dry_run }) => {
         const siteId = siteFor(ctx, given);
+        if (action === 'chrome') {
+            // SKIPPED WHEN THE SITE ALREADY SHARES ONE, because a second header is
+            // two headers rather than a menu — and below two pages, because a menu
+            // to one page is a link to itself. Both are `sb_import_site`'s own
+            // rules, kept because they were right there.
+            const kind = footer === true ? 'footer' : 'header';
+            if (await hasGlobal(ctx, siteId, kind)) {
+                return text({
+                    skipped: `this site already shares a ${kind} — a second one is two of them, not a menu`,
+                });
+            }
+            const pages = await sitePages(ctx, siteId);
+            if (pages.length < 2) {
+                return text({ skipped: 'a menu to one page is a link to itself' });
+            }
+            const links = chromeLinks(pages);
+            // RULE 0: the look comes off the HOME page, which is the one page whose
+            // pattern the rest of the site already follows. A site with nothing on
+            // its home page yields no tokens rather than an invented palette.
+            const home = pages.find((p) => p.isHome) ?? pages[0];
+            await session.open(siteId, home.id);
+            const tokens = tokensFromPage(session.current().doc);
+            if (dry_run !== false) {
+                return text({
+                    dry_run: true,
+                    would_create: kind,
+                    menu: links,
+                    onto: pages.map((p) => p.slug),
+                    tokens_from: Object.keys(tokens).length ? 'the home page' : 'nothing — the home page is blank',
+                    note: 'Nothing was sent. This creates ONE shared master and gives every page a reference ' +
+                        'to it, so the menu becomes one edit instead of one per page.',
+                });
+            }
+            const out = await shareChrome(ctx, session, siteId, kind, links, pages, tokens);
+            return text({
+                ...out,
+                next: 'Publish the pages: a global section reaches a visitor through each page it is ' +
+                    'composed onto, so a saved page keeps the old chrome until it is published again.',
+            });
+        }
         if (action === 'form') {
             if (!template) {
                 throw new Error(`sbuilder: action:"form" needs a template. One of: ${FORM_TEMPLATE_KEYS.join(', ')}.`);

package/package.json

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