flagsmith-nodejs 4.0.0 → 5.0.1

package/build/cjs/index.d.ts

@@ -1,3 +1,4 @@
 export { AnalyticsProcessor, FlagsmithAPIError, FlagsmithClientError, EnvironmentDataPollingManager, FlagsmithCache, DefaultFlag, Flags, Flagsmith, } from './sdk/index.js';
+export { BaseOfflineHandler, LocalFileHandler, } from './sdk/offline_handlers.js';
 export { FlagsmithConfig } from './sdk/types.js';
 export { EnvironmentModel, FeatureStateModel, IdentityModel, TraitModel, SegmentModel, OrganisationModel } from './flagsmith-engine/index.js';

package/build/cjs/index.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.OrganisationModel = exports.SegmentModel = exports.TraitModel = exports.IdentityModel = exports.FeatureStateModel = exports.EnvironmentModel = exports.Flagsmith = exports.Flags = exports.DefaultFlag = exports.EnvironmentDataPollingManager = exports.FlagsmithClientError = exports.FlagsmithAPIError = exports.AnalyticsProcessor = void 0;
+exports.OrganisationModel = exports.SegmentModel = exports.TraitModel = exports.IdentityModel = exports.FeatureStateModel = exports.EnvironmentModel = exports.LocalFileHandler = exports.BaseOfflineHandler = exports.Flagsmith = exports.Flags = exports.DefaultFlag = exports.EnvironmentDataPollingManager = exports.FlagsmithClientError = exports.FlagsmithAPIError = exports.AnalyticsProcessor = void 0;
 var index_js_1 = require("./sdk/index.js");
 Object.defineProperty(exports, "AnalyticsProcessor", { enumerable: true, get: function () { return index_js_1.AnalyticsProcessor; } });
 Object.defineProperty(exports, "FlagsmithAPIError", { enumerable: true, get: function () { return index_js_1.FlagsmithAPIError; } });
