@splitsoftware/splitio-commons 1.13.2-rc.9 → 1.14.1-rc.0

package/esm/evaluator/matchers/all.js

@@ -1,7 +1,5 @@
-import { ENGINE_MATCHER_ALL } from '../../logger/constants';
-export function allMatcherContext(log) {
+export function allMatcherContext() {
     return function allMatcher(runtimeAttr) {
-        log.debug(ENGINE_MATCHER_ALL);
         return runtimeAttr != null;
     };
 }

package/esm/evaluator/matchers/between.js

@@ -1,8 +1,6 @@
-import { ENGINE_MATCHER_BETWEEN } from '../../logger/constants';
-export function betweenMatcherContext(log, ruleVO) {
+export function betweenMatcherContext(ruleVO) {
     return function betweenMatcher(runtimeAttr) {
         var isBetween = runtimeAttr >= ruleVO.start && runtimeAttr <= ruleVO.end;
-        log.debug(ENGINE_MATCHER_BETWEEN, [runtimeAttr, ruleVO.start, ruleVO.end, isBetween]);
         return isBetween;
     };
 }

package/esm/evaluator/matchers/boolean.js

@@ -1,8 +1,6 @@
-import { ENGINE_MATCHER_BOOLEAN } from '../../logger/constants';
-export function booleanMatcherContext(log, ruleAttr) {
+export function booleanMatcherContext(ruleAttr) {
     return function booleanMatcher(runtimeAttr) {
         var booleanMatches = ruleAttr === runtimeAttr;
-        log.debug(ENGINE_MATCHER_BOOLEAN, [ruleAttr, runtimeAttr]);
         return booleanMatches;
     };
 }

package/esm/evaluator/matchers/cont_all.js

@@ -1,6 +1,5 @@
-import { ENGINE_MATCHER_CONTAINS_ALL } from '../../logger/constants';
 import { findIndex } from '../../utils/lang';
-export function containsAllSetMatcherContext(log, ruleAttr) {
+export function containsAllSetMatcherContext(ruleAttr) {
     return function containsAllMatcher(runtimeAttr) {
         var containsAll = true;
         if (runtimeAttr.length < ruleAttr.length) {
@@ -15,7 +14,6 @@ export function containsAllSetMatcherContext(log, ruleAttr) {
                 _loop_1(i);
             }
         }
-        log.debug(ENGINE_MATCHER_CONTAINS_ALL, [runtimeAttr, ruleAttr, containsAll]);
         return containsAll;
     };
 }

package/esm/evaluator/matchers/cont_any.js

@@ -1,6 +1,5 @@
-import { ENGINE_MATCHER_CONTAINS_ANY } from '../../logger/constants';
 import { findIndex } from '../../utils/lang';
-export function containsAnySetMatcherContext(log, ruleAttr) {
+export function containsAnySetMatcherContext(ruleAttr) {
     return function containsAnyMatcher(runtimeAttr) {
         var containsAny = false;
         var _loop_1 = function (i) {
@@ -10,7 +9,6 @@ export function containsAnySetMatcherContext(log, ruleAttr) {
         for (var i = 0; i < ruleAttr.length && !containsAny; i++) {
             _loop_1(i);
         }
-        log.debug(ENGINE_MATCHER_CONTAINS_ANY, [runtimeAttr, ruleAttr, containsAny]);
         return containsAny;
     };
 }

package/esm/evaluator/matchers/cont_str.js

@@ -1,9 +1,7 @@
 import { isString } from '../../utils/lang';
-import { ENGINE_MATCHER_CONTAINS_STRING } from '../../logger/constants';
-export function containsStringMatcherContext(log, ruleAttr) {
+export function containsStringMatcherContext(ruleAttr) {
     return function containsStringMatcher(runtimeAttr) {
         var contains = ruleAttr.some(function (e) { return isString(runtimeAttr) && runtimeAttr.indexOf(e) > -1; });
-        log.debug(ENGINE_MATCHER_CONTAINS_STRING, [runtimeAttr, ruleAttr, contains]);
         return contains;
     };
 }

package/esm/evaluator/matchers/dependency.js

@@ -1,6 +1,6 @@
 import { thenable } from '../../utils/promise/thenable';
 import { ENGINE_MATCHER_DEPENDENCY, ENGINE_MATCHER_DEPENDENCY_PRE } from '../../logger/constants';
-export function dependencyMatcherContext(log, _a, storage) {
+export function dependencyMatcherContext(_a, storage, log) {
     var split = _a.split, treatments = _a.treatments;
     function checkTreatment(evaluation, acceptableTreatments, parentName) {
         var matches = false;

package/esm/evaluator/matchers/eq.js

@@ -1,8 +1,6 @@
-import { ENGINE_MATCHER_EQUAL } from '../../logger/constants';
-export function equalToMatcherContext(log, ruleAttr) {
+export function equalToMatcherContext(ruleAttr) {
     return function equalToMatcher(runtimeAttr) {
         var isEqual = runtimeAttr === ruleAttr;
-        log.debug(ENGINE_MATCHER_EQUAL, [runtimeAttr, ruleAttr, isEqual]);
         return isEqual;
     };
 }

package/esm/evaluator/matchers/eq_set.js

@@ -1,6 +1,5 @@
-import { ENGINE_MATCHER_EQUAL_TO_SET } from '../../logger/constants';
 import { findIndex } from '../../utils/lang';
-export function equalToSetMatcherContext(log, ruleAttr) {
+export function equalToSetMatcherContext(ruleAttr) {
     return function equalToSetMatcher(runtimeAttr) {
         // Length being the same is the first condition.
         var isEqual = runtimeAttr.length === ruleAttr.length;
@@ -12,7 +11,6 @@ export function equalToSetMatcherContext(log, ruleAttr) {
         for (var i = 0; i < runtimeAttr.length && isEqual; i++) {
             _loop_1(i);
         }
-        log.debug(ENGINE_MATCHER_EQUAL_TO_SET, [runtimeAttr, ruleAttr, isEqual]);
         return isEqual;
     };
 }

package/esm/evaluator/matchers/ew.js

@@ -1,9 +1,7 @@
-import { ENGINE_MATCHER_ENDS_WITH } from '../../logger/constants';
 import { endsWith } from '../../utils/lang';
-export function endsWithMatcherContext(log, ruleAttr) {
+export function endsWithMatcherContext(ruleAttr) {
     return function endsWithMatcher(runtimeAttr) {
         var strEndsWith = ruleAttr.some(function (e) { return endsWith(runtimeAttr, e); });
-        log.debug(ENGINE_MATCHER_ENDS_WITH, [runtimeAttr, ruleAttr, strEndsWith]);
         return strEndsWith;
     };
 }

package/esm/evaluator/matchers/gte.js

@@ -1,8 +1,6 @@
-import { ENGINE_MATCHER_GREATER } from '../../logger/constants';
-export function greaterThanEqualMatcherContext(log, ruleAttr) {
+export function greaterThanEqualMatcherContext(ruleAttr) {
     return function greaterThanEqualMatcher(runtimeAttr) {
         var isGreaterThanEqual = runtimeAttr >= ruleAttr;
-        log.debug(ENGINE_MATCHER_GREATER, [runtimeAttr, ruleAttr, isGreaterThanEqual]);
         return isGreaterThanEqual;
     };
 }

package/esm/evaluator/matchers/index.js

@@ -53,6 +53,6 @@ export function matcherFactory(log, matcherDto, storage) {
     var matcherFn;
     // @ts-ignore
     if (matchers[type])
-        matcherFn = matchers[type](log, value, storage); // There is no index-out-of-bound exception in JavaScript
+        matcherFn = matchers[type](value, storage, log); // There is no index-out-of-bound exception in JavaScript
     return matcherFn;
 }

package/esm/evaluator/matchers/lte.js

@@ -1,8 +1,6 @@
-import { ENGINE_MATCHER_LESS } from '../../logger/constants';
-export function lessThanEqualMatcherContext(log, ruleAttr) {
+export function lessThanEqualMatcherContext(ruleAttr) {
     return function lessThanEqualMatcher(runtimeAttr) {
         var isLessThanEqual = runtimeAttr <= ruleAttr;
-        log.debug(ENGINE_MATCHER_LESS, [runtimeAttr, ruleAttr, isLessThanEqual]);
         return isLessThanEqual;
     };
 }

package/esm/evaluator/matchers/part_of.js

@@ -1,6 +1,5 @@
 import { findIndex } from '../../utils/lang';
-import { ENGINE_MATCHER_PART_OF } from '../../logger/constants';
-export function partOfSetMatcherContext(log, ruleAttr) {
+export function partOfSetMatcherContext(ruleAttr) {
     return function partOfMatcher(runtimeAttr) {
         // To be part of the length should be minor or equal.
         var isPartOf = runtimeAttr.length <= ruleAttr.length;
@@ -12,7 +11,6 @@ export function partOfSetMatcherContext(log, ruleAttr) {
         for (var i = 0; i < runtimeAttr.length && isPartOf; i++) {
             _loop_1(i);
         }
-        log.debug(ENGINE_MATCHER_PART_OF, [runtimeAttr, ruleAttr, isPartOf]);
         return isPartOf;
     };
 }

package/esm/evaluator/matchers/segment.js

@@ -1,17 +1,12 @@
 import { thenable } from '../../utils/promise/thenable';
-import { ENGINE_MATCHER_SEGMENT } from '../../logger/constants';
-export function segmentMatcherContext(log, segmentName, storage) {
+export function segmentMatcherContext(segmentName, storage) {
     return function segmentMatcher(key) {
         var isInSegment = storage.segments.isInSegment(segmentName, key);
         if (thenable(isInSegment)) {
             isInSegment.then(function (result) {
-                log.debug(ENGINE_MATCHER_SEGMENT, [segmentName, key, isInSegment]);
                 return result;
             });
         }
-        else {
-            log.debug(ENGINE_MATCHER_SEGMENT, [segmentName, key, isInSegment]);
-        }
         return isInSegment;
     };
 }

package/esm/evaluator/matchers/semver_between.js

@@ -1,5 +1,5 @@
 import { Semver } from '../../utils/Semver';
-export function betweenSemverMatcherContext(log, ruleAttr) {
+export function betweenSemverMatcherContext(ruleAttr) {
     var startSemver = new Semver(ruleAttr.start);
     var endSemver = new Semver(ruleAttr.end);
     return function betweenSemverMatcher(runtimeAttr) {

package/esm/evaluator/matchers/semver_eq.js

@@ -1,5 +1,5 @@
 import { Semver } from '../../utils/Semver';
-export function equalToSemverMatcherContext(log, ruleAttr) {
+export function equalToSemverMatcherContext(ruleAttr) {
     var ruleSemver = new Semver(ruleAttr);
     return function equalToSemverMatcher(runtimeAttr) {
         var runtimeSemver = new Semver(runtimeAttr);

package/esm/evaluator/matchers/semver_gte.js

@@ -1,5 +1,5 @@
 import { Semver } from '../../utils/Semver';
-export function greaterThanEqualToSemverMatcherContext(log, ruleAttr) {
+export function greaterThanEqualToSemverMatcherContext(ruleAttr) {
     var ruleSemver = new Semver(ruleAttr);
     return function greaterThanEqualToSemverMatcher(runtimeAttr) {
         var runtimeSemver = new Semver(runtimeAttr);

package/esm/evaluator/matchers/semver_inlist.js

@@ -1,6 +1,6 @@
 import { _Set } from '../../utils/lang/sets';
 import { Semver } from '../../utils/Semver';
-export function inListSemverMatcherContext(log, ruleAttr) {
+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');

package/esm/evaluator/matchers/semver_lte.js

@@ -1,5 +1,5 @@
 import { Semver } from '../../utils/Semver';
-export function lessThanEqualToSemverMatcherContext(log, ruleAttr) {
+export function lessThanEqualToSemverMatcherContext(ruleAttr) {
     var ruleSemver = new Semver(ruleAttr);
     return function lessThanEqualToSemverMatcher(runtimeAttr) {
         var runtimeSemver = new Semver(runtimeAttr);

package/esm/evaluator/matchers/string.js

@@ -1,9 +1,7 @@
-import { ENGINE_MATCHER_STRING } from '../../logger/constants';
-export function stringMatcherContext(log, ruleAttr) {
+export function stringMatcherContext(ruleAttr) {
     var regex = new RegExp(ruleAttr);
     return function stringMatcher(runtimeAttr) {
         var regexMatches = regex.test(runtimeAttr);
-        log.debug(ENGINE_MATCHER_STRING, [runtimeAttr, ruleAttr, regexMatches ? 'yes' : 'no']);
         return regexMatches;
     };
 }

package/esm/evaluator/matchers/sw.js

@@ -1,9 +1,7 @@
-import { ENGINE_MATCHER_STARTS_WITH } from '../../logger/constants';
 import { startsWith } from '../../utils/lang';
-export function startsWithMatcherContext(log, ruleAttr) {
+export function startsWithMatcherContext(ruleAttr) {
     return function startsWithMatcher(runtimeAttr) {
         var matches = ruleAttr.some(function (e) { return startsWith(runtimeAttr, e); });
-        log.debug(ENGINE_MATCHER_STARTS_WITH, [runtimeAttr, ruleAttr, matches]);
         return matches;
     };
 }

package/esm/evaluator/matchers/whitelist.js

@@ -1,10 +1,8 @@
 import { _Set } from '../../utils/lang/sets';
-import { ENGINE_MATCHER_WHITELIST } from '../../logger/constants';
-export function whitelistMatcherContext(log, ruleAttr) {
+export function whitelistMatcherContext(ruleAttr) {
     var whitelistSet = new _Set(ruleAttr);
     return function whitelistMatcher(runtimeAttr) {
         var isInWhitelist = whitelistSet.has(runtimeAttr);
-        log.debug(ENGINE_MATCHER_WHITELIST, [runtimeAttr, ruleAttr.join(','), isInWhitelist]);
         return isInWhitelist;
     };
 }

package/esm/evaluator/parser/index.js

@@ -6,7 +6,7 @@ import { conditionContext } from '../condition';
 import { ifElseIfCombinerContext } from '../combiners/ifelseif';
 import { andCombinerContext } from '../combiners/and';
 import { thenable } from '../../utils/promise/thenable';
-import { ENGINE_MATCHER_ERROR } from '../../logger/constants';
+import { ENGINE_MATCHER_ERROR, ENGINE_MATCHER_RESULT } from '../../logger/constants';
 export function parser(log, conditions, storage) {
     var predicates = [];
     for (var i = 0; i < conditions.length; i++) {
@@ -34,10 +34,13 @@ export function parser(log, conditions, storage) {
                         log.error(ENGINE_MATCHER_ERROR, [matcherDto.name, error]);
                     }
                 }
-                if (thenable(result)) { // @ts-ignore
-                    return result.then(function (res) { return Boolean(res ^ matcherDto.negate); });
-                } // @ts-ignore
-                return Boolean(result ^ matcherDto.negate);
+                function handleResult(result) {
+                    log.debug(ENGINE_MATCHER_RESULT, [matcherDto.name, result, matcherDto.value, value]); // @ts-ignore
+                    return Boolean(result ^ matcherDto.negate);
+                }
+                return thenable(result) ?
+                    result.then(handleResult) :
+                    handleResult(result);
             };
         });
         // if matcher's factory can't instantiate the matchers, the expressions array will be empty

package/esm/logger/constants.js

@@ -9,24 +9,8 @@ export var ENGINE_COMBINER_AND = 0;
 export var ENGINE_COMBINER_IFELSEIF = 1;
 export var ENGINE_COMBINER_IFELSEIF_NO_TREATMENT = 2;
 export var ENGINE_BUCKET = 3;
-export var ENGINE_MATCHER_ALL = 4;
-export var ENGINE_MATCHER_BETWEEN = 5;
-export var ENGINE_MATCHER_BOOLEAN = 6;
-export var ENGINE_MATCHER_CONTAINS_ALL = 7;
-export var ENGINE_MATCHER_CONTAINS_ANY = 8;
-export var ENGINE_MATCHER_CONTAINS_STRING = 9;
 export var ENGINE_MATCHER_DEPENDENCY = 10;
 export var ENGINE_MATCHER_DEPENDENCY_PRE = 11;
-export var ENGINE_MATCHER_EQUAL = 12;
-export var ENGINE_MATCHER_EQUAL_TO_SET = 13;
-export var ENGINE_MATCHER_ENDS_WITH = 14;
-export var ENGINE_MATCHER_GREATER = 15;
-export var ENGINE_MATCHER_LESS = 16;
-export var ENGINE_MATCHER_PART_OF = 17;
-export var ENGINE_MATCHER_SEGMENT = 18;
-export var ENGINE_MATCHER_STRING = 19;
-export var ENGINE_MATCHER_STARTS_WITH = 21;
-export var ENGINE_MATCHER_WHITELIST = 22;
 export var ENGINE_VALUE = 23;
 export var ENGINE_SANITIZE = 24;
 export var CLEANUP_REGISTERING = 25;
@@ -44,6 +28,7 @@ export var SYNC_TASK_START = 36;
 export var SYNC_TASK_EXECUTE = 37;
 export var SYNC_TASK_STOP = 38;
 export var SETTINGS_SPLITS_FILTER = 39;
+export var ENGINE_MATCHER_RESULT = 40;
 export var CLIENT_READY_FROM_CACHE = 100;
 export var CLIENT_READY = 101;
 export var IMPRESSION = 102;

package/esm/logger/messages/debug.js

@@ -6,26 +6,11 @@ export var codesDebug = codesInfo.concat([
     [c.ENGINE_COMBINER_IFELSEIF, c.LOG_PREFIX_ENGINE_COMBINER + 'Treatment found: %s'],
     [c.ENGINE_COMBINER_IFELSEIF_NO_TREATMENT, c.LOG_PREFIX_ENGINE_COMBINER + 'All predicates evaluated, no treatment found.'],
     [c.ENGINE_BUCKET, c.LOG_PREFIX_ENGINE + ': using algo "murmur" bucket %s for key %s using seed %s - treatment %s'],
-    [c.ENGINE_MATCHER_ALL, c.LOG_PREFIX_ENGINE_MATCHER + '[allMatcher] is always true'],
-    [c.ENGINE_MATCHER_BETWEEN, c.LOG_PREFIX_ENGINE_MATCHER + '[betweenMatcher] is %s between %s and %s? %s'],
-    [c.ENGINE_MATCHER_BOOLEAN, c.LOG_PREFIX_ENGINE_MATCHER + '[booleanMatcher] %s === %s'],
-    [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 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'],
-    [c.ENGINE_MATCHER_GREATER, c.LOG_PREFIX_ENGINE_MATCHER + '[greaterThanEqualMatcher] is %s greater than %s? %s'],
-    [c.ENGINE_MATCHER_LESS, c.LOG_PREFIX_ENGINE_MATCHER + '[lessThanEqualMatcher] is %s less than %s? %s'],
-    [c.ENGINE_MATCHER_PART_OF, c.LOG_PREFIX_ENGINE_MATCHER + '[partOfMatcher] %s is part of %s? %s'],
-    [c.ENGINE_MATCHER_SEGMENT, c.LOG_PREFIX_ENGINE_MATCHER + '[segmentMatcher] evaluated %s / %s => %s'],
-    [c.ENGINE_MATCHER_STRING, c.LOG_PREFIX_ENGINE_MATCHER + '[stringMatcher] does %s matches with %s? %s'],
-    [c.ENGINE_MATCHER_STARTS_WITH, c.LOG_PREFIX_ENGINE_MATCHER + '[startsWithMatcher] %s starts with %s? %s'],
-    [c.ENGINE_MATCHER_WHITELIST, c.LOG_PREFIX_ENGINE_MATCHER + '[whitelistMatcher] evaluated %s in [%s] => %s'],
-    [c.ENGINE_VALUE, c.LOG_PREFIX_ENGINE_VALUE + 'Extracted attribute [%s], [%s] will be used for matching.'],
-    [c.ENGINE_SANITIZE, c.LOG_PREFIX_ENGINE + ':sanitize: Attempted to sanitize [%s] which should be of type [%s]. Sanitized and processed value => [%s]'],
+    [c.ENGINE_MATCHER_DEPENDENCY, c.LOG_PREFIX_ENGINE_MATCHER + '[IN_SPLIT_TREATMENT] 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 + '[IN_SPLIT_TREATMENT] will evaluate parent feature flag: "%s" with key: %s %s'],
+    [c.ENGINE_VALUE, c.LOG_PREFIX_ENGINE_VALUE + 'Extracted attribute `%s`. %s will be used for matching.'],
+    [c.ENGINE_SANITIZE, c.LOG_PREFIX_ENGINE + ':sanitize: Attempted to sanitize %s which should be of type %s. Sanitized and processed value => %s'],
+    [c.ENGINE_MATCHER_RESULT, c.LOG_PREFIX_ENGINE_MATCHER + '[%s] Result: %s. Rule value: %s. Evaluation value: %s'],
     // SDK
     [c.CLEANUP_REGISTERING, c.LOG_PREFIX_CLEANUP + 'Registering cleanup handler %s'],
     [c.CLEANUP_DEREGISTERING, c.LOG_PREFIX_CLEANUP + 'Deregistering cleanup handler %s'],

package/esm/logger/messages/warn.js

@@ -3,7 +3,7 @@ import { codesError } from './error';
 export var codesWarn = codesError.concat([
     // evaluator
     [c.ENGINE_VALUE_INVALID, c.LOG_PREFIX_ENGINE_VALUE + 'Value %s doesn\'t match with expected type.'],
-    [c.ENGINE_VALUE_NO_ATTRIBUTES, c.LOG_PREFIX_ENGINE_VALUE + 'Defined attribute [%s], no attributes received.'],
+    [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 feature flags. %s'],

package/esm/services/splitApi.js

@@ -1,6 +1,6 @@
 import { splitHttpClientFactory } from './splitHttpClient';
 import { objectAssign } from '../utils/lang/objectAssign';
-import { SPLITS, IMPRESSIONS, IMPRESSIONS_COUNT, EVENTS, TELEMETRY, TOKEN, SEGMENT, MY_SEGMENT, FLAGS_SPEC } from '../utils/constants';
+import { SPLITS, IMPRESSIONS, IMPRESSIONS_COUNT, EVENTS, TELEMETRY, TOKEN, SEGMENT, MY_SEGMENT } from '../utils/constants';
 import { ERROR_TOO_MANY_SETS } from '../logger/constants';
 var noCacheHeaderOptions = { headers: { 'Cache-Control': 'no-cache' } };
 function userKeyToQueryParam(userKey) {
@@ -17,6 +17,7 @@ export function splitApiFactory(settings, platform, telemetryTracker) {
     var urls = settings.urls;
     var filterQueryString = settings.sync.__splitFiltersValidation && settings.sync.__splitFiltersValidation.queryString;
     var SplitSDKImpressionsMode = settings.sync.impressionsMode;
+    var flagSpecVersion = settings.sync.flagSpecVersion;
     var splitHttpClient = splitHttpClientFactory(settings, platform.getFetch);
     return {
         // @TODO throw errors if health check requests fail, to log them in the Synchronizer
@@ -29,7 +30,7 @@ export function splitApiFactory(settings, platform, telemetryTracker) {
             return splitHttpClient(url).then(function () { return true; }).catch(function () { return false; });
         },
         fetchAuth: function (userMatchingKeys) {
-            var url = urls.auth + "/v2/auth?s=" + FLAGS_SPEC;
+            var url = urls.auth + "/v2/auth?s=" + flagSpecVersion;
             if (userMatchingKeys) { // `userMatchingKeys` is undefined in server-side
                 var queryParams = userMatchingKeys.map(userKeyToQueryParam).join('&');
                 if (queryParams)
@@ -38,7 +39,7 @@ export function splitApiFactory(settings, platform, telemetryTracker) {
             return splitHttpClient(url, undefined, telemetryTracker.trackHttp(TOKEN));
         },
         fetchSplitChanges: function (since, noCache, till) {
-            var url = urls.sdk + "/splitChanges?s=" + FLAGS_SPEC + "&since=" + since + (filterQueryString || '') + (till ? '&till=' + till : '');
+            var url = urls.sdk + "/splitChanges?s=" + flagSpecVersion + "&since=" + since + (filterQueryString || '') + (till ? '&till=' + till : '');
             return splitHttpClient(url, noCache ? noCacheHeaderOptions : undefined, telemetryTracker.trackHttp(SPLITS))
                 .catch(function (err) {
                 if (err.statusCode === 414)

package/esm/storages/KeyBuilder.js

@@ -1,4 +1,3 @@
-import { FLAGS_SPEC } from '../utils/constants';
 import { startsWith } from '../utils/lang';
 import { hash } from '../utils/murmur3/murmur3';
 var everythingAtTheEnd = /[^.]+$/;
@@ -67,5 +66,5 @@ export { KeyBuilder };
  * The hash is in hexadecimal format (8 characters max, 32 bits).
  */
 export function getStorageHash(settings) {
-    return hash(settings.core.authorizationKey + "::" + settings.sync.__splitFiltersValidation.queryString + "::" + FLAGS_SPEC).toString(16);
+    return hash(settings.core.authorizationKey + "::" + settings.sync.__splitFiltersValidation.queryString + "::" + settings.sync.flagSpecVersion).toString(16);
 }

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

@@ -1,5 +1,5 @@
 import { isString } from '../../../utils/lang';
-var VERSION = '1.1';
+var ABLY_API_VERSION = '1.1';
 var CONTROL_CHANNEL_REGEX = /^control_/;
 /**
  * Build metadata headers for SSE connection.
@@ -57,7 +57,7 @@ var SSEClient = /** @class */ (function () {
             var params = CONTROL_CHANNEL_REGEX.test(channel) ? '[?occupancy=metrics.publishers]' : '';
             return encodeURIComponent(params + channel);
         }).join(',');
-        var url = this.streamingUrl + "?channels=" + channelsQueryParam + "&accessToken=" + authToken.token + "&v=" + VERSION + "&heartbeats=true"; // same results using `&heartbeats=false`
+        var url = this.streamingUrl + "?channels=" + channelsQueryParam + "&accessToken=" + authToken.token + "&v=" + ABLY_API_VERSION + "&heartbeats=true"; // same results using `&heartbeats=false`
         this.connection = new this.eventSource(
         // For client-side SDKs, SplitSDKClientKey and SplitSDKClientKey metadata is passed as query params,
         // because native EventSource implementations for browser doesn't support headers.

package/esm/utils/constants/index.js

@@ -86,4 +86,4 @@ export var NON_REQUESTED = 1;
 export var DISABLED = 0;
 export var ENABLED = 1;
 export var PAUSED = 2;
-export var FLAGS_SPEC = '1.1';
+export var FLAG_SPEC_VERSION = '1.1';

package/esm/utils/settingsValidation/index.js

@@ -1,7 +1,7 @@
 import { merge, get } from '../lang';
 import { validateMode } from './mode';
 import { validateSplitFilters } from './splitFilters';
-import { STANDALONE_MODE, OPTIMIZED, LOCALHOST_MODE, DEBUG } from '../constants';
+import { STANDALONE_MODE, OPTIMIZED, LOCALHOST_MODE, DEBUG, FLAG_SPEC_VERSION } from '../constants';
 import { validImpressionsMode } from './impressionsMode';
 import { validateKey } from '../inputValidation/key';
 import { validateTrafficType } from '../inputValidation/trafficType';
@@ -71,7 +71,8 @@ export var base = {
         // impressions collection mode
         impressionsMode: OPTIMIZED,
         localhostMode: undefined,
-        enabled: true
+        enabled: true,
+        flagSpecVersion: FLAG_SPEC_VERSION
     },
     // Logger
     log: undefined
@@ -87,7 +88,7 @@ function fromSecondsToMillis(n) {
  * @param validationParams defaults and fields validators used to validate and creates a settings object from a given config
  */
 export function settingsValidation(config, validationParams) {
-    var defaults = validationParams.defaults, runtime = validationParams.runtime, storage = validationParams.storage, integrations = validationParams.integrations, logger = validationParams.logger, localhost = validationParams.localhost, consent = validationParams.consent;
+    var defaults = validationParams.defaults, runtime = validationParams.runtime, storage = validationParams.storage, integrations = validationParams.integrations, logger = validationParams.logger, localhost = validationParams.localhost, consent = validationParams.consent, flagSpec = validationParams.flagSpec;
     // creates a settings object merging base, defaults and config objects.
     var withDefaults = merge({}, base, defaults, config);
     // ensure a valid logger.
@@ -95,7 +96,8 @@ export function settingsValidation(config, validationParams) {
     var log = logger(withDefaults); // @ts-ignore, modify readonly prop
     withDefaults.log = log;
     // ensure a valid impressionsMode
-    withDefaults.sync.impressionsMode = validImpressionsMode(log, withDefaults.sync.impressionsMode);
+    var sync = withDefaults.sync;
+    sync.impressionsMode = validImpressionsMode(log, sync.impressionsMode);
     function validateMinValue(paramName, actualValue, minValue) {
         if (actualValue >= minValue)
             return actualValue;
@@ -111,7 +113,7 @@ export function settingsValidation(config, validationParams) {
     scheduler.eventsPushRate = fromSecondsToMillis(scheduler.eventsPushRate);
     scheduler.telemetryRefreshRate = fromSecondsToMillis(validateMinValue('telemetryRefreshRate', scheduler.telemetryRefreshRate, 60));
     // Default impressionsRefreshRate for DEBUG mode is 60 secs
-    if (get(config, 'scheduler.impressionsRefreshRate') === undefined && withDefaults.sync.impressionsMode === DEBUG)
+    if (get(config, 'scheduler.impressionsRefreshRate') === undefined && sync.impressionsMode === DEBUG)
         scheduler.impressionsRefreshRate = 60;
     scheduler.impressionsRefreshRate = fromSecondsToMillis(scheduler.impressionsRefreshRate);
     // Log deprecation for old telemetry param
@@ -163,22 +165,23 @@ export function settingsValidation(config, validationParams) {
     if (integrations)
         withDefaults.integrations = integrations(withDefaults);
     if (localhost)
-        withDefaults.sync.localhostMode = localhost(withDefaults);
+        sync.localhostMode = localhost(withDefaults);
     // validate push options
     if (withDefaults.streamingEnabled !== false) { // @ts-ignore, modify readonly prop
         withDefaults.streamingEnabled = true;
         // Backoff bases.
-        // We are not checking if bases are positive numbers. Thus, we might be reauthenticating immediately (`setTimeout` with NaN or negative number)
+        // We are not checking if bases are positive numbers. Thus, we might be re-authenticating immediately (`setTimeout` with NaN or negative number)
         scheduler.pushRetryBackoffBase = fromSecondsToMillis(scheduler.pushRetryBackoffBase);
     }
     // validate sync enabled
-    if (withDefaults.sync.enabled !== false) { // @ts-ignore, modify readonly prop
-        withDefaults.sync.enabled = true;
+    if (sync.enabled !== false) {
+        sync.enabled = true;
     }
     // validate the `splitFilters` settings and parse splits query
-    var splitFiltersValidation = validateSplitFilters(log, withDefaults.sync.splitFilters, withDefaults.mode);
-    withDefaults.sync.splitFilters = splitFiltersValidation.validFilters;
-    withDefaults.sync.__splitFiltersValidation = splitFiltersValidation;
+    var splitFiltersValidation = validateSplitFilters(log, sync.splitFilters, withDefaults.mode);
+    sync.splitFilters = splitFiltersValidation.validFilters;
+    sync.__splitFiltersValidation = splitFiltersValidation;
+    sync.flagSpecVersion = flagSpec ? flagSpec(withDefaults) : FLAG_SPEC_VERSION;
     // ensure a valid user consent value
     // @ts-ignore, modify readonly prop
     withDefaults.userConsent = consent(withDefaults);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@splitsoftware/splitio-commons",
-  "version": "1.13.2-rc.9",
+  "version": "1.14.1-rc.0",
   "description": "Split JavaScript SDK common components",
   "main": "cjs/index.js",
   "module": "esm/index.js",

package/src/evaluator/matchers/all.ts

@@ -1,10 +1,6 @@
-import { ENGINE_MATCHER_ALL } from '../../logger/constants';
-import { ILogger } from '../../logger/types';
+export function allMatcherContext() {
 
-export function allMatcherContext(log: ILogger) {
   return function allMatcher(runtimeAttr: string): boolean {
-    log.debug(ENGINE_MATCHER_ALL);
-
     return runtimeAttr != null;
   };
 }

package/src/evaluator/matchers/between.ts

@@ -1,13 +1,9 @@
 import { IBetweenMatcherData } from '../../dtos/types';
-import { ENGINE_MATCHER_BETWEEN } from '../../logger/constants';
-import { ILogger } from '../../logger/types';
 
-export function betweenMatcherContext(log: ILogger, ruleVO: IBetweenMatcherData) {
-  return function betweenMatcher(runtimeAttr: number): boolean {
-
-    let isBetween = runtimeAttr >= ruleVO.start && runtimeAttr <= ruleVO.end;
+export function betweenMatcherContext(ruleVO: IBetweenMatcherData) {
 
-    log.debug(ENGINE_MATCHER_BETWEEN, [runtimeAttr, ruleVO.start, ruleVO.end, isBetween]);
+  return function betweenMatcher(runtimeAttr: number): boolean {
+    const isBetween = runtimeAttr >= ruleVO.start && runtimeAttr <= ruleVO.end;
 
     return isBetween;
   };

package/src/evaluator/matchers/boolean.ts

@@ -1,11 +1,7 @@
-import { ENGINE_MATCHER_BOOLEAN } from '../../logger/constants';
-import { ILogger } from '../../logger/types';
+export function booleanMatcherContext(ruleAttr: boolean) {
 
-export function booleanMatcherContext(log: ILogger, ruleAttr: boolean) {
   return function booleanMatcher(runtimeAttr: boolean): boolean {
-    let booleanMatches = ruleAttr === runtimeAttr;
-
-    log.debug(ENGINE_MATCHER_BOOLEAN, [ruleAttr, runtimeAttr]);
+    const booleanMatches = ruleAttr === runtimeAttr;
 
     return booleanMatches;
   };

package/src/evaluator/matchers/cont_all.ts

@@ -1,8 +1,6 @@
-import { ENGINE_MATCHER_CONTAINS_ALL } from '../../logger/constants';
-import { ILogger } from '../../logger/types';
 import { findIndex } from '../../utils/lang';
 
-export function containsAllSetMatcherContext(log: ILogger, ruleAttr: string[]) {
+export function containsAllSetMatcherContext(ruleAttr: string[]) {
   return function containsAllMatcher(runtimeAttr: string[]): boolean {
     let containsAll = true;
 
@@ -15,8 +13,6 @@ export function containsAllSetMatcherContext(log: ILogger, ruleAttr: string[]) {
       }
     }
 
-    log.debug(ENGINE_MATCHER_CONTAINS_ALL, [runtimeAttr, ruleAttr, containsAll]);
-
     return containsAll;
   };
 }

package/src/evaluator/matchers/cont_any.ts

@@ -1,8 +1,6 @@
-import { ENGINE_MATCHER_CONTAINS_ANY } from '../../logger/constants';
-import { ILogger } from '../../logger/types';
 import { findIndex } from '../../utils/lang';
 
-export function containsAnySetMatcherContext(log: ILogger, ruleAttr: string[]) {
+export function containsAnySetMatcherContext(ruleAttr: string[]) {
   return function containsAnyMatcher(runtimeAttr: string[]): boolean {
     let containsAny = false;
 
@@ -10,8 +8,6 @@ export function containsAnySetMatcherContext(log: ILogger, ruleAttr: string[]) {
       if (findIndex(runtimeAttr, e => e === ruleAttr[i]) >= 0) containsAny = true;
     }
 
-    log.debug(ENGINE_MATCHER_CONTAINS_ANY, [runtimeAttr, ruleAttr, containsAny]);
-
     return containsAny;
   };
 }