@splitsoftware/splitio-commons 1.2.1-rc.1 → 1.2.1-rc.12

package/cjs/sdkClient/sdkClientMethodCS.js

@@ -19,13 +19,8 @@ var method = 'Client instantiation';
  */
 function sdkClientMethodCSFactory(params) {
     var storage = params.storage, syncManager = params.syncManager, sdkReadinessManager = params.sdkReadinessManager, _a = params.settings, key = _a.core.key, readyTimeout = _a.startup.readyTimeout, log = _a.log;
-    // Keeping similar behaviour as in the isomorphic JS SDK: if settings key is invalid,
-    // `false` value is used as binded key of the default client, but trafficType is ignored
-    // @TODO handle as a non-recoverable error
-    var validKey = key_1.validateKey(log, key, method);
-    var mainClientInstance = clientCS_1.clientCSDecorator(log, sdkClient_1.sdkClientFactory(params), // @ts-ignore
-    validKey);
-    var parsedDefaultKey = key_2.keyParser(key);
+    var mainClientInstance = (0, clientCS_1.clientCSDecorator)(log, (0, sdkClient_1.sdkClientFactory)(params), key);
+    var parsedDefaultKey = (0, key_2.keyParser)(key);
     var defaultInstanceId = buildInstanceId(parsedDefaultKey);
     // Cache instances created per factory.
     var clientInstances = {};
@@ -36,13 +31,13 @@ function sdkClientMethodCSFactory(params) {
             return mainClientInstance;
         }
         // Validate the key value. The trafficType (2nd argument) is ignored
-        var validKey = key_1.validateKey(log, key, method);
+        var validKey = (0, key_1.validateKey)(log, key, method);
         if (validKey === false) {
             throw new Error('Shared Client needs a valid key.');
         }
         var instanceId = buildInstanceId(validKey);
         if (!clientInstances[instanceId]) {
-            var matchingKey = key_2.getMatching(validKey);
+            var matchingKey = (0, key_2.getMatching)(validKey);
             var sharedSdkReadiness_1 = sdkReadinessManager.shared(readyTimeout);
             var sharedStorage = storage.shared && storage.shared(matchingKey, function (err) {
                 if (err)
@@ -58,13 +53,12 @@ function sdkClientMethodCSFactory(params) {
             var sharedSyncManager = syncManager && sharedStorage && syncManager.shared(matchingKey, sharedSdkReadiness_1.readinessManager, sharedStorage);
             // As shared clients reuse all the storage information, we don't need to check here if we
             // will use offline or online mode. We should stick with the original decision.
-            clientInstances[instanceId] = clientCS_1.clientCSDecorator(log, sdkClient_1.sdkClientFactory(objectAssign_1.objectAssign({}, params, {
+            clientInstances[instanceId] = (0, clientCS_1.clientCSDecorator)(log, (0, sdkClient_1.sdkClientFactory)((0, objectAssign_1.objectAssign)({}, params, {
                 sdkReadinessManager: sharedSdkReadiness_1,
                 storage: sharedStorage || storage,
                 syncManager: sharedSyncManager,
-                signalListener: undefined,
-                sharedClient: true
-            })), validKey);
+                signalListener: undefined, // only the main client "destroy" method stops the signal listener
+            }), true), validKey);
             sharedSyncManager && sharedSyncManager.start();
             log.info(constants_1.NEW_SHARED_CLIENT);
         }

package/cjs/sdkClient/sdkClientMethodCSWithTT.js

@@ -21,17 +21,8 @@ var method = 'Client instantiation';
  */
 function sdkClientMethodCSFactory(params) {
     var storage = params.storage, syncManager = params.syncManager, sdkReadinessManager = params.sdkReadinessManager, _a = params.settings, _b = _a.core, key = _b.key, trafficType = _b.trafficType, readyTimeout = _a.startup.readyTimeout, log = _a.log;
-    // Keeping the behaviour as in the isomorphic JS SDK: if settings key or TT are invalid,
-    // `false` value is used as binded key/TT of the default client, which leads to several issues.
-    // @TODO update when supporting non-recoverable errors
-    var validKey = key_1.validateKey(log, key, method);
-    var validTrafficType;
-    if (trafficType !== undefined) {
-        validTrafficType = trafficType_1.validateTrafficType(log, trafficType, method);
-    }
-    var mainClientInstance = clientCS_1.clientCSDecorator(log, sdkClient_1.sdkClientFactory(params), // @ts-ignore
-    validKey, validTrafficType);
-    var parsedDefaultKey = key_2.keyParser(key);
+    var mainClientInstance = (0, clientCS_1.clientCSDecorator)(log, (0, sdkClient_1.sdkClientFactory)(params), key, trafficType);
+    var parsedDefaultKey = (0, key_2.keyParser)(key);
     var defaultInstanceId = buildInstanceId(parsedDefaultKey, trafficType);
     // Cache instances created per factory.
     var clientInstances = {};
@@ -42,20 +33,20 @@ function sdkClientMethodCSFactory(params) {
             return mainClientInstance;
         }
         // Validate the key value
-        var validKey = key_1.validateKey(log, key, "Shared " + method);
+        var validKey = (0, key_1.validateKey)(log, key, "Shared " + method);
         if (validKey === false) {
             throw new Error('Shared Client needs a valid key.');
         }
         var validTrafficType;
         if (trafficType !== undefined) {
-            validTrafficType = trafficType_1.validateTrafficType(log, trafficType, "Shared " + method);
+            validTrafficType = (0, trafficType_1.validateTrafficType)(log, trafficType, "Shared " + method);
             if (validTrafficType === false) {
                 throw new Error('Shared Client needs a valid traffic type or no traffic type at all.');
             }
         }
         var instanceId = buildInstanceId(validKey, validTrafficType);
         if (!clientInstances[instanceId]) {
-            var matchingKey = key_2.getMatching(validKey);
+            var matchingKey = (0, key_2.getMatching)(validKey);
             var sharedSdkReadiness_1 = sdkReadinessManager.shared(readyTimeout);
             var sharedStorage = storage.shared && storage.shared(matchingKey, function (err) {
                 if (err)
@@ -71,13 +62,12 @@ function sdkClientMethodCSFactory(params) {
             var sharedSyncManager = syncManager && sharedStorage && syncManager.shared(matchingKey, sharedSdkReadiness_1.readinessManager, sharedStorage);
             // As shared clients reuse all the storage information, we don't need to check here if we
             // will use offline or online mode. We should stick with the original decision.
-            clientInstances[instanceId] = clientCS_1.clientCSDecorator(log, sdkClient_1.sdkClientFactory(objectAssign_1.objectAssign({}, params, {
+            clientInstances[instanceId] = (0, clientCS_1.clientCSDecorator)(log, (0, sdkClient_1.sdkClientFactory)((0, objectAssign_1.objectAssign)({}, params, {
                 sdkReadinessManager: sharedSdkReadiness_1,
                 storage: sharedStorage || storage,
                 syncManager: sharedSyncManager,
-                signalListener: undefined,
-                sharedClient: true
-            })), validKey, validTrafficType);
+                signalListener: undefined, // only the main client "destroy" method stops the signal listener
+            }), true), validKey, validTrafficType);
             sharedSyncManager && sharedSyncManager.start();
             log.info(constants_1.NEW_SHARED_CLIENT);
         }

package/cjs/sdkFactory/index.js

@@ -11,24 +11,26 @@ var sdkLogger_1 = require("../logger/sdkLogger");
 var constants_1 = require("../logger/constants");
 var metadataBuilder_1 = require("../storages/metadataBuilder");
 var constants_2 = require("../readiness/constants");
+var objectAssign_1 = require("../utils/lang/objectAssign");
 /**
  * Modular SDK factory
  */
 function sdkFactory(params) {
-    var settings = params.settings, platform = params.platform, storageFactory = params.storageFactory, splitApiFactory = params.splitApiFactory, syncManagerFactory = params.syncManagerFactory, SignalListener = params.SignalListener, impressionsObserverFactory = params.impressionsObserverFactory, impressionListener = params.impressionListener, integrationsManagerFactory = params.integrationsManagerFactory, sdkManagerFactory = params.sdkManagerFactory, sdkClientMethodFactory = params.sdkClientMethodFactory;
+    var settings = params.settings, platform = params.platform, storageFactory = params.storageFactory, splitApiFactory = params.splitApiFactory, extraProps = params.extraProps, syncManagerFactory = params.syncManagerFactory, SignalListener = params.SignalListener, impressionsObserverFactory = params.impressionsObserverFactory, integrationsManagerFactory = params.integrationsManagerFactory, sdkManagerFactory = params.sdkManagerFactory, sdkClientMethodFactory = params.sdkClientMethodFactory;
     var log = settings.log;
     // @TODO handle non-recoverable errors: not start sync, mark the SDK as destroyed, etc.
     // We will just log and allow for the SDK to end up throwing an SDK_TIMEOUT event for devs to handle.
-    apiKey_1.validateAndTrackApiKey(log, settings.core.authorizationKey);
+    (0, apiKey_1.validateAndTrackApiKey)(log, settings.core.authorizationKey);
     // @TODO handle non-recoverable error, such as, `fetch` api not available, invalid API Key, etc.
-    var sdkReadinessManager = sdkReadinessManager_1.sdkReadinessManagerFactory(log, platform.EventEmitter, settings.startup.readyTimeout);
+    var sdkReadinessManager = (0, sdkReadinessManager_1.sdkReadinessManagerFactory)(log, platform.EventEmitter, settings.startup.readyTimeout);
     var readinessManager = sdkReadinessManager.readinessManager;
     // @TODO consider passing the settings object, so that each storage access only what it needs
     var storageFactoryParams = {
+        impressionsQueueSize: settings.scheduler.impressionsQueueSize,
         eventsQueueSize: settings.scheduler.eventsQueueSize,
-        optimize: utils_1.shouldBeOptimized(settings),
+        optimize: (0, utils_1.shouldBeOptimized)(settings),
         // ATM, only used by InLocalStorage
-        matchingKey: key_1.getMatching(settings.core.key),
+        matchingKey: (0, key_1.getMatching)(settings.core.key),
         splitFiltersValidation: settings.sync.__splitFiltersValidation,
         // ATM, only used by PluggableStorage
         mode: settings.mode,
@@ -40,7 +42,7 @@ function sdkFactory(params) {
             readinessManager.splits.emit(constants_2.SDK_SPLITS_ARRIVED);
             readinessManager.segments.emit(constants_2.SDK_SEGMENTS_ARRIVED);
         },
-        metadata: metadataBuilder_1.metadataBuilder(settings),
+        metadata: (0, metadataBuilder_1.metadataBuilder)(settings),
         log: log
     };
     var storage = storageFactory(storageFactoryParams);
@@ -57,28 +59,29 @@ function sdkFactory(params) {
     var integrationsManager = integrationsManagerFactory && integrationsManagerFactory({ settings: settings, storage: storage });
     // trackers
     var observer = impressionsObserverFactory && impressionsObserverFactory();
-    var impressionsTracker = impressionsTracker_1.impressionsTrackerFactory(log, storage.impressions, settings, impressionListener, integrationsManager, observer, storage.impressionCounts);
-    var eventTracker = eventTracker_1.eventTrackerFactory(log, storage.events, integrationsManager);
+    var impressionsTracker = (0, impressionsTracker_1.impressionsTrackerFactory)(settings, storage.impressions, integrationsManager, observer, storage.impressionCounts);
+    var eventTracker = (0, eventTracker_1.eventTrackerFactory)(settings, storage.events, integrationsManager);
     // signal listener
     var signalListener = SignalListener && new SignalListener(syncManager, settings, storage, splitApi);
     // Sdk client and manager
-    var clientMethod = sdkClientMethodFactory({ eventTracker: eventTracker, impressionsTracker: impressionsTracker, sdkReadinessManager: sdkReadinessManager, settings: settings, storage: storage, syncManager: syncManager, signalListener: signalListener });
+    var ctx = { eventTracker: eventTracker, impressionsTracker: impressionsTracker, sdkReadinessManager: sdkReadinessManager, settings: settings, storage: storage, syncManager: syncManager, signalListener: signalListener };
+    var clientMethod = sdkClientMethodFactory(ctx);
     var managerInstance = sdkManagerFactory(log, storage.splits, sdkReadinessManager);
     syncManager && syncManager.start();
     signalListener && signalListener.start();
     log.info(constants_1.NEW_FACTORY);
-    return {
+    // @ts-ignore
+    return (0, objectAssign_1.objectAssign)({
         // Split evaluation and event tracking engine
         client: clientMethod,
         // Manager API to explore available information
-        // @ts-ignore
         manager: function () {
             log.debug(constants_1.RETRIEVE_MANAGER);
             return managerInstance;
         },
         // Logger wrapper API
-        Logger: sdkLogger_1.createLoggerAPI(settings.log),
+        Logger: (0, sdkLogger_1.createLoggerAPI)(settings.log),
         settings: settings,
-    };
+    }, extraProps && extraProps(ctx));
 }
 exports.sdkFactory = sdkFactory;

package/cjs/sdkManager/index.js

@@ -8,7 +8,7 @@ var inputValidation_1 = require("../utils/inputValidation");
 function collectTreatments(splitObject) {
     var conditions = splitObject.conditions;
     // Rollout conditions are supposed to have the entire partitions list, so we find the first one.
-    var allTreatmentsCondition = lang_1.find(conditions, function (cond) { return cond.conditionType === 'ROLLOUT'; });
+    var allTreatmentsCondition = (0, lang_1.find)(conditions, function (cond) { return cond.conditionType === 'ROLLOUT'; });
     // Localstorage mode could fall into a no rollout conditions state. Take the first condition in that case.
     if (!allTreatmentsCondition)
         allTreatmentsCondition = conditions[0];
@@ -47,36 +47,36 @@ function objectsToViews(jsons) {
 function sdkManagerFactory(log, splits, _a) {
     var readinessManager = _a.readinessManager, sdkStatus = _a.sdkStatus;
     var SPLIT_FN_LABEL = 'split';
-    return objectAssign_1.objectAssign(
+    return (0, objectAssign_1.objectAssign)(
     // Proto-linkage of the readiness Event Emitter
     Object.create(sdkStatus), {
         /**
          * Get the Split object corresponding to the given split name if valid
          */
         split: function (maybeSplitName) {
-            var splitName = inputValidation_1.validateSplit(log, maybeSplitName, SPLIT_FN_LABEL);
-            if (!inputValidation_1.validateIfNotDestroyed(log, readinessManager, SPLIT_FN_LABEL) || !inputValidation_1.validateIfOperational(log, readinessManager, SPLIT_FN_LABEL) || !splitName) {
+            var splitName = (0, inputValidation_1.validateSplit)(log, maybeSplitName, SPLIT_FN_LABEL);
+            if (!(0, inputValidation_1.validateIfNotDestroyed)(log, readinessManager, SPLIT_FN_LABEL) || !(0, inputValidation_1.validateIfOperational)(log, readinessManager, SPLIT_FN_LABEL) || !splitName) {
                 return null;
             }
             var split = splits.getSplit(splitName);
-            if (thenable_1.thenable(split)) {
+            if ((0, thenable_1.thenable)(split)) {
                 return split.catch(function () { return null; }).then(function (result) {
-                    inputValidation_1.validateSplitExistance(log, readinessManager, splitName, result, SPLIT_FN_LABEL);
+                    (0, inputValidation_1.validateSplitExistance)(log, readinessManager, splitName, result, SPLIT_FN_LABEL);
                     return objectToView(result);
                 });
             }
-            inputValidation_1.validateSplitExistance(log, readinessManager, splitName, split, SPLIT_FN_LABEL);
+            (0, inputValidation_1.validateSplitExistance)(log, readinessManager, splitName, split, SPLIT_FN_LABEL);
             return objectToView(split);
         },
         /**
          * Get the Split objects present on the factory storage
          */
         splits: function () {
-            if (!inputValidation_1.validateIfNotDestroyed(log, readinessManager, 'splits') || !inputValidation_1.validateIfOperational(log, readinessManager, 'splits')) {
+            if (!(0, inputValidation_1.validateIfNotDestroyed)(log, readinessManager, 'splits') || !(0, inputValidation_1.validateIfOperational)(log, readinessManager, 'splits')) {
                 return [];
             }
             var currentSplits = splits.getAll();
-            return thenable_1.thenable(currentSplits) ?
+            return (0, thenable_1.thenable)(currentSplits) ?
                 currentSplits.catch(function () { return []; }).then(objectsToViews) : // handle possible rejections when using pluggable storage
                 objectsToViews(currentSplits);
         },
@@ -84,11 +84,11 @@ function sdkManagerFactory(log, splits, _a) {
          * Get the Split names present on the factory storage
          */
         names: function () {
-            if (!inputValidation_1.validateIfNotDestroyed(log, readinessManager, 'names') || !inputValidation_1.validateIfOperational(log, readinessManager, 'names')) {
+            if (!(0, inputValidation_1.validateIfNotDestroyed)(log, readinessManager, 'names') || !(0, inputValidation_1.validateIfOperational)(log, readinessManager, 'names')) {
                 return [];
             }
             var splitNames = splits.getSplitNames();
-            return thenable_1.thenable(splitNames) ?
+            return (0, thenable_1.thenable)(splitNames) ?
                 splitNames.catch(function () { return []; }) : // handle possible rejections when using pluggable storage
                 splitNames;
         }

package/cjs/services/splitApi.js

@@ -17,7 +17,7 @@ function splitApiFactory(settings, platform) {
     var urls = settings.urls;
     var filterQueryString = settings.sync.__splitFiltersValidation && settings.sync.__splitFiltersValidation.queryString;
     var SplitSDKImpressionsMode = settings.sync.impressionsMode;
-    var splitHttpClient = splitHttpClient_1.splitHttpClientFactory(settings, platform.getFetch, platform.getOptions);
+    var splitHttpClient = (0, splitHttpClient_1.splitHttpClientFactory)(settings, platform.getFetch, platform.getOptions);
     return {
         getSdkAPIHealthCheck: function () {
             var url = urls.sdk + "/version";
@@ -75,7 +75,8 @@ function splitApiFactory(settings, platform) {
             return splitHttpClient(url, {
                 // Adding extra headers to send impressions in OPTIMIZED or DEBUG modes.
                 method: 'POST',
-                body: body, headers: objectAssign_1.objectAssign({ SplitSDKImpressionsMode: SplitSDKImpressionsMode }, headers)
+                body: body,
+                headers: (0, objectAssign_1.objectAssign)({ SplitSDKImpressionsMode: SplitSDKImpressionsMode }, headers)
             });
         },
         /**

package/cjs/services/splitHttpClient.js

@@ -31,8 +31,8 @@ function splitHttpClientFactory(settings, getFetch, getOptions) {
     return function httpClient(url, reqOpts, logErrorsAsInfo) {
         if (reqOpts === void 0) { reqOpts = {}; }
         if (logErrorsAsInfo === void 0) { logErrorsAsInfo = false; }
-        var request = objectAssign_1.objectAssign({
-            headers: reqOpts.headers ? objectAssign_1.objectAssign({}, headers, reqOpts.headers) : headers,
+        var request = (0, objectAssign_1.objectAssign)({
+            headers: reqOpts.headers ? (0, objectAssign_1.objectAssign)({}, headers, reqOpts.headers) : headers,
             method: reqOpts.method || 'GET',
             body: reqOpts.body
         }, options);

package/cjs/storages/KeyBuilder.js

@@ -5,11 +5,7 @@ var lang_1 = require("../utils/lang");
 var everythingAtTheEnd = /[^.]+$/;
 var DEFAULT_PREFIX = 'SPLITIO';
 function validatePrefix(prefix) {
-    return prefix && typeof prefix === 'string' ?
-        lang_1.endsWith(prefix, '.' + DEFAULT_PREFIX) ?
-            prefix : // suffix already appended
-            prefix + '.' + DEFAULT_PREFIX : // append suffix
-        DEFAULT_PREFIX; // use default prefix if none is provided
+    return prefix ? prefix + '.SPLITIO' : 'SPLITIO';
 }
 exports.validatePrefix = validatePrefix;
 var KeyBuilder = /** @class */ (function () {
@@ -31,7 +27,7 @@ var KeyBuilder = /** @class */ (function () {
     //   return `${this.prefix}.splits.ready`;
     // }
     KeyBuilder.prototype.isSplitKey = function (key) {
-        return lang_1.startsWith(key, this.prefix + ".split.");
+        return (0, lang_1.startsWith)(key, this.prefix + ".split.");
     };
     KeyBuilder.prototype.buildSplitKeyPrefix = function () {
         return this.prefix + ".split.";

package/cjs/storages/KeyBuilderCS.js

@@ -5,7 +5,7 @@ var tslib_1 = require("tslib");
 var lang_1 = require("../utils/lang");
 var KeyBuilder_1 = require("./KeyBuilder");
 var KeyBuilderCS = /** @class */ (function (_super) {
-    tslib_1.__extends(KeyBuilderCS, _super);
+    (0, tslib_1.__extends)(KeyBuilderCS, _super);
     function KeyBuilderCS(prefix, matchingKey) {
         var _this = _super.call(this, prefix) || this;
         _this.matchingKey = matchingKey;
@@ -16,11 +16,21 @@ var KeyBuilderCS = /** @class */ (function (_super) {
      * @override
      */
     KeyBuilderCS.prototype.buildSegmentNameKey = function (segmentName) {
-        return this.matchingKey + "." + this.prefix + ".segment." + segmentName;
+        return this.prefix + "." + this.matchingKey + ".segment." + segmentName;
     };
     KeyBuilderCS.prototype.extractSegmentName = function (builtSegmentKeyName) {
+        var prefix = this.prefix + "." + this.matchingKey + ".segment.";
+        if ((0, lang_1.startsWith)(builtSegmentKeyName, prefix))
+            return builtSegmentKeyName.substr(prefix.length);
+    };
+    // @BREAKING: The key used to start with the matching key instead of the prefix, this was changed on version 10.17.3
+    KeyBuilderCS.prototype.buildOldSegmentNameKey = function (segmentName) {
+        return this.matchingKey + "." + this.prefix + ".segment." + segmentName;
+    };
+    // @BREAKING: The key used to start with the matching key instead of the prefix, this was changed on version 10.17.3
+    KeyBuilderCS.prototype.extractOldSegmentKey = function (builtSegmentKeyName) {
         var prefix = this.matchingKey + "." + this.prefix + ".segment.";
-        if (lang_1.startsWith(builtSegmentKeyName, prefix))
+        if ((0, lang_1.startsWith)(builtSegmentKeyName, prefix))
             return builtSegmentKeyName.substr(prefix.length);
     };
     KeyBuilderCS.prototype.buildLastUpdatedKey = function () {

package/cjs/storages/KeyBuilderSS.js

@@ -7,7 +7,7 @@ var KeyBuilder_1 = require("./KeyBuilder");
 // const everythingAfterCount = /count\.([^/]+)$/;
 // const latencyMetricNameAndBucket = /latency\.([^/]+)\.bucket\.([0-9]+)$/;
 var KeyBuilderSS = /** @class */ (function (_super) {
-    tslib_1.__extends(KeyBuilderSS, _super);
+    (0, tslib_1.__extends)(KeyBuilderSS, _super);
     function KeyBuilderSS(prefix, metadata) {
         var _this = _super.call(this, prefix) || this;
         _this.metadata = metadata;

package/cjs/storages/findLatencyIndex.js

@@ -8,6 +8,6 @@ function findLatencyIndex(latency, min, max, base) {
     if (max === void 0) { max = 23; }
     if (base === void 0) { base = 1.5; }
     var index = Math.min(max, Math.max(min, Math.floor(Math.log(latency) / Math.log(base))));
-    return lang_1.isNaNNumber(index) ? 0 : index; // index is NaN if latency is not a positive number
+    return (0, lang_1.isNaNNumber)(index) ? 0 : index; // index is NaN if latency is not a positive number
 }
 exports.findLatencyIndex = findLatencyIndex;

package/cjs/storages/inLocalStorage/MySegmentsCacheInLocal.js

@@ -5,7 +5,7 @@ var tslib_1 = require("tslib");
 var AbstractSegmentsCacheSync_1 = require("../AbstractSegmentsCacheSync");
 var constants_1 = require("./constants");
 var MySegmentsCacheInLocal = /** @class */ (function (_super) {
-    tslib_1.__extends(MySegmentsCacheInLocal, _super);
+    (0, tslib_1.__extends)(MySegmentsCacheInLocal, _super);
     function MySegmentsCacheInLocal(log, keys) {
         var _this = _super.call(this) || this;
         _this.log = log;
@@ -60,9 +60,29 @@ var MySegmentsCacheInLocal = /** @class */ (function (_super) {
         var index;
         // Scan current values from localStorage
         var storedSegmentNames = Object.keys(localStorage).reduce(function (accum, key) {
-            var name = _this.keys.extractSegmentName(key);
-            if (name)
-                accum.push(name);
+            var segmentName = _this.keys.extractSegmentName(key);
+            if (segmentName) {
+                accum.push(segmentName);
+            }
+            else {
+                // @BREAKING: This is only to clean up "old" keys. Remove this whole else code block.
+                segmentName = _this.keys.extractOldSegmentKey(key);
+                if (segmentName) { // this was an old segment key, let's clean up.
+                    var newSegmentKey = _this.keys.buildSegmentNameKey(segmentName);
+                    try {
+                        // If the new format key is not there, create it.
+                        if (!localStorage.getItem(newSegmentKey) && names.indexOf(segmentName) > -1) {
+                            localStorage.setItem(newSegmentKey, constants_1.DEFINED);
+                            // we are migrating a segment, let's track it.
+                            accum.push(segmentName);
+                        }
+                        localStorage.removeItem(key); // we migrated the current key, let's delete it.
+                    }
+                    catch (e) {
+                        _this.log.error(e);
+                    }
+                }
+            }
             return accum;
         }, []);
         // Extreme fast => everything is empty

package/cjs/storages/inLocalStorage/SplitsCacheInLocal.js

@@ -9,7 +9,7 @@ var constants_1 = require("./constants");
  * ISplitsCacheSync implementation that stores split definitions in browser LocalStorage.
  */
 var SplitsCacheInLocal = /** @class */ (function (_super) {
-    tslib_1.__extends(SplitsCacheInLocal, _super);
+    (0, tslib_1.__extends)(SplitsCacheInLocal, _super);
     /**
      * @param {KeyBuilderCS} keys
      * @param {number | undefined} expirationTimestamp
@@ -27,7 +27,7 @@ var SplitsCacheInLocal = /** @class */ (function (_super) {
         return _this;
     }
     SplitsCacheInLocal.prototype._decrementCount = function (key) {
-        var count = lang_1.toNumber(localStorage.getItem(key)) - 1;
+        var count = (0, lang_1.toNumber)(localStorage.getItem(key)) - 1;
         // @ts-expect-error
         if (count > 0)
             localStorage.setItem(key, count);
@@ -41,7 +41,7 @@ var SplitsCacheInLocal = /** @class */ (function (_super) {
                     var ttKey = this.keys.buildTrafficTypeKey(split.trafficTypeName);
                     this._decrementCount(ttKey);
                 }
-                if (AbstractSplitsCacheSync_1.usesSegments(split)) {
+                if ((0, AbstractSplitsCacheSync_1.usesSegments)(split)) {
                     var segmentsCountKey = this.keys.buildSplitsWithSegmentCountKey();
                     this._decrementCount(segmentsCountKey);
                 }
@@ -57,12 +57,12 @@ var SplitsCacheInLocal = /** @class */ (function (_super) {
                 if (split.trafficTypeName) {
                     var ttKey = this.keys.buildTrafficTypeKey(split.trafficTypeName);
                     // @ts-expect-error
-                    localStorage.setItem(ttKey, lang_1.toNumber(localStorage.getItem(ttKey)) + 1);
+                    localStorage.setItem(ttKey, (0, lang_1.toNumber)(localStorage.getItem(ttKey)) + 1);
                 }
-                if (AbstractSplitsCacheSync_1.usesSegments(split)) {
+                if ((0, AbstractSplitsCacheSync_1.usesSegments)(split)) {
                     var segmentsCountKey = this.keys.buildSplitsWithSegmentCountKey();
                     // @ts-expect-error
-                    localStorage.setItem(segmentsCountKey, lang_1.toNumber(localStorage.getItem(segmentsCountKey)) + 1);
+                    localStorage.setItem(segmentsCountKey, (0, lang_1.toNumber)(localStorage.getItem(segmentsCountKey)) + 1);
                 }
             }
         }
@@ -161,7 +161,7 @@ var SplitsCacheInLocal = /** @class */ (function (_super) {
         var value = localStorage.getItem(this.keys.buildSplitsTillKey());
         if (value !== null) {
             value = parseInt(value, 10);
-            return lang_1.isNaNNumber(value) ? n : value;
+            return (0, lang_1.isNaNNumber)(value) ? n : value;
         }
         return n;
     };
@@ -178,16 +178,16 @@ var SplitsCacheInLocal = /** @class */ (function (_super) {
         return accum;
     };
     SplitsCacheInLocal.prototype.trafficTypeExists = function (trafficType) {
-        var ttCount = lang_1.toNumber(localStorage.getItem(this.keys.buildTrafficTypeKey(trafficType)));
-        return lang_1.isFiniteNumber(ttCount) && ttCount > 0;
+        var ttCount = (0, lang_1.toNumber)(localStorage.getItem(this.keys.buildTrafficTypeKey(trafficType)));
+        return (0, lang_1.isFiniteNumber)(ttCount) && ttCount > 0;
     };
     SplitsCacheInLocal.prototype.usesSegments = function () {
         // If cache hasn't been synchronized with the cloud, assume we need them.
         if (!this.hasSync)
             return true;
         var storedCount = localStorage.getItem(this.keys.buildSplitsWithSegmentCountKey());
-        var splitsWithSegmentsCount = storedCount === null ? 0 : lang_1.toNumber(storedCount);
-        if (lang_1.isFiniteNumber(splitsWithSegmentsCount)) {
+        var splitsWithSegmentsCount = storedCount === null ? 0 : (0, lang_1.toNumber)(storedCount);
+        if ((0, lang_1.isFiniteNumber)(splitsWithSegmentsCount)) {
             return splitsWithSegmentsCount > 0;
         }
         else {
@@ -212,7 +212,7 @@ var SplitsCacheInLocal = /** @class */ (function (_super) {
         var value = localStorage.getItem(this.keys.buildLastUpdatedKey());
         if (value !== null) {
             value = parseInt(value, 10);
-            if (!lang_1.isNaNNumber(value) && expirationTimestamp && value < expirationTimestamp)
+            if (!(0, lang_1.isNaNNumber)(value) && expirationTimestamp && value < expirationTimestamp)
                 this.clear();
         }
     };

package/cjs/storages/inLocalStorage/index.js

@@ -20,12 +20,12 @@ var constants_2 = require("../../utils/constants");
  */
 function InLocalStorage(options) {
     if (options === void 0) { options = {}; }
-    var prefix = KeyBuilder_1.validatePrefix(options.prefix);
+    var prefix = (0, KeyBuilder_1.validatePrefix)(options.prefix);
     function InLocalStorageCSFactory(params) {
         // Fallback to InMemoryStorage if LocalStorage API is not available
-        if (!isLocalStorageAvailable_1.isLocalStorageAvailable()) {
+        if (!(0, isLocalStorageAvailable_1.isLocalStorageAvailable)()) {
             params.log.warn(constants_1.LOG_PREFIX + 'LocalStorage API is unavailable. Fallbacking into default MEMORY storage');
-            return InMemoryStorageCS_1.InMemoryStorageCSFactory(params);
+            return (0, InMemoryStorageCS_1.InMemoryStorageCSFactory)(params);
         }
         var log = params.log;
         var keys = new KeyBuilderCS_1.KeyBuilderCS(prefix, params.matchingKey);
@@ -33,7 +33,7 @@ function InLocalStorage(options) {
         return {
             splits: new SplitsCacheInLocal_1.SplitsCacheInLocal(log, keys, expirationTimestamp, params.splitFiltersValidation),
             segments: new MySegmentsCacheInLocal_1.MySegmentsCacheInLocal(log, keys),
-            impressions: new ImpressionsCacheInMemory_1.ImpressionsCacheInMemory(),
+            impressions: new ImpressionsCacheInMemory_1.ImpressionsCacheInMemory(params.impressionsQueueSize),
             impressionCounts: params.optimize ? new ImpressionCountsCacheInMemory_1.ImpressionCountsCacheInMemory() : undefined,
             events: new EventsCacheInMemory_1.EventsCacheInMemory(params.eventsQueueSize),
             destroy: function () {

package/cjs/storages/inMemory/AttributesCacheInMemory.js

@@ -33,7 +33,7 @@ var AttributesCacheInMemory = /** @class */ (function () {
      * @returns {boolean} attributes were stored
      */
     AttributesCacheInMemory.prototype.setAttributes = function (attributes) {
-        this.attributesCache = objectAssign_1.objectAssign(this.attributesCache, attributes);
+        this.attributesCache = (0, objectAssign_1.objectAssign)(this.attributesCache, attributes);
         return true;
     };
     /**

package/cjs/storages/inMemory/ImpressionCountsCacheInMemory.js

@@ -10,7 +10,7 @@ var ImpressionCountsCacheInMemory = /** @class */ (function () {
     * Builds key to be stored in the cache with the featureName and the timeFrame truncated.
     */
     ImpressionCountsCacheInMemory.prototype._makeKey = function (featureName, timeFrame) {
-        return featureName + "::" + time_1.truncateTimeFrame(timeFrame);
+        return featureName + "::" + (0, time_1.truncateTimeFrame)(timeFrame);
     };
     /**
     * Increments the quantity of impressions with the passed featureName and timeFrame.

package/cjs/storages/inMemory/ImpressionsCacheInMemory.js

@@ -2,15 +2,29 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.ImpressionsCacheInMemory = void 0;
 var ImpressionsCacheInMemory = /** @class */ (function () {
-    function ImpressionsCacheInMemory() {
+    /**
+     *
+     * @param impressionsQueueSize number of queued impressions to call onFullQueueCb.
+     * Default value is 0, that means no maximum value, in case we want to avoid this being triggered.
+     */
+    function ImpressionsCacheInMemory(impressionsQueueSize) {
+        if (impressionsQueueSize === void 0) { impressionsQueueSize = 0; }
+        this.maxQueue = impressionsQueueSize;
         this.queue = [];
     }
+    ImpressionsCacheInMemory.prototype.setOnFullQueueCb = function (cb) {
+        this.onFullQueue = cb;
+    };
     /**
      * Store impressions in sequential order
      */
     ImpressionsCacheInMemory.prototype.track = function (data) {
         var _a;
         (_a = this.queue).push.apply(_a, data);
+        // Check if the cache queue is full and we need to flush it.
+        if (this.maxQueue > 0 && this.queue.length >= this.maxQueue && this.onFullQueue) {
+            this.onFullQueue();
+        }
     };
     /**
      * Clear the data stored on the cache.

package/cjs/storages/inMemory/InMemoryStorage.js

@@ -16,7 +16,7 @@ function InMemoryStorageFactory(params) {
     return {
         splits: new SplitsCacheInMemory_1.SplitsCacheInMemory(),
         segments: new SegmentsCacheInMemory_1.SegmentsCacheInMemory(),
-        impressions: new ImpressionsCacheInMemory_1.ImpressionsCacheInMemory(),
+        impressions: new ImpressionsCacheInMemory_1.ImpressionsCacheInMemory(params.impressionsQueueSize),
         impressionCounts: params.optimize ? new ImpressionCountsCacheInMemory_1.ImpressionCountsCacheInMemory() : undefined,
         events: new EventsCacheInMemory_1.EventsCacheInMemory(params.eventsQueueSize),
         // When using MEMORY we should clean all the caches to leave them empty

package/cjs/storages/inMemory/InMemoryStorageCS.js

@@ -16,7 +16,7 @@ function InMemoryStorageCSFactory(params) {
     return {
         splits: new SplitsCacheInMemory_1.SplitsCacheInMemory(),
         segments: new MySegmentsCacheInMemory_1.MySegmentsCacheInMemory(),
-        impressions: new ImpressionsCacheInMemory_1.ImpressionsCacheInMemory(),
+        impressions: new ImpressionsCacheInMemory_1.ImpressionsCacheInMemory(params.impressionsQueueSize),
         impressionCounts: params.optimize ? new ImpressionCountsCacheInMemory_1.ImpressionCountsCacheInMemory() : undefined,
         events: new EventsCacheInMemory_1.EventsCacheInMemory(params.eventsQueueSize),
         // When using MEMORY we should clean all the caches to leave them empty

package/cjs/storages/inMemory/LatenciesCacheInMemory.js

@@ -17,7 +17,7 @@ var LatenciesCacheInMemory = /** @class */ (function () {
             ];
         }
         // +1 based on the latency number
-        this.counters[metricName][findLatencyIndex_1.findLatencyIndex(latency)]++;
+        this.counters[metricName][(0, findLatencyIndex_1.findLatencyIndex)(latency)]++;
         return true;
     };
     /**

package/cjs/storages/inMemory/MySegmentsCacheInMemory.js

@@ -8,7 +8,7 @@ var AbstractSegmentsCacheSync_1 = require("../AbstractSegmentsCacheSync");
  * Supported by all JS runtimes.
  */
 var MySegmentsCacheInMemory = /** @class */ (function (_super) {
-    tslib_1.__extends(MySegmentsCacheInMemory, _super);
+    (0, tslib_1.__extends)(MySegmentsCacheInMemory, _super);
     function MySegmentsCacheInMemory() {
         var _this = _super !== null && _super.apply(this, arguments) || this;
         _this.segmentCache = {};

package/cjs/storages/inMemory/SegmentsCacheInMemory.js

@@ -10,7 +10,7 @@ var lang_1 = require("../../utils/lang");
  * Supported by all JS runtimes.
  */
 var SegmentsCacheInMemory = /** @class */ (function (_super) {
-    tslib_1.__extends(SegmentsCacheInMemory, _super);
+    (0, tslib_1.__extends)(SegmentsCacheInMemory, _super);
     function SegmentsCacheInMemory() {
         var _this = _super !== null && _super.apply(this, arguments) || this;
         _this.segmentCache = {};
@@ -63,7 +63,7 @@ var SegmentsCacheInMemory = /** @class */ (function (_super) {
     };
     SegmentsCacheInMemory.prototype.getChangeNumber = function (name) {
         var value = this.segmentChangeNumber[name];
-        return lang_1.isIntegerNumber(value) ? value : -1;
+        return (0, lang_1.isIntegerNumber)(value) ? value : -1;
     };
     return SegmentsCacheInMemory;
 }(AbstractSegmentsCacheSync_1.AbstractSegmentsCacheSync));