fix(electron): detect electron context (#1856)

* fix(electron): detect electron env * fix(electron): cleanup code * fix: fixed wrong operator * fix(electron): improved code and add some comments * Update src/lib/is-browser.ts Co-authored-by: Daniel Lando <[email protected]> * fix: typo and lint --------- Co-authored-by: Daniel Lando <[email protected]>
mqttjs · May 6, 2024 · 6a03d29 · 6a03d29 · cmoniz-ocean · May 16, 2024
1 parent c6580a6
commit 6a03d29
Showing 1 changed file with 25 additions and 2 deletions.
diff --git a/src/lib/is-browser.ts b/src/lib/is-browser.ts
@@ -1,5 +1,28 @@
-const isStandardBrowserEnv = () =>
-	typeof window !== 'undefined' && typeof window.document !== 'undefined'
+const isStandardBrowserEnv = () => {
+	// window is only defined when it is a browser
+	if (typeof window !== 'undefined') {
+		// Is the process an electron application
+		// check if we are in electron `renderer`
+		const electronRenderCheck =
+			navigator?.userAgent?.toLowerCase().indexOf(' electron/') > -1
+		if (electronRenderCheck && process?.versions) {
+			const electronMainCheck = Object.prototype.hasOwnProperty.call(
+				process.versions,
+				'electron',
+			)
+			// Both electron checks are only true if the following webPreferences are set in the main electron BrowserWindow()
+			//   webPreferences: {
+			//     sandbox: false,
+			//     nodeIntegration: true
+			//     contextIsolation: false
+			// }
+			return !electronMainCheck
+		}
+		return typeof window.document !== 'undefined'
+	}
+	// return false if nothing is detected
+	return false
+}
 
 const isWebWorkerEnv = () =>
 	Boolean(