fix: properly set variable type mismatch as eval details on Cloud Bucketing Client

Author: kaushalkapasi

src/main/java/com/devcycle/sdk/server/cloud/api/DevCycleCloudClient.java

@@ -162,7 +162,12 @@ public <T> Variable<T> variable(DevCycleUser user, String key, T defaultValue) {
                         .defaultValue(defaultValue)
                         .isDefaulted(true)
                         .build();
-                variable.setEval(EvalReason.defaultReason(EvalReason.DefaultReasonDetailsEnum.USER_NOT_TARGETED));
+
+                if (exception.getMessage().equals("Variable type mismatch, returning default value")) {
+                    variable.setEval(EvalReason.defaultReason(EvalReason.DefaultReasonDetailsEnum.VARIABLE_TYPE_MISMATCH));
+                } else {
+                    variable.setEval(EvalReason.defaultReason(EvalReason.DefaultReasonDetailsEnum.USER_NOT_TARGETED));
+                }
             }
 
             evalHooksRunner.executeError(reversedHooks, context, exception);

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

@@ -1,6 +1,7 @@
 package com.devcycle.sdk.server.common.model;
 
 import com.fasterxml.jackson.annotation.JsonIgnoreProperties;
+import com.fasterxml.jackson.annotation.JsonInclude;
 import com.fasterxml.jackson.annotation.JsonProperty;
 
 import io.swagger.v3.oas.annotations.media.Schema;
@@ -23,6 +24,7 @@ public class EvalReason {
 
     @Schema(description = "Target ID")
     @JsonProperty("target_id")
+    @JsonInclude(value=JsonInclude.Include.NON_EMPTY, content=JsonInclude.Include.NON_NULL)
     private String targetId;
 
     private EvalReason(String reason, String details) {
@@ -41,7 +43,7 @@ public String getReason() {
     public enum DefaultReasonDetailsEnum {
         MISSING_CONFIG("Missing Config"),
         USER_NOT_TARGETED("User Not Targeted"),
-        VARIABLE_TYPE_MISMATCH("Type Mismatch"),
+        VARIABLE_TYPE_MISMATCH("Variable Type Mismatch"),
         ERROR("Error");
 
         private final String value;

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

@@ -116,10 +116,13 @@ public void queueAggregateEvent(DevCycleEvent event, BucketedUserConfig bucketed
             return;
         }
 
+        String eventString = OBJECT_MAPPER.writeValueAsString(event);
+
         if (bucketedConfig != null) {
-            this.localBucketing.queueAggregateEvent(this.sdkKey, OBJECT_MAPPER.writeValueAsString(event), OBJECT_MAPPER.writeValueAsString(bucketedConfig.variableVariationMap));
+            String variableVariationMapString = OBJECT_MAPPER.writeValueAsString(bucketedConfig.variableVariationMap);
+            this.localBucketing.queueAggregateEvent(this.sdkKey, eventString, variableVariationMapString);
         } else {
-            this.localBucketing.queueAggregateEvent(this.sdkKey, OBJECT_MAPPER.writeValueAsString(event), "{}");
+            this.localBucketing.queueAggregateEvent(this.sdkKey, eventString, "{}");
         }
     }