sbuilder-mcp 0.9.0 → 0.9.1

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.9.1] - 2026-09-09
+
+### Fixed
+- sb_set now refuses a `config.stuckAfter` write the renderer would drop on the floor — a negative, non-finite, or empty value, or one set on a node that cannot pin — instead of silently storing, saving, and publishing a threshold the runtime island never reads and quietly falling back to its automatic answer.
+
 ## [0.9.0] - 2026-09-08
 
 ### 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.9.1] - 2026-09-09
+
+### Fixed
+- sb_set giờ từ chối ghi `config.stuckAfter` nếu giá trị sẽ bị renderer âm thầm bỏ qua — số âm, không hữu hạn, rỗng, hoặc được đặt trên một node không thể ghim — thay vì lưu, save rồi publish một ngưỡng mà runtime island không bao giờ đọc và lặng lẽ quay về đáp án tự động của nó.
+
 ## [0.9.0] - 2026-09-08
 
 ### Added

package/dist/domains/site/builder.js

@@ -2,7 +2,7 @@ import { isOverlay, subtreeIds, ancestors, appBlockRoot, SPEC_GLOBAL_ID, SPEC_GL
 import { ELEMENTS, ELEMENT_SEEDS, SATELLITE_RULES } from '../../catalog/elements.generated.js';
 import { bindingsForConfig, createNode, mintSatellites } from './node.js';
 import { refuseSecondTemplate } from './traps.js';
-import { STUCK_STATE, refuseStuckConfig, requireStuckHost, stickySeeds } from './sticky.js';
+import { STUCK_STATE, refuseStuckAfter, refuseStuckConfig, requireStuckHost, stickySeeds, } from './sticky.js';
 import { genId } from './ids.js';
 /** Append sentinel: splice clamps a too-large index, and `isSyncablePatch`
  *  deliberately allows one — an append is a legitimate thing to describe. */
@@ -182,6 +182,12 @@ export function setKeys(doc, id, keys, opts) {
     // the same silent shape as an unbound element, which cost a whole page of
     // "$0.00" cards to find once.
     const rebind = namespace === 'config' ? rebindPatch(doc, id, keys) : null;
+    // The scroll THRESHOLD is a plain config key, and the renderer drops it in two
+    // different ways without a word — see sticky.ts. Checked here rather than in
+    // the state branch because it is written on the node that PINS, at base or at
+    // a breakpoint, never inside a stuck slot.
+    if (namespace === 'config')
+        refuseStuckAfter(doc.doc, id, keys);
     if (namespace === 'specials') {
         refuseComposedStamp(keys);
         // `specials` is content and identity, base-only by definition, and states

package/dist/domains/site/sticky.js

@@ -36,6 +36,17 @@ import { ancestors } from '../../core/tree.js';
  */
 /** The state key, in `node.states` and `responsive[bp].states`. */
 export const STUCK_STATE = 'stuck';
+/**
+ * The config key holding "pinned from WHEN?", in px of page scroll.
+ *
+ * It reaches the runtime island as a CSS custom property (`--wb-stuck-after`)
+ * rather than through `wb:props`, because it is per breakpoint and the cascade
+ * is what picks the right number at the right width. Unset is the common case:
+ * the island then decides from the element itself — a sticky one the moment it
+ * is held away from its place in the flow, a fixed one the moment the page
+ * scrolls past where it would have sat.
+ */
+export const STUCK_AFTER = 'stuckAfter';
 /**
  * The positions that HAVE a pinned moment.
  *
@@ -206,3 +217,32 @@ export function stickyWarning(doc, id, bp) {
         `scrolls. The node will not move. Clear overflowX/overflowY on ${blocker}, or pin a node ` +
         'outside it.');
 }
+/**
+ * Refuse a `stuckAfter` that the renderer would drop on the floor.
+ *
+ * `stuckAfterCss` emits the custom property only for a node that CAN pin and
+ * that carries a usable number, and returns an empty string otherwise — so both
+ * mistakes here are stored, saved, published and read by nothing. The value
+ * rules are the platform's own: zero is honoured ("as soon as the page moves at
+ * all" is a real answer, and dropping it would silently fall back to the
+ * automatic one); negative and non-finite are not, because no scroll position
+ * can satisfy them.
+ */
+export function refuseStuckAfter(doc, id, keys) {
+    if (!(STUCK_AFTER in keys))
+        return;
+    const raw = keys[STUCK_AFTER];
+    const n = typeof raw === 'number' ? raw : Number(raw);
+    if (raw == null || raw === '' || !Number.isFinite(n) || n < 0) {
+        throw new Error(`sbuilder: config.${STUCK_AFTER} is px of page scroll — a finite number, zero or more ` +
+            `(0 means "as soon as the page moves at all"). ${JSON.stringify(raw)} is dropped by ` +
+            "the renderer, which then falls back to the island's automatic answer without saying " +
+            'so. To go back to automatic, remove the key.');
+    }
+    if (!isPinnedNode(doc.nodes[id])) {
+        throw new Error(`sbuilder: config.${STUCK_AFTER} says WHEN a pinned element counts as stuck, and ${id} ` +
+            'cannot pin — the renderer emits the threshold only for a node that can ' +
+            '(stickyState.ts stuckAfterCss), so this would be stored and read by nothing. ' +
+            `Pin it first: sb_set ${id} style { position: "sticky" }.`);
+    }
+}

package/package.json

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