fix(config): clean up editor temp files on signals

cspell-dictionary.txt

@@ -99,6 +99,7 @@ envl
 esac
 eventsource
 execa
+execvpe
 expialidocious
 extnamespace
 extwlist
@@ -426,8 +427,12 @@ vcpu
 VERSINFO
 webfactory
 WINDIR
+WIFSIGNALED
+WNOHANG
+WTERMSIG
 WININICHANGE
 withfig
+waitpid
 wrapline
 writeout
 wubalubadubdub

src/lib/analytics-telemetry/worker-client.ts

@@ -1,5 +1,6 @@
 /* eslint-disable n/no-process-exit */
 
+import {awaitProcessShutdown, type ShutdownSignal} from '../process-shutdown.js'
 import {
   CLIError, spawnTelemetryWorker, telemetryDebug, TelemetryGlobal,
 } from './telemetry-utils.js'
@@ -43,24 +44,36 @@ export function setupTelemetryHandlers(options: SetupTelemetryOptions): void {
     }
   })
 
-  process.on('SIGINT', () => {
-    // Spawn background process to send telemetry
-    const error: CLIError = Object.assign(new Error('Received SIGINT'), {
+  let shutdownStarted = false
+  const handleSignal = async (signal: ShutdownSignal): Promise<void> => {
+    if (shutdownStarted) return
+
+    shutdownStarted = true
+
+    // Snapshot the error before waiting so signal-time duration and context are preserved.
+    const error: CLIError = Object.assign(new Error(`Received ${signal}`), {
       cliRunDuration: computeDuration(cliStartTime),
-      context: {
-        isTTY: process.stdin.isTTY,
-      },
+      ...(signal === 'SIGINT' ? {context: {isTTY: process.stdin.isTTY}} : {}),
     })
+
+    try {
+      await awaitProcessShutdown(signal)
+    } catch (shutdownError) {
+      telemetryDebug('Error waiting for process shutdown before sending signal telemetry: %s', shutdownError)
+    }
+
     spawnTelemetryWorker(error)
-    process.exit(1)
-  })
+    // Let the coordinator replay distinct unowned signals after the shutdown barrier settles.
+    setImmediate(() => {
+      // eslint-disable-next-line unicorn/no-process-exit -- preserve the CLI's signal exit status
+      process.exit(1)
+    })
+  }
 
+  process.on('SIGINT', () => {
+    handleSignal('SIGINT').catch(error => telemetryDebug('Error handling SIGINT shutdown: %s', error))
+  })
   process.on('SIGTERM', () => {
-    // Spawn background process to send telemetry
-    const error: CLIError = Object.assign(new Error('Received SIGTERM'), {
-      cliRunDuration: computeDuration(cliStartTime),
-    })
-    spawnTelemetryWorker(error)
-    process.exit(1)
+    handleSignal('SIGTERM').catch(error => telemetryDebug('Error handling SIGTERM shutdown: %s', error))
   })
 }