@splitsoftware/splitio-commons 1.8.2-rc.0 → 1.8.2-rc.2

package/cjs/storages/inRedis/SplitsCacheInRedis.js

@@ -38,19 +38,15 @@ var SplitsCacheInRedis = /** @class */ (function (_super) {
     }
     SplitsCacheInRedis.prototype._decrementCounts = function (split) {
         var _this = this;
-        if (split.trafficTypeName) {
-            var ttKey_1 = this.keys.buildTrafficTypeKey(split.trafficTypeName);
-            return this.redis.decr(ttKey_1).then(function (count) {
-                if (count === 0)
-                    return _this.redis.del(ttKey_1);
-            });
-        }
+        var ttKey = this.keys.buildTrafficTypeKey(split.trafficTypeName);
+        return this.redis.decr(ttKey).then(function (count) {
+            if (count === 0)
+                return _this.redis.del(ttKey);
+        });
     };
     SplitsCacheInRedis.prototype._incrementCounts = function (split) {
-        if (split.trafficTypeName) {
-            var ttKey = this.keys.buildTrafficTypeKey(split.trafficTypeName);
-            return this.redis.incr(ttKey);
-        }
+        var ttKey = this.keys.buildTrafficTypeKey(split.trafficTypeName);
+        return this.redis.incr(ttKey);
     };
     /**
      * Add a given split.
@@ -68,18 +64,19 @@ var SplitsCacheInRedis = /** @class */ (function (_super) {
                 newStringifiedSplit = JSON.stringify(split);
             }
             catch (e) {
-                throw new Error('Error parsing split definition: ' + e);
+                throw new Error('Error parsing feature flag definition: ' + e);
             }
-            return Promise.all([
-                _this.redis.set(splitKey, newStringifiedSplit),
-                _this._incrementCounts(split),
-                // If it's an update, we decrement the traffic type of the existing split,
-                parsedPreviousSplit && _this._decrementCounts(parsedPreviousSplit)
-            ]);
-        }).then(function (_a) {
-            var status = _a[0];
-            return status === 'OK';
-        });
+            return _this.redis.set(splitKey, newStringifiedSplit).then(function () {
+                // avoid unnecessary increment/decrement operations
+                if (parsedPreviousSplit && parsedPreviousSplit.trafficTypeName === split.trafficTypeName)
+                    return;
+                // update traffic type counts
+                return _this._incrementCounts(split).then(function () {
+                    if (parsedPreviousSplit)
+                        return _this._decrementCounts(parsedPreviousSplit);
+                });
+            });
+        }).then(function () { return true; });
     };
     /**
      * Add a list of splits.
@@ -229,7 +226,7 @@ var SplitsCacheInRedis = /** @class */ (function (_super) {
             return Promise.resolve(splits);
         })
             .catch(function (e) {
-            _this.log.error(constants_1.LOG_PREFIX + ("Could not grab splits due to an error: " + e + "."));
+            _this.log.error(constants_1.LOG_PREFIX + ("Could not grab feature flags due to an error: " + e + "."));
             return Promise.reject(e);
         });
     };

package/cjs/storages/pluggable/SplitsCachePluggable.js

