fix: don't leave behind electron processes

nbbeeken · nbbeeken · commit 47bcf1387857 · 2025-09-29T11:32:07.000+02:00
diff --git a/scripts/start.mts b/scripts/start.mts
@@ -76,7 +76,7 @@ async function cleanup(signal: NodeJS.Signals) {
   console.log('\nstart    | requested termination.');
   await timers.setTimeout(10_000);
   const stillRunning = subProcesses.filter((p) => p.exitCode === null);
-  for (const p of stillRunning) p.kill('SIGKILL');
+  for (const p of stillRunning) p.kill('SIGTERM');
   console.log('\nstart    | done.');
   process.exit(0);
 }

Original file line number	Diff line number	Diff line change
`@@ -76,7 +76,7 @@ async function cleanup(signal: NodeJS.Signals) {`
`76`	`76`	`console.log('\nstart \| requested termination.');`
`77`	`77`	`await timers.setTimeout(10_000);`
`78`	`78`	`const stillRunning = subProcesses.filter((p) => p.exitCode === null);`
`79`		`- for (const p of stillRunning) p.kill('SIGKILL');`
	`79`	`+ for (const p of stillRunning) p.kill('SIGTERM');`
`80`	`80`	`console.log('\nstart \| done.');`
`81`	`81`	`process.exit(0);`
`82`	`82`	`}`