refactor: cleanup/prettier

Author: rasendubi

src/application-logger.ts

@@ -1,8 +1,9 @@
 import pino from 'pino';
 
+/** @internal */
 export const loggerPrefix = '[Eppo SDK]';
 
-// Create a Pino logger instance
+/** @internal */
 export const logger = pino({
   // eslint-disable-next-line no-restricted-globals
   level: process.env.LOG_LEVEL ?? (process.env.NODE_ENV === 'production' ? 'warn' : 'info'),

src/broadcast.ts

@@ -2,7 +2,7 @@ export type Listener<T extends unknown[]> = (...args: T) => void;
 
 /**
  * A broadcast channel for dispatching events to multiple listeners.
- * 
+ *
  * @internal
  */
 export class BroadcastChannel<T extends unknown[]> {
@@ -29,4 +29,4 @@ export class BroadcastChannel<T extends unknown[]> {
       }
     }
   }
-}
+}

src/cache/tlru-cache.ts

@@ -8,10 +8,7 @@ import { LRUCache } from './lru-cache';
  **/
 export class TLRUCache extends LRUCache {
   private readonly cacheEntriesTTLRegistry = new Map<string, Date>();
-  constructor(
-    readonly maxSize: number,
-    readonly ttl: number,
-  ) {
+  constructor(readonly maxSize: number, readonly ttl: number) {
     super(maxSize);
   }
 

src/client/eppo-client-assignment-details.spec.ts

@@ -1,9 +1,6 @@
 import * as fs from 'fs';
 
-import {
-  IAssignmentTestCase,
-  readMockUfcConfiguration,
-} from '../../test/testHelpers';
+import { IAssignmentTestCase, readMockUfcConfiguration } from '../../test/testHelpers';
 import { AllocationEvaluationCode } from '../flag-evaluation-details-builder';
 import { Variation, VariationType } from '../interfaces';
 import { OperatorType } from '../rules';

src/client/eppo-client-experiment-container.spec.ts

@@ -17,7 +17,7 @@ describe('getExperimentContainerEntry', () => {
   let loggerWarnSpy: jest.SpyInstance;
 
   beforeEach(async () => {
-    client = new EppoClient({ 
+    client = new EppoClient({
       configuration: {
         initializationStrategy: 'none',
         initialConfiguration: readMockUfcConfiguration(),

src/client/eppo-client-with-bandits.spec.ts

@@ -517,7 +517,7 @@ describe('EppoClient Bandits E2E test', () => {
               unmatchedAllocations: [],
               unevaluatedAllocations: [],
             };
-            
+
             return {
               assignmentDetails: {
                 flagKey,
@@ -549,7 +549,7 @@ describe('EppoClient Bandits E2E test', () => {
                 },
                 evaluationDetails,
                 entityId: null,
-              }
+              },
             };
           });
 
@@ -681,11 +681,7 @@ describe('EppoClient Bandits E2E test', () => {
       subjectAttributes: ContextAttributes,
       banditActions: Record<string, BanditActions>,
     ): Configuration {
-      return client.getPrecomputedConfiguration(
-        subjectKey,
-        subjectAttributes,
-        banditActions,
-      );
+      return client.getPrecomputedConfiguration(subjectKey, subjectAttributes, banditActions);
     }
 
     describe('obfuscated results', () => {

src/client/eppo-client-with-overrides.spec.ts

@@ -5,9 +5,7 @@ import * as overrideValidatorModule from '../override-validator';
 import EppoClient from './eppo-client';
 
 describe('EppoClient', () => {
-  function setUnobfuscatedFlagEntries(
-    entries: Record<string, Flag>,
-  ): EppoClient {
+  function setUnobfuscatedFlagEntries(entries: Record<string, Flag>): EppoClient {
     return new EppoClient({
       sdkKey: 'dummy',
       sdkName: 'js-client-sdk-common',
@@ -26,8 +24,8 @@ describe('EppoClient', () => {
               },
               banditReferences: {},
             },
-          }
-        })
+          },
+        }),
       },
     });
   }

src/client/eppo-client.precomputed.spec.ts

@@ -11,7 +11,9 @@ import EppoClient from './eppo-client';
 
 describe('EppoClient Precomputed Mode', () => {
   // Read both configurations for test reference
-  const precomputedConfigurationWire = readMockConfigurationWireResponse(MOCK_PRECOMPUTED_WIRE_FILE);
+  const precomputedConfigurationWire = readMockConfigurationWireResponse(
+    MOCK_PRECOMPUTED_WIRE_FILE,
+  );
   const initialConfiguration = Configuration.fromString(precomputedConfigurationWire);
 
   let client: EppoClient;
@@ -21,7 +23,7 @@ describe('EppoClient Precomputed Mode', () => {
   beforeEach(() => {
     mockAssignmentLogger = { logAssignment: jest.fn() } as jest.Mocked<IAssignmentLogger>;
     mockBanditLogger = { logBanditAction: jest.fn() } as jest.Mocked<IBanditLogger>;
-    
+
     // Create EppoClient with precomputed configuration
     client = new EppoClient({
       sdkKey: 'test-key',
@@ -33,7 +35,7 @@ describe('EppoClient Precomputed Mode', () => {
         enablePolling: false,
       },
     });
-    
+
     client.setAssignmentLogger(mockAssignmentLogger);
     client.setBanditLogger(mockBanditLogger);
   });
@@ -67,29 +69,40 @@ describe('EppoClient Precomputed Mode', () => {
     expect(result).toEqual({ key: 'value', number: 123 });
     expect(mockAssignmentLogger.logAssignment).toHaveBeenCalledTimes(1);
   });
-  
+
   it('correctly evaluates flag with extra logging', () => {
-    const result = client.getStringAssignment('string-flag-with-extra-logging', 'test-subject-key', {}, 'default');
+    const result = client.getStringAssignment(
+      'string-flag-with-extra-logging',
+      'test-subject-key',
+      {},
+      'default',
+    );
     expect(result).toBe('red');
     expect(mockAssignmentLogger.logAssignment).toHaveBeenCalledTimes(1);
   });
 
   it('logs bandit evaluation for flag with bandit data', () => {
     const banditActions = {
-      'show_red_button': {
+      show_red_button: {
         expectedConversion: 0.23,
         expectedRevenue: 15.75,
         category: 'promotion',
-        placement: 'home_screen'
-      }
+        placement: 'home_screen',
+      },
     };
-    
-    const result = client.getBanditAction('string-flag', 'test-subject-key', {}, banditActions, 'default');
-    
+
+    const result = client.getBanditAction(
+      'string-flag',
+      'test-subject-key',
+      {},
+      banditActions,
+      'default',
+    );
+
     expect(result.variation).toBe('red');
     expect(result.action).toBe('show_red_button');
     expect(mockBanditLogger.logBanditAction).toHaveBeenCalledTimes(1);
-    
+
     const call = mockBanditLogger.logBanditAction.mock.calls[0][0];
     expect(call.bandit).toBe('recommendation-model-v1');
     expect(call.action).toBe('show_red_button');
@@ -99,27 +112,42 @@ describe('EppoClient Precomputed Mode', () => {
   });
 
   it('returns default values for nonexistent flags', () => {
-    const stringResult = client.getStringAssignment('nonexistent-flag', 'test-subject-key', {}, 'default-string');
+    const stringResult = client.getStringAssignment(
+      'nonexistent-flag',
+      'test-subject-key',
+      {},
+      'default-string',
+    );
     expect(stringResult).toBe('default-string');
-    
-    const boolResult = client.getBooleanAssignment('nonexistent-flag', 'test-subject-key', {}, true);
+
+    const boolResult = client.getBooleanAssignment(
+      'nonexistent-flag',
+      'test-subject-key',
+      {},
+      true,
+    );
     expect(boolResult).toBe(true);
-    
+
     const intResult = client.getIntegerAssignment('nonexistent-flag', 'test-subject-key', {}, 100);
     expect(intResult).toBe(100);
   });
 
   it('correctly handles assignment details', () => {
-    const details = client.getStringAssignmentDetails('string-flag', 'test-subject-key', {}, 'default');
-    
+    const details = client.getStringAssignmentDetails(
+      'string-flag',
+      'test-subject-key',
+      {},
+      'default',
+    );
+
     expect(details.variation).toBe('red');
     expect(details.evaluationDetails.variationKey).toBe('variation-123');
-    
+
     // Assignment should be logged
     expect(mockAssignmentLogger.logAssignment).toHaveBeenCalledTimes(1);
     const call = mockAssignmentLogger.logAssignment.mock.calls[0][0];
     expect(call.allocation).toBe('allocation-123');
     expect(call.featureFlag).toBe('string-flag');
     expect(call.subject).toBe('test-subject-key');
   });
-}); 
+});

src/client/eppo-client.spec.ts

@@ -2,10 +2,7 @@ import * as base64 from 'js-base64';
 import { times } from 'lodash';
 import * as td from 'testdouble';
 
-import {
-  MOCK_UFC_RESPONSE_FILE,
-  readMockUFCResponse,
-} from '../../test/testHelpers';
+import { MOCK_UFC_RESPONSE_FILE, readMockUFCResponse } from '../../test/testHelpers';
 import { IAssignmentLogger } from '../assignment-logger';
 import { AssignmentCache } from '../cache/abstract-assignment-cache';
 import {
@@ -28,7 +25,6 @@ jest.mock('../salt', () => ({
 }));
 
 describe('EppoClient E2E test', () => {
-
   // Configure fetch mock for tests that still need it
   global.fetch = jest.fn(() => {
     const ufc = readMockUFCResponse(MOCK_UFC_RESPONSE_FILE);