fix: skip graph edge tracking events when node keys are blank

Author: mattrmc1

lib/sdk/server-ai/src/main/java/com/launchdarkly/sdk/server/ai/AIGraphTracker.java

@@ -207,6 +207,10 @@ public void trackPath(List<String> path) {
    * @param redirectedTarget the key of the node that was actually used
    */
   public void trackRedirect(String sourceKey, String redirectedTarget) {
+    if (isNullOrBlank(sourceKey) || isNullOrBlank(redirectedTarget)) {
+      logger.debug("Skipping trackRedirect: sourceKey or redirectedTarget was null or blank.");
+      return;
+    }
     LDValue data = baseData()
         .put("sourceKey", sourceKey)
         .put("redirectedTarget", redirectedTarget)
@@ -223,6 +227,10 @@ public void trackRedirect(String sourceKey, String redirectedTarget) {
    * @param targetKey the key of the target node
    */
   public void trackHandoffSuccess(String sourceKey, String targetKey) {
+    if (isNullOrBlank(sourceKey) || isNullOrBlank(targetKey)) {
+      logger.debug("Skipping trackHandoffSuccess: sourceKey or targetKey was null or blank.");
+      return;
+    }
     LDValue data = baseData()
         .put("sourceKey", sourceKey)
         .put("targetKey", targetKey)
@@ -239,6 +247,10 @@ public void trackHandoffSuccess(String sourceKey, String targetKey) {
    * @param targetKey the key of the target node
    */
   public void trackHandoffFailure(String sourceKey, String targetKey) {
+    if (isNullOrBlank(sourceKey) || isNullOrBlank(targetKey)) {
+      logger.debug("Skipping trackHandoffFailure: sourceKey or targetKey was null or blank.");
+      return;
+    }
     LDValue data = baseData()
         .put("sourceKey", sourceKey)
         .put("targetKey", targetKey)
@@ -284,4 +296,8 @@ private ObjectBuilder baseData() {
     return b;
   }
 
+  private static boolean isNullOrBlank(String value) {
+    return value == null || value.trim().isEmpty();
+  }
+
 }

lib/sdk/server-ai/src/test/java/com/launchdarkly/sdk/server/ai/AIGraphTrackerTest.java

@@ -265,6 +265,17 @@ public void trackRedirectIsMultiFire() {
         eq("$ld:ai:graph:redirect"), any(), any(), anyDouble());
   }
 
+  @Test
+  public void trackRedirectSkipsWhenKeyBlankOrNull() {
+    tracker.trackRedirect(null, "target-b");
+    tracker.trackRedirect("source-a", "  ");
+    tracker.trackRedirect("", "target-b");
+    verify(client, never()).trackMetric(
+        eq("$ld:ai:graph:redirect"), any(), any(), anyDouble());
+    assertThat(debugs().stream().anyMatch(
+        w -> w.contains("Skipping trackRedirect")), is(true));
+  }
+
   // ---- trackHandoffSuccess --------------------------------------------------
 
   @Test
@@ -288,6 +299,17 @@ public void trackHandoffSuccessIsMultiFire() {
         eq("$ld:ai:graph:handoff_success"), any(), any(), anyDouble());
   }
 
+  @Test
+  public void trackHandoffSuccessSkipsWhenKeyBlankOrNull() {
+    tracker.trackHandoffSuccess(null, "target-b");
+    tracker.trackHandoffSuccess("source-a", "  ");
+    tracker.trackHandoffSuccess("source-a", "");
+    verify(client, never()).trackMetric(
+        eq("$ld:ai:graph:handoff_success"), any(), any(), anyDouble());
+    assertThat(debugs().stream().anyMatch(
+        w -> w.contains("Skipping trackHandoffSuccess")), is(true));
+  }
+
   // ---- trackHandoffFailure --------------------------------------------------
 
   @Test
@@ -311,6 +333,17 @@ public void trackHandoffFailureIsMultiFire() {
         eq("$ld:ai:graph:handoff_failure"), any(), any(), anyDouble());
   }
 
+  @Test
+  public void trackHandoffFailureSkipsWhenKeyBlankOrNull() {
+    tracker.trackHandoffFailure(null, "target-b");
+    tracker.trackHandoffFailure("source-a", "  ");
+    tracker.trackHandoffFailure("source-a", "");
+    verify(client, never()).trackMetric(
+        eq("$ld:ai:graph:handoff_failure"), any(), any(), anyDouble());
+    assertThat(debugs().stream().anyMatch(
+        w -> w.contains("Skipping trackHandoffFailure")), is(true));
+  }
+
   // ---- getSummary -----------------------------------------------------------
 
   @Test