lint/reporter: collect and output unused files

Author: Sergey Melyukov

lib/lint/reporter/collectFiles.js

@@ -0,0 +1,44 @@
+var path = require('path');
+var fs = require('fs');
+
+module.exports = function(base, extFilter){
+  var stack = [base];
+  var cursor;
+  var collectedFiles = {};
+
+  while (cursor = stack.pop())
+  {
+    if (!fs.existsSync(cursor))
+      continue;
+
+    var stat = fs.lstatSync(cursor);
+
+    if (stat.isSymbolicLink())
+    {
+      var resolvedLink = path.resolve(cursor, fs.readlinkSync(cursor));
+
+      stack.push(resolvedLink);
+    }
+    else if (stat.isDirectory())
+    {
+      var items = fs.readdirSync(cursor);
+
+      for (var i = 0; i < items.length; i++)
+        stack.push(path.join(cursor, items[i]));
+    }
+    else
+    {
+      if (extFilter)
+      {
+        var fileExt = path.extname(cursor);
+
+        if (fileExt.toLowerCase() === extFilter.toLowerCase())
+          collectedFiles[cursor] = true;
+      }
+      else
+        collectedFiles[cursor] = true;
+    }
+  }
+
+  return collectedFiles;
+};

lib/lint/reporter/parallel-process-warns.js

@@ -1,5 +1,10 @@
+var collectFiles = require('./collectFiles');
+
 module.exports = function(tasks){
   var result = {};
+  var basePaths = {};
+  var usedFiles = {};
+  var collectedFiles = {};
 
   tasks.forEach(function(task){
     var failures = result[task.name] = [];
@@ -20,7 +25,39 @@ module.exports = function(tasks){
           message: warn.message + ' at ' + filename
         });
       });
+
+    if (task.result.usedFiles)
+    {
+      basePaths[task.result.usedFiles.collectPath] = true;
+      task.result.usedFiles.items.forEach(function(filename){
+        usedFiles[task.result.usedFiles.basePath + filename] = true;
+      });
+    }
   });
 
+  if (Object.keys(basePaths).length)
+  {
+    for (var basePath in basePaths)
+    {
+      var files = collectFiles(basePath);
+
+      for (var fileName in files)
+        collectedFiles[fileName] = true;
+    }
+
+    for (var usedFile in usedFiles)
+      delete collectedFiles[usedFile];
+
+    for (var unusedName in collectedFiles)
+    {
+      unusedName = unusedName.slice(process.cwd().length);
+      result['unused files'] = result['unused files'] || [];
+      result['unused files'].push({
+        loc: unusedName,
+        message: unusedName
+      });
+    }
+  }
+
   return result;
 };

lib/lint/reporter/process-warns.js

@@ -1,5 +1,10 @@
-module.exports = function(warns, fileFilter){
+var isChildProcess = typeof process.send == 'function'; // child process has send method
+var collectFiles = require('./collectFiles');
+
+module.exports = function(flow, options){
   var result = {};
+  var warns = flow.warns;
+  var fileFilter = options.filter;
 
   warns.forEach(function(warn){
     var filename = warn.file || '<UnknownFile>';
@@ -16,5 +21,36 @@ module.exports = function(warns, fileFilter){
     });
   });
 
+  if (!isChildProcess)
+  {
+    var basePath;
+    var collectedFiles;
+
+    if (options.warnUnusedFiles && flow.usedFiles)
+    {
+      var usedFilesInfo = flow.usedFiles;
+      var usedFiles = {};
+
+      basePath = usedFilesInfo.collectPath;
+      collectedFiles = collectFiles(basePath);
+
+      usedFilesInfo.items.forEach(function(filename){
+        usedFiles[usedFilesInfo.basePath + filename] = true;
+      });
+
+      for (var usedFile in usedFiles)
+        delete collectedFiles[usedFile];
+
+      for (var unusedName in collectedFiles) {
+        unusedName = unusedName.slice(process.cwd().length);
+        result['unused files'] = result['unused files'] || [];
+        result['unused files'].push({
+          loc: unusedName,
+          message: unusedName
+        });
+      }
+    }
+  }
+
   return result;
 };