test: add integration tests for daemon server lifecycle (#5)

Add full-pipeline integration tests covering CLI client -> Unix socket ->
server -> PTY -> storage -> read. These are the first tests that exercise
the complete data flow end-to-end.

- Add setupTestServer helper with temp socket dir and MemoryStorage
- Add WithSocketDir server option and NewClientWithSocketPath for testability
- TestLifecycle: create -> send -> read -> stop -> read -> kill
- TestSearchBoundsValidation: negative before/after produces error (not panic)
- TestConcurrentAccess: 5 parallel sessions with send/read
- TestPerCursorReads: independent cursor position tracking
- TestSessionErrorCases: invalid names, duplicates, nonexistent sessions
- Fix data race: remove duplicate cmd.Wait()/proc.Wait() from handleStop
  and handleKill goroutines (captureOutput already handles process reaping)

Author: schovi

internal/daemon/client.go

@@ -11,12 +11,18 @@ import (
 	"github.com/schovi/shelli/internal/wait"
 )
 
-type Client struct{}
+type Client struct {
+	customSocketPath string
+}
 
 func NewClient() *Client {
 	return &Client{}
 }
 
+func NewClientWithSocketPath(path string) *Client {
+	return &Client{customSocketPath: path}
+}
+
 func (c *Client) EnsureDaemon() error {
 	if c.Ping() {
 		return nil
@@ -425,7 +431,11 @@ func (c *Client) Exec(name string, opts ExecOptions) (*ExecResult, error) {
 }
 
 func (c *Client) send(req Request) (*Response, error) {
-	conn, err := net.Dial("unix", SocketPath())
+	sockPath := SocketPath()
+	if c.customSocketPath != "" {
+		sockPath = c.customSocketPath
+	}
+	conn, err := net.Dial("unix", sockPath)
 	if err != nil {
 		return nil, err
 	}

internal/daemon/server.go

@@ -84,6 +84,12 @@ func WithStoppedTTL(ttl time.Duration) ServerOption {
 	}
 }
 
+func WithSocketDir(dir string) ServerOption {
+	return func(s *Server) {
+		s.socketDir = dir
+	}
+}
+
 // Deprecated: use WithStorage instead
 func WithMaxOutputSize(size int) ServerOption {
 	return func(s *Server) {
@@ -159,8 +165,12 @@ func SocketPath() string {
 	return filepath.Join(homeDir, ".shelli", "shelli.sock")
 }
 
+func (s *Server) socketPath() string {
+	return filepath.Join(s.socketDir, "shelli.sock")
+}
+
 func (s *Server) Start() error {
-	sockPath := SocketPath()
+	sockPath := s.socketPath()
 	os.Remove(sockPath)
 
 	listener, err := net.Listen("unix", sockPath)
@@ -242,7 +252,7 @@ func (s *Server) Shutdown() {
 		s.listener.Close()
 		s.listener = nil
 	}
-	os.Remove(SocketPath())
+	os.Remove(s.socketPath())
 }
 
 type Request struct {
@@ -834,14 +844,12 @@ func (s *Server) handleStop(req Request) Response {
 	}
 
 	if h.cmd != nil {
-		cmd := h.cmd
-		proc := cmd.Process
+		proc := h.cmd.Process
 		h.cmd = nil
 		proc.Signal(syscall.SIGTERM)
 		go func() {
 			time.Sleep(KillGracePeriod)
 			proc.Signal(syscall.SIGKILL)
-			cmd.Wait()
 		}()
 	}
 	h.frameDetector = nil
@@ -891,7 +899,6 @@ func (s *Server) handleKill(req Request) Response {
 			proc.Signal(syscall.SIGTERM)
 			time.Sleep(KillGracePeriod)
 			proc.Signal(syscall.SIGKILL)
-			proc.Wait()
 		}()
 	}