@splitsoftware/splitio-commons 2.11.1-rc.2 → 3.0.0-rc.0

package/cjs/sync/polling/pollingManagerCS.js

@@ -3,7 +3,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.pollingManagerCSFactory = void 0;
 var lang_1 = require("../../utils/lang");
 var mySegmentsSyncTask_1 = require("./syncTasks/mySegmentsSyncTask");
-var splitsSyncTask_1 = require("./syncTasks/splitsSyncTask");
+var definitionsSyncTask_1 = require("./syncTasks/definitionsSyncTask");
 var key_1 = require("../../utils/key");
 var constants_1 = require("../../readiness/constants");
 var constants_2 = require("../../logger/constants");
@@ -12,10 +12,10 @@ var AbstractSplitsCacheSync_1 = require("../../storages/AbstractSplitsCacheSync"
  * Expose start / stop mechanism for polling data from services.
  * For client-side API with multiple clients.
  */
-function pollingManagerCSFactory(params) {
+function pollingManagerCSFactory(params, definitionChangesFetcher) {
     var splitApi = params.splitApi, storage = params.storage, readiness = params.readiness, settings = params.settings;
     var log = settings.log;
-    var splitsSyncTask = (0, splitsSyncTask_1.splitsSyncTaskFactory)(splitApi.fetchSplitChanges, storage, readiness, settings, true);
+    var definitionsSyncTask = (0, definitionsSyncTask_1.definitionsSyncTaskFactory)(definitionChangesFetcher, storage, readiness, settings, true);
     // Map of matching keys to their corresponding MySegmentsSyncTask.
     var mySegmentsSyncTasks = {};
     var matchingKey = (0, key_1.getMatching)(settings.core.key);
@@ -33,7 +33,7 @@ function pollingManagerCSFactory(params) {
     }
     // smart pausing
     readiness.splits.on(constants_1.SDK_SPLITS_ARRIVED, function () {
-        if (!splitsSyncTask.isRunning())
+        if (!definitionsSyncTask.isRunning())
             return; // noop if not doing polling
         var usingSegments = (0, AbstractSplitsCacheSync_1.usesSegmentsSync)(storage);
         if (usingSegments !== mySegmentsSyncTask.isRunning()) {
@@ -61,27 +61,27 @@ function pollingManagerCSFactory(params) {
         return mySegmentsSyncTask;
     }
     return {
-        splitsSyncTask: splitsSyncTask,
+        definitionsSyncTask: definitionsSyncTask,
         segmentsSyncTask: mySegmentsSyncTask,
         // Start periodic fetching (polling)
         start: function () {
             log.info(constants_2.POLLING_START);
-            splitsSyncTask.start();
+            definitionsSyncTask.start();
             if ((0, AbstractSplitsCacheSync_1.usesSegmentsSync)(storage))
                 startMySegmentsSyncTasks();
         },
         // Stop periodic fetching (polling)
         stop: function () {
             log.info(constants_2.POLLING_STOP);
-            if (splitsSyncTask.isRunning())
-                splitsSyncTask.stop();
+            if (definitionsSyncTask.isRunning())
+                definitionsSyncTask.stop();
             stopMySegmentsSyncTasks();
         },
         // Used by SyncManager to know if running in polling mode.
-        isRunning: splitsSyncTask.isRunning,
-        // fetch splits and segments
+        isRunning: definitionsSyncTask.isRunning,
+        // fetch definitions and segments
         syncAll: function () {
-            var promises = [splitsSyncTask.execute()];
+            var promises = [definitionsSyncTask.execute()];
             (0, lang_1.forOwn)(mySegmentsSyncTasks, function (mySegmentsSyncTask) {
                 promises.push(mySegmentsSyncTask.execute());
             });

package/cjs/sync/polling/pollingManagerSS.js

@@ -1,29 +1,29 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.pollingManagerSSFactory = void 0;
-var splitsSyncTask_1 = require("./syncTasks/splitsSyncTask");
+var definitionsSyncTask_1 = require("./syncTasks/definitionsSyncTask");
 var segmentsSyncTask_1 = require("./syncTasks/segmentsSyncTask");
 var constants_1 = require("../../logger/constants");
 /**
  * Expose start / stop mechanism for pulling data from services.
  */
-function pollingManagerSSFactory(params) {
+function pollingManagerSSFactory(params, definitionChangesFetcher) {
     var splitApi = params.splitApi, storage = params.storage, readiness = params.readiness, settings = params.settings;
     var log = settings.log;
-    var splitsSyncTask = (0, splitsSyncTask_1.splitsSyncTaskFactory)(splitApi.fetchSplitChanges, storage, readiness, settings);
+    var definitionsSyncTask = (0, definitionsSyncTask_1.definitionsSyncTaskFactory)(definitionChangesFetcher, storage, readiness, settings);
     var segmentsSyncTask = (0, segmentsSyncTask_1.segmentsSyncTaskFactory)(splitApi.fetchSegmentChanges, storage, readiness, settings);
     return {
-        splitsSyncTask: splitsSyncTask,
+        definitionsSyncTask: definitionsSyncTask,
         segmentsSyncTask: segmentsSyncTask,
         // Start periodic fetching (polling)
         start: function () {
             log.info(constants_1.POLLING_START);
-            log.debug(constants_1.LOG_PREFIX_SYNC_POLLING + ("Splits will be refreshed each " + settings.scheduler.featuresRefreshRate + " millis"));
-            log.debug(constants_1.LOG_PREFIX_SYNC_POLLING + ("Segments will be refreshed each " + settings.scheduler.segmentsRefreshRate + " millis"));
-            var startingUp = splitsSyncTask.start();
+            log.debug(constants_1.LOG_PREFIX_SYNC_POLLING + "".concat(definitionChangesFetcher.type, " will be refreshed each ").concat(settings.scheduler.featuresRefreshRate, " millis"));
+            log.debug(constants_1.LOG_PREFIX_SYNC_POLLING + "segments will be refreshed each ".concat(settings.scheduler.segmentsRefreshRate, " millis"));
+            var startingUp = definitionsSyncTask.start();
             if (startingUp) {
                 startingUp.then(function () {
-                    if (splitsSyncTask.isRunning())
+                    if (definitionsSyncTask.isRunning())
                         segmentsSyncTask.start();
                 });
             }
@@ -31,16 +31,16 @@ function pollingManagerSSFactory(params) {
         // Stop periodic fetching (polling)
         stop: function () {
             log.info(constants_1.POLLING_STOP);
-            if (splitsSyncTask.isRunning())
-                splitsSyncTask.stop();
+            if (definitionsSyncTask.isRunning())
+                definitionsSyncTask.stop();
             if (segmentsSyncTask.isRunning())
                 segmentsSyncTask.stop();
         },
         // Used by SyncManager to know if running in polling mode.
-        isRunning: splitsSyncTask.isRunning,
+        isRunning: definitionsSyncTask.isRunning,
         syncAll: function () {
-            // fetch splits and segments. There is no need to catch this promise (`SplitChangesUpdater` is always resolved with a boolean value)
-            return splitsSyncTask.execute().then(function () {
+            // fetch definitions and segments. There is no need to catch this promise (`DefinitionChangesUpdater` is always resolved with a boolean value)
+            return definitionsSyncTask.execute().then(function () {
                 return segmentsSyncTask.execute();
             });
         }

package/cjs/sync/polling/syncTasks/definitionsSyncTask.js

@@ -0,0 +1,12 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.definitionsSyncTaskFactory = void 0;
+var syncTask_1 = require("../../syncTask");
+var definitionChangesUpdater_1 = require("../updaters/definitionChangesUpdater");
+/**
+ * Creates a sync task that periodically executes a `definitionChangesUpdater` task
+ */
+function definitionsSyncTaskFactory(definitionChangesFetcher, storage, readiness, settings, isClientSide) {
+    return (0, syncTask_1.syncTaskFactory)(settings.log, (0, definitionChangesUpdater_1.definitionChangesUpdaterFactory)(settings.log, definitionChangesFetcher, storage, settings.sync.__splitFiltersValidation, readiness.splits, settings.startup.requestTimeoutBeforeReady, settings.startup.retriesOnFailureBeforeReady, isClientSide), settings.scheduler.featuresRefreshRate, 'definitionChangesUpdater');
+}
+exports.definitionsSyncTaskFactory = definitionsSyncTaskFactory;

package/cjs/sync/polling/updaters/{splitChangesUpdater.js → definitionChangesUpdater.js}

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.splitChangesUpdaterFactory = exports.computeMutation = exports.parseSegments = void 0;
+exports.definitionChangesUpdaterFactory = exports.computeMutation = exports.parseSegments = void 0;
 var timeout_1 = require("../../../utils/promise/timeout");
 var constants_1 = require("../../../readiness/constants");
 var constants_2 = require("../../../logger/constants");
@@ -24,7 +24,7 @@ function checkAllSegmentsExist(segments) {
  */
 function parseSegments(ruleEntity, matcherType) {
     if (matcherType === void 0) { matcherType = constants_3.IN_SEGMENT; }
-    var _a = ruleEntity, _b = _a.conditions, conditions = _b === void 0 ? [] : _b, excluded = _a.excluded;
+    var _a = ruleEntity, conditions = _a.conditions, excluded = _a.excluded;
     var segments = new Set();
     if (excluded && excluded.segments) {
         excluded.segments.forEach(function (_a) {
@@ -34,122 +34,130 @@ function parseSegments(ruleEntity, matcherType) {
             }
         });
     }
-    for (var i = 0; i < conditions.length; i++) {
-        var matchers = conditions[i].matcherGroup.matchers;
-        matchers.forEach(function (matcher) {
-            if (matcher.matcherType === matcherType)
-                segments.add(matcher.userDefinedSegmentMatcherData.segmentName);
-        });
+    if (conditions) {
+        for (var i = 0; i < conditions.length; i++) {
+            var matchers = conditions[i].matcherGroup.matchers;
+            matchers.forEach(function (matcher) {
+                if (matcher.matcherType === matcherType)
+                    segments.add(matcher.userDefinedSegmentMatcherData.segmentName);
+            });
+        }
     }
     return segments;
 }
 exports.parseSegments = parseSegments;
 /**
- * If there are defined filters and one feature flag doesn't match with them, its status is changed to 'ARCHIVE' to avoid storing it
+ * If there are defined filters and one definition doesn't match with them, its status is changed to 'ARCHIVE' to avoid storing it
  * If there is `bySet` filter, `byName` and `byPrefix` filters are ignored
  *
- * @param featureFlag - feature flag to be evaluated
+ * @param definition - definition to be evaluated
  * @param filters - splitFiltersValidation bySet | byName
  */
-function matchFilters(featureFlag, filters) {
+function matchFilters(definition, filters) {
     var _a = filters.groupedFilters, setsFilter = _a.bySet, namesFilter = _a.byName, prefixFilter = _a.byPrefix;
     if (setsFilter.length > 0)
-        return featureFlag.sets && featureFlag.sets.some(function (featureFlagSet) { return setsFilter.indexOf(featureFlagSet) > -1; });
+        return definition.sets && definition.sets.some(function (definitionSet) { return setsFilter.indexOf(definitionSet) > -1; });
     var namesFilterConfigured = namesFilter.length > 0;
     var prefixFilterConfigured = prefixFilter.length > 0;
     if (!namesFilterConfigured && !prefixFilterConfigured)
         return true;
-    var matchNames = namesFilterConfigured && namesFilter.indexOf(featureFlag.name) > -1;
-    var matchPrefix = prefixFilterConfigured && prefixFilter.some(function (prefix) { return (0, lang_1.startsWith)(featureFlag.name, prefix); });
+    var matchNames = namesFilterConfigured && namesFilter.indexOf(definition.name) > -1;
+    var matchPrefix = prefixFilterConfigured && prefixFilter.some(function (prefix) { return (0, lang_1.startsWith)(definition.name, prefix); });
     return matchNames || matchPrefix;
 }
 /**
- * Given the list of splits from /splitChanges endpoint, it returns the mutations,
- * i.e., an object with added splits, removed splits and used segments.
+ * Given the list of definitions from /splitChanges or /configs endpoint, it returns the mutations,
+ * i.e., an object with added definitions, removed definitions, and used segments.
  * Exported for testing purposes.
  */
-function computeMutation(rules, segments, filters) {
-    return rules.reduce(function (accum, ruleEntity) {
-        if (ruleEntity.status === 'ACTIVE' && (!filters || matchFilters(ruleEntity, filters))) {
+function computeMutation(update, segments, filters) {
+    return update.updated.reduce(function (accum, ruleEntity) {
+        if (!filters || matchFilters(ruleEntity, filters)) {
             accum.added.push(ruleEntity);
             parseSegments(ruleEntity).forEach(function (segmentName) {
                 segments.add(segmentName);
             });
         }
         else {
-            accum.removed.push(ruleEntity);
+            accum.removed.push(ruleEntity.name);
         }
         accum.names.push(ruleEntity.name);
         return accum;
-    }, { added: [], removed: [], names: [] });
+    }, { added: [], removed: update.removed, names: update.removed.slice() });
 }
 exports.computeMutation = computeMutation;
+function convertInstantUpdateToDefinitionChanges(instantUpdate) {
+    var payload = instantUpdate.payload, changeNumber = instantUpdate.changeNumber;
+    return payload.status === 'ARCHIVED' ?
+        { till: changeNumber, updated: [], removed: [payload.name] } :
+        { till: changeNumber, updated: [payload], removed: [] };
+}
 /**
- * factory of SplitChanges updater, a task that:
- *  - fetches split changes using `splitChangesFetcher`
- *  - updates `splitsCache`
- *  - uses `splitsEventEmitter` to emit events related to split data updates
+ * Factory of DefinitionChanges updater, a task that:
+ *  - fetches definition changes using `definitionChangesFetcher`
+ *  - updates definitions storage
+ *  - uses `definitionsEventEmitter` to emit events related to definition data updates
  *
  * @param log -  Logger instance
- * @param splitChangesFetcher -  Fetcher of `/splitChanges`
- * @param splits -  Splits storage, with sync or async methods
+ * @param definitionChangesFetcher -  Fetcher of `/splitChanges` or `/configs`
+ * @param definitions -  Definitions storage, with sync or async methods
  * @param segments -  Segments storage, with sync or async methods
- * @param splitsEventEmitter -  Optional readiness manager. Not required for synchronizer or producer mode.
+ * @param definitionsEventEmitter -  Optional readiness manager. Not required for synchronizer or producer mode.
  * @param requestTimeoutBeforeReady -  How long the updater will wait for the request to timeout. Default 0, i.e., never timeout.
- * @param retriesOnFailureBeforeReady -  How many retries on `/splitChanges` we the updater do in case of failure or timeout. Default 0, i.e., no retries.
+ * @param retriesOnFailureBeforeReady -  How many retries on `/splitChanges` or `/configs` we the updater do in case of failure or timeout. Default 0, i.e., no retries.
  */
-function splitChangesUpdaterFactory(log, splitChangesFetcher, storage, splitFiltersValidation, splitsEventEmitter, requestTimeoutBeforeReady, retriesOnFailureBeforeReady, isClientSide) {
+function definitionChangesUpdaterFactory(log, definitionChangesFetcher, storage, splitFiltersValidation, splitsEventEmitter, requestTimeoutBeforeReady, retriesOnFailureBeforeReady, isClientSide) {
     if (requestTimeoutBeforeReady === void 0) { requestTimeoutBeforeReady = 0; }
     if (retriesOnFailureBeforeReady === void 0) { retriesOnFailureBeforeReady = 0; }
     var splits = storage.splits, rbSegments = storage.rbSegments, segments = storage.segments;
     var startingUp = true;
-    /** timeout decorator for `splitChangesFetcher` promise  */
+    /** timeout decorator for `definitionChangesFetcher` promise  */
     function _promiseDecorator(promise) {
         if (startingUp && requestTimeoutBeforeReady)
             promise = (0, timeout_1.timeout)(requestTimeoutBeforeReady, promise);
         return promise;
     }
     /**
-     * SplitChanges updater returns a promise that resolves with a `false` boolean value if it fails to fetch splits or synchronize them with the storage.
+     * DefinitionChanges updater returns a promise that resolves with a `false` boolean value if it fails to fetch definitions or synchronize them with the storage.
      * Returned promise will not be rejected.
      *
      * @param noCache - true to revalidate data to fetch
      * @param till - query param to bypass CDN requests
      */
-    return function splitChangesUpdater(noCache, till, instantUpdate) {
+    return function definitionChangesUpdater(noCache, till, instantUpdate) {
         /**
-         * @param since - current changeNumber at splitsCache
+         * @param since - current changeNumber at definitionsCache
          * @param retry - current number of retry attempts
          */
-        function _splitChangesUpdater(sinces, retry) {
+        function _definitionChangesUpdater(sinces, retry) {
             if (retry === void 0) { retry = 0; }
             var since = sinces[0], rbSince = sinces[1];
-            log.debug(constants_2.SYNC_SPLITS_FETCH, sinces);
+            log.debug(constants_2.SYNC_FETCH, [definitionChangesFetcher.type, since, rbSince]);
             return Promise.resolve(instantUpdate ?
                 instantUpdate.type === constants_4.SPLIT_UPDATE ?
-                    // IFFU edge case: a change to a flag that adds an IN_RULE_BASED_SEGMENT matcher that is not present yet
+                    // IFFU edge case: a change to definition that adds an IN_RULE_BASED_SEGMENT matcher that is not present yet
                     Promise.resolve(rbSegments.contains(parseSegments(instantUpdate.payload, constants_3.IN_RULE_BASED_SEGMENT))).then(function (contains) {
                         return contains ?
-                            { ff: { d: [instantUpdate.payload], t: instantUpdate.changeNumber } } :
-                            splitChangesFetcher(since, noCache, till, rbSince, _promiseDecorator);
+                            { d: convertInstantUpdateToDefinitionChanges(instantUpdate) } :
+                            definitionChangesFetcher(since, noCache, till, rbSince, _promiseDecorator);
                     }) :
-                    { rbs: { d: [instantUpdate.payload], t: instantUpdate.changeNumber } } :
-                splitChangesFetcher(since, noCache, till, rbSince, _promiseDecorator))
-                .then(function (splitChanges) {
+                    { rbs: convertInstantUpdateToDefinitionChanges(instantUpdate) } :
+                definitionChangesFetcher(since, noCache, till, rbSince, _promiseDecorator))
+                .then(function (definitionChanges) {
                 var usedSegments = new Set();
-                var updatedFlags = [];
+                var updatedDefinitions = [];
                 var ffUpdate = false;
-                if (splitChanges.ff) {
-                    var _a = computeMutation(splitChanges.ff.d, usedSegments, splitFiltersValidation), added = _a.added, removed = _a.removed, names = _a.names;
-                    updatedFlags = names;
-                    log.debug(constants_2.SYNC_SPLITS_UPDATE, [added.length, removed.length]);
-                    ffUpdate = splits.update(added, removed, splitChanges.ff.t);
+                if (definitionChanges.d) {
+                    var _a = computeMutation(definitionChanges.d, usedSegments, splitFiltersValidation), added = _a.added, removed = _a.removed, names = _a.names;
+                    updatedDefinitions = names;
+                    log.debug(constants_2.SYNC_UPDATE, [definitionChangesFetcher.type, added.length, removed.length]);
+                    ffUpdate = splits.update(added, removed, definitionChanges.d.till);
                 }
                 var rbsUpdate = false;
-                if (splitChanges.rbs) {
-                    var _b = computeMutation(splitChanges.rbs.d, usedSegments), added = _b.added, removed = _b.removed;
-                    log.debug(constants_2.SYNC_RBS_UPDATE, [added.length, removed.length]);
-                    rbsUpdate = rbSegments.update(added, removed, splitChanges.rbs.t);
+                if (definitionChanges.rbs) {
+                    var _b = computeMutation(definitionChanges.rbs, usedSegments), added = _b.added, removed = _b.removed;
+                    log.debug(constants_2.SYNC_UPDATE, ['rule-based segments', added.length, removed.length]);
+                    rbsUpdate = rbSegments.update(added, removed, definitionChanges.rbs.till);
                 }
                 return Promise.all([ffUpdate, rbsUpdate,
                     // @TODO if at least 1 segment fetch fails due to 404 and other segments are updated in the storage, SDK_UPDATE is not emitted
@@ -167,8 +175,8 @@ function splitChangesUpdaterFactory(log, splitChangesFetcher, storage, splitFilt
                             // emit SDK events
                             if (emitSplitsArrivedEvent) {
                                 var metadata = {
-                                    type: updatedFlags.length > 0 ? constants_1.FLAGS_UPDATE : constants_1.SEGMENTS_UPDATE,
-                                    names: updatedFlags.length > 0 ? updatedFlags : []
+                                    type: updatedDefinitions.length > 0 ? definitionChangesFetcher.type === 'configs' ? constants_1.CONFIGS_UPDATE : constants_1.FLAGS_UPDATE : constants_1.SEGMENTS_UPDATE,
+                                    names: updatedDefinitions.length > 0 ? updatedDefinitions : []
                                 };
                                 splitsEventEmitter.emit(constants_1.SDK_SPLITS_ARRIVED, metadata);
                             }
@@ -181,18 +189,18 @@ function splitChangesUpdaterFactory(log, splitChangesFetcher, storage, splitFilt
                 .catch(function (error) {
                 if (startingUp && retriesOnFailureBeforeReady > retry) {
                     retry += 1;
-                    log.warn(constants_2.SYNC_SPLITS_FETCH_RETRY, [retry, error]);
-                    return _splitChangesUpdater(sinces, retry);
+                    log.warn(constants_2.SYNC_FETCH_RETRY, [definitionChangesFetcher.type, retry, error]);
+                    return _definitionChangesUpdater(sinces, retry);
                 }
                 else {
                     startingUp = false;
-                    log.warn(constants_2.SYNC_SPLITS_FETCH_FAILS, [error]);
+                    log.warn(constants_2.SYNC_FETCH_FAILS, [definitionChangesFetcher.type, error]);
                 }
                 return false;
             });
         }
         // `getChangeNumber` never rejects or throws error
-        return Promise.all([splits.getChangeNumber(), rbSegments.getChangeNumber()]).then(_splitChangesUpdater);
+        return Promise.all([splits.getChangeNumber(), rbSegments.getChangeNumber()]).then(_definitionChangesUpdater);
     };
 }
-exports.splitChangesUpdaterFactory = splitChangesUpdaterFactory;
+exports.definitionChangesUpdaterFactory = definitionChangesUpdaterFactory;

package/cjs/sync/polling/updaters/mySegmentsUpdater.js

@@ -56,7 +56,7 @@ function mySegmentsUpdaterFactory(log, mySegmentsFetcher, storage, segmentsEvent
         return updaterPromise.catch(function (error) {
             if (startingUp && retriesOnFailureBeforeReady > retry) {
                 retry += 1;
-                log.warn(constants_2.SYNC_MYSEGMENTS_FETCH_RETRY, [retry, error]);
+                log.warn(constants_2.SYNC_FETCH_RETRY, ['memberships', retry, error]);
                 return _mySegmentsUpdater(retry); // no need to forward `segmentList` and `noCache` params
             }
             else {

package/cjs/sync/polling/updaters/segmentChangesUpdater.js

@@ -23,7 +23,7 @@ function segmentChangesUpdaterFactory(log, segmentChangesFetcher, segments, read
         return promise;
     }
     function updateSegment(segmentName, noCache, till, fetchOnlyNew, retries) {
-        log.debug(constants_2.LOG_PREFIX_SYNC_SEGMENTS + "Processing segment " + segmentName);
+        log.debug("".concat(constants_2.LOG_PREFIX_SYNC, "Processing segment ").concat(segmentName));
         var sincePromise = Promise.resolve(segments.getChangeNumber(segmentName));
         return sincePromise.then(function (since) {
             // if fetchOnlyNew flag, avoid processing already fetched segments
@@ -31,14 +31,14 @@ function segmentChangesUpdaterFactory(log, segmentChangesFetcher, segments, read
                 false :
                 segmentChangesFetcher(since || -1, segmentName, noCache, till, _promiseDecorator).then(function (changes) {
                     return Promise.all(changes.map(function (x) {
-                        log.debug(constants_2.LOG_PREFIX_SYNC_SEGMENTS + "Processing " + segmentName + " with till = " + x.till + ". Added: " + x.added.length + ". Removed: " + x.removed.length);
+                        log.debug("".concat(constants_2.LOG_PREFIX_SYNC, "Processing ").concat(segmentName, " with till = ").concat(x.till, ". Added: ").concat(x.added.length, ". Removed: ").concat(x.removed.length));
                         return segments.update(segmentName, x.added, x.removed, x.till);
                     })).then(function (updates) {
                         return updates.some(function (update) { return update; });
                     });
                 }).catch(function (error) {
                     if (retries) {
-                        log.warn(constants_2.LOG_PREFIX_SYNC_SEGMENTS + "Retrying fetch of segment " + segmentName + " (attempt #" + retries + "). Reason: " + error);
+                        log.warn("".concat(constants_2.LOG_PREFIX_SYNC, "Retrying fetch of segment ").concat(segmentName, " (attempt #").concat(retries, "). Reason: ").concat(error));
                         return updateSegment(segmentName, noCache, till, fetchOnlyNew, retries - 1);
                     }
                     throw error;
@@ -57,7 +57,7 @@ function segmentChangesUpdaterFactory(log, segmentChangesFetcher, segments, read
      * @param till - till target for the provided segmentName, for CDN bypass.
      */
     return function segmentChangesUpdater(fetchOnlyNew, segmentName, noCache, till) {
-        log.debug(constants_2.LOG_PREFIX_SYNC_SEGMENTS + "Started segments update");
+        log.debug("".concat(constants_2.LOG_PREFIX_SYNC, "Started segments update"));
         // If not a segment name provided, read list of available segments names to be updated.
         var segmentsPromise = Promise.resolve(segmentName ? [segmentName] : segments.getRegisteredSegments());
         return segmentsPromise.then(function (segmentNames) {
@@ -84,10 +84,10 @@ function segmentChangesUpdaterFactory(log, segmentChangesFetcher, segments, read
                 // @TODO although factory status is destroyed, synchronization is not stopped
                 if (readiness)
                     readiness.setDestroyed();
-                log.error(constants_2.LOG_PREFIX_INSTANTIATION + ": you passed a client-side type authorizationKey, please grab an SDK Key from the Split user interface that is of type server-side.");
+                log.error("".concat(constants_2.LOG_PREFIX_INSTANTIATION, ": you passed a client-side type authorizationKey, please grab an SDK Key from Harness UI that is of type server-side."));
             }
             else {
-                log.warn(constants_2.LOG_PREFIX_SYNC_SEGMENTS + "Error while doing fetch of segments. " + error);
+                log.warn("".concat(constants_2.LOG_PREFIX_SYNC, "Error while doing fetch of segments. ").concat(error));
             }
             return false;
         });

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/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/UpdateWorkers/SplitsUpdateWorker.js

@@ -11,7 +11,7 @@ var constants_5 = require("./constants");
 /**
  * SplitsUpdateWorker factory
  */
-function SplitsUpdateWorker(log, storage, splitsSyncTask, splitsEventEmitter, telemetryTracker, segmentsSyncTask) {
+function SplitsUpdateWorker(log, storage, definitionsSyncTask, splitsEventEmitter, telemetryTracker, segmentsSyncTask) {
     var ff = SplitsUpdateWorker(storage.splits);
     var rbs = SplitsUpdateWorker(storage.rbSegments);
     function SplitsUpdateWorker(cache) {
@@ -26,7 +26,7 @@ function SplitsUpdateWorker(log, storage, splitsSyncTask, splitsEventEmitter, te
             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) {
@@ -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,7 +49,7 @@ 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 {

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

@@ -46,7 +46,7 @@ function pushManagerFactory(params, pollingManager) {
     // 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);
+    var splitsUpdateWorker = (0, SplitsUpdateWorker_1.SplitsUpdateWorker)(log, storage, pollingManager.definitionsSyncTask, readiness.splits, 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.
@@ -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);
                 }