@@ -25,19 +25,15 @@ var SplitsCachePluggable = /** @class */ (function (_super) {
     }
     SplitsCachePluggable.prototype._decrementCounts = function (split) {
         var _this = this;
-        if (split.trafficTypeName) {
-            var ttKey_1 = this.keys.buildTrafficTypeKey(split.trafficTypeName);
-            return this.wrapper.decr(ttKey_1).then(function (count) {
-                if (count === 0)
-                    return _this.wrapper.del(ttKey_1);
-            });
-        }
+        var ttKey = this.keys.buildTrafficTypeKey(split.trafficTypeName);
+        return this.wrapper.decr(ttKey).then(function (count) {
+            if (count === 0)
+                return _this.wrapper.del(ttKey);
+        });
     };
     SplitsCachePluggable.prototype._incrementCounts = function (split) {
-        if (split.trafficTypeName) {
-            var ttKey = this.keys.buildTrafficTypeKey(split.trafficTypeName);
-            return this.wrapper.incr(ttKey);
-        }
+        var ttKey = this.keys.buildTrafficTypeKey(split.trafficTypeName);
+        return this.wrapper.incr(ttKey);
     };
     /**
      * Add a given split.
@@ -55,14 +51,18 @@ var SplitsCachePluggable = /** @class */ (function (_super) {
                 stringifiedNewSplit = JSON.stringify(split);
             }
             catch (e) {
-                throw new Error('Error parsing split definition: ' + e);
+                throw new Error('Error parsing feature flag definition: ' + e);
             }
-            return Promise.all([
-                _this.wrapper.set(splitKey, stringifiedNewSplit),
-                _this._incrementCounts(split),
-                // If it's an update, we decrement the traffic type and segment count of the existing split,
-                parsedPreviousSplit && _this._decrementCounts(parsedPreviousSplit)
-            ]);
+            return _this.wrapper.set(splitKey, stringifiedNewSplit).then(function () {
+                // avoid unnecessary increment/decrement operations
+                if (parsedPreviousSplit && parsedPreviousSplit.trafficTypeName === split.trafficTypeName)
+                    return;
+                // update traffic type counts
+                return _this._incrementCounts(split).then(function () {
+                    if (parsedPreviousSplit)
+                        return _this._decrementCounts(parsedPreviousSplit);
+                });
+            });
         }).then(function () { return true; });
     };
     /**

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

@@ -84,7 +84,7 @@ function segmentChangesUpdaterFactory(log, segmentChangesFetcher, segments, read
                 // @TODO although factory status is destroyed, synchronization is not stopped
                 if (readiness)
                     readiness.destroy();
-                log.error(constants_2.LOG_PREFIX_INSTANTIATION + ": you passed a client-side type authorizationKey, please grab an Api Key from the Split web console that is of type Server-side.");
+                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.");
             }
             else {
                 log.warn(constants_2.LOG_PREFIX_SYNC_SEGMENTS + "Error while doing fetch of segments. " + error);

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

@@ -5,7 +5,6 @@ var sets_1 = require("../../../utils/lang/sets");
 var timeout_1 = require("../../../utils/promise/timeout");
 var constants_1 = require("../../../readiness/constants");
 var constants_2 = require("../../../logger/constants");
-var maps_1 = require("../../../utils/lang/maps");
 // Checks that all registered segments have been fetched (changeNumber !== -1 for every segment).
 // Returns a promise that could be rejected.
 // @TODO review together with Segments and MySegments storage APIs
@@ -39,25 +38,19 @@ exports.parseSegments = parseSegments;
  * Exported for testing purposes.
  */
 function computeSplitsMutation(entries) {
-    var uniqueEntries = new maps_1._Map();
-    entries.forEach(function (split) {
-        var prevSplit = uniqueEntries.get(split.name);
-        if (!prevSplit || prevSplit.changeNumber < split.changeNumber)
-            uniqueEntries.set(split.name, split);
-    });
     var segments = new sets_1._Set();
-    var computed = { added: [], removed: [], segments: [] };
-    uniqueEntries.forEach(function (split) {
+    var computed = entries.reduce(function (accum, split) {
         if (split.status === 'ACTIVE') {
-            computed.added.push([split.name, split]);
+            accum.added.push([split.name, split]);
             parseSegments(split).forEach(function (segmentName) {
                 segments.add(segmentName);
             });
         }
         else {
-            computed.removed.push(split.name);
+            accum.removed.push(split.name);
         }
-    });
+        return accum;
+    }, { added: [], removed: [], segments: [] });
     computed.segments = (0, sets_1.setToArray)(segments);
     return computed;
 }

package/cjs/sync/streaming/pushManager.js

