hexai-lsp: Go Coverage Report

- -

// Summary: Hexai LSP entrypoint; parses flags and delegates to internal/hexailsp.
-package main
-
-import (
-        "flag"
-        "fmt"
-        "log"
-        "os"
-        "strings"
-
-        "codeberg.org/snonux/hexai/internal"
-        "codeberg.org/snonux/hexai/internal/appconfig"
-        "codeberg.org/snonux/hexai/internal/hexailsp"
-)
-
-func main() {
-        logPath := flag.String("log", "/tmp/hexai-lsp.log", "path to log file (optional)")
-        defaultCfg := defaultConfigPath()
-        configPath := flag.String("config", "", fmt.Sprintf("path to config file (default: %s)", defaultCfg))
-        showVersion := flag.Bool("version", false, "print version and exit")
-        flag.Parse()
-        if *showVersion {
-                log.Println(internal.Version)
-                return
-        }
-
-        path := strings.TrimSpace(*configPath)
-        if err := hexailsp.RunWithConfig(*logPath, path, os.Stdin, os.Stdout, os.Stderr); err != nil {
-                log.Fatalf("server error: %v", err)
-        }
-}
-
-func defaultConfigPath() string {
-        path, err := appconfig.ConfigPath()
-        if err != nil {
-                return "$XDG_CONFIG_HOME/hexai/config.toml"
-        }
-        return path
-}
-

- -

package main
-
-import (
-        "context"
-        "flag"
-        "fmt"
-        "os"
-        "strings"
-
-        "codeberg.org/snonux/hexai/internal/appconfig"
-        "codeberg.org/snonux/hexai/internal/hexaiaction"
-)
-
-func main() {
-        infile := flag.String("infile", "", "Read input from this file instead of stdin")
-        outfile := flag.String("outfile", "", "Write output to this file instead of stdout")
-        uiChild := flag.Bool("ui-child", false, "INTERNAL: run interactive UI and write to -outfile atomically")
-        defaultPath := defaultConfigPath()
-        configPath := flag.String("config", "", fmt.Sprintf("path to config file (default: %s)", defaultPath))
-        tmuxTarget := flag.String("tmux-target", "", "tmux split target (advanced)")
-        tmuxSplit := flag.String("tmux-split", "v", "tmux split orientation: v or h")
-        tmuxPercent := flag.Int("tmux-percent", 33, "tmux split size percentage (1-100)")
-        flag.Parse()
-
-        opts := hexaiaction.Options{
-                Infile: *infile, Outfile: *outfile,
-                UIChild: *uiChild, TmuxTarget: *tmuxTarget, TmuxSplit: *tmuxSplit, TmuxPercent: *tmuxPercent,
-        }
-        ctx := context.Background()
-        if path := strings.TrimSpace(*configPath); path != "" {
-                ctx = hexaiaction.WithConfigPath(ctx, path)
-        }
-        if err := hexaiaction.RunCommand(ctx, opts, os.Stdin, os.Stdout, os.Stderr); err != nil {
-                fmt.Fprintln(os.Stderr, err)
-                os.Exit(1)
-        }
-}
-
-func defaultConfigPath() string {
-        path, err := appconfig.ConfigPath()
-        if err != nil {
-                return "$XDG_CONFIG_HOME/hexai/config.toml"
-        }
-        return path
-}
-

- -

// Summary: Hexai CLI entrypoint; parses flags and delegates to internal/hexaicli.
-package main
-
-import (
-        "context"
-        "flag"
-        "fmt"
-        "io"
-        "log"
-        "os"
-        "strconv"
-        "strings"
-
-        "codeberg.org/snonux/hexai/internal"
-        "codeberg.org/snonux/hexai/internal/appconfig"
-        "codeberg.org/snonux/hexai/internal/hexaicli"
-)
-
-func main() {
-        configPath, remaining := splitConfigPath(os.Args[1:])
-        logger := log.New(io.Discard, "", 0)
-        cfg := appconfig.LoadWithOptions(logger, appconfig.LoadOptions{ConfigPath: configPath})
-        cliEntries := cfg.CLIConfigs
-        if len(cliEntries) == 0 {
-                cliEntries = []appconfig.SurfaceConfig{{Provider: cfg.Provider}}
-        }
-        fs := flag.NewFlagSet(os.Args[0], flag.ExitOnError)
-        defaultPath := defaultConfigPath()
-        configFlag := fs.String("config", configPath, fmt.Sprintf("path to config file (default: %s)", defaultPath))
-        showVersion := fs.Bool("version", false, "print version and exit")
-        selectedFlags := make([]bool, len(cliEntries))
-        for i, entry := range cliEntries {
-                name := strconv.Itoa(i)
-                provider := strings.TrimSpace(entry.Provider)
-                if provider == "" {
-                        provider = cfg.Provider
-                }
-                model := strings.TrimSpace(entry.Model)
-                if model == "" {
-                        model = pickDefaultModel(cfg, provider)
-                }
-                desc := fmt.Sprintf("use only provider #%d (%s:%s)", i, provider, model)
-                fs.BoolVar(&selectedFlags[i], name, false, desc)
-        }
-        _ = fs.Parse(remaining)
-        if *showVersion {
-                fmt.Fprintln(os.Stdout, internal.Version)
-                return
-        }
-        var selection []int
-        for i, sel := range selectedFlags {
-                if sel {
-                        selection = append(selection, i)
-                }
-        }
-        finalPath := strings.TrimSpace(*configFlag)
-        if finalPath == "" {
-                finalPath = configPath
-        }
-        ctx := context.Background()
-        if finalPath != "" {
-                ctx = hexaicli.WithCLIConfigPath(ctx, finalPath)
-        }
-        if len(selection) > 0 {
-                ctx = hexaicli.WithCLISelection(ctx, selection)
-        }
-        if err := hexaicli.Run(ctx, fs.Args(), os.Stdin, os.Stdout, os.Stderr); err != nil {
-                os.Exit(1)
-        }
-}
-
-func splitConfigPath(args []string) (string, []string) {
-        var path string
-        rest := make([]string, 0, len(args))
-        skip := false
-        for i := 0; i < len(args); i++ {
-                if skip {
-                        skip = false
-                        continue
-                }
-                arg := args[i]
-                switch {
-                case arg == "--config" || arg == "-config":
-                        if i+1 < len(args) {
-                                path = args[i+1]
-                                skip = true
-                        }
-                case strings.HasPrefix(arg, "--config="):
-                        path = arg[len("--config="):]
-                case strings.HasPrefix(arg, "-config="):
-                        path = arg[len("-config="):]
-                default:
-                        rest = append(rest, arg)
-                }
-        }
-        return strings.TrimSpace(path), rest
-}
-
-func pickDefaultModel(cfg appconfig.App, provider string) string {
-        switch strings.ToLower(strings.TrimSpace(provider)) {
-        case "ollama":
-                return strings.TrimSpace(cfg.OllamaModel)
-        case "copilot":
-                return strings.TrimSpace(cfg.CopilotModel)
-        default:
-                return strings.TrimSpace(cfg.OpenAIModel)
-        }
-}
-
-func defaultConfigPath() string {
-        cfgPath, err := appconfig.ConfigPath()
-        if err != nil {
-                return "$XDG_CONFIG_HOME/hexai/config.toml"
-        }
-        return cfgPath
-}
-

