Deduplicate complex event payloads

Author: MattIPv4

packages/shared/src/open-feature.ts

@@ -251,11 +251,13 @@ export abstract class OpenFeatureCommonAPI<
       .initialize(domain ? (this._domainScopedContext.get(domain) ?? this._context) : this._context)
       .then(() => {
         wrapper.status = this._statusEnumType.READY;
+        const payload = { clientName: domain, domain, providerName: wrapper.provider.metadata.name };
+
         // fetch the most recent event emitters, some may have been added during init
         this.getAssociatedEventEmitters(domain).forEach((emitter) => {
-          emitter?.emit(AllProviderEvents.Ready, { clientName: domain, domain, providerName: wrapper.provider.metadata.name });
+          emitter?.emit(AllProviderEvents.Ready, { ...payload });
         });
-        this._apiEmitter?.emit(AllProviderEvents.Ready, { clientName: domain, domain, providerName: wrapper.provider.metadata.name });
+        this._apiEmitter?.emit(AllProviderEvents.Ready, { ...payload });
       })
       .catch((error) => {
         // if this is a fatal error, transition to FATAL status
@@ -264,20 +266,19 @@ export abstract class OpenFeatureCommonAPI<
         } else {
           wrapper.status = this._statusEnumType.ERROR;
         }
-        this.getAssociatedEventEmitters(domain).forEach((emitter) => {
-          emitter?.emit(AllProviderEvents.Error, {
-            clientName: domain,
-            domain,
-            providerName: wrapper.provider.metadata.name,
-            message: error?.message,
-          });
-        });
-        this._apiEmitter?.emit(AllProviderEvents.Error, {
+        const payload = {
           clientName: domain,
           domain,
           providerName: wrapper.provider.metadata.name,
           message: error?.message,
+        };
+
+        // fetch the most recent event emitters, some may have been added during init
+        this.getAssociatedEventEmitters(domain).forEach((emitter) => {
+          emitter?.emit(AllProviderEvents.Error, { ...payload });
         });
+        this._apiEmitter?.emit(AllProviderEvents.Error, { ...payload });
+
         // rethrow after emitting error events, so that public methods can control error handling
         throw error;
       })

packages/web/src/open-feature.ts

@@ -282,21 +282,17 @@ export class OpenFeatureAPI
         : this._defaultProvider;
       if (wrapper) {
         wrapper.status = this._statusEnumType.ERROR;
-        const providerName = wrapper.provider?.metadata?.name || 'unnamed-provider';
-        this.getAssociatedEventEmitters(domain).forEach((emitter) => {
-          emitter?.emit(ProviderEvents.Error, {
-            clientName: domain,
-            domain,
-            providerName,
-            message: `Error validating context during setProvider: ${validateContextError instanceof Error ? validateContextError.message : String(validateContextError)}`,
-          });
-        });
-        this._apiEmitter?.emit(ProviderEvents.Error, {
+        const payload = {
           clientName: domain,
           domain,
-          providerName,
+          providerName: wrapper.provider?.metadata?.name || 'unnamed-provider',
           message: `Error validating context during setProvider: ${validateContextError instanceof Error ? validateContextError.message : String(validateContextError)}`,
+        };
+
+        this.getAssociatedEventEmitters(domain).forEach((emitter) => {
+          emitter?.emit(ProviderEvents.Error, { ...payload });
         });
+        this._apiEmitter?.emit(ProviderEvents.Error, { ...payload });
         this._logger.error('Error validating context during setProvider:', validateContextError);
       }
     }