@splitsoftware/splitio-commons 1.17.1-rc.4 → 2.0.0-rc.1

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

@@ -1,12 +1,12 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.splitChangesUpdaterFactory = exports.computeSplitsMutation = exports.parseSegments = void 0;
-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 lang_1 = require("../../../utils/lang");
 var constants_3 = require("../../../utils/constants");
+var sets_1 = require("../../../utils/lang/sets");
 // 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
@@ -23,7 +23,7 @@ function checkAllSegmentsExist(segments) {
  */
 function parseSegments(_a) {
     var conditions = _a.conditions;
-    var segments = new sets_1._Set();
+    var segments = new Set();
     for (var i = 0; i < conditions.length; i++) {
         var matchers = conditions[i].matcherGroup.matchers;
         matchers.forEach(function (matcher) {
@@ -59,7 +59,7 @@ function matchFilters(featureFlag, filters) {
  * Exported for testing purposes.
  */
 function computeSplitsMutation(entries, filters) {
-    var segments = new sets_1._Set();
+    var segments = new Set();
     var computed = entries.reduce(function (accum, split) {
         if (split.status === 'ACTIVE' && matchFilters(split, filters)) {
             accum.added.push([split.name, split]);

package/cjs/sync/streaming/parseUtils.js

@@ -11,7 +11,6 @@ function Uint8ArrayToString(myUint8Arr) {
 }
 function StringToUint8Array(myString) {
     var charCodes = myString.split('').map(function (e) { return e.charCodeAt(0); });
-    // eslint-disable-next-line compat/compat
     return new Uint8Array(charCodes);
 }
 /**

package/cjs/sync/streaming/pushManager.js

@@ -15,7 +15,6 @@ var constants_1 = require("./constants");
 var constants_2 = require("../../logger/constants");
 var types_1 = require("./SSEHandler/types");
 var parseUtils_1 = require("./parseUtils");
-var sets_1 = require("../../utils/lang/sets");
 var murmur3_64_1 = require("../../utils/murmur3/murmur3_64");
 var constants_3 = require("../../utils/constants");
 /**
@@ -221,8 +220,8 @@ function pushManagerFactory(params, pollingManager) {
                 var keyList = void 0, added_1, removed_1;
                 try {
                     keyList = (0, parseUtils_1.parseKeyList)(parsedData.d, parsedData.c);
-                    added_1 = new sets_1._Set(keyList.a);
-                    removed_1 = new sets_1._Set(keyList.r);
+                    added_1 = new Set(keyList.a);
+                    removed_1 = new Set(keyList.r);
                 }
                 catch (e) {
                     log.warn(constants_2.STREAMING_PARSING_MEMBERSHIPS_UPDATE, ['KeyList', e]);

package/cjs/utils/LRUCache/index.js

@@ -1,12 +1,11 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.LRUCache = void 0;
-var maps_1 = require("../lang/maps");
 var LinkedList_1 = require("./LinkedList");
 var LRUCache = /** @class */ (function () {
     function LRUCache(maxSize) {
         this.maxLen = maxSize || 1;
-        this.items = new maps_1._Map();
+        this.items = new Map();
         this.lru = new LinkedList_1.LinkedList();
     }
     LRUCache.prototype.get = function (key) {

package/cjs/utils/constants/browser.js

@@ -1,8 +1,5 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.DEFAULT_CACHE_EXPIRATION_IN_MILLIS = exports.SPLIT_TO_GOOGLE_ANALYTICS = exports.GOOGLE_ANALYTICS_TO_SPLIT = void 0;
-// Integration types
-exports.GOOGLE_ANALYTICS_TO_SPLIT = 'GOOGLE_ANALYTICS_TO_SPLIT';
-exports.SPLIT_TO_GOOGLE_ANALYTICS = 'SPLIT_TO_GOOGLE_ANALYTICS';
+exports.DEFAULT_CACHE_EXPIRATION_IN_MILLIS = void 0;
 // This value might be eventually set via a config parameter
 exports.DEFAULT_CACHE_EXPIRATION_IN_MILLIS = 864000000; // 10 days

package/cjs/utils/lang/index.js

@@ -121,11 +121,9 @@ exports.isBoolean = isBoolean;
 function isFiniteNumber(val) {
     if (val instanceof Number)
         val = val.valueOf();
-    // @TODO remove `isFinite` once `Number.isFinite` is fully supported by targets
-    // eslint-disable-next-line compat/compat
-    if (typeof val === 'number')
-        return Number.isFinite ? Number.isFinite(val) : isFinite(val);
-    return false;
+    return typeof val === 'number' ?
+        Number.isFinite ? Number.isFinite(val) : isFinite(val) :
+        false;
 }
 exports.isFiniteNumber = isFiniteNumber;
 /**
@@ -135,10 +133,9 @@ exports.isFiniteNumber = isFiniteNumber;
 function isIntegerNumber(val) {
     if (val instanceof Number)
         val = val.valueOf();
-    // eslint-disable-next-line compat/compat
-    if (typeof val === 'number')
-        return Number.isInteger ? Number.isInteger(val) : isFinite(val) && Math.floor(val) === val;
-    return false;
+    return typeof val === 'number' ?
+        Number.isInteger ? Number.isInteger(val) : isFinite(val) && Math.floor(val) === val :
+        false;
 }
 exports.isIntegerNumber = isIntegerNumber;
 /**

package/cjs/utils/lang/objectAssign.js

@@ -1,85 +1,20 @@
 "use strict";
-/*
-Adaptation of "object-assign" library (https://www.npmjs.com/package/object-assign)
-exported as an ES module instead of CommonJS, to avoid extra configuration steps when using
-the ESM build of the SDK with tools that doesn't support CommonJS by default (e.g. Rollup).
-
-object-assign
-(c) Sindre Sorhus
-@license MIT
-*/
+/**
+ * A tiny polyfill for Object.assign
+ */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.objectAssign = void 0;
-/* eslint-disable */
-// @ts-nocheck
-var getOwnPropertySymbols = Object.getOwnPropertySymbols;
-var hasOwnProperty = Object.prototype.hasOwnProperty;
-var propIsEnumerable = Object.prototype.propertyIsEnumerable;
-function toObject(val) {
-    if (val === null || val === undefined) {
+exports.objectAssign = Object.assign || function (target) {
+    if (target === null || target === undefined)
         throw new TypeError('Object.assign cannot be called with null or undefined');
-    }
-    return Object(val);
-}
-function shouldUseNative() {
-    try {
-        if (!Object.assign) {
-            return false;
-        }
-        // Detect buggy property enumeration order in older V8 versions.
-        // https://bugs.chromium.org/p/v8/issues/detail?id=4118
-        var test1 = new String('abc');
-        test1[5] = 'de';
-        if (Object.getOwnPropertyNames(test1)[0] === '5') {
-            return false;
-        }
-        // https://bugs.chromium.org/p/v8/issues/detail?id=3056
-        var test2 = {};
-        for (var i = 0; i < 10; i++) {
-            test2['_' + String.fromCharCode(i)] = i;
-        }
-        var order2 = Object.getOwnPropertyNames(test2).map(function (n) {
-            return test2[n];
-        });
-        if (order2.join('') !== '0123456789') {
-            return false;
-        }
-        // https://bugs.chromium.org/p/v8/issues/detail?id=3056
-        var test3 = {};
-        'abcdefghijklmnopqrst'.split('').forEach(function (letter) {
-            test3[letter] = letter;
-        });
-        if (Object.keys(Object.assign({}, test3)).join('') !==
-            'abcdefghijklmnopqrst') {
-            return false;
-        }
-        return true;
-    }
-    catch (err) {
-        // We don't expect any of the above to throw, but better to be safe.
-        return false;
-    }
-}
-exports.objectAssign = shouldUseNative() ? Object.assign : function (target, source) {
-    var from;
-    var to = toObject(target);
-    var symbols;
-    for (var s = 1; s < arguments.length; s++) {
-        from = Object(arguments[s]);
-        // eslint-disable-next-line no-restricted-syntax
-        for (var key in from) {
-            if (hasOwnProperty.call(from, key)) {
-                to[key] = from[key];
-            }
-        }
-        if (getOwnPropertySymbols) {
-            symbols = getOwnPropertySymbols(from);
-            for (var i = 0; i < symbols.length; i++) {
-                if (propIsEnumerable.call(from, symbols[i])) {
-                    to[symbols[i]] = from[symbols[i]];
-                }
+    target = Object(target);
+    for (var i = 1; i < arguments.length; i++) {
+        var source = Object(arguments[i]); // eslint-disable-next-line no-restricted-syntax
+        for (var key in source) {
+            if (Object.prototype.hasOwnProperty.call(source, key)) {
+                target[key] = source[key];
             }
         }
     }
-    return to;
+    return target;
 };

package/cjs/utils/lang/sets.js

@@ -1,120 +1,23 @@
 "use strict";
-/**
- * Set implementation based on es6-set polyfill (https://github.com/medikoo/es6-set/blob/master/polyfill.js),
- * with the minimal features used by the SDK.
-
-Copyright (C) 2013 Mariusz Nowak (www.medikoo.com)
-
-Permission is hereby granted, free of charge, to any person obtaining a copy
-of this software and associated documentation files (the "Software"), to deal
-in the Software without restriction, including without limitation the rights
-to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
-copies of the Software, and to permit persons to whom the Software is
-furnished to do so, subject to the following conditions:
-
-The above copyright notice and this permission notice shall be included in
-all copies or substantial portions of the Software.
-
-THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
-AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
-LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
-OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
-THE SOFTWARE.
-**/
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.returnDifference = exports.returnSetsUnion = exports._Set = exports.__getSetConstructor = exports.setToArray = exports.SetPoly = void 0;
-var SetPoly = /** @class */ (function () {
-    // unlike ES6 `Set`, it only accepts an array as first argument iterable
-    function SetPoly(values) {
-        var _this = this;
-        this.__setData__ = [];
-        if (Array.isArray(values))
-            values.forEach(function (value) { _this.add(value); });
-    }
-    SetPoly.prototype.clear = function () {
-        if (!this.__setData__.length)
-            return;
-        this.__setData__.length = 0;
-    };
-    SetPoly.prototype.add = function (value) {
-        if (this.has(value))
-            return this;
-        this.__setData__.push(value);
-        return this;
-    };
-    SetPoly.prototype.delete = function (value) {
-        var index = this.__setData__.indexOf(value);
-        if (index === -1)
-            return false;
-        this.__setData__.splice(index, 1);
-        return true;
-    };
-    SetPoly.prototype.has = function (value) {
-        return this.__setData__.indexOf(value) !== -1;
-    };
-    SetPoly.prototype.forEach = function (callbackfn, thisArg) {
-        if (typeof callbackfn !== 'function')
-            throw new TypeError(callbackfn + ' is not a function');
-        for (var i = 0; i < this.__setData__.length; i++) {
-            var value = this.__setData__[i];
-            callbackfn.call(thisArg, value, value, this);
-        }
-    };
-    Object.defineProperty(SetPoly.prototype, "size", {
-        get: function () {
-            return this.__setData__.length;
-        },
-        enumerable: false,
-        configurable: true
-    });
-    return SetPoly;
-}());
-exports.SetPoly = SetPoly;
-/**
- * return an array containing the items of the given set.
- * @param set Set or SetPoly instance
- */
+exports.returnDifference = exports.returnSetsUnion = exports.setToArray = void 0;
 function setToArray(set) {
-    if (set instanceof SetPoly) {
-        return set.__setData__.slice();
-    }
-    // if not using SetPoly as set, it means Array.from is supported
-    // eslint-disable-next-line compat/compat
-    return Array.from(set);
+    if (Array.from)
+        return Array.from(set);
+    var array = [];
+    set.forEach(function (value) {
+        array.push(value);
+    });
+    return array;
 }
 exports.setToArray = setToArray;
-/**
- * return the Set constructor to use. If `Array.from` built-in or native Set is not available or it doesn't support the required features,
- * a ponyfill with minimal features is returned instead.
- *
- * Exported for testing purposes only.
- */
-function __getSetConstructor() {
-    // eslint-disable-next-line compat/compat
-    if (typeof Array.from === 'function' && typeof Set === 'function' && Set.prototype && Set.prototype.values) {
-        return Set;
-    }
-    return SetPoly;
-}
-exports.__getSetConstructor = __getSetConstructor;
-exports._Set = __getSetConstructor();
 function returnSetsUnion(set, set2) {
-    var result = new exports._Set(setToArray(set));
-    set2.forEach(function (value) {
-        result.add(value);
-    });
-    return result;
+    return new Set(setToArray(set).concat(setToArray(set2)));
 }
 exports.returnSetsUnion = returnSetsUnion;
 function returnDifference(list, list2) {
     if (list === void 0) { list = []; }
     if (list2 === void 0) { list2 = []; }
-    var result = new exports._Set(list);
-    list2.forEach(function (item) {
-        result.delete(item);
-    });
-    return setToArray(result);
+    return list.filter(function (item) { return list2.indexOf(item) === -1; });
 }
 exports.returnDifference = returnDifference;

package/cjs/utils/settingsValidation/index.js

@@ -7,7 +7,6 @@ var splitFilters_1 = require("./splitFilters");
 var constants_1 = require("../constants");
 var impressionsMode_1 = require("./impressionsMode");
 var key_1 = require("../inputValidation/key");
-var trafficType_1 = require("../inputValidation/trafficType");
 var constants_2 = require("../../logger/constants");
 // Exported for telemetry
 exports.base = {
@@ -18,8 +17,6 @@ exports.base = {
         authorizationKey: undefined,
         // key used in your system (only required for browser version)
         key: undefined,
-        // traffic type for the given key (only used on browser version)
-        trafficType: undefined,
         // toggle impressions tracking of labels
         labelsEnabled: true,
         // toggle sendind (true) or not sending (false) IP and Host Name with impressions, events, and telemetries requests (only used on nodejs version)
@@ -148,12 +145,6 @@ function settingsValidation(config, validationParams) {
             // @ts-ignore, @TODO handle invalid keys as a non-recoverable error?
             withDefaults.core.key = (0, key_1.validateKey)(log, maybeKey, constants_2.LOG_PREFIX_CLIENT_INSTANTIATION);
         }
-        if (validationParams.acceptTT) {
-            var maybeTT = withDefaults.core.trafficType;
-            if (maybeTT !== undefined) { // @ts-ignore
-                withDefaults.core.trafficType = (0, trafficType_1.validateTrafficType)(log, maybeTT, constants_2.LOG_PREFIX_CLIENT_INSTANTIATION);
-            }
-        }
     }
     else {
         // On server-side, key is undefined and used to distinguish from client-side

package/cjs/utils/settingsValidation/logger/builtinLogger.js

@@ -5,9 +5,8 @@ var logger_1 = require("../../../logger");
 var isLocalStorageAvailable_1 = require("../../env/isLocalStorageAvailable");
 var isNode_1 = require("../../env/isNode");
 var debug_1 = require("../../../logger/messages/debug");
-var maps_1 = require("../../lang/maps");
 var commons_1 = require("./commons");
-var allCodes = new maps_1._Map(debug_1.codesDebug);
+var allCodes = new Map(debug_1.codesDebug);
 // @TODO set default debug setting instead of initialLogLevel when integrating in JS and Node packages
 var LS_KEY = 'splitio_debug';
 var ENV_VAR_KEY = 'SPLITIO_DEBUG';

package/esm/evaluator/Engine.js

@@ -14,7 +14,7 @@ var Engine = /** @class */ (function () {
     function Engine(baseInfo, evaluator) {
         this.baseInfo = baseInfo;
         this.evaluator = evaluator;
-        // in case we don't have a default treatment in the instanciation, use 'control'
+        // in case we don't have a default treatment in the instantiation, use 'control'
         if (typeof this.baseInfo.defaultTreatment !== 'string') {
             this.baseInfo.defaultTreatment = CONTROL;
         }

package/esm/evaluator/index.js

@@ -2,7 +2,7 @@ import { Engine } from './Engine';
 import { thenable } from '../utils/promise/thenable';
 import { EXCEPTION, SPLIT_NOT_FOUND } from '../utils/labels';
 import { CONTROL } from '../utils/constants';
-import { setToArray, returnSetsUnion, _Set } from '../utils/lang/sets';
+import { returnSetsUnion, setToArray } from '../utils/lang/sets';
 import { WARN_FLAGSET_WITHOUT_FLAGS } from '../logger/constants';
 var treatmentException = {
     treatment: CONTROL,
@@ -54,7 +54,7 @@ export function evaluateFeatures(log, key, splitNames, attributes, storage) {
 export function evaluateFeaturesByFlagSets(log, key, flagSets, attributes, storage, method) {
     var storedFlagNames;
     function evaluate(featureFlagsByFlagSets) {
-        var featureFlags = new _Set();
+        var featureFlags = new Set();
         for (var i = 0; i < flagSets.length; i++) {
             var featureFlagByFlagSet = featureFlagsByFlagSets[i];
             if (featureFlagByFlagSet.size) {

package/esm/evaluator/matchers/semver_inlist.js

@@ -1,10 +1,9 @@
-import { _Set } from '../../utils/lang/sets';
 import { Semver } from '../../utils/Semver';
 export function inListSemverMatcherContext(ruleAttr) {
     // @TODO ruleAttr validation should be done at the `parser` or `matchersTransform` level to reuse for all matchers
     if (!ruleAttr || ruleAttr.length === 0)
         throw new Error('whitelistMatcherData is required for IN_LIST_SEMVER matcher type');
-    var listOfSemvers = new _Set(ruleAttr.map(function (version) { return new Semver(version).version; }));
+    var listOfSemvers = new Set(ruleAttr.map(function (version) { return new Semver(version).version; }));
     return function inListSemverMatcher(runtimeAttr) {
         var runtimeSemver = new Semver(runtimeAttr).version;
         var isInList = listOfSemvers.has(runtimeSemver);

package/esm/evaluator/matchers/whitelist.js

@@ -1,6 +1,5 @@
-import { _Set } from '../../utils/lang/sets';
 export function whitelistMatcherContext(ruleAttr) {
-    var whitelistSet = new _Set(ruleAttr);
+    var whitelistSet = new Set(ruleAttr);
     return function whitelistMatcher(runtimeAttr) {
         var isInWhitelist = whitelistSet.has(runtimeAttr);
         return isInWhitelist;

package/esm/listeners/browser.js

@@ -97,7 +97,6 @@ var BrowserSignalListener = /** @class */ (function () {
      * Returns true if beacon API was used successfully, false otherwise.
      */
     BrowserSignalListener.prototype._sendBeacon = function (url, data, extraMetadata) {
-        // eslint-disable-next-line compat/compat
         if (typeof navigator !== 'undefined' && navigator.sendBeacon) {
             var json = {
                 entries: data,
@@ -110,7 +109,7 @@ var BrowserSignalListener = /** @class */ (function () {
             // Stringify the payload
             var payload = JSON.stringify(json);
             // https://xgwang.me/posts/you-may-not-know-beacon/#it-may-throw-error%2C-be-sure-to-catch
-            try { // eslint-disable-next-line compat/compat
+            try {
                 return navigator.sendBeacon(url, payload);
             }
             catch (e) {

package/esm/logger/browser/DebugLogger.js

@@ -1,6 +1,5 @@
 import { Logger } from '../index';
 import { codesDebug } from '../messages/debug';
-import { _Map } from '../../utils/lang/maps';
 export function DebugLogger() {
-    return new Logger({ logLevel: 'DEBUG' }, new _Map(codesDebug));
+    return new Logger({ logLevel: 'DEBUG' }, new Map(codesDebug));
 }

package/esm/logger/browser/ErrorLogger.js

@@ -1,6 +1,5 @@
 import { Logger } from '../index';
 import { codesError } from '../messages/error';
-import { _Map } from '../../utils/lang/maps';
 export function ErrorLogger() {
-    return new Logger({ logLevel: 'ERROR' }, new _Map(codesError));
+    return new Logger({ logLevel: 'ERROR' }, new Map(codesError));
 }

package/esm/logger/browser/InfoLogger.js

@@ -1,6 +1,5 @@
 import { Logger } from '../index';
 import { codesInfo } from '../messages/info';
-import { _Map } from '../../utils/lang/maps';
 export function InfoLogger() {
-    return new Logger({ logLevel: 'INFO' }, new _Map(codesInfo));
+    return new Logger({ logLevel: 'INFO' }, new Map(codesInfo));
 }

package/esm/logger/browser/WarnLogger.js

@@ -1,6 +1,5 @@
 import { Logger } from '../index';
 import { codesWarn } from '../messages/warn';
-import { _Map } from '../../utils/lang/maps';
 export function WarnLogger() {
-    return new Logger({ logLevel: 'WARN' }, new _Map(codesWarn));
+    return new Logger({ logLevel: 'WARN' }, new Map(codesWarn));
 }

package/esm/logger/index.js

@@ -1,6 +1,5 @@
 import { objectAssign } from '../utils/lang/objectAssign';
 import { find, isObject } from '../utils/lang';
-import { _Map } from '../utils/lang/maps';
 export var LogLevels = {
     DEBUG: 'DEBUG',
     INFO: 'INFO',
@@ -42,7 +41,7 @@ var defaultOptions = {
 var Logger = /** @class */ (function () {
     function Logger(options, codes) {
         this.options = objectAssign({}, defaultOptions, options);
-        this.codes = codes || new _Map();
+        this.codes = codes || new Map();
         this.logLevel = LogLevelIndexes[this.options.logLevel];
     }
     Logger.prototype.setLogLevel = function (logLevel) {

package/esm/sdkClient/clientCS.js

@@ -1,16 +1,15 @@
 import { objectAssign } from '../utils/lang/objectAssign';
 import { clientAttributesDecoration } from './clientAttributesDecoration';
 /**
- * Decorator that binds a key and (optionally) a traffic type to client methods
+ * Decorator that binds a key to client methods
  *
  * @param client sync client instance
  * @param key validated split key
- * @param trafficType validated traffic type
  */
-export function clientCSDecorator(log, client, key, trafficType) {
+export function clientCSDecorator(log, client, key) {
     var clientCS = clientAttributesDecoration(log, client);
     return objectAssign(clientCS, {
-        // In the client-side API, we bind a key to the client `getTreatment*` methods
+        // In the client-side API, we bind a key to the client `getTreatment*` and `track` methods
         getTreatment: clientCS.getTreatment.bind(clientCS, key),
         getTreatmentWithConfig: clientCS.getTreatmentWithConfig.bind(clientCS, key),
         getTreatments: clientCS.getTreatments.bind(clientCS, key),
@@ -19,11 +18,9 @@ export function clientCSDecorator(log, client, key, trafficType) {
         getTreatmentsWithConfigByFlagSets: clientCS.getTreatmentsWithConfigByFlagSets.bind(clientCS, key),
         getTreatmentsByFlagSet: clientCS.getTreatmentsByFlagSet.bind(clientCS, key),
         getTreatmentsWithConfigByFlagSet: clientCS.getTreatmentsWithConfigByFlagSet.bind(clientCS, key),
-        // Key is bound to the `track` method. Same thing happens with trafficType but only if provided
-        track: trafficType ? clientCS.track.bind(clientCS, key, trafficType) : clientCS.track.bind(clientCS, key),
+        track: clientCS.track.bind(clientCS, key),
         // Not part of the public API. These properties are used to support other modules (e.g., Split Suite)
         isClientSide: true,
-        key: key,
-        trafficType: trafficType
+        key: key
     });
 }

package/esm/sdkClient/sdkClientMethodCS.js

@@ -22,7 +22,7 @@ export function sdkClientMethodCSFactory(params) {
             log.debug(RETRIEVE_CLIENT_DEFAULT);
             return mainClientInstance;
         }
-        // Validate the key value. The trafficType (2nd argument) is ignored
+        // Validate the key value
         var validKey = validateKey(log, key, LOG_PREFIX_CLIENT_INSTANTIATION);
         if (validKey === false) {
             throw new Error('Shared Client needs a valid key.');

package/esm/services/decorateHeaders.js

@@ -1,6 +1,5 @@
 import { objectAssign } from '../utils/lang/objectAssign';
-import { _Set } from '../utils/lang/sets';
-var FORBIDDEN_HEADERS = new _Set([
+var FORBIDDEN_HEADERS = new Set([
     'splitsdkclientkey',
     'splitsdkversion',
     'splitsdkmachineip',

package/esm/storages/KeyBuilderCS.js

@@ -20,12 +20,6 @@ var KeyBuilderCS = /** @class */ (function (_super) {
         if (startsWith(builtSegmentKeyName, prefix))
             return builtSegmentKeyName.substr(prefix.length);
     };
-    // @BREAKING: The key used to start with the matching key instead of the prefix, this was changed on version 10.17.3
-    KeyBuilderCS.prototype.extractOldSegmentKey = function (builtSegmentKeyName) {
-        var prefix = this.matchingKey + "." + this.prefix + ".segment.";
-        if (startsWith(builtSegmentKeyName, prefix))
-            return builtSegmentKeyName.substr(prefix.length);
-    };
     KeyBuilderCS.prototype.buildLastUpdatedKey = function () {
         return this.prefix + ".splits.lastUpdated";
     };
@@ -48,9 +42,6 @@ export function myLargeSegmentsKeyBuilder(prefix, matchingKey) {
             if (startsWith(builtSegmentKeyName, p))
                 return builtSegmentKeyName.substr(p.length);
         },
-        extractOldSegmentKey: function () {
-            return undefined;
-        },
         buildTillKey: function () {
             return prefix + "." + matchingKey + ".largeSegments.till";
         }

package/esm/storages/inLocalStorage/MySegmentsCacheInLocal.js

@@ -45,28 +45,8 @@ var MySegmentsCacheInLocal = /** @class */ (function (_super) {
         // Scan current values from localStorage
         return Object.keys(localStorage).reduce(function (accum, key) {
             var segmentName = _this.keys.extractSegmentName(key);
-            if (segmentName) {
+            if (segmentName)
                 accum.push(segmentName);
-            }
-            else {
-                // @TODO @BREAKING: This is only to clean up "old" keys. Remove this whole else code block
-                segmentName = _this.keys.extractOldSegmentKey(key);
-                if (segmentName) { // this was an old segment key, let's clean up.
-                    var newSegmentKey = _this.keys.buildSegmentNameKey(segmentName);
-                    try {
-                        // If the new format key is not there, create it.
-                        if (!localStorage.getItem(newSegmentKey)) {
-                            localStorage.setItem(newSegmentKey, DEFINED);
-                            // we are migrating a segment, let's track it.
-                            accum.push(segmentName);
-                        }
-                        localStorage.removeItem(key); // we migrated the current key, let's delete it.
-                    }
-                    catch (e) {
-                        _this.log.error(e);
-                    }
-                }
-            }
             return accum;
         }, []);
     };