tinyspawn - npm Package Compare versions

Comparing version 1.3.1 to 1.3.2

package.json

@@ -5,3 +5,3 @@ {
   "homepage": "https://github.com/microlinkhq/tinyspawn",
-  "version": "1.3.1",
+  "version": "1.3.2",
   "types": "src/index.d.ts",
@@ -8,0 +8,0 @@ "main": "src/index.js",

README.md

@@ -127,33 +127,46 @@ <div align="center">
 ```js
-const error = await $(`node -e 'require("notfound")'`).catch(error => error)
+const subprocess = $('node', ['child.js'], {
+  timeout: 500,
+  killSignal: 'SIGKILL'
+})
 
-console.error(error)
-// The command spawned as:
-//
-//   /Users/kikobeats/.n/bin/node -e 'require("notfound")'
-//
-// exited with `{ code: 1 }` and the following trace:
-//
-//   node:internal/modules/cjs/loader:1147
-//     throw err;
-//     ^
-//
-//   Error: Cannot find module 'notfound'
-//   Require stack:
-//   - /Users/kikobeats/Downloads/tinyspawn/[eval]
-//       at Module._resolveFilename (node:internal/modules/cjs/loader:1144:15)
-//       at Module._load (node:internal/modules/cjs/loader:985:27)
-//       at Module.require (node:internal/modules/cjs/loader:1235:19)
-//       at require (node:internal/modules/helpers:176:18)
-//       at [eval]:1:1
-//       at runScriptInThisContext (node:internal/vm:144:10)
-//       at node:internal/process/execution:109:14
-//       at [eval]-wrapper:6:24
-//       at runScript (node:internal/process/execution:92:62)
-//       at evalScript (node:internal/process/execution:123:10) {
-//     code: 'MODULE_NOT_FOUND',
-//     requireStack: [ '/Users/kikobeats/Downloads/tinyspawn/[eval]' ]
-//   }
+console.log(await subprocess.catch(error => error))
+// Error [ChildProcessError]: The command spawned as:
 
-//   Node.js v20.10.0
+//   `node child.js`
+
+// exited with:
+
+//   `{ signal: 'null', code: 1 }`
+
+// with the following trace:
+
+//     at createChildProcessError (/Users/kikobeats/Projects/microlink/tinyspawn/src/index.js:20:17)
+//     at ChildProcess.<anonymous> (/Users/kikobeats/Projects/microlink/tinyspawn/src/index.js:63:18)
+//     at ChildProcess.emit (node:events:531:35)
+//     at ChildProcess._handle.onexit (node:internal/child_process:294:12) {
+//   command: 'node child.js',
+//   connected: false,
+//   signalCode: null,
+//   exitCode: 1,
+//   killed: false,
+//   spawnfile: 'node',
+//   spawnargs: [ 'node', 'child.js' ],
+//   pid: 63467,
+//   stdout: '',
+//   stderr: 'node:internal/modules/cjs/loader:1148\n' +
+//     '  throw err;\n' +
+//     '  ^\n' +
+//     '\n' +
+//     "Error: Cannot find module '/Users/kikobeats/Projects/microlink/tinyspawn/child.js'\n" +
+//     '    at Module._resolveFilename (node:internal/modules/cjs/loader:1145:15)\n' +
+//     '    at Module._load (node:internal/modules/cjs/loader:986:27)\n' +
+//     '    at Function.executeUserEntryPoint [as runMain] (node:internal/modules/run_main:174:12)\n' +
+//     '    at node:internal/main/run_main_module:28:49 {\n' +
+//     "  code: 'MODULE_NOT_FOUND',\n" +
+//     '  requireStack: []\n' +
+//     '}\n' +
+//     '\n' +
+//     'Node.js v20.15.1'
+// }
 ```
@@ -160,0 +173,0 @@

src/index.js

@@ -13,17 +13,19 @@ 'use strict'
 
-const createChildProcessError = ({ cmd, cmdArgs, exitCode, stderr, childProcess }) => {
+const createChildProcessError = ({ cmd, cmdArgs, childProcess }) => {
   const command = `${cmd} ${cmdArgs.join(' ')}`
   let message = `The command spawned as:${EOL}${EOL}`
-  message += `  ${command}${EOL}${EOL}`
-  message += `exited with \`{ code: ${exitCode} }\` and the following trace:${EOL}${EOL}`
-  message += String(stderr)
-    .split(EOL)
-    .map(line => `  ${line}`)
-    .join(EOL)
+  message += `  \`${command}\`${EOL}${EOL}`
+  message += `exited with:${EOL}${EOL}`
+  message += `  \`{ signal: '${childProcess.signalCode}', code: ${childProcess.exitCode} }\` ${EOL}${EOL}`
+  message += `with the following trace:${EOL}`
   const error = new Error(message)
   error.command = command
   error.name = 'ChildProcessError'
-  Object.keys(childProcess).forEach(key => {
-    error[key] = childProcess[key]
-  })
+
+  Object.keys(childProcess)
+    .filter(key => !key.startsWith('_') && !['stdio', 'stdin'].includes(key))
+    .forEach(key => {
+      error[key] = childProcess[key]
+    })
+
   return error
@@ -62,3 +64,3 @@ }
         ? resolve(childProcess)
-        : reject(createChildProcessError({ cmd, cmdArgs, exitCode, stderr, childProcess }))
+        : reject(createChildProcessError({ cmd, cmdArgs, childProcess }))
     })
@@ -65,0 +67,0 @@ })

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

13046

increased by3.56%

Number of lines in readme file

200

increased by6.95%

No dependency changes