@useinsider/guido 3.18.1-beta.e39fb14 → 3.18.1-beta.e489d08

package/dist/src/composables/useConfig.d.ts

@@ -99,7 +99,6 @@ export declare const useConfig: () => {
             autosave: boolean;
             allowlistEnabled: boolean;
             linkTrackingIds: boolean;
-            allowSaveWithErrors: boolean;
             reusableRecommendationStrategy: boolean;
         };
         blocks: {
@@ -235,7 +234,6 @@ export declare const useConfig: () => {
         autosave: boolean;
         allowlistEnabled: boolean;
         linkTrackingIds: boolean;
-        allowSaveWithErrors: boolean;
         reusableRecommendationStrategy: boolean;
     } | null>;
     blocks: import("vue").ComputedRef<{

package/dist/src/composables/useHtmlValidator.d.ts

@@ -1,12 +1,5 @@
 import type { DynamicContent } from '@@/Types/generic';
 import type { FiltersResponse } from '@@/Types/recommendation';
-/**
- * Extracts recommendation block IDs directly from the HTML by parsing
- * the recommendation-id attribute on block elements.
- * This avoids depending on store state which may not be populated
- * when the settings panel hasn't been opened yet (e.g., saved templates).
- */
-export declare function getRecommendationBlockIds(html: string): string[];
 /**
  * Parses a `{{recoId_index_attributeName}}` variable (already stripped of the
  * braces/whitespace) into its recommendation id and attribute name. Returns
@@ -34,20 +27,6 @@ export declare function buildPartnerAttributeNameSet(filterList: FiltersResponse
  * before, so flagging it would false-positive on merge tags like `{{user.name}}`.
  */
 export declare function findInvalidCustomFields(html: string, validCustomFields: string[], recommendationBlockIds: string[], ignoredIds: Set<string>, ignoredAttributeNames: Set<string> | null): string[];
-/**
- * Returns the invalid `{{ ... }}` dynamic-content tags found in the HTML.
- *
- * A tag's variable name is the part before the first `|` (everything after it is the fallback).
- * Surrounding whitespace is always allowed (trimmed). A tag is valid only when the variable name
- * is an allowed system token, or it is a clean identifier (`[A-Za-z0-9_.]`) and — when a `|` is
- * present — the fallback after it is not empty. Otherwise the tag is invalid (e.g.
- * `{{Api Response: OA1 Main Item Url}}` — spaces/`:` — or `{{bounce|}}` — empty fallback).
- *
- * `data-*` block-config attribute values are excluded from the scan: blocks such as Items/cart
- * store product merge tags (`data-product_button_link="{{Abandoned Cart Item (1) Url}}"`) that the
- * backend resolves through its own pipeline, so they must not be treated as user-authored tags.
- */
-export declare function getInvalidDynamicContentTags(html: string): string[];
 export declare const useHtmlValidator: () => {
     validateHtml: (html: string, customFields: DynamicContent[], isOnSaveValidation?: boolean) => Promise<boolean>;
 };

package/dist/src/composables/useSave.d.ts

@@ -1,9 +1,4 @@
 import type { SavedTemplateDetails } from '@@/Types/stripo';
-type SaveDetails = Omit<SavedTemplateDetails, 'metadata' | 'silent'>;
-type SaveResult = SaveDetails | undefined;
 export declare const useSave: () => {
-    save: (suppressEmit?: boolean, silent?: boolean) => Promise<SaveResult>;
-    saveWithErrors: () => Promise<SaveResult>;
-    saveRawOnly: () => Promise<SaveResult>;
+    save: (suppressEmit?: boolean, silent?: boolean) => Promise<Omit<SavedTemplateDetails, "metadata" | "silent"> | undefined>;
 };
-export {};

package/dist/src/composables/validators/useCouponBlockValidator.d.ts

@@ -1,6 +1,3 @@
-export declare const COUPON_TAG_MODIFIED_KEY = "newsletter.coupon-tag-modified";
-/** Pure core: the tag text of every coupon block whose code was modified. */
-export declare function findInvalidCouponBlocks(html: string): string[];
 export declare const useCouponBlockValidator: () => {
     validateCouponBlockTags: (html: string) => boolean;
 };

package/dist/src/composables/validators/useUnsubscribeBlockValidator.d.ts

@@ -1,14 +1,4 @@
-declare const SINGLETON_BLOCK_RULES: {
-    pageType: number;
-    messageKey: string;
-}[];
-export declare const EMPTY_BLOCK_KEY = "unsubscription-preference.select-page-to-continue";
-/** Pure core: the singleton rules whose page type appears more than once. */
-export declare function findDuplicateUnsubPageTypes(html: string): typeof SINGLETON_BLOCK_RULES;
-/** Pure core: count of unsubscribe blocks with no page type or page list chosen. */
-export declare function countUnsubBlocksWithoutPage(html: string): number;
 export declare const useUnsubscribeBlockValidator: () => {
     validateUnsubscribeBlockUniqueness: (html: string) => boolean;
     validateUnsubscribeBlockHasTemplate: (html: string) => boolean;
 };
-export {};

package/dist/src/enums/extensions/recommendationBlock.d.ts

@@ -5,7 +5,9 @@ export declare const URLS: {
     PRODUCT_ATTRIBUTES_PATH: string;
     RECOMMENDATION_STRATEGIES_PATH: string;
     RECOMMENDATION_STRATEGIES_PANEL_PATH: string;
+    RECOMMENDATION_STRATEGY_CREATE_PATH: string;
 };
+export declare const strategyEditPath: (strategyId: string | number) => string;
 export declare const QUERY_PARAMS: {
     CLIENT_ID: string;
 };

package/dist/src/extensions/Blocks/Recommendation/controls/main/index.d.ts

@@ -66,6 +66,8 @@ export declare class RecommendationBlockControl extends CommonControl {
      * edits are never clobbered (SD-141049).
      */
     private _syncNodeConfigIfChanged;
+    /** Whether the deferred write would move the node's strategyId — see the caller (SD-149916). */
+    private _isPendingStrategyIdChange;
     private _capturePendingNodeConfigWrite;
     /**
      * Accumulates the pending write into `documentModifier` without applying, so the

package/dist/src/extensions/Blocks/Recommendation/controls/main/strategy.d.ts

@@ -2,6 +2,7 @@ import { CommonControl } from '../../../common-control';
 export declare const STRATEGY_CONTROL_ID = "recommendation-strategy-control";
 export declare class StrategyControl extends CommonControl {
     private store;
+    private onboardingStore;
     private storeUnsubscription;
     private listeners;
     private lastPaintedState;
@@ -9,6 +10,14 @@ export declare class StrategyControl extends CommonControl {
     getTemplate(): string;
     private _strategyAction;
     onRender(): void;
+    /**
+     * 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.
+     */
+    private _isNodeUnassigned;
     onDestroy(): void;
     private _teardown;
     private _setupButtonListeners;

package/dist/src/extensions/Blocks/Recommendation/store/recommendation.d.ts

@@ -71,8 +71,10 @@ interface StoreState {
     strategies: RrsStrategy[];
     strategiesLoading: boolean;
     strategyUrlTemplates: Record<string, string>;
+    strategiesFetched: boolean;
     blockStates: Record<number, PerBlockState>;
     currentRecommendationId: number | null;
+    strategyPanelOpen: boolean;
     configVersion: number;
 }
 export declare const useRecommendationExtensionStore: import("pinia").StoreDefinition<"guidoRecommendationExtension", StoreState, {
@@ -99,8 +101,10 @@ export declare const useRecommendationExtensionStore: import("pinia").StoreDefin
         }[];
         strategiesLoading: boolean;
         strategyUrlTemplates: Record<string, string>;
+        strategiesFetched: boolean;
         blockStates: Record<number, PerBlockState>;
         currentRecommendationId: number | null;
+        strategyPanelOpen: boolean;
         configVersion: number;
     } & import("pinia").PiniaCustomStateProperties<StoreState>): PerBlockConfigs;
     /**
@@ -125,8 +129,10 @@ export declare const useRecommendationExtensionStore: import("pinia").StoreDefin
         }[];
         strategiesLoading: boolean;
         strategyUrlTemplates: Record<string, string>;
+        strategiesFetched: boolean;
         blockStates: Record<number, PerBlockState>;
         currentRecommendationId: number | null;
+        strategyPanelOpen: boolean;
         configVersion: number;
     } & import("pinia").PiniaCustomStateProperties<StoreState>): RecommendationProduct[];
     /**
@@ -151,8 +157,10 @@ export declare const useRecommendationExtensionStore: import("pinia").StoreDefin
         }[];
         strategiesLoading: boolean;
         strategyUrlTemplates: Record<string, string>;
+        strategiesFetched: boolean;
         blockStates: Record<number, PerBlockState>;
         currentRecommendationId: number | null;
+        strategyPanelOpen: boolean;
         configVersion: number;
     } & import("pinia").PiniaCustomStateProperties<StoreState>): boolean;
     assignStrategyDrawerStatus(state: {
@@ -174,8 +182,10 @@ export declare const useRecommendationExtensionStore: import("pinia").StoreDefin
         }[];
         strategiesLoading: boolean;
         strategyUrlTemplates: Record<string, string>;
+        strategiesFetched: boolean;
         blockStates: Record<number, PerBlockState>;
         currentRecommendationId: number | null;
+        strategyPanelOpen: boolean;
         configVersion: number;
     } & import("pinia").PiniaCustomStateProperties<StoreState>): boolean;
     removeStrategyModalStatus(state: {
@@ -197,8 +207,10 @@ export declare const useRecommendationExtensionStore: import("pinia").StoreDefin
         }[];
         strategiesLoading: boolean;
         strategyUrlTemplates: Record<string, string>;
+        strategiesFetched: boolean;
         blockStates: Record<number, PerBlockState>;
         currentRecommendationId: number | null;
+        strategyPanelOpen: boolean;
         configVersion: number;
     } & import("pinia").PiniaCustomStateProperties<StoreState>): boolean;
     assignedStrategy(state: {
@@ -220,8 +232,10 @@ export declare const useRecommendationExtensionStore: import("pinia").StoreDefin
         }[];
         strategiesLoading: boolean;
         strategyUrlTemplates: Record<string, string>;
+        strategiesFetched: boolean;
         blockStates: Record<number, PerBlockState>;
         currentRecommendationId: number | null;
+        strategyPanelOpen: boolean;
         configVersion: number;
     } & import("pinia").PiniaCustomStateProperties<StoreState>): RrsStrategy | null;
     /**
@@ -246,8 +260,10 @@ export declare const useRecommendationExtensionStore: import("pinia").StoreDefin
         }[];
         strategiesLoading: boolean;
         strategyUrlTemplates: Record<string, string>;
+        strategiesFetched: boolean;
         blockStates: Record<number, PerBlockState>;
         currentRecommendationId: number | null;
+        strategyPanelOpen: boolean;
         configVersion: number;
     } & import("pinia").PiniaCustomStateProperties<StoreState>): boolean;
     /**
@@ -272,8 +288,10 @@ export declare const useRecommendationExtensionStore: import("pinia").StoreDefin
         }[];
         strategiesLoading: boolean;
         strategyUrlTemplates: Record<string, string>;
+        strategiesFetched: boolean;
         blockStates: Record<number, PerBlockState>;
         currentRecommendationId: number | null;
+        strategyPanelOpen: boolean;
         configVersion: number;
     } & import("pinia").PiniaCustomStateProperties<StoreState>): number;
     hasFilters(): boolean;
@@ -299,8 +317,10 @@ export declare const useRecommendationExtensionStore: import("pinia").StoreDefin
         }[];
         strategiesLoading: boolean;
         strategyUrlTemplates: Record<string, string>;
+        strategiesFetched: boolean;
         blockStates: Record<number, PerBlockState>;
         currentRecommendationId: number | null;
+        strategyPanelOpen: boolean;
         configVersion: number;
     } & import("pinia").PiniaCustomStateProperties<StoreState>) => {
         text: string;
@@ -325,8 +345,10 @@ export declare const useRecommendationExtensionStore: import("pinia").StoreDefin
         }[];
         strategiesLoading: boolean;
         strategyUrlTemplates: Record<string, string>;
+        strategiesFetched: boolean;
         blockStates: Record<number, PerBlockState>;
         currentRecommendationId: number | null;
+        strategyPanelOpen: boolean;
         configVersion: number;
     } & import("pinia").PiniaCustomStateProperties<StoreState>) => {
         text: string;
@@ -351,8 +373,10 @@ export declare const useRecommendationExtensionStore: import("pinia").StoreDefin
         }[];
         strategiesLoading: boolean;
         strategyUrlTemplates: Record<string, string>;
+        strategiesFetched: boolean;
         blockStates: Record<number, PerBlockState>;
         currentRecommendationId: number | null;
+        strategyPanelOpen: boolean;
         configVersion: number;
     } & import("pinia").PiniaCustomStateProperties<StoreState>) => {
         text: string;
@@ -433,6 +457,38 @@ export declare const useRecommendationExtensionStore: import("pinia").StoreDefin
     openRemoveStrategyModal(): void;
     closeRemoveStrategyModal(): void;
     fetchStrategies(): Promise<void>;
+    invalidateStrategies(): void;
+    /**
+     * Opens a Smart Recommender strategy page in a new tab and refreshes the list every time
+     * the user comes back. `visibilitychange` covers the tab case; `focus` covers the page
+     * landing in a separate window, where this document never goes hidden.
+     *
+     * Deliberately NOT one-shot: the strategy tab stays open, so a user can create a second
+     * strategy in it and come back again — a listener that disarmed on the first return would
+     * leave that one missing from the list. Armed only once the user actually opens a strategy
+     * page, so anyone who never does pays nothing.
+     */
+    _armStrategyPageReturn(url: string): void;
+    /**
+     * Reloads the products of the block the user is on, so an edit made in the strategy tab
+     * lands on the canvas instead of waiting for the assign drawer to be reopened.
+     *
+     * Only the selected block: the canvas regeneration is driven off the active block's
+     * product array, so refreshing an unselected block would fetch without redrawing it.
+     */
+    _refreshAssignedStrategyProducts(): Promise<void>;
+    /** Lets the editor tear the return listeners down when it unmounts. */
+    disposeStrategyPageReturn(): void;
+    /**
+     * Opens Smart Recommender's strategy creation page. Every return refreshes the list so the
+     * new strategy is offered; the user picks it themselves.
+     */
+    openStrategyCreation(): void;
+    /**
+     * Opens the strategy's edit page. Every return refreshes the list so the block reflects the
+     * edited configuration.
+     */
+    openStrategyEdit(strategyId: string | number): void;
     resolveStrategyUrlTemplate(strategy: RrsStrategy, currency: string): Promise<string>;
     assignStrategy(strategy: RrsStrategy): void;
     removeStrategy(): void;
@@ -451,10 +507,10 @@ export declare const useRecommendationExtensionStore: import("pinia").StoreDefin
      */
     hasInvalidBlock(): boolean;
     /**
-     * Per-instance counterpart to `hasInvalidBlock`: the ids of every block
-     * that fails required-field validation, one drawer row each (SD-147430).
+     * Which required fields are missing across the design's blocks, so the save
+     * toaster can name the cause instead of always showing the generic message.
      */
-    invalidBlockIds(): string[];
+    invalidBlockFields(): string[];
     fetchRecommendationProducts(): Promise<void>;
     _doFetchProducts(): Promise<void>;
     _doFetchStrategyProducts(blockId: number): Promise<void>;

package/dist/src/extensions/Blocks/Recommendation/validation/requiredFields.d.ts

@@ -15,7 +15,11 @@ export interface RequiredField {
     getAvailableOptions?: (store: ExtensionStoreSlice) => string[];
     condition?: (config: PerBlockConfigs) => boolean;
 }
+/** Field key of the strategy/algorithm descriptor, used to pick the toaster message. */
+export declare const STRATEGY_FIELD_KEY = "strategy";
 export declare const REQUIRED_RECOMMENDATION_FIELDS: RequiredField[];
 export declare const RecommendationRequiredFieldsKey = "newsletter.recommendation-fill-required-fields";
+/** Shown instead of the generic message when the only thing missing is the strategy. */
+export declare const RecommendationAssignStrategyKey = "newsletter.recommendation-assign-strategy";
 export declare function getInvalidFields(config: PerBlockConfigs, store: ExtensionStoreSlice, fields?: RequiredField[]): string[];
 export declare function isConfigValid(config: PerBlockConfigs, store: ExtensionStoreSlice): boolean;

package/dist/src/stores/config.d.ts

@@ -104,7 +104,6 @@ export declare const useConfigStore: import("pinia").StoreDefinition<"guido-conf
                 autosave: boolean;
                 allowlistEnabled: boolean;
                 linkTrackingIds: boolean;
-                allowSaveWithErrors: boolean;
                 reusableRecommendationStrategy: boolean;
             };
             blocks: {
@@ -246,7 +245,6 @@ export declare const useConfigStore: import("pinia").StoreDefinition<"guido-conf
                 autosave: boolean;
                 allowlistEnabled: boolean;
                 linkTrackingIds: boolean;
-                allowSaveWithErrors: boolean;
                 reusableRecommendationStrategy: boolean;
             };
             blocks: {
@@ -388,7 +386,6 @@ export declare const useConfigStore: import("pinia").StoreDefinition<"guido-conf
                 autosave: boolean;
                 allowlistEnabled: boolean;
                 linkTrackingIds: boolean;
-                allowSaveWithErrors: boolean;
                 reusableRecommendationStrategy: boolean;
             };
             blocks: {
@@ -530,7 +527,6 @@ export declare const useConfigStore: import("pinia").StoreDefinition<"guido-conf
                 autosave: boolean;
                 allowlistEnabled: boolean;
                 linkTrackingIds: boolean;
-                allowSaveWithErrors: boolean;
                 reusableRecommendationStrategy: boolean;
             };
             blocks: {
@@ -672,7 +668,6 @@ export declare const useConfigStore: import("pinia").StoreDefinition<"guido-conf
                 autosave: boolean;
                 allowlistEnabled: boolean;
                 linkTrackingIds: boolean;
-                allowSaveWithErrors: boolean;
                 reusableRecommendationStrategy: boolean;
             };
             blocks: {
@@ -814,7 +809,6 @@ export declare const useConfigStore: import("pinia").StoreDefinition<"guido-conf
                 autosave: boolean;
                 allowlistEnabled: boolean;
                 linkTrackingIds: boolean;
-                allowSaveWithErrors: boolean;
                 reusableRecommendationStrategy: boolean;
             };
             blocks: {
@@ -956,7 +950,6 @@ export declare const useConfigStore: import("pinia").StoreDefinition<"guido-conf
                 autosave: boolean;
                 allowlistEnabled: boolean;
                 linkTrackingIds: boolean;
-                allowSaveWithErrors: boolean;
                 reusableRecommendationStrategy: boolean;
             };
             blocks: {
@@ -1098,7 +1091,6 @@ export declare const useConfigStore: import("pinia").StoreDefinition<"guido-conf
                 autosave: boolean;
                 allowlistEnabled: boolean;
                 linkTrackingIds: boolean;
-                allowSaveWithErrors: boolean;
                 reusableRecommendationStrategy: boolean;
             };
             blocks: {
@@ -1240,7 +1232,6 @@ export declare const useConfigStore: import("pinia").StoreDefinition<"guido-conf
                 autosave: boolean;
                 allowlistEnabled: boolean;
                 linkTrackingIds: boolean;
-                allowSaveWithErrors: boolean;
                 reusableRecommendationStrategy: boolean;
             };
             blocks: {
@@ -1382,7 +1373,6 @@ export declare const useConfigStore: import("pinia").StoreDefinition<"guido-conf
                 autosave: boolean;
                 allowlistEnabled: boolean;
                 linkTrackingIds: boolean;
-                allowSaveWithErrors: boolean;
                 reusableRecommendationStrategy: boolean;
             };
             blocks: {
@@ -1524,7 +1514,6 @@ export declare const useConfigStore: import("pinia").StoreDefinition<"guido-conf
                 autosave: boolean;
                 allowlistEnabled: boolean;
                 linkTrackingIds: boolean;
-                allowSaveWithErrors: boolean;
                 reusableRecommendationStrategy: boolean;
             };
             blocks: {
@@ -1666,7 +1655,6 @@ export declare const useConfigStore: import("pinia").StoreDefinition<"guido-conf
                 autosave: boolean;
                 allowlistEnabled: boolean;
                 linkTrackingIds: boolean;
-                allowSaveWithErrors: boolean;
                 reusableRecommendationStrategy: boolean;
             };
             blocks: {
@@ -1808,7 +1796,6 @@ export declare const useConfigStore: import("pinia").StoreDefinition<"guido-conf
                 autosave: boolean;
                 allowlistEnabled: boolean;
                 linkTrackingIds: boolean;
-                allowSaveWithErrors: boolean;
                 reusableRecommendationStrategy: boolean;
             };
             blocks: {
@@ -1950,7 +1937,6 @@ export declare const useConfigStore: import("pinia").StoreDefinition<"guido-conf
                 autosave: boolean;
                 allowlistEnabled: boolean;
                 linkTrackingIds: boolean;
-                allowSaveWithErrors: boolean;
                 reusableRecommendationStrategy: boolean;
             };
             blocks: {
@@ -2092,7 +2078,6 @@ export declare const useConfigStore: import("pinia").StoreDefinition<"guido-conf
                 autosave: boolean;
                 allowlistEnabled: boolean;
                 linkTrackingIds: boolean;
-                allowSaveWithErrors: boolean;
                 reusableRecommendationStrategy: boolean;
             };
             blocks: {
@@ -2136,18 +2121,6 @@ export declare const useConfigStore: import("pinia").StoreDefinition<"guido-conf
             };
         } | null;
     } & import("pinia").PiniaCustomStateProperties<ConfigStoreState>) => number;
