Updated the handling of invalid package.json

Cleaned up the styling and added a new contributor. <3

Related to #92

Author: Matt Hernandez

lib/demeteorizer.js

@@ -239,38 +239,46 @@ Demeteorizer.prototype.findDependenciesInFolder = function (folder, inNodeModule
       if (keepGoing) {
         this.findDependenciesInFolder(path.join(folder, file), inNodeMod, context);
       }
-    }
-    else {
+    } else {
       if (file === 'package.json') {
         var packageData = fs.readFileSync(path.join(folder, file));
 
-        // Test that the package.json contains data. There is a module that
-        //    includes a package.json fixture that is empty.
-        // Let's also test for invalid JSON in the package.json
-        var packageJson;
-        var isValidJSON = true;
-        try {
-        	packageJson = JSON.parse(packageData);
-    	} catch (e) {
-    		console.log('*** Exception when parsing packageData: ' + e);
-    		console.log('packageData path: ' + path.join(folder, file));
-        	console.log('packageData: ' + packageData);
-        	isValidJSON = false;
-    	}
-        
-        if (isValidJSON && packageData.length > 0) {
-           
-          var filtered = this.filterDep(packageJson.name, packageJson._resolved || packageJson.version);
-          if (filtered) {
-            context.dependencies[filtered.name] = filtered.version;
+        // Test that the package.json contains data.
+        if (packageData.length > 0) {
+          var packageJson = null;
+
+          try {
+            packageJson = JSON.parse(packageData);
+          } catch (e) {
+            this.emit('progress', util.format(
+              'Warning: failed to parse package.json for %s',
+              path.basename(folder)
+            ));
           }
-          if (packageJson.dependencies) {
-            Object.keys(packageJson.dependencies).forEach(function (dep) {
-              var filtered = this.filterDep(dep, packageJson.dependencies[dep]);
-              if (filtered) {
-                context.dependencies[filtered.name] = filtered.version;
-              }
-            }.bind(this));
+
+          // Only add the dependency if the package is valid.
+          if (packageJson) {
+            var filtered = this.filterDep(
+              packageJson.name,
+              packageJson._resolved || packageJson.version
+            );
+
+            if (filtered) {
+              context.dependencies[filtered.name] = filtered.version;
+            }
+
+            if (packageJson.dependencies) {
+              Object.keys(packageJson.dependencies).forEach(function (dep) {
+                var filtered = this.filterDep(
+                  dep,
+                  packageJson.dependencies[dep]
+                );
+
+                if (filtered) {
+                  context.dependencies[filtered.name] = filtered.version;
+                }
+              }.bind(this));
+            }
           }
         }
       }

package.json

@@ -14,7 +14,8 @@
     "Brandon Cannaday <[email protected]>",
     "Bret Fisher <[email protected]>",
     "Matt Hernandez <[email protected]>",
-    "Tarang Patel <[email protected]>"
+    "Tarang Patel <[email protected]>",
+    "Vincil Bishop <[email protected]>"
   ],
   "license": "MIT",
   "repository": {