"not a constructor" error when initialising WebPubSubClient using Vite

[Chris-Greaves]

• Package Name: @azure/web-pubsub-client
• Package Version: ^1.0.0-beta.3
• Operating system: Windows
• nodejs
  • version: v21.1.0
• browser
  • name/version: Google Chrome/119.0.6045.200
• typescript
  • version: ^5.0.0

Describe the bug
Can't initialise a new WebPubSubClient instance in my Sveltekit project. I'm getting the error TypeError: import_events.default is not a constructor (see screenshot to show where)

To Reproduce
Steps to reproduce the behavior:

1. Create a new skeleton Sveltekit project with Typescript: npm create svelte@latest bug-test
2. Add the following code to the +page.svelte file

   <script lang="ts">
   import { WebPubSubClient } from '@azure/web-pubsub-client';
   
   let tokenUrl = "<add token url>";
   let client: WebPubSubClient;
   
   async function connect() {
   	client = new WebPubSubClient(tokenUrl);
   	await client.start();
   
   	client.on('connected', () => console.log('Web PubSub Connected'));
   	client.on('group-message', m => console.log('Message received: %s', JSON.stringify(m)));
   }
   </script>
   
   <button on:click={connect}>Connect</button>
   

3. Run the server and click the button

Expected behavior
The client should be initialised so I can connect to the Web PubSub

Screenshots

Additional context
I'm not a frontend developer in any way, but I'll try my best to give as much info as possible.

[Chris-Greaves]

Also fails in Edge and Firefox.

I noticed that the EventEmitter doesn't have curly brackets {} when it's imported.

src: https://github.com/Azure/azure-sdk-for-js/blob/main/sdk/web-pubsub/web-pubsub-client/src/webPubSubClient.ts#L6

If changing this would fix it, I don't mind doing a PR to change every instance of EventEmitter being imported to use curly brackets.

[c-richard]

We were able to workaround this issue by installing this events package as a dependency https://www.npmjs.com/package/events.

This seems like an issue using this library with vite -> vitejs/vite#7821