@firebase/remote-config 0.8.3 → 0.8.4-eap-crashlytics.558ee841d

package/dist/esm/src/client/realtime_handler.d.ts

@@ -94,6 +94,9 @@ export declare class RealtimeHandler {
      * A key is considered changed if it's new, removed, or has a different value.
      */
     private getChangedParams;
+    private areExperimentsEqual;
+    /** Creates a map where the key is the config key and the value is the experiment description. */
+    private createExperimentsMap;
     private fetchLatestConfig;
     private autoFetch;
     /**

package/dist/index.cjs.js

@@ -9,7 +9,7 @@ var logger = require('@firebase/logger');
 require('@firebase/installations');
 
 const name = "@firebase/remote-config";
-const version = "0.8.3";
+const version = "0.8.4-eap-crashlytics.558ee841d";
 
 /**
  * @license
@@ -303,9 +303,7 @@ async function activate(remoteConfig) {
         return false;
     }
     const experiment = new Experiment(rc);
-    const updateActiveExperiments = lastSuccessfulFetchResponse.experiments
-        ? experiment.updateActiveExperiments(lastSuccessfulFetchResponse.experiments)
-        : Promise.resolve();
+    const updateActiveExperiments = experiment.updateActiveExperiments(lastSuccessfulFetchResponse.experiments || []);
     await Promise.all([
         rc._storageCache.setActiveConfig(lastSuccessfulFetchResponse.config),
         rc._storage.setActiveConfigEtag(lastSuccessfulFetchResponse.eTag),
@@ -1552,6 +1550,7 @@ const NO_FAILED_REALTIME_STREAMS = 0;
 const REALTIME_DISABLED_KEY = 'featureDisabled';
 const REALTIME_RETRY_INTERVAL = 'retryIntervalSeconds';
 const TEMPLATE_VERSION_KEY = 'latestTemplateVersionNumber';
+const ROLLOUT_ID_PREFIX = '_exp_rollout';
 class RealtimeHandler {
     constructor(firebaseInstallations, storage, sdkVersion, namespace, projectId, apiKey, appId, logger, storageCache, cachingClient) {
         this.firebaseInstallations = firebaseInstallations;
@@ -1769,14 +1768,29 @@ class RealtimeHandler {
      * Compares two configuration objects and returns a set of keys that have changed.
      * A key is considered changed if it's new, removed, or has a different value.
      */
-    getChangedParams(newConfig, oldConfig) {
+    getChangedParams(newConfig, oldConfig, newFetchResponse, oldFetchResponse) {
         const changedKeys = new Set();
         const newKeys = new Set(Object.keys(newConfig || {}));
         const oldKeys = new Set(Object.keys(oldConfig || {}));
+        const newExperiments = newFetchResponse.experiments || [];
+        const oldExperiments = oldFetchResponse?.experiments || [];
+        const newExperimentsMap = this.createExperimentsMap(newExperiments);
+        const oldExperimentsMap = this.createExperimentsMap(oldExperiments);
         for (const key of newKeys) {
             if (!oldKeys.has(key) || newConfig[key] !== oldConfig[key]) {
                 changedKeys.add(key);
             }
+            if (newExperimentsMap.has(key) !== oldExperimentsMap.has(key)) {
+                changedKeys.add(key);
+                continue;
+            }
+            const newExperiment = newExperimentsMap.get(key);
+            const oldExperiment = oldExperimentsMap.get(key);
+            if (newExperiment &&
+                oldExperiment &&
+                !this.areExperimentsEqual(newExperiment, oldExperiment)) {
+                changedKeys.add(key);
+            }
         }
         for (const key of oldKeys) {
             if (!newKeys.has(key)) {
@@ -1785,6 +1799,26 @@ class RealtimeHandler {
         }
         return changedKeys;
     }
+    areExperimentsEqual(newExperiment, oldExperiment) {
+        return (newExperiment.experimentId === oldExperiment.experimentId &&
+            newExperiment.variantId === oldExperiment.variantId &&
+            newExperiment.timeToLiveMillis === oldExperiment.timeToLiveMillis &&
+            newExperiment.triggerTimeoutMillis === oldExperiment.triggerTimeoutMillis);
+    }
+    /** Creates a map where the key is the config key and the value is the experiment description. */
+    createExperimentsMap(experimentDescriptions) {
+        const experimentsMap = new Map();
+        for (const experimentDescription of experimentDescriptions) {
+            if (!experimentDescription.affectedParameterKeys ||
+                experimentDescription.experimentId.startsWith(ROLLOUT_ID_PREFIX)) {
+                continue;
+            }
+            for (const key of experimentDescription.affectedParameterKeys) {
+                experimentsMap.set(key, experimentDescription);
+            }
+        }
+        return experimentsMap;
+    }
     async fetchLatestConfig(remainingAttempts, targetVersion) {
         const remainingAttemptsAfterFetch = remainingAttempts - 1;
         const currentAttempt = MAXIMUM_FETCH_ATTEMPTS - remainingAttemptsAfterFetch;
@@ -1801,6 +1835,7 @@ class RealtimeHandler {
                 fetchType: 'REALTIME',
                 fetchAttempt: currentAttempt
             };
+            const lastFetchResponse = await this.storage.getLastSuccessfulFetchResponse();
             const fetchResponse = await this.cachingClient.fetch(fetchRequest);
             let activatedConfigs = await this.storage.getActiveConfig();
             if (!this.fetchResponseIsUpToDate(fetchResponse, targetVersion)) {
@@ -1817,7 +1852,7 @@ class RealtimeHandler {
             if (activatedConfigs == null) {
                 activatedConfigs = {};
             }
-            const updatedKeys = this.getChangedParams(fetchResponse.config, activatedConfigs);
+            const updatedKeys = this.getChangedParams(fetchResponse.config, activatedConfigs, fetchResponse, lastFetchResponse);
             if (updatedKeys.size === 0) {
                 this.logger.debug('Config was fetched, but no params changed.');
                 return;