sbuilder-mcp 0.57.0 → 0.59.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.59.0] - 2026-09-15
+
+### Added
+- `sb_review` now reports a `maintenancePage` readiness gap when a site has maintenance mode switched on but no published page of the `maintain` type, since every public address then answers a bare 503 sentence with no header, logo, or word about when the shop is back; the check reads the site's own maintenance flag and stays silent for a site that is not currently in maintenance.
+
+## [0.58.0] - 2026-09-15
+
+### Added
+- `sb_review` now reports `blogCategoryPage` and `coursePage` readiness gaps when a site has blog categories or courses but no published page of the `blog` or `course` type, since `/blog-categories/{slug}` and `/courses/{slug}` fall back to those templates and every entity link 404s without them, completing the check against all five entity prefixes the storefront registers.
+
 ## [0.57.0] - 2026-09-15
 
 ### 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.59.0] - 2026-09-15
+
+### Added
+- `sb_review` giờ báo lỗi sẵn sàng `maintenancePage` khi site đã bật chế độ bảo trì nhưng chưa có trang loại `maintain` nào được publish, vì khi đó mọi địa chỉ công khai sẽ chỉ trả về một câu 503 trơn, không header, không logo, không một dòng nào báo khi nào shop mở lại; việc kiểm tra đọc thẳng cờ bảo trì của site và im lặng nếu site hiện không ở chế độ bảo trì.
+
+## [0.58.0] - 2026-09-15
+
+### Added
+- `sb_review` giờ báo lỗi sẵn sàng `blogCategoryPage` và `coursePage` khi site có danh mục blog hoặc khoá học nhưng chưa có trang loại `blog` hoặc `course` nào được publish, vì `/blog-categories/{slug}` và `/courses/{slug}` sẽ dùng các template đó, và mọi liên kết sẽ trả về 404 nếu thiếu — hoàn tất việc kiểm tra cho cả năm tiền tố thực thể mà storefront đăng ký.
+
 ## [0.57.0] - 2026-09-15
 
 ### Added

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

@@ -24,7 +24,7 @@ export async function gatherReadiness(ctx, siteId, pageNodes) {
         }
     };
     const site = encodeURIComponent(siteId);
