Fix WS types, current ones are broken

[ruizmarc]

WS library has changed how types are exported and graphql-ws is breaking builds if type checking for libraries is active:

node_modules/graphql-ws/lib/use/ws.d.ts(5,28): error TS2339: Property 'prototype' does not exist on type 'typeof import("/ptp/node_modules/@types/ws/index")'.

node_modules/graphql-ws/lib/use/ws.d.ts(6,27): error TS2694: Namespace '"/ptp/node_modules/@types/ws/index"' has no exported member 'Server'.

I have just properly imported types so we can use the library without problems.

Let me know if you need anything else from me. Hope to see it released soon!

[enisdenjo]

Thanks for the update! One small change request.

Also, does this work for older versions for ws? It wont be a breaking change? Furthermore, why aren't typechecks detecting this - does the @types/ws package need an update?

[ruizmarc]

Thanks for the update! One small change request.

Also, does this work for older versions for ws? It wont be a breaking change? Furthermore, why aren't typechecks detecting this - does the @types/ws package need an update?

Checking at how types are defined it seems Server and WebSocketServer are equivalents, but Server as name is no longer exported. As useServer uses the interface of WebSocketServer as parameter, anyone using Server should have compatible types.

I guess types were not crashing because in tsconfig you have: skipLibCheck as true, which skips types from libraries. As in my project we check lib types, I could see the problem.