diff options
| author | Paul Buetow <paul@buetow.org> | 2026-03-23 09:04:17 +0200 |
|---|---|---|
| committer | Paul Buetow <paul@buetow.org> | 2026-03-23 09:04:17 +0200 |
| commit | 462184dff3eef32f01f06634305da1355ac1bec2 (patch) | |
| tree | 026ffaaeacfe152957298c985e1df77ff661b723 /internal/lsp/server.go | |
| parent | 667f2d3384643aa877de2eefcbad3923965bad09 (diff) | |
chore: bump version to v0.25.9v0.25.9
Code quality fixes from audit:
- Log silently discarded errors in status sinks and stats.Update call sites
- Fix json.Marshal errors silently ignored in LSP handlers
- Replace time.Sleep in tests with channel signaling (mcp) and fake clock (stats)
- Make context cancellation work in production time.Sleep sites (handlers_document, cmdentry)
- Remove init()-based provider registration from llm package; use explicit RegisterAllProviders()
- Add WaitGroup goroutine tracking to MCP server Run()
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Diffstat (limited to 'internal/lsp/server.go')
| -rw-r--r-- | internal/lsp/server.go | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/internal/lsp/server.go b/internal/lsp/server.go index 25c5e5c..7675d34 100644 --- a/internal/lsp/server.go +++ b/internal/lsp/server.go @@ -344,13 +344,17 @@ func (s *Server) cancelRequests() { func (s *Server) emitLLMStartStatus(provider, model string) { if s.statusSink != nil { - _ = s.statusSink.SetLLMStart(provider, model) + if err := s.statusSink.SetLLMStart(provider, model); err != nil { + logging.Logf("lsp ", "status sink SetLLMStart error: %v", err) + } } } func (s *Server) emitGlobalStatus(gs GlobalStatus) { if s.statusSink != nil { - _ = s.statusSink.SetGlobal(gs) + if err := s.statusSink.SetGlobal(gs); err != nil { + logging.Logf("lsp ", "status sink SetGlobal error: %v", err) + } } } |
