sbuilder-mcp 0.87.0 → 0.88.0

package/CHANGELOG.md

@@ -6,6 +6,16 @@ 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.88.0] - 2026-10-12
+
+### Added
+- `sb_review` reports `default_seed_copy` for pages built earlier that still carry the English seed copy, such as "Add To Cart", on a Vietnamese site, with the Vietnamese wording to use instead.
+
+### Changed
+- `sb_page_create` writes the store page seeds in Vietnamese for a Vietnamese site, taking the locale from the site's settings when none is passed.
+- `sb_page_create` uses English thank-you copy on the completed-order page of an `en-US` site.
+- `sb_review` words its `categoryImages` note to say that a collection tile without an image now shows a placeholder instead of a broken image.
+
 ## [0.87.0] - 2026-10-12
 
 ### Added

package/CHANGELOG.vi.md

@@ -6,6 +6,16 @@ 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.88.0] - 2026-10-12
+
+### Added
+- `sb_review` báo `default_seed_copy` cho trang dựng trước đó còn mang chữ mẫu tiếng Anh, như "Add To Cart", trên site tiếng Việt, kèm bản tiếng Việt nên dùng.
+
+### Changed
+- `sb_page_create` viết seed trang cửa hàng bằng tiếng Việt cho site tiếng Việt, lấy locale từ cài đặt của site khi không truyền.
+- `sb_page_create` dùng câu cảm ơn tiếng Anh cho trang hoàn tất đơn hàng của site `en-US`.
+- `sb_review` sửa câu báo `categoryImages`: ô danh mục thiếu ảnh giờ hiện placeholder thay vì ảnh vỡ.
+
 ## [0.87.0] - 2026-10-12
 
 ### Added

package/dist/catalog/deadkeys.generated.js

@@ -1,6 +1,6 @@
 export const DEAD_KEY_SOURCE = {
-    "files": 5691,
-    "identifiers": 104085,
+    "files": 5692,
+    "identifiers": 104086,
     "seededKeys": 445,
     "dead": 0
 };

package/dist/catalog/source.generated.js

@@ -1,7 +1,7 @@
 // GENERATED by scripts/gen-catalog.ts — do not edit by hand.
 // The platform commit every other generated file here describes.
 export const PLATFORM_SOURCE = {
-    "commit": "df9accca9c8a0e402ca5520beddcb52c0843f7f5",
-    "committedAt": "2026-10-12T09:14:30+07:00",
+    "commit": "74ca59dd2340133e4ecf7f2faaf0947874ec3554",
+    "committedAt": "2026-10-12T10:37:24+07:00",
     "dirty": false
 };

package/dist/domains/site/readiness.js

@@ -612,11 +612,11 @@ export function readinessGaps(input) {
                 '/api/media/{siteId}/from-url ingests straight from a url in one hop.',
         });
     }
-    // A COLLECTION TILE WITH NO PICTURE IS A BROKEN-IMAGE ICON, not a grey card:
-    // render/nodes/collection-media writes `<img src="">` (the image element has a
-    // placeholder; this one does not). Measured on the sb_category_strip a store
-    // built here put on its home page — two broken icons above the category names.
-    // Asked only where a collection tile is actually on screen.
+    // A COLLECTION TILE WITH NO PICTURE. Measured on the sb_category_strip a store
+    // built here put on its home page: render/nodes/collection-media wrote
+    // `<img src="">`, two broken icons above the category names. web_builder PR
+    // #118 (74ca59dd2) gives it the image element's grey placeholder instead —
+    // better, still not the category. Asked only where a tile is on screen.
     if ((input.categoriesUnpictured ?? 0) > 0 &&
         [...input.pageNodes, ...(input.globalNodes ?? [])].some((n) => n.data?.type === 'collection-media')) {
         const n = input.categoriesUnpictured;
@@ -624,7 +624,8 @@ export function readinessGaps(input) {
             id: 'categoryImages',
             draft: false,
             problem: `${n} product categor${n === 1 ? 'y has' : 'ies have'} no image, and this page shows category ` +
-                'tiles — each renders a broken-image icon above its name.',
+                "tiles — each draws the platform's grey placeholder (a broken-image icon on a platform " +
+                'older than web_builder 74ca59dd2) where the picture goes.',
             fix: 'Upload a picture (sb_media_upload), then PUT the category back with image: url ' +
                 '(sb_api_call put:/api/sites/{siteId}/product-categories/{id}, merge:true).',
         });

package/dist/domains/site/review.js

@@ -3,6 +3,7 @@ import { childrenOf, childrenWithSatellites, isOverlay, isSharedSection, pageChi
 import { STUCK_STATE, stickyBlockedBy, stuckHostOf, isPinnedNode } from './sticky.js';
 import { bandOf, isGlobal } from './traps.js';
 import { exampleCopy } from './examplecopy.js';
+import { SEED_COPY_VI, langOf } from './seedcopy.js';
 import { HOVER_STATE, hoverHome } from './hover.js';
 import { ELEMENTS, BINDING_SOURCES, BOUND_SPECIALS, FIRST_CHILD_ONLY, SATELLITE_RULES, ELEMENT_SEEDS } from '../../catalog/elements.generated.js';
 import { fill } from './findings.js';
@@ -867,6 +868,23 @@ opts = {}) {
                     fix: fill('default_seed_copy', { id, key }),
                 });
             }
