sbuilder-mcp 0.58.0 → 0.59.0

package/CHANGELOG.md

@@ -6,6 +6,11 @@ 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

package/CHANGELOG.vi.md

@@ -6,6 +6,11 @@ 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

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, blogCategoryList, courseList] = 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`),
@@ -59,6 +59,11 @@ export async function gatherReadiness(ctx, siteId, pageNodes) {
         // 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:
@@ -116,5 +121,8 @@ export async function gatherReadiness(ctx, siteId, pageNodes) {
         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

package/package.json

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