@splitsoftware/splitio-commons 2.12.0 → 3.0.0-rc.1

package/cjs/sync/streaming/SSEClient/index.js

@@ -60,11 +60,11 @@ var SSEClient = /** @class */ (function () {
             var params = CONTROL_CHANNEL_REGEX.test(channel) ? '[?occupancy=metrics.publishers]' : '';
             return encodeURIComponent(params + channel);
         }).join(',');
-        var url = this.settings.urls.streaming + "/sse?channels=" + channelsQueryParam + "&accessToken=" + authToken.token + "&v=" + ABLY_API_VERSION + "&heartbeats=true"; // same results using `&heartbeats=false`
+        var url = "".concat(this.settings.urls.streaming, "/sse?channels=").concat(channelsQueryParam, "&accessToken=").concat(authToken.token, "&v=").concat(ABLY_API_VERSION, "&heartbeats=true"); // same results using `&heartbeats=false`
         var isServerSide = (0, key_1.checkIfServerSide)(this.settings);
         this.connection = new this.eventSource(
         // For client-side SDKs, metadata is passed as query param to avoid CORS issues and because native EventSource implementations in browsers do not support headers
-        isServerSide ? url : url + ("&SplitSDKVersion=" + this.headers.SplitSDKVersion + "&SplitSDKClientKey=" + this.headers.SplitSDKClientKey), 
+        isServerSide ? url : url + "&SplitSDKVersion=".concat(this.headers.SplitSDKVersion, "&SplitSDKClientKey=").concat(this.headers.SplitSDKClientKey), 
         // For server-side SDKs, metadata is passed via headers
         (0, objectAssign_1.objectAssign)(isServerSide ?
             { headers: (0, decorateHeaders_1.decorateHeaders)(this.settings, this.headers) } :

package/cjs/sync/streaming/SSEHandler/NotificationKeeper.js

@@ -73,7 +73,7 @@ function notificationKeeperFactory(pushEmitter, telemetryTracker) {
                         c.cTime = timestamp;
                         if (controlType === constants_2.ControlType.STREAMING_DISABLED) {
                             telemetryTracker.streamingEvent(constants_1.STREAMING_STATUS, constants_1.DISABLED);
-                            pushEmitter.emit(constants_2.PUSH_NONRETRYABLE_ERROR);
+                            pushEmitter.emit(constants_2.PUSH_NON_RETRYABLE_ERROR);
                         }
                         else if (hasPublishers) {
                             if (controlType === constants_2.ControlType.STREAMING_PAUSED && hasResumed) {

package/cjs/sync/streaming/SSEHandler/index.js

@@ -51,7 +51,7 @@ function SSEHandlerFactory(log, pushEmitter, telemetryTracker) {
                 pushEmitter.emit(constants_1.PUSH_RETRYABLE_ERROR);
             }
             else {
-                pushEmitter.emit(constants_1.PUSH_NONRETRYABLE_ERROR);
+                pushEmitter.emit(constants_1.PUSH_NON_RETRYABLE_ERROR);
             }
         },
         /* NotificationProcessor */

package/cjs/sync/streaming/UpdateWorkers/{SplitsUpdateWorker.js → DefinitionsUpdateWorker.js}

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.SplitsUpdateWorker = void 0;
+exports.DefinitionsUpdateWorker = void 0;
 var constants_1 = require("../../../logger/constants");
 var constants_2 = require("../../../readiness/constants");
 var Backoff_1 = require("../../../utils/Backoff");
@@ -9,28 +9,28 @@ var constants_4 = require("../constants");
 var parseUtils_1 = require("../parseUtils");
 var constants_5 = require("./constants");
 /**
- * SplitsUpdateWorker factory
+ * DefinitionsUpdateWorker factory
  */
-function SplitsUpdateWorker(log, storage, splitsSyncTask, splitsEventEmitter, telemetryTracker, segmentsSyncTask) {
-    var ff = SplitsUpdateWorker(storage.splits);
-    var rbs = SplitsUpdateWorker(storage.rbSegments);
-    function SplitsUpdateWorker(cache) {
+function DefinitionsUpdateWorker(log, storage, definitionsSyncTask, definitionsEventEmitter, telemetryTracker, segmentsSyncTask) {
+    var ff = DefinitionsUpdateWorker(storage.definitions);
+    var rbs = DefinitionsUpdateWorker(storage.rbSegments);
+    function DefinitionsUpdateWorker(cache) {
         var maxChangeNumber = -1;
         var handleNewEvent = false;
         var isHandlingEvent;
         var cdnBypass;
         var instantUpdate;
-        var backoff = new Backoff_1.Backoff(__handleSplitUpdateCall, constants_5.FETCH_BACKOFF_BASE, constants_5.FETCH_BACKOFF_MAX_WAIT);
-        function __handleSplitUpdateCall() {
+        var backoff = new Backoff_1.Backoff(__handleDefinitionUpdateCall, constants_5.FETCH_BACKOFF_BASE, constants_5.FETCH_BACKOFF_MAX_WAIT);
+        function __handleDefinitionUpdateCall() {
             isHandlingEvent = true;
             if (maxChangeNumber > cache.getChangeNumber()) {
                 handleNewEvent = false;
                 // fetch splits revalidating data if cached
-                splitsSyncTask.execute(true, cdnBypass ? maxChangeNumber : undefined, instantUpdate).then(function () {
+                definitionsSyncTask.execute(true, cdnBypass ? maxChangeNumber : undefined, instantUpdate).then(function () {
                     if (!isHandlingEvent)
                         return; // halt if `stop` has been called
                     if (handleNewEvent) {
-                        __handleSplitUpdateCall();
+                        __handleDefinitionUpdateCall();
                     }
                     else {
                         if (instantUpdate)
@@ -40,7 +40,7 @@ function SplitsUpdateWorker(log, storage, splitsSyncTask, splitsEventEmitter, te
                             segmentsSyncTask.execute(true);
                         var attempts = backoff.attempts + 1;
                         if (ff.isSync() && rbs.isSync()) {
-                            log.debug("Refresh completed" + (cdnBypass ? ' bypassing the CDN' : '') + " in " + attempts + " attempts.");
+                            log.debug("Refresh completed".concat(cdnBypass ? ' bypassing the CDN' : '', " in ").concat(attempts, " attempts."));
                             isHandlingEvent = false;
                             return;
                         }
@@ -49,13 +49,13 @@ function SplitsUpdateWorker(log, storage, splitsSyncTask, splitsEventEmitter, te
                             return;
                         }
                         if (cdnBypass) {
-                            log.debug("No changes fetched after " + attempts + " attempts with CDN bypassed.");
+                            log.debug("No changes fetched after ".concat(attempts, " attempts with CDN bypassed."));
                             isHandlingEvent = false;
                         }
                         else {
                             backoff.reset();
                             cdnBypass = true;
-                            __handleSplitUpdateCall();
+                            __handleDefinitionUpdateCall();
                         }
                     }
                 });
@@ -83,7 +83,7 @@ function SplitsUpdateWorker(log, storage, splitsSyncTask, splitsEventEmitter, te
                     instantUpdate = { payload: payload, changeNumber: changeNumber, type: type };
                 }
                 if (backoff.timeoutID || !isHandlingEvent)
-                    __handleSplitUpdateCall();
+                    __handleDefinitionUpdateCall();
                 backoff.reset();
             },
             stop: function () {
@@ -118,11 +118,11 @@ function SplitsUpdateWorker(log, storage, splitsSyncTask, splitsEventEmitter, te
          * @param splitName - name of split to kill
          * @param defaultTreatment - default treatment value
          */
-        killSplit: function (_a) {
+        killDefinition: function (_a) {
             var changeNumber = _a.changeNumber, splitName = _a.splitName, defaultTreatment = _a.defaultTreatment;
-            if (storage.splits.killLocally(splitName, defaultTreatment, changeNumber)) {
+            if (storage.definitions.killLocally(splitName, defaultTreatment, changeNumber)) {
                 // trigger an SDK_UPDATE if Split was killed locally
-                splitsEventEmitter.emit(constants_2.SDK_SPLITS_ARRIVED, true);
+                definitionsEventEmitter.emit(constants_2.SDK_DEFINITIONS_ARRIVED, true);
             }
             // queues the SplitChanges fetch (only if changeNumber is newer)
             ff.put({ changeNumber: changeNumber });
@@ -133,4 +133,4 @@ function SplitsUpdateWorker(log, storage, splitsSyncTask, splitsEventEmitter, te
         }
     };
 }
-exports.SplitsUpdateWorker = SplitsUpdateWorker;
+exports.DefinitionsUpdateWorker = DefinitionsUpdateWorker;

package/cjs/sync/streaming/UpdateWorkers/MySegmentsUpdateWorker.js

@@ -51,7 +51,7 @@ function MySegmentsUpdateWorker(log, storage, mySegmentsSyncTask, telemetryTrack
                             telemetryTracker.trackUpdatesFromSSE(constants_1.MEMBERSHIPS);
                         var attempts = backoff.attempts + 1;
                         if (maxChangeNumber <= currentChangeNumber) {
-                            log.debug("Refresh completed" + (cdnBypass ? ' bypassing the CDN' : '') + " in " + attempts + " attempts.");
+                            log.debug("Refresh completed".concat(cdnBypass ? ' bypassing the CDN' : '', " in ").concat(attempts, " attempts."));
                             isHandlingEvent = false;
                             return;
                         }
@@ -60,7 +60,7 @@ function MySegmentsUpdateWorker(log, storage, mySegmentsSyncTask, telemetryTrack
                             return;
                         }
                         if (cdnBypass) {
-                            log.debug("No changes fetched after " + attempts + " attempts with CDN bypassed.");
+                            log.debug("No changes fetched after ".concat(attempts, " attempts with CDN bypassed."));
                             isHandlingEvent = false;
                         }
                         else {

package/cjs/sync/streaming/UpdateWorkers/SegmentsUpdateWorker.js

@@ -28,7 +28,7 @@ function SegmentsUpdateWorker(log, segmentsSyncTask, segmentsCache) {
                     else {
                         var attempts = backoff.attempts + 1;
                         if (maxChangeNumber <= (segmentsCache.getChangeNumber(segment) || -1)) {
-                            log.debug("Refresh completed" + (cdnBypass ? ' bypassing the CDN' : '') + " in " + attempts + " attempts.");
+                            log.debug("Refresh completed".concat(cdnBypass ? ' bypassing the CDN' : '', " in ").concat(attempts, " attempts."));
                             isHandlingEvent = false;
                             return;
                         }
@@ -37,7 +37,7 @@ function SegmentsUpdateWorker(log, segmentsSyncTask, segmentsCache) {
                             return;
                         }
                         if (cdnBypass) {
-                            log.debug("No changes fetched after " + attempts + " attempts with CDN bypassed.");
+                            log.debug("No changes fetched after ".concat(attempts, " attempts with CDN bypassed."));
                             isHandlingEvent = false;
                         }
                         else {

package/cjs/sync/streaming/constants.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.ControlType = exports.OCCUPANCY = exports.CONTROL = exports.RB_SEGMENT_UPDATE = exports.SPLIT_UPDATE = exports.SPLIT_KILL = exports.SEGMENT_UPDATE = exports.MEMBERSHIPS_LS_UPDATE = exports.MEMBERSHIPS_MS_UPDATE = exports.PUSH_SUBSYSTEM_DOWN = exports.PUSH_SUBSYSTEM_UP = exports.PUSH_RETRYABLE_ERROR = exports.PUSH_NONRETRYABLE_ERROR = exports.SECONDS_BEFORE_EXPIRATION = void 0;
+exports.ControlType = exports.OCCUPANCY = exports.CONTROL = exports.RB_SEGMENT_UPDATE = exports.SPLIT_UPDATE = exports.SPLIT_KILL = exports.SEGMENT_UPDATE = exports.MEMBERSHIPS_LS_UPDATE = exports.MEMBERSHIPS_MS_UPDATE = exports.PUSH_SUBSYSTEM_DOWN = exports.PUSH_SUBSYSTEM_UP = exports.PUSH_RETRYABLE_ERROR = exports.PUSH_NON_RETRYABLE_ERROR = exports.SECONDS_BEFORE_EXPIRATION = void 0;
 // time for refresh token
 exports.SECONDS_BEFORE_EXPIRATION = 600;
 // Internal SDK events, subscribed by SyncManager and PushManager
@@ -8,7 +8,7 @@ exports.SECONDS_BEFORE_EXPIRATION = 600;
  * emitted on SSE and Authenticate non-recoverable errors, STREAMING_DISABLED control notification and authentication with pushEnabled false
  * triggers `handleNonRetryableError` call
  */
-exports.PUSH_NONRETRYABLE_ERROR = 'PUSH_NONRETRYABLE_ERROR';
+exports.PUSH_NON_RETRYABLE_ERROR = 'PUSH_NON_RETRYABLE_ERROR';
 /**
  * emitted on SSE and Authenticate recoverable errors
  * triggers `handleRetryableError` call
@@ -20,7 +20,7 @@ exports.PUSH_RETRYABLE_ERROR = 'PUSH_RETRYABLE_ERROR';
  */
 exports.PUSH_SUBSYSTEM_UP = 'PUSH_SUBSYSTEM_UP';
 /**
- * emitted on STREAMING_PAUSED control notification, OCCUPANCY equal to 0, PUSH_NONRETRYABLE_ERROR and PUSH_RETRYABLE_ERROR events.
+ * emitted on STREAMING_PAUSED control notification, OCCUPANCY equal to 0, PUSH_NON_RETRYABLE_ERROR and PUSH_RETRYABLE_ERROR events.
  * triggers `startPolling` and `stopWorkers` calls
  */
 exports.PUSH_SUBSYSTEM_DOWN = 'PUSH_SUBSYSTEM_DOWN';

package/cjs/sync/streaming/parseUtils.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.getDelay = exports.parseFFUpdatePayload = exports.isInBitmap = exports.parseBitmap = exports.parseKeyList = void 0;
+exports.getDelay = exports.parseFFUpdatePayload = exports.isInBitmap = exports.parseBitmap = exports.parseCompressedData = void 0;
 var decompress_1 = require("../../utils/decompress");
 var base64_1 = require("../../utils/base64");
 var murmur3_1 = require("../../utils/murmur3/murmur3");
@@ -30,27 +30,27 @@ function decompress(data, compression) {
         return decompress_1.algorithms.gunzipSync(binData);
     if (compression === ZLIB)
         return decompress_1.algorithms.unzlibSync(binData);
-    throw new Error("Invalid compression algorithm #" + compression);
+    throw new Error("Invalid compression algorithm #".concat(compression));
 }
 /**
- * Decode, decompress and parse the provided 'data' into a KeyList object
+ * Decode, decompress and parse the provided 'data' into an object of type T
  *
  * @param data - base64 encoded string
  * @param compression - 1 GZIP, 2 ZLIB
  * @param avoidPrecisionLoss - true as default, set it as false if dont need to avoid precission loss
- * @returns keyList
+ * @returns parsed object
  * @throws if data string cannot be decoded, decompressed or parsed
  */
-function parseKeyList(data, compression, avoidPrecisionLoss) {
+function parseCompressedData(data, compression, avoidPrecisionLoss) {
     if (avoidPrecisionLoss === void 0) { avoidPrecisionLoss = true; }
-    var binKeyList = decompress(data, compression);
-    var strKeyList = Uint8ArrayToString(binKeyList);
-    // replace numbers to strings, to avoid losing precision
+    var binData = decompress(data, compression);
+    var str = Uint8ArrayToString(binData);
+    // replace numbers to strings, to avoid losing precision (e.g., 64-bit IDs in KeyList)
     if (avoidPrecisionLoss)
-        strKeyList = strKeyList.replace(/\d+/g, '"$&"');
-    return JSON.parse(strKeyList);
+        str = str.replace(/\d+/g, '"$&"');
+    return JSON.parse(str);
 }
-exports.parseKeyList = parseKeyList;
+exports.parseCompressedData = parseCompressedData;
 /**
  * Decode, decompress and parse the provided 'data' into a Bitmap object
  *
@@ -83,7 +83,7 @@ exports.isInBitmap = isInBitmap;
  */
 function parseFFUpdatePayload(compression, data) {
     return compression > 0 ?
-        parseKeyList(data, compression, false) :
+        parseCompressedData(data, compression, false) :
         JSON.parse((0, base64_1.decodeFromBase64)(data));
 }
 exports.parseFFUpdatePayload = parseFFUpdatePayload;

package/cjs/sync/streaming/pushManager.js

@@ -6,7 +6,7 @@ var Backoff_1 = require("../../utils/Backoff");
 var SSEHandler_1 = require("./SSEHandler");
 var MySegmentsUpdateWorker_1 = require("./UpdateWorkers/MySegmentsUpdateWorker");
 var SegmentsUpdateWorker_1 = require("./UpdateWorkers/SegmentsUpdateWorker");
-var SplitsUpdateWorker_1 = require("./UpdateWorkers/SplitsUpdateWorker");
+var DefinitionsUpdateWorker_1 = require("./UpdateWorkers/DefinitionsUpdateWorker");
 var AuthClient_1 = require("./AuthClient");
 var lang_1 = require("../../utils/lang");
 var SSEClient_1 = require("./SSEClient");
@@ -45,8 +45,8 @@ function pushManagerFactory(params, pollingManager) {
     // init workers
     // MySegmentsUpdateWorker (client-side) are initiated in `add` method
     var segmentsUpdateWorker = userKey ? undefined : (0, SegmentsUpdateWorker_1.SegmentsUpdateWorker)(log, pollingManager.segmentsSyncTask, storage.segments);
-    // For server-side we pass the segmentsSyncTask, used by SplitsUpdateWorker to fetch new segments
-    var splitsUpdateWorker = (0, SplitsUpdateWorker_1.SplitsUpdateWorker)(log, storage, pollingManager.splitsSyncTask, readiness.splits, telemetryTracker, userKey ? undefined : pollingManager.segmentsSyncTask);
+    // For server-side we pass the segmentsSyncTask, used by DefinitionsUpdateWorker to fetch new segments
+    var definitionsUpdateWorker = (0, DefinitionsUpdateWorker_1.DefinitionsUpdateWorker)(log, storage, pollingManager.definitionsSyncTask, readiness.definitions, telemetryTracker, userKey ? undefined : pollingManager.segmentsSyncTask);
     // [Only for client-side] map of hashes to user keys, to dispatch membership update events to the corresponding MySegmentsUpdateWorker
     var userKeyHashes = {};
     // [Only for client-side] map of user keys to their corresponding hash64 and MySegmentsUpdateWorkers.
@@ -54,10 +54,10 @@ function pushManagerFactory(params, pollingManager) {
     var clients = {};
     // [Only for client-side] variable to flag that a new client was added. It is needed to reconnect streaming.
     var connectForNewClient = false;
-    // flag that indicates if `stop/disconnectPush` was called, either by the SyncManager, when the client is destroyed, or due to a PUSH_NONRETRYABLE_ERROR error.
+    // flag that indicates if `stop/disconnectPush` was called, either by the SyncManager, when the client is destroyed, or due to a PUSH_NON_RETRYABLE_ERROR error.
     // It is used to halt the `connectPush` process if it was in progress.
     var disconnected;
-    // flag that indicates a PUSH_NONRETRYABLE_ERROR, condition with which starting pushManager again is ignored.
+    // flag that indicates a PUSH_NON_RETRYABLE_ERROR, condition with which starting pushManager again is ignored.
     // true if STREAMING_DISABLED control event, or 'pushEnabled: false', or non-recoverable SSE or Auth errors.
     var disabled; // `disabled` implies `disconnected === true`
     /** PushManager functions related to initialization */
@@ -96,11 +96,11 @@ function pushManagerFactory(params, pollingManager) {
         authenticate(userKeys).then(function (authData) {
             if (disconnected)
                 return;
-            // 'pushEnabled: false' is handled as a PUSH_NONRETRYABLE_ERROR instead of PUSH_SUBSYSTEM_DOWN, in order to
+            // 'pushEnabled: false' is handled as a PUSH_NON_RETRYABLE_ERROR instead of PUSH_SUBSYSTEM_DOWN, in order to
             // close the sseClient in case the org has been bloqued while the instance was connected to streaming
             if (!authData.pushEnabled) {
                 log.info(constants_2.STREAMING_DISABLED);
-                pushEmitter.emit(constants_1.PUSH_NONRETRYABLE_ERROR);
+                pushEmitter.emit(constants_1.PUSH_NON_RETRYABLE_ERROR);
                 return;
             }
             // [Only for client-side] don't open SSE connection if a new shared client was added, since it means that a new authentication is taking place
@@ -115,7 +115,7 @@ function pushManagerFactory(params, pollingManager) {
             // Handle 4XX HTTP errors: 401 (invalid SDK Key) or 400 (using incorrect SDK Key, i.e., client-side SDK Key on server-side)
             if (error.statusCode >= 400 && error.statusCode < 500) {
                 telemetryTracker.streamingEvent(constants_3.AUTH_REJECTION);
-                pushEmitter.emit(constants_1.PUSH_NONRETRYABLE_ERROR);
+                pushEmitter.emit(constants_1.PUSH_NON_RETRYABLE_ERROR);
                 return;
             }
             // Handle other HTTP and network errors as recoverable errors
@@ -139,7 +139,7 @@ function pushManagerFactory(params, pollingManager) {
     }
     // cancel scheduled fetch retries of Splits, Segments, and MySegments Update Workers
     function stopWorkers() {
-        splitsUpdateWorker.stop();
+        definitionsUpdateWorker.stop();
         if (userKey)
             (0, lang_1.forOwn)(clients, function (_a) {
                 var worker = _a.worker;
@@ -155,7 +155,7 @@ function pushManagerFactory(params, pollingManager) {
         connectPushRetryBackoff.reset();
     });
     /** Fallback to polling without retry due to: STREAMING_DISABLED control event, or 'pushEnabled: false', or non-recoverable SSE and Authentication errors */
-    pushEmitter.on(constants_1.PUSH_NONRETRYABLE_ERROR, function handleNonRetryableError() {
+    pushEmitter.on(constants_1.PUSH_NON_RETRYABLE_ERROR, function handleNonRetryableError() {
         disabled = true;
         // Note: `stopWorkers` is been called twice, but it is not harmful
         disconnectPush();
@@ -181,9 +181,9 @@ function pushManagerFactory(params, pollingManager) {
         connectPush();
     });
     /** Functions related to synchronization (Queues and Workers in the spec) */
-    pushEmitter.on(constants_1.SPLIT_KILL, splitsUpdateWorker.killSplit);
-    pushEmitter.on(constants_1.SPLIT_UPDATE, splitsUpdateWorker.put);
-    pushEmitter.on(constants_1.RB_SEGMENT_UPDATE, splitsUpdateWorker.put);
+    pushEmitter.on(constants_1.SPLIT_KILL, definitionsUpdateWorker.killDefinition);
+    pushEmitter.on(constants_1.SPLIT_UPDATE, definitionsUpdateWorker.put);
+    pushEmitter.on(constants_1.RB_SEGMENT_UPDATE, definitionsUpdateWorker.put);
     function handleMySegmentsUpdate(parsedData) {
         switch (parsedData.u) {
             case types_1.UpdateStrategy.BoundedFetchRequest: {
@@ -206,7 +206,7 @@ function pushManagerFactory(params, pollingManager) {
             case types_1.UpdateStrategy.KeyList: {
                 var keyList = void 0, added_1, removed_1;
                 try {
-                    keyList = (0, parseUtils_1.parseKeyList)(parsedData.d, parsedData.c);
+                    keyList = (0, parseUtils_1.parseCompressedData)(parsedData.d, parsedData.c);
                     added_1 = new Set(keyList.a);
                     removed_1 = new Set(keyList.r);
                 }

package/cjs/sync/submitters/impressionsSubmitter.js

@@ -7,14 +7,14 @@ var constants_1 = require("../../logger/constants");
 /**
  * Converts `impressions` data from cache into request payload.
  */
-function fromImpressionsCollector(sendLabels, data) {
+function fromImpressionsCollector(sendLabels, entityType, data) {
     var groupedByFeature = (0, lang_1.groupBy)(data, 'feature');
     var dto = [];
     (0, lang_1.forOwn)(groupedByFeature, function (value, name) {
         dto.push({
             f: name,
             i: value.map(function (entry) {
-                var keyImpression = {
+                return {
                     k: entry.keyName,
                     t: entry.treatment,
                     m: entry.time,
@@ -22,9 +22,9 @@ function fromImpressionsCollector(sendLabels, data) {
                     r: sendLabels ? entry.label : undefined,
                     b: entry.bucketingKey,
                     pt: entry.pt,
-                    properties: entry.properties // Properties
+                    properties: entry.properties,
+                    et: entityType, // Definition type
                 };
-                return keyImpression;
             })
         });
     });
@@ -35,9 +35,9 @@ exports.fromImpressionsCollector = fromImpressionsCollector;
  * Submitter that periodically posts impressions data
  */
 function impressionsSubmitterFactory(params) {
-    var _a = params.settings, log = _a.log, impressionsRefreshRate = _a.scheduler.impressionsRefreshRate, labelsEnabled = _a.core.labelsEnabled, postTestImpressionsBulk = params.splitApi.postTestImpressionsBulk, impressions = params.storage.impressions;
+    var _a = params.settings, log = _a.log, impressionsRefreshRate = _a.scheduler.impressionsRefreshRate, labelsEnabled = _a.core.labelsEnabled, postTestImpressionsBulk = params.splitApi.postTestImpressionsBulk, impressions = params.storage.impressions, entityType = params.entityType;
     // retry impressions only once.
-    var syncTask = (0, submitter_1.submitterFactory)(log, postTestImpressionsBulk, impressions, impressionsRefreshRate, fromImpressionsCollector.bind(undefined, labelsEnabled), 1);
+    var syncTask = (0, submitter_1.submitterFactory)(log, postTestImpressionsBulk, impressions, impressionsRefreshRate, fromImpressionsCollector.bind(undefined, labelsEnabled, entityType), 1);
     // register impressions submitter to be executed when impressions cache is full
     impressions.setOnFullQueueCb(function () {
         if (syncTask.isRunning()) {

package/cjs/sync/submitters/submitter.js

@@ -16,7 +16,7 @@ function submitterFactory(log, postClient, sourceCache, postRate, fromCacheToPay
         if (sourceCache.isEmpty() && !data)
             return Promise.resolve();
         data = sourceCache.pop(data);
-        var dataCountMessage = typeof data.length === 'number' ? data.length + " " + dataName : dataName;
+        var dataCountMessage = typeof data.length === 'number' ? "".concat(data.length, " ").concat(dataName) : dataName;
         log[debugLogs ? 'debug' : 'info'](constants_1.SUBMITTERS_PUSH, [dataCountMessage]);
         var jsonPayload = JSON.stringify(fromCacheToPayload ? fromCacheToPayload(data) : data);
         if (!maxRetries)

package/cjs/sync/submitters/telemetrySubmitter.js

@@ -107,13 +107,12 @@ function telemetrySubmitterFactory(params) {
     var telemetry = params.storage.telemetry, now = params.platform.now;
     if (!telemetry || !now)
         return; // No submitter created if telemetry cache is not defined
-    var settings = params.settings, _a = params.settings, log = _a.log, telemetryRefreshRate = _a.scheduler.telemetryRefreshRate, splitApi = params.splitApi, readiness = params.readiness, sdkReadinessManager = params.sdkReadinessManager;
+    var settings = params.settings, _a = params.settings, log = _a.log, telemetryRefreshRate = _a.scheduler.telemetryRefreshRate, splitApi = params.splitApi, readiness = params.readiness;
     var stopTimer = (0, timer_1.timer)(now);
     var submitter = (0, submitter_1.firstPushWindowDecorator)((0, submitter_1.submitterFactory)(log, splitApi.postMetricsUsage, telemetry, telemetryRefreshRate, undefined, 0, true), telemetryRefreshRate);
     readiness.gate.once(constants_2.SDK_READY_FROM_CACHE, function () {
         telemetry.recordTimeUntilReadyFromCache(stopTimer());
     });
-    sdkReadinessManager.incInternalReadyCbCount();
     readiness.gate.once(constants_2.SDK_READY, function () {
         telemetry.recordTimeUntilReady(stopTimer());
         // Post config data when the SDK is ready and if the telemetry submitter was started

package/cjs/sync/syncManagerOnline.js

@@ -7,7 +7,8 @@ var constants_2 = require("../logger/constants");
 var consent_1 = require("../consent");
 var constants_3 = require("../utils/constants");
 var constants_4 = require("../readiness/constants");
-var AbstractSplitsCacheSync_1 = require("../storages/AbstractSplitsCacheSync");
+var AbstractDefinitionsCacheSync_1 = require("../storages/AbstractDefinitionsCacheSync");
+var splitChangesFetcher_1 = require("./polling/fetchers/splitChangesFetcher");
 /**
  * Online SyncManager factory.
  * Can be used for server-side API, and client-side API with or without multiple clients.
@@ -15,15 +16,17 @@ var AbstractSplitsCacheSync_1 = require("../storages/AbstractSplitsCacheSync");
  * @param pollingManagerFactory - allows to specialize the SyncManager for server-side or client-side API by passing
  * `pollingManagerSSFactory` or `pollingManagerCSFactory` respectively.
  * @param pushManagerFactory - optional to build a SyncManager with or without streaming support
+ * @param definitionChangesFetcherFactory - optional to replace the default split changes fetcher
  */
-function syncManagerOnlineFactory(pollingManagerFactory, pushManagerFactory) {
+function syncManagerOnlineFactory(pollingManagerFactory, pushManagerFactory, definitionChangesFetcherFactory) {
+    if (definitionChangesFetcherFactory === void 0) { definitionChangesFetcherFactory = splitChangesFetcher_1.splitChangesFetcherFactory; }
     /**
      * SyncManager factory for modular SDK
      */
     return function (params) {
         var settings = params.settings, _a = params.settings, log = _a.log, streamingEnabled = _a.streamingEnabled, syncEnabled = _a.sync.enabled, telemetryTracker = params.telemetryTracker, storage = params.storage, readiness = params.readiness;
         /** Polling Manager */
-        var pollingManager = pollingManagerFactory && pollingManagerFactory(params);
+        var pollingManager = pollingManagerFactory && pollingManagerFactory(params, definitionChangesFetcherFactory(params));
         /** Push Manager */
         var pushManager = syncEnabled && streamingEnabled && pollingManager && pushManagerFactory ?
             pushManagerFactory(params, pollingManager) :
@@ -77,7 +80,7 @@ function syncManagerOnlineFactory(pollingManagerFactory, pushManagerFactory) {
                     if (startFirstTime) {
                         // Emits SDK_READY_FROM_CACHE
                         if (!cacheMetadata.initialCacheLoad) {
-                            readiness.splits.emit(constants_4.SDK_SPLITS_CACHE_LOADED, cacheMetadata);
+                            readiness.definitions.emit(constants_4.SDK_DEFINITIONS_CACHE_LOADED, cacheMetadata);
                         }
                     }
                     // start syncing splits and segments
@@ -136,7 +139,7 @@ function syncManagerOnlineFactory(pollingManagerFactory, pushManagerFactory) {
                         if (pushManager) {
                             if (pollingManager.isRunning()) {
                                 // if doing polling, we must start the periodic fetch of data
-                                if ((0, AbstractSplitsCacheSync_1.usesSegmentsSync)(storage))
+                                if ((0, AbstractDefinitionsCacheSync_1.usesSegmentsSync)(storage))
                                     mySegmentsSyncTask.start();
                             }
                             else {
@@ -146,7 +149,7 @@ function syncManagerOnlineFactory(pollingManagerFactory, pushManagerFactory) {
                             }
                         }
                         else {
-                            if ((0, AbstractSplitsCacheSync_1.usesSegmentsSync)(storage))
+                            if ((0, AbstractDefinitionsCacheSync_1.usesSegmentsSync)(storage))
                                 mySegmentsSyncTask.start();
                         }
                     }

package/cjs/trackers/eventTracker.js

@@ -12,26 +12,24 @@ var mode_1 = require("../utils/settingsValidation/mode");
  * @param eventsCache - cache to save events
  * @param integrationsManager - optional event handler used for integrations
  */
-function eventTrackerFactory(settings, eventsCache, whenInit, integrationsManager, telemetryCache) {
+function eventTrackerFactory(settings, eventsCache, integrationsManager, telemetryCache) {
     var log = settings.log, mode = settings.mode;
     var isAsync = (0, mode_1.isConsumerMode)(mode);
     function queueEventsCallback(eventData, tracked) {
         var eventTypeId = eventData.eventTypeId, trafficTypeName = eventData.trafficTypeName, key = eventData.key, value = eventData.value, timestamp = eventData.timestamp, properties = eventData.properties;
         // Logging every prop would be too much.
-        var msg = "event of type \"" + eventTypeId + "\" for traffic type \"" + trafficTypeName + "\". Key: " + key + ". Value: " + value + ". Timestamp: " + timestamp + ". " + (properties ? 'With properties.' : 'With no properties.');
+        var msg = "event of type \"".concat(eventTypeId, "\" for traffic type \"").concat(trafficTypeName, "\". Key: ").concat(key, ". Value: ").concat(value, ". Timestamp: ").concat(timestamp, ". ").concat(properties ? 'With properties.' : 'With no properties.');
         if (tracked) {
             log.info(constants_1.EVENTS_TRACKER_SUCCESS, [msg]);
             if (integrationsManager) {
-                whenInit(function () {
-                    // Wrap in a timeout because we don't want it to be blocking.
-                    setTimeout(function () {
-                        // copy of event, to avoid unexpected behavior if modified by integrations
-                        var eventDataCopy = (0, objectAssign_1.objectAssign)({}, eventData);
-                        if (properties)
-                            eventDataCopy.properties = (0, objectAssign_1.objectAssign)({}, properties);
-                        // integrationsManager does not throw errors (they are internally handled by each integration module)
-                        integrationsManager.handleEvent(eventDataCopy);
-                    });
+                // Wrap in a timeout because we don't want it to be blocking.
+                setTimeout(function () {
+                    // copy of event, to avoid unexpected behavior if modified by integrations
+                    var eventDataCopy = (0, objectAssign_1.objectAssign)({}, eventData);
+                    if (properties)
+                        eventDataCopy.properties = (0, objectAssign_1.objectAssign)({}, properties);
+                    // integrationsManager does not throw errors (they are internally handled by each integration module)
+                    integrationsManager.handleEvent(eventDataCopy);
                 });
             }
         }

package/cjs/trackers/impressionObserver/buildKey.js

@@ -2,6 +2,6 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.buildKey = void 0;
 function buildKey(impression) {
-    return impression.keyName + ":" + impression.feature + ":" + impression.treatment + ":" + impression.label + ":" + impression.changeNumber;
+    return "".concat(impression.keyName, ":").concat(impression.feature, ":").concat(impression.treatment, ":").concat(impression.label, ":").concat(impression.changeNumber);
 }
 exports.buildKey = buildKey;

package/cjs/trackers/impressionsTracker.js

@@ -5,12 +5,31 @@ var objectAssign_1 = require("../utils/lang/objectAssign");
 var thenable_1 = require("../utils/promise/thenable");
 var constants_1 = require("../logger/constants");
 var constants_2 = require("../utils/constants");
+var strategyDebug_1 = require("./strategy/strategyDebug");
+var strategyNone_1 = require("./strategy/strategyNone");
+var strategyOptimized_1 = require("./strategy/strategyOptimized");
+var uniqueKeysTracker_1 = require("./uniqueKeysTracker");
 /**
  * Impressions tracker stores impressions in cache and pass them to the listener and integrations manager if provided.
  */
-function impressionsTrackerFactory(settings, impressionsCache, noneStrategy, strategy, whenInit, integrationsManager, telemetryCache) {
-    var log = settings.log, impressionListener = settings.impressionListener, _a = settings.runtime, ip = _a.ip, hostname = _a.hostname, version = settings.version;
+function impressionsTrackerFactory(params, storage, integrationsManager) {
+    var settings = params.settings, impressionsObserverFactory = params.impressionsObserverFactory, filterAdapterFactory = params.filterAdapterFactory;
+    var log = settings.log, impressionListener = settings.impressionListener, _a = settings.runtime, ip = _a.ip, hostname = _a.hostname, version = settings.version, impressionsMode = settings.sync.impressionsMode;
+    var observer = impressionsObserverFactory();
+    var uniqueKeysTracker = (0, uniqueKeysTracker_1.uniqueKeysTrackerFactory)(log, storage.uniqueKeys, filterAdapterFactory && filterAdapterFactory());
+    var noneStrategy = (0, strategyNone_1.strategyNoneFactory)(storage.impressionCounts, uniqueKeysTracker);
+    var strategy = impressionsMode === constants_2.OPTIMIZED ?
+        (0, strategyOptimized_1.strategyOptimizedFactory)(observer, storage.impressionCounts) :
+        impressionsMode === constants_2.DEBUG ?
+            (0, strategyDebug_1.strategyDebugFactory)(observer) :
+            noneStrategy;
     return {
+        start: function () {
+            uniqueKeysTracker.start();
+        },
+        stop: function () {
+            uniqueKeysTracker.stop();
+        },
         track: function (impressions, attributes) {
             if (settings.userConsent === constants_2.CONSENT_DECLINED)
                 return;
@@ -23,7 +42,7 @@ function impressionsTrackerFactory(settings, impressionsCache, noneStrategy, str
             var impressionsLength = impressions.length;
             var impressionsToStoreLength = impressionsToStore.length;
             if (impressionsToStoreLength) {
-                var res = impressionsCache.track(impressionsToStore.map(function (item) { return item.imp; }));
+                var res = storage.impressions.track(impressionsToStore.map(function (item) { return item.imp; }));
                 // If we're on an async storage, handle error and log it.
                 if ((0, thenable_1.thenable)(res)) {
                     res.then(function () {
@@ -35,9 +54,9 @@ function impressionsTrackerFactory(settings, impressionsCache, noneStrategy, str
                 else {
                     // Record when impressionsCache is sync only (standalone mode)
                     // @TODO we are not dropping impressions on full queue yet, so DROPPED stats are not recorded
-                    if (telemetryCache) {
-                        telemetryCache.recordImpressionStats(constants_2.QUEUED, impressionsToStoreLength);
-                        telemetryCache.recordImpressionStats(constants_2.DEDUPED, impressionsLength - impressionsToStoreLength);
+                    if (storage.telemetry) {
+                        storage.telemetry.recordImpressionStats(constants_2.QUEUED, impressionsToStoreLength);
+                        storage.telemetry.recordImpressionStats(constants_2.DEDUPED, impressionsLength - impressionsToStoreLength);
                     }
                 }
             }
@@ -52,20 +71,18 @@ function impressionsTrackerFactory(settings, impressionsCache, noneStrategy, str
                         hostname: hostname,
                         sdkLanguageVersion: version
                     };
-                    whenInit(function () {
-                        // Wrap in a timeout because we don't want it to be blocking.
-                        setTimeout(function () {
-                            // integrationsManager.handleImpression does not throw errors
-                            if (integrationsManager)
-                                integrationsManager.handleImpression(impressionData);
-                            try { // @ts-ignore. An exception on the listeners should not break the SDK.
-                                if (impressionListener)
-                                    impressionListener.logImpression(impressionData);
-                            }
-                            catch (err) {
-                                log.error(constants_1.ERROR_IMPRESSIONS_LISTENER, [err]);
-                            }
-                        });
+                    // Wrap in a timeout because we don't want it to be blocking.
+                    setTimeout(function () {
+                        // integrationsManager.handleImpression does not throw errors
+                        if (integrationsManager)
+                            integrationsManager.handleImpression(impressionData);
+                        try { // @ts-ignore. An exception on the listeners should not break the SDK.
+                            if (impressionListener)
+                                impressionListener.logImpression(impressionData);
+                        }
+                        catch (err) {
+                            log.error(constants_1.ERROR_IMPRESSIONS_LISTENER, [err]);
+                        }
                     });
                 };
                 for (var i = 0; i < impressionsLength; i++) {