chore: refactor to ensure Default events before the SDK is initialized are tracked correctly

Author: kaushalkapasi

src/main/java/com/devcycle/sdk/server/common/model/EvalReason.java

@@ -17,16 +17,21 @@ public class EvalReason {
     @JsonProperty("reason")
     private String reason;
 
-    @Schema(description = "Details", required = false)
+    @Schema(description = "Details")
     @JsonProperty("details")
     private String details;
 
-    @Schema(description = "Target ID", required = false)
-    @JsonProperty("targetId")
+    @Schema(description = "Target ID")
+    @JsonProperty("target_id")
     private String targetId;
 
+    private EvalReason(String reason, String details) {
+        this.reason = reason;
+        this.details = details;
+    }
+
     public static EvalReason defaultReason(DefaultReasonDetailsEnum details) {
-        return new EvalReason("DEFAULT", details.getValue(), null);
+        return new EvalReason("DEFAULT", details.getValue());
     }
 
     public String getReason() {

src/main/java/com/devcycle/sdk/server/common/model/Variable.java

@@ -33,6 +33,10 @@ public class Variable<T> {
     @Builder.Default
     private Boolean isDefaulted = false;
 
+    @Builder.Default
+    @Deprecated()
+    private String evalReason = null;
+
     @Schema(description = "Evaluation reason")
     private EvalReason eval;
 

src/main/java/com/devcycle/sdk/server/local/api/DevCycleLocalClient.java

@@ -155,12 +155,17 @@ public <T> Variable<T> variable(DevCycleUser user, String key, T defaultValue) {
 
         if (!isInitialized()) {
             DevCycleLogger.info("Variable called before DevCycleLocalClient has initialized, returning default value");
+            defaultVariable.setEval(EvalReason.defaultReason(EvalReason.DefaultReasonDetailsEnum.MISSING_CONFIG));
             try {
-                eventQueueManager.queueAggregateEvent(DevCycleEvent.builder().type("aggVariableDefaulted").target(key).build(), null);
+                eventQueueManager.queueAggregateEvent(DevCycleEvent.builder()
+                    .type("aggVariableDefaulted")
+                    .target(key)
+                    .metaData(Map.of("evalReason", defaultVariable.getEval().getReason()))
+                    .build(),
+                null);
             } catch (Exception e) {
                 DevCycleLogger.error("Unable to parse aggVariableDefaulted event for Variable " + key + " due to error: " + e, e);
             }
-            defaultVariable.setEval(EvalReason.defaultReason(EvalReason.DefaultReasonDetailsEnum.MISSING_CONFIG));
             return defaultVariable;
         }
 

src/main/java/com/devcycle/sdk/server/local/managers/EventQueueManager.java

@@ -1,5 +1,11 @@
 package com.devcycle.sdk.server.local.managers;
 
+import java.io.IOException;
+import java.util.Arrays;
+import java.util.concurrent.Executors;
+import java.util.concurrent.ScheduledExecutorService;
+import java.util.concurrent.TimeUnit;
+
 import com.devcycle.sdk.server.common.api.IDevCycleApi;
 import com.devcycle.sdk.server.common.logging.DevCycleLogger;
 import com.devcycle.sdk.server.common.model.DevCycleEvent;
@@ -13,15 +19,10 @@
 import com.devcycle.sdk.server.local.model.FlushPayload;
 import com.fasterxml.jackson.annotation.JsonInclude;
 import com.fasterxml.jackson.databind.ObjectMapper;
+
 import retrofit2.Call;
 import retrofit2.Response;
 
-import java.io.IOException;
-import java.util.Arrays;
-import java.util.concurrent.Executors;
-import java.util.concurrent.ScheduledExecutorService;
-import java.util.concurrent.TimeUnit;
-
 public class EventQueueManager {
 
     private static final ObjectMapper OBJECT_MAPPER = new ObjectMapper();