@useinsider/guido 3.16.0-beta.f1a5d40 → 3.16.0-beta.ff4f1bd

package/dist/utils/templatePreparation.js

@@ -1,7 +1,7 @@
 import { useActionsApi as k } from "../composables/useActionsApi.js";
 import { useConfig as U } from "../composables/useConfig.js";
 import { useHtmlCompiler as x } from "../composables/useHtmlCompiler.js";
-import { DEFAULT_CURRENCY as d, DEFAULT_NODE_CONFIG as s } from "../extensions/Blocks/Recommendation/constants/defaultConfig.js";
+import { DEFAULT_CURRENCY as d, DEFAULT_NODE_CONFIG as a } from "../extensions/Blocks/Recommendation/constants/defaultConfig.js";
 import { useRecommendationExtensionStore as C } from "../extensions/Blocks/Recommendation/store/recommendation.js";
 import { mapLegacyStrategy as L } from "../extensions/Blocks/Recommendation/utils/legacyStrategyMap.js";
 import { DATA_ATTRIBUTES as A } from "../extensions/Blocks/Unsubscribe/utils/constants.js";
@@ -11,20 +11,20 @@ import { useEditorStore as z } from "../stores/editor.js";
 import { useUnsubscribeStore as _ } from "../stores/unsubscribe.js";
 import { assignTrackingIds as v } from "./linkTrackingIds.js";
 function O(m) {
-  const n = new DOMParser().parseFromString(m, "text/html").querySelectorAll(`[${A.PAGE_LIST}]`), t = [];
-  return n.forEach((i) => {
-    const o = i.getAttribute(A.PAGE_LIST);
+  const i = new DOMParser().parseFromString(m, "text/html").querySelectorAll(`[${A.PAGE_LIST}]`), t = [];
+  return i.forEach((s) => {
+    const o = s.getAttribute(A.PAGE_LIST);
     o && t.push(...R(o));
   }), [...new Set(t)];
 }
 async function M(m) {
-  const a = new DOMParser().parseFromString(m, "text/html").querySelectorAll(".recommendation-block-v2");
-  if (a.length === 0)
+  const n = new DOMParser().parseFromString(m, "text/html").querySelectorAll(".recommendation-block-v2");
+  if (n.length === 0)
     return;
-  const n = C();
-  a.forEach((t) => {
+  const i = C();
+  n.forEach((t) => {
     var r, l, g, f, c;
-    const i = t.getAttribute("recommendation-id"), o = i ? Number(i) : NaN;
+    const s = t.getAttribute("recommendation-id"), o = s ? Number(s) : NaN;
     if (!Number.isFinite(o))
       return;
     const p = t.getAttribute("esd-ext-config");
@@ -42,24 +42,27 @@ async function M(m) {
       // 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: L(e.strategy) || s.strategy,
-      language: e.language ?? s.language,
-      size: e.size ?? s.size,
+      strategy: L(e.strategy) || a.strategy,
+      // esd-ext-config is the source of truth for the assigned RRS —
+      // a strategy-backed block must survive save/reopen from it alone.
+      strategyId: e.strategyId ?? a.strategyId,
+      language: e.language ?? a.language,
+      size: e.size ?? a.size,
       // Spread the default arrays so each block gets a fresh reference
       // instead of sharing the singleton in DEFAULT_NODE_CONFIG.
-      productIds: e.productIds ?? [...s.productIds],
-      filters: e.filters ?? [...s.filters],
-      shuffleProducts: e.shuffleProducts ?? s.shuffleProducts,
+      productIds: e.productIds ?? [...a.productIds],
+      filters: e.filters ?? [...a.filters],
+      shuffleProducts: e.shuffleProducts ?? a.shuffleProducts,
       currencyCode: ((r = e.currency) == null ? void 0 : r.code) ?? d.code,
       currencyAlignment: ((l = e.currency) == null ? void 0 : l.alignment) ?? d.alignment,
       currencyDecimalCount: ((g = e.currency) == null ? void 0 : g.decimalCount) ?? d.decimalCount,
       currencyDecimalSeparator: ((f = e.currency) == null ? void 0 : f.decimalSeparator) ?? d.decimalSeparator,
       currencyThousandSeparator: ((c = e.currency) == null ? void 0 : c.thousandSeparator) ?? d.thousandSeparator
     };
-    n.seedBlockUrlConfig(o, h);
+    i.seedBlockUrlConfig(o, h);
   });
   try {
-    await n.fetchRecommendationCreateData();
+    await i.fetchRecommendationCreateData();
   } catch (t) {
     console.warn(
       "Recommendation reference data pre-load failed; validator will skip the availability check.",
@@ -68,39 +71,39 @@ async function M(m) {
   }
 }
 const Z = () => {
-  const m = N(), u = z(), a = _(), { getCompiledEmail: n, getTemplateData: t, updateHtmlAndCss: i } = k(), { compileHtml: o, compileAmpHtml: p } = x(), { isFeatureEnabled: e } = U();
+  const m = N(), u = z(), n = _(), { getCompiledEmail: i, getTemplateData: t, updateHtmlAndCss: s } = k(), { compileHtml: o, compileAmpHtml: p } = x(), { isFeatureEnabled: e } = U();
   return {
     prepareTemplateDetails: async () => {
       let { html: r, css: l, syncModulesIds: g = [] } = await t();
       if (e("linkTrackingIds"))
         try {
-          const { html: y, changed: H, nextSeq: I } = v(r, u.trackIdSessionFloor);
-          u.trackIdSessionFloor = I, H && (r = y, await i(y, l));
-        } catch (y) {
-          console.error("Tracking-id step failed, saving anyway:", y);
+          const { html: S, changed: F, nextSeq: H } = v(r, u.trackIdSessionFloor);
+          u.trackIdSessionFloor = H, F && (r = S, await s(S, l));
+        } catch (S) {
+          console.error("Tracking-id step failed, saving anyway:", S);
         }
-      const { html: f, ampHtml: c = "", ampErrors: T = [] } = await n({
+      const { html: f, ampHtml: c = "", ampErrors: T = [] } = await i({
         minimize: !0,
         resetDataSavedFlag: !1
       });
-      a.selectedUnsubscribePages.length && await a.fetchTemplates(), await M(r);
-      const { compiledHtml: w, stats: S, appliedRules: D } = o(f), E = c && p(c).compiledHtml, P = m.getSelectedDynamicContentList, F = C(), b = O(r);
+      n.selectedUnsubscribePages.length && await n.fetchTemplates(), await M(r);
+      const { compiledHtml: I, stats: y, appliedRules: w } = o(f), D = c && p(c).compiledHtml, E = m.getSelectedDynamicContentList, P = C(), b = O(r);
       return console.debug("HTML Compilation Stats:", {
-        originalSize: S.originalSize,
-        compiledSize: S.compiledSize,
-        reduction: `${S.reductionPercentage.toFixed(2)}%`,
-        appliedRules: D,
-        executionTime: `${S.executionTime.toFixed(2)}ms`
+        originalSize: y.originalSize,
+        compiledSize: y.compiledSize,
+        reduction: `${y.reductionPercentage.toFixed(2)}%`,
+        appliedRules: w,
+        executionTime: `${y.executionTime.toFixed(2)}ms`
       }), {
-        dynamicContentList: P,
-        compiledHtml: w,
+        dynamicContentList: E,
+        compiledHtml: I,
         rawHtml: r,
         css: l,
-        ampHtml: E,
+        ampHtml: D,
         ampErrors: T,
         modules: g.map(Number),
         recommendation: {
-          campaignUrls: F.recommendationCampaignUrls,
+          campaignUrls: P.recommendationCampaignUrls,
           configs: {}
         },
         unsubscribe: {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@useinsider/guido",
-  "version": "3.16.0-beta.f1a5d40",
+  "version": "3.16.0-beta.ff4f1bd",
   "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",
@@ -25,7 +25,7 @@
     "start": "vite",
     "build": "vite build && vue-tsc --declaration --emitDeclarationOnly --project tsconfig.app.json",
     "preview": "vite preview",
-    "lint": "NODE_OPTIONS=--max-old-space-size=2048 eslint ./ && npm run type-check",
+    "lint": "NODE_OPTIONS=--max-old-space-size=2048 eslint ./ && bun run type-check",
     "lint:fix": "NODE_OPTIONS=--max-old-space-size=2048 eslint --fix ./",
     "lint:e2e-quality": "node scripts/lint-e2e-quality.mjs",
     "lint:e2e-quality:ci": "node scripts/lint-e2e-quality.mjs --ci",
@@ -48,6 +48,7 @@
   "license": "ISC",
   "dependencies": {
     "@stripoinc/ui-editor-extensions": "3.9.0",
+    "@useinsider/design-system-organisms": "0.0.3-beta.fd01ebd",
     "@useinsider/design-system-vue": "1.5.0",
     "@vueuse/core": "11.3.0",
     "lodash-es": "4.17.21",