fix: wrong mqttjs version printed (#1847)

Author: robertsLando

esbuild.js

@@ -16,9 +16,6 @@ const options = {
     format: 'iife',
     platform: 'browser',
     globalName: 'mqtt',
-    define: {
-        'process.env.npm_package_version': JSON.stringify(version),
-    },
     sourcemap: false, // this can be enabled while debugging, if we decide to keep this enabled we should also ship the `src` folder to npm
     plugins: [
         polyfillNode({
@@ -34,6 +31,27 @@ const options = {
                 navigator: true, // Needed for WeChat, ref #1789
             }
         }),
+        {
+            name: 'resolve-package-json',
+            setup(build) {
+                // when importing 'package.json' we want to provide a custom object like { version: '1.2.3' }
+
+                build.onResolve({ filter: /package\.json$/ }, args => {
+                    return {
+                        path: args.path,
+                        namespace: 'package-json'
+                    }
+                })
+
+                build.onLoad({ filter: /.*/, namespace: 'package-json' }, args => {
+                    return {
+                        contents: JSON.stringify({ version }),
+                        loader: 'json'
+                    }
+                }
+                )
+            }
+        },
     ],
 }
 

src/lib/client.ts

@@ -31,6 +31,7 @@ import {
 	ErrorWithReasonCode,
 	GenericCallback,
 	IStream,
+	MQTTJS_VERSION,
 	StreamBuilder,
 	TimerVariant,
 	VoidCallback,
@@ -396,7 +397,7 @@ export interface MqttClientEventCallbacks {
  * (see Connection#connect)
  */
 export default class MqttClient extends TypedEventEmitter<MqttClientEventCallbacks> {
-	public static VERSION = process.env.npm_package_version
+	public static VERSION = MQTTJS_VERSION
 
 	/** Public fields */
 

src/lib/shared.ts

@@ -82,3 +82,6 @@ export const nextTick =
 		: (callback: () => void) => {
 				setTimeout(callback, 0)
 		  }
+
+// eslint-disable-next-line @typescript-eslint/no-var-requires
+export const MQTTJS_VERSION = require('../../package.json').version