fix: address PR review comments for WebMultiProvider naming and documentation

Signed-off-by: Jonathan Norris <[email protected]>

Author: jonathannorris

packages/web/README.md

@@ -155,9 +155,9 @@ The Multi-Provider is a powerful tool for performing migrations between flag pro
 - **Multiple Data Sources**: The Multi-Provider allows you to seamlessly combine many sources of flagging data, such as environment variables, local files, database values and SaaS hosted feature management systems.
 
 ```ts
-import { MultiProvider } from '@openfeature/web-sdk';
+import { WebMultiProvider } from '@openfeature/web-sdk';
 
-const multiProvider = new MultiProvider([
+const multiProvider = new WebMultiProvider([
   { provider: new ProviderA() },
   { provider: new ProviderB() }
 ]);
@@ -176,12 +176,12 @@ The Multi-Provider comes with three strategies out of the box:
 
 - **FirstMatchStrategy** (default): Evaluates all providers in order and returns the first successful result. Providers that indicate FLAG_NOT_FOUND error will be skipped and the next provider will be evaluated.
 - **FirstSuccessfulStrategy**: Evaluates all providers in order and returns the first successful result. Any error will cause that provider to be skipped.
-- **ComparisonStrategy**: Evaluates all providers in parallel. If every provider returns a successful result with the same value, then that result is returned. Otherwise, the result returned by the configured "fallback provider" will be used.
+- **ComparisonStrategy**: Evaluates all providers sequentially. If every provider returns a successful result with the same value, then that result is returned. Otherwise, the result returned by the configured "fallback provider" will be used.
 
 ```ts
-import { MultiProvider, FirstSuccessfulStrategy } from '@openfeature/web-sdk';
+import { WebMultiProvider, FirstSuccessfulStrategy } from '@openfeature/web-sdk';
 
-const multiProvider = new MultiProvider(
+const multiProvider = new WebMultiProvider(
   [
     { provider: new ProviderA() },
     { provider: new ProviderB() }

packages/web/src/provider/multi-provider/README.md

@@ -56,10 +56,10 @@ const multiProvider = new WebMultiProvider(
 
 The Multi-Provider comes with three strategies out of the box:
 
-- `FirstMatchStrategy` (default): Evaluates all providers in order and returns the first successful result. Providers that indicate FLAG_NOT_FOUND error will be skipped and the next provider will be evaluated. Any other error will cause the operation to fail and the set of errors to be thrown. 
+- `FirstMatchStrategy` (default): Evaluates all providers in order and returns the first successful result. Providers that indicate FLAG_NOT_FOUND error will be skipped and the next provider will be evaluated. Any other error will cause the operation to fail and the set of errors to be thrown.
 - `FirstSuccessfulStrategy`: Evaluates all providers in order and returns the first successful result. Any error will cause that provider to be skipped.
 If no successful result is returned, the set of errors will be thrown.
-- `ComparisonStrategy`: Evaluates all providers in parallel. If every provider returns a successful result with the same value, then that result is returned.
+- `ComparisonStrategy`: Evaluates all providers sequentially. If every provider returns a successful result with the same value, then that result is returned.
 Otherwise, the result returned by the configured "fallback provider" will be used. When values do not agree, an optional callback will be executed to notify
 you of the mismatch. This can be useful when migrating between providers that are expected to contain identical configuration. You can easily spot mismatches
 in configuration without affecting flag behaviour.

packages/web/src/provider/multi-provider/status-tracker.ts

@@ -55,9 +55,7 @@ export class StatusTracker {
     this.providerStatuses[name] = status;
     const newStatus = this.getStatusFromProviderStatuses();
     if (currentStatus !== newStatus) {
-      if (newStatus === ProviderStatus.FATAL) {
-        this.events.emit(ProviderEvents.Error, details);
-      } else if (newStatus === ProviderStatus.ERROR) {
+      if (newStatus === ProviderStatus.FATAL || newStatus === ProviderStatus.ERROR) {
         this.events.emit(ProviderEvents.Error, details);
       } else if (newStatus === ProviderStatus.STALE) {
         this.events.emit(ProviderEvents.Stale, details);