review changes

1 files changed, 40 insertions, 41 deletions

diff --git a/internal/llm/provider.go b/internal/llm/provider.go
index 09e97e6..c605081 100644
--- a/internal/llm/provider.go
+++ b/internal/llm/provider.go
@@ -1,5 +1,4 @@
 // Summary: LLM provider interfaces, request options, configuration, and factory to build a client from config.
-// Not yet reviewed by a human
 package llm
 
 import (
@@ -17,22 +16,22 @@ 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
 }
 
 // Streamer is an optional interface that providers may implement to support
 // token-by-token streaming responses. Callers can type-assert to Streamer and
 // fall back to Client.Chat when not implemented.
 type Streamer interface {
-    // ChatStream sends chat messages and invokes onDelta with incremental text
-    // chunks as they are produced by the model. Implementations should call
-    // onDelta with empty strings sparingly (prefer only non-empty chunks).
-    ChatStream(ctx context.Context, messages []Message, onDelta func(string), opts ...RequestOption) error
+	// ChatStream sends chat messages and invokes onDelta with incremental text
+	// chunks as they are produced by the model. Implementations should call
+	// onDelta with empty strings sparingly (prefer only non-empty chunks).
+	ChatStream(ctx context.Context, messages []Message, onDelta func(string), opts ...RequestOption) error
 }
 
 // Options for a request. Providers may ignore unsupported fields.
@@ -55,40 +54,40 @@ 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
-    // Copilot options
-    CopilotBaseURL string
-    CopilotModel   string
+	Provider string
+	// OpenAI options
+	OpenAIBaseURL string
+	OpenAIModel   string
+	// Ollama options
+	OllamaBaseURL string
+	OllamaModel   string
+	// Copilot options
+	CopilotBaseURL string
+	CopilotModel   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, copilotAPIKey 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
-    case "copilot":
-        if strings.TrimSpace(copilotAPIKey) == "" {
-            return nil, errors.New("missing COPILOT_API_KEY for provider copilot")
-        }
-        return newCopilot(cfg.CopilotBaseURL, cfg.CopilotModel, copilotAPIKey), 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
+	case "copilot":
+		if strings.TrimSpace(copilotAPIKey) == "" {
+			return nil, errors.New("missing COPILOT_API_KEY for provider copilot")
+		}
+		return newCopilot(cfg.CopilotBaseURL, cfg.CopilotModel, copilotAPIKey), nil
+	default:
+		return nil, errors.New("unknown LLM provider: " + p)
+	}
 }