@splitsoftware/splitio 10.17.3 → 10.17.4-rc.2

package/lib/settings/storage/browser.js

@@ -0,0 +1,33 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.validateStorage = void 0;
+var isLocalStorageAvailable_1 = require("@splitsoftware/splitio-commons/cjs/utils/env/isLocalStorageAvailable");
+var constants_1 = require("@splitsoftware/splitio-commons/cjs/utils/constants");
+var STORAGE_LOCALSTORAGE = 'LOCALSTORAGE';
+function validateStorage(settings) {
+    var log = settings.log, mode = settings.mode, _a = settings.storage, _b = _a === void 0 ? { type: constants_1.STORAGE_MEMORY } : _a, type = _b.type, _c = _b.options, options = _c === void 0 ? {} : _c, prefix = _b.prefix;
+    var __originalType;
+    var fallbackToMemory = function () {
+        __originalType = type;
+        type = constants_1.STORAGE_MEMORY;
+    };
+    // 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_1.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_1.STORAGE_MEMORY && type !== STORAGE_LOCALSTORAGE ||
+        type === STORAGE_LOCALSTORAGE && !(0, isLocalStorageAvailable_1.isLocalStorageAvailable)()) {
+        fallbackToMemory();
+        log.error('Invalid or unavailable storage. Fallbacking into MEMORY storage');
+    }
+    return {
+        type: type,
+        options: options,
+        prefix: prefix,
+        __originalType: __originalType
+    };
+}
+exports.validateStorage = validateStorage;

package/lib/settings/storage/node.js

@@ -0,0 +1,67 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.validateStorage = void 0;
+var constants_1 = require("@splitsoftware/splitio-commons/cjs/utils/constants");
+function validateStorage(settings) {
+    var log = settings.log, mode = settings.mode, _a = settings.storage, _b = _a === void 0 ? { type: constants_1.STORAGE_MEMORY } : _a, type = _b.type, _c = _b.options, options = _c === void 0 ? {} : _c, prefix = _b.prefix;
+    // We can have MEMORY, REDIS or an invalid storage type
+    switch (type) {
+        case constants_1.STORAGE_REDIS: {
+            // If passing REDIS storage in localhost or standalone mode, we log an error and fallback to MEMORY storage
+            if (mode === constants_1.STANDALONE_MODE || mode === constants_1.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_1.STORAGE_MEMORY,
+                    prefix: prefix
+                };
+            }
+            var host = options.host, port = options.port, db = options.db, pass = options.pass, url = options.url, tls = options.tls, connectionTimeout = options.connectionTimeout, operationTimeout = options.operationTimeout;
+            if (process.env.REDIS_HOST)
+                host = process.env.REDIS_HOST;
+            if (process.env.REDIS_PORT)
+                port = process.env.REDIS_PORT;
+            if (process.env.REDIS_DB)
+                db = process.env.REDIS_DB;
+            if (process.env.REDIS_PASS)
+                pass = process.env.REDIS_PASS;
+            if (process.env.REDIS_URL)
+                url = process.env.REDIS_URL;
+            var newOpts = {
+                connectionTimeout: connectionTimeout,
+                operationTimeout: operationTimeout
+            };
+            if (url) {
+                newOpts.url = url;
+            }
+            else {
+                newOpts.host = host;
+                newOpts.port = port;
+                newOpts.db = db;
+                newOpts.pass = pass;
+            }
+            if (tls) {
+                newOpts.tls = tls;
+            }
+            return {
+                type: type,
+                prefix: prefix,
+                options: newOpts
+            };
+        }
+        // For now, we don't have modifiers or settings for MEMORY in NodeJS
+        case constants_1.STORAGE_MEMORY:
+        default: {
+            // If passing MEMORY storage in consumer mode, throw an error (no way to fallback to REDIS storage)
+            if (mode === constants_1.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_1.STORAGE_MEMORY)
+                log.error("The provided '" + type + "' storage type is invalid. Fallbacking into default MEMORY storage.");
+            return {
+                type: constants_1.STORAGE_MEMORY,
+                prefix: prefix
+            };
+        }
+    }
+}
+exports.validateStorage = validateStorage;

