blackwell-systems/agent-lsp

Bug fixes and test coverage for the daemon broker, warmup gate, and scope config shipped in v0.6.0.

Fixed

• Data race on DaemonInfo: warmup goroutine and main event loop wrote to shared struct without synchronization. Added mutex.
• Data race on lastDisconn: inactivity timeout read without lock. Now synchronized.
• Global firstRefDone state: package-level atomic leaked state between clients in multi-server mode. Moved to per-warmupState field.
• Unprotected socketConn write: daemon Shutdown path set socketConn=nil without holding c.mu. Now locked.
• Panic recovery in broker goroutines: warmup and socket accept goroutines now recover from panics with warning-level logging instead of crashing the daemon silently.
• WriteDaemonInfo error now logged: previously discarded with _ =, causing daemon to appear permanently stuck in "indexing" if the write failed.
• Content-Length parse error handling: malformed headers from socket clients now return an error instead of silently producing contentLength=0.
• Error wrapping in StopDaemon: now includes PID and operation context.

Testing

• 17 new unit tests covering daemon lifecycle (NeedsDaemon, DaemonDir, WriteDaemonInfo round-trip, CleanupStaleDaemons), scope config generation (Python pyrightconfig.json, TypeScript tsconfig.json, Go no-op, backup/restore), and warmup state (FirstRefTimeout, MarkReady, NotifyDiagnostic).