Small fixes

Author: Jonas-Isr

orchestration/src/main/java/com/sap/ai/sdk/orchestration/ImageItem.java

@@ -18,17 +18,19 @@ public record ImageItem(@Nonnull String imageUrl, @Nonnull DetailLevel detailLev
    * @param imageUrl the URL of the image
    */
   public ImageItem(@Nonnull final String imageUrl) {
-    this(imageUrl, DetailLevel.auto);
+    this(imageUrl, DetailLevel.AUTO);
   }
 
   /** The detail level of the image. */
   public enum DetailLevel {
     /** Low detail level. */
-    low,
+    LOW("low"),
     /** High detail level. */
-    high,
+    HIGH("high"),
     /** Automatic detail level. */
-    auto;
+    AUTO("auto");
+
+    private final String level;
 
     /**
      * Converts a string to a detail level.
@@ -38,7 +40,21 @@ public enum DetailLevel {
      */
     @Nonnull
     static DetailLevel fromString(@Nonnull final String str) {
-      return DetailLevel.valueOf(str.toLowerCase(Locale.ENGLISH));
+      return DetailLevel.valueOf(str.toUpperCase(Locale.ENGLISH));
+    }
+
+    /**
+     * Get the string representation of the DetailLevel
+     *
+     * @return the DetailLevel as string
+     */
+    @Nonnull
+    public String toString() {
+      return level;
+    }
+
+    DetailLevel(@Nonnull final String level) {
+      this.level = level;
     }
   }
 }

orchestration/src/main/java/com/sap/ai/sdk/orchestration/MessageContent.java

@@ -10,8 +10,6 @@
 import javax.annotation.Nonnull;
 import javax.annotation.Nullable;
 
-import static java.util.stream.Collectors.toList;
-
 /**
  * Represents the content of a chat message.
  *
@@ -58,12 +56,11 @@ public static MessageContent image(@Nonnull final String imageUrl) {
   @Nonnull
   public static MessageContent text(
       @Nonnull final String message, @Nullable final String... additionalMessages) {
-    var messagesStream =
+    final var messagesStream =
         (additionalMessages != null)
             ? Stream.concat(Stream.of(message), Stream.of(additionalMessages))
             : Stream.of(message);
-    var contentList =
-         messagesStream.map(text -> (ContentItem) new TextItem(text)).toList();
+    final var contentList = messagesStream.map(text -> (ContentItem) new TextItem(text)).toList();
     return new MessageContent(contentList);
   }
 

orchestration/src/main/java/com/sap/ai/sdk/orchestration/UserMessage.java

@@ -1,7 +1,6 @@
 package com.sap.ai.sdk.orchestration;
 
 import com.google.common.annotations.Beta;
-
 import java.util.LinkedList;
 import java.util.List;
 import java.util.stream.Stream;
@@ -46,11 +45,11 @@ public UserMessage(@Nonnull final String message, @Nullable final String... addi
   @Nonnull
   public UserMessage andText(
       @Nonnull final String message, @Nullable final String... additionalMessages) {
-    var messagesStream =
+    final var messagesStream =
         (additionalMessages != null)
             ? Stream.concat(Stream.of(message), Stream.of(additionalMessages))
             : Stream.of(message);
-    var contentList =
+    final var contentList =
         Stream.concat(content.contentItemList().stream(), messagesStream.map(TextItem::new))
             .toList();
     return new UserMessage(new MessageContent(contentList));

orchestration/src/test/java/com/sap/ai/sdk/orchestration/OrchestrationConvenienceUnitTest.java

@@ -32,9 +32,9 @@ void testMessageConstructionImage() {
     assertThat(userMessageWithImageAndImage).isEqualTo(userMessageWithImageAnd);
 
     var userMessageWithImageDetailAndImage =
-        Message.user("Text 1").andImage("url", ImageItem.DetailLevel.low);
+        Message.user("Text 1").andImage("url", ImageItem.DetailLevel.LOW);
     var userMessageWithImageDetailAnd =
-        Message.user("Text 1").and(MessageContent.image("url", ImageItem.DetailLevel.low));
+        Message.user("Text 1").and(MessageContent.image("url", ImageItem.DetailLevel.LOW));
     assertThat(userMessageWithImageDetailAndImage).isEqualTo(userMessageWithImageDetailAnd);
   }
 }

sample-code/spring-app/src/main/java/com/sap/ai/sdk/app/services/OrchestrationService.java

@@ -67,7 +67,7 @@ public OrchestrationChatResponse imageInput(@Nonnull final String pathToImage) {
         new OrchestrationModuleConfig().withLlmConfig(GPT_4O_MINI);
 
     final var multiMessage =
-        Message.user("What is in this image?").andImage(pathToImage, ImageItem.DetailLevel.low);
+        Message.user("What is in this image?").andImage(pathToImage, ImageItem.DetailLevel.LOW);
     final var prompt = new OrchestrationPrompt(multiMessage);
     return client.chatCompletion(prompt, llmWithImageSupportConfig);
   }