@terreno/feature-flags 57.8.0 → 57.9.0

package/dist/featureFlagsApp.js

@@ -89,9 +89,14 @@ class FeatureFlagsApp {
                 throw new api_1.APIError({ status: 401, title: "Authentication required" });
             }
             const targetingKey = String(user._id ?? user.id);
-            const flags = await featureFlagModel_1.FeatureFlag.find({ archived: { $ne: true }, enabled: true });
+            const flags = await featureFlagModel_1.FeatureFlag.find({ archived: { $ne: true } });
             const config = {};
             for (const flag of flags) {
+                // Disabled boolean flags are sent as `off` so clients can tell a turned-off
+                // flag from one that was never created. Disabled variant flags stay omitted.
+                if (!flag.enabled && flag.type !== "boolean") {
+                    continue;
+                }
                 config[flag.key] = (0, flagConfiguration_1.buildFlagDefinition)(flag, targetingKey, user, this.segments);
             }
             const body = { data: config };

package/package.json

@@ -5,14 +5,14 @@
   },
   "dependencies": {
     "@openfeature/server-sdk": "^1.22.0",
-    "@terreno/api": "57.8.0",
+    "@terreno/api": "57.9.0",
     "express": "^5.2.1",
     "lodash": "^4.18.1"
   },
   "description": "Feature flags and A/B testing plugin for @terreno/api — part of Terreno, the batteries-included TypeScript framework for universal apps.",
   "devDependencies": {
     "@biomejs/biome": "^2.3.6",
-    "@terreno/test": "57.8.0",
+    "@terreno/test": "57.9.0",
     "@types/bun": "^1.4.0",
     "@types/express": "^5.0.6",
     "@types/lodash": "^4.17.17",
@@ -53,5 +53,5 @@
     "test:coverage": "bun run ../scripts/check-coverage.ts"
   },
   "types": "dist/index.d.ts",
-  "version": "57.8.0"
+  "version": "57.9.0"
 }

package/src/featureFlagsApp.ts

@@ -123,10 +123,15 @@ export class FeatureFlagsApp implements TerrenoPlugin {
         }
 
         const targetingKey = String(user._id ?? user.id);
-        const flags = await FeatureFlag.find({archived: {$ne: true}, enabled: true});
+        const flags = await FeatureFlag.find({archived: {$ne: true}});
         const config: Record<string, FlagDefinition> = {};
 
         for (const flag of flags) {
+          // Disabled boolean flags are sent as `off` so clients can tell a turned-off
+          // flag from one that was never created. Disabled variant flags stay omitted.
+          if (!flag.enabled && flag.type !== "boolean") {
+            continue;
+          }
           config[flag.key] = buildFlagDefinition(flag, targetingKey, user, this.segments);
         }
 

package/src/tests/featureFlagsApp.test.ts

@@ -230,7 +230,7 @@ describe("FeatureFlagsApp", () => {
       await unauth.get("/feature-flags/flagConfiguration").expect(401);
     });
 
-    it("returns OpenFeature-shaped entries for enabled flags and omits disabled and archived", async () => {
+    it("returns OpenFeature-shaped entries and resolves disabled boolean flags to off", async () => {
       await FeatureFlag.create({
         enabled: true,
         key: "bool-active",
@@ -281,7 +281,11 @@ describe("FeatureFlagsApp", () => {
       expect(data.profile?.disabled).toBe(false);
       expect(["compact", "detailed"].includes(data.profile?.defaultVariant ?? "")).toBe(true);
       expect(data.profile?.variants.compact).toBe("compact");
-      expect(data["disabled-one"]).toBeUndefined();
+      expect(data["disabled-one"]).toEqual({
+        defaultVariant: "off",
+        disabled: false,
+        variants: {off: false, on: true},
+      });
       expect(data["archived-one"]).toBeUndefined();
     });