node-version-utils 0.1.19 → 0.3.1

package/.github/dependabot.yml

@@ -0,0 +1,11 @@
+version: 2
+updates:
+  - package-ecosystem: npm
+    directory: '/'
+    schedule:
+      interval: daily
+  - package-ecosystem: 'github-actions'
+    directory: '/'
+    schedule:
+      # Check for updates to GitHub Actions every weekday
+      interval: 'daily'

package/.github/workflows/main.yml

@@ -0,0 +1,22 @@
+name: CI
+on:
+  - push
+  - pull_request
+jobs:
+  test:
+    name: Node.js ${{ matrix.node-version }} ${{ matrix.os }}
+    runs-on: ${{ matrix.os }}
+    strategy:
+      matrix:
+        node: ['latest']
+        os: [ubuntu-latest, windows-latest, macOS-latest]
+    steps:
+      - uses: actions/checkout@v2
+      - uses: actions/setup-node@v2.4.1
+        with:
+          node-version: ${{ matrix.node-version }}
+      - run: git config --global user.name "Github Actions"
+      - run: git config --global user.email "actions@users.noreply.github.com"
+      - run: npm ci
+      - run: npm run lint
+      - run: npm run test:engines

package/index.js

@@ -1,3 +1,4 @@
 module.exports = {
   spawn: require('./lib/spawn'),
+  spawnSync: require('./lib/spawnSync'),
 };

package/lib/spawn.js

@@ -1,15 +1,9 @@
-var assign = require('object-assign');
 var crossSpawn = require('cross-spawn-cb');
-var prepend = require('path-string-prepend');
 
-var envForInstallPath = require('./envForInstallPath');
-var PATH_KEY = require('./constants').PATH_KEY;
+var spawnOptions = require('./spawnOptions');
 
 function spawn(installPath, command, args, options, callback) {
-  // put the path to node and npm at the front and remove nvs
-  var env = envForInstallPath(process.env, installPath);
-  env[PATH_KEY] = prepend(env[PATH_KEY] || '', env.npm_config_binroot);
-  crossSpawn(command, args, assign({}, options, { cwd: process.cwd(), env: env, execPath: env.npm_node_execpath, path: env[PATH_KEY] }), callback);
+  crossSpawn(command, args, spawnOptions(installPath, options), callback);
 }
 
 module.exports = function spawnWrapper(installPath, command, args, options, callback) {

package/lib/spawnOptions.js

@@ -0,0 +1,43 @@
+var path = require('path');
+var assign = require('just-extend');
+var prepend = require('path-string-prepend');
+var NODE = process.platform === 'win32' ? 'node.exe' : 'node';
+var PATH_KEY = require('env-path-key')();
+
+var isWindows = process.platform === 'win32';
+
+module.exports = function envForInstallPath(installPath, options) {
+  var processEnv = process.env;
+  var env = {};
+  env.npm_config_binroot = isWindows ? installPath : path.join(installPath, 'bin');
+  env.npm_config_root = isWindows ? installPath : path.join(installPath, 'lib');
+  env.npm_config_man = isWindows ? installPath : path.join(installPath, 'man');
+  env.npm_config_prefix = installPath;
+  env.npm_node_execpath = path.join(env.npm_config_binroot, NODE);
+
+  // copy the environment not for npm and skip case-insesitive additional paths
+  for (var key in processEnv) {
+    // skip npm_ variants
+    if (key.length > 4 && (key[0] === 'n' || key[0] === 'N') && (key[1] === 'p' || key[1] === 'P') && (key[2] === 'm' || key[2] === 'M') && key[3] === '_')
+      continue;
+
+    // skip non-matching path
+    if (
+      key.length === 4 &&
+      (key[0] === 'p' || key[0] === 'P') &&
+      (key[1] === 'a' || key[1] === 'A') &&
+      (key[2] === 't' || key[2] === 'T') &&
+      (key[3] === 'h' || key[3] === 'H') &&
+      key !== PATH_KEY
+    )
+      continue;
+    env[key] = processEnv[key];
+  }
+
+  // override node
+  if (env.NODE !== undefined) env.NODE = env.npm_node_execpath;
+
+  // put the path to node and npm at the front and remove nvs
+  env[PATH_KEY] = prepend(env[PATH_KEY] || '', env.npm_config_binroot);
+  return assign({}, options, { cwd: process.cwd(), env: env, execPath: env.npm_node_execpath, path: env[PATH_KEY] });
+};

package/lib/spawnSync.js

@@ -0,0 +1,6 @@
+var crossSpawn = require('cross-spawn-cb');
+var spawnOptions = require('./spawnOptions');
+
+module.exports = function spawnSync(installPath, command, args, options) {
+  return crossSpawn.sync(command, args, spawnOptions(installPath, options));
+};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "node-version-utils",
-  "version": "0.1.19",
+  "version": "0.3.1",
   "description": "Utilities for running commands on a specific version of node by installed path",
   "keywords": [
     "node",
@@ -24,18 +24,20 @@
     "format": "prettier --write .",
     "lint": "eslint .",
     "prepublishOnly": "dtd \"npm run lint\" \"depcheck\"",
-    "test": "mocha-compat test/spec/**/*.test.js --no-timeouts"
+    "test": "mocha-compat test/spec/**/*.test.js --no-timeouts",
+    "test:engines": "nvu engines npm test"
   },
   "dependencies": {
-    "cross-spawn-cb": "^0.3.1",
-    "object-assign": "^4.1.1",
+    "cross-spawn-cb": "^0.5.2",
+    "env-path-key": "^0.2.0",
+    "just-extend": "^6.0.1",
     "path-string-prepend": "^0.2.0"
   },
   "devDependencies": {
-    "babel-eslint": "^10.1.0",
+    "@typescript-eslint/parser": "^5.30.0",
     "cr": "^0.1.0",
-    "depcheck": "^1.0.0",
-    "dis-dat": "^0.1.3",
+    "depcheck": "^1.4.3",
+    "dis-dat": "^0.1.5",
     "eslint": "^6.8.0",
     "eslint-config-prettier": "^6.11.0",
     "eslint-config-standard": "^14.1.1",
@@ -47,12 +49,13 @@
     "lodash.find": "^4.6.0",
     "match-semver": "^0.1.0",
     "mocha-compat": "^3.5.5",
-    "node-install-release": "^0.1.31",
-    "prettier": "^2.3.2",
+    "node-install-release": "^0.2.4",
+    "node-version-use": "^0.2.2",
+    "prettier": "^2.7.1",
     "rimraf": "^2.7.1",
     "semver": "^5.7.1"
   },
   "engines": {
-    "node": ">=0.12"
+    "node": ">=0.10"
   }
 }

