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

athiramanu · 2025-10-03T06:20:24Z

This change is part of the feature to support Web Experiments in Remote Config. Changes can be broken down as below:

Introduces the experiment details in the fetch response type
Extracts experiments received from the fetch response and saves in indexedDB
Sets the experiment and variant received as user property

Design doc (internal): go/experiments-web

* feat: Process experiment metadata in RC fetch response * [Fix] Storage cache is not updating when there are no experiments in response * Add result of running yarn docgen:all * Address review comments * Fix yarn format failures * yarn docgen changes added * Export firebaseExperimentDescription --------- Co-authored-by: Athira M <athiramanu@google.com>

* feat: Process experiment metadata in RC fetch response * [Fix] Storage cache is not updating when there are no experiments in response * Add result of running yarn docgen:all * feat: Process experiment metadata in RC fetch response * feat: Add ABT support for remote config * [Fix] Storage cache is not updating when there are no experiments in response * Merge conflict fix * Yarn format fix * Fix merge conflicts * Address review comments * Fix yarn format failures * yarn docgen changes added * Export firebaseExperimentDescription * Address review comments --------- Co-authored-by: Athira M <athiramanu@google.com>

* feat: Process experiment metadata in RC fetch response * feat: Add ABT support for remote config * feat: Integrate firebase internal analytics with ABT * [Fix] Storage cache is not updating when there are no experiments in response * feat: Process experiment metadata in RC fetch response * [Fix] Storage cache is not updating when there are no experiments in response * Add result of running yarn docgen:all * feat: Process experiment metadata in RC fetch response * feat: Add ABT support for remote config * [Fix] Storage cache is not updating when there are no experiments in response * Merge conflict fix * Yarn format fix * Fix merge conflicts * Integrate ABT with Firebase analytics to add experiment as UP * Fix yarn format errors * Address review comments * Fix yarn format failures * yarn docgen changes added * Export firebaseExperimentDescription * Address review comments * Address review comments * Add unit tests * Add error handling * Remove log --------- Co-authored-by: Athira M <athiramanu@google.com>

changeset-bot · 2025-10-03T06:20:28Z

🦋 Changeset detected

Latest commit: 2ef04a7

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages

Name	Type
@firebase/remote-config	Minor
firebase	Minor
@firebase/remote-config-compat	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

google-oss-bot · 2025-10-03T06:30:12Z

Size Report ¹

Affected Products

@firebase/remote-config
Type Base (8209266) Merge (e72fa8e) Diff
browser 39.0 kB 41.4 kB +2.40 kB (+6.1%)
main 40.2 kB 42.6 kB +2.40 kB (+6.0%)
module 39.0 kB 41.4 kB +2.40 kB (+6.1%)
bundle
Type Base (8209266) Merge (e72fa8e) Diff
remote-config (getAndFetch) 59.4 kB 60.9 kB +1.51 kB (+2.5%)
firebase
Type Base (8209266) Merge (e72fa8e) Diff
firebase-compat.js 809 kB 811 kB +1.43 kB (+0.2%)
firebase-remote-config-compat.js 38.7 kB 40.1 kB +1.43 kB (+3.7%)
firebase-remote-config.js 43.5 kB 45.0 kB +1.48 kB (+3.4%)

Test Logs

https://storage.googleapis.com/firebase-sdk-metric-reports/KLyyupqxvD.html

google-oss-bot · 2025-10-03T06:41:58Z

Size Analysis Report ¹

Affected Products

`@firebase/remote-config`

`activate`

Size

Type	Base (`8209266`)	Merge (`e72fa8e`)	Diff
size	23.5 kB	25.0 kB	+1.51 kB (+6.4%)
size-with-ext-deps	50.0 kB	51.5 kB	+1.51 kB (+3.0%)

Dependency

Type Base (8209266) Merge (e72fa8e) Diff

classes

12 dependencies

CachingClient
EventEmitter
InMemoryStorage
IndexedDbStorage
RealtimeHandler
RemoteConfig
RemoteConfigAbortSignal
RestClient
RetryingClient
Storage
StorageCache
VisibilityMonitor

13 dependencies

CachingClient
EventEmitter
Experiment
InMemoryStorage
IndexedDbStorage
RealtimeHandler
RemoteConfig
RemoteConfigAbortSignal
RestClient
RetryingClient
Storage
StorageCache
VisibilityMonitor

+ Experiment

ensureInitialized
Size
Type Base (8209266) Merge (e72fa8e) Diff
size 23.1 kB 23.5 kB +347 B (+1.5%)
size-with-ext-deps 49.6 kB 50.0 kB +347 B (+0.7%)

`fetchAndActivate`

Size

Type	Base (`8209266`)	Merge (`e72fa8e`)	Diff
size	24.2 kB	25.7 kB	+1.51 kB (+6.3%)
size-with-ext-deps	50.7 kB	52.2 kB	+1.51 kB (+3.0%)

