5793301c60
~(PR https://github.com/jsx-eslint/eslint-plugin-react/pull/3377) introduced a change in `eslint-plugin-react@7.31.2` that will now show an error when unknown properties appear on elements. We can opt out of this by overriding the default.~ As discussed internally, we are turning `react/no-unknown-property` off, as it might be confusing even if different props are being used, (eg.: `css` for `emotion`). It's easy to fix https://github.com/jsx-eslint/eslint-plugin-react/blob/master/docs/rules/no-unknown-property.md#rule-options, but it might not be clear at first glance that Next.js is using `eslint-plugin-react` internally. If the user wants to enforce this rule, they can still add it to their own `rules` config. Fixes #40321, ref: https://github.com/vercel/next.js/discussions/40269, https://github.com/vercel/next.js/issues/38333
125 lines
3.4 KiB
JavaScript
125 lines
3.4 KiB
JavaScript
/*
|
|
* @rushstack/eslint-patch is used to include plugins as dev
|
|
* dependencies instead of imposing them as peer dependencies
|
|
*
|
|
* https://www.npmjs.com/package/@rushstack/eslint-patch
|
|
*/
|
|
const keptPaths = []
|
|
const sortedPaths = []
|
|
const cwd = process.cwd().replace(/\\/g, '/')
|
|
const originalPaths = require.resolve.paths('eslint-plugin-import')
|
|
|
|
// eslint throws a conflict error when plugins resolve to different
|
|
// locations, since we want to lock our dependencies by default
|
|
// but also need to allow using user dependencies this updates
|
|
// our resolve paths to first check the cwd and iterate to
|
|
// eslint-config-next's dependencies if needed
|
|
|
|
for (let i = originalPaths.length - 1; i >= 0; i--) {
|
|
const currentPath = originalPaths[i]
|
|
|
|
if (currentPath.replace(/\\/g, '/').startsWith(cwd)) {
|
|
sortedPaths.push(currentPath)
|
|
} else {
|
|
keptPaths.unshift(currentPath)
|
|
}
|
|
}
|
|
|
|
// maintain order of node_modules outside of cwd
|
|
sortedPaths.push(...keptPaths)
|
|
|
|
const hookPropertyMap = new Map(
|
|
[
|
|
['eslint-plugin-import', 'eslint-plugin-import'],
|
|
['eslint-plugin-react', 'eslint-plugin-react'],
|
|
['eslint-plugin-jsx-a11y', 'eslint-plugin-jsx-a11y'],
|
|
].map(([request, replacement]) => [
|
|
request,
|
|
require.resolve(replacement, { paths: sortedPaths }),
|
|
])
|
|
)
|
|
|
|
const mod = require('module')
|
|
const resolveFilename = mod._resolveFilename
|
|
mod._resolveFilename = function (request, parent, isMain, options) {
|
|
const hookResolved = hookPropertyMap.get(request)
|
|
if (hookResolved) {
|
|
request = hookResolved
|
|
}
|
|
return resolveFilename.call(mod, request, parent, isMain, options)
|
|
}
|
|
|
|
require('@rushstack/eslint-patch/modern-module-resolution')
|
|
|
|
module.exports = {
|
|
extends: [
|
|
'plugin:react/recommended',
|
|
'plugin:react-hooks/recommended',
|
|
'plugin:@next/next/recommended',
|
|
],
|
|
plugins: ['import', 'react', 'jsx-a11y'],
|
|
rules: {
|
|
'import/no-anonymous-default-export': 'warn',
|
|
'react/no-unknown-property': 'off',
|
|
'react/react-in-jsx-scope': 'off',
|
|
'react/prop-types': 'off',
|
|
'jsx-a11y/alt-text': [
|
|
'warn',
|
|
{
|
|
elements: ['img'],
|
|
img: ['Image'],
|
|
},
|
|
],
|
|
'jsx-a11y/aria-props': 'warn',
|
|
'jsx-a11y/aria-proptypes': 'warn',
|
|
'jsx-a11y/aria-unsupported-elements': 'warn',
|
|
'jsx-a11y/role-has-required-aria-props': 'warn',
|
|
'jsx-a11y/role-supports-aria-props': 'warn',
|
|
},
|
|
parser: './parser.js',
|
|
parserOptions: {
|
|
requireConfigFile: false,
|
|
sourceType: 'module',
|
|
allowImportExportEverywhere: true,
|
|
babelOptions: {
|
|
presets: ['next/babel'],
|
|
caller: {
|
|
// Eslint supports top level await when a parser for it is included. We enable the parser by default for Babel.
|
|
supportsTopLevelAwait: true,
|
|
},
|
|
},
|
|
},
|
|
overrides: [
|
|
{
|
|
files: ['**/*.ts?(x)'],
|
|
parser: '@typescript-eslint/parser',
|
|
parserOptions: {
|
|
sourceType: 'module',
|
|
ecmaFeatures: {
|
|
jsx: true,
|
|
},
|
|
warnOnUnsupportedTypeScriptVersion: true,
|
|
},
|
|
},
|
|
],
|
|
settings: {
|
|
react: {
|
|
version: 'detect',
|
|
},
|
|
'import/parsers': {
|
|
[require.resolve('@typescript-eslint/parser')]: ['.ts', '.tsx', '.d.ts'],
|
|
},
|
|
'import/resolver': {
|
|
[require.resolve('eslint-import-resolver-node')]: {
|
|
extensions: ['.js', '.jsx', '.ts', '.tsx'],
|
|
},
|
|
[require.resolve('eslint-import-resolver-typescript')]: {
|
|
alwaysTryTypes: true,
|
|
},
|
|
},
|
|
},
|
|
env: {
|
|
browser: true,
|
|
node: true,
|
|
},
|
|
}
|