fix: allow Ctrl+C to interrupt build-scss command (#4155)

* fix: allow Ctrl+C to interrupt build-scss command

* refactor: after review

* test: revert build-scss test to synchronous invocation

Author: peterkulko

bin/paragon-scripts.js

@@ -265,6 +265,10 @@ const COMMANDS = {
  * @function executeParagonCommand
  */
 (async () => {
+  process.on('SIGINT', () => {
+    process.exit(130);
+  });
+
   const [command, ...commandArgs] = process.argv.slice(2);
   const resolvedCommand = commandAliases[command] || command;
   const executor = COMMANDS[resolvedCommand];

lib/__tests__/build-scss.test.js

@@ -1,4 +1,7 @@
 const fs = require('fs');
+const path = require('path');
+const { spawn } = require('child_process');
+const os = require('os');
 const sass = require('sass');
 
 const buildScssCommand = require('../build-scss');
@@ -135,17 +138,17 @@ describe('buildScssCommand', () => {
     );
   });
 
-  it('should use default arguments when none provided', () => {
-    buildScssCommand([]);
+  it('should use default arguments when none provided', async () => {
+    await buildScssCommand([]);
 
     expect(sass.compile).toHaveBeenCalledWith(
       expect.stringContaining('core.scss'),
       expect.any(Object),
     );
   });
 
-  it('should exclude core properly', () => {
-    buildScssCommand(['--excludeCore']);
+  it('should exclude core properly', async () => {
+    await buildScssCommand(['--excludeCore']);
 
     expect(sass.compile).not.toHaveBeenCalledWith(
       expect.stringContaining('core.scss'),
@@ -158,3 +161,25 @@ describe('buildScssCommand', () => {
     );
   });
 });
+
+describe('SIGINT handling', () => {
+  it('should exit with code 130 when SIGINT is received during build', async () => {
+    const child = spawn(process.execPath, [
+      path.resolve(__dirname, '../../bin/paragon-scripts.js'),
+      'build-scss',
+      `--outDir=${path.join(os.tmpdir(), 'build-scss-sigint-test')}`,
+    ], {
+      cwd: path.resolve(__dirname, '../..'),
+      stdio: 'ignore',
+    });
+
+    const { code, signal } = await new Promise((resolve) => {
+      child.on('exit', (exitCode, exitSignal) => resolve({ code: exitCode, signal: exitSignal }));
+      setTimeout(() => child.kill('SIGINT'), 500);
+    });
+
+    // Process was terminated by SIGINT — either our handler called process.exit(130)
+    // or default signal handling killed it before the handler was registered
+    expect(code === 130 || signal === 'SIGINT').toBe(true);
+  }, 30000);
+});

lib/build-scss.js

@@ -177,18 +177,19 @@ function buildScssCommand(commandArgs) {
     });
   }
 
-  // Theme Variants CSS
-  fs.readdirSync(themesPath, { withFileTypes: true })
-    .filter((item) => item.isDirectory())
-    .forEach((themeDir) => {
-      compileAndWriteStyleSheets({
-        name: themeDir.name,
-        stylesPath: `${themesPath}/${themeDir.name}/index.css`,
-        outDir,
-        isThemeVariant: true,
-        isDefaultThemeVariant: defaultThemeVariants.includes(themeDir.name),
-      });
+  // Theme variants CSS
+  const themeDirs = fs.readdirSync(themesPath, { withFileTypes: true })
+    .filter((item) => item.isDirectory());
+
+  for (const themeDir of themeDirs) {
+    compileAndWriteStyleSheets({
+      name: themeDir.name,
+      stylesPath: `${themesPath}/${themeDir.name}/index.css`,
+      outDir,
+      isThemeVariant: true,
+      isDefaultThemeVariant: defaultThemeVariants.includes(themeDir.name),
     });
+  }
 }
 
 module.exports = buildScssCommand;