Consider fixing types to prevent erros when installing the package in an ESM project

[TomasHubelbauer]

Hi, I am trying to use your project through https://www.npmjs.com/package/eufy-security-client which relies on it. I set up a new Vite project using npm create vite@latest my-react-app -- --template react and installed the package but when I run npm run build, I run into an error which says:

Failed to resolve entry for package "mqtt". The package may have incorrect main/module/exports specified in its package.json. [plugin vite:dep-pre-bundle]

I believe this might be caused by incorrectly configured types. I checked your package using https://arethetypeswrong.github.io/?p=mqtt%405.3.4 and this is the result:

I am not a 100 % certain, but I believe the errors reported there might be what's causing the above issue.

I wonder if these could be fixed to make the package compatible with both CJS and ESM.

[robertsLando]

Could you submit a PR to fix this issue?

[mschuettlerTNG]

Our code base broke with similar errors when upgrading from 5.1.3 to 5.3.4

This might be related to
#1740
and is possibly explained by
evanw/esbuild#532 (comment)

[TomasHubelbauer]

@robertsLando As far as whether I can fix this, I don't know how to. I am not able to parse the information on the site I linked well enough to be able to figure out what the supposed fix is and the thread linked by Markus goes way over my head. I've personally been in the ESM-only land for several years now so my experience with dealing with CJS has (for the most part, thankfully, but in this specific instance, unfortunately) all but vanished.

[robertsLando]

@mschuettlerTNG I will give a look at this when I have some time in the meanwhile if you find a fix feel free to submit a PR

[TomasHubelbauer]

BTW this happens to me when using MQTT with Vite and I was able to make the error go away with this Vite config but then ran into a runtime error instead:

import { defineConfig } from 'vite'
import react from '@vitejs/plugin-react'
import path from 'path'

// https://vitejs.dev/config/
export default defineConfig({
  plugins: [react()],
  resolve: {
    alias: {
      'mqtt': path.resolve(__dirname, 'node_modules/mqtt/dist/mqtt.min.js')
    }
  },
})

[robertsLando]

Hi @TomasHubelbauer , did you checked the vite example in this repo? I made it work without any additional configuration, see https://github.com/mqttjs/MQTT.js/tree/main/examples/vite-example

[github-actions]

This is an automated message to let you know that this issue has
gone 365 days without any activity. In order to ensure that we work
on issues that still matter, this issue will be closed in 14 days.

If this issue is still important, you can simply comment with a
"bump" to keep it open.

Thank you for your contribution.