chore: update OpenFeature SDKs to support hook data (#1392)

Signed-off-by: Lukas Reining <[email protected]>
Signed-off-by: Thomas Poignant <[email protected]>

Author: lukas-reining

libs/hooks/open-telemetry/src/lib/traces/tracing-hook.spec.ts

@@ -1,4 +1,5 @@
 import type { EvaluationDetails, HookContext } from '@openfeature/server-sdk';
+import { MapHookData } from '@openfeature/server-sdk';
 
 const addEvent = jest.fn();
 const recordException = jest.fn();
@@ -30,6 +31,7 @@ describe('OpenTelemetry Hooks', () => {
     defaultValue: true,
     flagValueType: 'boolean',
     logger: console,
+    hookData: new MapHookData(),
   };
 
   let tracingHook: TracingHook;

libs/providers/multi-provider-web/src/lib/multi-provider-web.spec.ts

@@ -10,6 +10,7 @@ import type {
   ProviderMetadata,
   TrackingEventDetails,
 } from '@openfeature/web-sdk';
+import { MapHookData } from '@openfeature/web-sdk';
 import {
   DefaultLogger,
   ErrorCode,
@@ -21,7 +22,6 @@ import {
 import { FirstMatchStrategy } from './strategies/FirstMatchStrategy';
 import { FirstSuccessfulStrategy } from './strategies/FirstSuccessfulStrategy';
 import { ComparisonStrategy } from './strategies/ComparisonStrategy';
-import type { BaseEvaluationStrategy } from './strategies/BaseEvaluationStrategy';
 
 class TestProvider implements Provider {
   public metadata: ProviderMetadata = {
@@ -64,6 +64,7 @@ const callBeforeHook = (
     clientMetadata: {} as any,
     providerMetadata: {} as any,
     logger: logger,
+    hookData: new MapHookData(),
   };
   multi.hooks[0].before?.(hookContext);
 };
@@ -271,6 +272,7 @@ describe('MultiProvider', () => {
           clientMetadata: {} as any,
           providerMetadata: {} as any,
           logger: logger,
+          hookData: new MapHookData(),
         };
 
         provider1.hooks = [
@@ -350,6 +352,7 @@ describe('MultiProvider', () => {
           clientMetadata: {} as any,
           providerMetadata: {} as any,
           logger: logger,
+          hookData: new MapHookData(),
         };
 
         provider1.hooks = [

libs/providers/multi-provider/src/lib/multi-provider.spec.ts

@@ -4,11 +4,13 @@ import type {
   FlagValue,
   FlagValueType,
   Hook,
+  HookContext,
   Logger,
   Provider,
   ProviderMetadata,
   TrackingEventDetails,
 } from '@openfeature/server-sdk';
+import { MapHookData } from '@openfeature/server-sdk';
 import {
   DefaultLogger,
   ErrorCode,
@@ -63,6 +65,7 @@ const callBeforeHook = async (
     clientMetadata: {} as any,
     providerMetadata: {} as any,
     logger: logger,
+    hookData: new MapHookData(),
   };
   await multi.hooks[0].before?.(hookContext);
 };
@@ -262,14 +265,15 @@ describe('MultiProvider', () => {
         const context = {
           test: true,
         };
-        const hookContext = {
+        const hookContext: HookContext = {
           context: context,
           flagKey: 'flag',
           flagValueType: 'boolean' as any,
           defaultValue: false,
           clientMetadata: {} as any,
           providerMetadata: {} as any,
           logger: logger,
+          hookData: new MapHookData(),
         };
 
         const weakMap = new WeakMap();
@@ -371,6 +375,7 @@ describe('MultiProvider', () => {
           clientMetadata: {} as any,
           providerMetadata: {} as any,
           logger: logger,
+          hookData: new MapHookData(),
         };
 
         const weakMap = new WeakMap();

package-lock.json

@@ -57,9 +57,9 @@
     "@nx/rollup": "20.3.1",
     "@nx/web": "20.3.1",
     "@nx/workspace": "20.3.1",
-    "@openfeature/core": "^1.6.0",
-    "@openfeature/server-sdk": "^1.17.1",
-    "@openfeature/web-sdk": "^1.4.1",
+    "@openfeature/core": "^1.9.1",
+    "@openfeature/server-sdk": "^1.19.0",
+    "@openfeature/web-sdk": "^1.6.2",
     "@opentelemetry/sdk-metrics": "^1.15.0",
     "@swc-node/register": "~1.10.0",
     "@swc/cli": "~0.7.0",