fix: reap child processes on session stop/kill to prevent zombies

captureOutput goroutine could exit via the done channel without calling
cmd.Wait(), leaving terminated child processes as zombies in the process
table. Moved cmd.Wait() and p.Close() into a defer so the child is
always reaped regardless of exit path.

Author: schovi

internal/daemon/server.go

@@ -477,6 +477,11 @@ func (s *Server) captureOutput(name string, h *sessionHandle) {
 
 	f := p.File()
 
+	defer func() {
+		cmd.Wait()
+		p.Close()
+	}()
+
 	if detector != nil {
 		defer func() {
 			if pending := detector.Flush(); len(pending) > 0 {
@@ -513,13 +518,10 @@ func (s *Server) captureOutput(name string, h *sessionHandle) {
 			}
 		}
 		if err != nil && !isTimeout(err) {
-			break
+			return
 		}
 	}
 
-	cmd.Wait()
-	p.Close()
-
 	s.mu.Lock()
 	defer s.mu.Unlock()