@useinsider/guido 3.18.2-beta.ec25841 → 3.18.2-beta.f42547e

package/dist/services/templateLibraryApi.js

@@ -4,13 +4,14 @@ import { useToaster as C } from "../composables/useToaster.js";
 import { useTranslations as h } from "../composables/useTranslations.js";
 import { useLiquidValidator as S } from "../composables/validators/useLiquidValidator.js";
 import { EditorType as v } from "../enums/defaults.js";
-import { ToasterTypeOptions as N } from "../enums/toaster.js";
-import { useRecommendationExtensionStore as x } from "../extensions/Blocks/Recommendation/store/recommendation.js";
-import { useSaveAsTemplateStore as E } from "../stores/save-as-template.js";
+import { ToasterTypeOptions as x } from "../enums/toaster.js";
+import { useRecommendationExtensionStore as E } from "../extensions/Blocks/Recommendation/store/recommendation.js";
+import { useSaveAsTemplateStore as N } from "../stores/save-as-template.js";
 import { base64EncodeWithSpecialChars as r } from "../utils/base64.js";
+import { isPlainObject as P } from "../utils/genericUtil.js";
 import { useTemplatePreparation as A } from "../utils/templatePreparation.js";
-const j = () => {
-  const { get: n, post: m } = b(), { handleError: o, showToaster: l } = C(), { config: c, isFeatureEnabled: p } = w(), { validateLiquidSyntax: g } = S(), u = h();
+const B = () => {
+  const { get: n, post: m } = b(), { handleError: o, showToaster: l } = C(), { config: c, isFeatureEnabled: p } = w(), { validateLiquidSyntax: f } = S(), u = h();
   return {
     getCategories: async () => {
       try {
@@ -40,10 +41,10 @@ const j = () => {
     createTemplate: async () => {
       var e, t, d, y;
       try {
-        const { prepareTemplateDetails: i } = A(), s = E(), a = await i();
-        if (p("liquidSyntax") && !await g(a.compiledHtml))
+        const { prepareTemplateDetails: i } = A(), s = N(), a = await i();
+        if (p("liquidSyntax") && !await f(a.compiledHtml))
           return !1;
-        const f = {
+        const T = {
           name: s.getTemplateName,
           categories: s.getSelectedCategoryIds,
           productId: ((t = (e = c.value) == null ? void 0 : e.partner) == null ? void 0 : t.productType) || 0,
@@ -54,7 +55,7 @@ const j = () => {
           unsubscriptionPreferencePageStatus: !1,
           unsubscriptionPreferencePages: [],
           recommendationCampaignUrls: r(
-            x().recommendationCampaignUrls
+            E().recommendationCampaignUrls
           ),
           recommendationConfigs: r({}),
           isGuido: !0,
@@ -66,16 +67,18 @@ const j = () => {
             css: a.css,
             dynamicContentList: a.dynamicContentList
           })
-        }, { data: { type: T } } = await m(
+        }, { data: g } = await m(
           "/newsletter/template-library/create-template",
-          f
+          T
         );
-        return T === "alert" ? (l({
+        if (!P(g))
+          throw new Error("Unexpected create-template response");
+        return g.type === "alert" ? (l({
           message: u(
             "newsletter.template-library-save-alert",
             { templateName: s.getTemplateName }
           ),
-          type: N.Error
+          type: x.Error
         }), !1) : (l({
           message: u(
             "newsletter.template-library-save-success",
@@ -90,5 +93,5 @@ const j = () => {
   };
 };
 export {
-  j as useTemplateLibraryApi
+  B as useTemplateLibraryApi
 };

package/dist/src/extensions/Blocks/Recommendation/controls/main/utils.d.ts

@@ -107,7 +107,7 @@ export interface RegenerateWithStylesOptions extends Omit<RegenerateProductRowsO
  * HTML removes the need for any post-rebuild reapply pass.
  * @param options - Configuration options for regeneration
  */
-export declare function regenerateProductRowsWithStyles(options: RegenerateWithStylesOptions): void;
+export declare function regenerateProductRowsWithStyles(options: RegenerateWithStylesOptions): boolean;
 /**
  * Adjusts a products array to a target size.
  * Used by ProductLayoutControl to synchronously match products to the new size
@@ -151,6 +151,11 @@ export interface UpdateProductContentInPlaceOptions {
     currentNode: ImmutableHtmlNode | null | undefined;
     documentModifier: DocumentModifier;
     products: RecommendationProduct[];
+    /**
+     * Called when the batch was actually committed. Identical products commit nothing, which
+     * leaves anything a caller staged into the same batch unwritten unless it applies itself.
+     */
+    onApplied?: () => void;
 }
 /**
  * Updates product content in-place without regenerating HTML structure.
@@ -161,7 +166,7 @@ export interface UpdateProductContentInPlaceOptions {
  * After updating desktop, also updates mobile container to keep them in sync.
  *
  * Falls back to false when product count changed (need to add/remove elements).
- * @param options - Configuration options
+ * @param options - Configuration options; `onApplied` fires only when a batch was committed
  * @returns true if in-place update was successful, false to fall back to full regeneration
  */
 export declare function updateProductContentInPlace(options: UpdateProductContentInPlaceOptions): boolean;

package/dist/src/extensions/Blocks/Recommendation/utils/strategyCommit.d.ts

@@ -0,0 +1,14 @@
+/**
+ * Writes a strategy change and its products straight to the document, without the settings panel.
+ *
+ * Every entry point is host-rendered UI outside `ui-editor`, and none of them can rely on the
+ * panel control: opening the strategy page deselects the block, so by the time the user comes back
+ * and presses Assign the control — and with it the subscription that does the node write, the
+ * product fetch and the repaint — is gone. The assignment then reaches Pinia and nothing else
+ * (SD-149916).
+ *
+ * Call AFTER the store change (`assignStrategy` / `removeStrategy`).
+ * @param description - labels the undo entry
+ * @returns whether the write reached a node
+ */
+export declare function commitStrategyChange(blockId: number, description: string): Promise<boolean>;

package/dist/src/utils/genericUtil.d.ts

@@ -30,3 +30,9 @@ export declare const replaceString: (string?: string, values?: Record<string, st
  * a string nor a number becomes ''.
  */
 export declare const asText: (value: unknown) => string;
+/**
+ * Narrows an unknown API value to a plain JSON object. Arrays and exotic
+ * objects (Blob, Date, FormData) are rejected, so a response body that useHttp
+ * parsed as text or blob can never be read as a JSON payload.
+ */
+export declare const isPlainObject: <T>(value: T) => value is T & Record<string, unknown>;

package/dist/utils/genericUtil.js

@@ -1,69 +1,75 @@
-const o = (t) => {
-  const a = t.value.match(/\{\{([^}]+)\}\}/)[1].split("|").map((l) => l.trim()), [s] = a, r = {
-    text: t.label,
+const o = (a) => {
+  const t = a.value.match(/\{\{([^}]+)\}\}/)[1].split("|").map((e) => e.trim()), [s] = t, r = {
+    text: a.label,
     value: s || ""
   };
-  if (a.length >= 2) {
-    const [, l, e] = a;
-    if (l.startsWith("default:")) {
-      let n = l.slice(8).trim();
+  if (t.length >= 2) {
+    const [, e, l] = t;
+    if (e.startsWith("default:")) {
+      let n = e.slice(8).trim();
       n.startsWith('"') && n.endsWith('"') && (n = n.slice(1, -1)), r.fallback = n;
     } else {
-      const n = l.includes("=") ? { key: l.split("=")[0].trim(), value: l.split("=")[1].trim() } : null;
-      n ? r.format = n : e || (r.fallback = l), e && (r.fallback = e);
+      const n = e.includes("=") ? { key: e.split("=")[0].trim(), value: e.split("=")[1].trim() } : null;
+      n ? r.format = n : l || (r.fallback = e), l && (r.fallback = l);
     }
   }
   return r;
-}, f = (t, c = !1) => t.map((a) => {
-  if (a.format)
+}, u = (a, c = !1) => a.map((t) => {
+  if (t.format)
     return {
-      label: `${a.text} | ${a.format.key}=${a.format.value}`,
-      value: `{{${a.value}|${a.format.key}=${a.format.value}}}`
+      label: `${t.text} | ${t.format.key}=${t.format.value}`,
+      value: `{{${t.value}|${t.format.key}=${t.format.value}}}`
     };
   if (c) {
-    if (!a.fallback)
+    if (!t.fallback)
       return {
-        label: a.text,
-        value: `{{ ${a.value} }}`
+        label: t.text,
+        value: `{{ ${t.value} }}`
       };
-    const r = !Number.isNaN(Number(a.fallback)) && a.fallback.trim() !== "" ? a.fallback : `"${a.fallback}"`;
+    const r = !Number.isNaN(Number(t.fallback)) && t.fallback.trim() !== "" ? t.fallback : `"${t.fallback}"`;
     return {
-      label: `${a.text} | ${a.fallback}`,
-      value: `{{ ${a.value} | default: ${r} }}`
+      label: `${t.text} | ${t.fallback}`,
+      value: `{{ ${t.value} | default: ${r} }}`
     };
   }
   return {
-    label: a.fallback ? `${a.text} | ${a.fallback}` : a.text,
-    value: a.fallback ? `{{${a.value}|${a.fallback}}}` : `{{${a.value}}}`
+    label: t.fallback ? `${t.text} | ${t.fallback}` : t.text,
+    value: t.fallback ? `{{${t.value}|${t.fallback}}}` : `{{${t.value}}}`
   };
-}), b = (t) => {
-  const c = [], a = /* @__PURE__ */ new Set(), s = (l) => {
-    Array.isArray(l) && l.forEach((e) => {
-      const n = [...e.children ?? [], ...e.select_items ?? []];
+}), b = (a) => {
+  const c = [], t = /* @__PURE__ */ new Set(), s = (e) => {
+    Array.isArray(e) && e.forEach((l) => {
+      const n = [...l.children ?? [], ...l.select_items ?? []];
       if (n.length > 0) {
         s(n);
         return;
       }
-      const u = e.text || e.tag_text;
-      u && e.value && !a.has(e.value) && (a.add(e.value), c.push({ text: u, value: e.value }));
+      const f = l.text || l.tag_text;
+      f && l.value && !t.has(l.value) && (t.add(l.value), c.push({ text: f, value: l.value }));
     });
-  }, [r] = Object.values(t ?? {});
+  }, [r] = Object.values(a ?? {});
   return s(r), c;
-}, i = (t, c, a = !1) => {
-  const s = new Set(t.map((l) => l.value)), r = c.filter((l) => s.has(l.value) ? !1 : (s.add(l.value), !0));
+}, i = (a, c, t = !1) => {
+  const s = new Set(a.map((e) => e.value)), r = c.filter((e) => s.has(e.value) ? !1 : (s.add(e.value), !0));
   return [
-    ...f(t, a),
-    ...f(r, a)
+    ...u(a, t),
+    ...u(r, t)
   ];
-}, v = () => {
-  var t;
-  return ((t = document.head.querySelector('meta[name="csrf-token"]')) == null ? void 0 : t.getAttribute("content")) ?? "";
-}, k = (t) => typeof t == "string" ? t : typeof t == "number" ? String(t) : "";
+}, k = () => {
+  var a;
+  return ((a = document.head.querySelector('meta[name="csrf-token"]')) == null ? void 0 : a.getAttribute("content")) ?? "";
+}, v = (a) => typeof a == "string" ? a : typeof a == "number" ? String(a) : "", y = (a) => {
+  if (a === null || typeof a != "object" || Array.isArray(a))
+    return !1;
+  const c = Object.getPrototypeOf(a);
+  return c === Object.prototype || c === null;
+};
 export {
-  k as asText,
+  v as asText,
   i as buildMergeTagEntries,
-  f as dynamicContentToMergeTags,
+  u as dynamicContentToMergeTags,
   b as flattenDynamicContentList,
-  v as getCsrfToken,
+  k as getCsrfToken,
+  y as isPlainObject,
   o as mergeTagToDynamicContent
 };

package/dist/utils/templatePreparation.js

@@ -1,30 +1,31 @@
 import { useActionsApi as R } from "../composables/useActionsApi.js";
 import { useConfig as k } from "../composables/useConfig.js";
 import { useHtmlCompiler as N } from "../composables/useHtmlCompiler.js";
-import { RRS_CONFIG_VERSION as U, DEFAULT_CURRENCY as m, DEFAULT_NODE_CONFIG as a } from "../extensions/Blocks/Recommendation/constants/defaultConfig.js";
+import { RRS_CONFIG_VERSION as U, DEFAULT_CURRENCY as m, DEFAULT_NODE_CONFIG as n } from "../extensions/Blocks/Recommendation/constants/defaultConfig.js";
 import { useRecommendationExtensionStore as I } from "../extensions/Blocks/Recommendation/store/recommendation.js";
 import { mapLegacyStrategy as x } from "../extensions/Blocks/Recommendation/utils/legacyStrategyMap.js";
-import { DATA_ATTRIBUTES as T } from "../extensions/Blocks/Unsubscribe/utils/constants.js";
+import { DATA_ATTRIBUTES as A } from "../extensions/Blocks/Unsubscribe/utils/constants.js";
 import { parsePageList as L } from "../extensions/Blocks/Unsubscribe/utils/utils.js";
 import { useDynamicContentStore as z } from "../stores/dynamic-content.js";
-import { useEditorStore as _ } from "../stores/editor.js";
-import { useUnsubscribeStore as O } from "../stores/unsubscribe.js";
-import { assignTrackingIds as v } from "./linkTrackingIds.js";
-function B(l) {
-  const s = new DOMParser().parseFromString(l, "text/html").querySelectorAll(`[${T.PAGE_LIST}]`), t = [];
-  return s.forEach((c) => {
-    const r = c.getAttribute(T.PAGE_LIST);
+import { useEditorStore as O } from "../stores/editor.js";
+import { useUnsubscribeStore as _ } from "../stores/unsubscribe.js";
+import { isPlainObject as v } from "./genericUtil.js";
+import { assignTrackingIds as B } from "./linkTrackingIds.js";
+function G(l) {
+  const i = new DOMParser().parseFromString(l, "text/html").querySelectorAll(`[${A.PAGE_LIST}]`), t = [];
+  return i.forEach((s) => {
+    const r = s.getAttribute(A.PAGE_LIST);
     r && t.push(...L(r));
   }), [...new Set(t)];
 }
-async function G(l) {
-  const n = new DOMParser().parseFromString(l, "text/html").querySelectorAll(".recommendation-block-v2");
-  if (n.length === 0)
+async function M(l) {
+  const a = new DOMParser().parseFromString(l, "text/html").querySelectorAll(".recommendation-block-v2");
+  if (a.length === 0)
     return;
-  const s = I();
-  n.forEach((t) => {
-    var d, g, f, i, y, S;
-    const c = t.getAttribute("recommendation-id"), r = c ? Number(c) : NaN;
+  const i = I();
+  a.forEach((t) => {
+    var d, g, f, c, y, S;
+    const s = t.getAttribute("recommendation-id"), r = s ? Number(s) : NaN;
     if (!Number.isFinite(r))
       return;
     const p = t.getAttribute("esd-ext-config");
@@ -36,32 +37,32 @@ async function G(l) {
     } catch {
       return;
     }
-    if (!e || typeof e != "object" || Array.isArray(e))
+    if (!v(e))
       return;
-    const A = !!e.strategyId || (e.configVersion ?? 0) >= U, o = {
+    const C = !!e.strategyId || (e.configVersion ?? 0) >= U, o = {
       // Normalize legacy strategy keys (e.g. `mostViewed` → `mostPopular`) so
       // already-saved blocks that never re-run the migrator don't emit a
       // strategy-less campaign URL. `||` also catches '' / undefined (SD-144882).
-      strategy: x(e.strategy) || (A ? "" : a.strategy),
-      strategyId: e.strategyId ?? a.strategyId,
-      language: e.language ?? a.language,
-      size: e.size ?? a.size,
+      strategy: x(e.strategy) || (C ? "" : n.strategy),
+      strategyId: e.strategyId ?? n.strategyId,
+      language: e.language ?? n.language,
+      size: e.size ?? n.size,
       // Spread the default arrays so each block gets a fresh reference
       // instead of sharing the singleton in DEFAULT_NODE_CONFIG.
-      productIds: e.productIds ?? [...a.productIds],
-      filters: e.filters ?? [...a.filters],
-      shuffleProducts: e.shuffleProducts ?? a.shuffleProducts,
+      productIds: e.productIds ?? [...n.productIds],
+      filters: e.filters ?? [...n.filters],
+      shuffleProducts: e.shuffleProducts ?? n.shuffleProducts,
       currencyCode: ((d = e.currency) == null ? void 0 : d.code) ?? m.code,
       currencySymbol: ((g = e.currency) == null ? void 0 : g.symbol) ?? m.symbol,
       currencyAlignment: ((f = e.currency) == null ? void 0 : f.alignment) ?? m.alignment,
-      currencyDecimalCount: ((i = e.currency) == null ? void 0 : i.decimalCount) ?? m.decimalCount,
+      currencyDecimalCount: ((c = e.currency) == null ? void 0 : c.decimalCount) ?? m.decimalCount,
       currencyDecimalSeparator: ((y = e.currency) == null ? void 0 : y.decimalSeparator) ?? m.decimalSeparator,
       currencyThousandSeparator: ((S = e.currency) == null ? void 0 : S.thousandSeparator) ?? m.thousandSeparator
     };
-    s.seedBlockUrlConfig(r, o);
+    i.seedBlockUrlConfig(r, o);
   });
   try {
-    await s.fetchRecommendationCreateData();
+    await i.fetchRecommendationCreateData();
   } catch (t) {
     console.warn(
       "Recommendation reference data pre-load failed; validator will skip the availability check.",
@@ -69,24 +70,24 @@ async function G(l) {
     );
   }
 }
-const ee = () => {
-  const l = z(), u = _(), n = O(), { getCompiledEmail: s, getTemplateData: t, updateHtmlAndCss: c } = R(), { compileHtml: r, compileAmpHtml: p } = N(), { isFeatureEnabled: e } = k();
+const re = () => {
+  const l = z(), u = O(), a = _(), { getCompiledEmail: i, getTemplateData: t, updateHtmlAndCss: s } = R(), { compileHtml: r, compileAmpHtml: p } = N(), { isFeatureEnabled: e } = k();
   return {
     prepareTemplateDetails: async () => {
       let { html: o, css: d, syncModulesIds: g = [] } = await t();
       if (e("linkTrackingIds"))
         try {
-          const { html: b, changed: P, nextSeq: H } = v(o, u.trackIdSessionFloor);
-          u.trackIdSessionFloor = H, P && (o = b, await c(b, d));
+          const { html: b, changed: F, nextSeq: H } = B(o, u.trackIdSessionFloor);
+          u.trackIdSessionFloor = H, F && (o = b, await s(b, d));
         } catch (b) {
           console.error("Tracking-id step failed, saving anyway:", b);
         }
-      const { html: f, ampHtml: i = "", ampErrors: y = [] } = await s({
+      const { html: f, ampHtml: c = "", ampErrors: y = [] } = await i({
         minimize: !0,
         resetDataSavedFlag: !1
       });
-      n.selectedUnsubscribePages.length && await n.fetchTemplates(), await G(o);
-      const { compiledHtml: S, stats: h, appliedRules: w } = r(f), D = i && p(i).compiledHtml, E = l.getSelectedDynamicContentList, F = I(), C = B(o);
+      a.selectedUnsubscribePages.length && await a.fetchTemplates(), await M(o);
+      const { compiledHtml: S, stats: h, appliedRules: w } = r(f), D = c && p(c).compiledHtml, E = l.getSelectedDynamicContentList, P = I(), T = G(o);
       return console.debug("HTML Compilation Stats:", {
         originalSize: h.originalSize,
         compiledSize: h.compiledSize,
@@ -102,18 +103,18 @@ const ee = () => {
         ampErrors: y,
         modules: g.map(Number),
         recommendation: {
-          campaignUrls: F.recommendationCampaignUrls,
+          campaignUrls: P.recommendationCampaignUrls,
           configs: {}
         },
         unsubscribe: {
-          status: C.length > 0,
-          config: C
+          status: T.length > 0,
+          config: T
         }
       };
     }
   };
 };
 export {
-  B as getHtmlReferencedUnsubscribePages,
-  ee as useTemplatePreparation
+  G as getHtmlReferencedUnsubscribePages,
+  re as useTemplatePreparation
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@useinsider/guido",
-  "version": "3.18.2-beta.ec25841",
+  "version": "3.18.2-beta.f42547e",
   "description": "Guido is a Vue + TypeScript wrapper for Email Plugin. Easily embed the email editor in your Vue applications.",
   "main": "./dist/guido.umd.cjs",
   "module": "./dist/library.js",

package/dist/extensions/Blocks/Recommendation/utils/strategyDetach.js

@@ -1,19 +0,0 @@
-import { useRecommendationExtensionStore as c } from "../store/recommendation.js";
-import { commitBlockChangeOutsidePanel as m } from "./nodeConfigWriter.js";
-async function s(o) {
-  const e = c(), { strategyId: t, size: r } = e.recommendationConfigs;
-  try {
-    await e.fetchRecommendationProducts();
-  } catch (n) {
-    console.error("[recommendation] product refetch after strategy removal failed", n);
-  }
-  m(
-    o,
-    { strategyId: t, size: r },
-    e.recommendationProducts,
-    "Removed recommendation strategy"
-  );
-}
-export {
-  s as commitStrategyDetach
-};

package/dist/src/extensions/Blocks/Recommendation/utils/strategyDetach.d.ts

@@ -1,12 +0,0 @@
-/**
- * Repaints a block whose strategy assignment was just cleared, straight to the document.
- *
- * Both callers are host-rendered UI outside `ui-editor`, so neither can leave this to the
- * settings panel: the remove modal's click deselects the block and destroys the control, and
- * the edit button opens the strategy page in the same gesture, so the panel's 500 ms debounced
- * refetch races a tab that is already gone. Either way the canvas kept the old strategy's
- * products while the block read as unassigned (SD-149916).
- *
- * Call AFTER the store assignment is cleared (`removeStrategy`).
- */
-export declare function commitStrategyDetach(blockId: number): Promise<void>;