chore(flag-evaluation): bump version to 0.1.5 and improve operator evaluations

- Updated version in package.json to 0.1.5.
- Refined evaluation logic for "SET" and "NOT_SET" operators to use strict equality checks.
- Added tests to ensure correct behavior for missing context fields with "SET" and "NOT_SET" operators, including handling empty string values.

Author: pavkam

packages/flag-evaluation/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bucketco/flag-evaluation",
-  "version": "0.1.4",
+  "version": "0.1.5",
   "license": "MIT",
   "repository": {
     "type": "git",

packages/flag-evaluation/src/index.ts

@@ -325,9 +325,9 @@ export function evaluate(
         : fieldValueDate < daysAgo.getTime();
     }
     case "SET":
-      return fieldValue != "";
+      return fieldValue !== "";
     case "NOT_SET":
-      return fieldValue == "";
+      return fieldValue === "";
     case "IS":
       return fieldValue === value;
     case "IS_NOT":
@@ -357,13 +357,17 @@ function evaluateRecursively(
     case "constant":
       return filter.value;
     case "context":
-      if (!(filter.field in context)) {
+      if (
+        !(filter.field in context) &&
+        filter.operator !== "SET" &&
+        filter.operator !== "NOT_SET"
+      ) {
         missingContextFieldsSet.add(filter.field);
         return false;
       }
 
       return evaluate(
-        context[filter.field],
+        context[filter.field] ?? "",
         filter.operator,
         filter.values || [],
         filter.valueSet,

packages/flag-evaluation/test/index.test.ts

@@ -311,6 +311,126 @@ describe("evaluate feature targeting integration ", () => {
     });
   });
 
+  it("should not report missing context fields for `SET` operator when field doesn't exist", () => {
+    const res = evaluateFeatureRules({
+      featureKey: "test_feature",
+      rules: [
+        {
+          value: true,
+          filter: {
+            type: "context",
+            field: "user.name",
+            operator: "SET",
+            values: [],
+          },
+        },
+      ],
+      context: {},
+    });
+
+    expect(res).toEqual({
+      featureKey: "test_feature",
+      value: undefined,
+      context: {},
+      ruleEvaluationResults: [false],
+      reason: "no matched rules",
+      missingContextFields: [],
+    });
+  });
+
+  it("should not report missing context fields for `NOT_SET` operator when field doesn't exist", () => {
+    const res = evaluateFeatureRules({
+      featureKey: "test_feature",
+      rules: [
+        {
+          value: true,
+          filter: {
+            type: "context",
+            field: "user.name",
+            operator: "NOT_SET",
+            values: [],
+          },
+        },
+      ],
+      context: {},
+    });
+
+    expect(res).toEqual({
+      featureKey: "test_feature",
+      value: true,
+      context: {},
+      ruleEvaluationResults: [true],
+      reason: "rule #0 matched",
+      missingContextFields: [],
+    });
+  });
+
+  it("should handle `SET` operator with empty string field value", () => {
+    const res = evaluateFeatureRules({
+      featureKey: "test_feature",
+      rules: [
+        {
+          value: true,
+          filter: {
+            type: "context",
+            field: "user.name",
+            operator: "SET",
+            values: [],
+          },
+        },
+      ],
+      context: {
+        user: {
+          name: "",
+        },
+      },
+    });
+
+    expect(res).toEqual({
+      featureKey: "test_feature",
+      value: undefined,
+      context: {
+        "user.name": "",
+      },
+      ruleEvaluationResults: [false],
+      reason: "no matched rules",
+      missingContextFields: [],
+    });
+  });
+
+  it("should handle `NOT_SET` operator with empty string field value", () => {
+    const res = evaluateFeatureRules({
+      featureKey: "test_feature",
+      rules: [
+        {
+          value: true,
+          filter: {
+            type: "context",
+            field: "user.name",
+            operator: "NOT_SET",
+            values: [],
+          },
+        },
+      ],
+      context: {
+        user: {
+          name: "",
+        },
+      },
+    });
+
+    expect(res).toEqual({
+      featureKey: "test_feature",
+      value: true,
+      context: {
+        "user.name": "",
+      },
+      ruleEvaluationResults: [true],
+      reason: "rule #0 matched",
+      missingContextFields: [],
+    });
+  });
+
   it.each([
     {
       context: { "company.id": "company1" },
@@ -376,6 +496,8 @@ describe("operator evaluation", () => {
 
     ["value", "SET", "", true],
     ["", "SET", "", false],
+    ["value", "NOT_SET", "", false],
+    ["", "NOT_SET", "", true],
 
     // non numeric values should return false
     ["value", "GT", "value", false],