refactor: eppo client cleanup

Author: rasendubi

src/client/eppo-client.ts

@@ -17,7 +17,7 @@ import { TLRUInMemoryAssignmentCache } from '../cache/tlru-in-memory-assignment-
 import { Configuration } from '../configuration';
 import ConfigurationRequestor from '../configuration-requestor';
 import { ConfigurationStore } from '../configuration-store';
-import { IConfigurationStore, ISyncStore } from '../configuration-store/configuration-store';
+import { ISyncStore } from '../configuration-store/configuration-store';
 import { MemoryOnlyConfigurationStore } from '../configuration-store/memory.store';
 import {
   ConfigurationWireV1,
@@ -44,12 +44,8 @@ import { FlagEvaluationError } from '../flag-evaluation-error';
 import FetchHttpClient from '../http-client';
 import {
   BanditModelData,
-  BanditParameters,
-  BanditVariation,
-  Flag,
   FormatEnum,
   IPrecomputedBandit,
-  ObfuscatedFlag,
   PrecomputedFlag,
   Variation,
   VariationType,
@@ -100,9 +96,6 @@ export type EppoClientParameters = {
   // Dispatcher for arbitrary, application-level events (not to be confused with Eppo specific assignment
   // or bandit events). These events are application-specific and captures by EppoClient#track API.
   eventDispatcher?: EventDispatcher;
-  flagConfigurationStore: IConfigurationStore<Flag | ObfuscatedFlag>;
-  banditVariationConfigurationStore?: IConfigurationStore<BanditVariation[]>;
-  banditModelConfigurationStore?: IConfigurationStore<BanditParameters>;
   overrideStore?: ISyncStore<Variation>;
   configurationRequestParameters?: FlagConfigurationRequestParameters;
   initialConfiguration?: Configuration;
@@ -136,29 +129,17 @@ export default class EppoClient {
   private readonly overrideValidator = new OverrideValidator();
 
   private readonly configurationStore;
-  /** @deprecated use configurationStore instead. */
-  private flagConfigurationStore: IConfigurationStore<Flag | ObfuscatedFlag>;
-  /** @deprecated use configurationStore instead. */
-  private banditModelConfigurationStore?: IConfigurationStore<BanditParameters>;
-  /** @deprecated use configurationStore instead. */
-  private banditVariationConfigurationStore?: IConfigurationStore<BanditVariation[]>;
 
   constructor({
     eventDispatcher = new NoOpEventDispatcher(),
     isObfuscated,
-    flagConfigurationStore,
-    banditVariationConfigurationStore,
-    banditModelConfigurationStore,
     overrideStore,
     configurationRequestParameters,
     initialConfiguration,
   }: EppoClientParameters) {
     this.configurationStore = new ConfigurationStore(initialConfiguration);
 
     this.eventDispatcher = eventDispatcher;
-    this.flagConfigurationStore = flagConfigurationStore;
-    this.banditVariationConfigurationStore = banditVariationConfigurationStore;
-    this.banditModelConfigurationStore = banditModelConfigurationStore;
     this.overrideStore = overrideStore;
     this.configurationRequestParameters = configurationRequestParameters;
 

src/client/test-utils.ts

@@ -1,12 +1,9 @@
 import ApiEndpoints from '../api-endpoints';
 import ConfigurationRequestor from '../configuration-requestor';
-import { IConfigurationStore } from '../configuration-store/configuration-store';
+import { ConfigurationStore } from '../configuration-store';
 import FetchHttpClient from '../http-client';
-import { Flag, ObfuscatedFlag } from '../interfaces';
 
-export async function initConfiguration(
-  configurationStore: IConfigurationStore<Flag | ObfuscatedFlag>,
-) {
+export async function initConfiguration(configurationStore: ConfigurationStore) {
   const apiEndpoints = new ApiEndpoints({
     baseUrl: 'http://127.0.0.1:4000',
     queryParams: {
@@ -16,11 +13,6 @@ export async function initConfiguration(
     },
   });
   const httpClient = new FetchHttpClient(apiEndpoints, 1000);
-  const configurationRequestor = new ConfigurationRequestor(
-    httpClient,
-    configurationStore,
-    null,
-    null,
-  );
+  const configurationRequestor = new ConfigurationRequestor(httpClient, configurationStore);
   await configurationRequestor.fetchAndStoreConfigurations();
 }

src/configuration-wire/configuration-wire-helper.spec.ts

@@ -33,7 +33,6 @@ import {
 } from './configuration-store/configuration-store';
 import { HybridConfigurationStore } from './configuration-store/hybrid.store';
 import { MemoryStore, MemoryOnlyConfigurationStore } from './configuration-store/memory.store';
-import { ConfigurationWireHelper } from './configuration-wire/configuration-wire-helper';
 import {
   IConfigurationWire,
   IObfuscatedPrecomputedConfigurationResponse,
@@ -156,7 +155,6 @@ export {
   IPrecomputedConfigurationResponse,
   PrecomputedFlag,
   FlagKey,
-  ConfigurationWireHelper,
 
   // Test helpers
   decodePrecomputedFlag,