node-sass preventing node from exiting

[mogelbrod]

• NPM version (npm -v): 6.9.0
• Node version (node -v): v12.1.0 and v10.15.3 tested
• Node Process (node -p process.versions):

{
  node: '12.1.0',
  v8: '7.4.288.21-node.16',
  uv: '1.28.0',
  zlib: '1.2.11',
  brotli: '1.0.7',
  ares: '1.15.0',
  modules: '72',
  nghttp2: '1.38.0',
  napi: '4',
  llhttp: '1.1.1',
  http_parser: '2.8.0',
  openssl: '1.1.1b',
  cldr: '35.1',
  icu: '64.2',
  tz: '2019a',
  unicode: '12.1'
}

• Node Platform (node -p process.platform): darwin (Mac OS X 10.14.3)
• Node architecture (node -p process.arch): x64
• node-sass version (node -p "require('node-sass').info"):

node-sass	4.12.0	(Wrapper)	[JavaScript]
libsass  	3.5.4	(Sass Compiler)	[C/C++]

• npm node-sass versions (npm ls node-sass):

node-sass-hangs@1.0.0 .../node-sass-hangs
└── node-sass@4.12.0

I noticed that webpack builds of our project froze node without exiting if a JS error was encountered during the bundling. Manually calling process.exit() didn't work either. Debugging pointed towards node-sass being the culprit as the process exited when removing all scss imports from the webpack build. I've only been able to repro the issue when including most of the codebase, making it hard to create a minimal example.

Instead I found a similar issue with the same symptom with an example I was able to reproduce myself: nodejs/node#17872

Updated reproducible minimal example that should work with both node v12.1.0 and v10.*.

Help much appreciated!

[saper]

First problem is that you never return from the importer properly. But even if you would do, it still hangs.

process.exit() is running while there are other things scheduled in the event loop. This is special with node-sass since we jump to native code and return to JavaScript as we need (for example for importers).

[saper]

After some reading of the code this looks as a duplicate of #1048 to me

[saper]

Looking at my proposed solutions at #857 (comment) I'd suggest bumping priority of sass/libsass#1108 with libsass developers... (as per sass/libsass#1892 (comment))

[saper]

Closing in favour of #1048