Improve docs and confirm checkstyle

Author: rpanackal

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

@@ -26,9 +26,17 @@
 @NoArgsConstructor
 @Accessors(fluent = true)
 public class AzureContentFilter implements ContentFilter {
+
+  /* The moderation policy for hate content. */
   @Nullable AzureModerationPolicy hate;
+
+  /* The moderation policy for self-harm content. */
   @Nullable AzureModerationPolicy selfHarm;
+
+  /* The moderation policy for sexual content. */
   @Nullable AzureModerationPolicy sexual;
+
+  /* The moderation policy for violence content. */
   @Nullable AzureModerationPolicy violence;
 
   /**

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

@@ -6,7 +6,9 @@
 import com.sap.ai.sdk.orchestration.client.model.MaskingModuleConfig;
 import com.sap.ai.sdk.orchestration.client.model.OutputFilteringConfig;
 import com.sap.ai.sdk.orchestration.client.model.TemplatingModuleConfig;
+import java.util.ArrayList;
 import java.util.Arrays;
+import java.util.Objects;
 import javax.annotation.Nonnull;
 import javax.annotation.Nullable;
 import lombok.AccessLevel;
@@ -98,13 +100,18 @@ public OrchestrationModuleConfig withMaskingConfig(
    */
   @Nonnull
   public OrchestrationModuleConfig withInputFiltering(
-      @Nonnull final ContentFilter... contentFilters) {
+      @Nonnull final ContentFilter contentFilter, @Nonnull final ContentFilter... contentFilters) {
 
-    var filterConfigs = Arrays.stream(contentFilters).map(ContentFilter::createConfig).toList();
+    final var allFilters = new ArrayList<ContentFilter>();
+    allFilters.add(contentFilter);
+    allFilters.addAll(Arrays.asList(contentFilters));
 
-    var inputFilter = new InputFilteringConfig().filters(filterConfigs);
+    final var filterConfigs =
+        allFilters.stream().filter(Objects::nonNull).map(ContentFilter::createConfig).toList();
 
-    var newFilteringConfig =
+    final var inputFilter = new InputFilteringConfig().filters(filterConfigs);
+
+    final var newFilteringConfig =
         new FilteringModuleConfig()
             .input(inputFilter)
             .output(this.filteringConfig != null ? this.filteringConfig.getOutput() : null);
@@ -124,12 +131,18 @@ public OrchestrationModuleConfig withInputFiltering(
    */
   @Nonnull
   public OrchestrationModuleConfig withOutputFiltering(
-      @Nonnull final ContentFilter... contentFilters) {
+      @Nonnull final ContentFilter contentFilter, @Nonnull final ContentFilter... contentFilters) {
+
+    final var allFilters = new ArrayList<ContentFilter>();
+    allFilters.add(contentFilter);
+    allFilters.addAll(Arrays.asList(contentFilters));
+
+    final var filterConfigs =
+        allFilters.stream().filter(Objects::nonNull).map(ContentFilter::createConfig).toList();
 
-    var filterConfigs = Arrays.stream(contentFilters).map(ContentFilter::createConfig).toList();
-    var outputFilter = new OutputFilteringConfig().filters(filterConfigs);
+    final var outputFilter = new OutputFilteringConfig().filters(filterConfigs);
 
-    var newFilteringConfig =
+    final var newFilteringConfig =
         new FilteringModuleConfig()
             .output(outputFilter)
             .input(this.filteringConfig != null ? this.filteringConfig.getInput() : null);

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

@@ -393,4 +393,31 @@ void testEmptyChoicesResponse() {
 
     assertThat(result.getContent()).isEmpty();
   }
+
+  @Test
+  void testStackingInputAndOutputFilter() {
+    final var filter =
+        new AzureContentFilter()
+            .hate(ALLOW_SAFE_LOW_MEDIUM)
+            .selfHarm(ALLOW_SAFE_LOW_MEDIUM)
+            .sexual(ALLOW_SAFE_LOW_MEDIUM)
+            .violence(ALLOW_SAFE_LOW_MEDIUM);
+
+    final var configWithInputFirst = config.withInputFiltering(filter).withOutputFiltering(filter);
+    assertThat(configWithInputFirst.getFilteringConfig()).isNotNull();
+    assertThat(configWithInputFirst.getFilteringConfig().getInput()).isNotNull();
+
+    final var configWithOutputFirst = config.withOutputFiltering(filter).withInputFiltering(filter);
+    assertThat(configWithOutputFirst.getFilteringConfig()).isNotNull();
+    assertThat(configWithOutputFirst.getFilteringConfig().getOutput()).isNotNull();
+  }
+
+  @Test
+  void testThrowOnEmptyFilterConfig() {
+    final var filter = new AzureContentFilter();
+
+    assertThatThrownBy(filter::createConfig)
+        .isInstanceOf(IllegalArgumentException.class)
+        .hasMessage("At least one filter moderation policy must be set");
+  }
 }