sbuilder-mcp 0.82.6 → 0.82.7

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.82.7] - 2026-10-11
+
+### Fixed
+- `sb_template_use` inserts a site's own template into the open page, instead of reporting success while the page stays empty; `dry_run` sends no request.
+- `sb_page_update` accepts `default_template` and `render_for` on brand template pages.
+
 ## [0.82.6] - 2026-10-10
 
 ### Fixed

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.82.7] - 2026-10-11
+
+### Fixed
+- `sb_template_use` chèn template của chính site vào trang đang mở, thay vì báo thành công mà trang vẫn trống; `dry_run` không gửi request nào.
+- `sb_page_update` nhận `default_template` và `render_for` trên trang template brand.
+
 ## [0.82.6] - 2026-10-10
 
 ### Fixed

package/README.md

@@ -95,7 +95,7 @@ in this client: one would put the very key the platform exists to hold back into
 | `sb_remove` | Remove a node and its subtree |
 | `sb_duplicate` | Copy a node and its subtree under fresh ids, right after the original — or onto another page or site, as the editor's copy/paste does (assets copied, references that cannot travel listed) |
 | `sb_templates` | The store's saved section templates, plus the BUILT-IN layouts — hero, feature trio, stats, FAQ, CTA band, product shelf, category strip, brand wall, trust band — composed against the page's own tokens |
-| `sb_template_use` | Instantiate a template into a page |
+| `sb_template_use` | Add a section template to a page |
 | `sb_page_list` | Every page on the site |
 | `sb_page_create` | Create a page — a store type arrives with the editor's own starting document; `type` is the route for checkout, product, category, post, course |
 | `sb_page_update` | Edit a page's name, slug, home flag, order, SEO (title, description, Open Graph, robots, meta tags, JSON-LD) and members-only flag — merged into `settings`, so the keys it does not name survive. On a template page: make it the type's default, or choose which products/categories/posts render through it |

package/dist/tools/page.js

@@ -1024,6 +1024,20 @@ export class PageSession {
         this.savedRev = d.rev;
     }
 }
+/**
+ * A foreign subtree as a document `copyNodeInto` can read: a holder root over
+ * it, so the copy re-mints every id and strips the shared-master stamps.
+ */
+function heldTree(rootId, nodes) {
+    const holder = '__held';
+    const rn = structuredClone(nodes);
+    rn[rootId].data.parent = holder;
+    return PageDoc.from({
+        schema_version: 2,
+        root_node_id: holder,
+        nodes: { ...rn, [holder]: { id: holder, data: { type: 'root', parent: null, nodes: [rootId] } } },
+    });
+}
 function formDocPath(siteId, formId) {
     return `/api/sites/${encodeURIComponent(siteId)}/forms/${encodeURIComponent(formId)}/document`;
 }