@@ -9,6 +9,9 @@ Object.defineProperty(exports, "EnvironmentDataPollingManager", { enumerable: tr
 Object.defineProperty(exports, "DefaultFlag", { enumerable: true, get: function () { return index_js_1.DefaultFlag; } });
 Object.defineProperty(exports, "Flags", { enumerable: true, get: function () { return index_js_1.Flags; } });
 Object.defineProperty(exports, "Flagsmith", { enumerable: true, get: function () { return index_js_1.Flagsmith; } });
+var offline_handlers_js_1 = require("./sdk/offline_handlers.js");
+Object.defineProperty(exports, "BaseOfflineHandler", { enumerable: true, get: function () { return offline_handlers_js_1.BaseOfflineHandler; } });
+Object.defineProperty(exports, "LocalFileHandler", { enumerable: true, get: function () { return offline_handlers_js_1.LocalFileHandler; } });
 var index_js_2 = require("./flagsmith-engine/index.js");
 Object.defineProperty(exports, "EnvironmentModel", { enumerable: true, get: function () { return index_js_2.EnvironmentModel; } });
 Object.defineProperty(exports, "FeatureStateModel", { enumerable: true, get: function () { return index_js_2.FeatureStateModel; } });

package/build/cjs/sdk/index.js

@@ -117,10 +117,6 @@ class Flagsmith {
             this.environment = this.offlineHandler.getEnvironment();
         }
         if (!!data.cache) {
-            const missingMethods = ['has', 'get', 'set'].filter(method => data.cache && !data.cache[method]);
-            if (missingMethods.length > 0) {
-                throw new Error(`Please implement the following methods in your cache: ${missingMethods.join(', ')}`);
-            }
             this.cache = data.cache;
         }
         if (!this.offlineMode) {

package/build/cjs/sdk/models.js

@@ -37,7 +37,7 @@ class Flag extends BaseFlag {
     static fromAPIFlag(flagData) {
         return new Flag({
             enabled: flagData['enabled'],
-            value: flagData['feature_state_value'] || flagData['value'],
+            value: flagData['feature_state_value'] ?? flagData['value'],
             featureId: flagData['feature']['id'],
             featureName: flagData['feature']['name']
         });

package/build/cjs/sdk/types.d.ts

@@ -4,11 +4,21 @@ import { Dispatcher } from 'undici-types';
 import { Logger } from 'pino';
 import { BaseOfflineHandler } from './offline_handlers.js';
 export type IFlagsmithValue<T = string | number | boolean | null> = T;
+/**
+ * Stores and retrieves {@link Flags} from a cache.
+ */
 export interface FlagsmithCache {
-    get(key: string): Promise<Flags | undefined> | undefined;
-    set(key: string, value: Flags, ttl?: string | number): boolean | Promise<boolean>;
-    has(key: string): boolean | Promise<boolean>;
-    [key: string]: any;
+    /**
+     * Retrieve the cached {@link Flags} for the given environment or identity, or `undefined` if no cached value exists.
+     * @param key An environment ID or identity identifier, which is used as the cache key.
+     */
+    get(key: string): Promise<Flags | undefined>;
+    /**
+     * Persist an environment or identity's {@link Flags} in the cache.
+     * @param key An environment ID or identity identifier, which is used as the cache key.
+     * @param value The {@link Flags} to be stored in the cache.
+     */
+    set(key: string, value: Flags): Promise<void>;
 }
 export type Fetch = typeof fetch;
 export interface FlagsmithConfig {

package/build/esm/index.d.ts

@@ -1,3 +1,4 @@
 export { AnalyticsProcessor, FlagsmithAPIError, FlagsmithClientError, EnvironmentDataPollingManager, FlagsmithCache, DefaultFlag, Flags, Flagsmith, } from './sdk/index.js';
+export { BaseOfflineHandler, LocalFileHandler, } from './sdk/offline_handlers.js';
 export { FlagsmithConfig } from './sdk/types.js';
 export { EnvironmentModel, FeatureStateModel, IdentityModel, TraitModel, SegmentModel, OrganisationModel } from './flagsmith-engine/index.js';

package/build/esm/index.js

@@ -1,2 +1,3 @@
 export { AnalyticsProcessor, FlagsmithAPIError, FlagsmithClientError, EnvironmentDataPollingManager, DefaultFlag, Flags, Flagsmith, } from './sdk/index.js';
+export { BaseOfflineHandler, LocalFileHandler, } from './sdk/offline_handlers.js';
 export { EnvironmentModel, FeatureStateModel, IdentityModel, TraitModel, SegmentModel, OrganisationModel } from './flagsmith-engine/index.js';

package/build/esm/sdk/index.js

@@ -108,10 +108,6 @@ export class Flagsmith {
             this.environment = this.offlineHandler.getEnvironment();
         }
         if (!!data.cache) {
-            const missingMethods = ['has', 'get', 'set'].filter(method => data.cache && !data.cache[method]);
-            if (missingMethods.length > 0) {
-                throw new Error(`Please implement the following methods in your cache: ${missingMethods.join(', ')}`);
-            }
             this.cache = data.cache;
         }
         if (!this.offlineMode) {

package/build/esm/sdk/models.js

@@ -32,7 +32,7 @@ export class Flag extends BaseFlag {
     static fromAPIFlag(flagData) {
         return new Flag({
             enabled: flagData['enabled'],
-            value: flagData['feature_state_value'] || flagData['value'],
+            value: flagData['feature_state_value'] ?? flagData['value'],
             featureId: flagData['feature']['id'],
             featureName: flagData['feature']['name']
         });

package/build/esm/sdk/types.d.ts

@@ -4,11 +4,21 @@ import { Dispatcher } from 'undici-types';
 import { Logger } from 'pino';
 import { BaseOfflineHandler } from './offline_handlers.js';
 export type IFlagsmithValue<T = string | number | boolean | null> = T;
+/**
+ * Stores and retrieves {@link Flags} from a cache.
+ */
 export interface FlagsmithCache {
-    get(key: string): Promise<Flags | undefined> | undefined;
-    set(key: string, value: Flags, ttl?: string | number): boolean | Promise<boolean>;
-    has(key: string): boolean | Promise<boolean>;
-    [key: string]: any;
+    /**
+     * Retrieve the cached {@link Flags} for the given environment or identity, or `undefined` if no cached value exists.
+     * @param key An environment ID or identity identifier, which is used as the cache key.
+     */
+    get(key: string): Promise<Flags | undefined>;
+    /**
+     * Persist an environment or identity's {@link Flags} in the cache.
+     * @param key An environment ID or identity identifier, which is used as the cache key.
+     * @param value The {@link Flags} to be stored in the cache.
+     */
+    set(key: string, value: Flags): Promise<void>;
 }
 export type Fetch = typeof fetch;
 export interface FlagsmithConfig {

package/index.ts

@@ -9,6 +9,11 @@ export {
   Flagsmith,
 } from './sdk/index.js';
 
+export {
+  BaseOfflineHandler,
+  LocalFileHandler,
+} from './sdk/offline_handlers.js';
+
 export {
   FlagsmithConfig
 } from './sdk/types.js'

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "flagsmith-nodejs",
-  "version": "4.0.0",
+  "version": "5.0.1",
   "description": "Flagsmith lets you manage features flags and remote config across web, mobile and server side applications. Deliver true Continuous Integration. Get builds out faster. Control who has access to new features.",
   "main": "./build/cjs/index.js",
   "type": "module",

package/sdk/index.ts

@@ -128,17 +128,6 @@ export class Flagsmith {
         }
 
         if (!!data.cache) {
-            const missingMethods: string[] = ['has', 'get', 'set'].filter(
-                method => data.cache && !data.cache[method]
-            );
-
-            if (missingMethods.length > 0) {
-                throw new Error(
-                    `Please implement the following methods in your cache: ${missingMethods.join(
-                        ', '
-                    )}`
-                );
-            }
             this.cache = data.cache;
         }
 

package/sdk/models.ts

@@ -54,7 +54,7 @@ export class Flag extends BaseFlag {
     static fromAPIFlag(flagData: any): Flag {
         return new Flag({
             enabled: flagData['enabled'],
-            value: flagData['feature_state_value'] || flagData['value'],
+            value: flagData['feature_state_value'] ?? flagData['value'],
             featureId: flagData['feature']['id'],
             featureName: flagData['feature']['name']
         });

package/sdk/types.ts

@@ -5,11 +5,24 @@ import { Logger } from 'pino';
 import { BaseOfflineHandler } from './offline_handlers.js';
 
 export type IFlagsmithValue<T = string | number | boolean | null> = T;
+
+
+/**
+ * Stores and retrieves {@link Flags} from a cache.
+ */
 export interface FlagsmithCache {
-    get(key: string): Promise<Flags | undefined> | undefined;
-    set(key: string, value: Flags, ttl?: string | number): boolean | Promise<boolean>;
-    has(key: string): boolean | Promise<boolean>;
-    [key: string]: any;
+    /**
+     * Retrieve the cached {@link Flags} for the given environment or identity, or `undefined` if no cached value exists.
+     * @param key An environment ID or identity identifier, which is used as the cache key.
+     */
+    get(key: string): Promise<Flags | undefined>;
+
+    /**
+     * Persist an environment or identity's {@link Flags} in the cache.
+     * @param key An environment ID or identity identifier, which is used as the cache key.
+     * @param value The {@link Flags} to be stored in the cache.
+     */
+    set(key: string, value: Flags): Promise<void>;
 }
 
 export type Fetch = typeof fetch

package/tests/sdk/flagsmith-cache.test.ts

@@ -4,15 +4,6 @@ beforeEach(() => {
   vi.clearAllMocks();
 });
 
-test('test_wrong_cache_interface_throws_an_error', async () => {
-  const cache = {
-    set: () => { },
-    get: () => { },
-  };
-
-  expect(() => { const flg = flagsmith({ cache }); }).toThrow();
-});
-
 test('test_empty_cache_not_read_but_populated', async () => {
   fetch.mockResolvedValue(new Response(flagsJSON));
 
@@ -23,7 +14,7 @@ test('test_empty_cache_not_read_but_populated', async () => {
   const allFlags = (await flg.getEnvironmentFlags()).allFlags();
 
   expect(set).toBeCalled();
-  expect(await cache.has('flags')).toBe(true);
+  expect(await cache.get('flags')).toBeTruthy();
 
   expect(fetch).toBeCalledTimes(1);
   expect(allFlags[0].enabled).toBe(true);
@@ -42,7 +33,7 @@ test('test_api_not_called_when_cache_present', async () => {
   const allFlags = await (await flg.getEnvironmentFlags()).allFlags();
 
   expect(set).toBeCalled();
-  expect(await cache.has('flags')).toBe(true);
+  expect(await cache.get('flags')).toBeTruthy();
 
   expect(fetch).toBeCalledTimes(1);
   expect(allFlags[0].enabled).toBe(true);
@@ -97,7 +88,7 @@ test('test_cache_used_for_identity_flags', async () => {
   const identityFlags = (await flg.getIdentityFlags(identifier, traits)).allFlags();
 
   expect(set).toBeCalled();
-  expect(await cache.has('flags-identifier')).toBe(true);
+  expect(await cache.get('flags-identifier')).toBeTruthy();
 
   expect(fetch).toBeCalledTimes(1);
 
@@ -124,7 +115,7 @@ test('test_cache_used_for_identity_flags_local_evaluation', async () => {
   const identityFlags = (await flg.getIdentityFlags(identifier, traits)).allFlags();
 
   expect(set).toBeCalled();
-  expect(await cache.has('flags-identifier')).toBe(true);
+  expect(await cache.get('flags-identifier')).toBeTruthy();
 
   expect(fetch).toBeCalledTimes(1);
 

package/tests/sdk/utils.ts

@@ -10,17 +10,12 @@ const DATA_DIR = __dirname + '/data/';
 export class TestCache implements FlagsmithCache {
     cache: Record<string, Flags> = {};
 
-    async get(name: string): Promise<Flags> {
+    async get(name: string): Promise<Flags | undefined> {
         return this.cache[name];
     }
 
-    async has(name: string): Promise<boolean> {
-        return !!this.cache[name];
-    }
-
-    async set(name: string, value: Flags, ttl: number|string) {
+    async set(name: string, value: Flags) {
         this.cache[name] = value;
-        return true
     }
 }