-    const [pageList, gateways, shipping, globals, productList, categoryList, pageLinks, formList, articleList] = await Promise.all([
+    const [pageList, gateways, shipping, globals, productList, categoryList, pageLinks, formList, articleList, blogCategoryList, courseList, siteRecord] = await Promise.all([
         get(`/api/sites/${site}/pages`),
         get(`/api/sites/${site}/payment-gateways`),
         get(`/api/sites/${site}/shipping-methods`),
@@ -52,6 +52,18 @@ export async function gatherReadiness(ctx, siteId, pageNodes) {
         // resolves to the site's `post` template, so a site that has written
         // articles and has no template 404s every one of them.
         get(`/api/sites/${site}/articles?limit=1`),
+        // THE LAST TWO ENTITY PREFIXES the storefront registers
+        // (storefront/entityroute.go): /blog-categories/{slug} and /courses/{slug}.
+        // Each resolves to the site's published page of its own TYPE, so each has
+        // the hole /products/{slug} has, and counting is how we know the site uses
+        // the prefix at all.
+        get(`/api/sites/${site}/blog-categories?limit=1`),
+        get(`/api/sites/${site}/courses?limit=1`),
+        // THE SWITCH ITSELF. `maintain` is the one page type whose absence matters
+        // only while a flag is on — see maintenanceGap. Read here rather than
+        // guessed, because the alternative is telling every site on earth to build
+        // a page for an outage it is not having.
+        get(`/api/sites/${site}`),
     ]);
     // A gateway counts only when it is BOTH enabled and configured — the editor's
     // `live` getter also filters by the store's currency, which is a narrowing:
@@ -62,6 +74,10 @@ export async function gatherReadiness(ctx, siteId, pageNodes) {
         : null;
     const methods = shipping?.shippingMethods ?? shipping?.methods;
     const shippingMethods = Array.isArray(methods) ? methods.length : null;
+    // Either key, like the product categories below: this reader has met both
+    // spellings from the platform and guessing one would read a real list as none.
+    const rawBlogCats = blogCategoryList?.blogCategories ?? blogCategoryList?.categories;
+    const blogCats = Array.isArray(rawBlogCats) ? rawBlogCats : null;
     const cats = categoryList?.categories ?? categoryList?.productCategories;
     const categories = Array.isArray(cats) ? (categoryList?.total ?? cats.length) : null;
     const links = pageLinks?.links ?? pageLinks?.pageLinks;
@@ -101,5 +117,12 @@ export async function gatherReadiness(ctx, siteId, pageNodes) {
         articles: Array.isArray(articleList?.articles)
             ? (articleList?.total ?? articleList.articles.length)
             : null,
+        blogCategories: blogCats ? (blogCategoryList?.total ?? blogCats.length) : null,
+        courses: Array.isArray(courseList?.courses)
+            ? (courseList?.total ?? courseList.courses.length)
+            : null,
+        maintenanceMode: typeof siteRecord?.site?.maintenanceMode === 'boolean'
+            ? siteRecord.site.maintenanceMode
+            : null,
     };
 }

package/dist/domains/site/readiness.js

@@ -96,6 +96,36 @@ export function readinessGaps(input) {
                 'globalRef instead of a copy, and one edit reaches every page.',
         });
     }
+    // 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
+    // a visitor might one day meet: with the switch on, every public address is
+    // answering 503 this second. `maintenance.go` serves the published `maintain`
+    // page as that body when there is one, and a PLAIN SENTENCE when there is
+    // not — and it FAILS CLOSED on purpose, so an unresolvable page does not
+    // reopen the shop. The owner gets the outage they asked for either way; what
+    // they lose is every word of it.
+    //
+    // ONLY WHILE THE SWITCH IS ON. Telling a site to build a page for an outage
+    // it is not having is advice about a page nobody will see, and it would
+    // arrive on every review of every site — the nag this file keeps warning
+    // about, at the widest possible scale. Silent when the site could not be
+    // read, like everything else.
+    if (pages && input.maintenanceMode === true && !published(pages, 'maintain')) {
+        const draft = drafted(pages, 'maintain');
+        gaps.push({
+            id: 'maintenancePage',
+            draft,
+            problem: 'This storefront is switched OFF right now and has no published page of the "maintain" ' +
+                'type, so every address is answering a bare 503 sentence — no header, no logo, no word ' +
+                'about when it is back, and nothing that looks like this shop.',
+            fix: draft
+                ? 'Publish the maintenance page that already exists.'
+                : 'Create a page of type "maintain" and publish it. It has no address of its own — the ' +
+                    'storefront serves it as the BODY of the 503 — so give it the shop\'s name and a line ' +
+                    'saying when you are back.',
+        });
+    }
     // A MISTYPED URL ANSWERS IN GO, NOT IN THE SHOP'S VOICE.
     //
     // storefront/errorpage.go: with no PUBLISHED page of type `error`, notFound
@@ -203,6 +233,35 @@ export function readinessGaps(input) {
                     'headline, date and body an article cannot be read without.',
         });
     }
+    // THE LAST TWO ENTITY PREFIXES, the same hole in the same shape.
+    //
+    // storefront/entityroute.go registers five: /products, /collections, /blog,
+    // /blog-categories and /courses. Each resolves to the site's published page of
+    // its own TYPE, so each 404s everything under it when that page is missing.
+    // Four of the five are now checked above; these are the other two.
+    //
+    // COUNTED, NOT ASSUMED. A site with no blog categories has nothing under
+    // /blog-categories to break, and a site with no courses has not installed the
+    // app — the count IS the gate, which is why neither needs to ask about the
+    // app separately. Both sit above the store gate: a course is sold, but a blog
+    // is not commerce, and a site can be neither and still have one.
+    for (const [count, type, id, prefix, what] of [
+        [input.blogCategories, 'blog', 'blogCategoryPage', '/blog-categories/{slug}', 'blog category'],
+        [input.courses, 'course', 'coursePage', '/courses/{slug}', 'course'],
+    ]) {
+        if (!pages || (count ?? 0) === 0 || published(pages, type))
+            continue;
+        const draft = drafted(pages, type);
+        gaps.push({
+            id,
+            draft,
+            problem: `This site has ${count} ${what}${count === 1 ? '' : count && count > 1 ? 's' : ''} and no ` +
+                `published page of the "${type}" type. ${prefix} needs one, so every ${what} link 404s.`,
+            fix: draft
+                ? `Publish the ${what} template page that already exists.`
+                : `Create a page of type "${type}" and publish it.`,
+        });
+    }
     if (!isStore(input))
         return gaps;
     if (pages && !published(pages, 'checkout')) {

package/package.json

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