fixup: dont merge ctx uneccessarily

Signed-off-by: Todd Baert <[email protected]>

Author: toddbaert

packages/server/src/client/internal/open-feature-client.ts

@@ -228,11 +228,10 @@ export class OpenFeatureClient implements Client {
     try {
       this.shortCircuitIfNotReady();
 
-      const mergedContext = this.mergeContexts(context);
-      Object.freeze(mergedContext);
-
       if (typeof this._provider.track === 'function') {
-        return this._provider.track?.(occurrenceKey, mergedContext, occurrenceDetails);
+        // freeze the merged context
+        const frozenContext = Object.freeze(this.mergeContexts(context));
+        return this._provider.track?.(occurrenceKey, frozenContext, occurrenceDetails);
       } else {
         this._logger.debug('Provider does not support the track function; will no-op.');
       }

packages/web/src/client/internal/open-feature-client.ts

@@ -186,14 +186,12 @@ export class OpenFeatureClient implements Client {
     try {
       this.shortCircuitIfNotReady();
 
-      // copy and freeze the context
-      const context = {
-        ...OpenFeature.getContext(this?.options?.domain),
-      };
-      Object.freeze(context);
-
       if (typeof this._provider.track === 'function') {
-        return this._provider.track?.(occurrenceKey, context, occurrenceDetails);
+        // copy and freeze the context
+        const frozenContext = Object.freeze({
+          ...OpenFeature.getContext(this?.options?.domain),
+        });
+        return this._provider.track?.(occurrenceKey, frozenContext, occurrenceDetails);
       } else {
         this._logger.debug('Provider does not support the track function; will no-op.');
       }

packages/web/src/provider/in-memory-provider/in-memory-provider.ts

@@ -38,8 +38,7 @@ export class InMemoryProvider implements Provider {
    */
   async putConfiguration(flagConfiguration: FlagConfiguration) {
     try {
-      const flagsChanged = Object.entries(flagConfiguration)
-        .filter(([key, value]) => this._flagConfiguration[key] !== value)
+      const flagsChanged = Object.entries({...flagConfiguration, ...this._flagConfiguration})
         .map(([key]) => key);
 
       this._flagConfiguration = { ...flagConfiguration };