fix: electron main and renderer process detection

src/lib/client.ts

@@ -39,7 +39,7 @@
 } from './shared'
 import TopicAliasSend from './topic-alias-send'
 import { TypedEventEmitter } from './TypedEmitter'
 import KeepaliveManager from './KeepaliveManager'
 import isBrowser, { isWebWorker } from './is-browser'
 
 const setImmediate =
@@ -405,6 +405,7 @@
  */
 export default class MqttClient extends TypedEventEmitter<MqttClientEventCallbacks> {
 	public static VERSION = MQTTJS_VERSION
+	public static NEED_CHECK_BROWSER_ENVIRONMENT = true
 
 	/** Public fields */
 
@@ -486,6 +487,10 @@
 		return `mqttjs_${Math.random().toString(16).substr(2, 8)}`
 	}
 
+	public get needCheckBrowserEnvironment() {
+		return MqttClient.NEED_CHECK_BROWSER_ENVIRONMENT
+	}
+
 	constructor(streamBuilder: StreamBuilder, options: IClientOptions) {
 		super()
 

src/lib/is-browser.ts

@@ -1,28 +1,7 @@
+import MqttClient from './client'
+
 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 =
-			typeof navigator !== 'undefined' &&
-			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
+	return typeof window !== 'undefined' && typeof window.document !== 'undefined'
 }
 
 const isWebWorkerEnv = () =>
@@ -37,7 +16,7 @@
 	typeof navigator !== 'undefined' && navigator.product === 'ReactNative'
 
 const isBrowser =
-	isStandardBrowserEnv() || isWebWorkerEnv() || isReactNativeEnv()
+	MqttClient.NEED_CHECK_BROWSER_ENVIRONMENT !== false && (isStandardBrowserEnv() || isWebWorkerEnv() || isReactNativeEnv())
 
 export const isWebWorker = isWebWorkerEnv()