@useinsider/guido 3.18.1-beta.280b4e1 → 3.18.1-beta.2e195ed

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

@@ -1,37 +1,37 @@
-var R = Object.defineProperty;
-var B = (a, c, t) => c in a ? R(a, c, { enumerable: !0, configurable: !0, writable: !0, value: t }) : a[c] = t;
-var p = (a, c, t) => B(a, typeof c != "symbol" ? c + "" : c, t);
-import { useConfig as D } from "../../../composables/useConfig.js";
-import { BlockId as y } from "../../../enums/block.js";
-import { getMigrationBannerHtml as C } from "../../../utils/migrationBannerHtml.js";
-import { Block as b, BlockCompositionType as A, ModificationDescription as g } from "../../../node_modules/@stripoinc/ui-editor-extensions/dist/esm/index.js";
-import { CURRENT_CONFIG_VERSION as E, PRE_RRS_CONFIG_VERSION as S } from "./constants/defaultConfig.js";
+var D = Object.defineProperty;
+var y = (l, c, t) => c in l ? D(l, c, { enumerable: !0, configurable: !0, writable: !0, value: t }) : l[c] = t;
+var f = (l, c, t) => y(l, typeof c != "symbol" ? c + "" : c, t);
+import { useConfig as C } from "../../../composables/useConfig.js";
+import { BlockId as b } from "../../../enums/block.js";
+import { getMigrationBannerHtml as A } from "../../../utils/migrationBannerHtml.js";
+import { Block as E, BlockCompositionType as S, ModificationDescription as g } from "../../../node_modules/@stripoinc/ui-editor-extensions/dist/esm/index.js";
+import { CURRENT_CONFIG_VERSION as N, PRE_RRS_CONFIG_VERSION as O } from "./constants/defaultConfig.js";
 import { RECOMMENDATION_ID_ATTR as m } from "./constants/selectors.js";
-import { regenerateMobileProductRows as N } from "./controls/main/utils.js";
-import { ensureMobileCssRulesExist as h, setMobileLayoutOptOut as f, hasMobileLayoutOptOut as O } from "./controls/mobileLayout/cssRules.js";
+import { regenerateMobileProductRows as M } from "./controls/main/utils.js";
+import { ensureMobileCssRulesExist as _, setMobileLayoutOptOut as I, hasMobileLayoutOptOut as w } from "./controls/mobileLayout/cssRules.js";
 import { RecommendationConfigService as s } from "./services/configService.js";
 import { useRecommendationExtensionStore as d } from "./store/recommendation.js";
