EventEmitter doesn't exist in browsers

[francislavoie]

Describe the bug

After playing around with some of my frontend project's package.json dependencies, I ended up yeeting some of the deps that provided node polyfills along the way, and that made this lib no longer work because of this line:

client-js/src/transports/TransportRequestManager.ts

Line 6 in f1c9b04

import { EventEmitter } from "events";

More specifically, when my HTTP Transport is constructed, it would hit this line, and I would get EventEmitter is not a constructor

client-js/src/transports/TransportRequestManager.ts

Line 24 in f1c9b04

this.transportEventChannel = new EventEmitter();

I had to install events@^3.3.0 to fix it.

Expected behavior

This lib should either depend on events so that I don't need to require it myself, or it should use APIs that actually exist in the browser.

[zcstarr]

Nice catch! Thanks for the issue we'll take dive into this.