+            // THE STORE SEEDS' ENGLISH COPY ON A VIETNAMESE SITE ("Add To Cart") —
+            // fine on an English site, so asked only when the site's language is
+            // known. sb_page_create now seeds it in Vietnamese; this is for pages
+            // made before that, or with locale:"en" on a Vietnamese site.
+            else if (typeof value === 'string' &&
+                SEED_COPY_VI[value] &&
+                langOf(opts.siteLocale) === 'vi') {
+                out.push({
+                    code: 'default_seed_copy',
+                    nodeId: id,
+                    type,
+                    problem: `Still the platform's English seed copy (${JSON.stringify(value)}) on a Vietnamese site — ` +
+                        `"${SEED_COPY_VI[value]}" is what this storefront should say.`,
+                    key,
+                    fix: fill('default_seed_copy', { id, key }),
+                });
+            }
             // A BUILT-IN LAYOUT'S EXAMPLE COPY — see examplecopy.ts.
             else if (typeof value === 'string' && exampleCopy().has(value)) {
                 out.push({

package/dist/domains/site/seedcopy.js

@@ -0,0 +1,28 @@
+/**
+ * THE STORE SEEDS' ENGLISH COPY, IN VIETNAMESE — applied when the page is made
+ * for a Vietnamese site.
+ *
+ * The platform seeds in English on purpose (editor/src/element/emptyState.ts:
+ * "a seed is a seed", decided 2026-09-17) and a table per language was offered
+ * there and declined. Here the agent IS the author, and it knows the site's
+ * language: measured on a store built end to end, the product page published
+ * "Add To Cart" / "Buy It Now" to a Vietnamese storefront and every page
+ * reviewed clean. sb_page_create seeds through this; sb_review (which reads the
+ * site's locale) reports a page still carrying the English. Exact strings only; a seed string
+ * this table does not carry is left as it was — and test/store-page-seeds
+ * fails when codegen brings one in, so the table cannot fall behind silently.
+ */
+export const SEED_COPY_VI = {
+    'Add To Cart': 'Thêm vào giỏ',
+    'Buy It Now': 'Mua ngay',
+    'This item is unavailable': 'Sản phẩm tạm hết',
+    'It may have been removed. Have a look at the rest of the shop.': 'Sản phẩm có thể đã ngừng bán. Mời bạn xem các sản phẩm khác.',
+    'No products yet': 'Chưa có sản phẩm',
+    'New arrivals will show up here. Check back soon.': 'Sản phẩm mới sẽ sớm có ở đây — mời bạn quay lại sau.',
+    'No posts yet': 'Chưa có bài viết',
+    'New stories will show up here. Check back soon.': 'Bài viết mới sẽ sớm có ở đây.',
+};
+/** The language part of a locale tag — "vi-VN", "vi_VN" and "VI" are all "vi". */
+export function langOf(locale) {
+    return String(locale ?? '').toLowerCase().split(/[-_]/)[0];
+}

package/dist/domains/site/storepage.js

@@ -1,3 +1,4 @@
+import { SEED_COPY_VI, langOf } from './seedcopy.js';
 import { COMPLETION_HEADLINE, COMPLETION_HEADLINE_SENTINEL, PAGE_LAYOUT_SEEDS, PAGE_TYPES, STORE_PAGE_SEEDS } from '../../catalog/storepages.generated.js';
 import { BINDING_SOURCES } from '../../catalog/elements.generated.js';
 import { withFreshIds } from './ids.js';
@@ -40,22 +41,44 @@ export function seededTypes() {
  * An explicit `headline` wins over both — a merchant's own wording is not the
  * client's to overrule.
  */
+/** Rewrite every exact seed string the table carries, in place. */
+function localizeSeed(doc, table) {
+    for (const n of Object.values(doc.nodes)) {
+        const sp = n.specials;
+        if (!sp)
+            continue;
+        for (const k of ['text', 'label']) {
+            const v = sp[k];
+            if (typeof v === 'string' && table[v])
+                sp[k] = table[v];
+        }
+    }
+}
+export { SEED_COPY_VI } from './seedcopy.js';
 export function seedDocument(type, opts = {}) {
     const seed = STORE_PAGE_SEEDS[type];
     if (!seed)
         return null;
     const raw = JSON.stringify(seed);
+    const vi = langOf(opts.locale ?? 'vi') === 'vi';
+    const out = (doc) => {
+        if (vi)
+            localizeSeed(doc, SEED_COPY_VI);
+        return withFreshIds(doc);
+    };
     // Fresh ids per create, so two pages from one seed share nothing but ROOT.
     if (!raw.includes(COMPLETION_HEADLINE_SENTINEL)) {
-        return withFreshIds(JSON.parse(raw));
+        return out(JSON.parse(raw));
     }
-    const lang = opts.locale && opts.locale in COMPLETION_HEADLINE ? opts.locale : 'vi';
+    // Normalized: a site whose locale is "en-US" reads its own language here too.
+    const tag = langOf(opts.locale);
+    const lang = tag && tag in COMPLETION_HEADLINE ? tag : 'vi';
     const headline = opts.headline || COMPLETION_HEADLINE[lang];
     // JSON.stringify already escaped the seed; the substitute has to be escaped
     // the same way or an apostrophe in a merchant's own headline breaks the
     // document. Slicing the quotes off a stringified string is that escaping.
     const escaped = JSON.stringify(headline).slice(1, -1);
-    return withFreshIds(JSON.parse(raw.split(COMPLETION_HEADLINE_SENTINEL).join(escaped)));
+    return out(JSON.parse(raw.split(COMPLETION_HEADLINE_SENTINEL).join(escaped)));
 }
 /**
  * What to put on an ENTITY TEMPLATE that opens empty, or null.

package/dist/tools/page.js

@@ -47,8 +47,8 @@ import { projectList, PAGE_FIELDS, TEMPLATE_FIELDS } from './project.js';
  * copies of a directive is how one of them quietly loses it. The fix for each
  * KIND is sent once under `fixes`, and the directive once per process.
  */
-export function reviewField(ctx, doc, formTypes, pageType, formDocType, formRedirects) {
-    const all = reviewDesign(doc, { formTypes, pageType, formDocType, formRedirects });
+export function reviewField(ctx, doc, formTypes, pageType, formDocType, formRedirects, siteLocale) {
+    const all = reviewDesign(doc, { formTypes, pageType, formDocType, formRedirects, siteLocale });
     if (all.length === 0)
         return {};
     // MAINTENANCE IS ADVICE, kept out of `findings`: it works for a visitor, so
@@ -1807,6 +1807,7 @@ export function registerPageTools(server, ctx) {
         // The page's TYPE says which record it supplies — `no_data_context` stays
         // silent without it rather than guessing.
         let pageType;
+        let siteLocale;
         try {
             const { siteId, pageId } = session.location();
             const input = await gatherReadiness(ctx, siteId, Object.values(doc.doc.nodes), pageId);
@@ -1817,6 +1818,7 @@ export function registerPageTools(server, ctx) {
                 return f.id && a?.action === 'redirect' && (a.url || a.pageId) ? [f.id] : [];
             }));
             pageType = input.pages?.find((p) => p.id === pageId)?.type;
+            siteLocale = input.siteLocale;
             const gaps = readinessGaps(input);
             const lang = siteLanguage(input.siteLocale, input.pageNodes);
             const said = lang ? ctx.notices.once(`site-language:${siteId}`, lang) : undefined;
@@ -1830,7 +1832,7 @@ export function registerPageTools(server, ctx) {
         catch {
             // Readiness is additional information, never the reason a review fails.
         }
-        const field = reviewField(ctx, doc, formTypes, pageType, session.formType(), formRedirects);
+        const field = reviewField(ctx, doc, formTypes, pageType, session.formType(), formRedirects, siteLocale);
         // Clean = no finding a visitor meets; `advice` (maintenance) rides beside the verdict.
         const clean = !field.findings;
         return text({
@@ -2237,7 +2239,7 @@ export function registerPageTools(server, ctx) {
             settings: z.record(z.unknown()).optional(),
             seed: z.boolean().optional().describe('Default true; false creates a blank page.'),
             chrome: z.boolean().optional().describe("Carry the site's header and footer, default true"),
-            locale: z.string().optional().describe("vi (default) or en — the complete page's wording."),
+            locale: z.string().optional().describe("vi or en — a store seed's wording; defaults to the site's language, else vi."),
             headline: z.string().optional().describe("The complete page's thank-you line."),
             dry_run: z.boolean().optional(),
         },
@@ -2420,7 +2422,25 @@ export function registerPageTools(server, ctx) {
         let seeded = null;
         const newId = res.page?.id;
         if ((willSeed || layout) && typeof newId === 'string' && newId) {
-            const document = layout ? layoutDocument(layout) : seedDocument(type ?? '', { locale, headline });
+            // The SITE's language when the caller named none — the seed's English
+            // copy is rewritten for a Vietnamese site (SEED_COPY_VI). Unreadable
+            // settings fall back to the old default rather than block the create.
+            const lang = locale ??
+                (layout
+                    ? undefined
+                    : await request({
+                        base: ctx.base,
+                        method: 'GET',
+                        path: `/api/sites/${encodeURIComponent(site_id)}/settings`,
+                        token: siteToken(ctx),
+                        fetchImpl: ctx.fetchImpl,
+                    })
+                        .then((r) => {
+                        const l = r?.settings?.locale;
+                        return typeof l === 'string' && l ? l : undefined;
+                    })
+                        .catch(() => undefined));
+            const document = layout ? layoutDocument(layout) : seedDocument(type ?? '', { locale: lang, headline });
             if (document) {
                 try {
                     await request({

package/package.json

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