run formatter

Author: kjelko

packages/remote-config/src/api.ts

@@ -42,7 +42,10 @@ import { LogLevel as FirebaseLogLevel } from '@firebase/logger';
  *
  * @public
  */
-export function getRemoteConfig(app: FirebaseApp = getApp(), options: RemoteConfigOptions = {}): RemoteConfig {
+export function getRemoteConfig(
+  app: FirebaseApp = getApp(),
+  options: RemoteConfigOptions = {}
+): RemoteConfig {
   app = getModularInstance(app);
   const rcProvider = _getProvider(app, RC_COMPONENT_NAME);
   if (rcProvider.isInitialized()) {
@@ -63,7 +66,9 @@ export function getRemoteConfig(app: FirebaseApp = getApp(), options: RemoteConf
       rc._storage.setActiveConfigEtag(options.initialFetchResponse?.eTag || ''),
       rc._storageCache.setLastSuccessfulFetchTimestampMillis(Date.now()),
       rc._storageCache.setLastFetchStatus('success'),
-      rc._storageCache.setActiveConfig(options.initialFetchResponse?.config || {})
+      rc._storageCache.setActiveConfig(
+        options.initialFetchResponse?.config || {}
+      )
     ]).then();
     // The storageCache methods above set their in-memory fields sycnhronously, so it's
     // safe to declare our initialization complete at this point.
@@ -249,7 +254,7 @@ export function getValue(remoteConfig: RemoteConfig, key: string): Value {
   if (!rc._isInitializationComplete) {
     rc._logger.debug(
       `A value was requested for key "${key}" before SDK initialization completed.` +
-      ' Await on ensureInitialized if the intent was to get a previously activated value.'
+        ' Await on ensureInitialized if the intent was to get a previously activated value.'
     );
   }
   const activeConfig = rc._storageCache.getActiveConfig();
@@ -260,7 +265,7 @@ export function getValue(remoteConfig: RemoteConfig, key: string): Value {
   }
   rc._logger.debug(
     `Returning static value for key "${key}".` +
-    ' Define a default or remote value if this is unintentional.'
+      ' Define a default or remote value if this is unintentional.'
   );
   return new ValueImpl('static');
 }

packages/remote-config/src/public_types.ts

@@ -18,11 +18,14 @@
 import { FirebaseApp } from '@firebase/app';
 import { FetchResponse } from './client/remote_config_fetch_client';
 
-export { FetchResponse, FirebaseRemoteConfigObject } from './client/remote_config_fetch_client';
+export {
+  FetchResponse,
+  FirebaseRemoteConfigObject
+} from './client/remote_config_fetch_client';
 
 /**
  * Options for Remote Config initialization.
- * 
+ *
  * @public
  */
 export interface RemoteConfigOptions {

packages/remote-config/src/register.ts

@@ -23,7 +23,7 @@ import { isIndexedDBAvailable } from '@firebase/util';
 import {
   Component,
   ComponentType,
-  ComponentContainer,
+  ComponentContainer
 } from '@firebase/component';
 import { Logger, LogLevel as FirebaseLogLevel } from '@firebase/logger';
 import { RemoteConfig, RemoteConfigOptions } from './public_types';
@@ -79,9 +79,9 @@ export function registerRemoteConfig(): void {
     }
     const namespace = options?.templateId || 'firebase';
 
-    const storage = isIndexedDBAvailable() ? 
-        new IndexedDbStorage(appId, app.name, namespace) :
-        new InMemoryStorage();
+    const storage = isIndexedDBAvailable()
+      ? new IndexedDbStorage(appId, app.name, namespace)
+      : new InMemoryStorage();
     const storageCache = new StorageCache(storage);
 
     const logger = new Logger(packageName);

packages/remote-config/src/storage/storage.ts

@@ -175,7 +175,9 @@ export abstract class Storage {
     return this.get<CustomSignals>('custom_signals');
   }
 
-  abstract setCustomSignals(customSignals: CustomSignals): Promise<CustomSignals>;
+  abstract setCustomSignals(
+    customSignals: CustomSignals
+  ): Promise<CustomSignals>;
   abstract get<T>(key: ProjectNamespaceKeyFieldValue): Promise<T | undefined>;
   abstract set<T>(key: ProjectNamespaceKeyFieldValue, value: T): Promise<void>;
   abstract delete(key: ProjectNamespaceKeyFieldValue): Promise<void>;
@@ -373,8 +375,8 @@ export class InMemoryStorage extends Storage {
 
   async setCustomSignals(customSignals: CustomSignals): Promise<CustomSignals> {
     const combinedSignals = {
-      ...this.storage['custom_signals'] as CustomSignals,
-      ...customSignals,
+      ...(this.storage['custom_signals'] as CustomSignals),
+      ...customSignals
     };
 
     const updatedSignals = Object.fromEntries(

packages/remote-config/test/api.test.ts

@@ -1,18 +1,39 @@
+/**
+ * @license
+ * Copyright 2025 Google LLC
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
 import { expect } from 'chai';
-import { ensureInitialized, fetchAndActivate, getRemoteConfig, getString } from '../src/index';
+import {
+  ensureInitialized,
+  fetchAndActivate,
+  getRemoteConfig,
+  getString
+} from '../src/index';
 import '../test/setup';
-import { deleteApp, FirebaseApp, initializeApp, _addOrOverwriteComponent } from '@firebase/app';
+import {
+  deleteApp,
+  FirebaseApp,
+  initializeApp,
+  _addOrOverwriteComponent
+} from '@firebase/app';
 import * as sinon from 'sinon';
 import { FetchResponse } from '../src/client/remote_config_fetch_client';
-import {
-  Component,
-  ComponentType
-} from '@firebase/component';
+import { Component, ComponentType } from '@firebase/component';
 import { FirebaseInstallations } from '@firebase/installations-types';
-import {
-  openDatabase,
-  APP_NAMESPACE_STORE,
-} from '../src/storage/storage';
+import { openDatabase, APP_NAMESPACE_STORE } from '../src/storage/storage';
 
 const fakeFirebaseConfig = {
   apiKey: 'api-key',
@@ -36,7 +57,7 @@ describe('Remote Config API', () => {
   const STUB_FETCH_RESPONSE: FetchResponse = {
     status: 200,
     eTag: 'asdf',
-    config: { 'foobar': 'hello world' },
+    config: { 'foobar': 'hello world' }
   };
   let fetchStub: sinon.SinonStub;
 
@@ -50,11 +71,11 @@ describe('Remote Config API', () => {
         () => {
           return {
             getId: () => Promise.resolve('fis-id'),
-            getToken: () => Promise.resolve('fis-token'),
+            getToken: () => Promise.resolve('fis-token')
           } as any as FirebaseInstallations;
         },
         ComponentType.PUBLIC
-      ) as any,
+      ) as any
     );
   });
 
@@ -65,16 +86,18 @@ describe('Remote Config API', () => {
   });
 
   function setFetchResponse(response: FetchResponse = { status: 200 }): void {
-    fetchStub.returns(Promise.resolve({
-      ok: response.status === 200,
-      status: response.status,
-      headers: new Headers({ ETag: response.eTag || '' }),
-      json: () =>
-        Promise.resolve({
-          entries: response.config,
-          state: 'OK'
-        })
-    } as Response));
+    fetchStub.returns(
+      Promise.resolve({
+        ok: response.status === 200,
+        status: response.status,
+        headers: new Headers({ ETag: response.eTag || '' }),
+        json: () =>
+          Promise.resolve({
+            entries: response.config,
+            state: 'OK'
+          })
+      } as Response)
+    );
   }
 
   it('allows multiple initializations if options are same', () => {
@@ -115,11 +138,14 @@ describe('Remote Config API', () => {
     await fetchAndActivate(rc);
     expect(fetchStub).to.be.calledOnceWith(
       'https://firebaseremoteconfig.googleapis.com/v1/projects/project-id/namespaces/altTemplate:fetch?key=api-key',
-      sinon.match.object);
+      sinon.match.object
+    );
   });
 
   it('hydrates with initialFetchResponse', async () => {
-    const rc = getRemoteConfig(app, { initialFetchResponse: STUB_FETCH_RESPONSE });
+    const rc = getRemoteConfig(app, {
+      initialFetchResponse: STUB_FETCH_RESPONSE
+    });
     await ensureInitialized(rc);
     expect(getString(rc, 'foobar')).to.equal('hello world');
   });

packages/remote-config/test/storage/storage.test.ts

@@ -36,15 +36,14 @@ async function clearDatabase(): Promise<void> {
 }
 
 describe('Storage', () => {
-
   const indexedDbTestCase = {
     getStorage: () => new IndexedDbStorage('appId', 'appName', 'namespace'),
-    name: 'IndexedDbStorage',
+    name: 'IndexedDbStorage'
   };
 
   const inMemoryStorage = {
     getStorage: () => new InMemoryStorage(),
-    name: 'InMemoryStorage',
+    name: 'InMemoryStorage'
   };
 
   beforeEach(async () => {
@@ -53,9 +52,9 @@ describe('Storage', () => {
 
   it(`${indexedDbTestCase.name} constructs a composite key`, async () => {
     // This is defensive, but the cost of accidentally changing the key composition is high.
-    expect(indexedDbTestCase.getStorage().createCompositeKey('throttle_metadata')).to.eq(
-      'appId,appName,namespace,throttle_metadata'
-    );
+    expect(
+      indexedDbTestCase.getStorage().createCompositeKey('throttle_metadata')
+    ).to.eq('appId,appName,namespace,throttle_metadata');
   });
 
   for (const { name, getStorage } of [indexedDbTestCase, inMemoryStorage]) {
@@ -92,7 +91,9 @@ describe('Storage', () => {
       it('sets and gets last successful fetch response', async () => {
         const lastSuccessfulFetchResponse = { status: 200 } as FetchResponse;
 
-        await storage.setLastSuccessfulFetchResponse(lastSuccessfulFetchResponse);
+        await storage.setLastSuccessfulFetchResponse(
+          lastSuccessfulFetchResponse
+        );
 
         const actualConfig = await storage.getLastSuccessfulFetchResponse();