@ninetailed/experience.js-plugin-preview 7.17.4 → 7.17.6

package/index.cjs.js

@@ -191,7 +191,7 @@ class NinetailedPreviewPlugin extends experience_jsPluginAnalytics.NinetailedPlu
      * Returns a middleware function that applies variable overwrites to changes
      */
     this.getChangesModificationMiddleware = () => {
-      if (!this.isActiveInstance || Object.keys(this.variableOverwrites).length === 0) {
+      if (!this.isActiveInstance || Object.keys(this.variableOverwrites).length === 0 && Object.keys(this.audienceOverwrites).length === 0) {
         return undefined;
       }
       return ({
@@ -202,9 +202,16 @@ class NinetailedPreviewPlugin extends experience_jsPluginAnalytics.NinetailedPlu
             changes: inputChanges
           };
         }
-        // Calculate and return overridden changes on demand instead of storing them
+        const filteredChanges = inputChanges.filter(change => {
+          var _b, _c;
+          const experienceId = (_b = change.meta) === null || _b === void 0 ? void 0 : _b.experienceId;
+          if (!experienceId) return true;
+          const experience = this.experiences.find(e => e.id === experienceId);
+          if (!((_c = experience === null || experience === void 0 ? void 0 : experience.audience) === null || _c === void 0 ? void 0 : _c.id)) return true;
+          return this.audienceOverwrites[experience.audience.id] !== false;
+        });
         return {
-          changes: this.getEffectiveChanges(inputChanges)
+          changes: this.getEffectiveChanges(filteredChanges)
         };
       };
     };
@@ -230,31 +237,7 @@ class NinetailedPreviewPlugin extends experience_jsPluginAnalytics.NinetailedPlu
         // Handle entry replacements as before
         const entryReplacementComponents = experience.components.filter(component => component.type === experience_jsShared.ComponentTypeEnum.EntryReplacement && 'id' in component.baseline);
         const baselineComponent = entryReplacementComponents.find(component => component.baseline.id === baseline.id);
-        // Get the selected variant index
         const variantIndex = this.pluginApi.experienceVariantIndexes[experience.id];
-        // Handle variable components for this experience (NEW CODE)
-        if (variantIndex !== undefined) {
-          // Process all variable components for this experience
-          const variableComponents = experience.components.filter(component => component.type === experience_jsShared.ComponentTypeEnum.InlineVariable);
-          // Set variable values based on the selected variant index
-          variableComponents.forEach(component => {
-            const key = component.key;
-            let value;
-            if (variantIndex === 0) {
-              value = component.baseline;
-            } else {
-              const variant = component.variants[variantIndex - 1];
-              value = variant && 'value' in variant ? variant.value : component.baseline;
-            }
-            // Set the variable in our changes system
-            this.setVariableValue({
-              experienceId: experience.id,
-              key,
-              value,
-              variantIndex
-            });
-          });
-        }
         // Continue with entry replacement handling
         if (!baselineComponent) {
           return {
@@ -398,29 +381,6 @@ class NinetailedPreviewPlugin extends experience_jsPluginAnalytics.NinetailedPlu
       experience_jsShared.logger.warn(`You cannot deactivate an unknown audience (id: ${id}). How did you get it in the first place?`);
       return;
     }
-    this.experienceVariantIndexOverwrites = Object.entries(this.experienceVariantIndexOverwrites).filter(([key, _]) => {
-      return !this.experiences.filter(experience => {
-        var _b;
-        return ((_b = experience.audience) === null || _b === void 0 ? void 0 : _b.id) === id;
-      }).map(experience => experience.id).includes(key);
-    }).reduce((acc, [key, value]) => {
-      return Object.assign(Object.assign({}, acc), {
-        [key]: value
-      });
-    }, {});
-    this.audienceOverwrites = Object.assign(Object.assign({}, this.audienceOverwrites), {
-      [id]: false
-    });
-    this.onChange();
-    this.experiences.filter(experience => {
-      var _b;
-      return ((_b = experience.audience) === null || _b === void 0 ? void 0 : _b.id) === id;
-    }).forEach(experience => {
-      this.setExperienceVariant({
-        experienceId: experience.id,
-        variantIndex: 0
-      });
-    });
     this.audienceOverwrites = Object.assign(Object.assign({}, this.audienceOverwrites), {
       [id]: false
     });
@@ -434,6 +394,22 @@ class NinetailedPreviewPlugin extends experience_jsPluginAnalytics.NinetailedPlu
       experience_jsShared.logger.warn(`You cannot reset an unknown audience (id: ${id}). How did you get it in the first place?`);
       return;
     }
