[New] : Add option

Author: luxaritas

CHANGELOG.md

@@ -17,6 +17,7 @@ This change log adheres to standards from [Keep a CHANGELOG](https://keepachange
 - [`order`]: new `alphabetize.orderImportKind` option to sort imports with same path based on their kind (`type`, `typeof`) ([#2544], thanks [@stropho])
 - [`consistent-type-specifier-style`]: add rule ([#2473], thanks [@bradzacher])
 - Add [`no-empty-named-blocks`] rule ([#2568], thanks [@guilhermelimak])
+- [`no-extraneous-dependencies`]: Add `considerInParents` option to support package.json files in parent folders ([#2481], thanks [@luxaritas])
 
 ### Fixed
 - [`order`]: move nested imports closer to main import entry ([#2396], thanks [@pri1311])
@@ -1028,6 +1029,7 @@ for info on changes for earlier releases.
 [#2506]: https://github.com/import-js/eslint-plugin-import/pull/2506
 [#2503]: https://github.com/import-js/eslint-plugin-import/pull/2503
 [#2490]: https://github.com/import-js/eslint-plugin-import/pull/2490
+[#2481]: https://github.com/import-js/eslint-plugin-import/pull/2481
 [#2473]: https://github.com/import-js/eslint-plugin-import/pull/2473
 [#2466]: https://github.com/import-js/eslint-plugin-import/pull/2466
 [#2459]: https://github.com/import-js/eslint-plugin-import/pull/2459
@@ -1672,6 +1674,7 @@ for info on changes for earlier releases.
 [@ludofischer]: https://github.com/ludofischer
 [@Lukas-Kullmann]: https://github.com/Lukas-Kullmann
 [@lukeapage]: https://github.com/lukeapage
+[@luxaritas]: https://github.com/luxaritas
 [@lydell]: https://github.com/lydell
 [@magarcia]: https://github.com/magarcia
 [@Mairu]: https://github.com/Mairu

docs/rules/no-extraneous-dependencies.md

@@ -37,7 +37,16 @@ There are 2 boolean options to opt into checking extra imports that are normally
 "import/no-extraneous-dependencies": ["error", {"includeInternal": true, "includeTypes": true}]
 ```
 
-Also there is one more option called `packageDir`, this option is to specify the path to the folder containing package.json.
+To take `package.json` files in parent directories into account, use the `considerInParents` option.
+This takes an array of strings, which can include `"prod"`, `"dev"`, `"peer"`, `"optional"`, `"bundled"`.
+
+For example, the following would allow imports when the relevant packages are found in either `dependencies`
+or `devDependencies` in either the closest `package.json` or any `package.json` found in parent directories:
+```js
+"import/no-extraneous-dependencies": ["error", {"considerInParents": ["prod", "dev"]}]
+```
+
+To specify the path to the folder containing package.json, use the `packageDir` option.
 
 If provided as a relative path string, will be computed relative to the current working directory at linter execution time. If this is not ideal (does not work with some editor integrations), consider using `__dirname` to provide a path relative to your configuration.
 

src/rules/no-extraneous-dependencies.js

@@ -1,9 +1,9 @@
 import path from 'path';
 import fs from 'fs';
-import pkgUp from 'eslint-module-utils/pkgUp';
 import minimatch from 'minimatch';
 import resolve from 'eslint-module-utils/resolve';
 import moduleVisitor from 'eslint-module-utils/moduleVisitor';
+import includes from 'array-includes';
 import importType from '../core/importType';
 import { getFilePackageName } from '../core/packagePath';
 import docsUrl from '../docsUrl';
@@ -40,16 +40,41 @@ function extractDepFields(pkg) {
   };
 }
 
-function getPackageDepFields(packageJsonPath, throwAtRead) {
-  if (!depFieldCache.has(packageJsonPath)) {
-    const depFields = extractDepFields(readJSON(packageJsonPath, throwAtRead));
-    depFieldCache.set(packageJsonPath, depFields);
+function getPackageDepFields(packageDir, considerInParents, requireInDir) {
+  const cacheKey = JSON.stringify({ packageDir, considerInParents });
+
+  if (!depFieldCache.has(cacheKey)) {
+    // try in the current directory, erroring if the user explicitly specified this directory
+    // and reading fails
+    const parsedPackage = readJSON(path.join(packageDir, 'package.json'), requireInDir);
+    const depFields = extractDepFields(parsedPackage || {});
+
+    // If readJSON returned nothing, we want to keep searching since the current directory didn't
+    // have a package.json. Also keep searching if we're merging in some set of parents dependencies.
+    // However, if we're already at the root, stop.
+    if ((!parsedPackage || considerInParents.length > 0) && packageDir !== path.parse(packageDir).root) {
+      const parentDepFields = getPackageDepFields(path.dirname(packageDir), considerInParents, false);
+
+      Object.keys(depFields).forEach(depsKey => {
+        if (
+          (depsKey === 'dependencies' && includes(considerInParents, 'prod') )||
+          (depsKey === 'devDependencies' && includes(considerInParents, 'dev')) ||
+          (depsKey === 'peerDependencies' && includes(considerInParents, 'peer')) ||
+          (depsKey === 'optionalDependencies' && includes(considerInParents, 'optional')) ||
+          (depsKey === 'bundledDependencies' && includes(considerInParents, 'bundled'))
+        ) {
+          Object.assign(depFields[depsKey], parentDepFields[depsKey]);
+        }
+      });
+    }
+
+    depFieldCache.set(cacheKey, depFields);
   }
 
-  return depFieldCache.get(packageJsonPath);
+  return depFieldCache.get(cacheKey);
 }
 
-function getDependencies(context, packageDir) {
+function getDependencies(context, packageDir, considerInParents) {
   let paths = [];
   try {
     const packageContent = {
@@ -71,22 +96,20 @@ function getDependencies(context, packageDir) {
     if (paths.length > 0) {
       // use rule config to find package.json
       paths.forEach(dir => {
-        const packageJsonPath = path.join(dir, 'package.json');
-        const _packageContent = getPackageDepFields(packageJsonPath, true);
+        const _packageContent = getPackageDepFields(dir, considerInParents, true);
         Object.keys(packageContent).forEach(depsKey =>
           Object.assign(packageContent[depsKey], _packageContent[depsKey]),
         );
       });
     } else {
-      const packageJsonPath = pkgUp({
-        cwd: context.getPhysicalFilename ? context.getPhysicalFilename() : context.getFilename(),
-        normalize: false,
-      });
-
       // use closest package.json
       Object.assign(
         packageContent,
-        getPackageDepFields(packageJsonPath, false),
+        getPackageDepFields(
+          path.dirname(context.getPhysicalFilename ? context.getPhysicalFilename() : context.getFilename()),
+          considerInParents,
+          false,
+        ),
       );
     }
 
@@ -268,6 +291,7 @@ module.exports = {
           'packageDir': { 'type': ['string', 'array'] },
           'includeInternal': { 'type': ['boolean'] },
           'includeTypes': { 'type': ['boolean'] },
+          'considerInParents': { 'type': ['array'] },
         },
         'additionalProperties': false,
       },
@@ -277,7 +301,7 @@ module.exports = {
   create(context) {
     const options = context.options[0] || {};
     const filename = context.getPhysicalFilename ? context.getPhysicalFilename() : context.getFilename();
-    const deps = getDependencies(context, options.packageDir) || extractDepFields({});
+    const deps = getDependencies(context, options.packageDir, options.considerInParents || []) || extractDepFields({});
 
     const depsOptions = {
       allowDevDeps: testConfig(options.devDependencies, filename) !== false,

tests/src/rules/no-extraneous-dependencies.js

@@ -116,6 +116,18 @@ ruleTester.run('no-extraneous-dependencies', rule, {
       code: 'import rightpad from "right-pad";',
       options: [{ packageDir: [packageDirMonoRepoRoot, packageDirMonoRepoWithNested] }],
     }),
+    test({
+      code: 'import leftpad from "left-pad";',
+      options: [{ packageDir: packageDirMonoRepoWithNested, considerInParents: ['prod', 'dev'] }],
+    }),
+    test({
+      code: 'import rightpad from "right-pad";',
+      options: [{ packageDir: packageDirMonoRepoWithNested, considerInParents: ['prod', 'dev'] }],
+    }),
+    test({
+      code: 'import leftpad from "left-pad";',
+      options: [{ packageDir: packageDirMonoRepoWithNested, considerInParents: ['dev'] }],
+    }),
     test({ code: 'import foo from "@generated/foo"' }),
     test({
       code: 'import foo from "@generated/foo"',
@@ -319,6 +331,13 @@ ruleTester.run('no-extraneous-dependencies', rule, {
         message: "'left-pad' should be listed in the project's dependencies. Run 'npm i -S left-pad' to add it",
       }],
     }),
+    test({
+      code: 'import leftpad from "left-pad";',
+      options: [{ packageDir: packageDirMonoRepoWithNested, considerInParents: ['prod'] }],
+      errors: [{
+        message: "'left-pad' should be listed in the project's dependencies. Run 'npm i -S left-pad' to add it",
+      }],
+    }),
     test({
       code: 'import react from "react";',
       filename: path.join(packageDirMonoRepoRoot, 'foo.js'),
@@ -438,7 +457,7 @@ describe('TypeScript', () => {
 
           test(Object.assign({
             code: 'import type T from "a";',
-            options: [{ 
+            options: [{
               packageDir: packageDirWithTypescriptDevDependencies,
               devDependencies: false,
               includeTypes: true,