Fix Cohere and remove lazy list initialization

Author: g-despot

src/main/java/io/weaviate/client6/v1/api/collections/generative/AnthropicGenerative.java

@@ -58,7 +58,7 @@ public static class Builder implements ObjectBuilder<AnthropicGenerative> {
     private Integer maxTokens;
     private Float temperature;
     private String baseUrl;
-    private List<String> stopSequences;
+    private final List<String> stopSequences = new ArrayList<>();
 
     /** Base URL of the generative provider. */
     public Builder baseUrl(String baseUrl) {
@@ -101,9 +101,6 @@ public Builder stopSequences(String... stopSequences) {
      * Set tokens which should signal the model to stop generating further output.
      */
     public Builder stopSequences(List<String> stopSequences) {
-      if (this.stopSequences == null) {
-        this.stopSequences = new ArrayList<>();
-      }
       this.stopSequences.addAll(stopSequences);
       return this;
     }
@@ -202,9 +199,9 @@ public static class Builder implements ObjectBuilder<AnthropicGenerative.Provide
       private String model;
       private Integer maxTokens;
       private Float temperature;
-      private List<String> stopSequences;
-      private List<String> images;
-      private List<String> imageProperties;
+      private final List<String> stopSequences = new ArrayList<>();
+      private final List<String> images = new ArrayList<>();
+      private final List<String> imageProperties = new ArrayList<>();
 
       /** Base URL of the generative provider. */
       public Builder baseUrl(String baseUrl) {
@@ -247,9 +244,6 @@ public Builder stopSequences(String... stopSequences) {
        * Set tokens which should signal the model to stop generating further output.
        */
       public Builder stopSequences(List<String> stopSequences) {
-        if (this.stopSequences == null) {
-          this.stopSequences = new ArrayList<>();
-        }
         this.stopSequences.addAll(stopSequences);
         return this;
       }
@@ -259,9 +253,6 @@ public Builder images(String... images) {
       }
 
       public Builder images(List<String> images) {
-        if (this.images == null) {
-          this.images = new ArrayList<>();
-        }
         this.images.addAll(images);
         return this;
       }
@@ -271,9 +262,6 @@ public Builder imageProperties(String... imageProperties) {
       }
 
       public Builder imageProperties(List<String> imageProperties) {
-        if (this.imageProperties == null) {
-          this.imageProperties = new ArrayList<>();
-        }
         this.imageProperties.addAll(imageProperties);
         return this;
       }

src/main/java/io/weaviate/client6/v1/api/collections/generative/AwsGenerative.java

@@ -90,7 +90,7 @@ public Builder(Service service, String region) {
     private Integer maxTokensToSample;
     private Float topP;
     private Integer topK;
-    private List<String> stopSequences;
+    private final List<String> stopSequences = new ArrayList<>();
 
     /** Base URL of the generative provider. */
     protected Builder endpoint(String endpoint) {
@@ -153,9 +153,6 @@ public Builder stopSequences(String... stopSequences) {
 
     /** Stop sequences for the model. */
     public Builder stopSequences(List<String> stopSequences) {
-      if (this.stopSequences == null) {
-        this.stopSequences = new ArrayList<>();
-      }
       this.stopSequences.addAll(stopSequences);
       return this;
     }

src/main/java/io/weaviate/client6/v1/api/collections/generative/CohereGenerative.java

@@ -20,7 +20,10 @@ public record CohereGenerative(
     @SerializedName("maxTokens") Integer maxTokens,
     @SerializedName("temperature") Float temperature,
     @SerializedName("returnLikelihoods") String returnLikelihoodsProperty,
-    @SerializedName("stopSequences") List<String> stopSequences) implements Generative {
+    @SerializedName("stopSequences") List<String> stopSequences,
+    @SerializedName("P") Float topP,
+    @SerializedName("presencePenalty") Float presencePenalty,
+    @SerializedName("frequencyPenalty") Float frequencyPenalty) implements Generative {
 
   @Override
   public Kind _kind() {
@@ -48,7 +51,10 @@ public CohereGenerative(Builder builder) {
         builder.maxTokens,
         builder.temperature,
         builder.returnLikelihoodsProperty,
-        builder.stopSequences);
+        builder.stopSequences,
+        builder.topP,
+        builder.presencePenalty,
+        builder.frequencyPenalty);
   }
 
   public static class Builder implements ObjectBuilder<CohereGenerative> {
@@ -58,7 +64,10 @@ public static class Builder implements ObjectBuilder<CohereGenerative> {
     private Integer maxTokens;
     private Float temperature;
     private String returnLikelihoodsProperty;
-    private List<String> stopSequences;
+    private final List<String> stopSequences = new ArrayList<>();
+    private Float topP;
+    private Float presencePenalty;
+    private Float frequencyPenalty;
 
     /** Base URL of the generative provider. */
     public Builder baseUrl(String baseUrl) {
@@ -72,6 +81,12 @@ public Builder topK(int topK) {
       return this;
     }
 
+    /** Top P value for nucleus sampling. */
+    public Builder topP(float topP) {
+      this.topP = topP;
+      return this;
+    }
+
     /** Select generative model. */
     public Builder model(String model) {
       this.model = model;
@@ -100,9 +115,6 @@ public Builder stopSequences(String... stopSequences) {
      * Set tokens which should signal the model to stop generating further output.
      */
     public Builder stopSequences(List<String> stopSequences) {
-      if (this.stopSequences == null) {
-        this.stopSequences = new ArrayList<>();
-      }
       this.stopSequences.addAll(stopSequences);
       return this;
     }
@@ -116,6 +128,16 @@ public Builder temperature(float temperature) {
       return this;
     }
 
+    public Builder presencePenalty(float presencePenalty) {
+      this.presencePenalty = presencePenalty;
+      return this;
+    }
+
+    public Builder frequencyPenalty(float frequencyPenalty) {
+      this.frequencyPenalty = frequencyPenalty;
+      return this;
+    }
+
     @Override
     public CohereGenerative build() {
       return new CohereGenerative(this);
@@ -211,7 +233,7 @@ public static class Builder implements ObjectBuilder<CohereGenerative.Provider>
       private Float temperature;
       private Float frequencyPenalty;
       private Float presencePenalty;
-      private List<String> stopSequences;
+      private final List<String> stopSequences = new ArrayList<>();
 
       /** Base URL of the generative provider. */
       public Builder baseUrl(String baseUrl) {
@@ -265,9 +287,6 @@ public Builder stopSequences(String... stopSequences) {
        * Set tokens which should signal the model to stop generating further output.
        */
       public Builder stopSequences(List<String> stopSequences) {
-        if (this.stopSequences == null) {
-          this.stopSequences = new ArrayList<>();
-        }
         this.stopSequences.addAll(stopSequences);
         return this;
       }