Merge pull request #104 from fjuma/spec

Add validation for parts and historyLength

Author: maeste

src/main/java/io/a2a/spec/Artifact.java

@@ -18,6 +18,9 @@ public record Artifact(String artifactId, String name, String description, List<
     public Artifact {
         Assert.checkNotNullParam("artifactId", artifactId);
         Assert.checkNotNullParam("parts", parts);
+        if (parts.isEmpty()) {
+            throw new IllegalArgumentException("Parts cannot be empty");
+        }
     }
 
     public static class Builder {

src/main/java/io/a2a/spec/Message.java

@@ -44,6 +44,9 @@ public Message(@JsonProperty("role") Role role, @JsonProperty("parts") List<Part
                    @JsonProperty("kind") String kind) {
         Assert.checkNotNullParam("kind", kind);
         Assert.checkNotNullParam("parts", parts);
+        if (parts.isEmpty()) {
+            throw new IllegalArgumentException("Parts cannot be empty");
+        }
         Assert.checkNotNullParam("role", role);
         if (! kind.equals(MESSAGE)) {
             throw new IllegalArgumentException("Invalid Message");

src/main/java/io/a2a/spec/MessageSendConfiguration.java

@@ -20,6 +20,9 @@ public record MessageSendConfiguration(List<String> acceptedOutputModes, Integer
 
     public MessageSendConfiguration {
         Assert.checkNotNullParam("acceptedOutputModes", acceptedOutputModes);
+        if (historyLength != null && historyLength < 0) {
+            throw new IllegalArgumentException("Invalid history length");
+        }
     }
 
     public static class Builder {

src/main/java/io/a2a/spec/TaskQueryParams.java

@@ -19,6 +19,9 @@ public record TaskQueryParams(String id, Integer historyLength, Map<String, Obje
 
     public TaskQueryParams {
         Assert.checkNotNullParam("id", id);
+        if (historyLength != null && historyLength < 0) {
+            throw new IllegalArgumentException("Invalid history length");
+        }
     }
 
     public TaskQueryParams(String id) {