chore: tsc enable strict type checks

package.json

@@ -72,5 +72,6 @@
     "md5": "^2.3.0",
     "pino": "^8.19.0",
     "semver": "^7.5.4"
-  }
+  },
+  "packageManager": "[email protected]+sha512.a6b2f7906b721bba3d67d4aff083df04dad64c399707841b7acf00f6b133b7ac24255f2652fa22ae3534329dc6180534e98d17432037ff6fd140556e2bb3137e"
 }

src/bandit-evaluator.ts

@@ -20,8 +20,8 @@ export interface BanditEvaluation {
 }
 
 export class BanditEvaluator {
-  private assignmentShards = BANDIT_ASSIGNMENT_SHARDS; // We just hard code this for now
-  private sharder: Sharder = new MD5Sharder();
+  private readonly assignmentShards = BANDIT_ASSIGNMENT_SHARDS; // We just hard code this for now
+  private readonly sharder: Sharder = new MD5Sharder();
 
   public evaluateBandit(
     flagKey: string,

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

@@ -13,6 +13,7 @@ import { AllocationEvaluationCode } from '../flag-evaluation-details-builder';
 import FetchHttpClient from '../http-client';
 import { Flag, ObfuscatedFlag, Variation, VariationType } from '../interfaces';
 import { OperatorType } from '../rules';
+import { AttributeType } from '../types';
 
 import EppoClient, { IAssignmentDetails } from './eppo-client';
 
@@ -345,7 +346,12 @@ describe('EppoClient get*AssignmentDetails', () => {
                 [VariationType.STRING]: client.getStringAssignmentDetails.bind(client),
                 [VariationType.JSON]: client.getJSONAssignmentDetails.bind(client),
               };
-              const assignmentFn = typeAssignmentDetailsFunctions[variationType];
+              const assignmentFn = typeAssignmentDetailsFunctions[variationType] as (
+                flagKey: string,
+                subjectKey: string,
+                subjectAttributes: Record<string, AttributeType>,
+                defaultValue: boolean | string | number | object,
+              ) => IAssignmentDetails<boolean | string | number | object>;
               if (!assignmentFn) {
                 throw new Error(`Unknown variation type: ${variationType}`);
               }

src/client/eppo-client.ts

@@ -509,13 +509,13 @@
         );
         evaluationDetails.banditAction = action;
       }
-    } catch (err) {
+    } catch (err: any) {
       logger.error('Error determining bandit action', err);
       if (!this.isGracefulFailureMode) {
         throw err;
       }
       if (variation) {
         // If we have a variation, the assignment succeeded and the error was with the bandit part.
         // Update the flag evaluation code to indicate that
         evaluationDetails.flagEvaluationCode = 'BANDIT_ERROR';
       }
@@ -700,7 +700,7 @@
         expectedVariationType,
       );
       return this.parseVariationWithDetails(result, defaultValue, expectedVariationType);
-    } catch (error) {
+    } catch (error: any) {
       const eppoValue = this.rethrowIfNotGraceful(error, defaultValue);
       if (error instanceof FlagEvaluationError && error.flagEvaluationDetails) {
         return {
@@ -738,7 +738,7 @@
         eppoValue: EppoValue.valueOf(result.variation.value, expectedVariationType),
         flagEvaluationDetails: result.flagEvaluationDetails,
       };
-    } catch (error) {
+    } catch (error: any) {
       const eppoValue = this.rethrowIfNotGraceful(error, defaultValue);
       return {
         eppoValue,
@@ -752,7 +752,7 @@
       logger.error(`[Eppo SDK] Error getting assignment: ${err.message}`);
       return defaultValue ?? EppoValue.Null();
     }
     throw err;
   }
 
   /**
@@ -790,7 +790,7 @@
       return noneResult(flagKey, subjectKey, subjectAttributes, flagEvaluationDetails);
     }
 
     if (!checkTypeMatch(expectedVariationType, flag.variationType)) {
       const errorMessage = `Variation value does not have the correct type. Found ${flag.variationType}, but expected ${expectedVariationType} for flag ${flagKey}`;
       if (this.isGracefulFailureMode) {
         const flagEvaluationDetails = flagEvaluationDetailsBuilder.buildForNoneResult(
@@ -958,7 +958,7 @@
     eventsToFlush.forEach((event) => {
       try {
         logFunction(event);
-      } catch (error) {
+      } catch (error: any) {
         logger.error(`[Eppo SDK] Error flushing event to logger: ${error.message}`);
       }
     });
@@ -1005,12 +1005,12 @@
         allocationKey: allocationKey ?? '__eppo_no_allocation',
         variationKey: variation?.key ?? '__eppo_no_variation',
       });
-    } catch (error) {
+    } catch (error: any) {
       logger.error(`[Eppo SDK] Error logging assignment event: ${error.message}`);
     }
   }
 
   private buildLoggerMetadata(): Record<string, unknown> {
     return {
       obfuscated: this.isObfuscated,
       sdkLanguage: 'javascript',

src/evaluator.ts

@@ -32,7 +32,7 @@
 }
 
 export class Evaluator {
-  sharder: Sharder;
+  private readonly sharder: Sharder;
 
   constructor(sharder?: Sharder) {
     this.sharder = sharder ?? new MD5Sharder();
@@ -133,7 +133,7 @@
           'No allocations matched. Falling back to "Default Allocation", serving NULL',
         ),
       );
-    } catch (err) {
+    } catch (err: any) {
       const flagEvaluationDetails = flagEvaluationDetailsBuilder.gracefulBuild(
         'ASSIGNMENT_ERROR',
         `Assignment Error: ${err.message}`,

src/http-client.ts

@@ -61,7 +61,7 @@
         throw new HttpRequestError('Failed to fetch data', response?.status);
       }
       return await response.json();
-    } catch (error) {
+    } catch (error: any) {
       if (error.name === 'AbortError') {
         throw new HttpRequestError('Request timed out', 408, error);
       } else if (error instanceof HttpRequestError) {

src/poller.ts

@@ -48,7 +48,7 @@
         startRequestSuccess = true;
         previousPollFailed = false;
         logger.info('Eppo SDK successfully requested initial configuration');
-      } catch (pollingError) {
+      } catch (pollingError: any) {
         previousPollFailed = true;
         logger.warn(
           `Eppo SDK encountered an error with initial poll of configurations: ${pollingError.message}`,
@@ -116,7 +116,7 @@
         previousPollFailed = false;
         logger.info('Eppo SDK poll successful; resuming normal polling');
       }
-    } catch (error) {
+    } catch (error: any) {
       previousPollFailed = true;
       logger.warn(`Eppo SDK encountered an error polling configurations: ${error.message}`);
       const maxTries = 1 + (options?.maxPollRetries ?? DEFAULT_POLL_CONFIG_REQUEST_RETRIES);

tsconfig.json

@@ -8,7 +8,7 @@
     "target": "es2017",
     "outDir": "dist",
     "noImplicitAny": true,
-    "strictNullChecks": true,
+    "strict": true,
     "strictPropertyInitialization": true
   },
   "include": ["src/**/*.ts"],