@useinsider/guido 3.21.0-beta.27e5667 → 3.21.0-beta.7e166b5

package/dist/extensions/Blocks/Recommendation/block.js

@@ -1,22 +1,22 @@
-var D = Object.defineProperty;
-var A = (l, c, e) => c in l ? D(l, c, { enumerable: !0, configurable: !0, writable: !0, value: e }) : l[c] = e;
-var m = (l, c, e) => A(l, typeof c != "symbol" ? c + "" : c, e);
-import { useConfig as u } from "../../../composables/useConfig.js";
-import { BlockId as C } from "../../../enums/block.js";
-import { getMigrationBannerHtml as E } from "../../../utils/migrationBannerHtml.js";
+var A = Object.defineProperty;
+var b = (c, l, e) => l in c ? A(c, l, { enumerable: !0, configurable: !0, writable: !0, value: e }) : c[l] = e;
+var m = (c, l, e) => b(c, typeof l != "symbol" ? l + "" : l, e);
+import { useConfig as d } from "../../../composables/useConfig.js";
+import { BlockId as E } from "../../../enums/block.js";
+import { getMigrationBannerHtml as C } from "../../../utils/migrationBannerHtml.js";
 import { Block as S, BlockCompositionType as N, ModificationDescription as p } from "../../../node_modules/@stripoinc/ui-editor-extensions/dist/esm/index.js";
-import { CURRENT_CONFIG_VERSION as v, PRE_RRS_CONFIG_VERSION as M } from "./constants/defaultConfig.js";
-import { RECOMMENDATION_ID_ATTR as d } from "./constants/selectors.js";
-import { regenerateMobileProductRows as w } from "./controls/main/utils.js";
-import { ensureMobileCssRulesExist as I, setMobileLayoutOptOut as k, hasMobileLayoutOptOut as O } from "./controls/mobileLayout/cssRules.js";
+import { CURRENT_CONFIG_VERSION as v, PRE_RRS_CONFIG_VERSION as w } from "./constants/defaultConfig.js";
+import { RECOMMENDATION_ID_ATTR as h } from "./constants/selectors.js";
+import { regenerateMobileProductRows as M } from "./controls/main/utils.js";
+import { ensureMobileCssRulesExist as I, setMobileLayoutOptOut as R, hasMobileLayoutOptOut as O } from "./controls/mobileLayout/cssRules.js";
 import { RecommendationConfigService as o } from "./services/configService.js";
 import { useRecommendationExtensionStore as f } from "./store/recommendation.js";
 import { getDefaultTemplate as T } from "./templates/grid/template.js";
 import { useRecommendationBlockWarning as x } from "./useRecommendationBlockWarning.js";