- -

// Summary: Application configuration model and loader; reads ~/.config/hexai/config.toml and merges defaults.
-package appconfig
-
-import (
-        "fmt"
-        "log"
-        "os"
-        "path/filepath"
-        "slices"
-        "strconv"
-        "strings"
-
-        "github.com/pelletier/go-toml/v2"
-)
-
-// SurfaceConfig describes a provider/model pairing (with optional temperature).
-type SurfaceConfig struct {
-        Provider    string
-        Model       string
-        Temperature *float64
-}
-
-// App holds user-configurable settings read from ~/.config/hexai/config.toml.
-type App struct {
-        MaxTokens          int    `json:"max_tokens" toml:"max_tokens"`
-        ContextMode        string `json:"context_mode" toml:"context_mode"`
-        ContextWindowLines int    `json:"context_window_lines" toml:"context_window_lines"`
-        MaxContextTokens   int    `json:"max_context_tokens" toml:"max_context_tokens"`
-        LogPreviewLimit    int    `json:"log_preview_limit" toml:"log_preview_limit"`
-        // Single knob for LSP requests; if set, overrides hardcoded temps in LSP.
-        CodingTemperature *float64 `json:"coding_temperature" toml:"coding_temperature"`
-        // Minimum identifier characters required for manual (TriggerKind=1) invoke
-        // to proceed without structural triggers. 0 means always allow.
-        ManualInvokeMinPrefix int `json:"manual_invoke_min_prefix" toml:"manual_invoke_min_prefix"`
-
-        // Completion debounce in milliseconds. When > 0, the server waits until
-        // there has been no text change for at least this duration before sending
-        // an LLM completion request.
-        CompletionDebounceMs int `json:"completion_debounce_ms" toml:"completion_debounce_ms"`
-        // Completion throttle in milliseconds. When > 0, caps the minimum spacing
-        // between LLM requests (both chat and code-completer paths).
-        CompletionThrottleMs int `json:"completion_throttle_ms" toml:"completion_throttle_ms"`
-
-        TriggerCharacters []string `json:"trigger_characters" toml:"trigger_characters"`
-        Provider          string   `json:"provider" toml:"provider"`
-
-        // Inline prompt trigger characters (default: >!text> and >>!text>)
-        InlineOpen  string `json:"inline_open" toml:"inline_open"`
-        InlineClose string `json:"inline_close" toml:"inline_close"`
-        // In-editor chat triggers (default: suffix ">" after one of [?, !, :, ;])
-        ChatSuffix   string   `json:"chat_suffix" toml:"chat_suffix"`
-        ChatPrefixes []string `json:"chat_prefixes" toml:"chat_prefixes"`
-
-        // Provider-specific options
-        OpenAIBaseURL string `json:"openai_base_url" toml:"openai_base_url"`
-        OpenAIModel   string `json:"openai_model" toml:"openai_model"`
-        // Default temperature for OpenAI requests (nil means use provider default)
-        OpenAITemperature *float64 `json:"openai_temperature" toml:"openai_temperature"`
-        OpenRouterBaseURL string   `json:"openrouter_base_url" toml:"openrouter_base_url"`
-        OpenRouterModel   string   `json:"openrouter_model" toml:"openrouter_model"`
-        // Default temperature for OpenRouter requests (nil means use provider default)
-        OpenRouterTemperature *float64 `json:"openrouter_temperature" toml:"openrouter_temperature"`
-        OllamaBaseURL         string   `json:"ollama_base_url" toml:"ollama_base_url"`
-        OllamaModel           string   `json:"ollama_model" toml:"ollama_model"`
-        // Default temperature for Ollama requests (nil means use provider default)
-        OllamaTemperature *float64 `json:"ollama_temperature" toml:"ollama_temperature"`
-        CopilotBaseURL    string   `json:"copilot_base_url" toml:"copilot_base_url"`
-        CopilotModel      string   `json:"copilot_model" toml:"copilot_model"`
-        // Default temperature for Copilot requests (nil means use provider default)
-        CopilotTemperature *float64 `json:"copilot_temperature" toml:"copilot_temperature"`
-
-        // Per-surface provider/model configurations (ordered; first entry is primary)
-        CompletionConfigs []SurfaceConfig `json:"-" toml:"-"`
-        CodeActionConfigs []SurfaceConfig `json:"-" toml:"-"`
-        ChatConfigs       []SurfaceConfig `json:"-" toml:"-"`
-        CLIConfigs        []SurfaceConfig `json:"-" toml:"-"`
-
-        // Prompt templates (configured only via file; no env overrides)
-        // Completion/chat/code action/CLI prompt strings. See config.toml.example for placeholders.
-        // Completion
-        PromptCompletionSystemGeneral string `json:"-" toml:"-"`
-        PromptCompletionSystemParams  string `json:"-" toml:"-"`
-        PromptCompletionSystemInline  string `json:"-" toml:"-"`
-        PromptCompletionUserGeneral   string `json:"-" toml:"-"`
-        PromptCompletionUserParams    string `json:"-" toml:"-"`
-        PromptCompletionExtraHeader   string `json:"-" toml:"-"`
-        // Provider-native code-completer
-        PromptNativeCompletion string `json:"-" toml:"-"`
-        // In-editor chat
-        PromptChatSystem string `json:"-" toml:"-"`
-        // Code actions
-        PromptCodeActionRewriteSystem     string `json:"-" toml:"-"`
-        PromptCodeActionDiagnosticsSystem string `json:"-" toml:"-"`
-        PromptCodeActionDocumentSystem    string `json:"-" toml:"-"`
-        PromptCodeActionRewriteUser       string `json:"-" toml:"-"`
-        PromptCodeActionDiagnosticsUser   string `json:"-" toml:"-"`
-        PromptCodeActionDocumentUser      string `json:"-" toml:"-"`
-        PromptCodeActionGoTestSystem      string `json:"-" toml:"-"`
-        PromptCodeActionGoTestUser        string `json:"-" toml:"-"`
-        PromptCodeActionSimplifySystem    string `json:"-" toml:"-"`
-        PromptCodeActionSimplifyUser      string `json:"-" toml:"-"`
-        // CLI
-        PromptCLIDefaultSystem string `json:"-" toml:"-"`
-        PromptCLIExplainSystem string `json:"-" toml:"-"`
-
-        // Custom code actions and tmux integration
-        CustomActions        []CustomAction `json:"-" toml:"-"`
-        TmuxCustomMenuHotkey string         `json:"-" toml:"-"`
-        // Stats
-        StatsWindowMinutes int `json:"-" toml:"-"`
-}
-
-// CustomAction describes a user-defined code action.
-type CustomAction struct {
-        ID          string
-        Title       string
-        Kind        string // optional; default "refactor"
-        Scope       string // "selection" (default) | "diagnostics"
-        Hotkey      string // optional, used by tmux submenu
-        Instruction string // optional; if set and User is empty, use global rewrite templates
-        System      string // optional; used only when User is set
-        User        string // optional; if set, render with available vars
-}
-
-// Constructor: defaults for App (kept first among functions)
-func newDefaultConfig() App {
-        // Coding-friendly default temperature across providers
-        // Users can override per provider in config.toml (including 0.0).
-        t := 0.2
-        return App{
-                MaxTokens:             4000,
-                ContextMode:           "always-full",
-                ContextWindowLines:    120,
-                MaxContextTokens:      4000,
-                LogPreviewLimit:       100,
-                CodingTemperature:     &t,
-                OpenAITemperature:     &t,
-                OllamaTemperature:     &t,
-                CopilotTemperature:    &t,
-                ManualInvokeMinPrefix: 0,
-                CompletionDebounceMs:  800,
-                CompletionThrottleMs:  0,
-                // Inline/chat trigger defaults
-                InlineOpen:   ">!",
-                InlineClose:  ">",
-                ChatSuffix:   ">",
-                ChatPrefixes: []string{"?", "!", ":", ";"},
-
-                // Default prompt templates (match current hard-coded strings)
-                PromptCompletionSystemParams:  "You are a code completion engine for function signatures. Return only the parameter list contents (without parentheses), no braces, no prose. Prefer idiomatic names and types.",
-                PromptCompletionUserParams:    "Cursor is inside the function parameter list. Suggest only the parameter list (no parentheses).\nFunction line: {{function}}\nCurrent line (cursor at {{char}}): {{current}}",
-                PromptCompletionSystemGeneral: "You are a terse code completion engine. Return only the code to insert, no surrounding prose or backticks. Only continue from the cursor; never repeat characters already present to the left of the cursor on the current line (e.g., if 'name :=' is already typed, only return the right-hand side expression).",
-                PromptCompletionUserGeneral:   "Provide the next likely code to insert at the cursor.\nFile: {{file}}\nFunction/context: {{function}}\nAbove line: {{above}}\nCurrent line (cursor at character {{char}}): {{current}}\nBelow line: {{below}}\nOnly return the completion snippet.",
-                PromptCompletionSystemInline:  "You are a precise code completion/refactoring engine. Output only the code to insert with no prose, no comments, and no backticks. Return raw code only.",
-                PromptCompletionExtraHeader:   "Additional context:\n{{context}}",
-
-                PromptNativeCompletion: "// Path: {{path}}\n{{before}}",
-
-                PromptChatSystem: "You are a helpful coding assistant. Answer concisely and clearly.",
-
-                PromptCodeActionRewriteSystem:     "You are a precise code refactoring engine. Rewrite the given code strictly according to the instruction. Return only the updated code with no prose or backticks. Preserve formatting where reasonable.",
-                PromptCodeActionDiagnosticsSystem: "You are a precise code fixer. Resolve the given diagnostics by editing only the selected code. Return only the corrected code with no prose or backticks. Keep behavior and style, and avoid unrelated changes.",
-                PromptCodeActionDocumentSystem:    "You are a precise code documentation engine. Add idiomatic documentation comments to the given code. Preserve exact behavior and formatting as much as possible. Return only the updated code with comments, no prose or backticks.",
-                PromptCodeActionRewriteUser:       "Instruction: {{instruction}}\n\nSelected code to transform:\n{{selection}}",
-                PromptCodeActionDiagnosticsUser:   "Diagnostics to resolve (selection only):\n{{diagnostics}}\n\nSelected code:\n{{selection}}",
-                PromptCodeActionDocumentUser:      "Add documentation comments to this code:\n{{selection}}",
-                PromptCodeActionGoTestSystem:      "You are a precise Go unit test generator. Given a Go function, write one or more Test* functions using the testing package. Do NOT include package or imports, only the test function(s). Prefer table-driven tests. Keep it minimal and idiomatic.",
-                PromptCodeActionGoTestUser:        "Function under test:\n{{function}}",
-                PromptCodeActionSimplifySystem:    "You are a precise code improvement engine. Simplify and improve the given code while preserving behavior. Return only the improved code with no prose or backticks.",
-                PromptCodeActionSimplifyUser:      "Improve this code:\n{{selection}}",
-
-                PromptCLIDefaultSystem: "You are Hexai CLI. Default to very short, concise answers. If the user asks for commands, output only the commands (one per line) with no commentary or explanation. Only when the word 'explain' appears in the prompt, produce a verbose explanation.",
-                PromptCLIExplainSystem: "You are Hexai CLI. The user requested an explanation. Provide a clear, verbose explanation with reasoning and details. If commands are needed, include them with brief context.",
-
-                // Stats
-                StatsWindowMinutes: 60,
-        }
-}
-
-// Load reads configuration from a file and merges with defaults.
-// It respects the XDG Base Directory Specification.
-func Load(logger *log.Logger) App { return LoadWithOptions(logger, LoadOptions{}) }
-
-// LoadOptions tune how configuration is loaded at runtime.
-type LoadOptions struct {
-        // IgnoreEnv skips applying environment overrides when true.
-        IgnoreEnv  bool
-        ConfigPath string
-}
-
-// LoadWithOptions reads configuration and applies the requested loading options.
-func LoadWithOptions(logger *log.Logger, opts LoadOptions) App {
-        cfg := newDefaultConfig()
-        if logger == nil {
-                return cfg // Return defaults if no logger is provided (e.g. in tests)
-        }
-
-        configPath := strings.TrimSpace(opts.ConfigPath)
-        if configPath != "" {
-                if fileCfg, err := loadFromFile(configPath, logger); err == nil && fileCfg != nil {
-                        cfg.mergeWith(fileCfg)
-                } else if err != nil {
-                        logger.Printf("cannot open config file %s: %v", configPath, err)
-                }
-        } else {
-                path, err := getConfigPath()
-                if err != nil {
-                        logger.Printf("%v", err)
-                } else if fileCfg, err := loadFromFile(path, logger); err == nil && fileCfg != nil {
-                        cfg.mergeWith(fileCfg)
-                }
-        }
-
-        if !opts.IgnoreEnv {
-                // Environment overrides (take precedence over file)
-                if envCfg := loadFromEnv(logger); envCfg != nil {
-                        cfg.mergeWith(envCfg)
-                }
-        }
-        return cfg
-}
-
-// Private helpers
-// Sectioned (table-based) file format only.
-type fileConfig struct {
-        // Section tables only (flat keys are not allowed)
-        General    sectionGeneral    `toml:"general"`
-        Logging    sectionLogging    `toml:"logging"`
-        Completion sectionCompletion `toml:"completion"`
-        Triggers   sectionTriggers   `toml:"triggers"`
-        Inline     sectionInline     `toml:"inline"`
-        Chat       sectionChat       `toml:"chat"`
-        Provider   sectionProvider   `toml:"provider"`
-        OpenAI     sectionOpenAI     `toml:"openai"`
-        OpenRouter sectionOpenRouter `toml:"openrouter"`
-        Copilot    sectionCopilot    `toml:"copilot"`
-        Ollama     sectionOllama     `toml:"ollama"`
-        Prompts    sectionPrompts    `toml:"prompts"`
-        Tmux       sectionTmux       `toml:"tmux"`
-        Stats      sectionStats      `toml:"stats"`
-}
-
-type sectionGeneral struct {
-        MaxTokens          int      `toml:"max_tokens"`
-        ContextMode        string   `toml:"context_mode"`
-        ContextWindowLines int      `toml:"context_window_lines"`
-        MaxContextTokens   int      `toml:"max_context_tokens"`
-        CodingTemperature  *float64 `toml:"coding_temperature"`
-}
-
-type sectionLogging struct {
-        LogPreviewLimit int `toml:"log_preview_limit"`
-}
-
-type sectionCompletion struct {
-        CompletionDebounceMs  int `toml:"completion_debounce_ms"`
-        CompletionThrottleMs  int `toml:"completion_throttle_ms"`
-        ManualInvokeMinPrefix int `toml:"manual_invoke_min_prefix"`
-}
-
-type sectionTriggers struct {
-        TriggerCharacters []string `toml:"trigger_characters"`
-}
-
-type sectionInline struct {
-        InlineOpen  string `toml:"inline_open"`
-        InlineClose string `toml:"inline_close"`
-}
-
-type sectionChat struct {
-        ChatSuffix   string   `toml:"chat_suffix"`
-        ChatPrefixes []string `toml:"chat_prefixes"`
-}
-
-type sectionProvider struct {
-        Name string `toml:"name"`
-}
-
-type sectionStats struct {
-        WindowMinutes int `toml:"window_minutes"`
-}
-
-type sectionOpenAI struct {
-        Model       string            `toml:"model"`
-        BaseURL     string            `toml:"base_url"`
-        Temperature *float64          `toml:"temperature"`
-        Presets     map[string]string `toml:"presets"`
-}
-
-func (s sectionOpenAI) isZero() bool {
-        return strings.TrimSpace(s.Model) == "" && strings.TrimSpace(s.BaseURL) == "" && s.Temperature == nil && len(s.Presets) == 0
-}
-
-func (s sectionOpenAI) resolvedModel() string {
-        model := strings.TrimSpace(s.Model)
-        if model == "" {
-                return ""
-        }
-        if len(s.Presets) == 0 {
-                return model
-        }
-        if mapped := strings.TrimSpace(s.Presets[model]); mapped != "" {
-                return mapped
-        }
-        lower := strings.ToLower(model)
-        for k, v := range s.Presets {
-                if strings.ToLower(strings.TrimSpace(k)) == lower {
-                        if mapped := strings.TrimSpace(v); mapped != "" {
-                                return mapped
-                        }
-                }
-        }
-        return model
-}
-
-type sectionOpenRouter struct {
-        Model       string   `toml:"model"`
-        BaseURL     string   `toml:"base_url"`
-        Temperature *float64 `toml:"temperature"`
-}
-
-type sectionCopilot struct {
-        Model       string   `toml:"model"`
-        BaseURL     string   `toml:"base_url"`
-        Temperature *float64 `toml:"temperature"`
-}
-
-type sectionOllama struct {
-        Model       string   `toml:"model"`
-        BaseURL     string   `toml:"base_url"`
-        Temperature *float64 `toml:"temperature"`
-}
-
-// Prompts sections
-type sectionPrompts struct {
-        Completion     sectionPromptsCompletion     `toml:"completion"`
-        Chat           sectionPromptsChat           `toml:"chat"`
-        CodeAction     sectionPromptsCodeAction     `toml:"code_action"`
-        CLI            sectionPromptsCLI            `toml:"cli"`
-        ProviderNative sectionPromptsProviderNative `toml:"provider_native"`
-}
-
-type sectionPromptsCompletion struct {
-        SystemGeneral string `toml:"system_general"`
-        SystemParams  string `toml:"system_params"`
-        SystemInline  string `toml:"system_inline"`
-        UserGeneral   string `toml:"user_general"`
-        UserParams    string `toml:"user_params"`
-        ExtraHeader   string `toml:"additional_context"`
-}
-
-type sectionPromptsChat struct {
-        System string `toml:"system"`
-}
-
-type sectionPromptsCodeAction struct {
-        RewriteSystem     string                `toml:"rewrite_system"`
-        DiagnosticsSystem string                `toml:"diagnostics_system"`
-        DocumentSystem    string                `toml:"document_system"`
-        RewriteUser       string                `toml:"rewrite_user"`
-        DiagnosticsUser   string                `toml:"diagnostics_user"`
-        DocumentUser      string                `toml:"document_user"`
-        GoTestSystem      string                `toml:"go_test_system"`
-        GoTestUser        string                `toml:"go_test_user"`
-        SimplifySystem    string                `toml:"simplify_system"`
-        SimplifyUser      string                `toml:"simplify_user"`
-        Custom            []sectionCustomAction `toml:"custom"`
-}
-
-type sectionPromptsCLI struct {
-        DefaultSystem string `toml:"default_system"`
-        ExplainSystem string `toml:"explain_system"`
-}
-
-type sectionPromptsProviderNative struct {
-        Completion string `toml:"completion"`
-}
-
-type sectionCustomAction struct {
-        ID          string `toml:"id"`
-        Title       string `toml:"title"`
-        Kind        string `toml:"kind"`
-        Scope       string `toml:"scope"`
-        Hotkey      string `toml:"hotkey"`
-        Instruction string `toml:"instruction"`
-        System      string `toml:"system"`
-        User        string `toml:"user"`
-}
-
-type sectionTmux struct {
-        CustomMenuHotkey string `toml:"custom_menu_hotkey"`
-}
-
-func (fc *fileConfig) toApp() App {
-        out := App{}
-
-        // Merge section: general
-        if (fc.General != sectionGeneral{}) || fc.General.CodingTemperature != nil {
-                tmp := App{
-                        MaxTokens:          fc.General.MaxTokens,
-                        ContextMode:        fc.General.ContextMode,
-                        ContextWindowLines: fc.General.ContextWindowLines,
-                        MaxContextTokens:   fc.General.MaxContextTokens,
-                        CodingTemperature:  fc.General.CodingTemperature,
-                }
-                out.mergeBasics(&tmp)
-        }
-
-        // logging
-        if (fc.Logging != sectionLogging{}) {
-                tmp := App{LogPreviewLimit: fc.Logging.LogPreviewLimit}
-                out.mergeBasics(&tmp)
-        }
-
-        // completion
-        if (fc.Completion != sectionCompletion{}) {
-                tmp := App{
-                        CompletionDebounceMs:  fc.Completion.CompletionDebounceMs,
-                        CompletionThrottleMs:  fc.Completion.CompletionThrottleMs,
-                        ManualInvokeMinPrefix: fc.Completion.ManualInvokeMinPrefix,
-                }
-                out.mergeBasics(&tmp)
-        }
-
-        // triggers
-        if len(fc.Triggers.TriggerCharacters) > 0 {
-                tmp := App{TriggerCharacters: fc.Triggers.TriggerCharacters}
-                out.mergeBasics(&tmp)
-        }
-
-        // inline
-        if (fc.Inline != sectionInline{}) {
-                tmp := App{InlineOpen: fc.Inline.InlineOpen, InlineClose: fc.Inline.InlineClose}
-                out.mergeBasics(&tmp)
-        }
-
-        // chat
-        if strings.TrimSpace(fc.Chat.ChatSuffix) != "" || len(fc.Chat.ChatPrefixes) > 0 {
-                tmp := App{ChatSuffix: fc.Chat.ChatSuffix, ChatPrefixes: fc.Chat.ChatPrefixes}
-                out.mergeBasics(&tmp)
-        }
-
-        // provider
-        if strings.TrimSpace(fc.Provider.Name) != "" {
-                tmp := App{Provider: fc.Provider.Name}
-                out.mergeBasics(&tmp)
-        }
-
-        // openai
-        if !fc.OpenAI.isZero() || fc.OpenAI.Temperature != nil {
-                tmp := App{
-                        OpenAIBaseURL:     fc.OpenAI.BaseURL,
-                        OpenAIModel:       fc.OpenAI.resolvedModel(),
-                        OpenAITemperature: fc.OpenAI.Temperature,
-                }
-                out.mergeProviderFields(&tmp)
-        }
-
-        // openrouter
-        if (fc.OpenRouter != sectionOpenRouter{}) || fc.OpenRouter.Temperature != nil {
-                tmp := App{
-                        OpenRouterBaseURL:     fc.OpenRouter.BaseURL,
-                        OpenRouterModel:       fc.OpenRouter.Model,
-                        OpenRouterTemperature: fc.OpenRouter.Temperature,
-                }
-                out.mergeProviderFields(&tmp)
-        }
-
-        // copilot
-        if (fc.Copilot != sectionCopilot{}) || fc.Copilot.Temperature != nil {
-                tmp := App{
-                        CopilotBaseURL:     fc.Copilot.BaseURL,
-                        CopilotModel:       fc.Copilot.Model,
-                        CopilotTemperature: fc.Copilot.Temperature,
-                }
-                out.mergeProviderFields(&tmp)
-        }
-
-        // ollama
-        if (fc.Ollama != sectionOllama{}) || fc.Ollama.Temperature != nil {
-                tmp := App{
-                        OllamaBaseURL:     fc.Ollama.BaseURL,
-                        OllamaModel:       fc.Ollama.Model,
-                        OllamaTemperature: fc.Ollama.Temperature,
-                }
-                out.mergeProviderFields(&tmp)
-        }
-
-        // prompts
-        // completion
-        if (fc.Prompts.Completion != sectionPromptsCompletion{}) {
-                if strings.TrimSpace(fc.Prompts.Completion.SystemGeneral) != "" {
-                        out.PromptCompletionSystemGeneral = fc.Prompts.Completion.SystemGeneral
-                }
-                if strings.TrimSpace(fc.Prompts.Completion.SystemParams) != "" {
-                        out.PromptCompletionSystemParams = fc.Prompts.Completion.SystemParams
-                }
-                if strings.TrimSpace(fc.Prompts.Completion.SystemInline) != "" {
-                        out.PromptCompletionSystemInline = fc.Prompts.Completion.SystemInline
-                }
-                if strings.TrimSpace(fc.Prompts.Completion.UserGeneral) != "" {
-                        out.PromptCompletionUserGeneral = fc.Prompts.Completion.UserGeneral
-                }
-                if strings.TrimSpace(fc.Prompts.Completion.UserParams) != "" {
-                        out.PromptCompletionUserParams = fc.Prompts.Completion.UserParams
-                }
-                if strings.TrimSpace(fc.Prompts.Completion.ExtraHeader) != "" {
-                        out.PromptCompletionExtraHeader = fc.Prompts.Completion.ExtraHeader
-                }
-        }
-        // chat
-        if strings.TrimSpace(fc.Prompts.Chat.System) != "" {
-                out.PromptChatSystem = fc.Prompts.Chat.System
-        }
-        // code action
-        if strings.TrimSpace(fc.Prompts.CodeAction.RewriteSystem) != "" ||
-                strings.TrimSpace(fc.Prompts.CodeAction.DiagnosticsSystem) != "" ||
-                strings.TrimSpace(fc.Prompts.CodeAction.DocumentSystem) != "" ||
-                strings.TrimSpace(fc.Prompts.CodeAction.RewriteUser) != "" ||
-                strings.TrimSpace(fc.Prompts.CodeAction.DiagnosticsUser) != "" ||
-                strings.TrimSpace(fc.Prompts.CodeAction.DocumentUser) != "" ||
-                strings.TrimSpace(fc.Prompts.CodeAction.GoTestSystem) != "" ||
-                strings.TrimSpace(fc.Prompts.CodeAction.GoTestUser) != "" ||
-                strings.TrimSpace(fc.Prompts.CodeAction.SimplifySystem) != "" ||
-                strings.TrimSpace(fc.Prompts.CodeAction.SimplifyUser) != "" ||
-                len(fc.Prompts.CodeAction.Custom) > 0 {
-                if strings.TrimSpace(fc.Prompts.CodeAction.RewriteSystem) != "" {
-                        out.PromptCodeActionRewriteSystem = fc.Prompts.CodeAction.RewriteSystem
-                }
-                if strings.TrimSpace(fc.Prompts.CodeAction.DiagnosticsSystem) != "" {
-                        out.PromptCodeActionDiagnosticsSystem = fc.Prompts.CodeAction.DiagnosticsSystem
-                }
-                if strings.TrimSpace(fc.Prompts.CodeAction.DocumentSystem) != "" {
-                        out.PromptCodeActionDocumentSystem = fc.Prompts.CodeAction.DocumentSystem
-                }
-                if strings.TrimSpace(fc.Prompts.CodeAction.RewriteUser) != "" {
-                        out.PromptCodeActionRewriteUser = fc.Prompts.CodeAction.RewriteUser
-                }
-                if strings.TrimSpace(fc.Prompts.CodeAction.DiagnosticsUser) != "" {
-                        out.PromptCodeActionDiagnosticsUser = fc.Prompts.CodeAction.DiagnosticsUser
-                }
-                if strings.TrimSpace(fc.Prompts.CodeAction.DocumentUser) != "" {
-                        out.PromptCodeActionDocumentUser = fc.Prompts.CodeAction.DocumentUser
-                }
-                if strings.TrimSpace(fc.Prompts.CodeAction.GoTestSystem) != "" {
-                        out.PromptCodeActionGoTestSystem = fc.Prompts.CodeAction.GoTestSystem
-                }
-                if strings.TrimSpace(fc.Prompts.CodeAction.GoTestUser) != "" {
-                        out.PromptCodeActionGoTestUser = fc.Prompts.CodeAction.GoTestUser
-                }
-                if strings.TrimSpace(fc.Prompts.CodeAction.SimplifySystem) != "" {
-                        out.PromptCodeActionSimplifySystem = fc.Prompts.CodeAction.SimplifySystem
-                }
-                if strings.TrimSpace(fc.Prompts.CodeAction.SimplifyUser) != "" {
-                        out.PromptCodeActionSimplifyUser = fc.Prompts.CodeAction.SimplifyUser
-                }
-                if len(fc.Prompts.CodeAction.Custom) > 0 {
-                        for _, ca := range fc.Prompts.CodeAction.Custom {
-                                out.CustomActions = append(out.CustomActions, CustomAction{
-                                        ID:          strings.TrimSpace(ca.ID),
-                                        Title:       strings.TrimSpace(ca.Title),
-                                        Kind:        strings.TrimSpace(ca.Kind),
-                                        Scope:       strings.ToLower(strings.TrimSpace(ca.Scope)),
-                                        Hotkey:      strings.TrimSpace(ca.Hotkey),
-                                        Instruction: ca.Instruction,
-                                        System:      ca.System,
-                                        User:        ca.User,
-                                })
-                        }
-                }
-        }
-        // cli
-        if (fc.Prompts.CLI != sectionPromptsCLI{}) {
-                if strings.TrimSpace(fc.Prompts.CLI.DefaultSystem) != "" {
-                        out.PromptCLIDefaultSystem = fc.Prompts.CLI.DefaultSystem
-                }
-                if strings.TrimSpace(fc.Prompts.CLI.ExplainSystem) != "" {
-                        out.PromptCLIExplainSystem = fc.Prompts.CLI.ExplainSystem
-                }
-        }
-        // provider-native
-        if strings.TrimSpace(fc.Prompts.ProviderNative.Completion) != "" {
-                out.PromptNativeCompletion = fc.Prompts.ProviderNative.Completion
-        }
-
-        // tmux
-        if (fc.Tmux != sectionTmux{}) {
-                out.TmuxCustomMenuHotkey = strings.TrimSpace(fc.Tmux.CustomMenuHotkey)
-        }
-
-        // stats
-        if fc.Stats.WindowMinutes > 0 {
-                out.StatsWindowMinutes = fc.Stats.WindowMinutes
-        }
-
-        return out
-}
-
-func loadFromFile(path string, logger *log.Logger) (*App, error) {
-        b, err := os.ReadFile(path)
-        if err != nil {
-                if !os.IsNotExist(err) && logger != nil {
-                        logger.Printf("cannot open TOML config file %s: %v", path, err)
-                }
-                return nil, err
-        }
-
-        var tables fileConfig
-        errTables := toml.NewDecoder(strings.NewReader(string(b))).Decode(&tables)
-        // Raw map for validation/presence checks
-        var raw map[string]any
-        _ = toml.Unmarshal(b, &raw)
-        if errTables != nil {
-                if logger != nil {
-                        logger.Printf("invalid TOML config file %s: %v", path, errTables)
-                }
-                return nil, errTables
-        }
-
-        // Reject legacy flat keys at top-level (sectioned-only config is allowed)
-        legacy := map[string]struct{}{
-                "max_tokens": {}, "context_mode": {}, "context_window_lines": {}, "max_context_tokens": {},
-                "log_preview_limit": {}, "completion_debounce_ms": {}, "completion_throttle_ms": {},
-                "manual_invoke_min_prefix": {}, "trigger_characters": {}, "inline_open": {}, "inline_close": {},
-                "chat_suffix": {}, "chat_prefixes": {}, "coding_temperature": {}, "provider": {},
-                "openai_model": {}, "openai_base_url": {}, "openai_temperature": {},
-                "ollama_model": {}, "ollama_base_url": {}, "ollama_temperature": {},
-                "copilot_model": {}, "copilot_base_url": {}, "copilot_temperature": {},
-        }
-        for k := range raw {
-                if _, isTable := map[string]struct{}{"general": {}, "logging": {}, "completion": {}, "triggers": {}, "inline": {}, "chat": {}, "provider": {}, "models": {}, "openai": {}, "copilot": {}, "ollama": {}, "prompts": {}}[k]; isTable {
-                        continue
-                }
-                if _, isLegacy := legacy[k]; isLegacy {
-                        return nil, fmt.Errorf("unsupported flat key '%s' in config; use sectioned tables (see config.toml.example)", k)
-                }
-        }
-
-        if logger != nil {
-                logger.Printf("loaded configuration from %s (TOML)", path)
-        }
-
-        // Merge order: flat first, then tables (so tables win over zero flat values)
-        // Build App from tables only
-        tab := tables.toApp()
-        // Ensure explicit values from raw map are respected (defensive for ints)
-        if t, ok := raw["completion"].(map[string]any); ok {
-                if v, present := t["manual_invoke_min_prefix"]; present {
-                        switch vv := v.(type) {
-                        case int64:
-                                tab.ManualInvokeMinPrefix = int(vv)
-                        case int:
-                                tab.ManualInvokeMinPrefix = vv
-                        case float64:
-                                tab.ManualInvokeMinPrefix = int(vv)
-                        }
-                }
-        }
-        if t, ok := raw["logging"].(map[string]any); ok {
-                if v, present := t["log_preview_limit"]; present {
-                        switch vv := v.(type) {
-                        case int64:
-                                tab.LogPreviewLimit = int(vv)
-                        case int:
-                                tab.LogPreviewLimit = vv
-                        case float64:
-                                tab.LogPreviewLimit = int(vv)
-                        }
-                }
-        }
-        if m := parseSurfaceModels(raw, logger); m != nil {
-                tab.mergeSurfaceModels(m)
-        }
-        return &tab, nil
-}
-
-func parseSurfaceModels(raw map[string]any, logger *log.Logger) *App {
-        modelsRaw, ok := raw["models"]
-        if !ok {
-                return nil
-        }
-        table, ok := modelsRaw.(map[string]any)
-        if !ok {
-                if logger != nil {
-                        logger.Printf("config: ignoring models section (expected table, got %T)", modelsRaw)
-                }
-                return nil
-        }
-        var out App
-        appendEntries := func(dest *[]SurfaceConfig, key string, val any) bool {
-                entries, ok := parseSurfaceEntries(val, key, logger)
-                if !ok || len(entries) == 0 {
-                        return false
-                }
-                *dest = append(*dest, entries...)
-                return true
-        }
-        any := appendEntries(&out.CompletionConfigs, "models.completion", table["completion"])
-        if ok := appendEntries(&out.CodeActionConfigs, "models.code_action", table["code_action"]); ok {
-                if len(out.CodeActionConfigs) > 1 {
-                        if logger != nil {
-                                logger.Printf("config: models.code_action supports a single entry; ignoring %d extra", len(out.CodeActionConfigs)-1)
-                        }
-                        out.CodeActionConfigs = out.CodeActionConfigs[:1]
-                }
-                any = true
-        }
-        any = appendEntries(&out.ChatConfigs, "models.chat", table["chat"]) || any
-        any = appendEntries(&out.CLIConfigs, "models.cli", table["cli"]) || any
-        if !any {
-                return nil
-        }
-        return &out
-}
-
-func parseSurfaceEntries(raw any, path string, logger *log.Logger) ([]SurfaceConfig, bool) {
-        switch v := raw.(type) {
-        case nil:
-                return nil, false
-        case []any:
-                var out []SurfaceConfig
-                for i, entry := range v {
-                        cfg, ok := decodeModelEntry(entry, fmt.Sprintf("%s[%d]", path, i), logger)
-                        if !ok || cfg == nil {
-                                continue
-                        }
-                        out = append(out, *cfg)
-                }
-                return out, len(out) > 0
-        default:
-                if cfg, ok := decodeModelEntry(v, path, logger); ok && cfg != nil {
-                        return []SurfaceConfig{*cfg}, true
-                }
-                return nil, false
-        }
-}
-
-func cloneSurfaceConfigs(src []SurfaceConfig) []SurfaceConfig {
-        if len(src) == 0 {
-                return nil
-        }
-        out := make([]SurfaceConfig, len(src))
-        copy(out, src)
-        return out
-}
-
-func decodeModelEntry(raw any, path string, logger *log.Logger) (*SurfaceConfig, bool) {
-        if raw == nil {
-                return nil, false
-        }
-        switch v := raw.(type) {
-        case string:
-                model := strings.TrimSpace(v)
-                if model == "" {
-                        return nil, false
-                }
-                return &SurfaceConfig{Model: model}, true
-        case map[string]any:
-                model := ""
-                provider := ""
-                if m, ok := v["model"]; ok {
-                        s, ok := m.(string)
-                        if !ok {
-                                if logger != nil {
-                                        logger.Printf("config: %s.model must be a string", path)
-                                }
-                                return nil, false
-                        }
-                        model = strings.TrimSpace(s)
-                }
-                if pRaw, ok := v["provider"]; ok {
-                        ps, ok := pRaw.(string)
-                        if !ok {
-                                if logger != nil {
-                                        logger.Printf("config: %s.provider must be a string", path)
-                                }
-                                return nil, false
-                        }
-                        provider = strings.TrimSpace(ps)
-                }
-                var tempPtr *float64
-                if tRaw, ok := v["temperature"]; ok {
-                        parsed, ok := parseTemperatureValue(tRaw, path, logger)
-                        if !ok {
-                                return nil, false
-                        }
-                        tempPtr = p