config-inspector icon indicating copy to clipboard operation
config-inspector copied to clipboard

Warn me when I use `module.exports.default`

Open mmkal opened this issue 1 year ago • 2 comments

👋 small request here that could have saved me a bunch of time debugging. When I accidentally do module.exports.default = myConfig, the config inspector will tell me what a great config I have and how it applies loads of interesting rules to all kinds of different files. But running npx eslint (with v8.57.1) just tells me everything is ignored.

By adding lots of console.logs to eslint in my node_modules, I was able to see that one of my configs looked like { default: [...] } and that told me what to fix, but it would have been handy if the config inspector had said the config looked bad - or at least didn't tell me it was working perfectly.

mmkal avatar Sep 16 '24 18:09 mmkal

My guess is that this is due to ESM / CJS compatibility stuff that differs between the implementations

I wonder where that difference is

voxpelli avatar Sep 16 '24 22:09 voxpelli

Yeah, maybe this library is doing a if (config.default) config = config.default check somewhere and eslint isn't.

mmkal avatar Sep 17 '24 11:09 mmkal