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

How to configure with `eslint.config.js` with ESM?

Open rakleed opened this issue 2 years ago • 11 comments

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?

rakleed avatar Feb 02 '24 13:02 rakleed

PRs are welcome, but i don’t use flat config yet.

ljharb avatar Feb 02 '24 15:02 ljharb

~~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!

rdebeasi avatar Feb 08 '24 18:02 rdebeasi

Indeed, but eslint has FlatCompat for that.

ljharb avatar Feb 08 '24 19:02 ljharb

Ah cool! So could this issue could be resolved by docs explaining how to use FlatCompat with eslint-plugin-react?

rdebeasi avatar Feb 08 '24 19:02 rdebeasi

That would be a fine interim solution, yes - altho we'd just link to eslint's explanation, we don't need any prose here.

ljharb avatar Feb 08 '24 19:02 ljharb

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?

rdebeasi avatar Feb 09 '24 21:02 rdebeasi

@rakleed, are you looking for a version of this documentation that uses ESM instead of CommonJS?

Yes, exactly.

rakleed avatar Feb 09 '24 21:02 rakleed

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?

rdebeasi avatar Feb 09 '24 21:02 rdebeasi

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.

ljharb avatar Feb 09 '24 21:02 ljharb

@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,
  …
];

rakleed avatar Feb 13 '24 17:02 rakleed

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 .eslintrc to 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! 😄

rdebeasi avatar Feb 14 '24 14:02 rdebeasi

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 avatar Jul 18 '24 11:07 GerroDen

@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.

ljharb avatar Jul 18 '24 16:07 ljharb

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 avatar Jul 22 '24 17:07 magne4000

@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.

ljharb avatar Jul 22 '24 17:07 ljharb

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.

ljharb avatar Jul 22 '24 17:07 ljharb