If one task crashes, others keep running

[JamesMessinger]

When running multiple tasks simultaneously, if one task crashes, the others continue running, with no easy way to kill them. For example:

"scripts": {
    "runall": "npm-run-all --parallel watch webserver",
    "watch": "gulp watch",
    "webserver": "node server.js"
}

When I run npm run runall, it runs watch and webserver in parallel. Let's say that watch crashes for some reason (exits with a non-zero exit code). When this happens, npm-run-all also exits. But webserver is still running. If I re-run npm run runall, I now get an error because the webserver can't start because the already-running instance is blocking the port number.

It would be nice if when one of the tasks crashes, npm-run-all would kill the other tasks too.

[JamesMessinger]

FYI - concurrently does this via a --kill-others argument

[mysticatea]

Thank you for the report.
I will check it this evening (JST).

[mysticatea]

I reproduced this issue.
You are right, when one of tasks clashed, npm-run-all --parallel should kill all tasks.

But I wonder, I thought child processes are killed by OS when the parent process exited....

[JamesMessinger]

Yeah, I'm confused by that as well. I thought the child tasks would automatically get killed, but they don't.

It might be worth looking at the source code for concurrently, since I've noticed that when I force-quit concurrently (via Ctrl+C), it does kill the child processes. Maybe it has something to do with how the processes are launched?

[mysticatea]

I tried to send SIGTERM to remained tasks, but.... npm/npm#4603 :(

[mysticatea]

Next, I will try to emulate npm run behavior without using npm tomorrow.

[JamesMessinger]

It sucks that npm has that issue. I wonder how soon they plan to fix it?

Thanks for looking into this. I appreciate it!

[mysticatea]

I published v1.2.6.

On this version, when npm-run-all runs tasks in parallel, if one of tasks exited with an error, npm-run-all kills other tasks.
npm does not forward signals to child process, so npm-run-all tries to find child processes of npm, and send a SIGTERM signal to them.

Its test is here: https://github.com/mysticatea/npm-run-all/blob/v1.2.6/test/parallel.js#L36-58

Please try it!

[JamesMessinger]

Awesome! Testing it right now....