From baa96ce400eba597600446915abef7ffd22736ed Mon Sep 17 00:00:00 2001
From: Jose V Sebastian <josevseb99@gmail.com>
Date: Wed, 26 Jun 2024 08:12:22 +0530
Subject: [PATCH 1/2] fix: browser detection

update browser detection logic:
detect either window or web worker (WorkerGlobalScope).

fixes #8299 #8284
---
 .changeset/chilly-moons-play.md  |  5 +++++
 packages/util/src/environment.ts | 18 +++++++++++++++++-
 2 files changed, 22 insertions(+), 1 deletion(-)
 create mode 100644 .changeset/chilly-moons-play.md

diff --git a/.changeset/chilly-moons-play.md b/.changeset/chilly-moons-play.md
new file mode 100644
index 00000000000..1d1db3c15c6
--- /dev/null
+++ b/.changeset/chilly-moons-play.md
@@ -0,0 +1,5 @@
+---
+'@firebase/util': patch
+---
+
+fix: browser detection (detect either window or web worker)
diff --git a/packages/util/src/environment.ts b/packages/util/src/environment.ts
index f2afaede0f1..30fc661bfc6 100644
--- a/packages/util/src/environment.ts
+++ b/packages/util/src/environment.ts
@@ -18,6 +18,11 @@
 import { CONSTANTS } from './constants';
 import { getDefaults } from './defaults';
 
+/**
+ * Type placeholder for `WorkerGlobalScope` from `webworker`
+ */
+declare class WorkerGlobalScope {}
+
 /**
  * Returns navigator.userAgent string or '' if it's not defined.
  * @return user agent string
@@ -77,7 +82,18 @@ export function isNode(): boolean {
  * Detect Browser Environment
  */
 export function isBrowser(): boolean {
-  return typeof self === 'object' && self.self === self;
+  return typeof window !== 'undefined' || isWebWorker();
+}
+
+/**
+ * Detect Web Worker context
+ */
+export function isWebWorker(): boolean {
+  return (
+    typeof WorkerGlobalScope !== 'undefined' &&
+    typeof self !== 'undefined' &&
+    self instanceof WorkerGlobalScope
+  );
 }
 
 /**

From 3a2a1badf48cfd52dadb062a035975e0a4bd3bff Mon Sep 17 00:00:00 2001
From: Jose V Sebastian <josevseb99@gmail.com>
Date: Thu, 27 Jun 2024 08:17:25 +0530
Subject: [PATCH 2/2] fix: server app should initialize in web workers

---
 .changeset/fifty-mayflies-hug.md | 5 +++++
 packages/app/src/api.ts          | 9 +++++++--
 2 files changed, 12 insertions(+), 2 deletions(-)
 create mode 100644 .changeset/fifty-mayflies-hug.md

diff --git a/.changeset/fifty-mayflies-hug.md b/.changeset/fifty-mayflies-hug.md
new file mode 100644
index 00000000000..ad1970ce863
--- /dev/null
+++ b/.changeset/fifty-mayflies-hug.md
@@ -0,0 +1,5 @@
+---
+'@firebase/app': patch
+---
+
+fix: server app should initialize in web workers
diff --git a/packages/app/src/api.ts b/packages/app/src/api.ts
index 5928fd737a8..b8ec25fc509 100644
--- a/packages/app/src/api.ts
+++ b/packages/app/src/api.ts
@@ -48,7 +48,12 @@ import {
   LogOptions,
   setUserLogHandler
 } from '@firebase/logger';
-import { deepEqual, getDefaultAppConfig, isBrowser } from '@firebase/util';
+import {
+  deepEqual,
+  getDefaultAppConfig,
+  isBrowser,
+  isWebWorker
+} from '@firebase/util';
 
 export { FirebaseError } from '@firebase/util';
 
@@ -230,7 +235,7 @@ export function initializeServerApp(
   _options: FirebaseOptions | FirebaseApp,
   _serverAppConfig: FirebaseServerAppSettings
 ): FirebaseServerApp {
-  if (isBrowser()) {
+  if (isBrowser() && !isWebWorker()) {
     // FirebaseServerApp isn't designed to be run in browsers.
     throw ERROR_FACTORY.create(AppError.INVALID_SERVER_APP_ENVIRONMENT);
   }