@tehw0lf/yaft 0.0.16 → 0.0.17

package/README.md

@@ -129,6 +129,19 @@ rules can be applied directly to a feature without going through a provider.
 A missing feature is off. Unset, `null` or unparseable dates are ignored rather
 than treated as an error, and never throw.
 
+### Boolean shape
+
+The boolean providers hold `{ "myToggle": true }` and have no time logic. Only
+the JSON boolean `true` is on. A JSON `false` is kept in the data and reads as
+off. Any value that is not a boolean -- `"true"`, `"false"`, `1`, `null` -- is
+dropped when the data loads, so its key is missing from the data and reads as
+off. `normaliseBooleans` is exported and applies the same rule.
+
+**Changed in 0.0.17.** Before, `isEnabled` returned whatever was stored, and a
+caller testing it with `if` turned the string `"false"` **on**, because a
+non-empty string is truthy. A configuration that relied on string or number
+values has to switch to real booleans.
+
 ### Date format
 
 Dates must be **RFC 3339 with an offset** (`2026-09-18T15:00:00Z` or
@@ -166,8 +179,8 @@ is tested against that suite rather than only against its own expectations.
 The version is pinned in `conformance.lock`:
 
 ```
-version=v1.1.0
-sha256=d83ff1c960ad29830c00b57727591da628f4323b777faea245f603565d1c9ae9
+version=v2.0.0
+sha256=a10c1342e9ef346f998c0a819d90ae7f4984b53c933ed555eefab09419e993aa
 ```
 
 `npm test` fetches that release, verifies the checksum and unpacks it before

package/examples/ApiServiceBooleanProvider.js

@@ -37,7 +37,7 @@ class ApiServiceBooleanProvider {
             // through the core normaliser, so the two providers cannot disagree
             // about what a response means.
             if (isKeyedBooleans(response.data)) {
-                this.data = response.data;
+                this.data = (0, mapping_1.normaliseBooleans)(response.data);
                 return;
             }
             // Only the value matters here. The boolean shape has no time logic by
@@ -59,24 +59,27 @@ class ApiServiceBooleanProvider {
         }
     }
     isEnabled(key) {
-        const feature = this.data[key];
-        if (feature === undefined || feature === null)
-            return false;
-        return feature;
+        return this.data[key] === true;
     }
 }
 exports.ApiServiceBooleanProvider = ApiServiceBooleanProvider;
 /**
- * True when the payload is already `{ "myToggle": true }`.
+ * True when the payload is in the boolean shape, `{ "myToggle": true }`.
  *
  * Distinguishing this from a feature-shaped response matters: running a keyed
  * boolean object through the feature normaliser would look for a `key` field,
  * find none and discard every entry.
+ *
+ * One boolean value is enough. A mixed payload such as
+ * `{ "a": true, "b": "x" }` is still the boolean shape; normaliseBooleans then
+ * drops `b` and keeps `a` (R29), exactly as the local provider does. Requiring
+ * every value to be boolean sent it to the feature normaliser instead, which
+ * lost `a` as well. Feature responses never carry a boolean -- `value` is a
+ * string -- so they are not caught by this.
  */
 function isKeyedBooleans(data) {
     if (data === null || typeof data !== 'object' || Array.isArray(data)) {
         return false;
     }
-    const values = Object.values(data);
-    return values.length > 0 && values.every((v) => typeof v === 'boolean');
+    return Object.values(data).some((v) => typeof v === 'boolean');
 }

package/examples/LocalStorageBooleanProvider.js

