node-version-utils 0.2.0 → 0.4.0

package/.eslintrc.json

@@ -0,0 +1,13 @@
+{
+  "extends": ["standard", "prettier"],
+  "parser": "@typescript-eslint/parser",
+  "env": {
+    "es6": true,
+    "node": true,
+    "mocha": true
+  },
+  "rules": {
+    "no-var": "off",
+    "object-shorthand": "off"
+  }
+}

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,23 @@
+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 install node-version-use -g
+      - run: npm ci
+      - run: npm run lint
+      - run: nvu engines --desc npm run test

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,37 @@
+var path = require('path');
+var assign = require('just-extend');
+var prepend = require('path-string-prepend');
+var NODE = process.platform === 'win32' ? 'node.exe' : 'node';
+var pathKey = require('cross-spawn-cb').pathKey;
+var startsCaseInsensitiveFn = require('./startsCaseInsensitiveFn');
+
+var isWindows = process.platform === 'win32';
+
+var startsNPM = startsCaseInsensitiveFn('npm_');
+var startsPath = startsCaseInsensitiveFn('path');
+
+module.exports = function spawnOptions(installPath, options) {
+  var PATH_KEY = pathKey();
+  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 and non-matching path
+    if (key.length > 4 && startsNPM(key)) continue;
+    if (key.length === 4 && startsPath(key) && 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/lib/startsCaseInsensitiveFn.js

@@ -0,0 +1,11 @@
+module.exports = function startsCaseInsensitiveFn(string) {
+  var lower = string.toLowerCase();
+  var upper = string.toUpperCase();
+  return function startsCaseInsensitive(key) {
+    if (key.length < string.length) return false;
+    for (var i = 0; i < string.length; i++) {
+      if (key[i] !== lower[i] && key[i] !== upper[i]) return false;
+    }
+    return true;
+  };
+};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "node-version-utils",
-  "version": "0.2.0",
+  "version": "0.4.0",
   "description": "Utilities for running commands on a specific version of node by installed path",
   "keywords": [
     "node",
@@ -27,32 +27,30 @@
     "test": "mocha-compat test/spec/**/*.test.js --no-timeouts"
   },
   "dependencies": {
-    "cross-spawn-cb": "^0.3.1",
-    "object-assign": "^4.1.1",
+    "cross-spawn-cb": "^0.5.9",
+    "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",
-    "eslint": "^6.8.0",
-    "eslint-config-prettier": "^6.11.0",
-    "eslint-config-standard": "^14.1.1",
-    "eslint-plugin-import": "^2.22.0",
+    "depcheck": "^1.4.3",
+    "dis-dat": "^0.1.6",
+    "eslint": "^8.18.0",
+    "eslint-config-prettier": "^8.5.0",
+    "eslint-config-standard": "^17.0.0",
+    "eslint-plugin-import": "^2.26.0",
     "eslint-plugin-node": "^11.1.0",
-    "eslint-plugin-promise": "^4.2.1",
-    "eslint-plugin-standard": "^4.0.1",
+    "eslint-plugin-promise": "^6.0.0",
     "is-version": "^0.2.0",
     "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",
-    "rimraf": "^2.7.1",
+    "node-install-release": "^0.2.5",
+    "prettier": "^2.7.1",
     "semver": "^5.7.1"
   },
   "engines": {
-    "node": ">=0.12"
+    "node": ">=0.8"
   }
 }

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;
-};