-import { registerNodeConfigWriter as R } from "./utils/nodeConfigWriter.js";
-import { refreshArrivedStrategyBlocks as H, refreshDuplicatedStrategyBlocks as $ } from "./utils/strategyCommit.js";
-const y = C.Recommendation, g = "recommendation-block-v2", B = "hide-price";
-let b = !1;
+import { registerNodeConfigWriter as k } from "./utils/nodeConfigWriter.js";
+import { refreshArrivedStrategyBlocks as $, refreshDuplicatedStrategyBlocks as H } from "./utils/strategyCommit.js";
+const y = E.Recommendation, u = "recommendation-block-v2", B = "hide-price";
+let D = !1;
 class Z extends S {
   constructor() {
     super();
@@ -30,6 +30,8 @@ class Z extends S {
     /** Ids handed to duplicates the document root does not show yet: a copied module renames all its blocks at once. */
     m(this, "_unsettledDuplicateIds", /* @__PURE__ */ new Set());
     m(this, "_arrivedNodes", []);
+    /** Ids whose arrival or duplicate refresh is in flight, so a second refresh of the same block does not race it. */
+    m(this, "_refreshingIds", /* @__PURE__ */ new Set());
   }
   getId() {
     return y;
@@ -49,7 +51,7 @@ class Z extends S {
     );
   }
   getSettingsPanelTitleHtml() {
-    return E(
+    return C(
       y,
       this.api.translate("Recommendation Block"),
       this.api.translate("This block is switched from the Old Version to the New Version. We recommend you check the Recommendation block and test your message to ensure it works properly.")
@@ -89,10 +91,10 @@ class Z extends S {
         this._handleDuplicate(e, t), this._scheduleArrivalReset(e);
         return;
       }
-      const h = this._pendingBlockId === null;
+      const g = this._pendingBlockId === null;
       setTimeout(() => {
         if (this._getBlockElement(e)) {
-          if (h && this._detectDuplicate(t)) {
+          if (g && this._detectDuplicate(t)) {
             this._handleDuplicate(e, t);
             return;
           }
@@ -103,28 +105,37 @@ class Z extends S {
     }
     const i = this._pendingBlockId ?? this._generateNextId();
     this._pendingBlockId = null, this._assignRecommendationId(e, i);
-    const r = u().isFeatureEnabled("reusableRecommendationStrategy"), n = r && !o.hasConfig(e), { config: s, wasFreshDrop: a } = o.initializeConfig(
+    const r = d().isFeatureEnabled("reusableRecommendationStrategy"), s = r && !o.hasConfig(e), { config: n, wasFreshDrop: a } = o.initializeConfig(
       this.api,
       e,
       {
         recommendationId: i,
-        configVersion: r ? v : M,
-        ...n ? { partnerName: u().partnerName.value } : {},
+        configVersion: r ? v : w,
+        ...s ? { partnerName: d().partnerName.value } : {},
         // An RRS block picks a strategy, not an algorithm, and its algorithm control is
         // hidden. Keeping the `mostPopular` default would bake a most-popular feed URL
         // at save for a block the user never configured.
-        ...n ? { strategy: "" } : {}
+        ...s ? { strategy: "" } : {}
       }
     ), _ = f();
-    if (_.setCurrentBlock(i), this._stampHidePrice(e, s.priceHideIfSameAsDiscounted), a) {
+    if (_.setCurrentBlock(i), this._stampHidePrice(e, n.priceHideIfSameAsDiscounted), a) {
       I(this.api);
-      const h = this._getBlockElement(e);
-      h && (k(this.api, h, !0), w({
+      const g = this._getBlockElement(e);
+      g && (R(this.api, g, !0), M({
         currentNode: e,
         documentModifier: this.api.getDocumentModifier()
       }));
     }
-    _.patchCurrentBlockConfig({ language: s.language }, { triggerRefetch: !1 }), this._warnIfMultipleBlocks();
+    _.patchCurrentBlockConfig({ language: n.language }, { triggerRefetch: !1 }), this._warnIfMultipleBlocks();
+  }
+  /** Refreshes every block of an opened template: Stripo restores a stored document without `onCreated`. */
+  onDocumentInit() {
+    try {
+      const e = this.api.getDocumentRoot();
+      e && "querySelectorAll" in e && e.querySelectorAll(`.${u}`).forEach((t) => this._queueArrivalRefresh(t));
+    } catch (e) {
+      console.error("[recommendation] refreshing an opened strategy failed", e);
+    }
   }
   /**
    * Called when the document changes or template is loaded
@@ -146,11 +157,11 @@ class Z extends S {
       }
       this._healLingeringDuplicate(e), o.needsMigration(e) && this._migrateFromLegacy(e), this._healHidePriceAttr(e);
       try {
-        b || (I(this.api), b = !0);
+        D || (I(this.api), D = !0);
         const t = o.getConfig(e), i = this._getBlockElement(e);
         if (i) {
           const r = !t.mobileLayoutEnabled;
-          O(i) !== r && k(this.api, i, r);
+          O(i) !== r && R(this.api, i, r);
         }
       } catch {
       }
@@ -193,9 +204,9 @@ class Z extends S {
     const t = /* @__PURE__ */ new Set();
     try {
       const i = this.api.getDocumentRoot();
-      i && "querySelectorAll" in i && i.querySelectorAll(`.${g}`).forEach((n) => {
-        if ("getAttribute" in n) {
-          const s = n.getAttribute(d), a = s ? parseInt(s) : 0;
+      i && "querySelectorAll" in i && i.querySelectorAll(`.${u}`).forEach((s) => {
+        if ("getAttribute" in s) {
+          const n = s.getAttribute(h), a = n ? parseInt(n) : 0;
           t.add(a), a > e && (e = a);
         }
       });
@@ -213,17 +224,17 @@ class Z extends S {
    * the renamed one depending on where the module was dropped (SD-149916).
    */
   _scheduleArrivalReset(e) {
-    u().isFeatureEnabled("reusableRecommendationStrategy") && setTimeout(() => {
+    d().isFeatureEnabled("reusableRecommendationStrategy") && setTimeout(() => {
       const t = this._getRecommendationId(e);
       t !== null && t > 0 && !this._resetSourceIds.delete(t) && f().resetArrivedBlock(t);
     }, 0);
   }
   /**
-   * Re-reads the strategy of a block that arrived with saved cards, without waiting for a click. Stripo
-   * creates every block of an opened template or dropped module, so arrivals of one tick refresh together.
+   * Re-reads the strategy of a block that arrived with saved cards, without waiting for a click.
+   * Blocks of one opened template or dropped module arrive in one tick and refresh together.
    */
   _queueArrivalRefresh(e) {
-    u().isFeatureEnabled("reusableRecommendationStrategy") && (this._arrivedNodes.push(e) > 1 || setTimeout(() => {
+    d().isFeatureEnabled("reusableRecommendationStrategy") && (this._arrivedNodes.push(e) > 1 || setTimeout(() => {
       const t = this._arrivedNodes;
       this._arrivedNodes = [];
       try {
@@ -235,10 +246,10 @@ class Z extends S {
   }
   _refreshArrivedBlocks(e) {
     const t = e.flatMap((i) => {
-      const r = this._getRecommendationId(i), n = this._getBlockElement(i);
-      return r && n ? [{ id: r, config: o.getConfig(n) }] : [];
+      const r = this._getRecommendationId(i), s = this._getBlockElement(i);
+      return !r || !s || this._refreshingIds.has(r) ? [] : (this._refreshingIds.add(r), [{ id: r, config: o.getConfig(s) }]);
     });
-    R(this.api), H(t).catch((i) => console.error("[recommendation] refreshing an arrived strategy failed", i));
+    t.length && (k(this.api), $(t).catch((i) => console.error("[recommendation] refreshing an arrived strategy failed", i)).finally(() => t.forEach(({ id: i }) => this._refreshingIds.delete(i))));
   }
   /**
    * Heals a duplicate that slipped past the create-time `_detectDuplicate`
@@ -260,20 +271,20 @@ class Z extends S {
         if (!i || !("querySelectorAll" in i))
           return;
         const r = i.querySelectorAll(
-          `.${g}[${d}="${t}"]`
+          `.${u}[${h}="${t}"]`
         );
         if (r.length <= 1)
           return;
-        const n = this._getBlockElement(e);
-        if (!n)
+        const s = this._getBlockElement(e);
+        if (!s)
           return;
-        let s = -1;
+        let n = -1;
         for (let a = 0; a < r.length; a++)
-          if (r[a] === n) {
-            s = a;
+          if (r[a] === s) {
+            n = a;
             break;
           }
-        if (s <= 0)
+        if (n <= 0)
           return;
         this._reassignDuplicateId(e, t);
       } catch {
@@ -285,7 +296,7 @@ class Z extends S {
       const t = this.api.getDocumentRoot();
       if (t && "querySelectorAll" in t)
         return t.querySelectorAll(
-          `.${g}[${d}="${e}"]`
+          `.${u}[${h}="${e}"]`
         ).length > 1;
     } catch {
     }
@@ -294,7 +305,12 @@ class Z extends S {
   /** Assigns a fresh id to a duplicated block and syncs DOM, node config and store. */
   _handleDuplicate(e, t) {
     const i = this._reassignDuplicateId(e, t), r = f();
-    r.cloneBlockState(t, i), r.setCurrentBlock(i), u().isFeatureEnabled("reusableRecommendationStrategy") && (this._resetSourceIds.add(t), R(this.api), $(t, i, o.getConfig(e)).catch((n) => console.error("[recommendation] refreshing a duplicated strategy failed", n))), this._warnIfMultipleBlocks();
+    if (r.cloneBlockState(t, i), r.setCurrentBlock(i), d().isFeatureEnabled("reusableRecommendationStrategy")) {
+      this._resetSourceIds.add(t), k(this.api);
+      const s = [t, i].filter((n) => !this._refreshingIds.has(n));
+      s.forEach((n) => this._refreshingIds.add(n)), H(t, i, o.getConfig(e)).catch((n) => console.error("[recommendation] refreshing a duplicated strategy failed", n)).finally(() => s.forEach((n) => this._refreshingIds.delete(n)));
+    }
+    this._warnIfMultipleBlocks();
   }
   /**
    * Core id-reassignment for a duplicated block: rewrites the DOM
@@ -326,9 +342,9 @@ class Z extends S {
   _reassignInstanceClass(e, t, i) {
     if (!("getAttribute" in e))
       return;
-    const r = `ins-recommendation-v3-block-${t}`, n = `ins-recommendation-v3-block-${i}`, s = this.api.getDocumentModifier();
-    s.modifyHtml(e).removeClass(r).setClass(n), s.apply(new p(
-      `Reassign recommendation instance class ${r} -> ${n}`
+    const r = `ins-recommendation-v3-block-${t}`, s = `ins-recommendation-v3-block-${i}`, n = this.api.getDocumentModifier();
+    n.modifyHtml(e).removeClass(r).setClass(s), n.apply(new p(
+      `Reassign recommendation instance class ${r} -> ${s}`
     ));
   }
   /**
@@ -342,7 +358,7 @@ class Z extends S {
     if (!i)
       return;
     const r = this.api.getDocumentModifier();
-    r.modifyHtml(i).setAttribute(d, t.toString()), r.apply(new p(`Assign recommendation ID ${t}`));
+    r.modifyHtml(i).setAttribute(h, t.toString()), r.apply(new p(`Assign recommendation ID ${t}`));
   }
   /**
    * Stamps the `hide-price` attribute (read by email-service at send time) on
@@ -370,7 +386,7 @@ class Z extends S {
     const t = this._getBlockElement(e);
     if (!t || !("getAttribute" in t))
       return null;
-    const i = t.getAttribute(d);
+    const i = t.getAttribute(h);
     if (!i)
       return null;
     const r = parseInt(i);
@@ -382,10 +398,10 @@ class Z extends S {
   _getBlockElement(e) {
     if ("getAttribute" in e) {
       const t = e.getAttribute("class");
-      if (t && t.includes(g))
+      if (t && t.includes(u))
         return e;
     }
-    return "querySelector" in e ? e.querySelector(`.${g}`) ?? null : null;
+    return "querySelector" in e ? e.querySelector(`.${u}`) ?? null : null;
   }
   /**
    * Migrate configuration from legacy format

package/dist/src/extensions/Blocks/Recommendation/block.d.ts

@@ -20,6 +20,8 @@ export declare class RecommendationBlock extends Block {
     /** Ids handed to duplicates the document root does not show yet: a copied module renames all its blocks at once. */
     private _unsettledDuplicateIds;
     private _arrivedNodes;
+    /** Ids whose arrival or duplicate refresh is in flight, so a second refresh of the same block does not race it. */
+    private _refreshingIds;
     constructor();
     getId(): string;
     getIcon(): string;
@@ -45,6 +47,8 @@ export declare class RecommendationBlock extends Block {
      * @param node - The newly created block node
      */
     onCreated(node: ImmutableHtmlNode): void;
+    /** Refreshes every block of an opened template: Stripo restores a stored document without `onCreated`. */
+    onDocumentInit(): void;
     /**
      * Called when the document changes or template is loaded
      *
@@ -87,8 +91,8 @@ export declare class RecommendationBlock extends Block {
      */
     private _scheduleArrivalReset;
     /**
-     * Re-reads the strategy of a block that arrived with saved cards, without waiting for a click. Stripo
-     * creates every block of an opened template or dropped module, so arrivals of one tick refresh together.
+     * Re-reads the strategy of a block that arrived with saved cards, without waiting for a click.
+     * Blocks of one opened template or dropped module arrive in one tick and refresh together.
      */
     private _queueArrivalRefresh;
     private _refreshArrivedBlocks;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@useinsider/guido",
-  "version": "3.21.0-beta.27e5667",
+  "version": "3.21.0-beta.7e166b5",
   "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",