Circular dependency

[alexbjorlig]

Am I the only one who gets this error when building?

Circular dependency: ../../node_modules/graphql-request/dist/index.js -> ../../node_modules/graphql-request/dist/graphql-ws.js -> ../../node_modules/graphql-request/dist/index.js

[jasonkuhrt]

What tool are you using that leads to this error?

[jonkoops]

FYI this is likely Rollup emitting this warning.

[jasonkuhrt]

I see this is coming from our ws module. Speaking of that @jonkoops there's a lot of duplication in that module from some other ws library. It would be nice to ditch all that if possible, but I haven't investigated recently.

[jonkoops]

Yeah, perhaps this is something we can get to for the next version. I am going to try and get all the open bugs resolved as best I can, then we should take a moment and decide if we want to make some API changes and improvements before the major release.