Revert "Fix: Javadoc issues in io.a2a.spec package"

This reverts commit 71ff747.

Author: kabir

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

@@ -14,6 +14,7 @@
  *   <li>{@link A2AClientInvalidStateError} - Invalid client state errors</li>
  *   <li>{@link A2AClientInvalidArgsError} - Invalid argument errors</li>
  * </ul>
+ *
  * @see A2AClientException for general client exceptions
  * @see A2AClientHTTPError for HTTP-specific errors
  * @see A2AClientJSONError for JSON-specific errors

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

@@ -13,6 +13,7 @@
  *   <li>HTTP errors ({@link A2AClientHTTPError})</li>
  *   <li>JSON parsing errors ({@link A2AClientJSONError})</li>
  * </ul>
+ *
  * @see A2AException for the base exception class
  * @see A2AServerException for server-side errors
  * @see A2AClientError for more specific client errors

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

@@ -12,6 +12,7 @@
  *   <li>{@link A2AServerException} - Server-side failures</li>
  *   <li>{@link A2AClientException} - Client-side failures</li>
  * </ul>
+ *
  * @see A2AServerException for server-side errors
  * @see A2AClientException for client-side errors
  * @see <a href="https://a2a-protocol.org/latest/">A2A Protocol Specification</a>

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

@@ -13,6 +13,7 @@
  *   <li>Resource exhaustion or limits exceeded</li>
  *   <li>Configuration errors</li>
  * </ul>
+ *
  * @see A2AException for the base exception class
  * @see A2AClientException for client-side errors
  */

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

@@ -149,9 +149,9 @@ public String getType() {
     }
 
     /**
-     * Creates a new builder for {@link APIKeySecurityScheme}.
+     * Create a new Builder
      *
-     * @return a new builder instance
+     * @return the builder
      */
     public static Builder builder() {
         return new Builder();

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

@@ -35,15 +35,15 @@ public record AgentCapabilities(boolean streaming, boolean pushNotifications, bo
                                 List<AgentExtension> extensions) {
 
     /**
-     * Creates a new builder for {@link AgentCapabilities}.
+     * Create a new Builder
      *
-     * @return a new builder instance
+     * @return the builder
      */
     public static Builder builder() {
         return new Builder();
     }
     /**
-     * Builder for constructing immutable {@code AgentCapabilities} instances.
+     * Builder for constructing immutable {@link AgentCapabilities} instances.
      * <p>
      * The Builder pattern provides a fluent API for setting capability flags.
      * All capabilities default to false if not explicitly set.

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

@@ -65,6 +65,24 @@ public record AgentCard(
 
     /**
      * Compact constructor that validates required fields and sets defaults.
+     *
+     * @param name the name parameter (see class-level JavaDoc)
+     * @param description the description parameter (see class-level JavaDoc)
+     * @param provider the provider parameter (see class-level JavaDoc)
+     * @param version the version parameter (see class-level JavaDoc)
+     * @param documentationUrl the documentationUrl parameter (see class-level JavaDoc)
+     * @param capabilities the capabilities parameter (see class-level JavaDoc)
+     * @param defaultInputModes the defaultInputModes parameter (see class-level JavaDoc)
+     * @param defaultOutputModes the defaultOutputModes parameter (see class-level JavaDoc)
+     * @param skills the skills parameter (see class-level JavaDoc)
+     * @param supportsExtendedAgentCard the supportsExtendedAgentCard parameter (see class-level JavaDoc)
+     * @param securitySchemes the securitySchemes parameter (see class-level JavaDoc)
+     * @param security the security parameter (see class-level JavaDoc)
+     * @param iconUrl the iconUrl parameter (see class-level JavaDoc)
+     * @param supportedInterfaces the supportedInterfaces parameter (see class-level JavaDoc)
+     * @param protocolVersion the protocolVersion parameter (see class-level JavaDoc)
+     * @param signatures the signatures parameter (see class-level JavaDoc)
+     * @throws IllegalArgumentException if any required field is null
      */
     public AgentCard {
         Assert.checkNotNullParam("capabilities", capabilities);
@@ -105,7 +123,7 @@ public static Builder builder(AgentCard card) {
 
 
     /**
-     * Builder for constructing immutable {@code AgentCard} instances.
+     * Builder for constructing immutable {@link AgentCard} instances.
      * <p>
      * The Builder pattern is used to enforce immutability of AgentCard objects while providing
      * a fluent API for setting required and optional fields. This approach ensures that once

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

@@ -37,6 +37,9 @@ public record AgentCardSignature(Map<String, Object> header, @SerializedName("pr
     /**
      * Compact constructor that validates required fields.
      *
+     * @param header the header parameter (see class-level JavaDoc)
+     * @param protectedHeader the protectedHeader parameter (see class-level JavaDoc)
+     * @param signature the signature parameter (see class-level JavaDoc)
      * @throws IllegalArgumentException if protectedHeader or signature is null
      */
     public AgentCardSignature {
@@ -45,9 +48,9 @@ public record AgentCardSignature(Map<String, Object> header, @SerializedName("pr
     }
 
     /**
-     * Creates a new builder for {@link AgentCardSignature}.
+     * Create a new Builder
      *
-     * @return a new builder instance
+     * @return the builder
      */
     public static Builder builder() {
         return new Builder();

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

@@ -28,16 +28,20 @@ public record AgentExtension (String description, Map<String, Object> params, bo
     /**
      * Compact constructor that validates required fields.
      *
+     * @param description the description parameter (see class-level JavaDoc)
+     * @param params the params parameter (see class-level JavaDoc)
+     * @param required the required parameter (see class-level JavaDoc)
+     * @param uri the uri parameter (see class-level JavaDoc)
      * @throws IllegalArgumentException if uri is null
      */
     public AgentExtension {
         Assert.checkNotNullParam("uri", uri);
     }
 
     /**
-     * Creates a new builder for {@link AgentExtension}.
+     * Create a new Builder
      *
-     * @return a new builder instance
+     * @return the builder
      */
     public static Builder builder() {
         return new Builder();

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

@@ -29,6 +29,9 @@ public record AgentInterface(String protocolBinding, String url, String tenant)
     /**
      * Compact constructor that validates required fields.
      *
+     * @param protocolBinding the protocolBinding parameter (see class-level JavaDoc)
+     * @param url the url parameter (see class-level JavaDoc)
+     * @param tenant the tenant parameter (see class-level JavaDoc)
      * @throws IllegalArgumentException if protocolBinding or url is null
      */
     public AgentInterface   {