fix(shutdown): stop Ctrl+C output racing the shell prompt (#62)

melvincarvalho · melvincarvalho · commit 23eaaf60cd7d · 2026-05-25T20:33:04.000+02:00
The JSS child shares the CLI's process group, so a terminal Ctrl+C delivers SIGINT to both. The child already prints "Shutting down..." and exits on its own; the parent was also printing "Shutting down gracefully..." and then racing the farewell against the returning prompt. Drop the parent's redundant line and let handle.stop() await the child's exit before printing the farewell, so output stays ordered. Keeps both in the same process group (no detached orphan risk). Closes #62
diff --git a/index.js b/index.js
@@ -519,12 +519,15 @@ handle.exit.then(({ code, signal }) => {
   }
 });
 
-// Graceful shutdown
+// Graceful shutdown. The JSS child shares our process group, so a terminal
+// Ctrl+C delivers SIGINT to it too — it prints its own "Shutting down..."
+// and exits. We don't duplicate that line; we just await its exit (which
+// handle.stop() does) and then print the farewell, so the output stays
+// ordered ahead of the returning shell prompt instead of racing it.
 process.on('SIGINT', async () => {
-  console.log('\n' + chalk.yellow('⚠  Shutting down gracefully...'));
   await handle.stop();
-  console.log(chalk.green('✓  Server stopped'));
-  console.log(chalk.dim('\nGoodbye! 👋\n'));
+  console.log(chalk.green('\n✓  Server stopped'));
+  console.log(chalk.dim('Goodbye! 👋\n'));
   process.exit(0);
 });
 
