Circular dependencies

[kamilkisiela]

store.js -> optimistic-data/store.js -> store.js
core/ObservableQuery.js -> core/QueryManager.js -> scheduler/scheduler.js -> core/ObservableQuery.js
core/QueryManager.js -> scheduler/scheduler.js -> core/QueryManager.js
optimistic-data/store.js -> store.js -> optimistic-data/store.js
scheduler/scheduler.js -> core/QueryManager.js -> scheduler/scheduler.js

@DxCx

[helfer]

@kamilkisiela what is this for? Some tool that cannot handle circular dependencies?

PS: I'm surprised there isn't one between ApolloClient.ts and QueryManager.ts.

[kamilkisiela]

@helfer yes, rollup + systemjs for an example. There's a webpack plugin that shows all circular dependencies.

We work on some cool thing and this stops us, last missing piece.

[carloscordon]

I also managed to hit this issue with rollup. It took quite a while to debug. FWIW, here's the rollup issue for tracking this: rollup/rollup-plugin-commonjs#105