fix: revert hook data to resolve bytecode incompatibility (#1613)

* revert v2

Signed-off-by: christian.lutnik <[email protected]>

* remove vmlens to speed up the build

Signed-off-by: christian.lutnik <[email protected]>

---------

Signed-off-by: christian.lutnik <[email protected]>

Author: chrfwow

pom.xml

@@ -348,7 +348,7 @@
             </activation>
             <build>
                 <plugins>
-                    <plugin>
+                    <!--<plugin>
                         <groupId>com.vmlens</groupId>
                         <artifactId>vmlens-maven-plugin</artifactId>
                         <version>1.2.14</version>
@@ -363,7 +363,7 @@
                                 </configuration>
                             </execution>
                         </executions>
-                    </plugin>
+                    </plugin>-->
                     <plugin>
                         <artifactId>maven-dependency-plugin</artifactId>
                         <version>3.8.1</version>

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

@@ -1,13 +1,32 @@
 package dev.openfeature.sdk;
 
-import dev.openfeature.sdk.HookContextWithoutData.HookContextWithoutDataBuilder;
+import lombok.Builder;
+import lombok.NonNull;
+import lombok.Value;
+import lombok.With;
 
 /**
- * A interface to hold immutable context that {@link Hook} instances use.
+ * A data class to hold immutable context that {@link Hook} instances use.
+ *
+ * @param <T> the type for the flag being evaluated
  */
-public interface HookContext<T> {
+@Value
+@Builder
+@With
+public class HookContext<T> {
+    @NonNull String flagKey;
+
+    @NonNull FlagValueType type;
+
+    @NonNull T defaultValue;
+
+    @NonNull EvaluationContext ctx;
+
+    ClientMetadata clientMetadata;
+    Metadata providerMetadata;
+
     /**
-     * Builds a {@link HookContextWithoutData} instances from request data.
+     * Builds a {@link HookContext} instances from request data.
      *
      * @param key              feature flag key
      * @param type             flag value type
@@ -17,39 +36,21 @@ public interface HookContext<T> {
      * @param defaultValue     Fallback value
      * @param <T>              type that the flag is evaluating against
      * @return resulting context for hook
-     * @deprecated this should not be instantiated outside the SDK anymore
      */
-    @Deprecated
-    static <T> HookContext<T> from(
+    public static <T> HookContext<T> from(
             String key,
             FlagValueType type,
             ClientMetadata clientMetadata,
             Metadata providerMetadata,
             EvaluationContext ctx,
             T defaultValue) {
-        return new HookContextWithoutData<>(key, type, defaultValue, ctx, clientMetadata, providerMetadata);
-    }
-
-    /**
-     * Returns a builder for our default HookContext object.
-     */
-    static <T> HookContextWithoutDataBuilder<T> builder() {
-        return HookContextWithoutData.<T>builder();
-    }
-
-    String getFlagKey();
-
-    FlagValueType getType();
-
-    T getDefaultValue();
-
-    EvaluationContext getCtx();
-
-    ClientMetadata getClientMetadata();
-
-    Metadata getProviderMetadata();
-
-    default HookData getHookData() {
-        return null;
+        return HookContext.<T>builder()
+                .flagKey(key)
+                .type(type)
+                .clientMetadata(clientMetadata)
+                .providerMetadata(providerMetadata)
+                .ctx(ctx)
+                .defaultValue(defaultValue)
+                .build();
     }
 }