eslint-webpack-plugin icon indicating copy to clipboard operation
eslint-webpack-plugin copied to clipboard

fix(plugin): isolate ESLint instance per webpack config in non-thread…

Open AhNosNimes opened this issue 5 months ago • 1 comments
trafficstars

Hello

Avoid reusing global ESLint instances across multiple webpack configurations. This change corrects issues with incorrect cwd and context resolution caused by shared state between configurations. Note: Threaded mode remains broken due to fundamental architectural flaws and would require a full rewrite to function correctly.

Fixes #286

This PR contains a:

  • [ ] bugfix

Motivation / Use-Case

This fix addresses the use case of multiple webpack configurations where the previous implementation shares the same incorrect eslint instance.

Breaking Changes

Threaded mode is currently broken and now considered unsupported.

Additional Info

This PR is not meant to be merged as-is. It serves as a proof of concept or inspiration. The threaded mode is fundamentally flawed: setup is invoked multiple times for the same worker, and most operations are incorrectly executed on the main thread instead of worker threads. Fixing this would require a complete architectural rewrite, which I am not undertaking, as I do not use threaded mode myself. The current implementation leads to redundant and duplicated work, making it non-viable in practice.

Have a nice day

AhNosNimes avatar Jun 17 '25 13:06 AhNosNimes