fixes module resolver and adds a test

package.json

@@ -22,6 +22,7 @@
     "enhanced-resolve": "^4.1.1",
     "express": "^4.17.1",
     "lodash": "^4.17.15",
+    "lodash.get": "^4.4.2",
     "moment": "^2.24.0",
     "mustache-express": "^1.3.0",
     "opn": "^6.0.0",

src/handlers/fileResolver.js

@@ -1,3 +1,4 @@
+const get = require("lodash.get");
 const path = require("path");
 const enhancedResolve = require("enhanced-resolve");
 const { defaultParsableExtensions } = require("../models/Parsables");
@@ -10,21 +11,29 @@ function checkNodeModules(absPath) {
 }
 
 function checkBabelModuleResolver(filePath, currentFile) {
-  const loadedConfig = findBabelConfig.sync(currentFile);
-  if (loadedConfig) {
-    // find module-resolver configuration
-    const moduleResolverConfig = loadedConfig.config.plugins.find(element => {
-      if (Array.isArray(element)) {
-        if (element[0] == "module-resolver") {
-          return element;
+  // path doesn't start with a '.' or with a '/' which should be local dependencies
+  if (!filePath.match(/^(\.|\/)/)) {
+    const loadedConfig = findBabelConfig.sync(currentFile);
+    if (get(loadedConfig, "config.plugins")) {
+      // find module-resolver configuration
+      const moduleResolverConfig = loadedConfig.config.plugins.find(element => {
+        if (Array.isArray(element)) {
+          if (element[0] == "module-resolver") {
+            return element;
+          }
         }
+      });
+      const opts = moduleResolverConfig[1];
+      // convert root to absolute path, otherwise module resolver won't work properly
+      opts.root = path.join(path.dirname(loadedConfig.file), opts.root[0]);
+      const currentFileDirectory = path.dirname(currentFile);
+      const resolved = moduleResolver.resolvePath(filePath, currentFile, opts);
+      let output = path.join(currentFileDirectory, resolved);
+      if (!output.match(/\.js$/)) {
+        output += ".js";
       }
-    });
-    const opts = moduleResolverConfig[1];
-    const currentFileDirectory = path.dirname(currentFile);
-    const resolved = moduleResolver.resolvePath(filePath, currentFile, opts);
-    const output = path.join(currentFileDirectory, resolved);
-    return output;
+      return output;
+    }
   }
 }
 // by default resolve Parsable files even without .ext in import
@@ -39,10 +48,6 @@ const resolveSync = enhancedResolve.create.sync({
  */
 function resolve(relPath = "", currentFile = "") {
   const currentDir = path.dirname(currentFile);
-  if (!relPath.match(/^(\.|\/)/)) {
-    // path doesn't start with a '.' or with a '/' which should be local dependencies
-    relPath = checkBabelModuleResolver(relPath, currentFile);
-  }
 
   try {
     const absPath = resolveSync(currentDir, relPath);
@@ -53,6 +58,11 @@ function resolve(relPath = "", currentFile = "") {
   } catch (err) {
     if (!err.missing) return;
 
+    // Check if it's resolvable by babel-module-resolver
+    const absPath = checkBabelModuleResolver(relPath, currentFile);
+    if (absPath) {
+      return { resolved: { absPath } };
+    }
     const notFound = checkNodeModules(relPath)
       ? relPath
       : path.resolve(currentDir, relPath);

src/index.js

@@ -7,15 +7,14 @@ const WriteReportFile = require("./cli/WriteReportFile");
 const ReportServer = require("./reportServer/server");
 class DeadFile {
   constructor(argv) {
-    const { _: entry, dir, exclude = [], output } = argv;
-
+    const { _: entry, dir, exclude = [], output, report } = argv;
     this.baseDir = this.dirToAbs(dir); // if dir is not absolute find based on pwd
     this.entry = this.entryToAbs(entry, dir); // if the entry is not absolute find the absolute
     this.exclude = exclude; // excluded files/folders
     this.output = output;
-
+    this.shouldReport = !(report == false);
     this.allAssets = AllAssets(dir, { exclude }); // Array
-
+    this.results = null;
     new UsedAssets(
       { entry: this.entry, dir: this.baseDir, exclude: this.exclude },
       this.setUsedAssets.bind(this)
@@ -75,7 +74,10 @@ class DeadFile {
       importedButNotFoundInScope
     };
 
-    this.reporting(data);
+    if (this.shouldReport) {
+      this.reporting(data);
+    }
+    this.results = data;
   }
 
   reporting(data) {

test/fixtures/babel-example/.babelrc

@@ -0,0 +1,13 @@
+{
+  "plugins": [
+    "@babel/plugin-transform-modules-commonjs",
+    [
+      "module-resolver",
+      {
+        "root": [
+          "./src"
+        ]
+      }
+    ]
+  ]
+}