Replace panics with returned errors in llm.RegisterProvider

RegisterProvider now returns an error for an empty name, a nil factory,
or a duplicate registration instead of panicking, making registration
composable and testable without recover(). RegisterAllProviders caches
the one-time registration error (sync.Once cannot return a value) and
returns it to every caller.

Updated all call sites: hexailsp, hexaicli and hexaiaction propagate the
error with context; test TestMains fail fast via panic. Added unit tests
covering empty name, nil factory, duplicate, success and idempotent
re-registration.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

1 files changed, 3 insertions, 1 deletions

diff --git a/internal/hexailsp/run.go b/internal/hexailsp/run.go
index 242a013..25d1929 100644
--- a/internal/hexailsp/run.go
+++ b/internal/hexailsp/run.go
@@ -54,7 +54,9 @@ 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()
+	if err := llm.RegisterAllProviders(); err != nil {
+		return fmt.Errorf("failed to register LLM providers: %w", err)
+	}
 	return runWithConfigDependencies(logPath, configPath, stdin, stdout, stderr, defaultRunDependencies())
 }