Move FlagEvalHook construction inside try/catch block

FlagEvalHook references FlagEvalMetrics in its field declaration.
On JVMs that eagerly verify field types during class loading,
constructing FlagEvalHook outside the try/catch could throw
NoClassDefFoundError if OTel classes failed to load. Moving it
inside the try block ensures both metrics and hook are null-safe
when OTel is absent.

Author: typotter

products/feature-flagging/feature-flagging-api/src/main/java/datadog/trace/api/openfeature/Provider.java

@@ -43,13 +43,15 @@ public Provider(final Options options) {
     this.options = options;
     this.evaluator = evaluator;
     FlagEvalMetrics metrics = null;
+    FlagEvalHook hook = null;
     try {
       metrics = new FlagEvalMetrics();
+      hook = new FlagEvalHook(metrics);
     } catch (NoClassDefFoundError | Exception e) {
       // OTel classes not on classpath — metrics disabled
     }
     this.flagEvalMetrics = metrics;
-    this.flagEvalHook = new FlagEvalHook(metrics);
+    this.flagEvalHook = hook;
   }
 
   @Override