fix(process): improve process cleanup by killing entire process group on Unix

Windows or no PID cases fall back to child.kill(). Also removes token display from sub-agent summary UI as it's no longer needed.

Author: moazbuilds

src/cli/tui/routes/workflow/components/timeline/sub-agent-summary.tsx

@@ -9,7 +9,6 @@
 import { Show } from "solid-js"
 import { useTheme } from "@tui/shared/context/theme"
 import type { SubAgentState } from "../../state/types"
-import { formatTokens } from "../../state/formatters"
 
 export interface SubAgentSummaryProps {
   subAgents: SubAgentState[]
@@ -27,12 +26,12 @@ export function SubAgentSummary(props: SubAgentSummaryProps) {
   const arrow = () => (props.isExpanded ? "▼" : "▶")
   const showArrow = () => props.isSelected
 
-  // Count by status
+  // Count by status (sub-agents can only be running, completed, or failed)
   const statusStr = () => {
     const counts = {
       completed: 0,
       running: 0,
-      pending: 0,
+      failed: 0,
     }
 
     for (const agent of props.subAgents) {
@@ -42,26 +41,13 @@ export function SubAgentSummary(props: SubAgentSummaryProps) {
     }
 
     const parts: string[] = []
-    if (counts.completed > 0) parts.push(`${counts.completed} completed`)
+    if (counts.completed > 0) parts.push(`${counts.completed} done`)
     if (counts.running > 0) parts.push(`${counts.running} running`)
-    if (counts.pending > 0) parts.push(`${counts.pending} pending`)
+    if (counts.failed > 0) parts.push(`${counts.failed} failed`)
 
     return parts.join(", ")
   }
 
-  // Aggregate tokens
-  const tokenStr = () => {
-    let totalIn = 0
-    let totalOut = 0
-
-    for (const agent of props.subAgents) {
-      totalIn += agent.telemetry.tokensIn
-      totalOut += agent.telemetry.tokensOut
-    }
-
-    return totalIn > 0 || totalOut > 0 ? formatTokens(totalIn, totalOut) : ""
-  }
-
   // Selection indicator
   const selectionPrefix = () => (showArrow() ? "> " : "  ")
 
@@ -71,7 +57,6 @@ export function SubAgentSummary(props: SubAgentSummaryProps) {
         <text fg={themeCtx.theme.text}>{selectionPrefix()}</text>
         <text fg={themeCtx.theme.textMuted}>
           {arrow()} Sub-agents: {statusStr()}
-          <Show when={tokenStr()}> • {tokenStr()}</Show>
         </text>
       </box>
     </Show>

src/infra/process/spawn.ts

@@ -59,17 +59,39 @@ function resolveCommandExecutable(command: string): string {
 export function killAllActiveProcesses(): void {
   for (const child of activeProcesses) {
     try {
-      child.kill('SIGTERM');
-      // Force kill after 1 second if still running
-      setTimeout(() => {
+      // Kill process group on Unix (for wrapper scripts like CCR/Claude that spawn children)
+      // Using negative PID kills the entire process group, not just the direct child
+      if (process.platform !== 'win32' && child.pid) {
         try {
+          process.kill(-child.pid, 'SIGTERM');
+          // Force kill after 1 second if still running
+          setTimeout(() => {
+            try {
+              process.kill(-child.pid!, 'SIGKILL');
+            } catch {
+              // Process group already dead
+            }
+          }, 1000);
+        } catch {
+          // Fallback to killing just the child if process group kill fails
           if (!child.killed) {
-            child.kill('SIGKILL');
+            child.kill('SIGTERM');
           }
-        } catch {
-          // Process already dead
         }
-      }, 1000);
+      } else {
+        // Windows or no PID: use child.kill()
+        child.kill('SIGTERM');
+        // Force kill after 1 second if still running
+        setTimeout(() => {
+          try {
+            if (!child.killed) {
+              child.kill('SIGKILL');
+            }
+          } catch {
+            // Process already dead
+          }
+        }, 1000);
+      }
     } catch {
       // Ignore errors during cleanup
     }