fixed some linting errors

.eslintrc.js

@@ -16,6 +16,7 @@ module.exports = {
     "array-bracket-spacing": 2,
     "block-spacing": 2,
     "comma-spacing": 2,
+    "comma-dangle": [2, "always-multiline"],
     "computed-property-spacing": 2,
     "eol-last": 2,
     "indent": [2, 2],

lib/BrowserifyCache.js

@@ -1,10 +1,6 @@
 var fs = require('fs');
-var path = require('path');
-var util = require('util');
 var assert = require('assert');
-var splicer = require('labeled-stream-splicer');
 var through = require('through2');
-var async = require('async');
 var assign = require('xtend/mutable');
 
 var assertExists = require('./assertExists');
@@ -69,13 +65,11 @@ BrowserifyCache.getPackageCache = function(b) {
 };
 
 function attachCacheHooksToPipeline(b) {
-  var cache = BrowserifyCache.getCache(b);
-
   var prevBundle = b.bundle;
   b.bundle = function(cb) {
     var outputStream = through.obj();
 
-    invalidateCacheBeforeBundling(b, function(err, invalidated) {
+    invalidateCacheBeforeBundling(b, function(err) {
       if (err) return outputStream.emit('error', err);
 
       var bundleStream = prevBundle.call(b, cb);

lib/invalidateDependentFiles.js

@@ -22,7 +22,7 @@ function invalidateDependentFiles(cache, invalidatedModules, done) {
     Object.keys(dependentFiles[dependentFile]).forEach(function(module) {
       delete cache.modules[module];
     })
-  }, function(err, invalidated, deleted) { done(err); });
+  }, function(err) { done(err); });
 }
 
 module.exports = invalidateDependentFiles;

lib/invalidateFilesPackagePaths.js

@@ -1,9 +1,9 @@
 var fs = require('fs');
 var path = require('path');
-var assert = require('assert');
 var async = require('async');
 
 var assertExists = require('./assertExists');
+var packageFileForPackagePath = require('./packageFilePathUtils').packageFileForPackagePath;
 
 // this is a big complex blob of code to deal with the small edge case where
 // the package associated with a file changes (due to the addition or deletion
@@ -66,7 +66,7 @@ function invalidateFilesPackagePaths(filesPackagePaths, done) {
         });
       }, pkgdirDone);
     });
-  }, function(err) { done(null); }); // don't really care about errors
+  }, function() { done(null); }); // don't really care about errors
 }
 
 // get all directories between a common base and a list of files

lib/invalidateModifiedFiles.js

@@ -20,7 +20,7 @@ function invalidateModifiedFiles(mtimes, files, invalidate, done) {
       mtimes[file] = mtimeNew;
       fileDone();
     });
-  }, function(err) {
+  }, function() {
     done(null, invalidated, deleted);
   });
 }

lib/invalidatePackageCache.js

@@ -1,5 +1,7 @@
 var assertExists = require('./assertExists');
 var invalidateModifiedFiles = require('./invalidateModifiedFiles');
+var packagePathForPackageFile = require('./packageFilePathUtils').packagePathForPackageFile;
+var packageFileForPackagePath = require('./packageFilePathUtils').packageFileForPackagePath;
 
 function invalidatePackageCache(mtimes, cache, done) {
   assertExists(mtimes);
@@ -9,14 +11,5 @@ function invalidatePackageCache(mtimes, cache, done) {
   }, done)
 }
 
-var packagePathTrimLength = '/package.json'.length;
-
-function packagePathForPackageFile(packageFilepath) {
-  packageFilepath.slice(0, packageFilepath.length - packageFileTrimLength);
-}
-
-function packageFileForPackagePath(packagePath) {
-  return path.join(packagePath, 'package.json');
-}
 
 module.exports = invalidatePackageCache;

lib/packageFilePathUtils.js

@@ -0,0 +1,15 @@
+var path = require('path');
+var packagePathTrimLength = '/package.json'.length;
+
+function packagePathForPackageFile(packageFilepath) {
+  packageFilepath.slice(0, packageFilepath.length - packagePathTrimLength);
+}
+
+function packageFileForPackagePath(packagePath) {
+  return path.join(packagePath, 'package.json');
+}
+
+module.exports = {
+  packageFileForPackagePath,
+  packagePathForPackageFile,
+};