@useinsider/guido 3.18.2-beta.c25d57c → 3.18.2-beta.d3f87c7

package/dist/extensions/Blocks/Recommendation/controls/main/index.js

@@ -1,27 +1,27 @@
 var m = Object.defineProperty;
-var C = (h, l, t) => l in h ? m(h, l, { enumerable: !0, configurable: !0, writable: !0, value: t }) : h[l] = t;
-var n = (h, l, t) => C(h, typeof l != "symbol" ? l + "" : l, t);
+var C = (g, c, t) => c in g ? m(g, c, { enumerable: !0, configurable: !0, writable: !0, value: t }) : g[c] = t;
+var n = (g, c, t) => C(g, typeof c != "symbol" ? c + "" : c, t);
 import { useConfig as p } from "../../../../../composables/useConfig.js";
 import { ModificationDescription as y } from "../../../../../node_modules/@stripoinc/ui-editor-extensions/dist/esm/index.js";
 import { CommonControl as N } from "../../../common-control.js";
-import { CURRENT_CONFIG_VERSION as _, DEFAULT_NODE_CONFIG as a } from "../../constants/defaultConfig.js";
+import { RRS_CONFIG_VERSION as _, DEFAULT_NODE_CONFIG as a } from "../../constants/defaultConfig.js";
 import { RECOMMENDATION_ID_ATTR as I } from "../../constants/selectors.js";
-import { RecommendationConfigService as c } from "../../services/configService.js";
+import { RecommendationConfigService as l } from "../../services/configService.js";
 import { useRecommendationExtensionStore as R } from "../../store/recommendation.js";
 import { registerNodeConfigWriter as b } from "../../utils/nodeConfigWriter.js";
 import { AlgorithmControl as T } from "./algorithm.js";
 import { ALGORITHM_CONTROL_ID as ut } from "./algorithm.js";
 import { CurrencyControl as P } from "./currency.js";
-import { CURRENCY_CONTROL_ID as ht } from "./currency.js";
+import { CURRENCY_CONTROL_ID as gt } from "./currency.js";
 import { FiltersControl as S } from "./filters.js";
-import { FILTERS_CONTROL_ID as gt } from "./filters.js";
+import { FILTERS_CONTROL_ID as ft } from "./filters.js";
 import { LayoutOrientationControl as O } from "./layoutOrientation.js";
 import { LAYOUT_ORIENTATION_CONTROL_ID as Ct } from "./layoutOrientation.js";
 import { LocaleControl as L } from "./locale.js";
 import { LOCALE_CONTROL_ID as yt } from "./locale.js";
-import { PricePlacementControl as E } from "./pricePlacement.js";
+import { PricePlacementControl as k } from "./pricePlacement.js";
 import { PRICE_PLACEMENT_CONTROL_ID as _t } from "./pricePlacement.js";
-import { ProductCountControl as k } from "./productCount.js";
+import { ProductCountControl as E } from "./productCount.js";
 import { PRODUCT_COUNT_CONTROL_ID as Rt } from "./productCount.js";
 import { ProductLayoutControl as F } from "./productLayout.js";
 import { PRODUCT_LAYOUT_CONTROL_ID as Tt } from "./productLayout.js";
@@ -30,9 +30,9 @@ import { SHUFFLE_CONTROL_ID as St } from "./shuffle.js";
 import { StrategyControl as w } from "./strategy.js";
 import { STRATEGY_CONTROL_ID as Lt } from "./strategy.js";
 import { setCurrencyAttributes as A, getBlockElement as v, isPartnerManagedBlock as x, updateProductContentInPlace as B, regenerateProductRowsWithStyles as U } from "./utils.js";
