🤖 fix: correctly detect signal-killed processes in isRunning()

- Check both exitCode and signalCode when determining if process is alive
- Fixes terminate() always sending SIGKILL after SIGTERM, even when process
  already stopped (signal-killed processes have signalCode set but exitCode null)
- Also gate background tools for SSH runtimes (TODO: replace duck-type check
  with proper capability flag once SSH implements spawnBackground)

Author: ethanndickson

src/common/utils/tools/tools.ts

@@ -107,17 +107,14 @@ export async function getToolsForModel(
     // and line number miscalculations. Use file_edit_replace_string instead.
     // file_edit_replace_lines: wrap(createFileEditReplaceLinesTool(config)),
     bash: wrap(createBashTool(config)),
+    // TODO: These aren't supported by the SSH runtime yet, but they will be,
+    // so always add them.
+    bash_background_read: wrap(createBashBackgroundReadTool(config)),
+    bash_background_list: wrap(createBashBackgroundListTool(config)),
+    bash_background_terminate: wrap(createBashBackgroundTerminateTool(config)),
     web_fetch: wrap(createWebFetchTool(config)),
   };
 
-  // Only include background tools when manager is available
-  // (not available in CLI/debug paths)
-  if (config.backgroundProcessManager) {
-    runtimeTools.bash_background_read = wrap(createBashBackgroundReadTool(config));
-    runtimeTools.bash_background_list = wrap(createBashBackgroundListTool(config));
-    runtimeTools.bash_background_terminate = wrap(createBashBackgroundTerminateTool(config));
-  }
-
   // Non-runtime tools execute immediately (no init wait needed)
   const nonRuntimeTools: Record<string, Tool> = {
     propose_plan: createProposePlanTool(config),

src/node/runtime/LocalBackgroundHandle.ts

@@ -76,7 +76,10 @@ export class LocalBackgroundHandle implements BackgroundHandle {
   }
 
   isRunning(): Promise<boolean> {
-    return Promise.resolve(this.disposable.underlying.exitCode === null);
+    const child = this.disposable.underlying;
+    // Process is dead if either exitCode or signalCode is set
+    // (signal-killed processes have signalCode set but exitCode remains null)
+    return Promise.resolve(child.exitCode === null && child.signalCode === null);
   }
 
   async terminate(): Promise<void> {