Revert "feat(rc): Web support for ABT & Rollouts (#9293)"

This reverts commit 6abe529.

Author: hsubox76

.changeset/wild-snakes-bathe.md

@@ -41,7 +41,6 @@ export function fetchConfig(remoteConfig: RemoteConfig): Promise<void>;
 export interface FetchResponse {
     config?: FirebaseRemoteConfigObject;
     eTag?: string;
-    experiments?: FirebaseExperimentDescription[];
     status: number;
     templateVersion?: number;
 }
@@ -52,22 +51,6 @@ export type FetchStatus = 'no-fetch-yet' | 'success' | 'failure' | 'throttle';
 // @public
 export type FetchType = 'BASE' | 'REALTIME';
 
-// @public
-export interface FirebaseExperimentDescription {
-    // (undocumented)
-    affectedParameterKeys?: string[];
-    // (undocumented)
-    experimentId: string;
-    // (undocumented)
-    experimentStartTime: string;
-    // (undocumented)
-    timeToLiveMillis: string;
-    // (undocumented)
-    triggerTimeoutMillis: string;
-    // (undocumented)
-    variantId: string;
-}
-
 // @public
 export interface FirebaseRemoteConfigObject {
     // (undocumented)

common/api-review/remote-config.api.md

@@ -661,8 +661,6 @@ toc:
     path: /docs/reference/js/remote-config.customsignals.md
   - title: FetchResponse
     path: /docs/reference/js/remote-config.fetchresponse.md
-  - title: FirebaseExperimentDescription
-    path: /docs/reference/js/remote-config.firebaseexperimentdescription.md
   - title: FirebaseRemoteConfigObject
     path: /docs/reference/js/remote-config.firebaseremoteconfigobject.md
   - title: RemoteConfig

docs-devsite/_toc.yaml

@@ -26,7 +26,6 @@ export interface FetchResponse
 |  --- | --- | --- |
 |  [config](./remote-config.fetchresponse.md#fetchresponseconfig) | [FirebaseRemoteConfigObject](./remote-config.firebaseremoteconfigobject.md#firebaseremoteconfigobject_interface) | Defines the map of parameters returned as "entries" in the fetch response body.<p>Only defined for 200 responses. |
 |  [eTag](./remote-config.fetchresponse.md#fetchresponseetag) | string | Defines the ETag response header value.<p>Only defined for 200 and 304 responses. |
-|  [experiments](./remote-config.fetchresponse.md#fetchresponseexperiments) | [FirebaseExperimentDescription](./remote-config.firebaseexperimentdescription.md#firebaseexperimentdescription_interface)<!-- -->\[\] | Metadata for A/B testing and Remote Config Rollout experiments. |
 |  [status](./remote-config.fetchresponse.md#fetchresponsestatus) | number | The HTTP status, which is useful for differentiating success responses with data from those without.<p>The Remote Config client is modeled after the native <code>Fetch</code> interface, so HTTP status is first-class.<p>Disambiguation: the fetch response returns a legacy "state" value that is redundant with the HTTP status code. The former is normalized into the latter. |
 |  [templateVersion](./remote-config.fetchresponse.md#fetchresponsetemplateversion) | number | The version number of the config template fetched from the server. |
 
@@ -54,18 +53,6 @@ Defines the ETag response header value.
 eTag?: string;
 ```
 
-## FetchResponse.experiments
-
-Metadata for A/B testing and Remote Config Rollout experiments.
-
-Only defined for 200 responses.
-
-<b>Signature:</b>
-
-```typescript
-experiments?: FirebaseExperimentDescription[];
-```
-
 ## FetchResponse.status
 
 The HTTP status, which is useful for differentiating success responses with data from those without.

docs-devsite/remote-config.fetchresponse.md

@@ -42,7 +42,6 @@ The Firebase Remote Config Web SDK. This SDK does not work in a Node.js environm
 |  [ConfigUpdateObserver](./remote-config.configupdateobserver.md#configupdateobserver_interface) | Observer interface for receiving real-time Remote Config update notifications.<!-- -->NOTE: Although an <code>complete</code> callback can be provided, it will never be called because the ConfigUpdate stream is never-ending. |
 |  [CustomSignals](./remote-config.customsignals.md#customsignals_interface) | Defines the type for representing custom signals and their values.<p>The values in CustomSignals must be one of the following types:<ul> <li><code>string</code> <li><code>number</code> <li><code>null</code> </ul> |
 |  [FetchResponse](./remote-config.fetchresponse.md#fetchresponse_interface) | Defines a successful response (200 or 304).<p>Modeled after the native <code>Response</code> interface, but simplified for Remote Config's use case. |
-|  [FirebaseExperimentDescription](./remote-config.firebaseexperimentdescription.md#firebaseexperimentdescription_interface) | Defines experiment and variant attached to a config parameter. |
 |  [FirebaseRemoteConfigObject](./remote-config.firebaseremoteconfigobject.md#firebaseremoteconfigobject_interface) | Defines a self-descriptive reference for config key-value pairs. |
 |  [RemoteConfig](./remote-config.remoteconfig.md#remoteconfig_interface) | The Firebase Remote Config service interface. |
 |  [RemoteConfigOptions](./remote-config.remoteconfigoptions.md#remoteconfigoptions_interface) | Options for Remote Config initialization. |

docs-devsite/remote-config.firebaseexperimentdescription.md

@@ -36,7 +36,6 @@ import { ERROR_FACTORY, ErrorCode, hasErrorCode } from './errors';
 import { RemoteConfig as RemoteConfigImpl } from './remote_config';
 import { Value as ValueImpl } from './value';
 import { LogLevel as FirebaseLogLevel } from '@firebase/logger';
-import { Experiment } from './abt/experiment';
 
 /**
  *
@@ -111,19 +110,12 @@ export async function activate(remoteConfig: RemoteConfig): Promise<boolean> {
     // config.
     return false;
   }
-  const experiment = new Experiment(rc);
-  const updateActiveExperiments = lastSuccessfulFetchResponse.experiments
-    ? experiment.updateActiveExperiments(
-        lastSuccessfulFetchResponse.experiments
-      )
-    : Promise.resolve();
   await Promise.all([
     rc._storageCache.setActiveConfig(lastSuccessfulFetchResponse.config),
     rc._storage.setActiveConfigEtag(lastSuccessfulFetchResponse.eTag),
     rc._storage.setActiveConfigTemplateVersion(
       lastSuccessfulFetchResponse.templateVersion
-    ),
-    updateActiveExperiments
+    )
   ]);
   return true;
 }

docs-devsite/remote-config.md

@@ -18,8 +18,7 @@
 import {
   CustomSignals,
   FetchResponse,
-  FirebaseRemoteConfigObject,
-  FirebaseExperimentDescription
+  FirebaseRemoteConfigObject
 } from '../public_types';
 import {
   RemoteConfigFetchClient,
@@ -144,7 +143,6 @@ export class RestClient implements RemoteConfigFetchClient {
     let config: FirebaseRemoteConfigObject | undefined;
     let state: string | undefined;
     let templateVersion: number | undefined;
-    let experiments: FirebaseExperimentDescription[] | undefined;
 
     // JSON parsing throws SyntaxError if the response body isn't a JSON string.
     // Requesting application/json and checking for a 200 ensures there's JSON data.
@@ -160,7 +158,6 @@ export class RestClient implements RemoteConfigFetchClient {
       config = responseBody['entries'];
       state = responseBody['state'];
       templateVersion = responseBody['templateVersion'];
-      experiments = responseBody['experimentDescriptions'];
     }
 
     // Normalizes based on legacy state.
@@ -171,7 +168,6 @@ export class RestClient implements RemoteConfigFetchClient {
     } else if (state === 'NO_TEMPLATE' || state === 'EMPTY_CONFIG') {
       // These cases can be fixed remotely, so normalize to safe value.
       config = {};
-      experiments = [];
     }
 
     // Normalize to exception-based control flow for non-success cases.
@@ -184,6 +180,6 @@ export class RestClient implements RemoteConfigFetchClient {
       });
     }
 
-    return { status, eTag: responseEtag, config, templateVersion, experiments };
+    return { status, eTag: responseEtag, config, templateVersion };
   }
 }

packages/remote-config/src/abt/experiment.ts

@@ -37,8 +37,7 @@ export const enum ErrorCode {
   CONFIG_UPDATE_STREAM_ERROR = 'stream-error',
   CONFIG_UPDATE_UNAVAILABLE = 'realtime-unavailable',
   CONFIG_UPDATE_MESSAGE_INVALID = 'update-message-invalid',
-  CONFIG_UPDATE_NOT_FETCHED = 'update-not-fetched',
-  ANALYTICS_UNAVAILABLE = 'analytics-unavailable'
+  CONFIG_UPDATE_NOT_FETCHED = 'update-not-fetched'
 }
 
 const ERROR_DESCRIPTION_MAP: { readonly [key in ErrorCode]: string } = {
@@ -85,9 +84,7 @@ const ERROR_DESCRIPTION_MAP: { readonly [key in ErrorCode]: string } = {
   [ErrorCode.CONFIG_UPDATE_MESSAGE_INVALID]:
     'The stream invalidation message was unparsable: {$originalErrorMessage}',
   [ErrorCode.CONFIG_UPDATE_NOT_FETCHED]:
-    'Unable to fetch the latest config: {$originalErrorMessage}',
-  [ErrorCode.ANALYTICS_UNAVAILABLE]:
-    'Connection to Firebase Analytics failed: {$originalErrorMessage}'
+    'Unable to fetch the latest config: {$originalErrorMessage}'
 };
 
 // Note this is effectively a type system binding a code to params. This approach overlaps with the
@@ -111,7 +108,6 @@ interface ErrorParams {
   [ErrorCode.CONFIG_UPDATE_UNAVAILABLE]: { originalErrorMessage: string };
   [ErrorCode.CONFIG_UPDATE_MESSAGE_INVALID]: { originalErrorMessage: string };
   [ErrorCode.CONFIG_UPDATE_NOT_FETCHED]: { originalErrorMessage: string };
-  [ErrorCode.ANALYTICS_UNAVAILABLE]: { originalErrorMessage: string };
 }
 
 export const ERROR_FACTORY = new ErrorFactory<ErrorCode, ErrorParams>(