esbuild-plugin-less icon indicating copy to clipboard operation
esbuild-plugin-less copied to clipboard

Recursive imports crashes

Open zouloux opened this issue 1 year ago • 0 comments

Hi :) Thanks for this plugin. I encountered this little bug on a project with circular dependencies. I assume the issue is in the recursive in the reduce, we may need a check if this dependency has already been discovered, and break the loop.

Expected Behavior

Build should not crash when 2 less files imports each others.

Actual Behavior

An infinite loop in starts in this code https://github.com/iam-medvedev/esbuild-plugin-less/blob/main/src/less-utils.ts

Steps to Reproduce the Problem

a.less imports b.less, b.less imports a.less It is a real use case when we load other less files as "reference" to have all exported mixins and variables, without importing the raw CSS from it.

Versions

  • esbuild: 0.19.10
  • esbuild-plugin-less: 1.3.1
  • Platform: Node / MacOS

zouloux avatar Jan 07 '24 14:01 zouloux