-import { adjustProductsToSize as kt, formatProductPrice as Ft, getCardComposition as Dt, getCurrentLayout as wt, getMainContainerTbody as At, regenerateMobileProductRows as vt, updatePricesInPlace as xt, updateSingleProductContent as Bt } from "./utils.js";
-import { useDebounceFn as f } from "../../../../../node_modules/@vueuse/shared/index.js";
-const $ = "ui-elements-recommendation-block";
+import { adjustProductsToSize as Et, formatProductPrice as Ft, getCardComposition as Dt, getCurrentLayout as wt, getMainContainerTbody as At, regenerateMobileProductRows as vt, updatePricesInPlace as xt, updateSingleProductContent as Bt } from "./utils.js";
+import { useDebounceFn as h } from "../../../../../node_modules/@vueuse/shared/index.js";
+const W = "ui-elements-recommendation-block";
 class at extends N {
   constructor() {
     super(...arguments);
@@ -58,7 +58,7 @@ class at extends N {
     /**
      * Debounced product fetch to prevent rapid API calls during config changes
      */
-    n(this, "_debouncedFetchProducts", f(async () => {
+    n(this, "_debouncedFetchProducts", h(async () => {
       try {
         await this.store.fetchRecommendationProducts();
       } finally {
@@ -76,7 +76,7 @@ class at extends N {
      * count. The store pads products to the configured size, so in-place only
      * fails when the size actually changed.
      */
-    n(this, "_debouncedRegenerateWithProducts", f(() => {
+    n(this, "_debouncedRegenerateWithProducts", h(() => {
       const t = this.store.recommendationProducts;
       if (!this.currentNode || !this.api)
         return;
@@ -97,7 +97,7 @@ class at extends N {
     }, 100));
   }
   getId() {
-    return $;
+    return W;
   }
   get _subControls() {
     return [
@@ -118,7 +118,7 @@ class at extends N {
     return this.rrsEnabled = p().isFeatureEnabled("reusableRecommendationStrategy"), this._subControls.forEach((e) => {
       var o;
       return (o = e == null ? void 0 : e.onDestroy) == null ? void 0 : o.call(e);
-    }), this.algorithmControl = new T(), this.localeControl = new L(), this.currencyControl = new P(), this.productCountControl = new k(), this.productLayoutControl = new F(), this.filtersControl = new S(), this.shuffleControl = new D(), this.pricePlacementControl = new E(), this.layoutOrientationControl = new O(), this.strategyControl = this.rrsEnabled ? new w() : null, this._subControls.forEach((e) => {
+    }), this.algorithmControl = new T(), this.localeControl = new L(), this.currencyControl = new P(), this.productCountControl = new E(), this.productLayoutControl = new F(), this.filtersControl = new S(), this.shuffleControl = new D(), this.pricePlacementControl = new k(), this.layoutOrientationControl = new O(), this.strategyControl = this.rrsEnabled ? new w() : null, this._subControls.forEach((e) => {
       e && (e.api = this.api);
     }), `
             <div class="recommendation-controls-container">
@@ -140,14 +140,14 @@ class at extends N {
     const e = (i = this.getContainer()) == null ? void 0 : i.querySelector(".recommendation-controls-container");
     if (!e)
       return;
-    const o = this._getRecommendationIdFromNode(t) === null ? null : c.getConfig(t), r = this.rrsEnabled && !!o && (!!(o != null && o.strategyId) || ((o == null ? void 0 : o.configVersion) ?? 0) >= _);
+    const o = this._getRecommendationIdFromNode(t) === null ? null : l.getConfig(t), r = this.rrsEnabled && !!o && (!!(o != null && o.strategyId) || ((o == null ? void 0 : o.configVersion) ?? 0) >= _);
     e.classList.toggle("strategy-block", r);
   }
   async onRender() {
     var o;
     b(this.api);
     const t = this._getRecommendationIdFromNode(this.currentNode), e = t ?? this.store.currentRecommendationId;
-    if (e !== null && this.store.setCurrentBlock(e), t !== null && this._syncNodeConfigIfChanged(t), this._listenStateUpdates(), this._showControlsForBlock(this.currentNode), e !== null && ((o = this.store.blockStates[e]) != null && o.isInitialized)) {
+    if (e !== null && this.store.setCurrentBlock(e), t !== null && this._syncNodeConfigIfChanged(t), this._listenStateUpdates(), this._showControlsForBlock(this.currentNode), t !== null && this.strategyControl && (this.strategyControl.api = this.api, this.strategyControl.currentNode = this.currentNode, this.strategyControl.onRender()), e !== null && ((o = this.store.blockStates[e]) != null && o.isInitialized)) {
       this._initializeSubControls();
       return;
     }
@@ -209,18 +209,23 @@ class at extends N {
     const e = this.store.blockStates[t];
     if (!e)
       return;
-    const o = JSON.stringify(c.getConfig(this.currentNode)), r = e.syncedConfigFingerprint;
+    const o = JSON.stringify(l.getConfig(this.currentNode)), r = e.syncedConfigFingerprint;
     e.syncedConfigFingerprint = o, r !== o && (this.pendingNodeConfigWrite || this._syncNodeConfigToStore());
   }
+  /** Whether the deferred write would move the node's strategyId — see the caller (SD-149916). */
+  _isPendingStrategyIdChange() {
+    const t = this.pendingNodeConfigWrite;
+    return t ? l.getConfig(this.currentNode).strategyId !== t.updates.strategyId : !1;
+  }
   _capturePendingNodeConfigWrite() {
     const t = this.store.currentRecommendationId;
     if (!this.currentNode || !this.api || t === null)
       return;
-    const { filters: e, strategyId: o, size: r } = this.store.recommendationConfigs;
+    const { filters: e, strategy: o, strategyId: r, size: i } = this.store.recommendationConfigs;
     this.pendingNodeConfigWrite = {
       blockId: t,
       node: this.currentNode,
-      updates: { filters: e, strategyId: o, size: r }
+      updates: { filters: e, strategy: o, strategyId: r, size: i }
     };
   }
   /**
@@ -235,7 +240,7 @@ class at extends N {
    */
   _stagePendingNodeConfig(t) {
     const e = this.pendingNodeConfigWrite;
-    return !e || !this.currentNode ? !1 : (c.stageConfig(t, this.currentNode, e.updates), !0);
+    return !e || !this.currentNode ? !1 : (l.stageConfig(t, this.currentNode, e.updates), !0);
   }
   /**
    * Arms the guaranteed fallback write, once the product fetch has settled. The
@@ -259,12 +264,12 @@ class at extends N {
     const t = this.pendingNodeConfigWrite;
     if (this.pendingNodeConfigWrite = null, this._clearConfigFlush(), !t || !this.api)
       return;
-    const e = t.blockId === this.store.currentRecommendationId && this.currentNode ? this.currentNode : t.node, o = c.getConfig(e);
-    JSON.stringify(o.filters) === JSON.stringify(t.updates.filters) && o.strategyId === t.updates.strategyId && o.size === t.updates.size || c.updateConfig(this.api, e, t.updates, "Update recommendation filters");
+    const e = t.blockId === this.store.currentRecommendationId && this.currentNode ? this.currentNode : t.node, o = l.getConfig(e);
+    JSON.stringify(o.filters) === JSON.stringify(t.updates.filters) && o.strategy === t.updates.strategy && o.strategyId === t.updates.strategyId && o.size === t.updates.size || l.updateConfig(this.api, e, t.updates, "Update recommendation filters");
   }
   _syncNodeConfigToStore() {
     var r;
-    const t = c.getConfig(this.currentNode), e = this.store.currentRecommendationId, o = e !== null && ((r = this.store.blockStates[e]) == null ? void 0 : r.isInitialized);
+    const t = l.getConfig(this.currentNode), e = this.store.currentRecommendationId, o = e !== null && ((r = this.store.blockStates[e]) == null ? void 0 : r.isInitialized);
     this.store.patchCurrentBlockConfig({
       strategy: t.strategy,
       strategyId: t.strategyId,
@@ -328,7 +333,7 @@ class at extends N {
   _applySmartDefaults() {
     if (!this.currentNode || !this.api)
       return;
-    const t = c.getConfig(this.currentNode), e = {};
+    const t = l.getConfig(this.currentNode), e = {};
     let o = null, r = null, i = null;
     if (t.currency.code === a.currency.code) {
       const { currencyList: s } = this.store;
@@ -352,7 +357,7 @@ class at extends N {
         (d) => d.value === a.language
       ) || (i = s[0].value, e.language = i));
     }
-    !o && !r && !i || (c.updateConfig(
+    !o && !r && !i || (l.updateConfig(
       this.api,
       this.currentNode,
       e,
@@ -413,19 +418,19 @@ class at extends N {
         return;
       }
       const s = t.$state.configVersion;
-      s !== o && (o = s, this._capturePendingNodeConfigWrite(), this._debouncedFetchProducts());
-      const u = t.recommendationProducts, d = u !== e, g = Array.isArray(u) && u.length > 0;
-      d && g && (e = u, this._debouncedRegenerateWithProducts());
+      s !== o && (o = s, this._capturePendingNodeConfigWrite(), this._isPendingStrategyIdChange() && this._flushPendingNodeConfig(), this._debouncedFetchProducts());
+      const u = t.recommendationProducts, d = u !== e, f = Array.isArray(u) && u.length > 0;
+      d && f && (e = u, this._debouncedRegenerateWithProducts());
     });
   }
 }
 export {
   ut as ALGORITHM_CONTROL_ID,
   T as AlgorithmControl,
-  $ as CONTROL_BLOCK_ID,
-  ht as CURRENCY_CONTROL_ID,
+  W as CONTROL_BLOCK_ID,
+  gt as CURRENCY_CONTROL_ID,
   P as CurrencyControl,
-  gt as FILTERS_CONTROL_ID,
+  ft as FILTERS_CONTROL_ID,
   S as FiltersControl,
   Ct as LAYOUT_ORIENTATION_CONTROL_ID,
   yt as LOCALE_CONTROL_ID,
@@ -434,15 +439,15 @@ export {
   _t as PRICE_PLACEMENT_CONTROL_ID,
   Rt as PRODUCT_COUNT_CONTROL_ID,
   Tt as PRODUCT_LAYOUT_CONTROL_ID,
-  E as PricePlacementControl,
-  k as ProductCountControl,
+  k as PricePlacementControl,
+  E as ProductCountControl,
   F as ProductLayoutControl,
   at as RecommendationBlockControl,
   St as SHUFFLE_CONTROL_ID,
   Lt as STRATEGY_CONTROL_ID,
   D as ShuffleControl,
   w as StrategyControl,
-  kt as adjustProductsToSize,
+  Et as adjustProductsToSize,
   Ft as formatProductPrice,
   v as getBlockElement,
   Dt as getCardComposition,

package/dist/extensions/Blocks/Recommendation/controls/main/strategy.js

@@ -1,21 +1,25 @@
-var d = Object.defineProperty;
-var y = (r, s, t) => s in r ? d(r, s, { enumerable: !0, configurable: !0, writable: !0, value: t }) : r[s] = t;
-var o = (r, s, t) => y(r, typeof s != "symbol" ? s + "" : s, t);
-import { ACADEMY_LINKS as m } from "../../../../../enums/academy.js";
-import { CommonControl as h } from "../../../common-control.js";
-import { useRecommendationExtensionStore as u } from "../../store/recommendation.js";
-const p = "recommendation-strategy-control";
-class f extends h {
+var u = Object.defineProperty;
+var S = (o, n, t) => n in o ? u(o, n, { enumerable: !0, configurable: !0, writable: !0, value: t }) : o[n] = t;
+var r = (o, n, t) => S(o, typeof n != "symbol" ? n + "" : n, t);
+import { ACADEMY_LINKS as _ } from "../../../../../enums/academy.js";
+import { useOnboardingStore as f } from "../../../../../stores/onboarding.js";
+import { CommonControl as v } from "../../../common-control.js";
+import { RRS_CONFIG_VERSION as b } from "../../constants/defaultConfig.js";
+import { RecommendationConfigService as h } from "../../services/configService.js";
+import { useRecommendationExtensionStore as C } from "../../store/recommendation.js";
+const k = "recommendation-strategy-control";
+class I extends v {
   constructor() {
     super(...arguments);
-    o(this, "store", u());
-    o(this, "storeUnsubscription", () => {
+    r(this, "store", C());
+    r(this, "onboardingStore", f());
+    r(this, "storeUnsubscription", () => {
     });
-    o(this, "listeners", []);
-    o(this, "lastPaintedState", null);
+    r(this, "listeners", []);
+    r(this, "lastPaintedState", null);
   }
   getId() {
-    return p;
+    return k;
   }
   getTemplate() {
     const t = this.api.translate(
@@ -23,21 +27,34 @@ class f extends h {
     ), e = this.api.translate("For more information, you can"), a = this.api.translate("visit Academy");
     return `
             <div class="strategy-control-container">
-                <div class="strategy-control-header">
-                    <h3 class="strategy-control-title">${this.api.translate("Recommendation Strategy")}</h3>
-                </div>
-                <div class="strategy-control-description-row">
-                    <p class="strategy-control-description">
-                        ${t}
-                        ${e}
-                        <a
-                            href="${m.EMAIL_RECOMMENDATION_BLOCK}"
-                            target="_blank"
-                            rel="noopener noreferrer">${a}</a>.
-                    </p>
+                <div data-strategy-loading-state>
+                    <div class="strategy-control-header">
+                        <span class="guido-skeleton guido-skeleton--strategy-title"></span>
+                    </div>
+                    <div class="strategy-control-description-row">
+                        <span class="guido-skeleton guido-skeleton--strategy-description"></span>
+                    </div>
+                    <div class="strategy-control-skeleton-row">
+                        <span class="guido-skeleton guido-skeleton--strategy-cta"></span>
+                        <span class="guido-skeleton guido-skeleton--strategy-cta"></span>
+                    </div>
                 </div>
-                <div data-strategy-empty-state>
-                    ${this._GuTwoColumns([
+                <div data-strategy-content style="display: none;">
+                    <div class="strategy-control-header">
+                        <h3 class="strategy-control-title">${this.api.translate("Recommendation Strategy")}</h3>
+                    </div>
+                    <div class="strategy-control-description-row">
+                        <p class="strategy-control-description">
+                            ${t}
+                            ${e}
+                            <a
+                                href="${_.EMAIL_RECOMMENDATION_BLOCK}"
+                                target="_blank"
+                                rel="noopener noreferrer">${a}</a>.
+                        </p>
+                    </div>
+                    <div data-strategy-empty-state style="display: none;">
+                        ${this._GuTwoColumns([
       this._GuButton({
         name: "assignStrategy",
         label: this.api.translate("Assign Strategy"),
@@ -48,26 +65,28 @@ class f extends h {
         name: "createStrategy",
         label: this.api.translate("Create New Strategy"),
         id: "guido__btn-create-strategy",
-        className: "strategy-control-cta primary",
-        disabled: !0
+        className: "strategy-control-cta primary"
       })
     ], "strategy-control-cta-row")}
-                </div>
-                <div data-strategy-assigned-state style="display: none;">
-                    <div class="strategy-control-name-band">
-                        <span class="strategy-control-name" data-strategy-name></span>
                     </div>
-                    <div class="strategy-control-actions">
-                        ${this._strategyAction({
+                    <div data-strategy-assigned-state style="display: none;">
+                        <div class="strategy-control-name-band">
+                            <span class="strategy-control-name" data-strategy-name></span>
+                            <span
+                                class="guido-skeleton guido-skeleton--strategy-name" data-strategy-name-skeleton></span>
+                        </div>
+                        <div class="strategy-control-actions">
+                            ${this._strategyAction({
       id: "guido__btn-change-strategy",
       label: this.api.translate("Change"),
       modifier: "change"
     })}
-                        ${this._strategyAction({
+                            ${this._strategyAction({
       id: "guido__btn-remove-strategy",
       label: this.api.translate("Remove"),
       modifier: "remove"
     })}
+                        </div>
                     </div>
                 </div>
             </div>
@@ -84,41 +103,54 @@ class f extends h {
         `;
   }
   onRender() {
-    this._teardown(), this._setupButtonListeners(), this._renderState(), this.storeUnsubscription = this.store.$subscribe(() => {
+    this._teardown(), this.store.strategyPanelOpen = !0, this._setupButtonListeners(), this._renderState(), this.storeUnsubscription = this.store.$subscribe(() => {
       this._renderState();
-    }), this.store.recommendationConfigs.strategyId && !this.store.strategies.length && this.store.fetchStrategies().catch(() => {
+    }), this.store.recommendationConfigs.strategyId && !this.store.strategiesFetched && this.store.fetchStrategies().catch(() => {
     });
   }
+  /**
+   * Whether the document itself says this block has no strategy.
+   *
+   * Read the node, not the store: when the main control cannot resolve the block id off the
+   * node it never syncs the persisted config, so the store still holds defaults (empty
+   * strategyId) for an assigned block.
+   */
+  _isNodeUnassigned() {
+    if (!h.hasConfig(this.currentNode))
+      return !1;
+    const t = h.getConfig(this.currentNode);
+    return !t.strategyId && (t.configVersion ?? 0) >= b;
+  }
   onDestroy() {
     this._teardown();
   }
   _teardown() {
-    this.storeUnsubscription(), this.storeUnsubscription = () => {
+    this.store.strategyPanelOpen = !1, this.storeUnsubscription(), this.storeUnsubscription = () => {
     }, this.lastPaintedState = null, this.listeners.forEach(({ element: t, handler: e }) => t.removeEventListener("click", e)), this.listeners = [];
   }
   _setupButtonListeners() {
     const t = this.getContainer();
     if (!t)
       return;
-    const e = (a, n) => {
-      const i = t.querySelector(a);
-      i && (i.addEventListener("click", n), this.listeners.push({ element: i, handler: n }));
+    const e = (a, i) => {
+      const s = t.querySelector(a);
+      s && (s.addEventListener("click", i), this.listeners.push({ element: s, handler: i }));
     };
-    e("#guido__btn-assign-strategy", () => this.store.openAssignStrategyDrawer()), e("#guido__btn-change-strategy", () => this.store.openAssignStrategyDrawer()), e("#guido__btn-remove-strategy", () => this.store.openRemoveStrategyModal());
+    e("#guido__btn-assign-strategy", () => this.store.openAssignStrategyDrawer()), e("#guido__btn-create-strategy", () => this.store.openStrategyCreation()), e("#guido__btn-change-strategy", () => this.store.openAssignStrategyDrawer()), e("#guido__btn-remove-strategy", () => this.store.openRemoveStrategyModal());
   }
   _renderState() {
-    var l;
-    const { strategyId: t } = this.store.recommendationConfigs, e = t ? ((l = this.store.assignedStrategy) == null ? void 0 : l.name) ?? "" : "", a = `${t}\0${e}`;
-    if (a === this.lastPaintedState)
+    var p;
+    const { strategyId: t } = this.store.recommendationConfigs, e = t ? ((p = this.store.assignedStrategy) == null ? void 0 : p.name) ?? "" : "", a = !!t && !e && (this.store.strategiesLoading || !this.store.strategiesSettled), i = `${t}\0${e}\0${a}`;
+    if (i === this.lastPaintedState)
       return;
-    const n = this.getContainer();
-    if (!n)
+    const s = this.getContainer();
+    if (!s)
       return;
-    const i = n.querySelector("[data-strategy-empty-state]"), c = n.querySelector("[data-strategy-assigned-state]"), g = n.querySelector("[data-strategy-name]");
-    !i || !c || (i.style.display = t ? "none" : "", c.style.display = t ? "" : "none", g && t && (g.textContent = e || `#${t}`), this.lastPaintedState = a);
+    const c = s.querySelector("[data-strategy-empty-state]"), g = s.querySelector("[data-strategy-assigned-state]"), l = s.querySelector("[data-strategy-loading-state]"), y = s.querySelector("[data-strategy-content]"), d = s.querySelector("[data-strategy-name]"), m = s.querySelector("[data-strategy-name-skeleton]");
+    !c || !g || (l && (l.style.display = "none"), y && (y.style.display = ""), c.style.display = t ? "none" : "", g.style.display = t ? "" : "none", !t && this._isNodeUnassigned() && this.onboardingStore.startOnboarding("recommendationStrategyOnboarding"), d && (d.style.display = a ? "none" : "", t && !a && (d.textContent = e || `#${t}`)), m && (m.style.display = a ? "block" : "none"), this.lastPaintedState = i);
   }
 }
 export {
-  p as STRATEGY_CONTROL_ID,
-  f as StrategyControl
+  k as STRATEGY_CONTROL_ID,
+  I as StrategyControl
 };