@splitsoftware/splitio 10.17.1 → 10.17.3-rc.1

package/lib/{utils/settings → settings}/runtime/node.js

@@ -3,16 +3,16 @@
 var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault");
 
 exports.__esModule = true;
-exports.default = generateRuntimeSettings;
+exports.validateRuntime = validateRuntime;
 
 var _os = _interopRequireDefault(require("os"));
 
 var _ip = _interopRequireDefault(require("ip"));
 
-var _constants = require("../../constants");
+var _constants = require("@splitsoftware/splitio-commons/cjs/utils/constants");
 
 /**
-Copyright 2016 Split Software
+Copyright 2022 Split Software
 
 Licensed under the Apache License, Version 2.0 (the "License");
 you may not use this file except in compliance with the License.
@@ -26,8 +26,10 @@ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 See the License for the specific language governing permissions and
 limitations under the License.
 **/
-function generateRuntimeSettings(isIPAddressesEnabled, isConsumerMode) {
-  // If the values are not available, default to false (for standalone) or "unknown" (for consumer mode, to be used on Redis keys)
+function validateRuntime(settings) {
+  var isIPAddressesEnabled = settings.core.IPAddressesEnabled === true;
+  var isConsumerMode = settings.mode === _constants.CONSUMER_MODE; // If the values are not available, default to false (for standalone) or "unknown" (for consumer mode, to be used on Redis keys)
+
   var ip = _ip.default.address() || (isConsumerMode ? _constants.UNKNOWN : false);
   var hostname = _os.default.hostname() || (isConsumerMode ? _constants.UNKNOWN : false);
 

package/lib/{utils/settings → settings}/storage/browser.js

@@ -1,18 +1,14 @@
 "use strict";
 
-var _interopRequireDefault = require("@babel/runtime/helpers/interopRequireDefault");
-
 exports.__esModule = true;
-exports.default = void 0;
-
-var _logger = _interopRequireDefault(require("../../../utils/logger"));
+exports.validateStorage = validateStorage;
 
-var _isAvailable = _interopRequireDefault(require("../../../utils/localstorage/isAvailable"));
+var _isLocalStorageAvailable = require("@splitsoftware/splitio-commons/cjs/utils/env/isLocalStorageAvailable");
 
-var _constants = require("../../../utils/constants");
+var _constants = require("@splitsoftware/splitio-commons/cjs/utils/constants");
 
 /**
-Copyright 2016 Split Software
+Copyright 2022 Split Software
 
 Licensed under the Apache License, Version 2.0 (the "License");
 you may not use this file except in compliance with the License.
@@ -26,13 +22,16 @@ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 See the License for the specific language governing permissions and
 limitations under the License.
 **/
-var log = (0, _logger.default)('splitio-settings');
-
-var ParseStorageSettings = function ParseStorageSettings(settings) {
-  var mode = settings.mode,
-      _settings$storage = settings.storage,
-      _settings$storage$typ = _settings$storage.type,
-      type = _settings$storage$typ === void 0 ? _constants.STORAGE_MEMORY : _settings$storage$typ,
+var STORAGE_LOCALSTORAGE = 'LOCALSTORAGE';
+
+function validateStorage(settings) {
+  var log = settings.log,
+      mode = settings.mode,
+      _settings$storage = settings.storage;
+  _settings$storage = _settings$storage === void 0 ? {
+    type: _constants.STORAGE_MEMORY
+  } : _settings$storage;
+  var type = _settings$storage.type,
       _settings$storage$opt = _settings$storage.options,
       options = _settings$storage$opt === void 0 ? {} : _settings$storage$opt,
       prefix = _settings$storage.prefix;
@@ -48,19 +47,19 @@ var ParseStorageSettings = function ParseStorageSettings(settings) {
   var fallbackToMemory = function fallbackToMemory() {
     __originalType = type;
     type = _constants.STORAGE_MEMORY;
-  }; // In localhost mode, fallback to Memory storage and track original
-  // type to emit SDK_READY_FROM_CACHE if corresponds
+  }; // In localhost mode, fallback to Memory storage and track original type to emit SDK_READY_FROM_CACHE if corresponds.
+  // ATM, other mode settings (e.g., 'consumer') are ignored in client-side API, and so treated as standalone.
 
 
-  if (mode === _constants.LOCALHOST_MODE && type === _constants.STORAGE_LOCALSTORAGE) {
+  if (mode === _constants.LOCALHOST_MODE && type === STORAGE_LOCALSTORAGE) {
     fallbackToMemory();
   } // If an invalid storage type is provided OR we want to use LOCALSTORAGE and
   // it's not available, fallback into MEMORY
 
 
-  if (type !== _constants.STORAGE_MEMORY && type !== _constants.STORAGE_LOCALSTORAGE || type === _constants.STORAGE_LOCALSTORAGE && !(0, _isAvailable.default)()) {
+  if (type !== _constants.STORAGE_MEMORY && type !== STORAGE_LOCALSTORAGE || type === STORAGE_LOCALSTORAGE && !(0, _isLocalStorageAvailable.isLocalStorageAvailable)()) {
     fallbackToMemory();
-    log.warn('Invalid or unavailable storage. Fallbacking into MEMORY storage');
+    log.error('Invalid or unavailable storage. Fallbacking into MEMORY storage');
   }
 
   return {
@@ -69,7 +68,4 @@ var ParseStorageSettings = function ParseStorageSettings(settings) {
     prefix: prefix,
     __originalType: __originalType
   };
-};
-
-var _default = ParseStorageSettings;
-exports.default = _default;
+}

package/lib/{utils/settings → settings}/storage/node.js

@@ -1,12 +1,12 @@
 "use strict";
 
 exports.__esModule = true;
-exports.default = void 0;
+exports.validateStorage = validateStorage;
 
-var _constants = require("../../constants");
+var _constants = require("@splitsoftware/splitio-commons/cjs/utils/constants");
 
 /**
-Copyright 2016 Split Software
+Copyright 2022 Split Software
 
 Licensed under the Apache License, Version 2.0 (the "License");
 you may not use this file except in compliance with the License.
@@ -20,10 +20,14 @@ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 See the License for the specific language governing permissions and
 limitations under the License.
 **/
-var ParseStorageSettings = function ParseStorageSettings(settings) {
-  var mode = settings.mode,
-      _settings$storage = settings.storage,
-      type = _settings$storage.type,
+function validateStorage(settings) {
+  var log = settings.log,
+      mode = settings.mode,
+      _settings$storage = settings.storage;
+  _settings$storage = _settings$storage === void 0 ? {
+    type: _constants.STORAGE_MEMORY
+  } : _settings$storage;
+  var type = _settings$storage.type,
       _settings$storage$opt = _settings$storage.options,
       options = _settings$storage$opt === void 0 ? {} : _settings$storage$opt,
       prefix = _settings$storage.prefix;
@@ -32,17 +36,21 @@ var ParseStorageSettings = function ParseStorageSettings(settings) {
     prefix += '.SPLITIO';
   } else {
     prefix = 'SPLITIO';
-  } // In localhost mode we should force the user to use the MEMORY storage
+  } // We can have MEMORY, REDIS or an invalid storage type
 
 
-  if (mode === _constants.LOCALHOST_MODE) return {
-    type: _constants.STORAGE_MEMORY,
-    prefix: prefix
-  }; // In other cases we can have MEMORY or REDIS
-
   switch (type) {
     case _constants.STORAGE_REDIS:
       {
+        // If passing REDIS storage in localhost or standalone mode, we log an error and fallback to MEMORY storage
+        if (mode === _constants.STANDALONE_MODE || mode === _constants.LOCALHOST_MODE) {
+          log.error('The provided REDIS storage is invalid for this mode. It requires consumer mode. Fallbacking into default MEMORY storage.');
+          return {
+            type: _constants.STORAGE_MEMORY,
+            prefix: prefix
+          };
+        }
+
         var host = options.host,
             port = options.port,
             db = options.db,
@@ -80,13 +88,14 @@ var ParseStorageSettings = function ParseStorageSettings(settings) {
     case _constants.STORAGE_MEMORY:
     default:
       {
+        // If passing MEMORY storage in consumer mode, throw an error (no way to fallback to REDIS storage)
+        if (mode === _constants.CONSUMER_MODE) throw new Error('A REDIS storage is required on consumer mode'); // If passing an invalid storage type, log an error
+
+        if (type !== _constants.STORAGE_MEMORY) log.error("The provided '" + type + "' storage type is invalid. Fallbacking into default MEMORY storage.");
         return {
           type: _constants.STORAGE_MEMORY,
           prefix: prefix
         };
       }
   }
-};
-
-var _default = ParseStorageSettings;
-exports.default = _default;
+}

package/lib/umd.js

@@ -5,5 +5,6 @@ exports.default = void 0;
 
 var _index = require("./index");
 
+// eslint-disable-next-line import/no-default-export
 var _default = _index.SplitFactory;
 exports.default = _default;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@splitsoftware/splitio",
-  "version": "10.17.1",
+  "version": "10.17.3-rc.1",
   "description": "Split SDK",
   "files": [
     "README.md",
@@ -8,9 +8,11 @@
     "LICENSE",
     "CHANGES.txt",
     "lib",
-    "types/*.d.ts",
+    "types",
     "es",
-    "src"
+    "src",
+    "client",
+    "server"
   ],
   "repository": "splitio/javascript-client",
   "homepage": "https://github.com/splitio/javascript-client#readme",
@@ -33,13 +35,12 @@
   },
   "dependencies": {
     "@babel/runtime": "^7.13.10",
+    "@splitsoftware/splitio-commons": "1.2.0",
     "@types/google.analytics": "0.0.40",
-    "events": "3.1.0",
     "ioredis": "^4.28.0",
     "ip": "1.1.5",
     "js-yaml": "3.13.1",
-    "node-fetch": "^2.6.1",
-    "object-assign": "^4.1.1",
+    "node-fetch": "^2.6.7",
     "unfetch": "^4.2.0"
   },
   "optionalDependencies": {
@@ -48,8 +49,10 @@
   "devDependencies": {
     "@babel/cli": "^7.15.7",
     "@babel/core": "^7.13.10",
+    "@babel/plugin-proposal-class-properties": "^7.16.7",
     "@babel/plugin-transform-runtime": "^7.13.10",
     "@babel/preset-env": "^7.13.10",
+    "@babel/preset-typescript": "^7.16.7",
     "@babel/register": "^7.13.8",
     "babel-loader": "^8.2.2",
     "copyfiles": "^2.4.1",
@@ -57,6 +60,7 @@
     "csv-streamify": "4.0.0",
     "eslint": "^8.0.1",
     "eslint-plugin-compat": "^3.13.0",
+    "eslint-plugin-import": "^2.25.4",
     "fetch-mock": "^9.11.0",
     "karma": "^6.3.2",
     "karma-chrome-launcher": "^3.1.0",
@@ -80,10 +84,10 @@
   },
   "scripts": {
     "build-es": "rimraf es && cross-env NODE_ENV=es babel ./src -d es --ignore 'src/**/__tests__/*'",
-    "postbuild-es": "cross-env NODE_ENV=es node scripts/copy.packages.json.js",
+    "postbuild-es": "cross-env NODE_ENV=es node scripts/copy.packages.json.js && ./scripts/build_esm_replace_imports.sh",
     "build-umd:stats": "webpack --progress --env production --json > ./stats/stat_results.json",
     "build-cjs": "rimraf lib && cross-env NODE_ENV=cjs babel ./src -d lib --ignore 'src/**/__tests__/*'",
-    "postbuild-cjs": "cross-env NODE_ENV=cjs node scripts/copy.packages.json.js",
+    "postbuild-cjs": "cross-env NODE_ENV=cjs node scripts/copy.packages.json.js && ./scripts/build_cjs_replace_imports.sh",
     "build-umd": "webpack --config webpack.ci.dev.js --env branch=$BUILD_BRANCH --env commit_hash=$BUILD_COMMIT && webpack --config webpack.ci.prod.js --env branch=$BUILD_BRANCH --env commit_hash=$BUILD_COMMIT && ./scripts/clean_umd_build.sh",
     "build-umd:local": "webpack --config webpack.dev.js && webpack --config webpack.prod.js && ./scripts/clean_umd_build.sh",
     "build:local": "rimraf lib umd es && npm run build-cjs && npm run build-es && npm run build-umd:local",
@@ -91,7 +95,7 @@
     "build": "rimraf lib umd es && npm run build-cjs && npm run build-es && npm run build-umd",
     "check": "npm run check:lint && npm run check:version",
     "check:lint": "eslint src",
-    "check:version": "cross-env NODE_ENV=test tape -r @babel/register src/utils/__tests__/settings/index.spec.js",
+    "check:version": "cross-env NODE_ENV=test tape -r ./babel.register src/settings/__tests__/defaults.spec.js",
     "test-browser-local": "cross-env NODE_ENV=test karma start karma/local.karma.conf.js",
     "test-browser-e2e-local": "cross-env NODE_ENV=test karma start karma/e2e.local.karma.conf.js",
     "test-browser": "npm run test-browser-unit && npm run test-browser-online && npm run test-browser-offline && npm run test-browser-destroy && npm run test-browser-errors && npm run test-browser-push && npm run test-browser-gaintegration",
@@ -103,13 +107,13 @@
     "test-browser-gaintegration": "cross-env NODE_ENV=test karma start karma/gaintegration.ci.karma.conf.js",
     "test-browser-push": "cross-env NODE_ENV=test karma start karma/push.ci.karma.conf.js",
     "test-node": "npm run test-node-unit && npm run test-node-online && npm run test-node-redis && npm run test-node-offline && npm run test-node-destroy && npm run test-node-errors && npm run test-node-push",
-    "test-node-unit": "cross-env NODE_ENV=test tape -r @babel/register \"src/*/**/__tests__/**/!(browser).spec.js\" | tap-min",
-    "test-node-online": "cross-env NODE_ENV=test tape -r @babel/register src/__tests__/node.spec.js | tap-min",
-    "test-node-destroy": "cross-env NODE_ENV=test tape -r @babel/register src/__tests__/destroy/node.spec.js | tap-min",
-    "test-node-errors": "cross-env NODE_ENV=test tape -r @babel/register src/__tests__/errorCatching/node.spec.js | tap-min",
-    "test-node-offline": "cross-env NODE_ENV=test tape -r @babel/register src/__tests__/offline/node.spec.js | tap-min",
-    "test-node-redis": "cross-env NODE_ENV=test tape -r @babel/register \"src/__tests__/node_redis.spec.js\" | tap-min",
-    "test-node-push": "cross-env NODE_ENV=test tape -r @babel/register src/__tests__/push/node.spec.js | tap-min",
+    "test-node-unit": "cross-env NODE_ENV=test tape -r ./babel.register \"src/*/**/__tests__/**/!(browser).spec.js\" | tap-min",
+    "test-node-online": "cross-env NODE_ENV=test tape -r ./babel.register src/__tests__/node.spec.js | tap-min",
+    "test-node-destroy": "cross-env NODE_ENV=test tape -r ./babel.register src/__tests__/destroy/node.spec.js | tap-min",
+    "test-node-errors": "cross-env NODE_ENV=test tape -r ./babel.register src/__tests__/errorCatching/node.spec.js | tap-min",
+    "test-node-offline": "cross-env NODE_ENV=test tape -r ./babel.register src/__tests__/offline/node.spec.js | tap-min",
+    "test-node-redis": "cross-env NODE_ENV=test tape -r ./babel.register \"src/__tests__/node_redis.spec.js\" | tap-min",
+    "test-node-push": "cross-env NODE_ENV=test tape -r ./babel.register src/__tests__/push/node.spec.js | tap-min",
     "pretest-ts-decls": "npm run build-es && npm run build-cjs && npm link",
     "test-ts-decls": "./scripts/ts-tests.sh",
     "posttest-ts-decls": "npm unlink && npm install",

package/server/package.json

@@ -0,0 +1,5 @@
+{
+  "main": "../lib/factory/node.js",
+  "module": "../es/factory/node.js",
+  "types": "../types/server/index.d.ts"
+}

package/src/factory/browser.js

@@ -0,0 +1,74 @@
+import { splitApiFactory } from '@splitsoftware/splitio-commons/src/services/splitApi';
+import { syncManagerOnlineFactory } from '@splitsoftware/splitio-commons/src/sync/syncManagerOnline';
+import { pushManagerFactory } from '@splitsoftware/splitio-commons/src/sync/streaming/pushManager';
+import { pollingManagerCSFactory } from '@splitsoftware/splitio-commons/src/sync/polling/pollingManagerCS';
+import { InLocalStorage } from '@splitsoftware/splitio-commons/src/storages/inLocalStorage';
+import { InMemoryStorageCSFactory } from '@splitsoftware/splitio-commons/src/storages/inMemory/InMemoryStorageCS';
+import { sdkManagerFactory } from '@splitsoftware/splitio-commons/src/sdkManager';
+import { sdkClientMethodCSFactory } from '@splitsoftware/splitio-commons/src/sdkClient/sdkClientMethodCSWithTT';
+import { impressionObserverCSFactory } from '@splitsoftware/splitio-commons/src/trackers/impressionObserver/impressionObserverCS';
+import { integrationsManagerFactory } from '@splitsoftware/splitio-commons/src/integrations/browser';
+import { __InLocalStorageMockFactory } from '@splitsoftware/splitio-commons/src/utils/settingsValidation/storage/storageCS';
+import { sdkFactory } from '@splitsoftware/splitio-commons/src/sdkFactory';
+import { LOCALHOST_MODE, STORAGE_LOCALSTORAGE } from '@splitsoftware/splitio-commons/src/utils/constants';
+import { shouldAddPt } from '@splitsoftware/splitio-commons/src/trackers/impressionObserver/utils';
+
+import { settingsFactory } from '../settings/browser';
+import { platform, SignalListener } from '../platform';
+
+const syncManagerOnlineCSFactory = syncManagerOnlineFactory(pollingManagerCSFactory, pushManagerFactory);
+
+function getStorage(settings) {
+  return settings.storage.type === STORAGE_LOCALSTORAGE ?
+    InLocalStorage(settings.storage)
+    : settings.storage.__originalType === STORAGE_LOCALSTORAGE ?
+      __InLocalStorageMockFactory
+      : InMemoryStorageCSFactory;
+}
+
+/**
+ *
+ * @param {import("@splitsoftware/splitio-commons/types/types").ISettings} settings
+ */
+function getModules(settings) {
+
+  const modules = {
+    settings,
+
+    platform,
+
+    storageFactory: getStorage(settings),
+
+    splitApiFactory,
+
+    syncManagerFactory: syncManagerOnlineCSFactory,
+
+    sdkManagerFactory,
+
+    sdkClientMethodFactory: sdkClientMethodCSFactory,
+
+    SignalListener,
+
+    impressionListener: settings.impressionListener,
+
+    integrationsManagerFactory: settings.integrations && settings.integrations.length > 0 ? integrationsManagerFactory.bind(null, settings.integrations) : undefined,
+
+    impressionsObserverFactory: shouldAddPt(settings) ? impressionObserverCSFactory : undefined,
+  };
+
+  switch (settings.mode) {
+    case LOCALHOST_MODE:
+      modules.splitApiFactory = undefined;
+      modules.syncManagerFactory = settings.sync.localhostMode;
+      modules.SignalListener = undefined;
+      break;
+  }
+
+  return modules;
+}
+
+export function SplitFactory(config) {
+  const settings = settingsFactory(config);
+  const modules = getModules(settings);
+  return sdkFactory(modules);
+}

package/src/factory/node.js

@@ -0,0 +1,71 @@
+import { splitApiFactory } from '@splitsoftware/splitio-commons/src/services/splitApi';
+import { syncManagerOnlineFactory } from '@splitsoftware/splitio-commons/src/sync/syncManagerOnline';
+import { pushManagerFactory } from '@splitsoftware/splitio-commons/src/sync/streaming/pushManager';
+import { pollingManagerSSFactory } from '@splitsoftware/splitio-commons/src/sync/polling/pollingManagerSS';
+import { InRedisStorage } from '@splitsoftware/splitio-commons/src/storages/inRedis';
+import { InMemoryStorageFactory } from '@splitsoftware/splitio-commons/src/storages/inMemory/InMemoryStorage';
+import { sdkManagerFactory } from '@splitsoftware/splitio-commons/src/sdkManager';
+import { sdkClientMethodFactory } from '@splitsoftware/splitio-commons/src/sdkClient/sdkClientMethod';
+import { impressionObserverSSFactory } from '@splitsoftware/splitio-commons/src/trackers/impressionObserver/impressionObserverSS';
+import { sdkFactory } from '@splitsoftware/splitio-commons/src/sdkFactory';
+import { CONSUMER_MODE, LOCALHOST_MODE } from '@splitsoftware/splitio-commons/src/utils/constants';
+import { shouldAddPt } from '@splitsoftware/splitio-commons/src/trackers/impressionObserver/utils';
+
+import { settingsFactory } from '../settings/node';
+import { platform, SignalListener } from '../platform';
+
+const syncManagerOnlineSSFactory = syncManagerOnlineFactory(pollingManagerSSFactory, pushManagerFactory);
+
+function getStorage(settings) {
+  return settings.storage.type === 'REDIS' ?
+    InRedisStorage(settings.storage) :
+    InMemoryStorageFactory;
+}
+
+/**
+ *
+ * @param {import("@splitsoftware/splitio-commons/types/types").ISettings} settings
+ */
+function getModules(settings) {
+
+  const modules = {
+    settings,
+
+    platform,
+
+    storageFactory: getStorage(settings),
+
+    splitApiFactory,
+
+    syncManagerFactory: syncManagerOnlineSSFactory,
+
+    sdkManagerFactory,
+
+    sdkClientMethodFactory,
+
+    SignalListener,
+
+    impressionListener: settings.impressionListener,
+
+    impressionsObserverFactory: shouldAddPt(settings) ? impressionObserverSSFactory : undefined,
+  };
+
+  switch (settings.mode) {
+    case LOCALHOST_MODE:
+      modules.splitApiFactory = undefined;
+      modules.syncManagerFactory = settings.sync.localhostMode;
+      modules.SignalListener = undefined;
+      break;
+    case CONSUMER_MODE:
+      modules.syncManagerFactory = undefined;
+      break;
+  }
+
+  return modules;
+}
+
+export function SplitFactory(config) {
+  const settings = settingsFactory(config);
+  const modules = getModules(settings);
+  return sdkFactory(modules);
+}

package/src/index.js

@@ -1,140 +1 @@
-import ManagerFactory from './manager';
-import StorageFactory from './storage';
-import ReadinessGateFacade from './readiness';
-import SettingsFactory from './utils/settings';
-import Context from './utils/context';
-import keyParser from './utils/key/parser';
-import logFactory, { API } from './utils/logger';
-const log = logFactory('splitio');
-import tracker from './utils/timeTracker';
-import SplitFactoryOnline from './factory/online';
-import SplitFactoryOffline from './factory/offline';
-import sdkStatusManager from './readiness/statusManager';
-import { LOCALHOST_MODE } from './utils/constants';
-import { validateApiKey, validateKey, validateTrafficType } from './utils/inputValidation';
-import IntegrationsManagerFactory from './integrations';
-import ImpressionCounter from './impressions/counter';
-
-const buildInstanceId = (key, trafficType) => `${key.matchingKey ? key.matchingKey : key}-${key.bucketingKey ? key.bucketingKey : key}-${trafficType !== undefined ? trafficType : ''}`;
-
-export function SplitFactory(config) {
-  // Cache instances created per factory.
-  const clientInstances = {};
-
-  // Tracking times. We need to do it here because we need the storage created.
-  const readyLatencyTrackers = {
-    splitsReadyTracker: tracker.start(tracker.TaskNames.SPLITS_READY),
-    segmentsReadyTracker: tracker.start(tracker.TaskNames.SEGMENTS_READY),
-    sdkReadyTracker: tracker.start(tracker.TaskNames.SDK_READY)
-  };
-  const context = new Context();
-
-  // Put settings config within context
-  const settings = SettingsFactory(config);
-  context.put(context.constants.SETTINGS, settings);
-
-  // We will just log and allow for the SDK to end up throwing an SDK_TIMEOUT event for devs to handle.
-  validateApiKey(settings.core.authorizationKey);
-
-  // Put readiness and statusManager within context
-  // Done before adding storage, to let it access readiness gate synchronously
-  const gateFactory = ReadinessGateFacade();
-  const readiness = gateFactory(settings.startup.readyTimeout);
-  context.put(context.constants.READINESS, readiness);
-  const statusManager = sdkStatusManager(context);
-  context.put(context.constants.STATUS_MANAGER, statusManager);
-
-  // Put storage config within context
-  const storage = StorageFactory(context);
-  context.put(context.constants.STORAGE, storage);
-
-  // Put counter
-  const impressionsCounter = new ImpressionCounter(); // Instantiates new counter for Impressions
-  context.put(context.constants.IMPRESSIONS_COUNTER, impressionsCounter);
-
-  // Put integrationsManager within context.
-  // It needs to access the storage, settings and potentially other pieces, so it's registered after them.
-  const integrationsManager = IntegrationsManagerFactory(context);
-  context.put(context.constants.INTEGRATIONS_MANAGER, integrationsManager);
-
-  // Define which type of factory to use
-  const splitFactory = settings.mode === LOCALHOST_MODE ? SplitFactoryOffline : SplitFactoryOnline;
-
-  const {
-    api: mainClientInstance,
-    metricCollectors: mainClientMetricCollectors
-  } = splitFactory(context, readyLatencyTrackers);
-
-  // It makes no sense to have multiple instances of the manager.
-  const managerInstance = ManagerFactory(storage.splits, context);
-
-  const parsedDefaultKey = keyParser(settings.core.key);
-  const defaultInstanceId = buildInstanceId(parsedDefaultKey, settings.core.trafficType);
-  clientInstances[defaultInstanceId] = mainClientInstance;
-
-  log.info('New Split SDK instance created.');
-
-  return {
-    // Split evaluation and event tracking engine
-    client(key, trafficType) {
-      if (key === undefined) {
-        log.debug('Retrieving default SDK client.');
-        return mainClientInstance;
-      }
-
-      if (typeof storage.shared != 'function') {
-        throw new Error('Shared Client not supported by the storage mechanism. Create isolated instances instead.');
-      }
-
-      // Validate the key value
-      const validKey = validateKey(key, 'Shared Client instantiation');
-      if (validKey === false) {
-        throw new Error('Shared Client needs a valid key.');
-      }
-
-      let validTrafficType;
-      if (trafficType !== undefined) {
-        validTrafficType = validateTrafficType(trafficType, 'Shared Client instantiation');
-        if (validTrafficType === false) {
-          throw new Error('Shared Client needs a valid traffic type or no traffic type at all.');
-        }
-      }
-      const instanceId = buildInstanceId(validKey, validTrafficType);
-
-      if (!clientInstances[instanceId]) {
-        const sharedSettings = settings.overrideKeyAndTT(validKey, validTrafficType);
-        const sharedContext = new Context();
-
-        const readiness = gateFactory(sharedSettings.startup.readyTimeout);
-        sharedContext.put(context.constants.READY_FROM_CACHE, context.get(context.constants.READY_FROM_CACHE, true));
-        sharedContext.put(context.constants.READINESS, readiness);
-        // for shared clients, the internal offset of added/removed SDK_READY callbacks is -1
-        sharedContext.put(context.constants.STATUS_MANAGER, sdkStatusManager(sharedContext, -1));
-        sharedContext.put(context.constants.SETTINGS, sharedSettings);
-        sharedContext.put(context.constants.STORAGE, storage.shared(sharedSettings));
-        sharedContext.put(context.constants.IMPRESSIONS_COUNTER, impressionsCounter);
-        // As shared clients reuse all the storage information, we don't need to check here if we
-        // will use offline or online mode. We should stick with the original decision.
-        clientInstances[instanceId] = splitFactory(sharedContext, false, mainClientMetricCollectors).api;
-
-        log.info('New shared client instance created.');
-      } else {
-        log.debug('Retrieving existing SDK client.');
-      }
-
-      return clientInstances[instanceId];
-    },
-
-    // Manager API to explore available information
-    manager() {
-      log.info('Manager instance retrieved.');
-      return managerInstance;
-    },
-
-    // Logger wrapper API
-    Logger: API,
-
-    // Expose SDK settings
-    settings
-  };
-}
+export { SplitFactory } from './factory';