Circular dependency warnings

[fix-me]

Hi,

I'm using swagger-parser module in a rollupJS library with the commonjs plugin.
Several circular dependencies appear while build process:

(!) Circular dependency: node_modules\z-schema\src\SchemaCache.js -> node_modules\z-schema\src\SchemaCompilation.js -> node_modules\z-schema\src\SchemaCache.js
(!) Circular dependency: node_modules\z-schema\src\SchemaCache.js -> node_modules\z-schema\src\SchemaCompilation.js ->  commonjs-proxy:***\node_modules\z-schema\src\SchemaCache.js -> node_modules\z-schema\src\SchemaCache.js
(!) Circular dependency: node_modules\json-schema-ref-parser\lib\ref.js -> node_modules\json-schema-ref-parser\lib\pointer.js -> node_modules\json-schema-ref-parser\lib\ref.js
(!) Circular dependency: node_modules\json-schema-ref-parser\lib\ref.js -> node_modules\json-schema-ref-parser\lib\pointer.js ->  commonjs-proxy:***\node_modules\json-schema-ref-parser\lib\ref.js -> node_modules\json-schema-ref-parser\lib\ref.js

Greetz!

[JamesMessinger]

Circular dependencies between modules are allowed in JavaScript. If Rollup throws errors on circular dependencies, then that's a bug in Rollup.

That said... I bet there's a way to get Rollup to work. Maybe there's an option or a plugin or something. I'm not very familiar with Rollup, so I don't know.

[JamesMessinger]

It appears that this is a known issue in Rollup

nodejs/readable-stream#348

[fix-me]

Oh, I see its a problem with cjs modules which get lifted to es static imports.
So it's more related to the commonJS plugin, I'll try my best there.

Thank you!