Add support for import() expressions

src/cli.js

@@ -89,7 +89,7 @@ export default function cli(args, log, error, exit) {
     .describe('json', 'Output results to JSON')
     .describe('ignores', 'Comma separated package list to ignore')
     .describe('ignore-dirs', 'Comma separated folder names to ignore')
-    .describe('parsers', 'Comma separated glob:pasers pair list')
+    .describe('parsers', 'Comma separated glob:parser pair list')
     .describe('detectors', 'Comma separated detector list')
     .describe('specials', 'Comma separated special parser list')
     .version('version', 'Show version number', version)

src/component.json

@@ -11,6 +11,7 @@
   "detector": [
     "expressViewEngine",
     "gruntLoadTaskCallExpression",
+    "importCallExpression",
     "importDeclaration",
     "requireCallExpression",
     "requireResolveCallExpression"

src/constants.js

@@ -46,6 +46,7 @@ export const defaultOptions = {
     availableDetectors.importDeclaration,
     availableDetectors.requireCallExpression,
     availableDetectors.requireResolveCallExpression,
+    availableDetectors.importCallExpression,
     availableDetectors.gruntLoadTaskCallExpression,
   ],
   specials: lodash.values(availableSpecials),

src/detector/importCallExpression.js

@@ -0,0 +1,23 @@
+import lodash from 'lodash';
+
+
+export default function importCallExpression(node) {
+  return (
+      node.type === 'CallExpression' &&
+      node.callee &&
+
+      (
+        ((node.callee.type === 'Identifier' && node.callee.name === 'import') ||
+         (node.callee.type === 'Import')) ||
+        (node.callee.type === 'MemberExpression' &&
+         node.callee.object &&
+         node.callee.object.name === 'System' &&
+         node.callee.property &&
+         node.callee.property.name === 'import')
+      ) &&
+      node.arguments[0] &&
+      lodash.isString(node.arguments[0].value)
+        ? [node.arguments[0].value]
+        : []
+  );
+}

test/fake_modules/import_function/index.js

@@ -0,0 +1 @@
+import('optimist');

test/fake_modules/import_function/package.json

@@ -0,0 +1,5 @@
+{
+  "dependencies":{
+    "optimist": "~0.6.0"
+  }
+}

test/fake_modules/import_function_missing/index.js

@@ -0,0 +1 @@
+import('anyone');

test/fake_modules/import_function_missing/package.json

@@ -0,0 +1 @@
+{}

test/fake_modules/import_function_webpack/index.js

@@ -0,0 +1 @@
+import(/* webpackChunkName: 'chunk' */ 'optimist');

test/fake_modules/import_function_webpack/package.json

@@ -0,0 +1,5 @@
+{
+  "dependencies":{
+    "optimist": "~0.6.0"
+  }
+}

test/spec.js

@@ -1,6 +1,53 @@
 import depcheck from '../src/index';
 
 export default [
+  {
+    name: 'detect missing module for dynamic import() when missing in package.json',
+    module: 'import_function_missing',
+    options: {
+      withoutDev: true,
+    },
+    expected: {
+      dependencies: [],
+      devDependencies: [],
+      missing: {
+        anyone: ['index.js'],
+      },
+      using: {
+        anyone: ['index.js'],
+      },
+    },
+  },
+  {
+    name: 'find module for dynamic import() when present',
+    module: 'import_function',
+    options: {
+      withoutDev: true,
+    },
+    expected: {
+      dependencies: [],
+      devDependencies: [],
+      missing: {},
+      using: {
+        optimist: ['index.js'],
+      },
+    },
+  },
+  {
+    name: 'find module for dynamic import() with magic Webpack comment',
+    module: 'import_function_webpack',
+    options: {
+      withoutDev: true,
+    },
+    expected: {
+      dependencies: [],
+      devDependencies: [],
+      missing: {},
+      using: {
+        optimist: ['index.js'],
+      },
+    },
+  },
   {
     name: 'missing module for require.resolve when missing in package.json',
     module: 'require_resolve_missing',