expf-sigma-node.js 0.1.0 → 0.1.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "expf-sigma-node.js",
-  "version": "0.1.0",
+  "version": "0.1.2",
   "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": [
@@ -17,7 +17,10 @@
   "scripts": {
     "release": "np",
     "build": "esbuild --bundle ./src/sigma.js --platform=node --format=cjs --outdir=public",
-    "dev": "esbuild --bundle ./src/sigma.js --platform=node --format=cjs --outdir=public --watch"
+    "dev": "esbuild --bundle ./src/sigma.js --platform=node --format=cjs --outdir=public --watch",
+    "lint": "eslint ./src/sigma.js",
+    "lint:all": "eslint \"**/*.js\"",
+    "test": "jest"
   },
   "dependencies": {
     "node-fetch": "^2.1.2"
@@ -27,6 +30,7 @@
     "@babel/plugin-proposal-class-properties": "^7.18.6",
     "@babel/preset-env": "^7.16.4",
     "babel-eslint": "^10.1.0",
+    "babel-jest": "^29.2.2",
     "babel-loader": "^8.2.3",
     "babel-plugin-add-module-exports": "^1.0.4",
     "clean-webpack-plugin": "^4.0.0",
@@ -36,6 +40,8 @@
     "eslint-plugin-jest": "^23.8.2",
     "eslint-plugin-node": "^11.1.0",
     "eslint-plugin-prettier": "^3.1.3",
+    "jest": "^29.2.2",
+    "jest-environment-jsdom": "^29.2.1",
     "js-cookie": "^3.0.1",
     "jshashes": "^1.0.8",
     "node-cache": "^5.1.2",

package/public/sigma.js

@@ -5620,11 +5620,14 @@ var SigmaUserData = class {
   getUserNavigator() {
     const userNavigator = browser_detect_es5_default();
     const { user } = this;
-    const userOs = userNavigator.os.split(" ");
-    user.browser.name = userNavigator.name;
-    user.browser.version = userNavigator.version;
-    user.os.name = userOs[0];
-    user.os.version = userOs[1];
+    let userOs;
+    if (userNavigator.os) {
+      userOs = userNavigator.os.split(" ");
+    }
+    user.browser.name = userNavigator.name ? userNavigator.name : void 0;
+    user.browser.version = userNavigator.version ? userNavigator.version : void 0;
+    user.os.name = userOs ? userOs[0] : void 0;
+    user.os.version = userOs ? userOs[1] : void 0;
   }
   setUserId(userId) {
     this.user.userId = userId;
@@ -5980,6 +5983,7 @@ var Sigma = class {
   async getUserGeoData() {
     try {
       const data = await this.getDataFile(`${defaultApi}/geo`);
+      this.sigmaUserData.setIp(data.ip);
       return data;
     } catch (error) {
       throw new Error(error);
@@ -6076,7 +6080,7 @@ var Sigma = class {
   }
   async checkFlag(flagName) {
     if (!flagName) {
-      return false;
+      return null;
     }
     await this.updateCache();
     const cacheKey = this.cache.parse("sigmaDataFile");
@@ -6092,7 +6096,7 @@ var Sigma = class {
       }
     }
     if (!flag)
-      return false;
+      return null;
     return this.findingSpotCondition(flag, this.sigmaUserData);
   }
   getFlag(key) {
@@ -6143,14 +6147,14 @@ var Sigma = class {
         return !conditionValues.includes(userValue);
       case "contains any of":
         for (let i = 0; i < conditionValues.length; i++) {
-          if (String(conditionValues[i]).includes(userValue)) {
+          if (String(userValue).includes(conditionValues[i])) {
             return true;
           }
         }
         return false;
       case "contains none of":
         for (let i = 0; i < conditionValues.length; i++) {
-          if (String(conditionValues[i]).includes(userValue)) {
+          if (String(userValue).includes(conditionValues[i])) {
             return false;
           }
         }
@@ -6228,7 +6232,7 @@ var Sigma = class {
   }
   async getExperiment(experimentName) {
     if (!experimentName) {
-      return false;
+      return null;
     }
     await this.updateCache();
     const sigmaDataLs = await this.cache.parse(sigmaDataFile);
@@ -6268,7 +6272,7 @@ var Sigma = class {
     }
     const getParamValue = (paramName) => {
       if (!paramName || !experiment) {
-        return false;
+        return null;
       }
       const groupName = this.cache.get(localStorageGroupName);
       let params = null;
@@ -6279,7 +6283,7 @@ var Sigma = class {
         }
       }
       if (!params) {
-        return false;
+        return null;
       }
       let paramValue = null;
       for (let item in params.values) {
@@ -6288,13 +6292,13 @@ var Sigma = class {
         }
       }
       if (!paramValue) {
-        return false;
+        return null;
       }
       return paramValue;
     };
     const getFeatureValue = (flagName) => {
       if (!flagName || !experiment) {
-        return false;
+        return null;
       }
       const groupName = this.cache.get(localStorageGroupName);
       let flagExperiment = null;
@@ -6305,7 +6309,7 @@ var Sigma = class {
         }
       }
       if (!flagExperiment)
-        return false;
+        return null;
       let flag = null;
       for (let i = 0; i < flagExperiment.groups.length; i++) {
         if (flagExperiment.groups[i]["group"] === groupName) {
@@ -6314,8 +6318,8 @@ var Sigma = class {
         }
       }
       if (!flag)
-        return false;
-      return this.findingSpotCondition(flag);
+        return null;
+      return this.findingSpotCondition(flag, false, experiment);
     };
     return {
       getParamValue,
@@ -6372,7 +6376,7 @@ var Sigma = class {
     }
     return groupName;
   }
-  findingSpotCondition(flag, saveToUser) {
+  findingSpotCondition(flag, saveToUser = false, exp = null) {
     let flagRules = flag.rules;
     let flagDefaultResult = null;
     for (let defaultRule in flagRules) {
@@ -6397,7 +6401,10 @@ var Sigma = class {
         const conditions = flagRules[rule].conditions;
         let results = [];
         for (let i in conditions) {
-          let userValue = this.getUserParamValue(conditions[i].name, flag.is_private);
+          let userValue = this.getUserParamValue(
+            conditions[i].name,
+            exp ? exp.is_private : flag.is_private
+          );
           results.push(
             this.conditionOperation(
               userValue,