@@ -1,6 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.LocalStorageBooleanProvider = void 0;
+const mapping_1 = require("../mapping");
 class LocalStorageBooleanProvider {
     data = {};
     constructor(configPath) {
@@ -8,8 +9,7 @@ class LocalStorageBooleanProvider {
     }
     getConfig(configPathOrUrl) {
         try {
-            const configData = require(configPathOrUrl);
-            this.data = configData;
+            this.data = (0, mapping_1.normaliseBooleans)(require(configPathOrUrl));
         }
         catch (error) {
             console.error("Failed to load configuration from local file:", error);
@@ -17,10 +17,7 @@ class LocalStorageBooleanProvider {
         }
     }
     isEnabled(key) {
-        const feature = this.data[key];
-        if (feature === undefined || feature === null)
-            return false;
-        return feature;
+        return this.data[key] === true;
     }
 }
 exports.LocalStorageBooleanProvider = LocalStorageBooleanProvider;

package/index.d.ts

@@ -1,3 +1,3 @@
 export { Feature, FeatureToggle, FeatureToggleBase, FeatureProvider, } from "./FeatureToggle";
 export { Clock, evaluate, parseTimestamp, systemClock } from "./evaluate";
-export { normaliseCollection, normaliseFeature } from "./mapping";
+export { normaliseBooleans, normaliseCollection, normaliseFeature } from "./mapping";

package/index.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.normaliseFeature = exports.normaliseCollection = exports.systemClock = exports.parseTimestamp = exports.evaluate = exports.FeatureToggleBase = exports.FeatureToggle = void 0;
+exports.normaliseFeature = exports.normaliseCollection = exports.normaliseBooleans = exports.systemClock = exports.parseTimestamp = exports.evaluate = exports.FeatureToggleBase = exports.FeatureToggle = void 0;
 var FeatureToggle_1 = require("./FeatureToggle");
 Object.defineProperty(exports, "FeatureToggle", { enumerable: true, get: function () { return FeatureToggle_1.FeatureToggle; } });
 Object.defineProperty(exports, "FeatureToggleBase", { enumerable: true, get: function () { return FeatureToggle_1.FeatureToggleBase; } });
@@ -9,5 +9,6 @@ Object.defineProperty(exports, "evaluate", { enumerable: true, get: function ()
 Object.defineProperty(exports, "parseTimestamp", { enumerable: true, get: function () { return evaluate_1.parseTimestamp; } });
 Object.defineProperty(exports, "systemClock", { enumerable: true, get: function () { return evaluate_1.systemClock; } });
 var mapping_1 = require("./mapping");
+Object.defineProperty(exports, "normaliseBooleans", { enumerable: true, get: function () { return mapping_1.normaliseBooleans; } });
 Object.defineProperty(exports, "normaliseCollection", { enumerable: true, get: function () { return mapping_1.normaliseCollection; } });
 Object.defineProperty(exports, "normaliseFeature", { enumerable: true, get: function () { return mapping_1.normaliseFeature; } });

package/mapping.d.ts

@@ -24,4 +24,12 @@ export declare function normaliseFeature(raw: RawFeature): Feature;
  * where `isEnabled("")` could reach it.
  */
 export declare function normaliseCollection(response: unknown): Record<string, Feature>;
+/**
+ * Normalises a boolean-shape payload, `{ "myToggle": true }`.
+ *
+ * Only real booleans are kept (R29). Anything else is dropped, so its key
+ * reads as missing and therefore off. Keeping it and testing its truthiness
+ * would turn `"false"` on, since a non-empty string is truthy.
+ */
+export declare function normaliseBooleans(response: unknown): Record<string, boolean>;
 export {};

package/mapping.js

@@ -2,6 +2,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.normaliseFeature = normaliseFeature;
 exports.normaliseCollection = normaliseCollection;
+exports.normaliseBooleans = normaliseBooleans;
 /**
  * Reads a field by presence, not by truthiness.
  *
@@ -77,3 +78,21 @@ function normaliseCollection(response) {
     }
     return data;
 }
+/**
+ * Normalises a boolean-shape payload, `{ "myToggle": true }`.
+ *
+ * Only real booleans are kept (R29). Anything else is dropped, so its key
+ * reads as missing and therefore off. Keeping it and testing its truthiness
+ * would turn `"false"` on, since a non-empty string is truthy.
+ */
+function normaliseBooleans(response) {
+    if (response === null || typeof response !== 'object' || Array.isArray(response)) {
+        return {};
+    }
+    const data = {};
+    for (const [key, value] of Object.entries(response)) {
+        if (typeof value === 'boolean')
+            data[key] = value;
+    }
+    return data;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tehw0lf/yaft",
-  "version": "0.0.16",
+  "version": "0.0.17",
   "description": "YaFT - Feature Toggles using Go&PostgreSQL or any source!",
   "type": "commonjs",
   "main": "index.js",