Remove is-browser.ts detection util in favor of config option

[someengineer-nordsec]

Is your feature request related to a problem? Please describe.

• It seems this detection adds unnecessary complexity
• There have been https://github.com/mqttjs/MQTT.js/commits/main/src/lib/is-browser.ts quite a few fixes for it, there is lot of edge cases
• Chrome extension detection is bit flaky (some of this pain has been alleviated by feat: timerVariant option to choose between native and worker timers #1818)
• The exception 'ws does not work in the browser' if I try connecting from a worker thread #1709

Describe the solution you'd like
Configurable by required option

mqtt.connect(credentials.endpoint, {
  environment: 'browser', // 'react-native' 'electron' 'node'
})

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Additional context
Add any other context or screenshots about the feature request here.

[robertsLando]

@olso-nordsec problem is that actually we also need that in order to know which modules to load and which not (as some wouldn't work on browser). For the other points in code I think your solution could work but we firstly should define how to fix this. Maybe there could be a way by using esbuild

If you have something in mind feel free to open a PR

Ref:

MQTT.js/src/lib/connect/index.ts

Lines 21 to 33 in cd0b044

	if (!IS_BROWSER) {
	protocols.mqtt = require('./tcp').default
	protocols.tcp = require('./tcp').default
	protocols.ssl = require('./tls').default
	protocols.tls = protocols.ssl
	protocols.mqtts = require('./tls').default
	} else {
	protocols.wx = require('./wx').default
	protocols.wxs = require('./wx').default
	protocols.ali = require('./ali').default
	protocols.alis = require('./ali').default
	}