cross-spawn-cb 0.6.2 → 0.6.5

package/lib/index.js

@@ -1,4 +1,18 @@
 require('./polyfills');
+
+// exclude from polyfils to avoid loading in functionExec
+var cp = require('child_process');
+if (!cp.spawnSync) {
+  var path = require('path')
+  var functionExec = require('function-exec-sync');
+
+  cp.spawnSync = function spawnSync(command, args, options) {
+    options = options || {};
+    var filePath = path.resolve(__dirname, 'spawnCallback.js');
+    return functionExec({ callbacks: true }, filePath, command, args, options);
+  };
+}
+
 module.exports = require('./spawnCallback');
 module.exports.spawn = require('./spawnCallback');
 module.exports.sync = require('./spawnSyncCallback');

package/lib/polyfills.js

@@ -1,13 +1,4 @@
+require('core-js/actual/object/assign');
+
 var path = require('path');
 if (typeof path.delimiter === 'undefined') path.delimiter = process.platform === 'win32' ? ';' : ':';
-
-var cp = require('child_process');
-if (!cp.spawnSync) {
-  var functionExec = require('function-exec-sync');
-
-  cp.spawnSync = function spawnSync(command, args, options) {
-    options = options || {};
-    var filePath = path.resolve(__dirname, 'spawnCallback.js');
-    return functionExec({ callbacks: true }, filePath, command, args, options);
-  };
-}

package/lib/spawnCallback.js

@@ -1,3 +1,4 @@
+require('./polyfills');
 var once = require('once');
 var nextTick = require('next-tick');
 