-    /**
-     * Transactional campaigns keep the hard save block. Reactive getter, so
-     * consumers stay safe in setup where a raw config read snapshots null (SD-147945).
-     */
-    isTransactional(): boolean;
-    /**
-     * Soft-save policy: may the editor persist a template with validation errors?
-     * Default true (host sets features.allowSaveWithErrors=false for transactional +
-     * unsub). Reactive getter defaulting true — reading via isFeatureEnabled would
-     * floor to false pre-init and momentarily hide the soft-save option (SD-147430).
-     */
-    allowSaveWithErrors(): boolean;
     /**
      * Get the username
      */
@@ -2246,7 +2219,6 @@ export declare const useConfigStore: import("pinia").StoreDefinition<"guido-conf
                 autosave: boolean;
                 allowlistEnabled: boolean;
                 linkTrackingIds: boolean;
-                allowSaveWithErrors: boolean;
                 reusableRecommendationStrategy: boolean;
             };
             blocks: {
@@ -2388,7 +2360,6 @@ export declare const useConfigStore: import("pinia").StoreDefinition<"guido-conf
                 autosave: boolean;
                 allowlistEnabled: boolean;
                 linkTrackingIds: boolean;
-                allowSaveWithErrors: boolean;
                 reusableRecommendationStrategy: boolean;
             };
             blocks: {
@@ -2532,7 +2503,6 @@ export declare const useConfigStore: import("pinia").StoreDefinition<"guido-conf
                 autosave: boolean;
                 allowlistEnabled: boolean;
                 linkTrackingIds: boolean;
-                allowSaveWithErrors: boolean;
                 reusableRecommendationStrategy: boolean;
             };
             blocks: {