update to use SuperBuilder for MultiProviderEvaluation

Signed-off-by: Parth Suthar <parth.suthar@dynatrace.com>

Author: suthar26

src/main/java/dev/openfeature/sdk/ProviderEvaluation.java

@@ -4,14 +4,15 @@
 import lombok.Builder;
 import lombok.Data;
 import lombok.NoArgsConstructor;
+import lombok.experimental.SuperBuilder;
 
 /**
  * Contains information about how the a flag was evaluated, including the resolved value.
  *
  * @param <T> the type of the flag being evaluated.
  */
 @Data
-@Builder
+@SuperBuilder
 @NoArgsConstructor
 @AllArgsConstructor
 public class ProviderEvaluation<T> implements BaseEvaluation<T> {

src/main/java/dev/openfeature/sdk/multiprovider/FirstMatchStrategy.java

@@ -64,7 +64,7 @@ public <T> ProviderEvaluation<T> evaluate(
         }
 
         // All providers either threw or returned FLAG_NOT_FOUND
-        return MultiProviderEvaluation.<T>multiProviderBuilder()
+        return MultiProviderEvaluation.<T>builder()
                 .errorMessage(ProviderError.buildAggregateMessage("Flag not found in any provider", collectedErrors))
                 .errorCode(FLAG_NOT_FOUND)
                 .providerErrors(collectedErrors)

src/main/java/dev/openfeature/sdk/multiprovider/FirstSuccessfulStrategy.java

@@ -49,7 +49,7 @@ public <T> ProviderEvaluation<T> evaluate(
             }
         }
 
-        return MultiProviderEvaluation.<T>multiProviderBuilder()
+        return MultiProviderEvaluation.<T>builder()
                 .errorMessage(
                         ProviderError.buildAggregateMessage("No provider successfully responded", collectedErrors))
                 .errorCode(ErrorCode.GENERAL)

src/main/java/dev/openfeature/sdk/multiprovider/MultiProviderEvaluation.java

@@ -1,10 +1,11 @@
 package dev.openfeature.sdk.multiprovider;
 
-import dev.openfeature.sdk.ErrorCode;
-import dev.openfeature.sdk.ImmutableMetadata;
 import dev.openfeature.sdk.ProviderEvaluation;
 import java.util.Collections;
 import java.util.List;
+import lombok.Builder;
+import lombok.Getter;
+import lombok.experimental.SuperBuilder;
 
 /**
  * A {@link ProviderEvaluation} subtype returned by multi-provider strategies that carries
@@ -28,96 +29,16 @@
  *
  * @param <T> the type of the flag being evaluated
  */
+@Getter
+@SuperBuilder
 public class MultiProviderEvaluation<T> extends ProviderEvaluation<T> {
 
-    private final List<ProviderError> providerErrors;
-
-    private MultiProviderEvaluation(
-            T value,
-            String variant,
-            String reason,
-            ErrorCode errorCode,
-            String errorMessage,
-            ImmutableMetadata flagMetadata,
-            List<ProviderError> providerErrors) {
-        super(value, variant, reason, errorCode, errorMessage, flagMetadata);
-        this.providerErrors =
-                providerErrors != null ? Collections.unmodifiableList(providerErrors) : Collections.emptyList();
-    }
-
     /**
-     * Returns the per-provider error details.
+     * Per-provider error details.
      *
      * <p>Each entry describes why a specific provider failed during multi-provider evaluation.
-     *
-     * @return an unmodifiable list of per-provider errors, never {@code null}
-     */
-    public List<ProviderError> getProviderErrors() {
-        return providerErrors;
-    }
-
-    /**
-     * Create a new builder for {@link MultiProviderEvaluation}.
-     *
-     * @param <T> the flag value type
-     * @return a new builder
+     * Defaults to an empty list when not set.
      */
-    public static <T> Builder<T> multiProviderBuilder() {
-        return new Builder<>();
-    }
-
-    /**
-     * Builder for {@link MultiProviderEvaluation}.
-     *
-     * @param <T> the flag value type
-     */
-    public static class Builder<T> {
-        private T value;
-        private String variant;
-        private String reason;
-        private ErrorCode errorCode;
-        private String errorMessage;
-        private ImmutableMetadata flagMetadata;
-        private List<ProviderError> providerErrors;
-
-        public Builder<T> value(T value) {
-            this.value = value;
-            return this;
-        }
-
-        public Builder<T> variant(String variant) {
-            this.variant = variant;
-            return this;
-        }
-
-        public Builder<T> reason(String reason) {
-            this.reason = reason;
-            return this;
-        }
-
-        public Builder<T> errorCode(ErrorCode errorCode) {
-            this.errorCode = errorCode;
-            return this;
-        }
-
-        public Builder<T> errorMessage(String errorMessage) {
-            this.errorMessage = errorMessage;
-            return this;
-        }
-
-        public Builder<T> flagMetadata(ImmutableMetadata flagMetadata) {
-            this.flagMetadata = flagMetadata;
-            return this;
-        }
-
-        public Builder<T> providerErrors(List<ProviderError> providerErrors) {
-            this.providerErrors = providerErrors;
-            return this;
-        }
-
-        public MultiProviderEvaluation<T> build() {
-            return new MultiProviderEvaluation<>(
-                    value, variant, reason, errorCode, errorMessage, flagMetadata, providerErrors);
-        }
-    }
+    @Builder.Default
+    private List<ProviderError> providerErrors = Collections.emptyList();
 }