expf-sigma-node.js 3.3.4 → 3.3.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "expf-sigma-node.js",
-  "version": "3.3.4",
+  "version": "3.3.5",
   "description": "expf-sigma-node.js lets you manage features flags and remote config across web, server side applications. Deliver true Continuous Integration. Get builds out faster. Control who has access to new features.",
   "main": "public/sigma.js",
   "keywords": [

package/public/sigma.js

@@ -1268,7 +1268,7 @@ function generateFilteredList(array, name) {
 }
 
 // package.json
-var version = "3.3.4";
+var version = "3.3.5";
 
 // src/helpers/sdkVersion.js
 var sdkVersion = version;
@@ -1663,7 +1663,7 @@ var Sigma = class {
             userValue,
             targets.conditions[index].value,
             targets.conditions[index].condition_sign,
-            this.sigmaUserData.user.data,
+            this.sigmaUserData.user.date,
             this.sigmaUserData.user.time
           )
         );
@@ -1876,7 +1876,7 @@ var Sigma = class {
               userValue,
               conditions[i].value,
               conditions[i].condition_sign,
-              this.sigmaUserData.user.data,
+              this.sigmaUserData.user.date,
               this.sigmaUserData.user.time
             )
           );
@@ -2110,25 +2110,12 @@ var Sigma = class {
         console.warn(`Sigma warn: Experiment ${experimentId} has not group with index ${groupIndex}`);
         return;
       }