Dependency

Type Base (8209266) Merge (e72fa8e) Diff

classes

12 dependencies

CachingClient
EventEmitter
InMemoryStorage
IndexedDbStorage
RealtimeHandler
RemoteConfig
RemoteConfigAbortSignal
RestClient
RetryingClient
Storage
StorageCache
VisibilityMonitor

13 dependencies

CachingClient
EventEmitter
Experiment
InMemoryStorage
IndexedDbStorage
RealtimeHandler
RemoteConfig
RemoteConfigAbortSignal
RestClient
RetryingClient
Storage
StorageCache
VisibilityMonitor

+ Experiment

fetchConfig
Size
Type Base (8209266) Merge (e72fa8e) Diff
size 23.7 kB 24.1 kB +347 B (+1.5%)
size-with-ext-deps 50.2 kB 50.6 kB +347 B (+0.7%)
getAll
Size
Type Base (8209266) Merge (e72fa8e) Diff
size 24.3 kB 24.7 kB +347 B (+1.4%)
size-with-ext-deps 50.8 kB 51.1 kB +347 B (+0.7%)
getBoolean
Size
Type Base (8209266) Merge (e72fa8e) Diff
size 24.2 kB 24.5 kB +347 B (+1.4%)
size-with-ext-deps 50.7 kB 51.0 kB +347 B (+0.7%)
getNumber
Size
Type Base (8209266) Merge (e72fa8e) Diff
size 24.2 kB 24.5 kB +347 B (+1.4%)
size-with-ext-deps 50.7 kB 51.0 kB +347 B (+0.7%)
getRemoteConfig
Size
Type Base (8209266) Merge (e72fa8e) Diff
size 23.9 kB 24.3 kB +347 B (+1.5%)
size-with-ext-deps 57.3 kB 57.7 kB +347 B (+0.6%)
getString
Size
Type Base (8209266) Merge (e72fa8e) Diff
size 24.2 kB 24.5 kB +347 B (+1.4%)
size-with-ext-deps 50.7 kB 51.0 kB +347 B (+0.7%)
getValue
Size
Type Base (8209266) Merge (e72fa8e) Diff
size 24.1 kB 24.5 kB +347 B (+1.4%)
size-with-ext-deps 50.6 kB 51.0 kB +347 B (+0.7%)
isSupported
Size
Type Base (8209266) Merge (e72fa8e) Diff
size 23.3 kB 23.6 kB +347 B (+1.5%)
size-with-ext-deps 49.7 kB 50.1 kB +347 B (+0.7%)
onConfigUpdate
Size
Type Base (8209266) Merge (e72fa8e) Diff
size 23.3 kB 23.6 kB +347 B (+1.5%)
size-with-ext-deps 49.7 kB 50.1 kB +347 B (+0.7%)
setCustomSignals
Size
Type Base (8209266) Merge (e72fa8e) Diff
size 23.6 kB 24.0 kB +347 B (+1.5%)
size-with-ext-deps 50.1 kB 50.5 kB +347 B (+0.7%)
setLogLevel
Size
Type Base (8209266) Merge (e72fa8e) Diff
size 23.3 kB 23.6 kB +347 B (+1.5%)
size-with-ext-deps 49.8 kB 50.1 kB +347 B (+0.7%)

Test Logs

https://storage.googleapis.com/firebase-sdk-metric-reports/21Y9G5ummL.html

packages/remote-config/src/public_types.ts

docs-devsite/remote-config.fetchresponse.md

jenh

Note: @nk-1983 also approved this CL, but it didn't get firebase-tw approval due to a group sync issue (should be resolved now!).

hsubox76

Add a changeset to ensure this will be published. Use yarn changeset and follow the dialog. It's fine if you mess up, you can edit the file after it's created. The bump should be minor. Manually add "firebase: minor" to the file after it's created - see https://g3doc.corp.google.com/firebase/jscore/g3doc/contributing/changeset.md?cl=head

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

* Bug fixes * yarn format fail * update event name * Ensure user property is parsed * Add firebase prefix to exp

github-actions · 2025-10-28T01:28:57Z

Changeset File Check ✅

No modified packages are missing from the changeset file.
No changeset formatting errors detected.

.changeset/wild-snakes-bathe.md

Review messages

hsubox76 · 2025-11-05T15:00:07Z

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

