Improve the code quality of the Registry class

Signed-off-by: AayushSaini101 <[email protected]>

Author: AayushSaini101

src/main/java/land/oras/Registry.java

@@ -172,41 +172,20 @@ public void deleteBlob(ContainerRef containerRef) {
         }
         handleError(response);
     }
-
-    /**
-     * Upload an ORAS artifact
-     * @param containerRef The container
-     * @param paths The paths
-     * @return The manifest
-     */
-    public Manifest pushArtifact(ContainerRef containerRef, Path... paths) {
-        return pushArtifact(containerRef, null, Annotations.empty(), Config.empty(), paths);
-    }
-
-    /**
-     * Upload an ORAS artifact
-     * @param containerRef The container
-     * @param artifactType The artifact type
-     * @param paths The paths
-     * @return The manifest
-     */
-    public Manifest pushArtifact(ContainerRef containerRef, String artifactType, Path... paths) {
-        return pushArtifact(containerRef, artifactType, Annotations.empty(), Config.empty(), paths);
-    }
-
-    /**
-     * Upload an ORAS artifact
-     * @param containerRef The container
-     * @param artifactType The artifact type
-     * @param annotations The annotations
-     * @param paths The paths
+      /**
+     * Push an ORAS artifact using the Builder pattern.
+     * @param builder The PushArtifactBuilder containing all configuration
      * @return The manifest
      */
-    public Manifest pushArtifact(
-            ContainerRef containerRef, String artifactType, Annotations annotations, Path... paths) {
-        return pushArtifact(containerRef, artifactType, annotations, Config.empty(), paths);
+    public Manifest pushArtifact(PushArtifactBuilder builder) {
+        return pushArtifact(
+                builder.containerRef,
+                builder.artifactType,
+                builder.annotations,
+                builder.config,
+                builder.paths
+        );
     }
-
     /**
      * Download an ORAS artifact
      * @param containerRef The container
@@ -238,6 +217,7 @@ public void pullArtifact(ContainerRef containerRef, Path path, boolean overwrite
         }
     }
 
+
     /**
      * Upload an ORAS artifact
      * @param containerRef The container
@@ -300,6 +280,46 @@ public Manifest pushArtifact(
         LOG.debug("Manifest pushed to: {}", location);
         return manifest;
     }
+     /**
+     * Builder class for configuring artifact uploads.
+     */
+    public static class PushArtifactBuilder {
+        private final ContainerRef containerRef; // Required parameter
+        private String artifactType = null;      // Optional parameter
+        private Annotations annotations = Annotations.empty(); // Default value
+        private Config config = Config.empty();  // Default value
+        private Path[] paths;                    // Required parameter
+
+        // Constructor for required fields
+        public PushArtifactBuilder(ContainerRef containerRef, Path... paths) {
+            this.containerRef = containerRef;
+            this.paths = paths;
+        }
+
+        // Setter for artifact type
+        public PushArtifactBuilder withArtifactType(String artifactType) {
+            this.artifactType = artifactType;
+            return this;
+        }
+
+        // Setter for annotations
+        public PushArtifactBuilder withAnnotations(Annotations annotations) {
+            this.annotations = annotations;
+            return this;
+        }
+
+        // Setter for config
+        public PushArtifactBuilder withConfig(Config config) {
+            this.config = config;
+            return this;
+        }
+
+        // Build method (optional if you just want chaining)
+        public PushArtifactBuilder build() {
+            return this;
+        }
+    }
+
 
     /**
      * Push a blob from file

src/test/java/land/oras/RegistryTest.java

@@ -184,7 +184,7 @@ void testShouldPushAndPullMinimalArtifact() throws IOException {
         Files.writeString(file1, "foobar");
 
         // Upload
-        Manifest manifest = registry.pushArtifact(containerRef, file1);
+        Manifest manifest = registry.pushArtifact(new Registry.PushArtifactBuilder(containerRef, file1));
         assertEquals(1, manifest.getLayers().size());
 
         Layer layer = manifest.getLayers().get(0);
@@ -223,7 +223,7 @@ void testShouldPushCompressedDirectory() throws IOException {
         Files.writeString(file3, "barfoo");
 
         // Upload blob dir
-        Manifest manifest = registry.pushArtifact(containerRef, blobDir);
+        Manifest manifest = registry.pushArtifact(new Registry.PushArtifactBuilder(containerRef, blobDir));
         assertEquals(1, manifest.getLayers().size());
 
         Layer layer = manifest.getLayers().get(0);