sbuilder-mcp 0.7.1 → 0.7.2

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.7.2] - 2026-09-08
+
+### Fixed
+- sb_import no longer classifies a short block-level link as a button; it previously stopped at "not inline", so a documentation sidebar's list of navigation links came back as dozens of buttons. A real call to action must now be painted with a fill or a border, and a border only counts when it has width, since a Tailwind-built page sets `border-style: solid; border-width: 0` on every element.
+- sb_import's page capture no longer waits a flat 600ms before reading the page; it now reuses sb_look's own settle check, which waits until the page actually stops changing rather than a fixed delay that is too long for a static page and too short for one that builds itself with scripts.
+
 ## [0.7.1] - 2026-09-08
 
 ### Added

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.7.2] - 2026-09-08
+
+### Fixed
+- sb_import không còn xếp một liên kết block-level ngắn vào loại button; trước đây rule chỉ dừng ở "không phải inline", nên danh sách liên kết điều hướng trong sidebar tài liệu bị trả về thành hàng chục button. Giờ một call-to-action thật phải được "tô" bằng màu nền hoặc viền, và viền chỉ được tính khi có độ dày, vì một trang dựng bằng Tailwind đặt `border-style: solid; border-width: 0` trên mọi phần tử.
+- Bước capture trang của sb_import không còn chờ cố định 600ms trước khi đọc trang; giờ nó dùng lại chính cơ chế chờ ổn định (settle) của sb_look, chờ đến khi trang thực sự ngừng thay đổi thay vì một khoảng chờ cố định vừa quá dài với trang tĩnh vừa quá ngắn với trang tự dựng bằng script.
+
 ## [0.7.1] - 2026-09-08
 
 ### Added

package/dist/vision/capture.js

@@ -1,4 +1,5 @@
 import { chromium } from 'playwright-core';
+import { settleDom } from './shoot.js';
 /**
  * The in-page walk, as a string the browser evaluates.
  *
@@ -41,11 +42,26 @@ function capturePage(limits) {
     // label in a box with a background. Everything else is prose with a link in
     // it, and turning those into buttons produces a page of buttons.
     const looksLikeButton = (el) => {
+        const t = clean(el.textContent);
+        if (t.length === 0 || t.length > 32)
+            return false;
         const cls = typeof el.className === 'string' ? el.className.toLowerCase() : '';
         if (/\b(btn|button|cta)\b/.test(cls))
             return true;
-        const t = clean(el.textContent);
-        return t.length > 0 && t.length <= 32 && getComputedStyle(el).display !== 'inline';
+        // A SHORT BLOCK-LEVEL LINK IS USUALLY NAVIGATION, not a call to action. The
+        // rule used to stop at "not inline" and a documentation sidebar came back as
+        // 38 buttons — a page of pink pills where the source had a list of links.
+        // A real CTA is PAINTED: it has a fill or a border. Nav links have neither.
+        const cs = getComputedStyle(el);
+        const filled = cs.backgroundColor !== '' &&
+            cs.backgroundColor !== 'transparent' &&
+            !cs.backgroundColor.startsWith('rgba(0, 0, 0, 0)');
+        // A BORDER NEEDS WIDTH, not just a style. Tailwind's preflight sets
+        // `border-style: solid; border-width: 0` on every element, so "has a border
+        // style" is true of an entire site built with it — which is how a
+        // documentation sidebar came back as 38 buttons even after the first fix.
+        const bordered = cs.borderStyle !== '' && cs.borderStyle !== 'none' && parseFloat(cs.borderWidth || '0') > 0;
+        return cs.display !== 'inline' && (filled || bordered);
     };
     const IGNORE = new Set([
         'SCRIPT', 'STYLE', 'NOSCRIPT', 'TEMPLATE', 'SVG', 'PATH', 'IFRAME', 'CANVAS',
@@ -182,7 +198,13 @@ export async function capture(url, opts = {}) {
     try {
         page = await browser.newPage({ viewport: { width: opts.width ?? 1440, height: 900 } });
         await page.goto(url, { waitUntil: 'load', timeout: 30_000 });
-        await page.waitForTimeout(600);
+        // THE SAME SETTLE `sb_look` USES, not a flat sleep. A fixed 600ms is wrong
+        // at both ends: example.com is finished long before it, and a page that
+        // builds itself with scripts is not finished after it — which is exactly the
+        // page an import is most likely to be pointed at. `settleDom` asks the
+        // question actually being asked (has the page stopped changing) and answers
+        // when it becomes true, bounded so a page that never settles is still read.
+        await settleDom(page);
         return (await page.evaluate(capturePage, limits));
     }
     finally {

package/dist/vision/shoot.js

@@ -161,7 +161,7 @@ export async function shoot(url, opts = {}) {
  * polling widget from holding the shot forever. A page that never settles is
  * photographed anyway — a late picture beats none.
  */
-async function settleDom(page) {
+export async function settleDom(page) {
     await page
         .evaluate(({ quiet, cap }) => new Promise((resolve) => {
         const start = Date.now();

package/package.json

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