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/hexailsp | |
| 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/hexailsp')
| -rw-r--r-- | internal/hexailsp/run.go | 1 | ||||
| -rw-r--r-- | internal/hexailsp/run_test.go | 7 |
2 files changed, 8 insertions, 0 deletions
diff --git a/internal/hexailsp/run.go b/internal/hexailsp/run.go index 57e7476..242a013 100644 --- a/internal/hexailsp/run.go +++ b/internal/hexailsp/run.go @@ -54,6 +54,7 @@ func Run(logPath string, stdin io.Reader, stdout io.Writer, stderr io.Writer) er // RunWithConfig is like Run but accepts an explicit config file path. func RunWithConfig(logPath string, configPath string, stdin io.Reader, stdout io.Writer, stderr io.Writer) error { + llm.RegisterAllProviders() return runWithConfigDependencies(logPath, configPath, stdin, stdout, stderr, defaultRunDependencies()) } diff --git a/internal/hexailsp/run_test.go b/internal/hexailsp/run_test.go index 583c6c8..badb27c 100644 --- a/internal/hexailsp/run_test.go +++ b/internal/hexailsp/run_test.go @@ -15,6 +15,13 @@ import ( "codeberg.org/snonux/hexai/internal/lsp" ) +// TestMain registers all built-in LLM providers before tests run, mirroring +// the explicit registration done in production binaries via RunWithConfig. +func TestMain(m *testing.M) { + llm.RegisterAllProviders() + os.Exit(m.Run()) +} + // fake server capturing options and recording run calls type fakeServer struct { ran bool |
