Revert non functions-compat changes

Author: hsubox76

packages-exp/functions-compat/src/register.ts

@@ -29,6 +29,8 @@ import {
 } from '@firebase/component';
 import { Functions as FunctionsServiceExp } from '@firebase/functions-exp';
 
+const DEFAULT_REGION = 'us-central1';
+
 declare module '@firebase/component' {
   interface NameServiceMapping {
     'app-compat': FirebaseApp;
@@ -39,14 +41,15 @@ declare module '@firebase/component' {
 
 const factory: InstanceFactory<'functions-compat'> = (
   container: ComponentContainer,
-  { instanceIdentifier: regionOrCustomDomain }: InstanceFactoryOptions
+  options?: InstanceFactoryOptions
 ) => {
   // Dependencies
   const app = container.getProvider('app-compat').getImmediate();
   const functionsServiceExp = container
     .getProvider('functions-exp')
     .getImmediate({
-      identifier: regionOrCustomDomain
+      // This value is used as regionOrCustomDomain
+      identifier: options?.instanceIdentifier ?? DEFAULT_REGION
     });
 
   return new FunctionsService(app, functionsServiceExp);

packages-exp/functions-exp/src/config.ts

@@ -21,16 +21,15 @@ import {
   Component,
   ComponentType,
   ComponentContainer,
-  InstanceFactory
+  InstanceFactory,
+  InstanceFactoryOptions
 } from '@firebase/component';
 import { FUNCTIONS_TYPE } from './constants';
 
-export const DEFAULT_REGION = 'us-central1';
-
 export function registerFunctions(fetchImpl: typeof fetch): void {
   const factory: InstanceFactory<'functions'> = (
     container: ComponentContainer,
-    { instanceIdentifier: regionOrCustomDomain }
+    options?: InstanceFactoryOptions
   ) => {
     // Dependencies
     const app = container.getProvider('app-exp').getImmediate();
@@ -42,7 +41,8 @@ export function registerFunctions(fetchImpl: typeof fetch): void {
       app,
       authProvider,
       messagingProvider,
-      regionOrCustomDomain,
+      // regionOrCustomDomain
+      options?.instanceIdentifier,
       fetchImpl
     );
   };

packages-exp/performance-exp/src/index.ts

@@ -91,7 +91,7 @@ export function trace(
 
 const factory: InstanceFactory<'performance-exp'> = (
   container: ComponentContainer,
-  { options: settings }: { options?: PerformanceSettings }
+  instanceFactoryOptions?: { options?: PerformanceSettings }
 ) => {
   // Dependencies
   const app = container.getProvider('app-exp').getImmediate();
@@ -107,7 +107,7 @@ const factory: InstanceFactory<'performance-exp'> = (
   }
   setupApi(window);
   const perfInstance = new PerformanceController(app, installations);
-  perfInstance._init(settings);
+  perfInstance._init(instanceFactoryOptions.options);
 
   return perfInstance;
 };

packages-exp/remote-config-compat/src/index.ts

@@ -49,12 +49,12 @@ function registerRemoteConfigCompat(
 
 function remoteConfigFactory(
   container: ComponentContainer,
-  { instanceIdentifier: namespace }: InstanceFactoryOptions
+  options?: InstanceFactoryOptions
 ): RemoteConfigCompatImpl {
   const app = container.getProvider('app-compat').getImmediate();
   // The following call will always succeed because rc `import {...} from '@firebase/remote-config-exp'`
   const remoteConfig = container.getProvider('remote-config-exp').getImmediate({
-    identifier: namespace
+    identifier: options?.instanceIdentifier // namespace
   });
 
   return new RemoteConfigCompatImpl(app, remoteConfig);

packages-exp/remote-config-exp/src/register.ts

@@ -54,7 +54,7 @@ export function registerRemoteConfig(): void {
 
   function remoteConfigFactory(
     container: ComponentContainer,
-    { instanceIdentifier: namespace }: InstanceFactoryOptions
+    options?: InstanceFactoryOptions
   ): RemoteConfig {
     /* Dependencies */
     // getImmediate for FirebaseApp will always succeed
@@ -80,7 +80,7 @@ export function registerRemoteConfig(): void {
     if (!appId) {
       throw ERROR_FACTORY.create(ErrorCode.REGISTRATION_APP_ID);
     }
-    namespace = namespace || 'firebase';
+    const namespace = options?.instanceIdentifier || 'firebase';
 
     const storage = new Storage(appId, app.name, namespace);
     const storageCache = new StorageCache(storage);

packages/database/exp/index.ts

@@ -17,7 +17,11 @@
 
 // eslint-disable-next-line import/no-extraneous-dependencies
 import { _registerComponent, registerVersion } from '@firebase/app-exp';
-import { Component, ComponentType } from '@firebase/component';
+import {
+  Component,
+  ComponentType,
+  InstanceFactoryOptions
+} from '@firebase/component';
 
 import { version } from '../package.json';
 import { FirebaseDatabase } from '../src/exp/Database';
@@ -35,10 +39,14 @@ function registerDatabase(): void {
   _registerComponent(
     new Component(
       'database-exp',
-      (container, { instanceIdentifier: url }) => {
+      (container, options?: InstanceFactoryOptions) => {
         const app = container.getProvider('app-exp').getImmediate()!;
         const authProvider = container.getProvider('auth-internal');
-        return new FirebaseDatabase(app, authProvider, url);
+        return new FirebaseDatabase(
+          app,
+          authProvider,
+          options?.instanceIdentifier // url
+        );
       },
       ComponentType.PUBLIC
     ).setMultipleInstances(true)

packages/database/index.node.ts

@@ -30,7 +30,11 @@ import { setSDKVersion } from './src/core/version';
 import { CONSTANTS, isNodeSdk } from '@firebase/util';
 import { setWebSocketImpl } from './src/realtime/WebSocketConnection';
 import { Client } from 'faye-websocket';
-import { Component, ComponentType } from '@firebase/component';
+import {
+  Component,
+  ComponentType,
+  InstanceFactoryOptions
+} from '@firebase/component';
 import { FirebaseAuthInternal } from '@firebase/auth-interop-types';
 
 import { name, version } from './package.json';
@@ -84,13 +88,18 @@ export function registerDatabase(instance: FirebaseNamespace) {
   const namespace = (instance as _FirebaseNamespace).INTERNAL.registerComponent(
     new Component(
       'database',
-      (container, { instanceIdentifier: url }) => {
+      (container, options?: InstanceFactoryOptions) => {
         /* Dependencies */
         // getImmediate for FirebaseApp will always succeed
         const app = container.getProvider('app').getImmediate();
         const authProvider = container.getProvider('auth-internal');
 
-        return repoManagerDatabaseFromApp(app, authProvider, url, undefined);
+        return repoManagerDatabaseFromApp(
+          app,
+          authProvider,
+          options?.instanceIdentifier, // url
+          undefined
+        );
       },
       ComponentType.PUBLIC
     )

packages/database/index.ts

@@ -30,7 +30,11 @@ import * as TEST_ACCESS from './src/api/test_access';
 import { isNodeSdk } from '@firebase/util';
 import * as types from '@firebase/database-types';
 import { setSDKVersion } from './src/core/version';
-import { Component, ComponentType } from '@firebase/component';
+import {
+  Component,
+  ComponentType,
+  InstanceFactoryOptions
+} from '@firebase/component';
 
 import { name, version } from './package.json';
 
@@ -44,13 +48,18 @@ export function registerDatabase(instance: FirebaseNamespace) {
   const namespace = (instance as _FirebaseNamespace).INTERNAL.registerComponent(
     new Component(
       'database',
-      (container, { instanceIdentifier: url }) => {
+      (container, options: InstanceFactoryOptions) => {
         /* Dependencies */
         // getImmediate for FirebaseApp will always succeed
         const app = container.getProvider('app').getImmediate();
         const authProvider = container.getProvider('auth-internal');
 
-        return repoManagerDatabaseFromApp(app, authProvider, url, undefined);
+        return repoManagerDatabaseFromApp(
+          app,
+          authProvider,
+          options?.instanceIdentifier, // url
+          undefined
+        );
       },
       ComponentType.PUBLIC
     )

packages/functions/src/config.ts

@@ -40,7 +40,7 @@ export function registerFunctions(
 
   function factory(
     container: ComponentContainer,
-    { instanceIdentifier: regionOrCustomDomain }: InstanceFactoryOptions
+    options?: InstanceFactoryOptions
   ): Service {
     // Dependencies
     const app = container.getProvider('app').getImmediate();
@@ -52,7 +52,8 @@ export function registerFunctions(
       app,
       authProvider,
       messagingProvider,
-      regionOrCustomDomain,
+      // regionOrCustomDomain
+      options?.instanceIdentifier,
       fetchImpl
     );
   }

packages/remote-config/index.ts

@@ -60,7 +60,7 @@ export function registerRemoteConfig(
 
   function remoteConfigFactory(
     container: ComponentContainer,
-    { instanceIdentifier: namespace }: InstanceFactoryOptions
+    options?: InstanceFactoryOptions
   ): RemoteConfig {
     /* Dependencies */
     // getImmediate for FirebaseApp will always succeed
@@ -84,7 +84,7 @@ export function registerRemoteConfig(
     if (!appId) {
       throw ERROR_FACTORY.create(ErrorCode.REGISTRATION_APP_ID);
     }
-    namespace = namespace || 'firebase';
+    const namespace = options?.instanceIdentifier || 'firebase';
 
     const storage = new Storage(appId, app.name, namespace);
     const storageCache = new StorageCache(storage);

packages/storage/compat/index.ts

@@ -41,7 +41,7 @@ const STORAGE_TYPE = 'storage';
 
 function factory(
   container: ComponentContainer,
-  { instanceIdentifier: url }: InstanceFactoryOptions
+  options?: InstanceFactoryOptions
 ): types.FirebaseStorage {
   // Dependencies
   // TODO: This should eventually be 'app-compat'
@@ -56,7 +56,7 @@ function factory(
       app,
       authProvider,
       new XhrIoPool(),
-      url,
+      options?.instanceIdentifier, // url
       firebase.SDK_VERSION
     )
   );

packages/storage/exp/index.ts

@@ -298,7 +298,7 @@ export function getStorage(
 
 function factory(
   container: ComponentContainer,
-  { instanceIdentifier: url }: InstanceFactoryOptions
+  options?: InstanceFactoryOptions
 ): StorageService {
   const app = container.getProvider('app-exp').getImmediate();
   const authProvider = container.getProvider('auth-internal');
@@ -307,7 +307,7 @@ function factory(
     app,
     authProvider,
     new XhrIoPool(),
-    url,
+    options?.instanceIdentifier, // url
     SDK_VERSION
   );
 }