Refactor Api, ApiClient, Utils, and MultipartBodyBuilder classes to add private constructors preventing instantiation. Enhance ApiRequest class with additional getter methods for request details, improving code readability. Remove unused JsonMap class.

Author: guptavishal-xm1

src/main/java/io/mochaapi/client/Api.java

@@ -24,6 +24,9 @@
  */
 public class Api {
 
+    /** Private constructor to prevent instantiation. */
+    private Api() {}
+    
     private static final HttpClientEngine DEFAULT_ENGINE = new DefaultHttpClientEngine();
     private static final Executor DEFAULT_EXECUTOR = createDefaultExecutor();
 

src/main/java/io/mochaapi/client/ApiClient.java

@@ -202,6 +202,9 @@ private ApiResponse applyResponseInterceptors(ApiResponse response) {
      * Builder class for creating configured ApiClient instances.
      */
     public static class Builder {
+        
+        /** Creates a new Builder instance. */
+        public Builder() {}
         private HttpClientEngine engine;
         private Executor executor;
         private Duration connectTimeout = Duration.ofSeconds(30);

src/main/java/io/mochaapi/client/ApiRequest.java

@@ -277,10 +277,39 @@ public void async(Consumer<ApiResponse> callback) {
     }
 
     // Getters for internal use
+    /**
+     * Gets the request URL.
+     * @return the request URL
+     */
     public String getUrl() { return url; }
+    
+    /**
+     * Gets the HTTP method.
+     * @return the HTTP method
+     */
     public String getMethod() { return method; }
+    
+    /**
+     * Gets the request headers.
+     * @return the request headers
+     */
     public Map<String, String> getHeaders() { return headers; }
+    
+    /**
+     * Gets the query parameters.
+     * @return the query parameters
+     */
     public Map<String, Object> getQueryParams() { return queryParams; }
+    
+    /**
+     * Gets the request body.
+     * @return the request body
+     */
     public Object getBody() { return body; }
+    
+    /**
+     * Checks if this is a multipart request.
+     * @return true if this is a multipart request
+     */
     public boolean isMultipart() { return isMultipart; }
 }

src/main/java/io/mochaapi/client/JsonMap …ain/java/io/mochaapi/client/JsonMap.javasrc/main/java/io/mochaapi/client/JsonMap renamed to src/main/java/io/mochaapi/client/JsonMap.java src/main/java/io/mochaapi/client/JsonMap renamed to src/main/java/io/mochaapi/client/JsonMap.java

@@ -14,8 +14,8 @@
  * ApiResponse response = Api.get("https://api.example.com/user").execute();
  * 
  * // Traditional approach (verbose with casting)
- * Map<String, Object> data = response.toMap();
- * String city = ((Map<String, Object>)((Map<String, Object>)data.get("user")).get("address")).get("city").toString();
+ * Map&lt;String, Object&gt; data = response.toMap();
+ * String city = ((Map&lt;String, Object&gt;)((Map&lt;String, Object&gt;)data.get("user")).get("address")).get("city").toString();
  * 
  * // JsonMap approach (clean chaining)
  * String city = response.toJsonMap().get("user").get("address").get("city").toString();
@@ -29,6 +29,7 @@
  */
 public class JsonMap extends HashMap<String, Object> {
 
+    /** Internal field to wrap primitive values for chaining. */
     private Object wrappedValue = null;
 
     /**

src/main/java/io/mochaapi/client/internal/Utils.java

@@ -14,6 +14,9 @@
  */
 public class Utils {
 
+    /** Private constructor to prevent instantiation. */
+    private Utils() {}
+    
     // Pattern to match dangerous URL schemes that could be used for open redirects
     private static final Pattern DANGEROUS_SCHEMES = Pattern.compile(
         "^(javascript|data|vbscript|file|ftp):", Pattern.CASE_INSENSITIVE);

src/main/java/io/mochaapi/client/multipart/MultipartBodyBuilder.java

@@ -17,6 +17,9 @@
  */
 public class MultipartBodyBuilder {
 
+    /** Private constructor to prevent instantiation. */
+    private MultipartBodyBuilder() {}
+    
     private static final String BOUNDARY_PREFIX = "----MochaAPI-";
     private static final String CRLF = "\r\n";
     private static final long DEFAULT_MAX_FILE_SIZE = 50 * 1024 * 1024; // 50MB default limit