package/lib/umd.js

@@ -1,9 +1,5 @@
 "use strict";
-
-exports.__esModule = true;
-exports.default = void 0;
-
-var _index = require("./index");
-
-var _default = _index.SplitFactory;
-exports.default = _default;
+Object.defineProperty(exports, "__esModule", { value: true });
+var index_1 = require("./index");
+// eslint-disable-next-line import/no-default-export
+exports.default = index_1.SplitFactory;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@splitsoftware/splitio",
-  "version": "10.17.3",
+  "version": "10.17.4-rc.2",
   "description": "Split SDK",
   "files": [
     "README.md",
@@ -8,7 +8,7 @@
     "LICENSE",
     "CHANGES.txt",
     "lib",
-    "types/*.d.ts",
+    "types",
     "es",
     "src"
   ],
@@ -32,33 +32,28 @@
     "node": ">=6"
   },
   "dependencies": {
-    "@babel/runtime": "^7.13.10",
+    "@splitsoftware/splitio-commons": "1.2.1-rc.9",
     "@types/google.analytics": "0.0.40",
-    "events": "3.1.0",
+    "@types/ioredis": "^4.28.0",
     "ioredis": "^4.28.0",
     "ip": "1.1.5",
     "js-yaml": "3.13.1",
     "node-fetch": "^2.6.7",
-    "object-assign": "^4.1.1",
     "unfetch": "^4.2.0"
   },
   "optionalDependencies": {
     "eventsource": "^1.0.7"
   },
   "devDependencies": {
-    "@babel/cli": "^7.15.7",
-    "@babel/core": "^7.13.10",
-    "@babel/plugin-transform-runtime": "^7.13.10",
-    "@babel/preset-env": "^7.13.10",
-    "@babel/register": "^7.13.8",
-    "babel-loader": "^8.2.2",
+    "@types/node-fetch": "^2.5.12",
     "copyfiles": "^2.4.1",
     "cross-env": "^7.0.3",
     "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": "^6.3.16",
     "karma-chrome-launcher": "^3.1.0",
     "karma-tap": "4.1.4",
     "karma-webpack": "^5.0.0",
@@ -74,24 +69,25 @@
     "tap-min": "^2.0.0",
     "tape": "4.13.2",
     "tape-catch": "1.0.6",
+    "ts-loader": "^9.2.6",
+    "ts-node": "^10.5.0",
+    "typescript": "4.4.4",
     "webpack": "^5.53.0",
     "webpack-cli": "^4.8.0",
     "webpack-merge": "^5.8.0"
   },
   "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",
+    "build-esm": "rimraf es && tsc -outDir es",
+    "postbuild-esm": "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",
-    "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",
-    "build:npm": "rimraf lib es && npm run build-cjs && npm run build-es",
-    "build": "rimraf lib umd es && npm run build-cjs && npm run build-es && npm run build-umd",
+    "build-cjs": "rimraf lib && tsc -outDir lib -m CommonJS",
+    "postbuild-cjs": "cross-env NODE_ENV=cjs node scripts/copy.packages.json.js && ./scripts/build_cjs_replace_imports.sh",
+    "build-umd": "webpack --config webpack.dev.js --env branch=$BUILD_BRANCH --env commit_hash=$BUILD_COMMIT && webpack --config webpack.prod.js --env branch=$BUILD_BRANCH --env commit_hash=$BUILD_COMMIT && ./scripts/clean_umd_build.sh",
+    "build:npm": "rimraf lib es && npm run build-cjs && npm run build-esm",
+    "build": "rimraf lib umd es && npm run build-cjs && npm run build-esm && 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 ./ts-node.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,14 +99,14 @@
     "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",
