Bunch of stuff

Author: jamesdaniels

docs/zones.md

@@ -14,8 +14,26 @@ Observables, Promise-based APIs, and those with callbacks will purposely destabi
 
 When using a Firebase or RxFire API without importing from AngularFire or if AngularFire APIs are used outside of an injection context you _may_ experience instability.
 
-instability can be difficult to track down ([see Resolving zone pollution in the Angular docs](https://angular.dev/best-practices/zone-pollution)). To help with debugging, AngularFire emits warnings when it is unable to Zone wrap an API while in dev-mode. **Often these messages can be safely ignored** but we'd rather be verbose.
-
 When an application is unstable change-detection, two-way binding, and rehydration may not work as expected—leading to both subtle and non-subtle bugs in your application. Further, server-side rendering (SSR) and static site generation (SSG/pre-rendering) may timeout or render a blank page.
 
 There are a number of situations where AngularFire's Zone wrapping is inconsequential such adding/deleting/updating a document in response to user-input, signing a user in, calling a Cloud Function, etc. So long as no long-lived side-effects are kicked off, your application should be ok. Most Promise based APIs are fairly safe without zone wrapping. 
+
+## Logging
+
+You may see a log warning, `Calling Firebase APIs outside of an Injection context may destabilize your application leading to subtle change-detection and hydration bugs. Find more at https://github.com/angular/angularfire/blob/main/docs/zones.md` when developing your application.
+
+Instability can be difficult to track down. To help with debugging, AngularFire emits warnings when it is unable to Zone wrap an API while in dev-mode. **Often these messages can be safely ignored** but we'd rather be verbose.
+
+There are three logging levels in AngularFire:
+
+* **Silent**: when the logging level is set to silent only the above banner is displayed when AngularFire APIs are called outside of an injection context, this is the default when using Zoneless change-detection.
+* **Warn**: when the logging level is set to warn, only blocking reads, long-lived tasks, and APIs with high risk of destabilizing your application are called, this is the default when using ZoneJS.
+* **Verbose**: when the logging level is set to verbose, all AngularFire APIs called outside of an injection context are logged—helping you track down APIs that may be destabilizing your application
+
+You can change the log-level like so:
+
+```ts
+import { setLogLevel, LogLevel } from "@angular/fire";
+
+setLogLevel(LogLevel.VERBOSE);
+```

sample/package-lock.json

@@ -15,7 +15,7 @@
     "@angular/common": "^19.0.0",
     "@angular/compiler": "^19.0.0",
     "@angular/core": "^19.0.0",
-    "@angular/fire": "file:../angularfire/angular-fire-19.0.0.tgz",
+    "@angular/fire": "file:../angular-fire-19.0.0.tgz",
     "@angular/forms": "^19.0.0",
     "@angular/platform-browser": "^19.0.0",
     "@angular/platform-browser-dynamic": "^19.0.0",

sample/package.json

@@ -1,6 +1,9 @@
 import { bootstrapApplication } from '@angular/platform-browser';
 import { config } from './app/app.config.client';
 import { AppComponent } from './app/app.component';
+import { setLogLevel, LogLevel } from "@angular/fire";
+
+setLogLevel(LogLevel.VERBOSE);
 
 bootstrapApplication(AppComponent, config)
   .catch((err) => console.error(err));

sample/src/main.ts

@@ -10,16 +10,16 @@ import {
   isDevMode,
   makeEnvironmentProviders,
 } from '@angular/core';
-import { VERSION, ɵAPP_CHECK_PROVIDER_NAME, ɵAngularFireSchedulers, ɵAppCheckInstances, ɵgetDefaultInstanceOf } from '@angular/fire';
+import { VERSION, ɵAngularFireSchedulers, ɵgetDefaultInstanceOf } from '@angular/fire';
 import { FirebaseApp, FirebaseApps } from '@angular/fire/app';
 import { registerVersion } from 'firebase/app';
 import { AppCheck as FirebaseAppCheck } from 'firebase/app-check';
-import { AppCheck } from './app-check';
+import { APP_CHECK_PROVIDER_NAME, AppCheck, AppCheckInstances } from './app-check';
 
 export const PROVIDED_APP_CHECK_INSTANCES = new InjectionToken<AppCheck[]>('angularfire2.app-check-instances');
 
 export function defaultAppCheckInstanceFactory(provided: FirebaseAppCheck[]|undefined, defaultApp: FirebaseApp) {
-  const defaultAppCheck = ɵgetDefaultInstanceOf<FirebaseAppCheck>(ɵAPP_CHECK_PROVIDER_NAME, provided, defaultApp);
+  const defaultAppCheck = ɵgetDefaultInstanceOf<FirebaseAppCheck>(APP_CHECK_PROVIDER_NAME, provided, defaultApp);
   return defaultAppCheck && new AppCheck(defaultAppCheck);
 }
 
@@ -38,7 +38,7 @@ export function appCheckInstanceFactory(fn: (injector: Injector) => FirebaseAppC
 }
 
 const APP_CHECK_INSTANCES_PROVIDER = {
-  provide: ɵAppCheckInstances,
+  provide: AppCheckInstances,
   deps: [
     [new Optional(), PROVIDED_APP_CHECK_INSTANCES ],
   ]

src/analytics/firebase.ts

@@ -1,8 +1,10 @@
-import { ɵAPP_CHECK_PROVIDER_NAME, ɵgetAllInstancesOf } from '@angular/fire';
+import { ɵgetAllInstancesOf } from '@angular/fire';
 import { AppCheck as FirebaseAppCheck } from 'firebase/app-check';
 import { from, timer } from 'rxjs';
 import { concatMap, distinct } from 'rxjs/operators';
 
+export const APP_CHECK_PROVIDER_NAME = 'app-check';
+
 // see notes in core/firebase.app.module.ts for why we're building the class like this
 // eslint-disable-next-line @typescript-eslint/no-empty-interface
 export interface AppCheck extends FirebaseAppCheck {}
@@ -13,7 +15,16 @@ export class AppCheck {
   }
 }
 
+// eslint-disable-next-line @typescript-eslint/no-empty-interface
+export interface AppCheckInstances extends Array<AppCheck> {}
+
+export class AppCheckInstances {
+  constructor() {
+    return ɵgetAllInstancesOf<AppCheck>(APP_CHECK_PROVIDER_NAME);
+  }
+}
+
 export const appCheckInstance$ = timer(0, 300).pipe(
-  concatMap(() => from(ɵgetAllInstancesOf<FirebaseAppCheck>(ɵAPP_CHECK_PROVIDER_NAME))),
+  concatMap(() => from(ɵgetAllInstancesOf<FirebaseAppCheck>(APP_CHECK_PROVIDER_NAME))),
   distinct(),
 );

src/app-check/app-check.module.ts

@@ -1,4 +1,3 @@
-export { AppCheck, appCheckInstance$ } from './app-check';
-export { ɵAppCheckInstances as AppCheckInstances } from '@angular/fire';
+export { AppCheck, appCheckInstance$, AppCheckInstances } from './app-check';
 export { provideAppCheck, AppCheckModule } from './app-check.module';
 export * from './firebase';

src/app-check/app-check.ts

@@ -8,8 +8,8 @@ import {
   makeEnvironmentProviders,
 } from '@angular/core';
 import { VERSION, ɵAngularFireSchedulers, ɵgetDefaultInstanceOf } from '@angular/fire';
-import { ɵAppCheckInstances } from '@angular/fire';
 import { FirebaseApp, FirebaseApps } from '@angular/fire/app';
+import { AppCheckInstances } from '@angular/fire/app-check';
 import { registerVersion } from 'firebase/app';
 import { Auth as FirebaseAuth } from 'firebase/auth';
 import { AUTH_PROVIDER_NAME, Auth, AuthInstances } from './auth';
@@ -70,7 +70,7 @@ export function provideAuth(fn: (injector: Injector) => FirebaseAuth, ...deps: a
         Injector,
         ɵAngularFireSchedulers,
         FirebaseApps,
-        [new Optional(), ɵAppCheckInstances ],
+        [new Optional(), AppCheckInstances ],
         ...deps,
       ]
     }