@@ -15,15 +16,15 @@ module.exports = function spawnCallback(command, args, options, callback) {
   var cp = spawn(command, args, options);
 
   // collect output
-  var stdout, stderr;
+  var res = { stdout: null, stderr: null };
   if (options.encoding) {
     if (cp.stdout) {
-      stdout = [];
-      cp.stdout.on('data', stdout.push.bind(stdout));
+      res.stdout = [];
+      cp.stdout.on('data', res.stdout.push.bind(res.stdout));
     }
     if (cp.stderr) {
-      stderr = [];
-      cp.stderr.on('data', stderr.push.bind(stderr));
+      res.stderr = [];
+      cp.stderr.on('data', res.stderr.push.bind(res.stderr));
     }
   }
 
@@ -35,34 +36,29 @@ module.exports = function spawnCallback(command, args, options, callback) {
   // done
   cp.on('close', function (status, signal) {
     nextTick(function closeNextTick() {
-      // process output
-      if (options.encoding) {
-        stdout = Buffer.concat(stdout);
-        stderr = Buffer.concat(stderr);
-
-        if (options.encoding !== 'buffer') {
-          stdout = stdout.toString(options.encoding);
-          stderr = stderr.toString(options.encoding);
-        }
+      // prepare result
+      res.pid = cp.pid;
+      res.status = status;
+      res.signal = signal;
+      if (res.stdout) {
+        res.stdout = Buffer.concat(res.stdout);
+        if (options.encoding !== 'buffer') res.stdout = res.stdout.toString(options.encoding);
+      }
+      if (res.stderr) {
+        res.stderr = Buffer.concat(res.stderr);
+        if (options.encoding !== 'buffer') res.stderr = res.stderr.toString(options.encoding);
       }
+      res.output = [null, res.stdout, res.stderr];
 
       // process errors
-      var err = status !== 0 ? new Error('Non-zero exit code: ' + status) : null;
-      if (stderr && stderr.length) {
+      var err = res.status !== 0 ? new Error('Non-zero exit code: ' + res.status) : null;
+      if (res.stderr && res.stderr.length) {
         err = err || new Error('stderr has content');
-        err.stderr = Buffer.isBuffer(stderr) ? stderr.toString('utf8') : stderr;
+        for (var key in res) {
+          err[key] = Buffer.isBuffer(res[key]) ? res[key].toString('utf8') : res[key];
+        }
       }
-      if (err) return callback(err);
-
-      // non error result
-      callback(null, {
-        pid: cp.pid,
-        output: [null, stdout, stderr],
-        stdout: stdout,
-        stderr: stderr,
-        status: status,
-        signal: signal,
-      });
+      err ? callback(err) : callback(null, res);
     });
   });
 

package/lib/spawnSyncCallback.js

@@ -10,20 +10,27 @@ module.exports = function spawnSyncCallback(command, args, options) {
     stdio: 'pipe',
     encoding: 'utf8',
   });
-  var result = spawnSync(command, args, syncOptions);
-  if (result.stdout && (options.stdout === 'inherit' || options.stdio === 'inherit')) {
-    process.stdout.write(result.stdout);
-    result.stdout = null;
+
+  var res = spawnSync(command, args, syncOptions);
+
+  // pipe if inherited
+  if (res.stdout && (options.stdout === 'inherit' || options.stdio === 'inherit')) {
+    process.stdout.write(res.stdout);
+    res.stdout = null;
   }
-  if (result.stderr && (options.stderr === 'inherit' || options.stdio === 'inherit')) {
-    process.stderr.write(result.stderr);
-    result.stderr = null;
+  if (res.stderr && (options.stderr === 'inherit' || options.stdio === 'inherit')) {
+    process.stderr.write(res.stderr);
+    res.stderr = null;
   }
-  var err = result.status ? new Error('Non-zero exit code: ' + result.status) : null;
-  if (result.stderr && result.stderr.length) {
+
+  // process errors
+  var err = res.status !== 0 ? new Error('Non-zero exit code: ' + res.status) : null;
+  if (res.stderr && res.stderr.length) {
     err = err || new Error('stderr has content');
-    err.stderr = result.stderr;
+    for (var key in res) {
+      err[key] = Buffer.isBuffer(res[key]) ? res[key].toString('utf8') : res[key];
+    }
   }
   if (err) throw err;
-  return result;
+  return res;
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cross-spawn-cb",
-  "version": "0.6.2",
+  "version": "0.6.5",
   "description": "Cross spawn with a completion callback",
   "keywords": [
     "cross-spawn",
@@ -14,9 +14,10 @@
   "license": "MIT",
   "main": "lib/index.js",
   "scripts": {
+    "test:engines": "nvu engines npm test",
     "format": "prettier --write .",
     "lint": "eslint .",
-    "prepublishOnly": "dtd \"npm run lint\" \"depcheck\"",
+    "prepublishOnly": "npm run lint && depcheck",
     "test": "mocha-compat test/spec/**/*.test.js",
     "build": "rollup -c config/rollup.cs.config.js"
   },
@@ -24,28 +25,28 @@
     "lib"
   ],
   "dependencies": {
+    "core-js": "^3.23.5",
     "cross-spawn": "^7.0.3",
-    "function-exec-sync": "^0.2.1",
+    "function-exec-sync": "^0.2.4",
     "just-extend": "^6.0.1",
     "next-tick": "^1.1.0",
     "once": "^1.4.0"
   },
   "devDependencies": {
-    "@babel/preset-env": "^7.18.6",
+    "@babel/preset-env": "^7.18.9",
     "@rollup/plugin-babel": "^5.3.1",
     "@rollup/plugin-commonjs": "^22.0.1",
     "@rollup/plugin-node-resolve": "^13.3.0",
-    "@typescript-eslint/parser": "^5.30.6",
+    "@typescript-eslint/parser": "^5.30.7",
     "depcheck": "^1.4.3",
-    "dis-dat": "^0.1.7",
-    "eslint": "^8.19.0",
+    "eslint": "^8.20.0",
     "eslint-config-prettier": "^8.5.0",
     "eslint-config-standard": "^17.0.0",
     "eslint-plugin-import": "^2.26.0",
     "eslint-plugin-promise": "^6.0.0",
     "mocha-compat": "^3.5.5",
     "prettier": "^2.7.1",
-    "rollup": "^2.76.0",
+    "rollup": "^2.77.0",
     "rollup-plugin-babel": "^4.4.0",
     "rollup-plugin-commonjs": "^10.1.0",
     "rollup-plugin-node-externals": "^4.1.1"