@@ -1902,15 +1916,7 @@ export function registerPageTools(server, ctx) {
             if (!r?.tree?.nodes || !r.tree.rootId || !(r.tree.rootId in r.tree.nodes)) {
                 throw new Error('sbuilder: the platform answered the reconcile with no tree — nothing was pasted.');
             }
-            // A holder root so the copy can read the reconciled subtree as a document.
-            const holder = '__clipboard';
-            const rn = structuredClone(r.tree.nodes);
-            rn[r.tree.rootId].data.parent = holder;
-            src = PageDoc.from({
-                schema_version: 2,
-                root_node_id: holder,
-                nodes: { ...rn, [holder]: { id: holder, data: { type: 'root', parent: null, nodes: [r.tree.rootId] } } },
-            });
+            src = heldTree(r.tree.rootId, r.tree.nodes);
             if (r.tree.rootId !== id)
                 id = r.tree.rootId;
             reconciled = { copiedAssets: r.copiedAssets ?? 0, droppedRefs: r.droppedRefs ?? [] };
@@ -1950,7 +1956,7 @@ export function registerPageTools(server, ctx) {
         });
     });
     server.registerTool('sb_template_use', {
-        description: 'Instantiate a section template into a page — the site\'s own (the server copies it) or ' +
+        description: 'Add a section template to a page, above the footer — a sb_templates row (another site\'s is copied in first) or ' +
             'one of the BUILT-IN layouts sb_templates lists, which are composed against this ' +
             "page's own tokens rather than copied.",
         inputSchema: {
@@ -1962,9 +1968,9 @@ export function registerPageTools(server, ctx) {
         annotations: { readOnlyHint: false, destructiveHint: false },
     }, async ({ site_id: given, template_id, page_id, dry_run }) => {
         const site_id = siteFor(ctx, given);
-        // A BUILT-IN IS NOT A SERVER COPY. The platform's own templates are
-        // instantiated by the platform, which is why the section arrives exactly
-        // as designed; a built-in has no row on the server, so it is composed HERE
+        // A BUILT-IN IS NOT A SERVER COPY. A server row is a stored tree
+        // (copied in by instantiate when another site owns it), which is why the
+        // section arrives exactly as designed; a built-in has no row on the server, so it is composed HERE
         // — against the target page's own tokens, which is the whole point of it
         // being a pattern rather than a snapshot.
         const pattern = PATTERN_BY_ID.get(template_id);
@@ -2068,23 +2074,68 @@ export function registerPageTools(server, ctx) {
                     : {}),
             });
         }
-        const path = `/api/sites/${encodeURIComponent(site_id)}/section-templates/${encodeURIComponent(template_id)}/instantiate`;
-        if (dry_run !== false) {
-            return text({ dry_run: true, would_post: path, body: { pageId: page_id } });
-        }
-        const out = await request({
+        // A SERVER-SIDE TEMPLATE IS A TREE THIS SERVER INSERTS, as the editor does
+        // (TemplateCards.vue). The site's own is placed straight from the list's
+        // `document`; anyone else's (org, platform) goes through instantiate, which
+        // reads NO body, touches no page, copies the template's images into this
+        // site and answers `{ document, droppedRefs }`
+        // (sectiontemplates/rest/rest.go). Either way the ids are minted fresh here,
+        // as the editor's `cloneTree` does.
+        const tplBase = `/api/sites/${encodeURIComponent(site_id)}/section-templates`;
+        const listed = (await request({
             base: ctx.base,
-            method: 'POST',
-            path,
+            method: 'GET',
+            path: tplBase,
             token: siteToken(ctx),
-            body: { pageId: page_id },
             fetchImpl: ctx.fetchImpl,
-        });
+        }));
+        const row = (listed.sectionTemplates ?? []).find((t) => t.id === template_id);
+        if (!row) {
+            throw new Error(`sbuilder: no template "${template_id}" on this site's shelf or among the built-ins — list them with sb_templates.`);
+        }
+        const own = row.source === 'site';
+        const path = `${tplBase}/${encodeURIComponent(template_id)}/instantiate`;
+        const insert = (tree, target) => {
+            if (!tree?.nodes || !tree.root_node_id || !(tree.root_node_id in tree.nodes))
+                return null;
+            return copyNodeInto(heldTree(tree.root_node_id, tree.nodes), tree.root_node_id, target, target.doc.root_node_id, middleEnd(target.doc));
+        };
+        const doc = dry_run !== false
+            ? PageDoc.from((await loadSource(ctx, site_id, page_id)).document)
+            : (await session.open(site_id, page_id), session.current());
+        // Checked on the listed document BEFORE instantiate: that call copies media,
+        // and a refusal after it would leave orphaned images in the library.
+        const planned = insert(row.document, doc);
+        const refuse = planned ? saveRefusal(doc, planned.patches).refusal : undefined;
+        if (dry_run !== false) {
+            return text({
+                dry_run: true,
+                ...(refuse ? { would_refuse: refuse } : {}),
+                would_add: row.name ?? template_id,
+                ...(planned ? { nodes: planned.ids.length } : {}),
+                into: page_id,
+                ...(own ? {} : { would_post: path, note: "instantiate copies the template's images into this site first" }),
+            });
+        }
+        if (refuse)
+            throw new Error(`sbuilder: ${refuse}`);
+        let placed = planned;
+        let dropped = [];
+        if (!own) {
+            const out = (await request({ base: ctx.base, method: 'POST', path, token: siteToken(ctx), fetchImpl: ctx.fetchImpl }));
+            placed = insert(out.document, doc);
+            dropped = out.droppedRefs ?? [];
+        }
+        if (!placed)
+            throw new Error(`sbuilder: template "${template_id}" carries no usable document — nothing was added.`);
+        await session.applyAndSave(placed.patches);
         return text({
-            instantiated: template_id,
+            added: row.name ?? template_id,
+            section: placed.ids[0],
+            nodes: placed.ids.length,
             into: page_id,
-            result: out,
-            note: 'Re-open the page with sb_page_open — this session still holds the old tree.',
+            rev: doc.rev,
+            ...(dropped.length ? { dropped_refs: dropped } : {}),
         });
     });
     server.registerTool('sb_page_list', {

package/dist/tools/pageupdate.js

@@ -39,13 +39,14 @@ const LINK_TYPE_FOR_PAGE_TYPE = {
     post: 'article',
     blog: 'blogCategory',
     course: 'course',
+    brand: 'brand',
 };
 const PUBLISH_NOTICE = 'Page settings are published verbatim: the storefront shows this change after sb_publish.';
 export function registerPageUpdateTools(server, ctx) {
     server.registerTool('sb_page_update', {
         description: "Edit a page's name, slug, home flag, order, SEO (merged key by key) or members_only. " +
             'Reads the page first and merges `settings`, so keys it does not name survive. On a ' +
-            'template page (product, category, post, blog, course): default_template makes it the ' +
+            'template page (product, category, post, blog, course, brand): default_template makes it the ' +
             'type\'s default, render_for/render_default move entities onto it or back to the default.',
         inputSchema: {
             site_id: z.string().optional(),

package/package.json

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