-import { getDefaultTemplate as M } from "./templates/grid/template.js";
-import { useRecommendationBlockWarning as w } from "./useRecommendationBlockWarning.js";
-const _ = y.Recommendation, u = "recommendation-block-v2", I = "hide-price";
-let k = !1;
-class W extends b {
+import { getDefaultTemplate as x } from "./templates/grid/template.js";
+import { useRecommendationBlockWarning as H } from "./useRecommendationBlockWarning.js";
+const k = b.Recommendation, u = "recommendation-block-v2", R = "hide-price";
+let B = !1;
+class j extends E {
   constructor() {
     super();
     /**
      * Stores the ID generated in getTemplate() so onCreated() can reuse it.
      * This avoids generating a new (different) ID in onCreated().
      */
-    p(this, "_pendingBlockId", null);
+    f(this, "_pendingBlockId", null);
   }
   getId() {
-    return _;
+    return k;
   }
   getIcon() {
     return "recommendation-icon";
   }
   getBlockCompositionType() {
-    return A.CONTAINER;
+    return S.CONTAINER;
   }
   getName() {
     return this.api.translate("Recommendation Block");
@@ -42,8 +42,8 @@ class W extends b {
     );
   }
   getSettingsPanelTitleHtml() {
-    return C(
-      _,
+    return A(
+      k,
       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.")
     );
@@ -66,7 +66,7 @@ class W extends b {
    */
   getTemplate() {
     const t = this._generateNextId();
-    return this._pendingBlockId = t, M(t);
+    return this._pendingBlockId = t, x(t);
   }
   /**
    * Called when a new block is dropped into the template
@@ -89,23 +89,27 @@ class W extends b {
     }
     const i = this._pendingBlockId ?? this._generateNextId();
     this._pendingBlockId = null, this._assignRecommendationId(t, i);
-    const { config: n, wasFreshDrop: o } = s.initializeConfig(
+    const n = C().isFeatureEnabled("reusableRecommendationStrategy"), o = n && !s.hasConfig(t), { config: r, wasFreshDrop: a } = s.initializeConfig(
       this.api,
       t,
       {
         recommendationId: i,
-        configVersion: D().isFeatureEnabled("reusableRecommendationStrategy") ? E : S
+        configVersion: n ? N : O,
+        // 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.
+        ...o ? { strategy: "" } : {}
       }
-    ), r = d();
-    if (r.setCurrentBlock(i), this._stampHidePrice(t, n.priceHideIfSameAsDiscounted), o) {
-      h(this.api);
-      const l = this._getBlockElement(t);
-      l && (f(this.api, l, !0), N({
+    ), p = d();
+    if (p.setCurrentBlock(i), this._stampHidePrice(t, r.priceHideIfSameAsDiscounted), a) {
+      _(this.api);
+      const h = this._getBlockElement(t);
+      h && (I(this.api, h, !0), M({
         currentNode: t,
         documentModifier: this.api.getDocumentModifier()
       }));
     }
-    r.patchCurrentBlockConfig({ language: n.language }, { triggerRefetch: !1 }), this._warnIfMultipleBlocks();
+    p.patchCurrentBlockConfig({ language: r.language }, { triggerRefetch: !1 }), this._warnIfMultipleBlocks();
   }
   /**
    * Called when the document changes or template is loaded
@@ -127,11 +131,11 @@ class W extends b {
       }
       this._healLingeringDuplicate(t), s.needsMigration(t) && this._migrateFromLegacy(t), this._healHidePriceAttr(t);
       try {
-        k || (h(this.api), k = !0);
+        B || (_(this.api), B = !0);
         const e = s.getConfig(t), i = this._getBlockElement(t);
         if (i) {
           const n = !e.mobileLayoutEnabled;
-          O(i) !== n && f(this.api, i, n);
+          w(i) !== n && I(this.api, i, n);
         }
       } catch {
       }
@@ -161,7 +165,7 @@ class W extends b {
    */
   _warnIfMultipleBlocks() {
     try {
-      w().warnIfMultipleBlocks(this.api.getDocumentRootHtmlNode());
+      H().warnIfMultipleBlocks(this.api.getDocumentRootHtmlNode());
     } catch {
     }
   }
@@ -211,9 +215,9 @@ class W extends b {
         if (!o)
           return;
         let r = -1;
-        for (let l = 0; l < n.length; l++)
-          if (n[l] === o) {
-            r = l;
+        for (let a = 0; a < n.length; a++)
+          if (n[a] === o) {
+            r = a;
             break;
           }
         if (r <= 0)
@@ -293,7 +297,7 @@ class W extends b {
    */
   _stampHidePrice(t, e) {
     const i = this._getBlockElement(t);
-    i && this.api.getDocumentModifier().modifyHtml(i).setAttribute(I, String(e)).apply(new g("Stamp hide-price"));
+    i && this.api.getDocumentModifier().modifyHtml(i).setAttribute(R, String(e)).apply(new g("Stamp hide-price"));
   }
   /**
    * Stamps `hide-price` from config only when the block lacks the attribute —
@@ -301,7 +305,7 @@ class W extends b {
    */
   _healHidePriceAttr(t) {
     const e = this._getBlockElement(t);
-    if (!e || !("getAttribute" in e) || e.getAttribute(I) !== null)
+    if (!e || !("getAttribute" in e) || e.getAttribute(R) !== null)
       return;
     const { priceHideIfSameAsDiscounted: i } = s.getConfig(t);
     this._stampHidePrice(t, i);
@@ -338,6 +342,6 @@ class W extends b {
   }
 }
 export {
-  _ as BLOCK_ID,
-  W as RecommendationBlock
+  k as BLOCK_ID,
+  j as RecommendationBlock
 };

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

@@ -1,20 +1,20 @@
 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 r = (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 { 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";
@@ -31,36 +31,36 @@ 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 Dt, getCardComposition as Ft, 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 { useDebounceFn as h } from "../../../../../node_modules/@vueuse/shared/index.js";
+const W = "ui-elements-recommendation-block";
 class at extends N {
   constructor() {
     super(...arguments);
-    n(this, "store", R());
-    n(this, "storeUnsubscription", () => {
+    r(this, "store", R());
+    r(this, "storeUnsubscription", () => {
     });
     // blockId -> fingerprint of the node config as it looked at the last sync.
-    n(this, "syncedConfigs", /* @__PURE__ */ new Map());
+    r(this, "syncedConfigs", /* @__PURE__ */ new Map());
     // A config change waiting to reach the node config. Deferred so it can ride the
     // re-render's single apply() — one user action must cost one undo press.
-    n(this, "pendingNodeConfigWrite", null);
-    n(this, "pendingConfigFlushTimer", null);
+    r(this, "pendingNodeConfigWrite", null);
+    r(this, "pendingConfigFlushTimer", null);
     // Sub-control instances for lifecycle management
-    n(this, "algorithmControl", null);
-    n(this, "localeControl", null);
-    n(this, "currencyControl", null);
-    n(this, "productCountControl", null);
-    n(this, "productLayoutControl", null);
-    n(this, "filtersControl", null);
-    n(this, "shuffleControl", null);
-    n(this, "pricePlacementControl", null);
-    n(this, "layoutOrientationControl", null);
-    n(this, "strategyControl", null);
-    n(this, "rrsEnabled", !1);
+    r(this, "algorithmControl", null);
+    r(this, "localeControl", null);
+    r(this, "currencyControl", null);
+    r(this, "productCountControl", null);
+    r(this, "productLayoutControl", null);
+    r(this, "filtersControl", null);
+    r(this, "shuffleControl", null);
+    r(this, "pricePlacementControl", null);
+    r(this, "layoutOrientationControl", null);
+    r(this, "strategyControl", null);
+    r(this, "rrsEnabled", !1);
     /**
      * Debounced product fetch to prevent rapid API calls during config changes
      */
-    n(this, "_debouncedFetchProducts", f(async () => {
+    r(this, "_debouncedFetchProducts", h(async () => {
       try {
         await this.store.fetchRecommendationProducts();
       } finally {
@@ -78,7 +78,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(() => {
+    r(this, "_debouncedRegenerateWithProducts", h(() => {
       const t = this.store.recommendationProducts;
       if (!this.currentNode || !this.api)
         return;
@@ -99,7 +99,7 @@ class at extends N {
     }, 100));
   }
   getId() {
-    return $;
+    return W;
   }
   get _subControls() {
     return [
@@ -142,14 +142,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) >= _);
-    e.classList.toggle("strategy-block", r);
+    const o = this._getRecommendationIdFromNode(t) === null ? null : l.getConfig(t), n = this.rrsEnabled && !!o && (!!(o != null && o.strategyId) || ((o == null ? void 0 : o.configVersion) ?? 0) >= _);
+    e.classList.toggle("strategy-block", n);
   }
   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;
     }
@@ -161,9 +161,9 @@ class at extends N {
     const e = this._getRecommendationIdFromNode(t);
     e !== null && e !== this.store.currentRecommendationId && this.store.setCurrentBlock(e), e !== null && this._syncNodeConfigIfChanged(e), this._showControlsForBlock(t), e !== null && !((o = this.store.blockStates[e]) != null && o.isInitialized) && this._fetchBlockData(e).then(() => {
       this._initializeSubControls();
-    }), this._subControls.forEach((r) => {
+    }), this._subControls.forEach((n) => {
       var i;
-      r != null && r.api && (r.currentNode = t, (i = r.onTemplateNodeUpdated) == null || i.call(r, t));
+      n != null && n.api && (n.currentNode = t, (i = n.onTemplateNodeUpdated) == null || i.call(n, t));
     });
   }
   onDestroy() {
@@ -203,18 +203,23 @@ class at extends N {
    */
   _syncNodeConfigIfChanged(t) {
     this.pendingNodeConfigWrite && this.pendingNodeConfigWrite.blockId !== t && this._flushPendingNodeConfig();
-    const e = JSON.stringify(c.getConfig(this.currentNode)), o = this.syncedConfigs.get(t);
+    const e = JSON.stringify(l.getConfig(this.currentNode)), o = this.syncedConfigs.get(t);
     this.syncedConfigs.set(t, e), o !== e && (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, strategyId: o, size: n } = this.store.recommendationConfigs;
     this.pendingNodeConfigWrite = {
       blockId: t,
       node: this.currentNode,
-      updates: { filters: e, strategyId: o, size: r }
+      updates: { filters: e, strategyId: o, size: n }
     };
   }
   /**
@@ -229,7 +234,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
@@ -253,12 +258,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.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);
+    var n;
+    const t = l.getConfig(this.currentNode), e = this.store.currentRecommendationId, o = e !== null && ((n = this.store.blockStates[e]) == null ? void 0 : n.isInitialized);
     this.store.patchCurrentBlockConfig({
       strategy: t.strategy,
       strategyId: t.strategyId,
@@ -300,8 +305,8 @@ class at extends N {
     t !== null && this.store.markBlockInitialized(t), (await Promise.allSettled([
       this.store.fetchRecommendationCreateData(),
       this.store.fetchRecommendationFilters()
-    ])).forEach((o, r) => {
-      o.status === "rejected" && console.warn(`Recommendation block: ${["fetchRecommendationCreateData", "fetchRecommendationFilters"][r]} failed`, o.reason);
+    ])).forEach((o, n) => {
+      o.status === "rejected" && console.warn(`Recommendation block: ${["fetchRecommendationCreateData", "fetchRecommendationFilters"][n]} failed`, o.reason);
     }), this._applySmartDefaults();
     try {
       await this.store.fetchRecommendationProducts();
@@ -322,8 +327,8 @@ class at extends N {
   _applySmartDefaults() {
     if (!this.currentNode || !this.api)
       return;
-    const t = c.getConfig(this.currentNode), e = {};
-    let o = null, r = null, i = null;
+    const t = l.getConfig(this.currentNode), e = {};
+    let o = null, n = null, i = null;
     if (t.currency.code === a.currency.code) {
       const { currencyList: s } = this.store;
       s.length > 0 && (s.some(
@@ -338,7 +343,7 @@ class at extends N {
       const s = this.store.getActivePredictiveAlgorithms;
       s.length > 0 && (s.some(
         (d) => d.value === a.strategy
-      ) || (r = s[0].value, e.strategy = r));
+      ) || (n = s[0].value, e.strategy = n));
     }
     if (t.language === a.language) {
       const s = this.store.getLanguages;
@@ -346,7 +351,7 @@ class at extends N {
         (d) => d.value === a.language
       ) || (i = s[0].value, e.language = i));
     }
-    !o && !r && !i || (c.updateConfig(
+    !o && !n && !i || (l.updateConfig(
       this.api,
       this.currentNode,
       e,
@@ -367,7 +372,7 @@ class at extends N {
           thousandSeparator: a.currency.thousandSeparator
         }
       } : {},
-      ...r ? { strategy: r } : {},
+      ...n ? { strategy: n } : {},
       ...i ? { language: i } : {}
     }, { triggerRefetch: !1 }));
   }
@@ -381,8 +386,8 @@ class at extends N {
     const o = e.getAttribute(I);
     if (!o)
       return null;
-    const r = parseInt(o);
-    return Number.isNaN(r) ? null : r;
+    const n = parseInt(o);
+    return Number.isNaN(n) ? null : n;
   }
   /**
    * Listen to store changes that require product refresh or regeneration.
@@ -399,27 +404,27 @@ class at extends N {
    */
   _listenStateUpdates() {
     const { store: t } = this;
-    let e = t.recommendationProducts, o = t.$state.configVersion, r = t.currentRecommendationId;
+    let e = t.recommendationProducts, o = t.$state.configVersion, n = t.currentRecommendationId;
     this.storeUnsubscription = t.$subscribe(() => {
       const i = t.currentRecommendationId;
-      if (i !== r) {
-        r = i, e = t.recommendationProducts, o = t.$state.configVersion;
+      if (i !== n) {
+        n = i, e = t.recommendationProducts, o = t.$state.configVersion;
         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,

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 { CURRENT_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
 };