sbuilder-mcp 0.7.0 → 0.7.2

package/CHANGELOG.md

@@ -6,6 +6,20 @@ 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
+- sb_import takes max_images (default 24), bounding how many images it uploads from the imported page, since every image is a real upload and a page shape like a sponsors wall can carry dozens of them in a single tool call.
+
+### Fixed
+- sb_import no longer duplicates content that sits inside a nested section; it previously matched every `<section>` on the page, so an outer band and the bands nested inside it were both captured and the inner content came back twice. Only the innermost matching section is now kept, since the outermost is a candidate too and keeping it would reduce the whole page to one band.
+
 ## [0.7.0] - 2026-09-08
 
 ### Added

package/CHANGELOG.vi.md

@@ -6,6 +6,20 @@ 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
+- sb_import nhận max_images (mặc định 24), giới hạn số ảnh nó tải lên từ trang được import, vì mỗi ảnh là một lần upload thật và một dạng trang như tường tài trợ có thể chứa hàng chục ảnh trong một lần gọi tool.
+
+### Fixed
+- sb_import không còn nhân đôi nội dung nằm trong một section lồng nhau; trước đây nó khớp với mọi `<section>` trên trang, nên một band bên ngoài và các band lồng bên trong nó đều bị lấy, khiến nội dung bên trong xuất hiện hai lần. Giờ chỉ section trong cùng khớp được giữ lại, vì section ngoài cùng cũng là một ứng viên và giữ nó sẽ làm cả trang chỉ còn một band.
+
 ## [0.7.0] - 2026-09-08
 
 ### Added

package/dist/tools/importpage.js

@@ -34,6 +34,13 @@ export function registerImportTools(server, ctx, session) {
             url: z.string().describe('The page to read'),
             site_id: z.string().optional(),
             max_sections: z.number().int().min(1).max(60).optional(),
+            max_images: z
+                .number()
+                .int()
+                .min(0)
+                .max(100)
+                .optional()
+                .describe('Default 24 — every image is an upload'),
             upload_images: z
                 .boolean()
                 .optional()
@@ -41,13 +48,13 @@ export function registerImportTools(server, ctx, session) {
             dry_run: z.boolean().optional(),
         },
         annotations: { readOnlyHint: false, destructiveHint: false, openWorldHint: true },
-    }, async ({ url, site_id: given, max_sections, upload_images, dry_run }) => {
+    }, async ({ url, site_id: given, max_sections, max_images, upload_images, dry_run }) => {
         const siteId = siteFor(ctx, given);
         // THE TARGET PAGE MUST BE OPEN, and not only because that is where the
         // nodes go: its own heading, button and section are where the tokens come
         // from, so an import with no open page is an import with no design.
         const doc = session.current();
-        const shot = await capture(url, { maxSections: max_sections });
+        const shot = await capture(url, { maxSections: max_sections, maxImages: max_images });
         const tokens = tokensFromPage(doc.doc);
         const images = imageSources(shot.sections);
         if (dry_run !== false) {

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.
  *
@@ -12,6 +13,7 @@ function capturePage(limits) {
     // page, as `ReferenceError: HEADINGS is not defined`, by which point the file
     // already carried a comment saying exactly that.
     const HEADINGS = new Set(['H1', 'H2', 'H3', 'H4', 'H5', 'H6']);
+    const taken = { images: 0 };
     const skipped = {};
     const skip = (why) => void (skipped[why] = (skipped[why] ?? 0) + 1);
     const here = location.href;
@@ -40,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',
@@ -73,11 +90,21 @@ function capturePage(limits) {
             }
             if (tag === 'IMG') {
                 const src = el.getAttribute('src');
-                if (src && !src.startsWith('data:')) {
-                    out.push({ kind: 'image', src: abs(src), alt: clean(el.getAttribute('alt')) });
-                }
-                else
+                if (!src || src.startsWith('data:')) {
                     skip('image-without-src');
+                    return;
+                }
+                // BOUNDED, because every image is an upload. A sponsors wall is a real
+                // page shape — one measured at 36 logos in four sections — and importing
+                // it means 36 sequential HTTP round trips inside a single tool call,
+                // which is slow, half-fails in interesting ways, and is almost never
+                // what the caller wanted from "import this page".
+                if (taken.images >= limits.maxImages) {
+                    skip('over-image-limit');
+                    return;
+                }
+                taken.images++;
+                out.push({ kind: 'image', src: abs(src), alt: clean(el.getAttribute('alt')) });
                 return;
             }
             if (tag === 'A' && looksLikeButton(el)) {
@@ -117,6 +144,16 @@ function capturePage(limits) {
         const main = document.querySelectorAll('main')[0] ?? document.body;
         candidates = Array.from(main.children);
     }
+    // INNERMOST ONLY. `section` matches nested ones too, so an outer band and the
+    // bands inside it were both taken — and the inner content came back TWICE,
+    // once through its parent's leaf walk and once on its own. Measured: 15
+    // duplicated strings out of 22 on one real page, 12 on another, which on an
+    // imported page reads as a stutter nobody typed.
+    //
+    // Innermost rather than outermost because a `<section>` that wraps the whole
+    // document is one candidate too, and keeping THAT would reduce every page to a
+    // single band. The finest ones are the page's actual bands.
+    candidates = candidates.filter((el) => !candidates.some((o) => o !== el && el.contains(o)));
     const sections = [];
     for (const el of candidates) {
         if (sections.length >= limits.maxSections) {
@@ -147,6 +184,7 @@ export async function capture(url, opts = {}) {
     const limits = {
         maxSections: opts.maxSections ?? 24,
         maxPerSection: opts.maxPerSection ?? 40,
+        maxImages: opts.maxImages ?? 24,
     };
     let browser;
     try {
@@ -160,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.0",
+  "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",