teen_process 2.0.2 → 2.0.4

package/lib/helpers.js

@@ -1,5 +1,5 @@
-import which from 'which';
-import fs from 'fs';
+import path from 'path';
+import fs from 'fs/promises';
 
 /**
  * Decorates ENOENT error received from a spawn system call
@@ -8,24 +8,30 @@ import fs from 'fs';
  * @param {NodeJS.ErrnoException} error Original error instance. !!! The instance is mutated after
  * this helper function invocation
  * @param {string} cmd Original command to execute
- * @param {string|URL?} [cwd] Optional path to the current working dir
- * @returns {NodeJS.ErrnoException} Mutated error instance with an improved description or an
+ * @param {string?} [cwd] Optional path to the current working dir
+ * @returns {Promise<NodeJS.ErrnoException>} Mutated error instance with an improved description or an
  * unchanged error instance
  */
-function formatEnoent (error, cmd, cwd = null) {
-  try {
-    which.sync(cmd);
-    if (cwd) {
-      try {
-        fs.accessSync(cwd, fs.constants.R_OK);
-      } catch (ign) {
-        error.message = `The current working directory '${cwd}' for '${cmd}' command ` +
-          `either does not exist or is not accessible`;
+async function formatEnoent (error, cmd, cwd = null) {
+  if (cwd) {
+    try {
+      const stat = await fs.stat(cwd);
+      if (!stat.isDirectory()) {
+        error.message = `The working directory '${cwd}' of '${cmd}' is not a valid folder path`;
+        return error;
+      }
+    } catch (e) {
+      if (e.code === 'ENOENT') {
+        error.message = `The working directory '${cwd}' of '${cmd}' does not exist`;
+        return error;
       }
     }
-  } catch (ign) {
-    error.message = `Command '${cmd}' not found. Is it installed?`;
   }
+
+  const curDir = path.resolve(cwd ?? process.cwd());
+  const pathMsg = process.env.PATH ?? 'which is not defined for the process';
+  error.message = `'${cmd}' executable is not found neither in the process working folder (${curDir}) ` +
+    `nor in any folders specified in the PATH environment variable (${pathMsg})`;
   return error;
 }
 

package/lib/index.js

@@ -1,9 +1,10 @@
-// transpile:main
+import {install} from 'source-map-support';
+install();
+
 import * as cp from 'child_process';
 import * as spIndex from './subprocess';
 import * as execIndex from './exec';
 
-
 const { spawn } = cp;
 const { SubProcess } = spIndex;
 const { exec } = execIndex;

package/lib/subprocess.js

@@ -198,12 +198,12 @@ class SubProcess extends EventEmitter {
       };
 
       // if we get an error spawning the proc, reject and clean up the proc
-      this.proc.on('error', /** @param {NodeJS.ErrnoException} err */ (err) => {
+      this.proc.on('error', /** @param {NodeJS.ErrnoException} err */ async (err) => {
         this.proc?.removeAllListeners('exit');
         this.proc?.kill('SIGINT');
 
         if (err.code === 'ENOENT') {
-          err = formatEnoent(err, this.cmd, this.opts?.cwd);
+          err = await formatEnoent(err, this.cmd, this.opts?.cwd);
         }
         reject(err);
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "teen_process",
-  "version": "2.0.2",
+  "version": "2.0.4",
   "description": "A grown up version of Node's spawn/exec",
   "keywords": [
     "child_process",
@@ -31,14 +31,16 @@
     "build/lib"
   ],
   "scripts": {
-    "build": "babel --out-dir=build/lib lib",
+    "build": "tsc -b",
+    "clean": "npm run build -- --clean",
     "dev": "npm run build -- --watch",
     "lint": "eslint .",
     "lint:fix": "npm run lint -- --fix",
     "lint:types": "tsc",
     "precommit-lint": "lint-staged",
     "precommit-msg": "echo 'Pre-commit checks...' && exit 0",
-    "prepare": "npm run build",
+    "prepare": "npm run rebuild",
+    "rebuild": "npm run clean && npm run build",
     "test": "mocha"
   },
   "pre-commit": [
@@ -56,46 +58,42 @@
     "singleQuote": true
   },
   "dependencies": {
-    "@babel/runtime": "7.19.0",
     "bluebird": "3.7.2",
     "lodash": "4.17.21",
-    "shell-quote": "1.7.3",
-    "source-map-support": "0.5.21",
-    "which": "2.0.2"
+    "shell-quote": "1.8.1",
+    "source-map-support": "0.5.21"
   },
   "devDependencies": {
-    "@appium/eslint-config-appium": "7.0.0",
-    "@appium/support": "2.60.0",
-    "@babel/cli": "7.18.10",
-    "@babel/core": "7.19.1",
-    "@babel/eslint-parser": "7.19.1",
-    "@babel/plugin-transform-runtime": "7.19.1",
-    "@babel/preset-env": "7.19.1",
-    "@babel/register": "7.18.9",
-    "@types/bluebird": "3.5.37",
-    "@types/chai": "4.3.3",
+    "@appium/eslint-config-appium": "8.0.3",
+    "@appium/tsconfig": "0.3.0",
+    "@appium/types": "0.11.1",
+    "@types/bluebird": "3.5.38",
+    "@types/chai": "4.3.5",
     "@types/chai-as-promised": "7.1.5",
-    "@types/lodash": "4.14.185",
-    "@types/mocha": "9.1.1",
-    "@types/node": "18.6.5",
+    "@types/lodash": "4.14.195",
+    "@types/mocha": "10.0.1",
+    "@types/node": "18.16.18",
     "@types/shell-quote": "1.7.1",
-    "@types/which": "2.0.1",
-    "babel-plugin-source-map-support": "2.2.0",
-    "chai": "4.3.6",
+    "@types/sinon": "10.0.15",
+    "@types/source-map-support": "0.5.6",
+    "@types/ws": "8.5.4",
+    "chai": "4.3.7",
     "chai-as-promised": "7.1.1",
-    "eslint": "8.24.0",
-    "eslint-config-prettier": "8.5.0",
-    "eslint-plugin-import": "2.26.0",
+    "eslint": "8.42.0",
+    "eslint-config-prettier": "8.8.0",
+    "eslint-plugin-import": "2.27.5",
     "eslint-plugin-mocha": "10.1.0",
-    "eslint-plugin-promise": "6.0.1",
-    "lint-staged": "13.0.3",
-    "mocha": "10.0.0",
+    "eslint-plugin-promise": "6.1.1",
+    "lint-staged": "13.2.2",
+    "mocha": "10.2.0",
     "pre-commit": "1.2.2",
-    "prettier": "2.7.1",
-    "typescript": "4.8.3"
+    "prettier": "2.8.8",
+    "sinon": "15.1.2",
+    "typescript": "5.1.3",
+    "ts-node": "10.9.1"
   },
   "engines": {
-    "node": ">=14",
-    "npm": ">=6"
+    "node": "^14.17.0 || ^16.13.0 || >=18.0.0",
+    "npm": ">=8"
   }
 }