module-deps - npm Package Compare versions

Comparing version 0.16.0 to 1.0.0

index.js

@@ -29,3 +29,2 @@ var fs = require('fs');
     var pending = 0;
-    var currentOrder = 0;
     
@@ -37,30 +36,6 @@ var output = through();
     
-    var pushResult = (function () {
-        var slots = {};
-        var upto = 0;
-        var offset = 0;
-        
-        return function (row, order) {
-            if (row && opts.includeIndex) {
-                row.index = order - offset;
-            }
-            
-            if (order === upto) {
-                if (row === false) offset ++;
-                else output.queue(row);
-                
-                for (upto ++; slots[upto] !== undefined; upto++) {
-                    if (slots[upto] === false) {
-                        offset ++
-                    }
-                    else output.queue(slots[upto]);
-                    
-                    delete slots[upto];
-                }
-            }
-            else {
-                slots[order] = row;
-            }
-        };
-    })();
+    function pushResult (row) {
+        output.queue(row);
+        if (--pending === 0) output.queue(null);
+    }
     
@@ -85,6 +60,5 @@ var top = { id: '/', filename: '/', paths: [] };
         pending ++;
-        var order = currentOrder ++;
         
         if (typeof id === 'object') {
-            return id.stream.pipe(concat(function (src) {
+            id.stream.pipe(concat(function (src) {
                 var pkgfile = path.join(basedir, 'package.json');
@@ -98,5 +72,7 @@ fs.readFile(pkgfile, function (err, pkgsrc) {
                     var trx = getTransform(pkg);
-                    applyTransforms(id.file, trx, src, pkg, order);
+                    applyTransforms(id.file, trx, src, pkg);
                 });
             }));
+            if (cb) cb(false);
+            return;
         }
@@ -124,8 +100,6 @@
             ].join('')));
+            
             if (cb) cb(file);
-            if (visited[file]) {
-                pushResult(false, order);
-                if (--pending === 0) pushResult(null, currentOrder ++);
-                return;
-            }
+            if (visited[file]) return;
+            
             visited[file] = true;
@@ -136,7 +110,7 @@
             if (cache && cache[file]) {
-                parseDeps(file, cache[file], pkg, order);
+                parseDeps(file, cache[file], pkg);
             }
             else fs.readFile(file, 'utf8', function (err, src) {
                 if (err) return output.emit('error', err);
-                applyTransforms(file, trx, src, pkg, order);
+                applyTransforms(file, trx, src, pkg);
             });
@@ -158,3 +132,3 @@ });
     
-    function applyTransforms (file, trx, src, pkg, order) {
+    function applyTransforms (file, trx, src, pkg) {
         var isTopLevel = mains.some(function (main) {
@@ -182,7 +156,7 @@ var m = path.relative(path.dirname(main), file);
         function done () {
-            parseDeps(file, src, pkg, order);
+            parseDeps(file, src, pkg);
         }
     }
     
-    function parseDeps (file, src, pkg, order) {
+    function parseDeps (file, src, pkg) {
         var deps;
@@ -211,2 +185,3 @@ if (!Buffer.isBuffer(src) && typeof src === 'object') {
         var resolved = {};
+        var indexes = {};
         
@@ -220,4 +195,5 @@ deps.forEach(function (id) {
             
-            walk(id, current, function (r) {
+            walk(id, current, function (r, index) {
                 resolved[id] = r;
+                indexes[id] = index;
                 if (--p === 0) done();
@@ -237,4 +213,3 @@ });
             }
-            pushResult(rec, order);
-            if (--pending === 0) pushResult(null, currentOrder ++);
+            pushResult(rec);
         }
@@ -241,0 +216,0 @@ }

package.json

 {
     "name": "module-deps",
-    "version": "0.16.0",
+    "version": "1.0.0",
     "description": "walk the dependency graph to generate json output that can be fed into browser-pack",
@@ -5,0 +5,0 @@ "main": "index.js",

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

No v1

Quality

Package is not semver >=1. This means it is not stable and does not support ^ ranges.

Found 1 instance in 1 package

Improved metrics

Number of low quality alerts

0

decreased by-100%

Worsened metrics

Total package byte prevSize

28185

decreased by-3.05%

Lines of code

652

decreased by-3.41%

No dependency changes