@ninetailed/experience.js-plugin-preview 7.17.5 → 7.18.0

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)
         };
       };
     };
@@ -231,29 +238,6 @@ class NinetailedPreviewPlugin extends experience_jsPluginAnalytics.NinetailedPlu
         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);
         const variantIndex = this.pluginApi.experienceVariantIndexes[experience.id];
-        // Natural evaluation should not create overwrites, letting the core SDK handle variable evaluation
-        if (variantIndex !== undefined && this.experienceVariantIndexOverwrites[experience.id] !== undefined) {
-          // Only set variable overwrites when there's an explicit preview override 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.value;
-            } else {
-              const variant = component.variants[variantIndex - 1];
-              value = variant && 'value' in variant ? variant.value : component.baseline.value;
-            }
-            // Set the variable in our changes system
-            this.setVariableValue({
-              experienceId: experience.id,
-              key,
-              value,
-              variantIndex
-            });
-          });
-        }
         // Continue with entry replacement handling
         if (!baselineComponent) {
           return {
@@ -397,28 +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;
     }
-    // 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;
-    });
-    // 1. Reset each experience to baseline BEFORE removing the audience so the guard in setExperienceVariant passes
-    experiencesToReset.forEach(experience => {
-      this.setExperienceVariant({
-        experienceId: experience.id,
-        variantIndex: 0
-      });
-    });
-    // 2. Remove experience variant index overwrites for these experiences
-    const experienceIdsToReset = experiencesToReset.map(e => e.id);
-    this.experienceVariantIndexOverwrites = Object.fromEntries(Object.entries(this.experienceVariantIndexOverwrites).filter(([key]) => !experienceIdsToReset.includes(key)));
-    // 3. Remove any variable overwrites that were set for these experiences (prevents being stuck on a variant value)
-    if (experienceIdsToReset.length > 0) {
-      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));
-      }));
-    }
     this.audienceOverwrites = Object.assign(Object.assign({}, this.audienceOverwrites), {
       [id]: false
     });
@@ -571,7 +533,7 @@ class NinetailedPreviewPlugin extends experience_jsPluginAnalytics.NinetailedPlu
   get pluginApi() {
     var _b;
     return {
-      version: "7.17.5" ,
+      version: "7.18.0" ,
       open: this.open.bind(this),
       close: this.close.bind(this),
       toggle: this.toggle.bind(this),
@@ -663,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)
         };
       };
     };
@@ -205,32 +211,6 @@ class NinetailedPreviewPlugin extends NinetailedPlugin {
         const baselineComponent = entryReplacementComponents.find(component => component.baseline.id === baseline.id);
         const variantIndex = this.pluginApi.experienceVariantIndexes[experience.id];
 
-        // Natural evaluation should not create overwrites, letting the core SDK handle variable evaluation
-        if (variantIndex !== undefined && this.experienceVariantIndexOverwrites[experience.id] !== undefined) {
-          // Only set variable overwrites when there's an explicit preview override 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.value;
-            } else {
-              const variant = component.variants[variantIndex - 1];
-              value = variant && 'value' in variant ? variant.value : component.baseline.value;
-            }
-
-            // Set the variable in our changes system
-            this.setVariableValue({
-              experienceId: experience.id,
-              key,
-              value,
-              variantIndex
-            });
-          });
-        }
-
         // Continue with entry replacement handling
         if (!baselineComponent) {
           return {
@@ -361,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
@@ -378,32 +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;
     }
-
-    // Identify all experiences that belong to this audience
-    const experiencesToReset = this.experiences.filter(experience => {
-      var _experience$audience2;
-      return ((_experience$audience2 = experience.audience) == null ? void 0 : _experience$audience2.id) === id;
-    });
-
-    // 1. Reset each experience to baseline BEFORE removing the audience so the guard in setExperienceVariant passes
-    experiencesToReset.forEach(experience => {
-      this.setExperienceVariant({
-        experienceId: experience.id,
-        variantIndex: 0
-      });
-    });
-
-    // 2. Remove experience variant index overwrites for these experiences
-    const experienceIdsToReset = experiencesToReset.map(e => e.id);
-    this.experienceVariantIndexOverwrites = Object.fromEntries(Object.entries(this.experienceVariantIndexOverwrites).filter(([key]) => !experienceIdsToReset.includes(key)));
-
-    // 3. Remove any variable overwrites that were set for these experiences (prevents being stuck on a variant value)
-    if (experienceIdsToReset.length > 0) {
-      this.variableOverwrites = Object.fromEntries(Object.entries(this.variableOverwrites).filter(([, change]) => {
-        var _change$meta;
-        return !((_change$meta = change.meta) != null && _change$meta.experienceId && experienceIdsToReset.includes(change.meta.experienceId));
-      }));
-    }
     this.audienceOverwrites = Object.assign({}, this.audienceOverwrites, {
       [id]: false
     });
@@ -564,7 +518,7 @@ class NinetailedPreviewPlugin extends NinetailedPlugin {
   get pluginApi() {
     var _this$profile;
     return {
-      version: "7.17.5" ,
+      version: "7.18.0" ,
       open: this.open.bind(this),
       close: this.close.bind(this),
       toggle: this.toggle.bind(this),
@@ -658,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) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ninetailed/experience.js-plugin-preview",
-  "version": "7.17.5",
+  "version": "7.18.0",
   "description": "Ninetailed SDK plugin for preview",
   "license": "MIT",
   "repository": {
@@ -14,13 +14,24 @@
     "personalization",
     "a/b testing"
   ],
+  "main": "./index.cjs.js",
+  "module": "./index.esm.js",
+  "types": "./index.cjs.d.ts",
+  "exports": {
+    ".": {
+      "types": "./index.cjs.d.ts",
+      "import": "./index.esm.js",
+      "require": "./index.cjs.js",
+      "default": "./index.esm.js"
+    },
+    "./package.json": "./package.json"
+  },
+  "sideEffects": false,
   "dependencies": {
-    "@ninetailed/experience.js-shared": "7.17.5",
-    "@ninetailed/experience.js": "7.17.5",
-    "@ninetailed/experience.js-preview-bridge": "7.17.5",
-    "@ninetailed/experience.js-plugin-analytics": "7.17.5",
+    "@ninetailed/experience.js-shared": "7.18.0",
+    "@ninetailed/experience.js": "7.18.0",
+    "@ninetailed/experience.js-preview-bridge": "7.18.0",
+    "@ninetailed/experience.js-plugin-analytics": "7.18.0",
     "radash": "10.9.0"
-  },
-  "module": "./index.esm.js",
-  "main": "./index.cjs.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;