Fix codestyle

Author: rpanackal

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

@@ -21,5 +21,4 @@ interface ModuleResultsOutputUnmaskingInnerMixIn {}
 
   @JsonTypeInfo(use = JsonTypeInfo.Id.NONE)
   interface NoneTypeInfoMixin {}
-  
 }

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

@@ -55,10 +55,11 @@ public TokenUsage getTokenUsage() {
    * @return A list of all messages.
    */
   @Nonnull
-  public List<Message> getAllMessages() throws UnsupportedOperationException{
+  public List<Message> getAllMessages() throws UnsupportedOperationException {
     final var messages = new ArrayList<Message>();
 
-    for (final ChatMessagesInner chatMessage : originalResponse.getModuleResults().getTemplating()) {
+    for (final ChatMessagesInner chatMessage :
+        originalResponse.getModuleResults().getTemplating()) {
       if (chatMessage instanceof ChatMessage simpleMsg) {
         final var message =
             switch (simpleMsg.getRole()) {
@@ -68,9 +69,9 @@ public List<Message> getAllMessages() throws UnsupportedOperationException{
               default -> throw new IllegalStateException("Unexpected role: " + simpleMsg.getRole());
             };
         messages.add(message);
-      }
-      else {
-        throw new UnsupportedOperationException("Currently MultiChatMessage type not supported by convenience API");
+      } else {
+        throw new UnsupportedOperationException(
+            "Currently MultiChatMessage type not supported by convenience API");
       }
     }
 

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

@@ -52,7 +52,7 @@ public class OrchestrationClient {
         ModuleResultsOutputUnmaskingInner.class,
         JacksonMixins.ModuleResultsOutputUnmaskingInnerMixIn.class);
 
-    var module =
+    final var module =
         new SimpleModule()
             .addDeserializer(
                 ChatMessagesInner.class,

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

@@ -3,34 +3,36 @@
 import com.fasterxml.jackson.annotation.JsonSubTypes;
 import com.fasterxml.jackson.core.JsonParser;
 import com.fasterxml.jackson.databind.DeserializationContext;
+import com.fasterxml.jackson.databind.JsonDeserializer;
 import com.fasterxml.jackson.databind.JsonMappingException;
-import com.fasterxml.jackson.databind.deser.std.StdDeserializer;
 import java.io.IOException;
 import java.util.ArrayList;
 import java.util.List;
+import javax.annotation.Nonnull;
 
 /**
  * Handles polymorphic deserialization for a base class or interface.
  *
- * <p>This deserializer attempts to deserialize JSON into a subtype of a base class or interface.
- * Subtypes are either discovered using the {@link JsonSubTypes} annotation or provided explicitly.
- * If deserialization fails for all candidates, a {@link JsonMappingException} is thrown with
- * suppressed exceptions.
+ * <p>This deserializer uses a fallback strategy by attempting to deserialize the JSON into known
+ * subtypes. Subtypes are either discovered using the {@link JsonSubTypes} annotation or provided
+ * explicitly. If deserialization fails for all candidates, a {@link JsonMappingException} is thrown
+ * with suppressed exceptions.
  *
  * @param <T> The base type for deserialization.
  */
-public class PolymorphicFallbackDeserializer<T> extends StdDeserializer<T> {
+public class PolymorphicFallbackDeserializer<T> extends JsonDeserializer<T> {
 
-  private final List<Class<? extends T>> candidates;
+  @Nonnull private final List<Class<? extends T>> candidates;
+  Class<T> baseClass;
 
   /**
    * Constructs the deserializer using the {@link JsonSubTypes} annotation.
    *
    * @param baseClass The base class or interface to be resolved.
    * @throws IllegalStateException If no subtypes are found.
    */
-  protected PolymorphicFallbackDeserializer(Class<T> baseClass) {
-    super(baseClass);
+  protected PolymorphicFallbackDeserializer(@Nonnull final Class<T> baseClass) {
+    this.baseClass = baseClass;
 
     final var subTypes = baseClass.getAnnotation(JsonSubTypes.class);
     if (subTypes == null || subTypes.value().length == 0) {
@@ -50,8 +52,8 @@ protected PolymorphicFallbackDeserializer(Class<T> baseClass) {
    * @param candidates A list of candidate classes to try deserialization.
    */
   protected PolymorphicFallbackDeserializer(
-      Class<T> baseClass, List<Class<? extends T>> candidates) {
-    super(baseClass);
+      @Nonnull final Class<T> baseClass, @Nonnull final List<Class<? extends T>> candidates) {
+    this.baseClass = baseClass;
     this.candidates = candidates;
   }
 
@@ -62,17 +64,20 @@ protected PolymorphicFallbackDeserializer(
    * @param deserializationContext The deserialization context.
    * @return The deserialized object of a matching candidate type.
    * @throws JsonMappingException If deserialization fails for all candidates.
-   * @throws IOException If an I/O error occurs.
+   * @throws IOException If json content cannot be consumed.
    */
+  @Nonnull
   @Override
-  public T deserialize(JsonParser jsonParser, DeserializationContext deserializationContext)
+  public T deserialize(
+      @Nonnull final JsonParser jsonParser,
+      @Nonnull final DeserializationContext deserializationContext)
       throws IOException {
 
     final var root = jsonParser.readValueAsTree();
     final var throwable =
         JsonMappingException.from(
             jsonParser,
-            "PolymorphicFallbackDeserializer failed to deserialize " + handledType().getName());
+            "PolymorphicFallbackDeserializer failed to deserialize " + this.baseClass.getName());
 
     for (final var candidate : candidates) {
       try {

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

@@ -130,7 +130,7 @@ void testGrounding() {
     assertThat(groundingData.get("grounding_result").toString())
         .startsWith("Joule is the AI copilot that truly understands your business.");
     assertThat(result.getModuleResults().getGrounding().getMessage()).isEqualTo("grounding result");
-    assertThat(((ChatMessage)result.getModuleResults().getTemplating().get(0)).getContent())
+    assertThat(((ChatMessage) result.getModuleResults().getTemplating().get(0)).getContent())
         .startsWith(
             "What does Joule do? Use the following information as additional context: Joule is the AI copilot that truly understands your business.");
     assertThat(llmChoice.getMessage().getContent())
@@ -578,7 +578,7 @@ void streamChatCompletionDeltas() throws IOException {
         assertThat(message0.get("content")).isEqualTo("");
         final var templating = deltaList.get(0).getModuleResults().getTemplating();
         assertThat(templating).hasSize(1);
-        
+
         final var templateItem = (ChatMessage) templating.get(0);
         assertThat(templateItem.getRole()).isEqualTo("user");
         assertThat(templateItem.getContent())