fix: resolve eslint errors in multi-provider migration

Author: jonathannorris

packages/server/src/provider/multi-provider/strategies/base-evaluation-strategy.ts

@@ -1,3 +1,4 @@
+/* eslint-disable @typescript-eslint/no-unused-vars */
 import type {
   ErrorCode,
   EvaluationContext,
@@ -67,7 +68,6 @@ export abstract class BaseEvaluationStrategy {
     return true;
   }
 
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
   shouldEvaluateNextProvider<T extends FlagValue>(
     _strategyContext?: StrategyPerProviderContext,
     _context?: EvaluationContext,
@@ -76,7 +76,6 @@ export abstract class BaseEvaluationStrategy {
     return true;
   }
 
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
   shouldTrackWithThisProvider(
     strategyContext: StrategyProviderContext,
     _context?: EvaluationContext,

packages/server/test/multi-provider.spec.ts

@@ -1,3 +1,4 @@
+/* eslint-disable @typescript-eslint/no-explicit-any */
 import { MultiProvider } from '../src/provider/multi-provider/multi-provider';
 import type {
   EvaluationContext,
@@ -43,7 +44,7 @@ class TestProvider implements Provider {
 }
 
 const callEvaluation = async (multi: MultiProvider, context: EvaluationContext, logger: Logger) => {
-  await callBeforeHook(multi, context, 'flag', 'boolean', false);
+  await callBeforeHook(multi, context, 'flag', 'boolean', false, logger);
   return multi.resolveBooleanEvaluation('flag', false, context);
 };
 

packages/web/src/provider/multi-provider/strategies/base-evaluation-strategy.ts

@@ -1,3 +1,4 @@
+/* eslint-disable @typescript-eslint/no-unused-vars */
 import type {
   ErrorCode,
   EvaluationContext,
@@ -65,7 +66,6 @@ export abstract class BaseEvaluationStrategy {
     return true;
   }
 
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
   shouldEvaluateNextProvider<T extends FlagValue>(
     strategyContext: StrategyPerProviderContext,
     context: EvaluationContext,
@@ -74,7 +74,6 @@ export abstract class BaseEvaluationStrategy {
     return true;
   }
 
-  // eslint-disable-next-line @typescript-eslint/no-unused-vars
   shouldTrackWithThisProvider(
     strategyContext: StrategyProviderContext,
     context: EvaluationContext,

packages/web/src/provider/multi-provider/strategies/ComparisonStrategy.ts renamed to packages/web/src/provider/multi-provider/strategies/comparison-strategy.ts

@@ -1,4 +1,4 @@
 export * from './base-evaluation-strategy';
-export * from './FirstMatchStrategy';
-export * from './FirstSuccessfulStrategy';
-export * from './ComparisonStrategy';
+export * from './first-match-strategy';
+export * from './first-successful-strategy';
+export * from './comparison-strategy';

packages/web/src/provider/multi-provider/strategies/FirstMatchStrategy.ts renamed to packages/web/src/provider/multi-provider/strategies/first-match-strategy.ts

@@ -1,3 +1,4 @@
+/* eslint-disable @typescript-eslint/no-explicit-any */
 import { WebMultiProvider } from '../src/provider/multi-provider/multi-provider-web';
 import type {
   EvaluationContext,