Transform leaves process running causing CI to fail

[Deseteral]

When the build process (babel) quits and the transform was used it leaves a child process running which causes CI build to hang and eventually fail.

Code to reproduce:

const babel = require("babel-core");

const babelOpts = {
  plugins: [
    ['transform-postcss', {
      config: 'path/to/postcss.config.js'
    }]
  ]
};

const source = 'const s = require("./path/to/postcss/file.css");';

const res = babel.transform(source, babelOpts);
console.log(res.code);

postcss.config.js:

module.exports = ctx => ({
  plugins: []
});

CSS file:

.link {
  color: red;
}

The transpiled code is correct the only problem is orphaned child process.

After looking for details I found in MacOS's Activity Monitor that this process is using /tmp/bptp-<projectname>.sock file.

[wbyoung]

In theory, this is handled by the fact that the server is launched & unrefed.

I run this via gulp in CI and it does run fine.

Is there any chance you'd have the time to debug this a little (or even put up a repo w/ the example so it easier to just npm install and debug it)?

[GreenGremlin]

I'm curious, what is the reason for unrefing the server process? It appears every time I run babel I end up with an orphan server process.

[wbyoung]

@GreenGremlin from the docs:

Calling unref on a server will allow the program to exit if this is the only active server in the event system. If the server is already unrefd calling unref again will have no effect.

The server should be killed on exit.

[MaxMotovilov]

Ran into the same issue: code in postcss-server.js handles 'SIGTERM' by removing the socket file but not exiting. PR to come.

[ChristopherBiscardi]

This unref situation is causing a boatload of orphaned processes to stick around on my system:

[MaxMotovilov]

I think @wbyoung was waiting for someone to review/confirm my PR #94. Would you like to try it and leave a review/approval there?

[wbyoung]

Also, I'm not really doing much coding any more, so if one of you wants to take over maintaining this, let me know.

[ChristopherBiscardi]

ok thanks for the heads up @wbyoung ❤️ hope all is well