-      if (splitById && experiment.forced_user_list.includes(String(this.userData[splitById]))) {
-        console.warn(`Sigma warn: User is already in experiment ${experimentId}. You need to exclude first then run method again`);
-        return;
-      }
-      if (splitById && this.userData[splitById] && !groupInExperiment.forced_user_list.includes(String(this.userData[splitById]))) {
+      this.clearForcedList(experiment, splitById);
+      if (this.userData[splitById] && !groupInExperiment.forced_user_list.includes(String(this.userData[splitById]))) {
         groupInExperiment.forced_user_list.push(String(this.userData[splitById]));
       }
     } else if (!groupIndex) {
-      let isUserInForcedGroup = false;
-      for (const group of experiment.groups) {
-        if (splitById && group.forced_user_list.includes(String(this.userData[splitById]))) {
-          isUserInForcedGroup = true;
-          break;
-        }
-      }
-      if (isUserInForcedGroup) {
-        console.warn(`Sigma warn: User is already group in experiment ${experimentId}. You need to exclude first then run method again`);
-        return;
-      }
+      this.clearForcedList(experiment, splitById);
       if (this.userData[splitById] && !experiment.forced_user_list.includes(String(this.userData[splitById]))) {
         experiment.forced_user_list.push(String(this.userData[splitById]));
       }
@@ -2136,68 +2123,64 @@ var Sigma = class {
     if (!this.cache.get(sigmaForcedExperiments)) {
       this.cache.set(sigmaForcedExperiments, JSON.stringify([{ experimentId, groupIndex }]));
     } else {
-      const forcedExperiments = JSON.parse(this.cache.get(sigmaForcedExperiments));
-      if (!forcedExperiments.find((el) => el.experimentId === experimentId)) {
+      const forcedExperiments = this.cache.parse(sigmaForcedExperiments);
+      const foundingForcedExperiment = forcedExperiments.find((el) => el.experimentId === experimentId);
+      if (!foundingForcedExperiment) {
         forcedExperiments.push({ experimentId, groupIndex });
         this.cache.set(sigmaForcedExperiments, JSON.stringify(forcedExperiments));
+      } else if (typeof groupIndex === "number" && groupIndex >= 0 && foundingForcedExperiment) {
+        foundingForcedExperiment.groupIndex = groupIndex;
+        this.cache.set(sigmaForcedExperiments, JSON.stringify(forcedExperiments));
+      } else if (!groupIndex && foundingForcedExperiment) {
+        foundingForcedExperiment.groupIndex = groupIndex;
+        this.cache.set(sigmaForcedExperiments, JSON.stringify(forcedExperiments));
       }
     }
     this.cache.set(sigmaDataFile, JSON.stringify(sigmaDataLs));
   }
-  excludeForceUser(experimentId) {
-    if (!experimentId || !this.cache.get(sigmaDataFile))
+  clearForcedList(experiment, splitById) {
+    if (experiment.forced_user_list.includes(String(this.userData[splitById]))) {
+      experiment.forced_user_list = experiment.forced_user_list.filter((el) => String(el) !== String(this.userData[splitById]));
+    }
+    for (const group of experiment.groups) {
+      if (group.forced_user_list.includes(String(this.userData[splitById]))) {
+        group.forced_user_list = group.forced_user_list.filter((el) => String(el) !== String(this.userData[splitById]));
+      }
+    }
+  }
+  async excludeForceUser(experimentId) {
+    if (!experimentId || !this.cache.get(sigmaDataFile) || !this.cache.get(sigmaForcedExperiments))
       return;
     const sigmaDataLs = this.cache.parse(sigmaDataFile);
     if (!sigmaDataLs || !sigmaDataLs.experiments)
       return;
     let experiment = sigmaDataLs.experiments.find((exp) => exp.name === experimentId);
-    if (!experiment)
+    if (!experiment) {
+      console.warn(`Sigma warn: Experiment ${experimentId} not found`);
       return;
-    if (!Object.keys(this.userData).includes(experiment.split_by))
-      return;
-    const splitById = Object.keys(this.userData).find((id) => id === experiment.split_by);
-    if (experiment.forced_user_list.length && experiment.forced_user_list.includes(String(this.userData[splitById]))) {
-      experiment.forced_user_list = experiment.forced_user_list.filter((el) => el !== String(this.userData[splitById]));
     }
-    const groups = experiment.groups;
-    for (const group of groups) {
-      if (group.forced_user_list.length && group.forced_user_list.includes(String(this.userData[splitById]))) {
-        group.forced_user_list = group.forced_user_list.filter((el) => el !== String(this.userData[splitById]));
-      }
+    if (!Object.keys(this.userData).includes(experiment.split_by)) {
+      console.warn(`Sigma warn: ${experiment.split_by} is missing in userData`);
+      return;
     }
-    if (this.cache.get(sigmaForcedExperiments)) {
-      let forcedExperiments = JSON.parse(this.cache.get(sigmaForcedExperiments));
-      forcedExperiments = forcedExperiments.filter((el) => el.experimentId !== experimentId);
-      if (forcedExperiments.length > 0) {
-        this.cache.set(sigmaForcedExperiments, JSON.stringify(forcedExperiments));
-      } else {
-        this.cache.remove(sigmaForcedExperiments);
-      }
+    let forcedExperiments = this.cache.parse(sigmaForcedExperiments);
+    if (!forcedExperiments.find((el) => el.experimentId === experimentId))
+      return;
+    forcedExperiments = forcedExperiments.filter((el) => el.experimentId !== experimentId);
+    if (forcedExperiments.length > 0) {
+      this.cache.set(sigmaForcedExperiments, JSON.stringify(forcedExperiments));
+    } else {
+      this.cache.remove(sigmaForcedExperiments);
     }
-    this.cache.set(sigmaDataFile, JSON.stringify(sigmaDataLs));
+    this.cache.set(sigmaHash, "");
+    await this.saveToCache();
   }
-  excludeForceUserAll() {
-    if (!this.cache.get(sigmaDataFile))
+  async excludeForceUserAll() {
+    if (!this.cache.get(sigmaDataFile) || !this.cache.get(sigmaForcedExperiments))
       return;
-    const sigmaDataLs = this.cache.parse(sigmaDataFile);
-    if (!sigmaDataLs || !sigmaDataLs.experiments)
-      return;
-    for (const experiment of sigmaDataLs.experiments) {
-      if (!Object.keys(this.userData).includes(experiment.split_by))
-        continue;
-      const splitById = Object.keys(this.userData).find((id) => id === experiment.split_by);
-      if (experiment.forced_user_list.length && experiment.forced_user_list.includes(String(this.userData[splitById]))) {
-        experiment.forced_user_list = experiment.forced_user_list.filter((el) => el !== String(this.userData[splitById]));
-      }
-      const groups = experiment.groups;
-      for (const group of groups) {
-        if (group.forced_user_list.length && group.forced_user_list.includes(String(this.userData[splitById]))) {
-          group.forced_user_list = group.forced_user_list.filter((el) => el !== String(this.userData[splitById]));
-        }
-      }
-    }
     this.cache.remove(sigmaForcedExperiments);
-    this.cache.set(sigmaDataFile, JSON.stringify(sigmaDataLs));
+    this.cache.set(sigmaHash, "");
+    await this.saveToCache();
   }
   checkForceExperiments() {
     if (!this.cache.get(sigmaForcedExperiments))