+        analytics.setUserProperties(customProperty);
+        analytics.logEvent(`set_firebase_experiment_state`);
+      } else {
+        this.logger.warn(`Analytics import failed`);


Shouldn't this throw the ANALYTICS_UNAVAILABLE error instead? Since "optional:true" is set, I don't think this will throw. If the user hasn't imported analytics you'll reach here instead. There should be a better message here. Have you tested this?

I remember we discussed whether we want to block fetch call if analytics import is not present and decided to log warning instead. Since we are setting all experiment user properties without doing a diff, there are no experiments getting missed. The ANALYTICS_UNAVAILABLE was added in case one of the analytics calls throw error.

I verified that the experiments are not lost.

What's the intention of this warning? To let the developer know they need to import analytics? In that case the message needs to be clearer and tell them what they need to do. I think we discussed earlier that this text was a placeholder that would be run by the tech writer. Right now the text makes it sound like something went wrong inside the SDK ("failed") and doesn't make it clear the developer needs to add an import statement. I mentioned that previous precedents for this were to either (1) provide specific instructions (such as "make sure you have import 'firebase/analytics' in your app code" or whatever wording the tech writer agrees on) or (2) to link to a devsite page that provides instructions (e.g. "Firebase Analytics was not imported. See https://www.docs//docs-about-this-feature for how to set up this feature").

I guess the message can read:
Analytics import failed. Verify if you have imported Firebase Analytics in your app code.

athiramanu and others added 3 commits September 27, 2025 16:42

athiramanu requested review from a team and erikeldridge as code owners October 3, 2025 06:20

athiramanu requested a review from nk-1983 October 3, 2025 06:47

nk-1983 reviewed Oct 3, 2025

View reviewed changes

packages/remote-config/src/public_types.ts Outdated Show resolved Hide resolved

nk-1983 reviewed Oct 3, 2025

View reviewed changes

docs-devsite/remote-config.fetchresponse.md Outdated Show resolved Hide resolved

Fix messages and docs

fa63f97

nk-1983 approved these changes Oct 7, 2025

View reviewed changes

athiramanu requested review from hsubox76, jenh, kjelko and nk-1983 October 8, 2025 04:48

jenh approved these changes Oct 8, 2025

View reviewed changes

hsubox76 reviewed Oct 9, 2025

View reviewed changes

packages/remote-config/src/abt/experiment.ts Outdated Show resolved Hide resolved

Update error messages

b5f96dd

athiramanu requested a review from ashish-kothari October 11, 2025 04:45

ashish-kothari approved these changes Oct 15, 2025

View reviewed changes

Fix feature defects if Web Experiments (#9327)

851a47f

* Bug fixes * yarn format fail * update event name * Ensure user property is parsed * Add firebase prefix to exp

athiramanu requested review from a team as code owners October 28, 2025 01:27

Add changeset

dc45a19

athiramanu force-pushed the web-experiment branch from ef61bbf to dc45a19 Compare October 28, 2025 01:32

Athira M added 2 commits October 28, 2025 09:23

Merge branch 'main' into web-experiment

34c5750

Update changeset

4aa9bd5

hsubox76 reviewed Oct 28, 2025

View reviewed changes

.changeset/wild-snakes-bathe.md Show resolved Hide resolved

Update changeset

2ef04a7

hsubox76 previously approved these changes Nov 5, 2025

View reviewed changes

hsubox76 reviewed Nov 5, 2025

View reviewed changes

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

Are you sure you want to change the base?

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

Conversation

athiramanu commented Oct 3, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

changeset-bot bot commented Oct 3, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🦋 Changeset detected

Uh oh!

google-oss-bot commented Oct 3, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Size Report 1

Affected Products

@firebase/remote-config

bundle

firebase

Test Logs

Uh oh!

google-oss-bot commented Oct 3, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Size Analysis Report 1

Affected Products

@firebase/remote-config

activate

ensureInitialized

fetchAndActivate

fetchConfig

getAll

getBoolean

getNumber

getRemoteConfig

getString

getValue

isSupported

onConfigUpdate

setCustomSignals

setLogLevel

Test Logs

Uh oh!

Uh oh!

Uh oh!

jenh left a comment

Choose a reason for hiding this comment

Uh oh!

hsubox76 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

github-actions bot commented Oct 28, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Changeset File Check ✅

Uh oh!

Uh oh!

hsubox76 Nov 5, 2025

Choose a reason for hiding this comment

Uh oh!

athiramanu Nov 5, 2025

Choose a reason for hiding this comment

Uh oh!

hsubox76 Nov 7, 2025

Choose a reason for hiding this comment

Uh oh!

nk-1983 Nov 7, 2025

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

athiramanu commented Oct 3, 2025 •

edited

Loading

changeset-bot bot commented Oct 3, 2025 •

edited

Loading

google-oss-bot commented Oct 3, 2025 •

edited

Loading

Size Report ¹

`@firebase/remote-config`

`bundle`

`firebase`

google-oss-bot commented Oct 3, 2025 •

edited

Loading

Size Analysis Report ¹

`@firebase/remote-config`

`activate`

`ensureInitialized`

`fetchAndActivate`

`fetchConfig`

`getAll`

`getBoolean`

`getNumber`

`getRemoteConfig`

`getString`

`getValue`

`isSupported`

`onConfigUpdate`

`setCustomSignals`

`setLogLevel`

github-actions bot commented Oct 28, 2025 •

edited

Loading