summaryrefslogtreecommitdiff
path: root/internal/config
diff options
context:
space:
mode:
authorClaudomator Agent <agent@claudomator>2026-03-10 04:36:19 +0000
committerClaudomator Agent <agent@claudomator>2026-03-10 04:36:19 +0000
commit8873187921c55d94be56364bf0b9d6b2d12127c2 (patch)
treecc277db491104ebec5645e1a0422be9d5d4cafd2 /internal/config
parent7d6943c5f9f4124c652377148a35bea5f61be4bf (diff)
cli: implement --config flag to load TOML config file
The --config flag was registered but silently ignored. Now: - config.LoadFile loads a TOML file on top of defaults - PersistentPreRunE applies the file when --config is set - Explicit CLI flags (--data-dir, --claude-bin) take precedence over the file Tests: TestLoadFile_OverridesDefaults, TestLoadFile_MissingFile_ReturnsError, TestRootCmd_ConfigFile_Loaded, TestRootCmd_ConfigFile_CLIFlagOverrides, TestRootCmd_ConfigFile_Missing_ReturnsError Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Diffstat (limited to 'internal/config')
-rw-r--r--internal/config/config.go15
-rw-r--r--internal/config/config_test.go31
2 files changed, 46 insertions, 0 deletions
diff --git a/internal/config/config.go b/internal/config/config.go
index daf42fe..8c5aebf 100644
--- a/internal/config/config.go
+++ b/internal/config/config.go
@@ -5,6 +5,8 @@ import (
"fmt"
"os"
"path/filepath"
+
+ "github.com/BurntSushi/toml"
)
type Config struct {
@@ -42,6 +44,19 @@ func Default() (*Config, error) {
}, nil
}
+// LoadFile loads a TOML config file on top of the defaults.
+// Fields not present in the file retain their default values.
+func LoadFile(path string) (*Config, error) {
+ cfg, err := Default()
+ if err != nil {
+ return nil, err
+ }
+ if _, err := toml.DecodeFile(path, cfg); err != nil {
+ return nil, fmt.Errorf("loading config file %q: %w", path, err)
+ }
+ return cfg, nil
+}
+
// EnsureDirs creates the data directory structure.
func (c *Config) EnsureDirs() error {
for _, dir := range []string{c.DataDir, c.LogDir} {
diff --git a/internal/config/config_test.go b/internal/config/config_test.go
index 766b856..2bba2c4 100644
--- a/internal/config/config_test.go
+++ b/internal/config/config_test.go
@@ -1,6 +1,8 @@
package config
import (
+ "os"
+ "path/filepath"
"testing"
)
@@ -22,3 +24,32 @@ func TestDefault_ValidHome_ReturnsConfig(t *testing.T) {
t.Errorf("DataDir = %q, want /tmp/testhome/.claudomator", cfg.DataDir)
}
}
+
+func TestLoadFile_OverridesDefaults(t *testing.T) {
+ dir := t.TempDir()
+ path := filepath.Join(dir, "config.toml")
+ if err := os.WriteFile(path, []byte("server_addr = \":9191\"\n"), 0600); err != nil {
+ t.Fatal(err)
+ }
+ t.Setenv("HOME", dir)
+
+ cfg, err := LoadFile(path)
+ if err != nil {
+ t.Fatalf("unexpected error: %v", err)
+ }
+ if cfg.ServerAddr != ":9191" {
+ t.Errorf("ServerAddr = %q, want :9191", cfg.ServerAddr)
+ }
+ // Unset fields retain defaults.
+ if cfg.MaxConcurrent != 3 {
+ t.Errorf("MaxConcurrent = %d, want 3", cfg.MaxConcurrent)
+ }
+}
+
+func TestLoadFile_MissingFile_ReturnsError(t *testing.T) {
+ t.Setenv("HOME", t.TempDir())
+ _, err := LoadFile("/nonexistent/config.toml")
+ if err == nil {
+ t.Fatal("expected error for missing file, got nil")
+ }
+}