Add rootDir setting to eslint-plugin-next (#27918)
## Introduction
This PR enables setting a `rootDir` for a Next.js project, and follows the same pattern used by [`@typescript-eslint/parser`](https://github.com/typescript-eslint/typescript-eslint/tree/master/packages/parser#parseroptionsproject).
## Details
Previously, users had to pass paths to the rule itself.
```js
module.exports = {
rules: {
"@next/next/no-html-link-for-pages": [
"error",
// This could be a string, or array of strings.
"/packages/my-app/pages",
],
},
};
```
With this PR, this has been simplified (the previous implementation still works as expected).
```js
module.exports = {
settings: {
next: {
rootDir: "/packages/my-app",
},
},
rules: {
"@next/next/no-html-link-for-pages": "error",
},
};
```
Further, this rule allows the use of globs, again aligning with `@typescript-eslint/parser`.
```js
module.exports = {
settings: {
next: {
// Globs
rootDir: "/packages/*",
rootDir: "/packages/{app-a,app-b}",
// Arrays
rootDir: ["/app-a", "/app-b"],
// Arrays with globs
rootDir: ["/main-app", "/other-apps/*"],
},
};
```
This enables users to either provide per-workspace configuration with overrides, or to use globs for situations like monorepos where the apps share a domain (micro-frontends).
This doesn't solve, but improves https://github.com/vercel/next.js/issues/26330.
## Feature
- [x] Implements an existing feature request or RFC. Make sure the feature request has been accepted for implementation before opening a PR.
- [x] Related issues linked using `fixes #number`
- [ ] Integration tests added
- [ ] Documentation added
- [ ] Telemetry added. In case of a feature if it's used or not.
- [ ] Errors have helpful link attached, see `contributing.md`
## Documentation / Examples
- [x] Make sure the linting passes
2021-08-11 12:37:55 +02:00
|
|
|
const NodeAttributes = require('../utils/node-attributes.js')
|
2021-05-10 20:35:11 +02:00
|
|
|
|
|
|
|
module.exports = {
|
|
|
|
meta: {
|
|
|
|
docs: {
|
|
|
|
description:
|
|
|
|
'Ensure passHref is assigned if child of Link component is a custom component',
|
|
|
|
category: 'HTML',
|
|
|
|
recommended: true,
|
2021-10-16 08:16:41 +02:00
|
|
|
url: 'https://nextjs.org/docs/messages/link-passhref',
|
2021-05-10 20:35:11 +02:00
|
|
|
},
|
|
|
|
fixable: null,
|
|
|
|
},
|
|
|
|
|
|
|
|
create: function (context) {
|
|
|
|
let linkImport = null
|
|
|
|
|
|
|
|
return {
|
|
|
|
ImportDeclaration(node) {
|
|
|
|
if (node.source.value === 'next/link') {
|
|
|
|
linkImport = node.specifiers[0].local.name
|
|
|
|
}
|
|
|
|
},
|
|
|
|
|
|
|
|
JSXOpeningElement(node) {
|
|
|
|
if (node.name.name !== 'Link' || node.name.name !== linkImport) {
|
|
|
|
return
|
|
|
|
}
|
|
|
|
|
|
|
|
const attributes = new NodeAttributes(node)
|
|
|
|
const children = node.parent.children
|
|
|
|
|
|
|
|
if (
|
|
|
|
!attributes.hasAny() ||
|
|
|
|
!attributes.has('href') ||
|
|
|
|
!children.some((attr) => attr.type === 'JSXElement')
|
|
|
|
) {
|
|
|
|
return
|
|
|
|
}
|
|
|
|
|
|
|
|
const hasPassHref =
|
|
|
|
attributes.has('passHref') &&
|
|
|
|
(typeof attributes.value('passHref') === 'undefined' ||
|
|
|
|
attributes.value('passHref') === true)
|
|
|
|
|
|
|
|
const hasAnchorChild = children.some(
|
|
|
|
(attr) =>
|
2021-06-18 15:17:53 +02:00
|
|
|
attr.type === 'JSXElement' && attr.openingElement.name.name === 'a'
|
2021-05-10 20:35:11 +02:00
|
|
|
)
|
|
|
|
|
|
|
|
if (!hasAnchorChild && !hasPassHref) {
|
|
|
|
context.report({
|
|
|
|
node,
|
|
|
|
message: `passHref ${
|
|
|
|
attributes.value('passHref') !== true
|
|
|
|
? 'must be set to true'
|
|
|
|
: 'is missing'
|
2021-06-21 00:55:03 +02:00
|
|
|
}. See https://nextjs.org/docs/messages/link-passhref`,
|
2021-05-10 20:35:11 +02:00
|
|
|
})
|
|
|
|
}
|
|
|
|
},
|
|
|
|
}
|
|
|
|
},
|
|
|
|
}
|