Track whether initialization has run in wrapper

Author: MattIPv4

packages/shared/src/open-feature.ts

@@ -27,6 +27,8 @@ type AnyProviderStatus = ClientProviderStatus | ServerProviderStatus;
  */
 export class ProviderWrapper<P extends CommonProvider<AnyProviderStatus>, S extends AnyProviderStatus> {
   private _pendingContextChanges = 0;
+  private _initializing = false;
+  private _initialized = false;
 
   constructor(
     private _provider: P,
@@ -49,6 +51,8 @@ export class ProviderWrapper<P extends CommonProvider<AnyProviderStatus>, S exte
         this._status = _statusEnumType.ERROR as S;
       }
     });
+
+    this._initialized = !(typeof _provider.initialize === 'function');
   }
 
   get provider(): P {
@@ -67,6 +71,22 @@ export class ProviderWrapper<P extends CommonProvider<AnyProviderStatus>, S exte
     this._status = status;
   }
 
+  get initializing() {
+    return this._initializing;
+  }
+
+  set initializing(initializing: boolean) {
+    this._initializing = initializing;
+  }
+
+  get initialized() {
+    return this._initialized;
+  }
+
+  set initialized(initialized: boolean) {
+    this._initialized = initialized;
+  }
+
   get allContextChangesSettled() {
     return this._pendingContextChanges === 0;
   }
@@ -219,10 +239,13 @@ export abstract class OpenFeatureCommonAPI<
 
   protected initializeProviderForDomain(wrapper: ProviderWrapper<P, AnyProviderStatus>, domain?: string): Promise<void> | void {
     if (typeof wrapper.provider.initialize !== 'function'
-      || this.allProviders.includes(wrapper.provider)) {
+      || this.allProviders.includes(wrapper.provider)
+      || wrapper.initializing
+      || wrapper.initialized) {
       return;
     }
 
+    wrapper.initializing = true;
     return wrapper.provider
       .initialize(domain ? (this._domainScopedContext.get(domain) ?? this._context) : this._context)
       .then(() => {
@@ -256,6 +279,10 @@ export abstract class OpenFeatureCommonAPI<
         });
         // rethrow after emitting error events, so that public methods can control error handling
         throw error;
+      })
+      .finally(() => {
+        wrapper.initialized = true;
+        wrapper.initializing = false;
       });
     }
 
@@ -291,6 +318,7 @@ export abstract class OpenFeatureCommonAPI<
       this._statusEnumType,
     );
 
+    // TODO: Don't initialize (nor set READY) if user requests delayed initialization until first context change
     // initialize the provider if it implements "initialize" and it's not already registered
     const initializationPromise = this.initializeProviderForDomain(wrappedProvider, domain);
     if (!initializationPromise) {