Nodemon child process not being killed with update to 1.18.0

[nathanchapman]

• nodemon -v: 1.18.0
• node -v: v8.11.2
• Operating system/terminal environment: macOS 10.13.5/ZSH
• Command you ran: nodemon --exec npx ...

Expected behaviour

In the previous version (1.17.5), Nodemon would properly kill the child process with ^C

Actual behaviour

After updating to 1.18.0, Nodemon no longer kills the child process and now throws an error:

npm ERR! code ELIFECYCLE
npm ERR! errno 1

Steps to reproduce

Update to 1.18.0 and run a child process like: nodemon --exec npx http-server. Hit control + C. See new error. The http-server will still be running and requires a killall node to exit.

---

If applicable, please append the --dump flag on your command and include the output here ensuring to remove any sensitive/personal details or tokens.

[remy]

This is me running that command on my own Mac 10.13.5 using zsh:

1.18.0 does change how stdin is handled, so I can believe there's a problem here, but I can't replicate at the moment.

This is a gist of the package.json and I'm able to ctrl-c the process and the sub-process (in my case) does exit.

Are you able to include any more detail on what you're seeing?

[remy]

(I'm shutting down for the night, but if you're able to give me more info I've pushed a version that has a tiny tweak that might help - I'm not sure, you can test from npm i nodemon@debug - it should give you nodemon@1.18.1-alpha.1)

[remy]

Related commit: a974892

[ianrosedev]

Mac 10.13.5 using Bash
I'm having the same issue even with nodemon@1.18.1-alpha.1.

[virzak]

Looks like the same issue is happening on Windows. error is the result of pressing ctrl+c. The node task is not killed.

[wenbolovesnz]

I can confirm this is an issue for me as well, I can kill the process on 1.17.5. The issue started appearing after I upgraded to 1.18.0. I am a Mac user.

[remy]

I've published a new debug version - please can both windows and all other OS types test with this version - again, for myself, both 1.18.0 and debug versions work - but I'm using the exact same example provided by @nathanchapman.

To installed: npm i -g nodemon@debug - should yield nodemon@1.18.1-apha.2

[AllanPooley]

Also encountered this error today in our dev shop (MacOS).

Temporary fix was to:

1. Kill process sudo lsof -i tcp:3000 & kill -9 <pid>
   2. Delete node modules rm -rf node_modules
   3. Clear npm cache (npm cache clean --force or yarn cache clean if you're using yarn)
   4. Replace nodemon entry in your npm package-lock.json with entry for 1.17.5 (copy here: https://pastebin.com/S0aC2HWA)
   5. Reinstall node modules (npm install) and run app (npm run dev).
2. Install 1.17.5 npm install nodemon@1.17.5 --save-dev --save-exact

Will test nodemon@1.18.1-apha.2 in a moment @remy.

[AllanPooley]

@remy Can confirm nodemon@1.18.1-alpha.2 exits gracefully (without issue error) for me (MacOS)

Before:

After:

[baileyandy]

I can also confirm nodemon@1.18.1-alpha.2 on GNU bash, version 3.2.57(1)-release (x86_64-apple-darwin17) is working as expected. Thanks for the quick fix!

[virzak]

Windows works as well

[remy]

Okay, the fix is shipping now. It'll be on npm as 1.18.1 as soon as the tests have finished.

[ianrosedev]

I'm still getting the error with 1.18.1:

npm ERR! code ELIFECYCLE
npm ERR! errno 130
npm ERR! todo-api@0.1.0 test-watch: `nodemon --exec npm test`
npm ERR! Exit status 130

1.18.1-alpha.2 is working correctly.

[remy]

@ianrosedev can you explain what you're doing to run that command? i.e. so I can replicate?

[remy]

Here's what I get when I hit control-c:

Left side shows running nodemon (which runs an http server from an npm command) - then on the right side, I curl the server - it responds ok.

I then control-c on the left, then try to curl on the right, and it fails (as the sub-process isn't running anymore).

[ianrosedev]

@remy I'm running this this Mocha test script:

"test": "export NODE_ENV=test || SET \"NODE_ENV=test\" && mocha -r esm --exit server/**/*.test.js",

[ianrosedev]

Here's what I get when I hit control-c:

[virzak]

I just noticed that on Windows too.

Happens with 1.18.1-alpha.2 and 1.18.2

correction
1.18.1-alpha.2 works properly

[remy]

@ianrosedev did you get your prompt back and did the sub-process end? - so far your screenshots don't answer that question.

[ianrosedev]

@remy sorry about that. Yes the prompt is coming back and the sub-process is ending.

[remy]

Please can you try with the current debug version: npm install -g nodemon@debug (1.18.2-alpha.1)

I'm going to revert the change I made for handling ctrl-l if this fixes the issue.

[MateuszJeziorski]

@remy I had the same issue on Mac with 1.18.1, but it works fine with 1.18.2-alpha.1

[ianrosedev]

@remy yes 1.18.2-alpha.1 is working correctly for me too.

[nathanchapman]

1.18.2-alpha.1 works as expected for me as well! Thank you!