resolve - npm Package Compare versions

Comparing version 1.4.0 to 1.5.0

lib/core.js

 var current = (process.versions && process.versions.node && process.versions.node.split('.')) || [];
 
-function versionIncluded(version) {
-    if (version === '*') return true;
-    var versionParts = version.split('.');
+function versionIncluded(specifier) {
+    if (specifier === true) { return true; }
+    var parts = specifier.split(' ');
+    var op = parts[0];
+    var versionParts = parts[1].split('.');
+
     for (var i = 0; i < 3; ++i) {
-        if ((current[i] || 0) >= (versionParts[i] || 0)) return true;
+        var cur = Number(current[i] || 0);
+        var ver = Number(versionParts[i] || 0);
+        if (cur === ver) {
+            continue; // eslint-disable-line no-restricted-syntax, no-continue
+        }
+        if (op === '<') {
+            return cur < ver;
+        } else if (op === '>=') {
+            return cur >= ver;
+        } else {
+            return false;
+        }
     }
@@ -15,9 +29,7 @@ return false;
 var core = {};
-for (var version in data) { // eslint-disable-line no-restricted-syntax
-    if (Object.prototype.hasOwnProperty.call(data, version) && versionIncluded(version)) {
-        for (var i = 0; i < data[version].length; ++i) {
-            core[data[version][i]] = true;
-        }
+for (var mod in data) { // eslint-disable-line no-restricted-syntax
+    if (Object.prototype.hasOwnProperty.call(data, mod)) {
+        core[mod] = versionIncluded(data[mod]);
     }
 }
 module.exports = core;

lib/core.json

 {
-    "*": [
-        "assert",
-        "buffer_ieee754",
-        "buffer",
-        "child_process",
-        "cluster",
-        "console",
-        "constants",
-        "crypto",
-        "_debugger",
-        "dgram",
-        "dns",
-        "domain",
-        "events",
-        "freelist",
-        "fs",
-        "http",
-        "https",
-        "_linklist",
-        "module",
-        "net",
-        "os",
-        "path",
-        "punycode",
-        "querystring",
-        "readline",
-        "repl",
-        "stream",
-        "string_decoder",
-        "sys",
-        "timers",
-        "tls",
-        "tty",
-        "url",
-        "util",
-        "vm",
-        "zlib"
-    ],
-    "0.11": [
-        "_http_server"
-    ],
-    "1.0": [
-        "process",
-        "v8"
-    ]
+    "assert": true,
+    "buffer_ieee754": "< 0.9.7",
+    "buffer": true,
+    "child_process": true,
+    "cluster": true,
+    "console": true,
+    "constants": true,
+    "crypto": true,
+    "_debugger": "< 8",
+    "dgram": true,
+    "dns": true,
+    "domain": true,
+    "events": true,
+    "freelist": "< 6",
+    "fs": true,
+    "http": true,
+    "http2": ">= 8.8",
+    "https": true,
+    "_http_server": ">= 0.11",
+    "_linklist": "< 8",
+    "module": true,
+    "net": true,
+    "os": true,
+    "path": true,
+    "perf_hooks": ">= 8.5",
+    "process": ">= 1",
+    "punycode": true,
+    "querystring": true,
+    "readline": true,
+    "repl": true,
+    "stream": true,
+    "string_decoder": true,
+    "sys": true,
+    "timers": true,
+    "tls": true,
+    "tty": true,
+    "url": true,
+    "util": true,
+    "v8": ">= 1",
+    "vm": true,
+    "zlib": true
 }

package.json

 {
   "name": "resolve",
   "description": "resolve like require.resolve() on behalf of files asynchronously and synchronously",
-  "version": "1.4.0",
+  "version": "1.5.0",
   "repository": {
     "type": "git",
-    "url": "git://github.com/substack/node-resolve.git"
+    "url": "git://github.com/browserify/node-resolve.git"
   },
@@ -24,8 +24,8 @@ "main": "index.js",
   "devDependencies": {
-    "@ljharb/eslint-config": "^12.2.0",
-    "eslint": "^4.3.0",
+    "@ljharb/eslint-config": "^12.2.1",
+    "eslint": "^4.9.0",
     "object-keys": "^1.0.11",
     "safe-publish-latest": "^1.1.1",
     "tap": "0.4.13",
-    "tape": "^4.7.0"
+    "tape": "^4.8.0"
   },
@@ -32,0 +32,0 @@ "license": "MIT",

test/core.js

 var test = require('tape');
+var keys = require('object-keys');
 var resolve = require('../');
@@ -16,9 +17,15 @@
     t.test('core list', function (st) {
-        st.plan(resolve.core.length);
+        var cores = keys(resolve.core);
+        st.plan(cores.length);
 
-        for (var i = 0; i < resolve.core.length; ++i) {
-            st.doesNotThrow(
-                function () { require(resolve.core[i]); }, // eslint-disable-line no-loop-func
-                'requiring ' + resolve.core[i] + ' does not throw'
-            );
+        for (var i = 0; i < cores.length; ++i) {
+            var mod = cores[i];
+            if (resolve.core[mod]) {
+                st.doesNotThrow(
+                    function () { require(mod); }, // eslint-disable-line no-loop-func
+                    'requiring ' + mod + ' does not throw'
+                );
+            } else {
+                st.skip(mod + ' not supported');
+            }
         }
@@ -25,0 +32,0 @@

New alerts

Dynamic require

Supply chain risk

Dynamic require can indicate the package is performing dangerous or unsafe dynamic code execution.

Found 1 instance in 1 package

Fixed alerts

Dynamic require

Supply chain risk

Dynamic require can indicate the package is performing dangerous or unsafe dynamic code execution.

Found 1 instance in 1 package

Improved metrics

Lines of code

1481

increased by0.95%

Worsened metrics

Total package byte prevSize

66314

decreased by-0.08%

Number of package files

70

decreased by-5.41%

No dependency changes