webcake-storefront-mcp 1.15.0 → 1.15.1

package/dist/builder/bindings.js

@@ -202,6 +202,8 @@ export function normalizeBindings(bindings) {
     if (!Array.isArray(bindings))
         return bindings;
     return bindings.map((b) => {
+        if (typeof b === "string")
+            return makeBinding(b); // bare "name::field" target
         if (!b || typeof b !== "object")
             return b;
         if (b.id && b.name)

package/dist/builder/catalog.js

@@ -166,7 +166,7 @@ export function buildElement(type, opts = {}) {
     if (Array.isArray(node.bindings) && node.bindings.length)
         node.bindings = normalizeBindings(node.bindings);
     if (Array.isArray(node.events) && node.events.length)
-        node.events = normalizeEvents(node.events);
+        node.events = normalizeEvents(node.events, node.type);
     return node;
 }
 export function isKnownType(type) {

package/dist/builder/events.js

@@ -80,25 +80,45 @@ export function isKnownTrigger(name) {
 export function getEventAction(action) {
     return ACTION_BY_NAME[action];
 }
+// Element types whose events only fire on a specific trigger — mirrors builderx_spa
+// components/editor/traits/Events.vue `addEvent()`. For these the element-type trigger
+// is the right default (a click handler on a form never fires); for everything else the
+// action's usual trigger (or "click") wins.
+const SPECIAL_EVENT_TRIGGER = {
+    form: "success",
+    swiper: "tab",
+    popup: "hide",
+    "input-search": "onenter",
+    "submit-button": "hover",
+};
+/** The trigger the builder defaults to for a given element type, if it is a special one. */
+export function defaultTriggerForType(type) {
+    return type ? SPECIAL_EVENT_TRIGGER[type] : undefined;
+}
 /**
  * Mint a structurally-valid event object. Pass at least `action`; extra action-specific
  * fields are merged verbatim. The id and a sensible eventName are filled in.
+ * eventName precedence (mirrors the editor): explicit > element-type default
+ * (form→success, swiper→tab, popup→hide, input-search→onenter, submit-button→hover) >
+ * the action's usual trigger > "click".
  *   makeEvent({ action: "scroll_to", scroll_to_id: "SECTION-x" })
- *   makeEvent({ action: "open_link", eventName: "click", link_target: "https://…" })
+ *   makeEvent({ action: "open_page", open_page_id: "…" }, "form") // eventName → "success"
  */
-export function makeEvent(spec) {
+export function makeEvent(spec, elementType) {
     if (!spec || typeof spec !== "object")
         throw new Error("Event spec must be an object with at least an `action`.");
     const def = spec.action ? ACTION_BY_NAME[spec.action] : undefined;
-    const eventName = spec.eventName || (def ? def.trigger : "click");
+    const eventName = spec.eventName || defaultTriggerForType(elementType) || (def ? def.trigger : "click");
     const { eventName: _e, action: _a, id: _id, ...rest } = spec;
     return { id: spec.id || `EVENT-${randomString(6)}`, eventName, ...(spec.action ? { action: spec.action } : {}), ...rest };
 }
-/** Ensure every event in an array has an id + an eventName (mint where missing). */
-export function normalizeEvents(events) {
+/** Ensure every event in an array has an id + an eventName (mint where missing).
+ *  Pass the owning element's type so special types get the right default trigger.
+ *  Round-trip safe: events that already carry id/eventName are preserved verbatim. */
+export function normalizeEvents(events, elementType) {
     if (!Array.isArray(events))
         return events;
-    return events.map((e) => (e && typeof e === "object" ? makeEvent(e) : e));
+    return events.map((e) => (e && typeof e === "object" ? makeEvent(e, elementType) : e));
 }
 /** Validate one element's events against the catalog + the set of ids present in the page. */
 export function validateEvents(node, allIds) {

package/dist/changelog.json

@@ -1,4 +1,11 @@
 [
+  {
+    "v": "1.15.1",
+    "d": "24/06/2026",
+    "type": "Fixed",
+    "en": "update_page_element, update_page_elements, update_global_source_element, and update_global_source_elements now normalize events and bindings arrays…",
+    "vi": "update_page_element, update_page_elements, update_global_source_element và update_global_source_elements nay chuẩn hóa mảng events và bindings khi…"
+  },
   {
     "v": "1.15.0",
     "d": "24/06/2026",
@@ -33,12 +40,5 @@
     "type": "Added",
     "en": "New update_product tool updates an existing product's name, description, images, category_ids, is_published flag, or variations (price/stock/SKU per…",
     "vi": "Tool mới update_product cập nhật name, description, images, category_ids, cờ is_published hoặc variations (giá/tồn kho/SKU theo từng biến thể) của…"
-  },
-  {
-    "v": "1.11.1",
-    "d": "24/06/2026",
-    "type": "Fixed",
-    "en": "create_site_from_template now calls the dedicated import_store_to_theme API instead of the generic site-duplicate endpoint, correctly cloning the…",
-    "vi": "create_site_from_template nay gọi đúng API import_store_to_theme thay vì endpoint nhân bản site thông thường, giúp clone đầy đủ các trang, global…"
   }
 ]

package/dist/tools/global-sources.js

@@ -1,5 +1,7 @@
 import { z } from "zod";
 import { getConfirmMode } from "./context.js";
+import { normalizeEvents } from "../builder/events.js";
+import { normalizeBindings } from "../builder/bindings.js";
 /**
  * Global Sources tools — manage site-wide components: cart, popup, overview, etc.
  *
@@ -151,10 +153,11 @@ function applyNodeUpdates(node, updates) {
         node.config = { ...(node.config || {}), ...updates.config };
     if (updates.specials)
         node.specials = { ...(node.specials || {}), ...updates.specials };
+    // Normalize so updated events/bindings get a valid id + eventName (renderer needs it).
     if (updates.events !== undefined)
-        node.events = updates.events;
+        node.events = normalizeEvents(updates.events, node.type);
     if (updates.bindings !== undefined)
-        node.bindings = updates.bindings;
+        node.bindings = normalizeBindings(updates.bindings);
     if (updates.responsive) {
         for (const [bp, val] of Object.entries(updates.responsive)) {
             if (/^bp\d+$/.test(bp)) {
@@ -432,7 +435,8 @@ STEP 1: Call with dry_run=true (default) → returns diff of what will change.
 STEP 2: Show the diff to the user and ask for confirmation. NEVER proceed without explicit user approval.
 STEP 3: Only after user confirms, call again with dry_run=false to apply.
 IMPORTANT: You MUST show the diff to the user and get explicit "yes/ok/confirm" before calling with dry_run=false. Skipping confirmation risks data loss.
-Merge rules: style/config/specials = shallow merge, events/bindings = replace array, responsive = merge by bp key.`, {
+Merge rules: style/config/specials = shallow merge, responsive = merge by bp key.
+events/bindings = REPLACE the whole array — pass the COMPLETE list (read it first); entries are auto-normalized (id + eventName filled in).`, {
         global_source_id: z.string().describe("Global source ID"),
         element_id: z.string().describe("Element ID to update (e.g. 'TEXT-3', 'BUTTON-1')"),
         component: z.string().optional().describe('Component hint for faster lookup'),

package/dist/tools/pages.js

@@ -1,6 +1,8 @@
 import { z } from "zod";
 import { CUSTOM_CODE_GUIDE } from "../guides.js";
 import { getConfirmMode } from "./context.js";
+import { normalizeEvents } from "../builder/events.js";
+import { normalizeBindings } from "../builder/bindings.js";
 /**
  * Page source utilities.
  *
@@ -145,10 +147,13 @@ function applyNodeUpdates(node, updates) {
         node.config = { ...(node.config || {}), ...updates.config };
     if (updates.specials)
         node.specials = { ...(node.specials || {}), ...updates.specials };
+    // Normalize so updated events/bindings carry a valid id + eventName (and the right
+    // element-type default trigger) — the storefront renderer won't dispatch an event with
+    // no eventName. Round-trip safe: already-formed entries pass through unchanged.
     if (updates.events !== undefined)
-        node.events = updates.events;
+        node.events = normalizeEvents(updates.events, node.type);
     if (updates.bindings !== undefined)
-        node.bindings = updates.bindings;
+        node.bindings = normalizeBindings(updates.bindings);
     // Responsive breakpoints (bp1, bp2, ...)
     if (updates.responsive) {
         for (const [bp, val] of Object.entries(updates.responsive)) {
@@ -460,7 +465,8 @@ STEP 1: Call with dry_run=true (default) → returns diff of what will change.
 STEP 2: Show the diff to the user and ask for confirmation. NEVER proceed without explicit user approval.
 STEP 3: Only after user confirms, call again with dry_run=false to apply.
 IMPORTANT: You MUST show the diff to the user and get explicit "yes/ok/confirm" before calling with dry_run=false. Skipping confirmation risks data loss.
-Merge rules: style/config/specials = shallow merge, events/bindings = replace array, responsive = merge by bp key.`, {
+Merge rules: style/config/specials = shallow merge, responsive = merge by bp key.
+events/bindings = REPLACE the whole array — pass the COMPLETE list (read it first with get_page_element so you don't drop the others); entries are auto-normalized (id + eventName filled in, so you can pass just { action, ...fields } / { target }).`, {
         page_id: z.string().describe("Page ID"),
         element_id: z.string().describe("Element ID to update (e.g. 'TEXT-3', 'BUTTON-1')"),
         dry_run: z.boolean().optional().describe("Preview only (true) or apply changes (false). Defaults to confirm_mode setting. Use toggle_confirm_mode to change default."),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "webcake-storefront-mcp",
-  "version": "1.15.0",
+  "version": "1.15.1",
   "description": "MCP server for the WebCake/StoreCake storefront builder — page CRUD, page authoring, products, orders, and more",
   "mcpName": "io.github.vuluu2k/webcake-storefront-mcp",
   "license": "MIT",