package/lib/constants.js

@@ -1,18 +0,0 @@
-var isWindows = process.platform === 'win32';
-
-function windowsPathKey() {
-  var pathKey = 'Path';
-
-  for (var key in process.env) {
-    if (key.toUpperCase() === 'PATH') {
-      pathKey = key;
-      if (pathKey !== 'PATH') return key; // 'which' in cross-spawn uses PATH in windows, but this causes issues in repeat spawn calls given PATH get propagated so PATH_KEY needs to select 'Path' over 'PATH' if both exist
-    }
-  }
-  return pathKey;
-}
-
-module.exports = {
-  PATH_KEY: isWindows ? windowsPathKey() : 'PATH',
-  NODE: isWindows ? 'node.exe' : 'node',
-};

package/lib/envForInstallPath.js

@@ -1,26 +0,0 @@
-var path = require('path');
-var NODE = require('./constants').NODE;
-var PATH_KEY = require('./constants').PATH_KEY;
-
-var isWindows = process.platform === 'win32';
-
-module.exports = function envForInstallPath(env, installPath) {
-  var installEnv = {};
-  installEnv.npm_config_binroot = isWindows ? installPath : path.join(installPath, 'bin');
-  installEnv.npm_config_root = isWindows ? installPath : path.join(installPath, 'lib');
-  installEnv.npm_config_man = isWindows ? installPath : path.join(installPath, 'man');
-  installEnv.npm_config_prefix = installPath;
-  installEnv.npm_node_execpath = path.join(installEnv.npm_config_binroot, NODE);
-
-  // copy the environment not for npm and skip case-insesitive additional paths
-  for (var key in env) {
-    var lowerKey = key.toLowerCase();
-    if (lowerKey.indexOf('npm_') === 0) continue;
-    if (lowerKey === 'path' && key !== PATH_KEY) continue;
-    installEnv[key] = env[key];
-  }
-
-  // override node
-  if (installEnv.NODE !== undefined) installEnv.NODE = installEnv.npm_node_execpath;
-  return installEnv;
-};