lsp/config: make completion trigger characters configurable

- Add trigger_characters to JSON config and ServerOptions
- Store on server and advertise in initialize
- Update README and example config
- Preserve previous defaults when unset

1 files changed, 31 insertions, 31 deletions

diff --git a/internal/llm/provider.go b/internal/llm/provider.go
index c7367ed..6c6cf04 100644
--- a/internal/llm/provider.go
+++ b/internal/llm/provider.go
@@ -1,9 +1,9 @@
 package llm
 
 import (
-    "context"
-    "errors"
-    "strings"
+	"context"
+	"errors"
+	"strings"
 )
 
 // Message represents a chat-style prompt message.
@@ -15,12 +15,12 @@ type Message struct {
 // Client is a minimal LLM provider interface.
 // Future providers (Ollama, etc.) should implement this.
 type Client interface {
-    // Chat sends chat messages and returns the assistant text.
-    Chat(ctx context.Context, messages []Message, opts ...RequestOption) (string, error)
-    // Name returns the provider's short name (e.g., "openai", "ollama").
-    Name() string
-    // DefaultModel returns the configured default model name.
-    DefaultModel() string
+	// Chat sends chat messages and returns the assistant text.
+	Chat(ctx context.Context, messages []Message, opts ...RequestOption) (string, error)
+	// Name returns the provider's short name (e.g., "openai", "ollama").
+	Name() string
+	// DefaultModel returns the configured default model name.
+	DefaultModel() string
 }
 
 // Options for a request. Providers may ignore unsupported fields.
@@ -43,32 +43,32 @@ func WithStop(stop ...string) RequestOption {
 
 // Config defines provider configuration read from the Hexai config file.
 type Config struct {
-    Provider       string
-    // OpenAI options
-    OpenAIBaseURL  string
-    OpenAIModel    string
-    // Ollama options
-    OllamaBaseURL  string
-    OllamaModel    string
+	Provider string
+	// OpenAI options
+	OpenAIBaseURL string
+	OpenAIModel   string
+	// Ollama options
+	OllamaBaseURL string
+	OllamaModel   string
 }
 
 // NewFromConfig creates an LLM client using only the supplied configuration.
 // The OpenAI API key is supplied separately and may be read from the environment
 // by the caller; other environment-based configuration is not used.
 func NewFromConfig(cfg Config, openAIAPIKey string) (Client, error) {
-    p := strings.ToLower(strings.TrimSpace(cfg.Provider))
-    if p == "" {
-        p = "openai"
-    }
-    switch p {
-    case "openai":
-        if strings.TrimSpace(openAIAPIKey) == "" {
-            return nil, errors.New("missing OPENAI_API_KEY for provider openai")
-        }
-        return newOpenAI(cfg.OpenAIBaseURL, cfg.OpenAIModel, openAIAPIKey), nil
-    case "ollama":
-        return newOllama(cfg.OllamaBaseURL, cfg.OllamaModel), nil
-    default:
-        return nil, errors.New("unknown LLM provider: " + p)
-    }
+	p := strings.ToLower(strings.TrimSpace(cfg.Provider))
+	if p == "" {
+		p = "openai"
+	}
+	switch p {
+	case "openai":
+		if strings.TrimSpace(openAIAPIKey) == "" {
+			return nil, errors.New("missing OPENAI_API_KEY for provider openai")
+		}
+		return newOpenAI(cfg.OpenAIBaseURL, cfg.OpenAIModel, openAIAPIKey), nil
+	case "ollama":
+		return newOllama(cfg.OllamaBaseURL, cfg.OllamaModel), nil
+	default:
+		return nil, errors.New("unknown LLM provider: " + p)
+	}
 }