ensure NODE_PATH environment variable is honored when checking for extraneous dependencies

Author: matoilic

packages/eslint-config-react-app/index.js

@@ -9,6 +9,8 @@
 
 'use strict';
 
+const resolveNodePath = require('./utils/resolveNodePath');
+
 // Inspired by https://github.com/airbnb/javascript but less opinionated.
 
 // We use eslint-loader so even warnings are very visible.
@@ -49,6 +51,7 @@ module.exports = {
     'import/resolver': {
       node: {
         extensions: ['.js', '.json'],
+        moduleDirectory: ['node_modules'].concat(resolveNodePath()),
       },
     },
   },

packages/eslint-config-react-app/package.json

@@ -8,6 +8,7 @@
     "url": "https://github.com/facebookincubator/create-react-app/issues"
   },
   "files": [
+    "utils",
     "index.js"
   ],
   "peerDependencies": {

packages/eslint-config-react-app/utils/resolveNodePath.js

@@ -0,0 +1,11 @@
+'use strict';
+
+function resolveNodePath() {
+  const nodePaths = (process.env.NODE_PATH || '')
+    .split(process.platform === 'win32' ? ';' : ':')
+    .filter(Boolean);
+
+  return nodePaths;
+}
+
+module.exports = resolveNodePath;