Fix spawn of cmd and bat files on Windows

[atrauzzi]

This ticket is specifically in reference to this: https://nodejs.org/api/child_process.html#child_process_spawning_bat_and_cmd_files_on_windows

I've put together a comparison as an example, basically it uses edgejs to spawn a command that's based off of a cmd file (electron): https://gist.github.com/atrauzzi/bbcc171b57a7ea276a2e6bf6419c7e51

The point of the comparison is to show that it is possible to run cmd and bat files without requiring a console window open the whole time. It would just be a matter of figuring out how .net is doing it and plugging into that same mechanism when running on Windows vs however nodejs is currently doing things.

But I really think this issue is long overdue to be addressed.

cc. #556

[mscdex]

For some reason, the process module in nodejs imposes that it be run in a console window.

I think this snippet from the C# code you referenced pretty much says it all:

WindowStyle = System.Diagnostics.ProcessWindowStyle.Hidden

A console window is still being created. So C# is really doing the same thing, except it is removing the window from the taskbar.

[atrauzzi]

Nice, that probably bodes well for a fix then because it means that nodejs isn't doing anything wrong in how it spawns, it's just not fully abstracted over this Windows-specific nuance?

[bnoordhuis]

Libuv supports it (the UV_PROCESS_WINDOWS_HIDE flag to uv_spawn()) but node.js hasn't exposed it so far.

[atrauzzi]

Cool, is this something that can be exposed as another option to spawn and exec? Without claiming to be a whiz with C++ or nodejs internals, conceptually this sounds pretty low risk. And I mean, this is an issue that's been spoken about since at least 2015 -- pretty low hanging fruit for such a high impact.

[cjihrig]

@atrauzzi if you follow the logic of windowsVerbatimArguments, you should be able to implement something similar for this flag.

[atrauzzi]

I suspect it would make more sense for someone familiar with C++ dev to implement this.

[cjihrig]

@atrauzzi I've opened #15380, which exposes UV_PROCESS_WINDOWS_HIDE as windowsHide in the child_process methods. I don't have a Windows box on hand to test with at the moment, so could you or another Windows user see if that fixes the problem.

[bzoz]

I've tested #15380 but it looks like it does not solve the issue. But I'm also not confident that I've tested the right thing.

@atrauzzi could you provide a small example of how to reproduce the console popping? It would be great if it would be in pure Node.js, but since this looks related to the PM2 internals, an example that uses PM2 will work for now.

[bnoordhuis]

Ping @atrauzzi.

[atrauzzi]

I put together a quick example as part of this ticket's initial description if that's enough?