[Bug]: Optional Fields As per Specifications are not handled in Task Class Constructor

[priyanshus1]

What happened?

mvn: a2a-java-sdk-client
class name: io.a2a.spec.Task

In the A2A specification document: https://a2a-protocol.org/latest/specification/#61-task-object
The task object contains optional field: artificat & history. If I take the ? suffix in my understading.
However the class constructor for the Task does not follow it.

Even in the https://github.com/a2aproject/A2A/blob/main/specification/json/a2a.json Task spec does not make artifact and history as mandatory.

Expectation:
The Task class should provide an additional constructor to handle the optional fields as per the specification

Relevant log output

Code of Conduct

• I agree to follow this project's Code of Conduct

[kabir]

I'm not sure I follow.

https://github.com/a2aproject/a2a-java/blob/main/spec/src/main/java/io/a2a/spec/Task.java#L41-L58

    public Task(@JsonProperty("id") String id, @JsonProperty("contextId") String contextId, @JsonProperty("status") TaskStatus status,
                @JsonProperty("artifacts") List<Artifact> artifacts, @JsonProperty("history") List<Message> history,
                @JsonProperty("metadata") Map<String, Object> metadata, @JsonProperty("kind") String kind) {
        Assert.checkNotNullParam("id", id);
        Assert.checkNotNullParam("contextId", contextId);
        Assert.checkNotNullParam("status", status);
        Assert.checkNotNullParam("kind", kind);
        if (! kind.equals(TASK)) {
            throw new IllegalArgumentException("Invalid Task");
        }
        this.id = id;
        this.contextId = contextId;
        this.status = status;
        this.artifacts = artifacts;
        this.history = history;
        this.metadata = metadata;
        this.kind = kind;
    }

artifacts and history are there and are not mandatory. What exactly differs from the spec in your opinion?

[kabir]

BTW the Task class has a Builder which allows you to set just what you want without overriding the constructor.

[priyanshus1]

Yes, that's right I missed the builder pattern.