sbuilder-mcp 0.37.0 → 0.38.1

package/CHANGELOG.md

@@ -6,6 +6,23 @@ 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.38.1] - 2026-09-11
+
+### Fixed
+- sb_look no longer photographs a reveal-on-scroll band (config.animation trigger:"view") as blank; every animation is now stopped before the shutter opens, since the lazy-image walk returns to the top of the page and would otherwise catch the band back at its opacity:0 starting keyframe.
+- sb_import and sb_import_site no longer drop a `<video>` element that carries a poster image and no explicit width/height, which previously measured as a zero-size box and was skipped like a genuinely hidden element.
+
+## [0.38.0] - 2026-09-11
+
+### Added
+- The catalog gained a 112th element, spline-scene: an interactive 3D scene embedded from a Spline viewer link, seeded by default with a real scene from Spline's own demo project, with a warning at creation time telling the caller to replace it with the merchant's own export before publishing.
+- config.animation grew from five keys to ten (intensity, trigger, range, repeat, and alternate join active, type, easing, delay, and duration), including a new `trigger: "view"` for reveal-on-scroll animation, compiled as a pure-CSS animation-timeline override with no added script.
+- sb_traits_for now returns an animation_values field listing every legal value for each config.animation key (all 46 types, 7 easings, every intensity and trigger) for any element that carries an animation control.
+- sb_set now warns when config.animation combines alternate:true with a finite repeat count, since that combination publishes a node that finishes on its opacity:0 keyframe and is invisible to every visitor, and when intensity or trigger is set to a value the platform does not recognize.
+
+### Changed
+- config.animation is no longer base-only: sb_set now writes it per breakpoint like any other config key, instead of always routing it to base, so an animation can now be turned off on mobile.
+
 ## [0.37.0] - 2026-09-11
 
 ### Added

package/CHANGELOG.vi.md

@@ -6,6 +6,23 @@ 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.38.1] - 2026-09-11
+
+### Fixed
+- sb_look không còn chụp một band hiệu ứng xuất hiện khi cuộn (config.animation trigger:"view") thành ảnh trống nữa; mọi animation giờ được dừng lại trước khi chụp, vì bước quét ảnh lazy-load quay về đầu trang và trước đây sẽ bắt band đó đứng yên ở keyframe khởi đầu opacity:0.
+- sb_import và sb_import_site không còn bỏ sót phần tử `<video>` có ảnh poster nhưng không khai báo width/height tường minh, trước đây phần tử này đo được kích thước bằng 0 nên bị bỏ qua như một phần tử thực sự ẩn.
+
+## [0.38.0] - 2026-09-11
+
+### Added
+- Catalog có thêm element thứ 112, spline-scene: một cảnh 3D tương tác được nhúng từ link viewer của Spline, mặc định được gán sẵn một cảnh thật từ project demo của chính Spline, kèm cảnh báo ngay lúc tạo yêu cầu caller thay bằng file export của merchant trước khi publish.
+- config.animation tăng từ năm key lên mười key (intensity, trigger, range, repeat và alternate được thêm bên cạnh active, type, easing, delay và duration), trong đó có `trigger: "view"` mới cho hiệu ứng xuất hiện khi cuộn trang, được biên dịch thành một override CSS thuần bằng animation-timeline, không cần thêm script.
+- sb_traits_for giờ trả về thêm trường animation_values liệt kê mọi giá trị hợp lệ cho từng key của config.animation (đủ 46 type, 7 easing, mọi intensity và trigger) với bất kỳ element nào có control animation.
+- sb_set giờ cảnh báo khi config.animation kết hợp alternate:true với một số lần repeat hữu hạn, vì tổ hợp này publish một node kết thúc ở keyframe opacity:0 và không visitor nào nhìn thấy được, cũng như khi intensity hoặc trigger được đặt một giá trị mà nền tảng không nhận ra.
+
+### Changed
+- config.animation không còn base-only nữa: sb_set giờ ghi giá trị này theo từng breakpoint như mọi config key khác, thay vì luôn dồn về base, nên giờ có thể tắt animation riêng trên mobile.
+
 ## [0.37.0] - 2026-09-11
 
 ### Added

