Make response data not optional

Author: sameerank

src/client/eppo-precomputed-client.ts

@@ -144,12 +144,8 @@ export default class EppoPrecomputedClient {
 
     // A callback to capture the salt and subject information
     precomputedRequestor.onPrecomputedResponse = (responseData) => {
-      if (responseData.salt) {
-        this.setDecodedFlagKeySalt(decodeBase64(responseData.salt));
-      }
-      if (responseData.subjectKey && responseData.subjectAttributes) {
-        this.setSubjectData(responseData.subjectKey, responseData.subjectAttributes);
-      }
+      this.setDecodedFlagKeySalt(decodeBase64(responseData.salt));
+      this.setSubjectData(responseData.subjectKey, responseData.subjectAttributes);
     };
 
     const pollingCallback = async () => {

src/precomputed-requestor.ts

@@ -5,9 +5,9 @@ import { PrecomputedFlag, UNKNOWN_ENVIRONMENT_NAME } from './interfaces';
 import { Attributes } from './types';
 
 export interface PrecomputedResponseData {
-  salt?: string;
-  subjectKey?: string;
-  subjectAttributes?: Attributes;
+  salt: string;
+  subjectKey: string;
+  subjectAttributes: Attributes;
 }
 
 // Requests AND stores precomputed flags, reuses the configuration store
@@ -27,9 +27,9 @@ export default class PrecomputedFlagRequestor {
       subject_attributes: this.subjectAttributes,
     });
 
-    if (this.onPrecomputedResponse) {
+    if (this.onPrecomputedResponse && precomputedResponse) {
       this.onPrecomputedResponse({
-        salt: precomputedResponse?.salt,
+        salt: precomputedResponse.salt,
         subjectKey: this.subjectKey,
         subjectAttributes: this.subjectAttributes,
       });