fix: downgrade null-arg track logs from warn to debug per spec

Author: mattrmc1

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

@@ -155,7 +155,7 @@ public String getResumptionToken() {
   @Override
   public void trackDuration(Duration duration) {
     if (duration == null) {
-      logger.warn("Skipping trackDuration: duration was null.");
+      logger.debug("Skipping trackDuration: duration was null.");
       return;
     }
     long ms = Math.max(0L, duration.toMillis());
@@ -181,7 +181,7 @@ public <T> T trackDurationOf(Callable<T> operation) throws Exception {
   @Override
   public void trackTimeToFirstToken(Duration duration) {
     if (duration == null) {
-      logger.warn("Skipping trackTimeToFirstToken: duration was null.");
+      logger.debug("Skipping trackTimeToFirstToken: duration was null.");
       return;
     }
     long ms = Math.max(0L, duration.toMillis());
@@ -213,7 +213,7 @@ public void trackError() {
   @Override
   public void trackFeedback(FeedbackKind kind) {
     if (kind == null) {
-      logger.warn("Skipping trackFeedback: kind was null.");
+      logger.debug("Skipping trackFeedback: kind was null.");
       return;
     }
     // Resolve event name BEFORE claiming the guard — an exception here must not burn the slot.
@@ -228,7 +228,7 @@ public void trackFeedback(FeedbackKind kind) {
   @Override
   public void trackTokens(TokenUsage tokens) {
     if (tokens == null) {
-      logger.warn("Skipping trackTokens: tokens was null.");
+      logger.debug("Skipping trackTokens: tokens was null.");
       return;
     }
     boolean hasPositive = tokens.getTotal() > 0 || tokens.getInput() > 0 || tokens.getOutput() > 0;
@@ -254,7 +254,7 @@ public void trackTokens(TokenUsage tokens) {
   @Override
   public void trackToolCall(String toolKey) {
     if (toolKey == null) {
-      logger.warn("Skipping trackToolCall: toolKey was null.");
+      logger.debug("Skipping trackToolCall: toolKey was null.");
       return;
     }
     toolCalls.add(toolKey);
@@ -275,7 +275,7 @@ public void trackToolCalls(List<String> toolKeys) {
   @Override
   public void trackJudgeResult(JudgeResult result) {
     if (result == null) {
-      logger.warn("Skipping trackJudgeResult: result was null.");
+      logger.debug("Skipping trackJudgeResult: result was null.");
       return;
     }
     if (!result.isSampled()) {

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

@@ -3,6 +3,7 @@
 import static org.hamcrest.MatcherAssert.assertThat;
 import static org.hamcrest.Matchers.containsInAnyOrder;
 import static org.hamcrest.Matchers.containsString;
+import static org.hamcrest.Matchers.empty;
 import static org.hamcrest.Matchers.greaterThanOrEqualTo;
 import static org.hamcrest.Matchers.is;
 import static org.hamcrest.Matchers.notNullValue;
@@ -82,6 +83,13 @@ private List<String> warnings() {
         .collect(Collectors.toList());
   }
 
+  private List<String> debugs() {
+    return logCapture.getMessages().stream()
+        .filter(m -> m.getLevel() == LDLogLevel.DEBUG)
+        .map(LogCapture.Message::getText)
+        .collect(Collectors.toList());
+  }
+
   private LDValue baseExpectedData() {
     return LDValue.buildObject()
         .put("runId", RUN_ID)
@@ -172,10 +180,11 @@ public void trackDurationAtMostOnce() {
   }
 
   @Test
-  public void trackDurationNullIsIgnoredWithWarning() {
+  public void trackDurationNullIsIgnoredWithDebugLog() {
     tracker.trackDuration(null);
     verify(client, never()).trackMetric(eq("$ld:ai:duration:total"), any(), any(), anyDouble());
-    assertThat(warnings().size(), greaterThanOrEqualTo(1));
+    assertThat(debugs().size(), greaterThanOrEqualTo(1));
+    assertThat(warnings(), is(empty()));
   }
 
   // ---- trackDurationOf ------------------------------------------------------
@@ -215,10 +224,11 @@ public void trackTimeToFirstTokenAtMostOnce() {
   }
 
   @Test
-  public void trackTimeToFirstTokenNullIsIgnoredWithWarning() {
+  public void trackTimeToFirstTokenNullIsIgnoredWithDebugLog() {
     tracker.trackTimeToFirstToken(null);
     verify(client, never()).trackMetric(eq("$ld:ai:tokens:ttf"), any(), any(), anyDouble());
-    assertThat(warnings().size(), greaterThanOrEqualTo(1));
+    assertThat(debugs().size(), greaterThanOrEqualTo(1));
+    assertThat(warnings(), is(empty()));
   }
 
   // ---- trackSuccess / trackError --------------------------------------------
@@ -299,9 +309,10 @@ public void trackFeedbackAtMostOnce() {
   }
 
   @Test
-  public void trackFeedbackNullIsIgnoredWithWarning_slotNotBurned() {
+  public void trackFeedbackNullIsIgnoredWithDebugLog_slotNotBurned() {
     tracker.trackFeedback(null);
-    assertThat(warnings().size(), greaterThanOrEqualTo(1));
+    assertThat(debugs().size(), greaterThanOrEqualTo(1));
+    assertThat(warnings(), is(empty()));
     // Slot should not be burned — a subsequent valid call should still work
     tracker.trackFeedback(FeedbackKind.POSITIVE);
     verify(client, times(1)).trackMetric(eq("$ld:ai:feedback:user:positive"), any(), any(), anyDouble());
@@ -342,10 +353,11 @@ public void trackTokensAtMostOnce() {
   }
 
   @Test
-  public void trackTokensNullIsIgnoredWithWarning() {
+  public void trackTokensNullIsIgnoredWithDebugLog() {
     tracker.trackTokens(null);
     verify(client, never()).trackMetric(eq("$ld:ai:tokens:total"), any(), any(), anyDouble());
-    assertThat(warnings().size(), greaterThanOrEqualTo(1));
+    assertThat(debugs().size(), greaterThanOrEqualTo(1));
+    assertThat(warnings(), is(empty()));
   }
 
   // ---- trackToolCall --------------------------------------------------------
@@ -382,10 +394,11 @@ public void trackToolCallsDelegate() {
   }
 
   @Test
-  public void trackToolCallNullIsIgnoredWithWarning() {
+  public void trackToolCallNullIsIgnoredWithDebugLog() {
     tracker.trackToolCall(null);
     verify(client, never()).trackMetric(eq("$ld:ai:tool_call"), any(), any(), anyDouble());
-    assertThat(warnings().size(), greaterThanOrEqualTo(1));
+    assertThat(debugs().size(), greaterThanOrEqualTo(1));
+    assertThat(warnings(), is(empty()));
   }
 
   // ---- trackJudgeResult -----------------------------------------------------
@@ -470,9 +483,10 @@ public void trackJudgeResultIsNotAtMostOnce() {
   }
 
   @Test
-  public void trackJudgeResultNullIsIgnoredWithWarning() {
+  public void trackJudgeResultNullIsIgnoredWithDebugLog() {
     tracker.trackJudgeResult(null);
-    assertThat(warnings().size(), greaterThanOrEqualTo(1));
+    assertThat(debugs().size(), greaterThanOrEqualTo(1));
+    assertThat(warnings(), is(empty()));
   }
 
   // ---- trackMetricsOf -------------------------------------------------------