From a945976c982fece748d9ea7c04c1fc6b7a7c43ab Mon Sep 17 00:00:00 2001 From: Ramiro Silveyra d'Avila Date: Sat, 24 Dec 2016 02:13:50 -0300 Subject: [PATCH] Add support for nested package.json. Fix #458 --- CHANGELOG.md | 3 +++ docs/rules/no-extraneous-dependencies.md | 8 +++++++- src/rules/no-extraneous-dependencies.js | 17 ++++++++++++----- tests/src/rules/no-extraneous-dependencies.js | 14 ++++++++++++++ 4 files changed, 36 insertions(+), 6 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index e1ff0c4e70..c501d071ec 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -6,6 +6,7 @@ This change log adheres to standards from [Keep a CHANGELOG](http://keepachangel ## [Unreleased] ### Changed - [`no-extraneous-dependencies`]: use `read-pkg-up` to simplify finding + loading `package.json` ([#680], thanks [@wtgtybhertgeghgtwtg]) +- Add support for nested package.json [`no-extraneous-dependencies`]. ([#458], thanks [@ramasilveyra]) ### Fixed - attempt to fix crash in [`no-mutable-exports`]. ([#660]) @@ -395,6 +396,7 @@ for info on changes for earlier releases. [#489]: https://github.com/benmosher/eslint-plugin-import/pull/489 [#485]: https://github.com/benmosher/eslint-plugin-import/pull/485 [#461]: https://github.com/benmosher/eslint-plugin-import/pull/461 +[#458]: https://github.com/benmosher/eslint-plugin-import/issues/458 [#444]: https://github.com/benmosher/eslint-plugin-import/pull/444 [#428]: https://github.com/benmosher/eslint-plugin-import/pull/428 [#395]: https://github.com/benmosher/eslint-plugin-import/pull/395 @@ -561,3 +563,4 @@ for info on changes for earlier releases. [@ntdb]: https://github.com/ntdb [@jakubsta]: https://github.com/jakubsta [@wtgtybhertgeghgtwtg]: https://github.com/wtgtybhertgeghgtwtg +[@ramasilveyra]: https://github.com/ramasilveyra diff --git a/docs/rules/no-extraneous-dependencies.md b/docs/rules/no-extraneous-dependencies.md index adb532e390..2ce36cab0d 100644 --- a/docs/rules/no-extraneous-dependencies.md +++ b/docs/rules/no-extraneous-dependencies.md @@ -1,7 +1,7 @@ # Forbid the use of extraneous packages Forbid the import of external modules that are not declared in the `package.json`'s `dependencies`, `devDependencies`, `optionalDependencies` or `peerDependencies`. -The closest parent `package.json` will be used. If no `package.json` is found, the rule will not lint anything. +The closest parent `package.json` will be used. If no `package.json` is found, the rule will not lint anything. This behaviour can be changed with the rule option `packagePath`. ### Options @@ -27,6 +27,12 @@ You can also use an array of globs instead of literal booleans: When using an array of globs, the setting will be activated if the name of the file being linted matches a single glob in the array. +Also there is one more option called `packagePath`, this option is to specify the path of the package.json. + +```js +"import/no-extraneous-dependencies": ["error", {"packagePath": './package.json'}] +``` + ## Rule Details Given the following `package.json`: diff --git a/src/rules/no-extraneous-dependencies.js b/src/rules/no-extraneous-dependencies.js index 166d4f843b..e080e8c0a7 100644 --- a/src/rules/no-extraneous-dependencies.js +++ b/src/rules/no-extraneous-dependencies.js @@ -1,16 +1,22 @@ import path from 'path' +import fs from 'fs' import readPkgUp from 'read-pkg-up' import minimatch from 'minimatch' import importType from '../core/importType' import isStaticRequire from '../core/staticRequire' -function getDependencies(context) { +function getDependencies(context, packagePath) { try { - const pkg = readPkgUp.sync({cwd: context.getFilename(), normalize: false}) - if (!pkg || !pkg.pkg) { + const pkg = packagePath + ? JSON.parse(fs.readFileSync(packagePath, 'utf8')) + : readPkgUp.sync({cwd: context.getFilename(), normalize: false}).pkg + + if (!pkg) { return null } - const packageContent = pkg.pkg + + const packageContent = pkg + return { dependencies: packageContent.dependencies || {}, devDependencies: packageContent.devDependencies || {}, @@ -93,6 +99,7 @@ module.exports = { 'devDependencies': { 'type': ['boolean', 'array'] }, 'optionalDependencies': { 'type': ['boolean', 'array'] }, 'peerDependencies': { 'type': ['boolean', 'array'] }, + 'packagePath': { 'type': 'string' }, }, 'additionalProperties': false, }, @@ -102,7 +109,7 @@ module.exports = { create: function (context) { const options = context.options[0] || {} const filename = context.getFilename() - const deps = getDependencies(context) + const deps = getDependencies(context, options.packagePath) if (!deps) { return {} diff --git a/tests/src/rules/no-extraneous-dependencies.js b/tests/src/rules/no-extraneous-dependencies.js index 8bb632d017..b4883fe083 100644 --- a/tests/src/rules/no-extraneous-dependencies.js +++ b/tests/src/rules/no-extraneous-dependencies.js @@ -56,6 +56,11 @@ ruleTester.run('no-extraneous-dependencies', rule, { filename: path.join(process.cwd(), 'foo.spec.js'), }), test({ code: 'require(6)' }), + + test({ + code: 'import "doctrine"', + options: [{packagePath: path.join(__dirname, '../../../package.json')}], + }), ], invalid: [ test({ @@ -154,5 +159,14 @@ ruleTester.run('no-extraneous-dependencies', rule, { message: '\'lodash.isarray\' should be listed in the project\'s dependencies, not optionalDependencies.', }], }), + + test({ + code: 'import "not-a-dependency"', + options: [{packagePath: path.join(__dirname, '../../../package.json')}], + errors: [{ + ruleId: 'no-extraneous-dependencies', + message: '\'not-a-dependency\' should be listed in the project\'s dependencies. Run \'npm i -S not-a-dependency\' to add it', + }], + }), ], })