How to configure with `eslint.config.js` with ESM?
Can you please add information in the README on how to set up the plugin and config when using flat config in ESLint (eslint.config.js) with ESM?
PRs are welcome, but i don’t use flat config yet.
~~My understanding is that eslint-plugin-react isn't yet compatible with the new flat config format: #3688~~
Correction: the new format is already supported!
Indeed, but eslint has FlatCompat for that.
Ah cool! So could this issue could be resolved by docs explaining how to use FlatCompat with eslint-plugin-react?
That would be a fine interim solution, yes - altho we'd just link to eslint's explanation, we don't need any prose here.
I realized that the Configuration (new: eslint.config.js) section in the readme covers setup with flat files.
@rakleed, are you looking for a version of this documentation that uses ESM instead of CommonJS? Or is this documentation missing some other information that you need?
@rakleed, are you looking for a version of this documentation that uses ESM instead of CommonJS?
Yes, exactly.
Ah, thanks for clearing that up! An ESM version of that documentation would replace require and module.exports with import and export default, like this:
import reactRecommended from 'eslint-plugin-react/configs/recommended';
export default [
…
reactRecommended, // This is not a plugin object, but a shareable config object
…
];
A readme a with two versions of each example would be helpful for folks who are using ESM, but it could be difficult to read and maintain. Perhaps the readme could include a single ESM example, or a link to more details about the difference between ESM and CommonJS?
I think that if translating the basic CJS example to ESM in your head requires documentation, then using native ESM is probably a bit premature.
@rdebeasi thanks, this works for most cases, although you need to add the .js extension, which is strange since all other configs with ESLint Flat Config support do not have such a requirement.
But in my case it doesn't work. I'm using the Airbnb config for React, which doesn't support Flat Config, so I use the FlatCompat utility. But when I then add reactJsxRuntime, the console displays the error Error: Key "plugins": Cannot redefine plugin "react".
Here is my example:
import reactJsxRuntime from 'eslint-plugin-react/configs/jsx-runtime.js';
export default [
…
...compat.extends('airbnb', 'airbnb/hooks'),
reactJsxRuntime,
…
];
Oh interesting! That sounds like a different problem from what's described in this issue. If you no longer need the readme update, consider closing this issue.
If you'd like to dig into that issue further, you could:
- Try the same thing with
.eslintrcto determine whether this issue occurs only when using flat files - Create a minimal, reproducible example of the problem
- File a new ticket with that example and steps to reproduce
Caveat: I'm not associated with the eslint-pluign-react project and I don't know much about this codebase. These are just some general troubleshooting steps that I use! 😄
Am I missing something? I see in docs there are flat configs exported. But I cannot find the flat configs that in the actual code.
@GerroDen be sure you're looking at the docs for the last released tag on github - everywhere on github, things on main might be unreleased.
The plugin lacks exports definitions in package.json, which is why we see errors with some Typescript or ESM configs. Example of what it looks like in eslint-plugin-prettier.
@magne4000 since adding exports is a breaking change, it can't be added within v7. Separately, there shouldn't be any need for exports definitions - without them, everything is accessible. If you mean that native ESM requires specifying file extensions, that's sadly true, and exports would indeed be able to omit them.
The latest version (v7.35.0) supports eslint 9 and flat config.
Please file a new issue if you're having trouble with that version.