-    "pretest-ts-decls": "npm run build-es && npm run build-cjs && npm link",
+    "test-node-unit": "cross-env NODE_ENV=test tape -r ./ts-node.register \"src/*/**/__tests__/**/!(browser).spec.js\" | tap-min",
+    "test-node-online": "cross-env NODE_ENV=test tape -r ./ts-node.register src/__tests__/node.spec.js | tap-min",
+    "test-node-destroy": "cross-env NODE_ENV=test tape -r ./ts-node.register src/__tests__/destroy/node.spec.js | tap-min",
+    "test-node-errors": "cross-env NODE_ENV=test tape -r ./ts-node.register src/__tests__/errorCatching/node.spec.js | tap-min",
+    "test-node-offline": "cross-env NODE_ENV=test tape -r ./ts-node.register src/__tests__/offline/node.spec.js | tap-min",
+    "test-node-redis": "cross-env NODE_ENV=test tape -r ./ts-node.register src/__tests__/node_redis.spec.js | tap-min",
+    "test-node-push": "cross-env NODE_ENV=test tape -r ./ts-node.register src/__tests__/push/node.spec.js | tap-min",
+    "pretest-ts-decls": "npm run build-esm && npm run build-cjs && npm link",
     "test-ts-decls": "./scripts/ts-tests.sh",
     "posttest-ts-decls": "npm unlink && npm install",
     "test": "npm run test-node && npm run test-browser",

package/src/factory/browser.js

@@ -0,0 +1,84 @@
+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 { userConsentProps } from '@splitsoftware/splitio-commons/src/sdkFactory/userConsentProps';
+
+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,
+
+    integrationsManagerFactory: settings.integrations && settings.integrations.length > 0 ? integrationsManagerFactory.bind(null, settings.integrations) : undefined,
+
+    impressionsObserverFactory: shouldAddPt(settings) ? impressionObserverCSFactory : undefined,
+
+    extraProps: userConsentProps
+  };
+
+  switch (settings.mode) {
+    case LOCALHOST_MODE:
+      modules.splitApiFactory = undefined;
+      modules.syncManagerFactory = settings.sync.localhostMode;
+      modules.SignalListener = undefined;
+      break;
+  }
+
+  return modules;
+}
+
+/**
+ * SplitFactory for client-side.
+ *
+ * @param {SplitIO.IBrowserSettings} config configuration object used to instantiate the SDK
+ * @param {Function=} __updateModules optional function that lets redefine internal SDK modules. Use with
+ * caution since, unlike `config`, this param is not validated neither considered part of the public API.
+ * @throws Will throw an error if the provided config is invalid.
+ */
+export function SplitFactory(config, __updateModules) {
+  const settings = settingsFactory(config);
+  const modules = getModules(settings);
+  if (__updateModules) __updateModules(modules);
+  return sdkFactory(modules);
+}

package/src/factory/node.js

@@ -0,0 +1,78 @@
+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,
+
+    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;
+}
+
+/**
+ * SplitFactory for server-side.
+ *
+ * @param {SplitIO.INodeSettings | SplitIO.INodeAsyncSettings} config configuration object used to instantiate the SDK
+ * @param {Function=} __updateModules optional function that lets redefine internal SDK modules. Use with
+ * caution since, unlike `config`, this param is not validated neither considered part of the public API.
+ * @throws Will throw an error if the provided config is invalid.
+ */
+export function SplitFactory(config, __updateModules) {
+  const settings = settingsFactory(config);
+  const modules = getModules(settings);
+  if (__updateModules) __updateModules(modules);
+  return sdkFactory(modules);
+}

package/src/index.js

@@ -1,140 +1,17 @@
-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';
+/**
+Copyright 2022 Split Software
 
-const buildInstanceId = (key, trafficType) => `${key.matchingKey ? key.matchingKey : key}-${key.bucketingKey ? key.bucketingKey : key}-${trafficType !== undefined ? trafficType : ''}`;
+Licensed under the Apache License, Version 2.0 (the "License");
+you may not use this file except in compliance with the License.
+You may obtain a copy of the License at
 
-export function SplitFactory(config) {
-  // Cache instances created per factory.
-  const clientInstances = {};
+    http://www.apache.org/licenses/LICENSE-2.0
 
-  // 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();
+Unless required by applicable law or agreed to in writing, software
+distributed under the License is distributed on an "AS IS" BASIS,
+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.
+**/
 
-  // 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';