@useinsider/guido 3.19.0-beta.f02eaa1 → 3.19.0-beta.f66e9de

package/dist/extensions/Blocks/Recommendation/constants/defaultConfig.js

@@ -1,13 +1,13 @@
-import { DEFAULT_COLUMN_SPACING as R, DEFAULT_ROW_SPACING as s, DEFAULT_MOBILE_ROW_SPACING as n, DEFAULT_MOBILE_COLUMN_SPACING as O, DEFAULT_MOBILE_CARDS_IN_ROW as e, DEFAULT_CARDS_IN_ROW as C } from "./layout.js";
-import { ATTR_PRODUCT_IMAGE as o, ATTR_PRODUCT_NAME as t, ATTR_PRODUCT_OLD_PRICE as _, ATTR_PRODUCT_PRICE as T, ATTR_PRODUCT_OMNIBUS_PRICE as r, ATTR_PRODUCT_OMNIBUS_DISCOUNT as I, ATTR_PRODUCT_BUTTON as i, ATTR_RECOMMENDATION_TITLE as U } from "./selectors.js";
-const A = 1, u = 2, D = {
+import { DEFAULT_COLUMN_SPACING as i, DEFAULT_ROW_SPACING as s, DEFAULT_MOBILE_ROW_SPACING as n, DEFAULT_MOBILE_COLUMN_SPACING as O, DEFAULT_MOBILE_CARDS_IN_ROW as e, DEFAULT_CARDS_IN_ROW as C } from "./layout.js";
+import { ATTR_PRODUCT_IMAGE as o, ATTR_PRODUCT_NAME as t, ATTR_PRODUCT_OLD_PRICE as _, ATTR_PRODUCT_PRICE as T, ATTR_PRODUCT_OMNIBUS_PRICE as I, ATTR_PRODUCT_OMNIBUS_DISCOUNT as R, ATTR_PRODUCT_BUTTON as r, ATTR_RECOMMENDATION_TITLE as E } from "./selectors.js";
+const U = 1, u = 2, m = 2, A = {
   code: "USD",
   symbol: "USD",
   alignment: "after",
   decimalCount: 2,
   decimalSeparator: ".",
   thousandSeparator: ","
-}, E = {
+}, D = {
   textBefore: "",
   textAfter: ""
 }, a = {
@@ -18,26 +18,26 @@ const A = 1, u = 2, D = {
   t,
   _,
   T,
-  r,
   I,
-  i
+  R,
+  r
 ], N = {
-  [U]: !0,
+  [E]: !0,
   [o]: !0,
   [t]: !0,
   [T]: !0,
   [_]: !0,
-  [r]: !1,
   [I]: !1,
-  [i]: !0
-}, m = {
+  [R]: !1,
+  [r]: !0
+}, L = {
   // Settings
   strategy: "mostPopular",
   productIds: [],
   size: "6",
   shuffleProducts: !1,
   language: "en_US",
-  currency: D,
+  currency: A,
   filters: [],
   strategyId: "",
   // Layout
@@ -46,7 +46,7 @@ const A = 1, u = 2, D = {
   mobileCardsInRow: e,
   mobileLayoutEnabled: !1,
   previousMobileCardsInRow: e,
-  columnSpacing: R,
+  columnSpacing: i,
   rowSpacing: s,
   mobileColumnSpacing: O,
   mobileRowSpacing: n,
@@ -55,7 +55,7 @@ const A = 1, u = 2, D = {
   visibility: N,
   previousTitleHtml: "",
   // Element settings
-  omnibusPrice: E,
+  omnibusPrice: D,
   omnibusDiscount: a,
   textTrimming: !1,
   // Price placement (block-level, affects all cards)
@@ -66,17 +66,18 @@ const A = 1, u = 2, D = {
   // of "false" (the actual hiding happens server-side at send).
   priceHideIfSameAsDiscounted: !1,
   // Meta
-  configVersion: A,
+  configVersion: U,
   recommendationId: 0
-}, L = [11, 12];
+}, P = [11, 12];
 export {
   u as CURRENT_CONFIG_VERSION,
   c as DEFAULT_COMPOSITION,
-  D as DEFAULT_CURRENCY,
-  m as DEFAULT_NODE_CONFIG,
+  A as DEFAULT_CURRENCY,
+  L as DEFAULT_NODE_CONFIG,
   a as DEFAULT_OMNIBUS_DISCOUNT,
-  E as DEFAULT_OMNIBUS_PRICE,
+  D as DEFAULT_OMNIBUS_PRICE,
   N as DEFAULT_VISIBILITY,
-  L as EXCLUDED_ALGORITHM_IDS,
-  A as PRE_RRS_CONFIG_VERSION
+  P as EXCLUDED_ALGORITHM_IDS,
+  U as PRE_RRS_CONFIG_VERSION,
+  m as RRS_CONFIG_VERSION
 };

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

@@ -1,20 +1,20 @@
 var m = Object.defineProperty;
-var C = (g, l, t) => l in g ? m(g, l, { enumerable: !0, configurable: !0, writable: !0, value: t }) : g[l] = t;
-var r = (g, l, t) => C(g, 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 i = (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 { ALGORITHM_CONTROL_ID as dt } from "./algorithm.js";
 import { CurrencyControl as P } from "./currency.js";
 import { CURRENCY_CONTROL_ID as gt } from "./currency.js";
 import { FiltersControl as S } from "./filters.js";
-import { FILTERS_CONTROL_ID as ht } 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,34 +31,34 @@ import { StrategyControl as w } from "./strategy.js";
 import { STRATEGY_CONTROL_ID as Lt } from "./strategy.js";
 import { setCurrencyAttributes as A, getBlockElement as B, isPartnerManagedBlock as v, updateProductContentInPlace as x, regenerateProductRowsWithStyles as U } from "./utils.js";
 import { adjustProductsToSize as Et, formatProductPrice as Ft, getCardComposition as Dt, getCurrentLayout as wt, getMainContainerTbody as At, regenerateMobileProductRows as Bt, updatePricesInPlace as vt, updateSingleProductContent as xt } 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);
-    r(this, "store", R());
-    r(this, "storeUnsubscription", () => {
+    i(this, "store", R());
+    i(this, "storeUnsubscription", () => {
     });
     // 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.
-    r(this, "pendingNodeConfigWrite", null);
-    r(this, "pendingConfigFlushTimer", null);
+    i(this, "pendingNodeConfigWrite", null);
+    i(this, "pendingConfigFlushTimer", null);
     // Sub-control instances for lifecycle management
-    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);
+    i(this, "algorithmControl", null);
+    i(this, "localeControl", null);
+    i(this, "currencyControl", null);
+    i(this, "productCountControl", null);
+    i(this, "productLayoutControl", null);
+    i(this, "filtersControl", null);
+    i(this, "shuffleControl", null);
+    i(this, "pricePlacementControl", null);
+    i(this, "layoutOrientationControl", null);
+    i(this, "strategyControl", null);
+    i(this, "rrsEnabled", !1);
     /**
      * Debounced product fetch to prevent rapid API calls during config changes
      */
-    r(this, "_debouncedFetchProducts", f(async () => {
+    i(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.
      */
-    r(this, "_debouncedRegenerateWithProducts", f(() => {
+    i(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 [
@@ -136,18 +136,18 @@ class at extends N {
         `;
   }
   _showControlsForBlock(t) {
-    var i;
-    const e = (i = this.getContainer()) == null ? void 0 : i.querySelector(".recommendation-controls-container");
+    var r;
+    const e = (r = this.getContainer()) == null ? void 0 : r.querySelector(".recommendation-controls-container");
     if (!e)
       return;
-    const o = this._getRecommendationIdFromNode(t) === null ? null : c.getConfig(t), n = this.rrsEnabled && !!o && (!!(o != null && o.strategyId) || ((o == null ? void 0 : o.configVersion) ?? 0) >= _), s = this.rrsEnabled && !!o && !n;
+    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) >= _), s = this.rrsEnabled && !!o && !n;
     e.classList.toggle("strategy-block", n), e.classList.toggle("legacy-strategy-block", s);
   }
   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)), n = e.syncedConfigFingerprint;
+    const o = JSON.stringify(l.getConfig(this.currentNode)), n = e.syncedConfigFingerprint;
     e.syncedConfigFingerprint = o, n !== 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: n, nodeConfigVersion: s } = this.store.recommendationConfigs;
+    const { filters: e, strategy: o, strategyId: n, size: s, nodeConfigVersion: r } = this.store.recommendationConfigs;
     this.pendingNodeConfigWrite = {
       blockId: t,
       node: this.currentNode,
-      updates: { filters: e, strategyId: o, size: n, configVersion: s }
+      updates: { filters: e, strategy: o, strategyId: n, size: s, configVersion: r }
     };
   }
   /**
@@ -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 n;
-    const t = c.getConfig(this.currentNode), e = this.store.currentRecommendationId, o = e !== null && ((n = this.store.blockStates[e]) == null ? void 0 : n.isInitialized);
+    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,
@@ -329,31 +334,31 @@ 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, n = null, s = null;
     if (t.currency.code === a.currency.code) {
-      const { currencyList: i } = this.store;
-      i.length > 0 && (i.some(
-        (d) => d.value === `price.${a.currency.code}`
-      ) || (o = i[0].value.replace("price.", ""), e.currency = {
+      const { currencyList: r } = this.store;
+      r.length > 0 && (r.some(
+        (u) => u.value === `price.${a.currency.code}`
+      ) || (o = r[0].value.replace("price.", ""), e.currency = {
         ...a.currency,
         code: o,
         symbol: o
       }));
     }
     if (t.strategy === a.strategy) {
-      const i = this.store.getActivePredictiveAlgorithms;
-      i.length > 0 && (i.some(
-        (d) => d.value === a.strategy
-      ) || (n = i[0].value, e.strategy = n));
+      const r = this.store.getActivePredictiveAlgorithms;
+      r.length > 0 && (r.some(
+        (u) => u.value === a.strategy
+      ) || (n = r[0].value, e.strategy = n));
     }
     if (t.language === a.language) {
-      const i = this.store.getLanguages;
-      i.length > 0 && (i.some(
-        (d) => d.value === a.language
-      ) || (s = i[0].value, e.language = s));
+      const r = this.store.getLanguages;
+      r.length > 0 && (r.some(
+        (u) => u.value === a.language
+      ) || (s = r[0].value, e.language = s));
     }
-    !o && !n && !s || (c.updateConfig(
+    !o && !n && !s || (l.updateConfig(
       this.api,
       this.currentNode,
       e,
@@ -413,20 +418,20 @@ class at extends N {
         n = s, e = t.recommendationProducts, o = t.$state.configVersion;
         return;
       }
-      const i = t.$state.configVersion;
-      i !== o && (o = i, this._capturePendingNodeConfigWrite(), this._debouncedFetchProducts());
-      const u = t.recommendationProducts, d = u !== e, h = Array.isArray(u) && u.length > 0;
-      d && h && (e = u, this._debouncedRegenerateWithProducts());
+      const r = t.$state.configVersion;
+      r !== o && (o = r, this._capturePendingNodeConfigWrite(), this._isPendingStrategyIdChange() && this._flushPendingNodeConfig(), this._debouncedFetchProducts());
+      const d = t.recommendationProducts, u = d !== e, f = Array.isArray(d) && d.length > 0;
+      u && f && (e = d, this._debouncedRegenerateWithProducts());
     });
   }
 }
 export {
-  ut as ALGORITHM_CONTROL_ID,
+  dt as ALGORITHM_CONTROL_ID,
   T as AlgorithmControl,
-  $ as CONTROL_BLOCK_ID,
+  W as CONTROL_BLOCK_ID,
   gt as CURRENCY_CONTROL_ID,
   P as CurrencyControl,
-  ht 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,49 +1,64 @@
-var p = Object.defineProperty;
-var v = (i, s, t) => s in i ? p(i, s, { enumerable: !0, configurable: !0, writable: !0, value: t }) : i[s] = t;
-var r = (i, s, t) => v(i, typeof s != "symbol" ? s + "" : s, t);
-import { useLegacyStrategyConversion as _ } from "../../../../../composables/useLegacyStrategyConversion.js";
-import { ACADEMY_LINKS as b } from "../../../../../enums/academy.js";
-import { useOnboardingStore as S } from "../../../../../stores/onboarding.js";
-import { CommonControl as f } from "../../../common-control.js";
-import { useRecommendationExtensionStore as $ } from "../../store/recommendation.js";
-const C = "recommendation-strategy-control";
-class N extends f {
+var C = Object.defineProperty;
+var $ = (c, n, t) => n in c ? C(c, n, { enumerable: !0, configurable: !0, writable: !0, value: t }) : c[n] = t;
+var i = (c, n, t) => $(c, typeof n != "symbol" ? n + "" : n, t);
+import { useLegacyStrategyConversion as k } from "../../../../../composables/useLegacyStrategyConversion.js";
+import { ACADEMY_LINKS as L } from "../../../../../enums/academy.js";
+import { useOnboardingStore as w } from "../../../../../stores/onboarding.js";
+import { CommonControl as A } from "../../../common-control.js";
+import { RRS_CONFIG_VERSION as N } from "../../constants/defaultConfig.js";
+import { RecommendationConfigService as b } from "../../services/configService.js";
+import { useRecommendationExtensionStore as R } from "../../store/recommendation.js";
+const E = "recommendation-strategy-control";
+class M extends A {
   constructor() {
     super(...arguments);
-    r(this, "store", $());
-    r(this, "onboardingStore", S());
-    r(this, "legacyConversion", _());
-    r(this, "storeUnsubscription", () => {
+    i(this, "store", R());
+    i(this, "onboardingStore", w());
+    i(this, "legacyConversion", k());
+    i(this, "storeUnsubscription", () => {
     });
-    r(this, "listeners", []);
-    r(this, "lastPaintedState", null);
+    i(this, "listeners", []);
+    i(this, "lastPaintedState", null);
   }
   getId() {
-    return C;
+    return E;
   }
   getTemplate() {
     const t = this.api.translate(
       "You can assign a recommendation strategy by selecting an existing one or creating a new one to show specific content."
-    ), e = this.api.translate("For more information, you can"), a = this.api.translate("visit Academy"), o = this.api.translate("Legacy Strategy"), n = this.api.translate(
+    ), e = this.api.translate("For more information, you can"), s = this.api.translate("visit Academy"), o = this.api.translate("Legacy Strategy"), a = this.api.translate(
       "Note that this is an older version of the Recommendation Strategy. We recommend you update it to the new Recommendation Strategy for an improved experience."
     );
     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="${b.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="${L.EMAIL_RECOMMENDATION_BLOCK}"
+                                target="_blank"
+                                rel="noopener noreferrer">${s}</a>.
+                        </p>
+                    </div>
+                    <div data-strategy-empty-state style="display: none;">
+                        ${this._GuTwoColumns([
       this._GuButton({
         name: "assignStrategy",
         label: this.api.translate("Assign Strategy"),
@@ -54,115 +69,131 @@ class N extends f {
         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-legacy-state style="display: none;">
-                    <div class="strategy-control-notice-row">
-                        <div class="strategy-control-notice">
-                            <p class="strategy-control-notice-text">${n}</p>
-                        </div>
                     </div>
-                    <div class="strategy-control-name-band">
-                        <span class="strategy-control-name">${o}</span>
-                        ${this._strategyAction({
+                    <div data-strategy-legacy-state style="display: none;">
+                        <div class="strategy-control-notice-row">
+                            <div class="strategy-control-notice">
+                                <p class="strategy-control-notice-text">${a}</p>
+                            </div>
+                        </div>
+                        <div class="strategy-control-name-band">
+                            <span class="strategy-control-name">${o}</span>
+                            ${this._strategyAction({
       id: "guido__btn-update-legacy-strategy",
       label: this.api.translate("Update"),
       modifier: "update"
     })}
-                    </div>
-                    <div class="strategy-control-actions">
-                        ${this._strategyAction({
+                        </div>
+                        <div class="strategy-control-actions">
+                            ${this._strategyAction({
       id: "guido__btn-change-legacy-strategy",
       label: this.api.translate("Change"),
       modifier: "change"
     })}
-                        ${this._strategyAction({
+                            ${this._strategyAction({
       id: "guido__btn-remove-legacy-strategy",
       label: this.api.translate("Remove"),
       modifier: "remove"
     })}
+                        </div>
                     </div>
-                </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>
         `;
   }
-  _strategyAction({ id: t, label: e, modifier: a }) {
+  _strategyAction({ id: t, label: e, modifier: s }) {
     return `
             <button
                 id="${t}"
                 type="button"
-                class="strategy-control-action strategy-control-action--${a}">
+                class="strategy-control-action strategy-control-action--${s}">
                 <span>${e}</span>
             </button>
         `;
   }
   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(() => {
     });
-    const t = this.store.currentRecommendationId, e = t !== null && !!this.store.blockStates[t], a = this.store.isLegacyStrategyBlock;
-    e && !this.store.recommendationConfigs.strategyId && !a && this.onboardingStore.startOnboarding("recommendationStrategyOnboarding"), e && a && this.onboardingStore.startOnboarding("recommendationLegacyStrategyOnboarding");
+  }
+  /**
+   * 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 (!b.hasConfig(this.currentNode))
+      return !1;
+    const t = b.getConfig(this.currentNode);
+    return !t.strategyId && (t.configVersion ?? 0) >= N;
   }
   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, o) => {
-      const n = t.querySelector(a);
-      n && (n.addEventListener("click", o), this.listeners.push({ element: n, handler: o }));
+    const e = (s, o) => {
+      const a = t.querySelector(s);
+      a && (a.addEventListener("click", o), this.listeners.push({ element: a, handler: o }));
     };
-    e("#guido__btn-assign-strategy", () => this.store.openAssignStrategyDrawer()), e("#guido__btn-change-strategy", () => this.store.openAssignStrategyDrawer()), e("#guido__btn-change-legacy-strategy", () => this.store.openAssignStrategyDrawer()), e("#guido__btn-remove-strategy", () => this.store.openRemoveStrategyModal()), e("#guido__btn-remove-legacy-strategy", () => this.store.openRemoveStrategyModal()), e("#guido__btn-update-legacy-strategy", () => {
+    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-change-legacy-strategy", () => this.store.openAssignStrategyDrawer()), e("#guido__btn-remove-strategy", () => this.store.openRemoveStrategyModal()), e("#guido__btn-remove-legacy-strategy", () => this.store.openRemoveStrategyModal()), e("#guido__btn-update-legacy-strategy", () => {
       this.legacyConversion.convertLegacyStrategy();
     });
   }
   _renderState() {
-    var h;
-    const { strategyId: t } = this.store.recommendationConfigs, e = t ? ((h = this.store.assignedStrategy) == null ? void 0 : h.name) ?? "" : "", a = this.store.legacyConversionLoading, o = this.store.isLegacyStrategyBlock, n = `${t}\0${e}\0${a}\0${o}`;
-    if (n === this.lastPaintedState)
+    var _;
+    const { strategyId: t } = this.store.recommendationConfigs, e = t ? ((_ = this.store.assignedStrategy) == null ? void 0 : _.name) ?? "" : "", s = !!t && !e && (this.store.strategiesLoading || !this.store.strategiesSettled), o = this.store.legacyConversionLoading, a = this.store.isLegacyStrategyBlock, h = `${t}\0${e}\0${s}\0${o}\0${a}`;
+    if (h === this.lastPaintedState)
       return;
-    const c = this.getContainer();
-    if (!c)
+    const r = this.getContainer();
+    if (!r)
       return;
-    const d = c.querySelector("[data-strategy-empty-state]"), y = c.querySelector("[data-strategy-assigned-state]"), g = c.querySelector("[data-strategy-legacy-state]"), m = c.querySelector("[data-strategy-name]");
+    const d = r.querySelector("[data-strategy-empty-state]"), y = r.querySelector("[data-strategy-assigned-state]"), g = r.querySelector("[data-strategy-legacy-state]"), p = r.querySelector("[data-strategy-loading-state]"), u = r.querySelector("[data-strategy-content]"), m = r.querySelector("[data-strategy-name]"), v = r.querySelector("[data-strategy-name-skeleton]");
     if (!d || !y || !g)
       return;
-    const u = t ? y : o ? g : d;
+    p && (p.style.display = "none"), u && (u.style.display = "");
+    const f = t ? y : a ? g : d;
     [d, y, g].forEach((l) => {
-      l.style.display = l === u ? "" : "none";
+      l.style.display = l === f ? "" : "none";
     }), g.querySelectorAll("button").forEach((l) => {
-      l.disabled = a;
-    }), g.classList.toggle("is-converting", a), m && t && (m.textContent = e || `#${t}`), this.lastPaintedState = n;
+      l.disabled = o;
+    }), g.classList.toggle("is-converting", o), !t && this._isNodeUnassigned() && this.onboardingStore.startOnboarding("recommendationStrategyOnboarding");
+    const S = this.store.currentRecommendationId;
+    a && S !== null && this.store.blockStates[S] && this.onboardingStore.startOnboarding("recommendationLegacyStrategyOnboarding"), m && (m.style.display = s ? "none" : "", t && !s && (m.textContent = e || `#${t}`)), v && (v.style.display = s ? "block" : "none"), this.lastPaintedState = h;
   }
 }
 export {
-  C as STRATEGY_CONTROL_ID,
-  N as StrategyControl
+  E as STRATEGY_CONTROL_ID,
+  M as StrategyControl
 };