+    // Identify all experiences that belong to this audience
+    const experiencesToReset = this.experiences.filter(experience => {
+      var _b;
+      return ((_b = experience.audience) === null || _b === void 0 ? void 0 : _b.id) === id;
+    });
+    if (experiencesToReset.length > 0) {
+      const experienceIdsToReset = experiencesToReset.map(e => e.id);
+      // 1. Clear any variable overwrites that were set for these experiences (allows natural evaluation)
+      this.variableOverwrites = Object.fromEntries(Object.entries(this.variableOverwrites).filter(([, change]) => {
+        var _b;
+        return !(((_b = change.meta) === null || _b === void 0 ? void 0 : _b.experienceId) && experienceIdsToReset.includes(change.meta.experienceId));
+      }));
+      // 2. Clear experience variant index overwrites for these experiences (allows natural variant selection)
+      this.experienceVariantIndexOverwrites = Object.fromEntries(Object.entries(this.experienceVariantIndexOverwrites).filter(([key]) => !experienceIdsToReset.includes(key)));
+    }
+    // 3. Remove audience override (allows natural audience evaluation)
     const _b = this.audienceOverwrites,
       _c = id;
       _b[_c];
@@ -557,7 +533,7 @@ class NinetailedPreviewPlugin extends experience_jsPluginAnalytics.NinetailedPlu
   get pluginApi() {
     var _b;
     return {
-      version: "7.17.4" ,
+      version: "7.17.6" ,
       open: this.open.bind(this),
       close: this.close.bind(this),
       toggle: this.toggle.bind(this),
@@ -649,7 +625,7 @@ class NinetailedPreviewPlugin extends experience_jsPluginAnalytics.NinetailedPlu
     return `${experienceId}:${key}`;
   }
   /**
-   * Get effective changes by applying overrides - compute on demand
+   * Get effective changes by applying overwrites - compute on demand
    */
   getEffectiveChanges(inputChanges = this.changes) {
     if (!inputChanges || Object.keys(this.variableOverwrites).length === 0) {

package/index.esm.js

@@ -162,7 +162,7 @@ class NinetailedPreviewPlugin extends NinetailedPlugin {
      * Returns a middleware function that applies variable overwrites to changes
      */
     this.getChangesModificationMiddleware = () => {
-      if (!this.isActiveInstance || Object.keys(this.variableOverwrites).length === 0) {
+      if (!this.isActiveInstance || Object.keys(this.variableOverwrites).length === 0 && Object.keys(this.audienceOverwrites).length === 0) {
         return undefined;
       }
       return ({
@@ -173,10 +173,16 @@ class NinetailedPreviewPlugin extends NinetailedPlugin {
             changes: inputChanges
           };
         }
-
-        // Calculate and return overridden changes on demand instead of storing them
+        const filteredChanges = inputChanges.filter(change => {
+          var _change$meta, _experience$audience;
+          const experienceId = (_change$meta = change.meta) == null ? void 0 : _change$meta.experienceId;
+          if (!experienceId) return true;
+          const experience = this.experiences.find(e => e.id === experienceId);
+          if (!(experience != null && (_experience$audience = experience.audience) != null && _experience$audience.id)) return true;
+          return this.audienceOverwrites[experience.audience.id] !== false;
+        });
         return {
-          changes: this.getEffectiveChanges(inputChanges)
+          changes: this.getEffectiveChanges(filteredChanges)
         };
       };
     };
@@ -203,36 +209,8 @@ class NinetailedPreviewPlugin extends NinetailedPlugin {
         // Handle entry replacements as before
         const entryReplacementComponents = experience.components.filter(component => component.type === ComponentTypeEnum.EntryReplacement && 'id' in component.baseline);
         const baselineComponent = entryReplacementComponents.find(component => component.baseline.id === baseline.id);
-
-        // Get the selected variant index
         const variantIndex = this.pluginApi.experienceVariantIndexes[experience.id];
 
-        // Handle variable components for this experience (NEW CODE)
-        if (variantIndex !== undefined) {
-          // Process all variable components for this experience
-          const variableComponents = experience.components.filter(component => component.type === ComponentTypeEnum.InlineVariable);
-
-          // Set variable values based on the selected variant index
-          variableComponents.forEach(component => {
-            const key = component.key;
-            let value;
-            if (variantIndex === 0) {
-              value = component.baseline;
-            } else {
-              const variant = component.variants[variantIndex - 1];
-              value = variant && 'value' in variant ? variant.value : component.baseline;
-            }
-
-            // Set the variable in our changes system
-            this.setVariableValue({
-              experienceId: experience.id,
-              key,
-              value,
-              variantIndex
-            });
-          });
-        }
-
         // Continue with entry replacement handling
         if (!baselineComponent) {
           return {
@@ -363,8 +341,8 @@ class NinetailedPreviewPlugin extends NinetailedPlugin {
       [id]: true
     });
     this.experienceVariantIndexOverwrites = Object.assign({}, this.experienceVariantIndexOverwrites, this.experiences.filter(experience => {
-      var _experience$audience;
-      return ((_experience$audience = experience.audience) == null ? void 0 : _experience$audience.id) === id;
+      var _experience$audience2;
+      return ((_experience$audience2 = experience.audience) == null ? void 0 : _experience$audience2.id) === id;
     }).map(experience => experience.id).reduce((acc, curr) => {
       return Object.assign({}, acc, {
         [curr]: this.experienceVariantIndexes[curr] || 0
@@ -380,29 +358,6 @@ class NinetailedPreviewPlugin extends NinetailedPlugin {
       logger.warn(`You cannot deactivate an unknown audience (id: ${id}). How did you get it in the first place?`);
       return;
     }
-    this.experienceVariantIndexOverwrites = Object.entries(this.experienceVariantIndexOverwrites).filter(([key, _]) => {
-      return !this.experiences.filter(experience => {
-        var _experience$audience2;
-        return ((_experience$audience2 = experience.audience) == null ? void 0 : _experience$audience2.id) === id;
-      }).map(experience => experience.id).includes(key);
-    }).reduce((acc, [key, value]) => {
-      return Object.assign({}, acc, {
-        [key]: value
-      });
-    }, {});
-    this.audienceOverwrites = Object.assign({}, this.audienceOverwrites, {
-      [id]: false
-    });
-    this.onChange();
-    this.experiences.filter(experience => {
-      var _experience$audience3;
-      return ((_experience$audience3 = experience.audience) == null ? void 0 : _experience$audience3.id) === id;
-    }).forEach(experience => {
-      this.setExperienceVariant({
-        experienceId: experience.id,
-        variantIndex: 0
-      });
-    });
     this.audienceOverwrites = Object.assign({}, this.audienceOverwrites, {
       [id]: false
     });
@@ -416,6 +371,26 @@ class NinetailedPreviewPlugin extends NinetailedPlugin {
       logger.warn(`You cannot reset an unknown audience (id: ${id}). How did you get it in the first place?`);
       return;
     }
+
+    // Identify all experiences that belong to this audience
+    const experiencesToReset = this.experiences.filter(experience => {
+      var _experience$audience3;
+      return ((_experience$audience3 = experience.audience) == null ? void 0 : _experience$audience3.id) === id;
+    });
+    if (experiencesToReset.length > 0) {
+      const experienceIdsToReset = experiencesToReset.map(e => e.id);
+
+      // 1. Clear any variable overwrites that were set for these experiences (allows natural evaluation)
+      this.variableOverwrites = Object.fromEntries(Object.entries(this.variableOverwrites).filter(([, change]) => {
+        var _change$meta2;
+        return !((_change$meta2 = change.meta) != null && _change$meta2.experienceId && experienceIdsToReset.includes(change.meta.experienceId));
+      }));
+
+      // 2. Clear experience variant index overwrites for these experiences (allows natural variant selection)
+      this.experienceVariantIndexOverwrites = Object.fromEntries(Object.entries(this.experienceVariantIndexOverwrites).filter(([key]) => !experienceIdsToReset.includes(key)));
+    }
+
+    // 3. Remove audience override (allows natural audience evaluation)
     const _this$audienceOverwri = this.audienceOverwrites,
       audienceOverwrites = _objectWithoutPropertiesLoose(_this$audienceOverwri, [id].map(_toPropertyKey));
     this.audienceOverwrites = audienceOverwrites;
@@ -543,7 +518,7 @@ class NinetailedPreviewPlugin extends NinetailedPlugin {
   get pluginApi() {
     var _this$profile;
     return {
-      version: "7.17.4" ,
+      version: "7.17.6" ,
       open: this.open.bind(this),
       close: this.close.bind(this),
       toggle: this.toggle.bind(this),
@@ -637,7 +612,7 @@ class NinetailedPreviewPlugin extends NinetailedPlugin {
   }
 
   /**
-   * Get effective changes by applying overrides - compute on demand
+   * Get effective changes by applying overwrites - compute on demand
    */
   getEffectiveChanges(inputChanges = this.changes) {
     if (!inputChanges || Object.keys(this.variableOverwrites).length === 0) {
@@ -646,9 +621,9 @@ class NinetailedPreviewPlugin extends NinetailedPlugin {
 
     // Filter out changes that we're overriding
     const filteredChanges = inputChanges.filter(change => {
-      var _change$meta;
+      var _change$meta3;
       if (change.type !== ChangeTypes.Variable) return true;
-      const changeKey = (_change$meta = change.meta) != null && _change$meta.experienceId ? this.getOverrideKey(change.meta.experienceId, change.key) : change.key;
+      const changeKey = (_change$meta3 = change.meta) != null && _change$meta3.experienceId ? this.getOverrideKey(change.meta.experienceId, change.key) : change.key;
       return !this.variableOverwrites[changeKey];
     });
     const effectiveChanges = [...filteredChanges, ...Object.values(this.variableOverwrites)];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ninetailed/experience.js-plugin-preview",
-  "version": "7.17.4",
+  "version": "7.17.6",
   "description": "Ninetailed SDK plugin for preview",
   "license": "MIT",
   "repository": {
@@ -15,10 +15,10 @@
     "a/b testing"
   ],
   "dependencies": {
-    "@ninetailed/experience.js-shared": "7.17.4",
-    "@ninetailed/experience.js": "7.17.4",
-    "@ninetailed/experience.js-preview-bridge": "7.17.4",
-    "@ninetailed/experience.js-plugin-analytics": "7.17.4",
+    "@ninetailed/experience.js-shared": "7.17.6",
+    "@ninetailed/experience.js": "7.17.6",
+    "@ninetailed/experience.js-preview-bridge": "7.17.6",
+    "@ninetailed/experience.js-plugin-analytics": "7.17.6",
     "radash": "10.9.0"
   },
   "module": "./index.esm.js",

package/src/lib/plugin/NinetailedPreviewPlugin.d.ts

@@ -85,7 +85,7 @@ export declare class NinetailedPreviewPlugin extends NinetailedPlugin implements
      */
     private getOverrideKey;
     /**
-     * Get effective changes by applying overrides - compute on demand
+     * Get effective changes by applying overwrites - compute on demand
      */
     private getEffectiveChanges;
     private onChange;