Socket.io should probably ESM export "Server"

[richardeschloss]

socket.io/lib/index.ts

Line 1002 in 945c84b

export { Socket, ServerOptions, Namespace, BroadcastOperator, RemoteSocket };

I think the ESM export should include Server too? It seems like it's only exported with the CommonJS. Some modern frameworks may not work well with the mixed CJS / ESM exports, and may throw an error when trying to import { Server } from "socket.io"

Related issue: https://github.com/richardeschloss/nuxt-socket-io/issues/278

I appreciate your help!

[darrachequesne]

Hi! The Server class is actually exported:

socket.io/lib/index.ts

Line 102 in 945c84b

export class Server<

But there might be an issue in the underlying engine.io package: socketio/engine.io#657

[richardeschloss]

Ok, will close this in favor of socketio/engine.io#657