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

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

@@ -665,8 +665,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>(