package/dist/catalog/element-search.js

@@ -1,5 +1,5 @@
 import { ELEMENTS, TRAIT_WRITES } from './elements.generated.js';
-import { animationVocabulary, vocabulariesForWrites } from '../domains/site/vocabulary.js';
+import { animationValues, animationVocabulary, vocabulariesForWrites } from '../domains/site/vocabulary.js';
 import { neverTranslatedOn, translatableSpecials } from '../domains/site/translate.js';
 /** Eight to choose from; the hints for the chosen one come with sb_traits_for. */
 export const DEFAULT_CATALOG_LIMIT = 8;
@@ -130,7 +130,9 @@ export function traitsFor(type, control) {
         // with a required gate, and the three ways to get it wrong all render
         // NOTHING rather than something else. An element that does not offer the
         // control says nothing, so this is silent on the other 38.
-        ...(el.controls.includes('animation') ? { animation: animationVocabulary() } : {}),
+        ...(el.controls.includes('animation')
+            ? { animation: animationVocabulary(), animation_values: animationValues() }
+            : {}),
         isContainer: el.isContainer,
         isRootOnly: el.isRootOnly,
         childAllows: el.childAllows,

package/dist/catalog/elements.generated.js

@@ -1,5 +1,5 @@
 export const ELEMENT_SOURCE = {
-    "count": 111,
+    "count": 112,
     "docSchemaVersion": 2
 };
 export const ELEMENTS = {
@@ -3686,6 +3686,146 @@ export const ELEMENTS = {
             "player"
         ]
     },
+    "spline-scene": {
+        "type": "spline-scene",
+        "label": "Spline scene",
+        "category": "media",
+        "isContainer": false,
+        "isRootOnly": false,
+        "locked": false,
+        "hideInLayer": false,
+        "childAllows": [],
+        "defaults": {
+            "specials": {
+                "sceneUrl": "https://prod.spline.design/HqdfCmOueigtautT/scene.splinecode",
+                "posterUrl": "",
+                "sceneControls": []
+            },
+            "style": {
+                "width": "100%",
+                "height": "480px"
+            },
+            "config": {
+                "background": "transparent",
+                "eventsTarget": "local",
+                "mobileMode": "scene"
+            }
+        },
+        "inspector": [
+            {
+                "tab": "general",
+                "groups": [
+                    {
+                        "key": "size",
+                        "label": "Size",
+                        "controls": [
+                            "width_select",
+                            "size_bounds"
+                        ]
+                    },
+                    {
+                        "key": "spline",
+                        "label": "Spline",
+                        "controls": [
+                            "spline_source"
+                        ]
+                    },
+                    {
+                        "key": "scene_display",
+                        "label": "Display",
+                        "controls": [
+                            "scene_display"
+                        ]
+                    },
+                    {
+                        "key": "scene_controls",
+                        "label": "Scene controls",
+                        "controls": [
+                            "scene_controls"
+                        ]
+                    },
+                    {
+                        "key": "shape",
+                        "label": "Shape",
+                        "controls": [
+                            "border",
+                            "corner",
+                            "shadow"
+                        ]
+                    }
+                ]
+            },
+            {
+                "tab": "advanced",
+                "groups": [
+                    {
+                        "key": "spacing",
+                        "label": "Spacing",
+                        "controls": [
+                            "padding_margin"
+                        ]
+                    },
+                    {
+                        "key": "display",
+                        "label": "Display",
+                        "controls": [
+                            "display"
+                        ]
+                    },
+                    {
+                        "key": "animation",
+                        "label": "Animation",
+                        "controls": [
+                            "animation"
+                        ]
+                    },
+                    {
+                        "key": "class_css",
+                        "label": "Class",
+                        "controls": [
+                            "class_css"
+                        ]
+                    }
+                ]
+            }
+        ],
+        "controls": [
+            "width_select",
+            "size_bounds",
+            "spline_source",
+            "scene_display",
+            "scene_controls",
+            "border",
+            "corner",
+            "shadow",
+            "padding_margin",
+            "display",
+            "animation",
+            "class_css"
+        ],
+        "description": "An interactive 3D scene made in Spline, embedded from its viewer URL.",
+        "useWhen": [
+            "For a hero or product moment that should move in 3D as the visitor scrolls or hovers",
+            "When the merchant already designs in Spline and has a .splinecode link"
+        ],
+        "avoidWhen": [
+            "For a plain product photo — use image",
+            "For a video — use video / youtube / vimeo",
+            "More than three 3D scenes on one page (each one costs ~600 KB of JS once it scrolls into view)"
+        ],
+        "contentTips": [
+            "Paste the link from Spline: Export → Viewer → copy link (…/scene.splinecode)",
+            "Add a poster image so the box is not blank while the scene loads"
+        ],
+        "semantics": [
+            "3d",
+            "spline",
+            "scene",
+            "webgl",
+            "animation",
+            "interactive"
+        ]
+    },
     "vimeo": {
         "type": "vimeo",
         "label": "Vimeo",
@@ -36195,7 +36335,6 @@ export const BASE_ONLY_CONFIG = [
     "collectionId",
     "collectionType",
     "quantity",
-    "animation",
     "rowLimit"
 ];
 /**
@@ -36207,20 +36346,38 @@ export const BASE_ONLY_EXCEPTIONS = [
     "quantity-button:iconSize"
 ];
 /**
- * The ENTRANCE ANIMATION's vocabulary — config.animation, offered by 73 of the
- * 111 element types and describable by nothing until now.
+ * The ENTRANCE ANIMATION's vocabulary — config.animation, offered by most of
+ * the element library and describable by nothing until this table existed.
  *
  * Three ways to miss, all silent (AnimationTypeOf answers "" and no keyframes,
  * no rule and no error are emitted, through save, publish and render):
- *   - it is an OBJECT, not a string: {active, type, easing, delay, duration}
+ *   - it is an OBJECT, not a string
  *   - active:true is REQUIRED; a stored type is deliberately NOT consent,
  *     because the panel keeps the type when the switch goes off
  *   - type is a keyframe key spelled with UNDERSCORES: fade_in, never fade-in
  *
  * easing is the mild one: an unrecognised value falls back to "ease".
  *
- * It is also BASE-ONLY (see BASE_ONLY_CONFIG) — render/css.go emits it into the
- * base lane because the config object is read with no responsive merge.
+ * TWO THINGS THIS TABLE USED TO SAY THAT ARE NO LONGER TRUE, kept as a
+ * correction because both were recorded here as settled facts:
+ *
+ *   - IT IS NOT BASE-ONLY ANY MORE. The compiler reads config through
+ *     MergeNamespace and emits per lane, so the key left the platform's
+ *     base-only ledger — and the side effect is the thing merchants ask for
+ *     most, an animation that is off on mobile. A caller still writing it to
+ *     base gets the cascade's fallback layer, which is correct but cannot vary.
+ *   - REVEAL-ON-SCROLL HAS AN ANSWER. trigger:"view" compiles to
+ *     animation-timeline: view() inside an @supports override, so it costs no
+ *     JavaScript and the engines without it keep animating at first paint.
+ *
+ * The object now carries ten keys. intensity travels as CSS variables (a
+ * distance is a quantity, so it reaches the page per breakpoint) and ABSENCE IS
+ * NOT medium — a document with no intensity keeps the old 0.5s fallback.
+ *
+ * alternateNeedsInfinite is the guard worth reading before using repeat:
+ * alternate with an EVEN finite count finishes on the from keyframe, and every
+ * entrance keyframe starts at opacity:0 — so the node publishes INVISIBLE. The
+ * compiler honours alternate only alongside an infinite repeat.
  */
 /**
  * The eight node-style keys a THEME TEXT STYLE controls, and the var prop each
@@ -36237,18 +36394,79 @@ export const BASE_ONLY_EXCEPTIONS = [
 export const TEXT_STYLE_KEYS = [["fontFamily", "family"], ["fontSize", "size"], ["fontWeight", "weight"], ["fontStyle", "style"], ["lineHeight", "line-height"], ["letterSpacing", "letter-spacing"], ["textTransform", "transform"], ["color", "color"]];
 export const ANIMATION = {
     "types": [
+        "blur_in",
+        "bounce",
+        "bounce_in",
+        "elastic_in",
         "fade_in",
+        "fade_in_bottom_left",
+        "fade_in_bottom_right",
+        "fade_in_down",
+        "fade_in_left",
+        "fade_in_right",
+        "fade_in_top_left",
+        "fade_in_top_right",
+        "fade_in_up",
+        "flash",
+        "flip",
+        "flip_in_x",
+        "flip_in_y",
+        "head_shake",
+        "heart_beat",
+        "jack_in_the_box",
+        "jello",
+        "light_speed_in_left",
+        "light_speed_in_right",
+        "pulse",
+        "roll_in",
+        "rotate_in",
+        "rotate_in_down_left",
+        "rotate_in_down_right",
+        "rotate_in_up_left",
+        "rotate_in_up_right",
+        "rubber_band",
+        "shake_x",
+        "shake_y",
         "slide_down",
+        "slide_in_down",
+        "slide_in_left",
+        "slide_in_right",
+        "slide_in_up",
+        "slide_left",
+        "slide_right",
         "slide_up",
-        "zoom_in"
+        "swing",
+        "tada",
+        "wobble",
+        "zoom_in",
+        "zoom_out"
     ],
     "easings": [
         "ease",
         "ease-in",
         "ease-out",
-        "linear"
+        "ease-in-out",
+        "linear",
+        "spring",
+        "bounce"
     ],
     "easingFallback": "ease",
     "durationDefault": 0.5,
+    "intensities": [
+        "soft",
+        "medium",
+        "strong"
+    ],
+    "intensityDurations": {
+        "soft": 0.4,
+        "medium": 0.6,
+        "strong": 0.9
+    },
+    "triggers": [
+        "view"
+    ],
+    "rangeDefault": 60,
+    "repeatMax": 100,
+    "alternateNeedsInfinite": true,
     "readBy": "AnimationTypeOf + CompileEntranceAnimationCSS"
 };

package/dist/catalog/translations.generated.js

@@ -3,7 +3,7 @@
 export const TRANSLATION_SOURCE = {
     "elements": 58,
     "pairs": 141,
-    "neverKeys": 156,
+    "neverKeys": 159,
     "entityTypes": 12
 };
 /** Every entity type a translation record can name, including "node". */
@@ -402,6 +402,7 @@ export const NEVER_TRANSLATED = [
     "payload",
     "picker",
     "poster",
+    "posterUrl",
     "prefillValue",
     "presetCode",
     "provinceField",
@@ -409,6 +410,8 @@ export const NEVER_TRANSLATED = [
     "required",
     "reviewForm",
     "sameDefault",
+    "sceneControls",
+    "sceneUrl",
     "searchBehavior",
     "searchClearSwap",
     "searchDebounceMs",

package/dist/domains/site/inert.js

@@ -26,6 +26,19 @@ export const INERT_ON_ADD = {
             'store that is not in English, set it (e.g. "Trang chủ") or the trail reads half-translated ' +
             'on every product page.',
     },
+    // THE SHARPEST OF THE THREE, because the placeholder is not a placeholder: it
+    // is a REAL scene on Spline's own servers, and it loads, renders and responds
+    // to the mouse. The other two entries here describe an element that looks
+    // finished; this one looks finished AND is somebody else's work, published on
+    // the merchant's domain. `sb_review` reads a correct tree, `sb_look`
+    // photographs a convincing 3D hero, and nothing anywhere says whose it is.
+    'spline-scene': {
+        note: "A spline-scene is born pointing at SPLINE'S OWN DEMO SCENE (specials.sceneUrl defaults to " +
+            'a real prod.spline.design link), so an unset one publishes a 3D hero that loads, moves, ' +
+            "and belongs to somebody else — it looks finished, so no check can flag it. Set sceneUrl " +
+            'to the merchant\'s own export (Spline: Export → Viewer → the …/scene.splinecode link), and ' +
+            'set specials.posterUrl too or the box is blank until the engine chunk arrives.',
+    },
 };
 /** The hint for a type, or null. */
 export function inertHint(type) {

package/dist/domains/site/vocabulary.js

@@ -79,9 +79,31 @@ export function unknownValueNote(key, value) {
  * falls back to `ease`, so the animation still runs — it is a wrong answer, not
  * a missing one.
  *
- * And it is BASE-ONLY, which is a fourth way to lose it — but that one is
- * ROUTED rather than warned about, by `baseonly.ts`, because the platform's
- * ledger names the key and `sb_set` can simply write it to the right layer.
+ * IT WAS BASE-ONLY AND IS NOT ANY MORE, and the correction is kept in that
+ * shape because this file stated it as a settled fact. The compiler used to
+ * index `node.Config["animation"]` with no responsive merge, so a per-breakpoint
+ * write landed where nothing looked and `baseonly.ts` ROUTED it. Adding an
+ * intensity ended that: a distance is a QUANTITY, and this repo's own mandate is
+ * that a quantity reaches the page per breakpoint. `CompileEntranceAnimationCSS`
+ * now takes a `bp` and emits per lane, the key left the platform's ledger, and
+ * the routing stopped on its own — the table is generated, which is exactly why
+ * it could. What the caller gains is the thing merchants ask for most: an
+ * animation that is off on mobile.
+ *
+ * THE OBJECT ALSO GREW TO TEN KEYS, and two of them change what is possible
+ * rather than merely how it looks:
+ *
+ *   - `trigger: "view"` IS REVEAL-ON-SCROLL, which this repo recorded as having
+ *     no answer at all ("the platform has nowhere to put it"). It does now:
+ *     `animation-timeline: view()` inside an `@supports` override, no island and
+ *     no JavaScript, with the engines that lack it still animating at first
+ *     paint. A note that says "you cannot" outlives the thing that made it true,
+ *     and this is the third time that has cost something here.
+ *   - `repeat` + `alternate` is the one combination that publishes an INVISIBLE
+ *     node. `alternate` with an EVEN finite count finishes on the `from`
+ *     keyframe and every entrance keyframe starts at `opacity: 0`, so the author
+ *     sees it on the canvas and a visitor never sees it at all. The compiler
+ *     honours `alternate` only alongside `"infinite"`.
  */
 export const ANIMATION_VOCAB = ANIMATION;
 /**
@@ -96,10 +118,68 @@ export const ANIMATION_VOCAB = ANIMATION;
  * mistake is actually made.
  */
 export function animationVocabulary() {
-    return (`config.animation is an OBJECT: { active: true, type: ${ANIMATION.types.join('|')}, ` +
-        `easing?: ${ANIMATION.easings.join('|')}, delay?, duration? }. active:true is REQUIRED ` +
-        '(a type alone is not consent), the type is UNDERSCORED, and it is base-only. Anything ' +
-        'else renders no animation at all, with no error.');
+    return ('config.animation is an OBJECT, never a string: { active: true, type, easing?, intensity?, ' +
+        'trigger?, delay?, duration?, repeat?, alternate?, range? }. active:true is REQUIRED (a type ' +
+        'alone is not consent) and the type is UNDERSCORED (fade_in, never fade-in); either miss ' +
+        'renders no animation at all, with no error. Written per breakpoint like any config, so it ' +
+        'can be off on mobile. See animation_values below for what each key accepts.');
+}
+/**
+ * THE SAME FACTS AS DATA, because forty-six type names are not a sentence.
+ *
+ * The line above used to enumerate every type, which worked while there were
+ * four and became 848 bytes of prose the moment the platform shipped 46 — and
+ * the budget test caught it, correctly, for the second time on this same field.
+ * The first catch (a six-field object, 12,396 bytes) moved the long form OUT of
+ * the trait sheet; this one moves the ENUMERATION out of the prose. What is left
+ * in the sentence is what decides whether a write does anything at all.
+ *
+ * It rides on `sb_traits_for`, which answers for ONE element per call, and not
+ * on `sb_catalog_search`, which answers for many — so the cost is paid once by
+ * the caller who has already chosen the element it is about to animate.
+ *
+ * Every value here is GENERATED from the Go that renders, so a platform that
+ * adds a trigger or an intensity brings it to this sheet on the next codegen,
+ * and one that drops a field stops claiming it.
+ */
+export function animationValues() {
+    const a = ANIMATION;
+    const mid = a.intensities[Math.floor(a.intensities.length / 2)];
+    return {
+        type: a.types,
+        easing: { values: a.easings, unknown_falls_back_to: a.easingFallback },
+        // `absence is not the middle setting` is the platform's own point and the
+        // one reading a caller gets wrong without being told; the rest of the
+        // intensity story is a duration table, which speaks for itself.
+        ...(a.intensities.length
+            ? {
+                intensity: {
+                    values: a.intensities,
+                    implies_duration_s: a.intensityDurations,
+                    note: `omitting it is NOT "${mid}" — it keeps the ${a.durationDefault}s default instead`,
+                },
+            }
+            : {}),
+        // THE ONE ENTRY THAT MUST BE PROACTIVE. Every other trap here is caught by
+        // animationNote at the moment of the mistake — but there is no mistake to
+        // catch in never knowing a capability exists, so reveal-on-scroll has to be
+        // discoverable from the sheet itself.
+        ...(a.triggers.length
+            ? {
+                trigger: {
+                    values: a.triggers,
+                    default: 'omitted = runs at first paint',
+                    view: `reveal-on-scroll (animation-timeline), range = % of entry, default ${a.rangeDefault}`,
+                },
+            }
+            : {}),
+        ...(a.repeatMax ? { repeat: `1-${a.repeatMax} or "infinite"; above that is clamped` } : {}),
+        ...(a.alternateNeedsInfinite
+            ? { alternate: 'requires repeat:"infinite" — DROPPED on a finite count, which would end invisible' }
+            : {}),
+        duration_default_s: a.durationDefault,
+        read_by: a.readBy,
+    };
 }
 /**
  * The warning for a `config.animation` write that will not animate.
@@ -138,6 +218,33 @@ export function animationNote(value) {
         notes.push(`easing ${JSON.stringify(e)} is outside ${ANIMATION.easings.join(', ')}, so the renderer ` +
             `uses "${ANIMATION.easingFallback}" — the animation runs, with a curve you did not choose`);
     }
+    const i = value.intensity;
+    if (i !== undefined && (typeof i !== 'string' || !ANIMATION.intensities.includes(i))) {
+        // Same family as easing — it runs — but the miss is quieter still: no
+        // variables are emitted and the keyframes fall back to their own literals,
+        // so the node animates at a distance nobody chose and nothing looks broken.
+        notes.push(`intensity ${JSON.stringify(i)} is outside ${ANIMATION.intensities.join(', ')}, so no ` +
+            'intensity variables are emitted and the keyframes use their built-in distances');
+    }
+    const tr = value.trigger;
+    if (tr !== undefined && (typeof tr !== 'string' || !ANIMATION.triggers.includes(tr))) {
+        notes.push(`trigger ${JSON.stringify(tr)} is not ${ANIMATION.triggers.map((v) => JSON.stringify(v)).join(' or ')}, ` +
+            'so the animation runs at first paint rather than on scroll — stored, published, and ignored');
+    }
+    // THE ONE THAT PUBLISHES AN INVISIBLE NODE. `alternate` is honoured only
+    // alongside an infinite repeat: with an even finite count the animation
+    // finishes on the `from` keyframe, and every entrance keyframe starts at
+    // opacity:0. The author sees it play on the canvas and the visitor sees
+    // nothing at all, with no error anywhere to explain it.
+    if (ANIMATION.alternateNeedsInfinite && value.alternate === true && value.repeat !== 'infinite') {
+        notes.push('alternate:true is DROPPED unless repeat is "infinite" — with an even finite count the ' +
+            'animation would finish on the from keyframe, which is opacity:0, and publish a node no ' +
+            'visitor can see; the renderer refuses that rather than shipping it');
+    }
+    if (typeof value.repeat === 'number' && ANIMATION.repeatMax && value.repeat > ANIMATION.repeatMax) {
+        notes.push(`repeat ${value.repeat} is clamped to ${ANIMATION.repeatMax} — the count reaches CSS as a ` +
+            'number, and a document is not a trusted source');
+    }
     if (!notes.length)
         return null;
     return `config.animation will not do what this says. ${notes.join('. ')}.`;

package/dist/vision/capture.js

@@ -22,7 +22,24 @@ function capturePage(limits) {
         if (cs.display === 'none' || cs.visibility === 'hidden' || cs.opacity === '0')
             return false;
         const r = el.getBoundingClientRect();
-        return r.width > 0 && r.height > 0;
+        if (r.width > 0 && r.height > 0)
+            return true;
+        // A ZERO BOX MEANS "HIDDEN" ONLY FOR AN ELEMENT THAT SIZES ITSELF FROM ITS
+        // CONTENT. A `<video>` sizes itself from its MEDIA, and a `poster` is what
+        // it measures before any media loads — so a video whose poster has not
+        // resolved measures 0×0 while being perfectly present, and Chrome only
+        // falls back to 300×150 once that load has actually failed.
+        //
+        // MEASURED: `<video src poster>` with no explicit width/height was dropped
+        // with nothing but a `hidden` count, while the identical element carrying
+        // `width`/`height` — or carrying NO poster — came through. So the one video
+        // most worth importing, the one with a still frame on it, was the one that
+        // disappeared, and only on the pages slow enough for the race to be lost.
+        //
+        // Narrow on purpose: a `<video>` alone, and only when it names something to
+        // play or to show. An empty `<video></video>` is a genuinely empty box and
+        // goes on being skipped.
+        return el.tagName === 'VIDEO' && !!(el.getAttribute('src') || el.getAttribute('poster'));
     };
     const clean = (s) => (s ?? '').replace(/\s+/g, ' ').trim();
     // ABSOLUTE where it can be, RAW where it cannot. `new URL(rel, base)` throws

package/dist/vision/shoot.js

@@ -250,6 +250,42 @@ async function settleLazyImages(page) {
         .waitForFunction(() => [...document.querySelectorAll('img')].every((i) => i.complete === true), undefined, { timeout: 3_000 })
         .catch(() => { });
 }
+/**
+ * SETTLE EVERY ENTRANCE ANIMATION, or photograph a band that is not there.
+ *
+ * A still picture wants the page a visitor ends up looking at, and the settled
+ * state of every entrance animation is "visible". Two ways a correct page
+ * photographs BLANK without this, both of them new since the platform shipped
+ * 46 effects and a scroll trigger, and both silent:
+ *
+ *   - `trigger: "view"` compiles to `animation-timeline: view()`, whose progress
+ *     is a function of where the element sits in the scrollport. The walk above
+ *     scrolls and then RETURNS TO THE TOP, so every revealed section is back at
+ *     its `from` keyframe — `opacity: 0` — at the moment the shutter opens.
+ *     MEASURED: a four-band page photographed with the third band entirely
+ *     empty, the other three correct.
+ *   - `fill: both` applies the `from` keyframe during a `delay`, so a section
+ *     with a delay long enough to outlast the settle photographs blank too.
+ *
+ * The honest reading of either picture is "this band is broken", which sends
+ * the caller to fix a page that works — the same failure the lazy-image walk
+ * above exists to prevent, arriving by a different route that the walk cannot
+ * fix because the walk's own return to the top is what causes it.
+ *
+ * `animation: none` rather than forcing the timeline to `auto`: that one
+ * RESTARTS the animation against the document timeline and the shot catches it
+ * mid-flight (measured at opacity 0.317). Removing the animation drops the
+ * element to its own declared style, which for every entrance effect here is
+ * exactly the end state — measured back at opacity 1, with the band's text on
+ * screen.
+ *
+ * Best-effort: a page that refuses a stylesheet is still worth photographing.
+ */
+async function settleAnimations(page) {
+    await page
+        .addStyleTag({ content: '*,*::before,*::after{animation:none !important}' })
+        .catch(() => { });
+}
 async function shootOne(page, url, width, format, opts) {
     // LOAD, then a BOUNDED settle — never `networkidle` alone.
     //
@@ -287,6 +323,7 @@ async function shootOne(page, url, width, format, opts) {
     }
     await settleDom(page);
     await settleLazyImages(page);
+    await settleAnimations(page);
     // A RENDERED page carries its node ids as the HTML `id` attribute — not as
     // `data-node-id`, which is the editor CANVAS's hook and never reaches the
     // renderer. Selecting the canvas attribute here returned an empty box list

package/package.json

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