When compiled, shows warnings: Circular dependency

[francoisromain]

Hello,

When compiled, Rollup shows some warnings:

(!) Circular dependency: node_modules/rollup-plugin-node-builtins/src/es6/readable-stream/duplex.js -> node_modules/rollup-plugin-node-builtins/src/es6/readable-stream/readable.js -> node_modules/rollup-plugin-node-builtins/src/es6/readable-stream/duplex.js
(!) Circular dependency: node_modules/rollup-plugin-node-builtins/src/es6/readable-stream/duplex.js -> node_modules/rollup-plugin-node-builtins/src/es6/readable-stream/writable.js -> node_modules/rollup-plugin-node-builtins/src/es6/readable-stream/duplex.js

Is is a problem?

[calvinmetcalf]

shouldn't be one

[juanjoDiaz]

I'm seeing the same. It would be great to get rid of the warning.

[linusschwalbe]

I do believe the circular problem stems from how Nodes stream is constructed. Though it would be preferable to be able to supres this specific warning in the plugin.

[kopax]

I have the same warning

[nicobrinkkemper]

Did any of you solve this yet? I've got the same issue.

[ydaniv]

Changing order of plugins around causes this to appear/disappear.
Moving node-resolve and commonjs before node-globals and node-builtins in the order of plugins causes it to appear, and vice versa it disappears, but then moduleContext doesn't work properly.

[calvinmetcalf]

folks as far as I know this is only a warning and not an error so my official position is 'don't worry about it'

the actual reason for it is because duplex streams inherit from both readable and writable streams but for various reasons both readable and writable streams need access to Duplex to check if the instance they are operating on is descended from a duplex or not (for reasons that really seamed good at the time) but is done here in a way that works fine for es6 module rules so probably isn't something to worry about

[orar]

I get same with htmparser2. Its not just a warning. I get the following error afterwards when running

var DomHandler$1 = lib.DomHandler,
                               ^
TypeError: Cannot read property 'DomHandler' of undefined

[barry800414]

I have the same warning and got same errors as @orar while running :(

[fabien-h]

Same warning here.

When I switch the order to have 'node-resolve' first, I get this error :

`An error occurred while building your code: TypeError: Cannot read property 'warn' of undefined. And the code does not compile.

[fabien-h]

Probably linked to this :

rollup/rollup-plugin-commonjs#248

This is an internal issue with the custom stream implementation.