@@ -113,7 +113,7 @@ function pushManagerFactory(params, pollingManager) {
             if (disconnected)
                 return;
             log.error(constants_2.ERROR_STREAMING_AUTH, [error.message]);
-            // Handle 4XX HTTP errors: 401 (invalid API Key) or 400 (using incorrect API Key, i.e., client-side API Key on server-side)
+            // 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);

package/cjs/sync/submitters/telemetrySubmitter.js

@@ -28,8 +28,8 @@ function getActiveFactories() {
     return Object.keys(apiKey_1.usedKeysMap).length;
 }
 function getRedundantActiveFactories() {
-    return Object.keys(apiKey_1.usedKeysMap).reduce(function (acum, apiKey) {
-        return acum + apiKey_1.usedKeysMap[apiKey] - 1;
+    return Object.keys(apiKey_1.usedKeysMap).reduce(function (acum, sdkKey) {
+        return acum + apiKey_1.usedKeysMap[sdkKey] - 1;
     }, 0);
 }
 function getTelemetryConfigStats(mode, storageType) {

package/cjs/utils/inputValidation/apiKey.js

@@ -3,51 +3,52 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.releaseApiKey = exports.validateAndTrackApiKey = exports.usedKeysMap = exports.validateApiKey = void 0;
 var constants_1 = require("../../logger/constants");
 var lang_1 = require("../lang");
-var item = 'api_key';
-/** validates the given api key */
-function validateApiKey(log, maybeApiKey) {
-    var apiKey = false;
-    if (maybeApiKey == undefined) { // eslint-disable-line eqeqeq
+var item = 'sdk_key';
+// @TODO replace ApiKey with SdkKey in function names
+/** validates the given SDK key */
+function validateApiKey(log, maybeSdkKey) {
+    var sdkKey = false;
+    if (maybeSdkKey == undefined) { // eslint-disable-line eqeqeq
         log.error(constants_1.ERROR_NULL, [constants_1.LOG_PREFIX_INSTANTIATION, item]);
     }
-    else if ((0, lang_1.isString)(maybeApiKey)) {
-        if (maybeApiKey.length > 0)
-            apiKey = maybeApiKey;
+    else if ((0, lang_1.isString)(maybeSdkKey)) {
+        if (maybeSdkKey.length > 0)
+            sdkKey = maybeSdkKey;
         else
             log.error(constants_1.ERROR_EMPTY, [constants_1.LOG_PREFIX_INSTANTIATION, item]);
     }
     else {
         log.error(constants_1.ERROR_INVALID, [constants_1.LOG_PREFIX_INSTANTIATION, item]);
     }
-    return apiKey;
+    return sdkKey;
 }
 exports.validateApiKey = validateApiKey;
 // Exported for telemetry
 exports.usedKeysMap = {};
-/** validates the given api key and also warns if it is in use */
-function validateAndTrackApiKey(log, maybeApiKey) {
-    var apiKey = validateApiKey(log, maybeApiKey);
-    // If the apiKey is correct, we'll save it as the instance creation should work.
-    if (apiKey) {
-        if (!exports.usedKeysMap[apiKey]) {
+/** validates the given SDK key and also warns if it is in use */
+function validateAndTrackApiKey(log, maybeSdkKey) {
+    var sdkKey = validateApiKey(log, maybeSdkKey);
+    // If sdkKey is correct, we'll save it as the instance creation should work.
+    if (sdkKey) {
+        if (!exports.usedKeysMap[sdkKey]) {
             // If this key is not present, only warning scenarios is that we have factories for other keys.
-            exports.usedKeysMap[apiKey] = 1;
+            exports.usedKeysMap[sdkKey] = 1;
             if (Object.keys(exports.usedKeysMap).length > 1) {
-                log.warn(constants_1.WARN_API_KEY, ['an instance of the Split factory']);
+                log.warn(constants_1.WARN_SDK_KEY, ['an instance of the Split factory']);
             }
         }
         else {
-            log.warn(constants_1.WARN_API_KEY, [exports.usedKeysMap[apiKey] + " factory/ies with this API Key"]);
-            exports.usedKeysMap[apiKey]++;
+            log.warn(constants_1.WARN_SDK_KEY, [exports.usedKeysMap[sdkKey] + " factory/ies with this SDK Key"]);
+            exports.usedKeysMap[sdkKey]++;
         }
     }
-    return apiKey;
+    return sdkKey;
 }
 exports.validateAndTrackApiKey = validateAndTrackApiKey;
-function releaseApiKey(apiKey) {
-    if (exports.usedKeysMap[apiKey])
-        exports.usedKeysMap[apiKey]--;
-    if (exports.usedKeysMap[apiKey] === 0)
-        delete exports.usedKeysMap[apiKey];
+function releaseApiKey(sdkKey) {
+    if (exports.usedKeysMap[sdkKey])
+        exports.usedKeysMap[sdkKey]--;
+    if (exports.usedKeysMap[sdkKey] === 0)
+        delete exports.usedKeysMap[sdkKey];
 }
 exports.releaseApiKey = releaseApiKey;

package/cjs/utils/inputValidation/preloadedData.js

@@ -13,12 +13,12 @@ function validateSplitsData(log, maybeSplitsData, method) {
     if ((0, lang_1.isObject)(maybeSplitsData)) {
         var splitNames = Object.keys(maybeSplitsData);
         if (splitNames.length === 0)
-            log.warn(method + ": preloadedData.splitsData doesn't contain split definitions.");
+            log.warn(method + ": preloadedData.splitsData doesn't contain feature flag definitions.");
         // @TODO in the future, consider handling the possibility of having parsed definitions of splits
         if (splitNames.every(function (splitName) { return (0, split_1.validateSplit)(log, splitName, method) && (0, lang_1.isString)(maybeSplitsData[splitName]); }))
             return true;
     }
-    log.error(method + ": preloadedData.splitsData must be a map of split names to their serialized definitions.");
+    log.error(method + ": preloadedData.splitsData must be a map of feature flag names to their stringified definitions.");
     return false;
 }
 function validateMySegmentsData(log, maybeMySegmentsData, method) {
@@ -40,7 +40,7 @@ function validateSegmentsData(log, maybeSegmentsData, method) {
         if (segmentNames.every(function (segmentName) { return (0, lang_1.isString)(maybeSegmentsData[segmentName]); }))
             return true;
     }
-    log.error(method + ": preloadedData.segmentsData must be a map of segment names to their serialized definitions.");
+    log.error(method + ": preloadedData.segmentsData must be a map of segment names to their stringified definitions.");
     return false;
 }
 function validatePreloadedData(log, maybePreloadedData, method) {

package/cjs/utils/inputValidation/split.js

@@ -6,7 +6,7 @@ var lang_1 = require("../lang");
 // include BOM and nbsp
 var TRIMMABLE_SPACES_REGEX = /^[\s\uFEFF\xA0]+|[\s\uFEFF\xA0]+$/;
 function validateSplit(log, maybeSplit, method, item) {
-    if (item === void 0) { item = 'split name'; }
+    if (item === void 0) { item = 'feature flag name'; }
     if (maybeSplit == undefined) { // eslint-disable-line eqeqeq
         log.error(constants_1.ERROR_NULL, [method, item]);
     }

package/cjs/utils/inputValidation/splits.js

@@ -5,8 +5,8 @@ var constants_1 = require("../../logger/constants");
 var lang_1 = require("../lang");
 var split_1 = require("./split");
 function validateSplits(log, maybeSplits, method, listName, item) {
-    if (listName === void 0) { listName = 'split_names'; }
-    if (item === void 0) { item = 'split name'; }
+    if (listName === void 0) { listName = 'feature flag names'; }
+    if (item === void 0) { item = 'feature flag name'; }
     if (Array.isArray(maybeSplits) && maybeSplits.length > 0) {
         var validatedArray_1 = [];
         // Remove invalid values

package/cjs/utils/settingsValidation/index.js

@@ -84,7 +84,7 @@ function fromSecondsToMillis(n) {
 }
 /**
  * Validates the given config and use it to build a settings object.
- * NOTE: it doesn't validate the Api Key. Call `validateApikey` or `validateAndTrackApiKey` for that after settings validation.
+ * NOTE: it doesn't validate the SDK Key. Call `validateApiKey` or `validateAndTrackApiKey` for that after settings validation.
  *
  * @param config user defined configuration
  * @param validationParams defaults and fields validators used to validate and creates a settings object from a given config

package/esm/logger/constants.js

@@ -92,7 +92,7 @@ export var WARN_INTEGRATION_INVALID = 218;
 export var WARN_SPLITS_FILTER_IGNORED = 219;
 export var WARN_SPLITS_FILTER_INVALID = 220;
 export var WARN_SPLITS_FILTER_EMPTY = 221;
-export var WARN_API_KEY = 222;
+export var WARN_SDK_KEY = 222;
 export var STREAMING_PARSING_MY_SEGMENTS_UPDATE_V2 = 223;
 export var ERROR_ENGINE_COMBINER_IFELSEIF = 300;
 export var ERROR_LOGLEVEL_INVALID = 301;
@@ -132,7 +132,7 @@ export var LOG_PREFIX_SYNC = 'sync';
 export var LOG_PREFIX_SYNC_MANAGER = LOG_PREFIX_SYNC + ':sync-manager: ';
 export var LOG_PREFIX_SYNC_OFFLINE = LOG_PREFIX_SYNC + ':offline: ';
 export var LOG_PREFIX_SYNC_STREAMING = LOG_PREFIX_SYNC + ':streaming: ';
-export var LOG_PREFIX_SYNC_SPLITS = LOG_PREFIX_SYNC + ':split-changes: ';
+export var LOG_PREFIX_SYNC_SPLITS = LOG_PREFIX_SYNC + ':featureflag-changes: ';
 export var LOG_PREFIX_SYNC_SEGMENTS = LOG_PREFIX_SYNC + ':segment-changes: ';
 export var LOG_PREFIX_SYNC_MYSEGMENTS = LOG_PREFIX_SYNC + ':my-segments: ';
 export var LOG_PREFIX_SYNC_POLLING = LOG_PREFIX_SYNC + ':polling-manager: ';

package/esm/logger/messages/debug.js

@@ -12,8 +12,8 @@ export var codesDebug = codesInfo.concat([
     [c.ENGINE_MATCHER_CONTAINS_ALL, c.LOG_PREFIX_ENGINE_MATCHER + '[containsAllMatcher] %s contains all elements of %s? %s'],
     [c.ENGINE_MATCHER_CONTAINS_ANY, c.LOG_PREFIX_ENGINE_MATCHER + '[containsAnyMatcher] %s contains at least an element of %s? %s'],
     [c.ENGINE_MATCHER_CONTAINS_STRING, c.LOG_PREFIX_ENGINE_MATCHER + '[containsStringMatcher] %s contains %s? %s'],
-    [c.ENGINE_MATCHER_DEPENDENCY, c.LOG_PREFIX_ENGINE_MATCHER + '[dependencyMatcher] parent split "%s" evaluated to "%s" with label "%s". %s evaluated treatment is part of [%s] ? %s.'],
-    [c.ENGINE_MATCHER_DEPENDENCY_PRE, c.LOG_PREFIX_ENGINE_MATCHER + '[dependencyMatcher] will evaluate parent split: "%s" with key: %s %s'],
+    [c.ENGINE_MATCHER_DEPENDENCY, c.LOG_PREFIX_ENGINE_MATCHER + '[dependencyMatcher] parent feature flag "%s" evaluated to "%s" with label "%s". %s evaluated treatment is part of [%s] ? %s.'],
+    [c.ENGINE_MATCHER_DEPENDENCY_PRE, c.LOG_PREFIX_ENGINE_MATCHER + '[dependencyMatcher] will evaluate parent feature flag: "%s" with key: %s %s'],
     [c.ENGINE_MATCHER_EQUAL, c.LOG_PREFIX_ENGINE_MATCHER + '[equalToMatcher] is %s equal to %s? %s'],
     [c.ENGINE_MATCHER_EQUAL_TO_SET, c.LOG_PREFIX_ENGINE_MATCHER + '[equalToSetMatcher] is %s equal to set %s? %s'],
     [c.ENGINE_MATCHER_ENDS_WITH, c.LOG_PREFIX_ENGINE_MATCHER + '[endsWithMatcher] %s ends with %s? %s'],
@@ -34,15 +34,15 @@ export var codesDebug = codesInfo.concat([
     [c.RETRIEVE_CLIENT_EXISTING, 'Retrieving existing SDK client.'],
     [c.RETRIEVE_MANAGER, 'Retrieving manager instance.'],
     // synchronizer
-    [c.SYNC_OFFLINE_DATA, c.LOG_PREFIX_SYNC_OFFLINE + 'Splits data: \n%s'],
-    [c.SYNC_SPLITS_FETCH, c.LOG_PREFIX_SYNC_SPLITS + 'Spin up split update using since = %s'],
-    [c.SYNC_SPLITS_NEW, c.LOG_PREFIX_SYNC_SPLITS + 'New splits %s'],
-    [c.SYNC_SPLITS_REMOVED, c.LOG_PREFIX_SYNC_SPLITS + 'Removed splits %s'],
+    [c.SYNC_OFFLINE_DATA, c.LOG_PREFIX_SYNC_OFFLINE + 'Feature flags data: \n%s'],
+    [c.SYNC_SPLITS_FETCH, c.LOG_PREFIX_SYNC_SPLITS + 'Spin up feature flags update using since = %s'],
+    [c.SYNC_SPLITS_NEW, c.LOG_PREFIX_SYNC_SPLITS + 'New feature flags %s'],
+    [c.SYNC_SPLITS_REMOVED, c.LOG_PREFIX_SYNC_SPLITS + 'Removed feature flags %s'],
     [c.SYNC_SPLITS_SEGMENTS, c.LOG_PREFIX_SYNC_SPLITS + 'Segment names collected %s'],
     [c.STREAMING_NEW_MESSAGE, c.LOG_PREFIX_SYNC_STREAMING + 'New SSE message received, with data: %s.'],
     [c.SYNC_TASK_START, c.LOG_PREFIX_SYNC + ': Starting %s. Running each %s millis'],
     [c.SYNC_TASK_EXECUTE, c.LOG_PREFIX_SYNC + ': Running %s'],
     [c.SYNC_TASK_STOP, c.LOG_PREFIX_SYNC + ': Stopping %s'],
     // initialization / settings validation
-    [c.SETTINGS_SPLITS_FILTER, c.LOG_PREFIX_SETTINGS + ': splits filtering criteria is "%s".']
+    [c.SETTINGS_SPLITS_FILTER, c.LOG_PREFIX_SETTINGS + ': feature flags filtering criteria is "%s".']
 ]);

package/esm/logger/messages/error.js

@@ -1,7 +1,7 @@
 import * as c from '../constants';
 export var codesError = [
     // evaluator
-    [c.ERROR_ENGINE_COMBINER_IFELSEIF, c.LOG_PREFIX_ENGINE_COMBINER + 'Invalid Split, no valid rules found'],
+    [c.ERROR_ENGINE_COMBINER_IFELSEIF, c.LOG_PREFIX_ENGINE_COMBINER + 'Invalid feature flag, no valid rules found'],
     // SDK
     [c.ERROR_LOGLEVEL_INVALID, 'logger: Invalid Log Level - No changes to the logs will be applied.'],
     [c.ERROR_CLIENT_CANNOT_GET_READY, 'The SDK will not get ready. Reason: %s'],
@@ -9,7 +9,7 @@ export var codesError = [
     [c.ERROR_IMPRESSIONS_LISTENER, c.LOG_PREFIX_IMPRESSIONS_TRACKER + 'Impression listener logImpression method threw: %s.'],
     [c.ERROR_EVENTS_TRACKER, c.LOG_PREFIX_EVENTS_TRACKER + 'Failed to queue %s'],
     // synchronizer
-    [c.ERROR_SYNC_OFFLINE_LOADING, c.LOG_PREFIX_SYNC_OFFLINE + 'There was an issue loading the mock Splits data, no changes will be applied to the current cache. %s'],
+    [c.ERROR_SYNC_OFFLINE_LOADING, c.LOG_PREFIX_SYNC_OFFLINE + 'There was an issue loading the mock feature flags data. No changes will be applied to the current cache. %s'],
     [c.ERROR_STREAMING_SSE, c.LOG_PREFIX_SYNC_STREAMING + 'Failed to connect or error on streaming connection, with error message: %s'],
     [c.ERROR_STREAMING_AUTH, c.LOG_PREFIX_SYNC_STREAMING + 'Failed to authenticate for streaming. Error: %s.'],
     [c.ERROR_HTTP, 'Response status is not OK. Status: %s. URL: %s. Message: %s'],

package/esm/logger/messages/info.js

@@ -6,7 +6,7 @@ export var codesInfo = codesWarn.concat([
     [c.CLIENT_READY_FROM_CACHE, READY_MSG + ' from cache'],
     [c.CLIENT_READY, READY_MSG],
     // SDK
-    [c.IMPRESSION, c.LOG_PREFIX_IMPRESSIONS_TRACKER + 'Split: %s. Key: %s. Evaluation: %s. Label: %s'],
+    [c.IMPRESSION, c.LOG_PREFIX_IMPRESSIONS_TRACKER + 'Feature flag: %s. Key: %s. Evaluation: %s. Label: %s'],
     [c.IMPRESSION_QUEUEING, c.LOG_PREFIX_IMPRESSIONS_TRACKER + 'Queueing corresponding impression.'],
     [c.NEW_SHARED_CLIENT, 'New shared client instance created.'],
     [c.NEW_FACTORY, 'New Split SDK instance created.'],
@@ -19,13 +19,13 @@ export var codesInfo = codesWarn.concat([
     [c.POLLING_SMART_PAUSING, c.LOG_PREFIX_SYNC_POLLING + 'Turning segments data polling %s.'],
     [c.POLLING_START, c.LOG_PREFIX_SYNC_POLLING + 'Starting polling'],
     [c.POLLING_STOP, c.LOG_PREFIX_SYNC_POLLING + 'Stopping polling'],
-    [c.SYNC_SPLITS_FETCH_RETRY, c.LOG_PREFIX_SYNC_SPLITS + 'Retrying download of splits #%s. Reason: %s'],
+    [c.SYNC_SPLITS_FETCH_RETRY, c.LOG_PREFIX_SYNC_SPLITS + 'Retrying download of feature flags #%s. Reason: %s'],
     [c.SUBMITTERS_PUSH_FULL_QUEUE, c.LOG_PREFIX_SYNC_SUBMITTERS + 'Flushing full %s queue and reseting timer.'],
     [c.SUBMITTERS_PUSH, c.LOG_PREFIX_SYNC_SUBMITTERS + 'Pushing %s.'],
     [c.STREAMING_REFRESH_TOKEN, c.LOG_PREFIX_SYNC_STREAMING + 'Refreshing streaming token in %s seconds, and connecting streaming in %s seconds.'],
     [c.STREAMING_RECONNECT, c.LOG_PREFIX_SYNC_STREAMING + 'Attempting to reconnect streaming in %s seconds.'],
     [c.STREAMING_CONNECTING, c.LOG_PREFIX_SYNC_STREAMING + 'Connecting streaming.'],
-    [c.STREAMING_DISABLED, c.LOG_PREFIX_SYNC_STREAMING + 'Streaming is disabled for given Api key. Switching to polling mode.'],
+    [c.STREAMING_DISABLED, c.LOG_PREFIX_SYNC_STREAMING + 'Streaming is disabled for given SDK key. Switching to polling mode.'],
     [c.STREAMING_DISCONNECTING, c.LOG_PREFIX_SYNC_STREAMING + 'Disconnecting streaming.'],
     [c.SYNC_START_POLLING, c.LOG_PREFIX_SYNC_MANAGER + 'Streaming not available. Starting polling.'],
     [c.SYNC_CONTINUE_POLLING, c.LOG_PREFIX_SYNC_MANAGER + 'Streaming couldn\'t connect. Continue polling.'],

package/esm/logger/messages/warn.js

@@ -6,7 +6,7 @@ export var codesWarn = codesError.concat([
     [c.ENGINE_VALUE_NO_ATTRIBUTES, c.LOG_PREFIX_ENGINE_VALUE + 'Defined attribute [%s], no attributes received.'],
     // synchronizer
     [c.SYNC_MYSEGMENTS_FETCH_RETRY, c.LOG_PREFIX_SYNC_MYSEGMENTS + 'Retrying download of segments #%s. Reason: %s'],
-    [c.SYNC_SPLITS_FETCH_FAILS, c.LOG_PREFIX_SYNC_SPLITS + 'Error while doing fetch of Splits. %s'],
+    [c.SYNC_SPLITS_FETCH_FAILS, c.LOG_PREFIX_SYNC_SPLITS + 'Error while doing fetch of feature flags. %s'],
     [c.STREAMING_PARSING_ERROR_FAILS, c.LOG_PREFIX_SYNC_STREAMING + 'Error parsing SSE error notification: %s'],
     [c.STREAMING_PARSING_MESSAGE_FAILS, c.LOG_PREFIX_SYNC_STREAMING + 'Error parsing SSE message notification: %s'],
     [c.STREAMING_FALLBACK, c.LOG_PREFIX_SYNC_STREAMING + 'Falling back to polling mode. Reason: %s'],
@@ -20,14 +20,14 @@ export var codesWarn = codesError.concat([
     [c.WARN_TRIMMING_PROPERTIES, '%s: Event has more than 300 properties. Some of them will be trimmed when processed.'],
     [c.WARN_CONVERTING, '%s: %s "%s" is not of type string, converting.'],
     [c.WARN_TRIMMING, '%s: %s "%s" has extra whitespace, trimming.'],
-    [c.WARN_NOT_EXISTENT_SPLIT, '%s: split "%s" does not exist in this environment, please double check what splits exist in the Split web console.'],
+    [c.WARN_NOT_EXISTENT_SPLIT, '%s: feature flag "%s" does not exist in this environment. Please double check what feature flags exist in the Split user interface.'],
     [c.WARN_LOWERCASE_TRAFFIC_TYPE, '%s: traffic_type_name should be all lowercase - converting string to lowercase.'],
-    [c.WARN_NOT_EXISTENT_TT, '%s: traffic type "%s" does not have any corresponding split in this environment, make sure you\'re tracking your events to a valid traffic type defined in the Split web console.'],
+    [c.WARN_NOT_EXISTENT_TT, '%s: traffic type "%s" does not have any corresponding feature flag in this environment, make sure you\'re tracking your events to a valid traffic type defined in the Split user interface.'],
     // initialization / settings validation
     [c.WARN_INTEGRATION_INVALID, c.LOG_PREFIX_SETTINGS + ': %s integration item(s) at settings is invalid. %s'],
-    [c.WARN_SPLITS_FILTER_IGNORED, c.LOG_PREFIX_SETTINGS + ': split filters have been configured but will have no effect if mode is not "%s", since synchronization is being deferred to an external tool.'],
-    [c.WARN_SPLITS_FILTER_INVALID, c.LOG_PREFIX_SETTINGS + ': split filter at position %s is invalid. It must be an object with a valid filter type ("byName" or "byPrefix") and a list of "values".'],
-    [c.WARN_SPLITS_FILTER_EMPTY, c.LOG_PREFIX_SETTINGS + ': splitFilters configuration must be a non-empty array of filter objects.'],
-    [c.WARN_API_KEY, c.LOG_PREFIX_SETTINGS + ': You already have %s. We recommend keeping only one instance of the factory at all times (Singleton pattern) and reusing it throughout your application'],
+    [c.WARN_SPLITS_FILTER_IGNORED, c.LOG_PREFIX_SETTINGS + ': feature flag filters have been configured but will have no effect if mode is not "%s", since synchronization is being deferred to an external tool.'],
+    [c.WARN_SPLITS_FILTER_INVALID, c.LOG_PREFIX_SETTINGS + ': feature flag filter at position %s is invalid. It must be an object with a valid filter type ("byName" or "byPrefix") and a list of "values".'],
+    [c.WARN_SPLITS_FILTER_EMPTY, c.LOG_PREFIX_SETTINGS + ': feature flag filter configuration must be a non-empty array of filter objects.'],
+    [c.WARN_SDK_KEY, c.LOG_PREFIX_SETTINGS + ': You already have %s. We recommend keeping only one instance of the factory at all times (Singleton pattern) and reusing it throughout your application'],
     [c.STREAMING_PARSING_MY_SEGMENTS_UPDATE_V2, c.LOG_PREFIX_SYNC_STREAMING + 'Fetching MySegments due to an error processing %s notification: %s'],
 ]);

package/esm/sdkClient/client.js

@@ -8,10 +8,10 @@ import { CONTROL, TREATMENT, TREATMENTS, TREATMENT_WITH_CONFIG, TREATMENTS_WITH_
 import { IMPRESSION, IMPRESSION_QUEUEING } from '../logger/constants';
 import { isStorageSync } from '../trackers/impressionObserver/utils';
 var treatmentNotReady = { treatment: CONTROL, label: SDK_NOT_READY };
-function treatmentsNotReady(splitNames) {
+function treatmentsNotReady(featureFlagNames) {
     var evaluations = {};
-    splitNames.forEach(function (splitName) {
-        evaluations[splitName] = treatmentNotReady;
+    featureFlagNames.forEach(function (featureFlagName) {
+        evaluations[featureFlagName] = treatmentNotReady;
     });
     return evaluations;
 }
@@ -21,59 +21,59 @@ function treatmentsNotReady(splitNames) {
 export function clientFactory(params) {
     var readinessManager = params.sdkReadinessManager.readinessManager, storage = params.storage, settings = params.settings, impressionsTracker = params.impressionsTracker, eventTracker = params.eventTracker, telemetryTracker = params.telemetryTracker;
     var log = settings.log, mode = settings.mode;
-    function getTreatment(key, splitName, attributes, withConfig) {
+    function getTreatment(key, featureFlagName, attributes, withConfig) {
         if (withConfig === void 0) { withConfig = false; }
         var stopTelemetryTracker = telemetryTracker.trackEval(withConfig ? TREATMENT_WITH_CONFIG : TREATMENT);
         var wrapUp = function (evaluationResult) {
             var queue = [];
-            var treatment = processEvaluation(evaluationResult, splitName, key, attributes, withConfig, "getTreatment" + (withConfig ? 'withConfig' : ''), queue);
+            var treatment = processEvaluation(evaluationResult, featureFlagName, key, attributes, withConfig, "getTreatment" + (withConfig ? 'withConfig' : ''), queue);
             impressionsTracker.track(queue, attributes);
             stopTelemetryTracker(queue[0] && queue[0].label);
             return treatment;
         };
         var evaluation = readinessManager.isReady() || readinessManager.isReadyFromCache() ?
-            evaluateFeature(log, key, splitName, attributes, storage) :
+            evaluateFeature(log, key, featureFlagName, attributes, storage) :
             isStorageSync(settings) ? // If the SDK is not ready, treatment may be incorrect due to having splits but not segments data, or storage is not connected
                 treatmentNotReady :
                 Promise.resolve(treatmentNotReady); // Promisify if async
         return thenable(evaluation) ? evaluation.then(function (res) { return wrapUp(res); }) : wrapUp(evaluation);
     }
-    function getTreatmentWithConfig(key, splitName, attributes) {
-        return getTreatment(key, splitName, attributes, true);
+    function getTreatmentWithConfig(key, featureFlagName, attributes) {
+        return getTreatment(key, featureFlagName, attributes, true);
     }
-    function getTreatments(key, splitNames, attributes, withConfig) {
+    function getTreatments(key, featureFlagNames, attributes, withConfig) {
         if (withConfig === void 0) { withConfig = false; }
         var stopTelemetryTracker = telemetryTracker.trackEval(withConfig ? TREATMENTS_WITH_CONFIG : TREATMENTS);
         var wrapUp = function (evaluationResults) {
             var queue = [];
             var treatments = {};
-            Object.keys(evaluationResults).forEach(function (splitName) {
-                treatments[splitName] = processEvaluation(evaluationResults[splitName], splitName, key, attributes, withConfig, "getTreatments" + (withConfig ? 'withConfig' : ''), queue);
+            Object.keys(evaluationResults).forEach(function (featureFlagName) {
+                treatments[featureFlagName] = processEvaluation(evaluationResults[featureFlagName], featureFlagName, key, attributes, withConfig, "getTreatments" + (withConfig ? 'withConfig' : ''), queue);
             });
             impressionsTracker.track(queue, attributes);
             stopTelemetryTracker(queue[0] && queue[0].label);
             return treatments;
         };
         var evaluations = readinessManager.isReady() || readinessManager.isReadyFromCache() ?
-            evaluateFeatures(log, key, splitNames, attributes, storage) :
+            evaluateFeatures(log, key, featureFlagNames, attributes, storage) :
             isStorageSync(settings) ? // If the SDK is not ready, treatment may be incorrect due to having splits but not segments data, or storage is not connected
-                treatmentsNotReady(splitNames) :
-                Promise.resolve(treatmentsNotReady(splitNames)); // Promisify if async
+                treatmentsNotReady(featureFlagNames) :
+                Promise.resolve(treatmentsNotReady(featureFlagNames)); // Promisify if async
         return thenable(evaluations) ? evaluations.then(function (res) { return wrapUp(res); }) : wrapUp(evaluations);
     }
-    function getTreatmentsWithConfig(key, splitNames, attributes) {
-        return getTreatments(key, splitNames, attributes, true);
+    function getTreatmentsWithConfig(key, featureFlagNames, attributes) {
+        return getTreatments(key, featureFlagNames, attributes, true);
     }
     // Internal function
-    function processEvaluation(evaluation, splitName, key, attributes, withConfig, invokingMethodName, queue) {
+    function processEvaluation(evaluation, featureFlagName, key, attributes, withConfig, invokingMethodName, queue) {
         var matchingKey = getMatching(key);
         var bucketingKey = getBucketing(key);
         var treatment = evaluation.treatment, label = evaluation.label, changeNumber = evaluation.changeNumber, _a = evaluation.config, config = _a === void 0 ? null : _a;
-        log.info(IMPRESSION, [splitName, matchingKey, treatment, label]);
-        if (validateSplitExistance(log, readinessManager, splitName, label, invokingMethodName)) {
+        log.info(IMPRESSION, [featureFlagName, matchingKey, treatment, label]);
+        if (validateSplitExistance(log, readinessManager, featureFlagName, label, invokingMethodName)) {
             log.info(IMPRESSION_QUEUEING);
             queue.push({
-                feature: splitName,
+                feature: featureFlagName,
                 keyName: matchingKey,
                 treatment: treatment,
                 time: Date.now(),

package/esm/sdkClient/clientAttributesDecoration.js

@@ -12,17 +12,17 @@ export function clientAttributesDecoration(log, client) {
     var clientGetTreatments = client.getTreatments;
     var clientGetTreatmentsWithConfig = client.getTreatmentsWithConfig;
     var clientTrack = client.track;
-    function getTreatment(maybeKey, maybeSplit, maybeAttributes) {
-        return clientGetTreatment(maybeKey, maybeSplit, combineAttributes(maybeAttributes));
+    function getTreatment(maybeKey, maybeFeatureFlagName, maybeAttributes) {
+        return clientGetTreatment(maybeKey, maybeFeatureFlagName, combineAttributes(maybeAttributes));
     }
-    function getTreatmentWithConfig(maybeKey, maybeSplit, maybeAttributes) {
-        return clientGetTreatmentWithConfig(maybeKey, maybeSplit, combineAttributes(maybeAttributes));
+    function getTreatmentWithConfig(maybeKey, maybeFeatureFlagName, maybeAttributes) {
+        return clientGetTreatmentWithConfig(maybeKey, maybeFeatureFlagName, combineAttributes(maybeAttributes));
     }
-    function getTreatments(maybeKey, maybeSplits, maybeAttributes) {
-        return clientGetTreatments(maybeKey, maybeSplits, combineAttributes(maybeAttributes));
+    function getTreatments(maybeKey, maybeFeatureFlagNames, maybeAttributes) {
+        return clientGetTreatments(maybeKey, maybeFeatureFlagNames, combineAttributes(maybeAttributes));
     }
-    function getTreatmentsWithConfig(maybeKey, maybeSplits, maybeAttributes) {
-        return clientGetTreatmentsWithConfig(maybeKey, maybeSplits, combineAttributes(maybeAttributes));
+    function getTreatmentsWithConfig(maybeKey, maybeFeatureFlagNames, maybeAttributes) {
+        return clientGetTreatmentsWithConfig(maybeKey, maybeFeatureFlagNames, combineAttributes(maybeAttributes));
     }
     function track(maybeKey, maybeTT, maybeEvent, maybeEventValue, maybeProperties) {
         return clientTrack(maybeKey, maybeTT, maybeEvent, maybeEventValue, maybeProperties);