conf: Add tilde (~) expansion support for config_path

Users can now use portable tilde notation in config files:
  config_path = '~/.config/jj/config.toml'

Instead of hardcoded absolute paths:
  config_path = '/Users/username/.config/jj/config.toml'

Implementation:
- Added ExpandPath() function to expand tilde to home directory
- GetTool() now automatically expands tilde in config_path
- DefaultConfig() uses tilde notation for portability
- Load() expands tilde when loading config from file

Testing:
- Added comprehensive tests for ExpandPath()
- Added TestLoadExpandsTildePaths integration test
- All package tests pass (config, editors, schemas, tools)

Benefits:
- Config files are portable across users/systems
- No need to update paths when home directory changes
- Cleaner, more readable configuration files

Closes: issues-36

🤖 Generated with [Claude Code](https://claude.com/claude-code)

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

Author: neongreen

.beads/issues.jsonl

@@ -433,6 +433,8 @@
 {"id":"issues-32","title":"Add comprehensive unit tests for declarative state management","description":"The new declarative state management system lacks unit tests for critical functionality.\n\nMissing test coverage:\n1. Config.SetToolValue/GetToolValue/UnsetToolValue methods\n2. Config.SetShim/GetShim/UnsetShim methods  \n3. applyTool() function with various scenarios\n4. showToolStatus() drift detection logic\n5. Integration between state recording and target file application\n\nTest scenarios needed:\n- State recording and retrieval\n- Drift detection (in sync, drift, missing values)\n- Apply operations (dry-run vs actual)\n- Error handling for invalid tools/paths\n- Edge cases (empty state, missing files)\n\nPriority 1 since this is core functionality without test coverage.","status":"closed","priority":1,"issue_type":"task","created_at":"2025-10-21T23:20:16.142583+02:00","updated_at":"2025-10-21T23:22:53.660666+02:00","closed_at":"2025-10-21T23:22:53.660666+02:00"}
 {"id":"issues-33","title":"Add unit tests for tools/jj, tools/shims, tools/starship packages","description":"Three tool packages have no unit test coverage.\n\nMissing tests for:\n1. pkg/tools/jj/ - JJ tool operations, schema validation, config handling\n2. pkg/tools/shims/ - Shim creation, listing, removal, validation \n3. pkg/tools/starship/ - Starship config operations, path validation\n\nEach package needs tests for:\n- Tool initialization (NewTool functions)\n- SetConfig/GetConfig operations\n- Path validation logic\n- Error handling\n- Dry-run modes\n- Config file handling\n\nPriority 2 since basic functionality works but lacks test safety net.","status":"closed","priority":2,"issue_type":"task","created_at":"2025-10-21T23:20:16.195189+02:00","updated_at":"2025-10-22T08:28:48.581102+02:00","closed_at":"2025-10-22T08:28:48.581102+02:00"}
 {"id":"issues-34","title":"Fix JJ schema compilation warning and refactor duplicated switch statements","description":"Technical debt cleanup for better maintainability.\n\n1. JJ Schema Issue:\nWarning: jj schema compilation had issues: duplicate enum values at indexes 10 and 11\nNeed to fix jj.json schema file to remove duplicate enum entries\n\n2. Code Duplication:\napplyToolValue() and getActualValue() have identical switch statements\nShould extract into a single tool factory/registry pattern\n\nRefactor to:\n- toolRegistry map[string]ToolFactory\n- Single function to get tool instance by name\n- Cleaner, more maintainable code\n\n3. Error Handling:\nSome edge cases not properly handled in new commands\nAdd proper validation and error messages\n\nPriority 2 for code quality and maintainability.","status":"closed","priority":2,"issue_type":"task","created_at":"2025-10-21T23:20:16.239878+02:00","updated_at":"2025-10-22T07:54:36.939519+02:00","closed_at":"2025-10-22T07:54:36.939519+02:00"}
+{"id":"issues-35","title":"conf: Make configuration management LLM-friendly","description":"Design and implement features to make conf a natural choice for LLM-driven configuration management:\n\nGoals:\n- Natural language to config operations\n- Clear, parseable status output for LLMs\n- Schema-based validation and suggestions\n- Atomic operations with clear success/failure\n- Readable diffs and previews\n\nAreas to explore:\n- JSON output modes for programmatic access\n- Structured error messages\n- Enhanced schema documentation\n- API/library mode for direct integration\n- Clear operation semantics\n\nWill break down into specific tasks after initial design review.","status":"open","priority":2,"issue_type":"epic","created_at":"2025-10-22T10:54:43.862155+02:00","updated_at":"2025-10-22T10:54:43.862155+02:00"}
+{"id":"issues-36","title":"conf: Support tilde (~) resolution in config_path","description":"Currently config_path in conf's config.toml requires full absolute paths like:\n  config_path = '/Users/artyom/.config/jj/config.toml'\n\nShould support tilde notation for better portability:\n  config_path = '~/.config/jj/config.toml'\n\nImplement tilde expansion when reading config_path values so configs are more portable across systems and users.","status":"open","priority":2,"issue_type":"task","created_at":"2025-10-22T13:29:40.593353+02:00","updated_at":"2025-10-22T13:29:40.593353+02:00"}
 {"id":"issues-5","title":"Create robust mise task for building preprocessor and example book","description":"Build system currently has no validation - preprocessor may or may not exist, book builds with stale JS files, no error checking.\n\nRequirements:\n- mise task that builds preprocessor if needed (check if Cargo project exists)\n- Verifies preprocessor is installed/available before building book\n- Builds TypeScript/JS assets\n- Builds example book with mdbook\n- Fails loudly if anything is missing or out of date\n- Should be the ONLY way to build for testing\n\nCurrent problems:\n- mdbook-comments preprocessor not found but book still builds\n- HTML has inlined JS from old builds\n- No way to ensure fresh, correct builds\n- Manual copying of files (bad!)\n\nThis task should replace all manual build steps and be used exclusively for testing.","status":"closed","priority":1,"issue_type":"task","created_at":"2025-10-21T10:49:03.911981+02:00","updated_at":"2025-10-21T10:56:08.81806+02:00","closed_at":"2025-10-21T10:56:08.81806+02:00"}
 {"id":"issues-6","title":"Clean up test task to kill servers and reset database","description":"The test:e2e task doesn't properly clean up before running:\n- Doesn't kill existing json-server or mdbook servers\n- Doesn't clean /tmp/test-db.json\n- Can lead to tests using stale data/code\n\nFix mise test:e2e task to:\n1. Kill any running json-server on port 54322\n2. Kill any running mdbook serve on port 3300  \n3. Remove /tmp/test-db.json\n4. Then run build:all\n5. Then run Playwright tests\n\nThis ensures fresh, clean state for every test run.","status":"closed","priority":1,"issue_type":"task","created_at":"2025-10-21T10:53:25.66892+02:00","updated_at":"2025-10-21T12:03:15.464857+02:00","closed_at":"2025-10-21T12:03:15.464857+02:00"}
 {"id":"issues-7","title":"Fix Preact reactivity for reply updates in Comment components","description":"The Comment component isn't re-rendering when replies are added because:\n\n1. buildReplyStructure() creates new comment objects, but handleUpdate doesn't properly pass updated references to Preact\n2. The key={`${comment.id}-replies-${comment.replies?.length || 0}`} strategy works in theory but the comment objects themselves aren't being updated in the component props\n3. Need to ensure that when handleUpdate re-renders the CommentSection, it gets the freshly built comment objects with updated replies arrays\n\nThis is causing test #6 'should reply to a comment' to fail - the reply saves to database but UI doesn't refresh.","status":"closed","priority":1,"issue_type":"bug","created_at":"2025-10-21T12:03:43.819407+02:00","updated_at":"2025-10-21T12:05:11.808144+02:00","closed_at":"2025-10-21T12:05:11.808144+02:00"}

conf/pkg/config/config.go

@@ -24,26 +24,21 @@ type ToolConfig struct {
 
 // DefaultConfig returns a new Config with default settings
 func DefaultConfig() *Config {
-	homeDir, err := os.UserHomeDir()
-	if err != nil {
-		homeDir = "~"
-	}
-
 	return &Config{
 		Tools: map[string]ToolConfig{
 			"jj": {
 				Name:       "jj",
-				ConfigPath: filepath.Join(homeDir, ".config/jj/config.toml"),
+				ConfigPath: "~/.config/jj/config.toml",
 				SchemaPath: "embedded://jj.json",
 			},
 			"mise": {
 				Name:       "mise",
-				ConfigPath: filepath.Join(homeDir, ".config", "mise", "config.toml"),
+				ConfigPath: "~/.config/mise/config.toml",
 				SchemaPath: "embedded://mise.toml",
 			},
 			"starship": {
 				Name:       "starship",
-				ConfigPath: filepath.Join(homeDir, ".config", "starship.toml"),
+				ConfigPath: "~/.config/starship.toml",
 			},
 		},
 	}
@@ -70,6 +65,31 @@ func ConfigPath() (string, error) {
 	return filepath.Join(configDir, "config.toml"), nil
 }
 
+// ExpandPath expands tilde (~) in paths to the user's home directory
+func ExpandPath(path string) (string, error) {
+	if path == "" {
+		return path, nil
+	}
+
+	// Only expand if path starts with ~
+	if path[0] != '~' {
+		return path, nil
+	}
+
+	homeDir, err := os.UserHomeDir()
+	if err != nil {
+		return "", fmt.Errorf("failed to get user home directory: %w", err)
+	}
+
+	// Handle ~/ or just ~
+	if len(path) == 1 || path[1] == '/' {
+		return filepath.Join(homeDir, path[1:]), nil
+	}
+
+	// Don't support ~user syntax for now
+	return path, nil
+}
+
 // Load loads the configuration from the config file, creating it if it doesn't exist
 func Load() (*Config, error) {
 	configPath, err := ConfigPath()
@@ -97,6 +117,14 @@ func Load() (*Config, error) {
 		return nil, fmt.Errorf("failed to parse config file: %w", err)
 	}
 
+	// Expand tilde in config paths
+	for name, tool := range config.Tools {
+		if expandedPath, err := ExpandPath(tool.ConfigPath); err == nil {
+			tool.ConfigPath = expandedPath
+			config.Tools[name] = tool
+		}
+	}
+
 	return &config, nil
 }
 
@@ -127,10 +155,19 @@ func (c *Config) Save() error {
 	return nil
 }
 
-// GetTool returns the configuration for a specific tool
+// GetTool returns the configuration for a specific tool with expanded paths
 func (c *Config) GetTool(name string) (ToolConfig, bool) {
 	tool, exists := c.Tools[name]
-	return tool, exists
+	if !exists {
+		return tool, false
+	}
+
+	// Expand tilde in config path
+	if expandedPath, err := ExpandPath(tool.ConfigPath); err == nil {
+		tool.ConfigPath = expandedPath
+	}
+
+	return tool, true
 }
 
 // SetTool sets the configuration for a specific tool

conf/pkg/config/config_test.go

@@ -13,8 +13,8 @@ func TestDefaultConfig(t *testing.T) {
 		t.Fatal("Default config should not be nil")
 	}
 
-	if len(config.Tools) != 2 {
-		t.Errorf("Default config should have 2 tools, got %d", len(config.Tools))
+	if len(config.Tools) != 3 {
+		t.Errorf("Default config should have 3 tools, got %d", len(config.Tools))
 	}
 
 	// Check jj tool config
@@ -25,9 +25,13 @@ func TestDefaultConfig(t *testing.T) {
 	if jjTool.Name != "jj" {
 		t.Errorf("jj tool name should be 'jj', got '%s'", jjTool.Name)
 	}
+	// GetTool() expands tilde, so check for expanded path
 	if !strings.Contains(jjTool.ConfigPath, ".config/jj/config.toml") {
 		t.Errorf("jj config path should contain '.config/jj/config.toml', got '%s'", jjTool.ConfigPath)
 	}
+	if strings.HasPrefix(jjTool.ConfigPath, "~") {
+		t.Errorf("jj config path should be expanded, got '%s'", jjTool.ConfigPath)
+	}
 
 	// Check mise tool config
 	miseTool, exists := config.GetTool("mise")
@@ -37,8 +41,92 @@ func TestDefaultConfig(t *testing.T) {
 	if miseTool.Name != "mise" {
 		t.Errorf("mise tool name should be 'mise', got '%s'", miseTool.Name)
 	}
-	if !strings.Contains(miseTool.ConfigPath, "mise/config.toml") {
-		t.Errorf("mise config path should contain 'mise/config.toml', got '%s'", miseTool.ConfigPath)
+	// GetTool() expands tilde, so check for expanded path
+	if !strings.Contains(miseTool.ConfigPath, ".config/mise/config.toml") {
+		t.Errorf("mise config path should contain '.config/mise/config.toml', got '%s'", miseTool.ConfigPath)
+	}
+	if strings.HasPrefix(miseTool.ConfigPath, "~") {
+		t.Errorf("mise config path should be expanded, got '%s'", miseTool.ConfigPath)
+	}
+
+	// Check starship tool config
+	starshipTool, exists := config.GetTool("starship")
+	if !exists {
+		t.Error("Default config should include starship tool")
+	}
+	if starshipTool.Name != "starship" {
+		t.Errorf("starship tool name should be 'starship', got '%s'", starshipTool.Name)
+	}
+	// GetTool() expands tilde, so check for expanded path
+	if !strings.Contains(starshipTool.ConfigPath, ".config/starship.toml") {
+		t.Errorf("starship config path should contain '.config/starship.toml', got '%s'", starshipTool.ConfigPath)
+	}
+	if strings.HasPrefix(starshipTool.ConfigPath, "~") {
+		t.Errorf("starship config path should be expanded, got '%s'", starshipTool.ConfigPath)
+	}
+}
+
+func TestExpandPath(t *testing.T) {
+	homeDir, err := os.UserHomeDir()
+	if err != nil {
+		t.Fatalf("Failed to get home dir: %v", err)
+	}
+
+	tests := []struct {
+		name     string
+		input    string
+		expected string
+		wantErr  bool
+	}{
+		{
+			name:     "expand tilde with slash",
+			input:    "~/.config/jj/config.toml",
+			expected: homeDir + "/.config/jj/config.toml",
+			wantErr:  false,
+		},
+		{
+			name:     "expand just tilde",
+			input:    "~",
+			expected: homeDir,
+			wantErr:  false,
+		},
+		{
+			name:     "no tilde - absolute path",
+			input:    "/etc/config.toml",
+			expected: "/etc/config.toml",
+			wantErr:  false,
+		},
+		{
+			name:     "no tilde - relative path",
+			input:    "./config.toml",
+			expected: "./config.toml",
+			wantErr:  false,
+		},
+		{
+			name:     "empty string",
+			input:    "",
+			expected: "",
+			wantErr:  false,
+		},
+		{
+			name:     "tilde in middle - no expansion",
+			input:    "/path/~/config.toml",
+			expected: "/path/~/config.toml",
+			wantErr:  false,
+		},
+	}
+
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			result, err := ExpandPath(tt.input)
+			if (err != nil) != tt.wantErr {
+				t.Errorf("ExpandPath() error = %v, wantErr %v", err, tt.wantErr)
+				return
+			}
+			if result != tt.expected {
+				t.Errorf("ExpandPath() = %v, want %v", result, tt.expected)
+			}
+		})
 	}
 }
 
@@ -135,3 +223,67 @@ func TestSaveAndLoad(t *testing.T) {
 		t.Errorf("Test tool name should be 'test', got '%s'", testTool.Name)
 	}
 }
+
+func TestLoadExpandsTildePaths(t *testing.T) {
+	// Create a temporary directory for testing
+	tempDir, err := os.MkdirTemp("", "conf-test")
+	if err != nil {
+		t.Fatalf("Failed to create temp dir: %v", err)
+	}
+	defer os.RemoveAll(tempDir)
+
+	// Override HOME for testing
+	originalHome := os.Getenv("HOME")
+	os.Setenv("HOME", tempDir)
+	defer os.Setenv("HOME", originalHome)
+
+	// Create config with tilde paths
+	config := &Config{
+		Tools: map[string]ToolConfig{
+			"jj": {
+				Name:       "jj",
+				ConfigPath: "~/.config/jj/config.toml",
+				SchemaPath: "embedded://jj.json",
+			},
+			"mise": {
+				Name:       "mise",
+				ConfigPath: "~/.config/mise/config.toml",
+				SchemaPath: "embedded://mise.toml",
+			},
+		},
+	}
+
+	// Save config
+	err = config.Save()
+	if err != nil {
+		t.Fatalf("Failed to save config: %v", err)
+	}
+
+	// Load config - should expand tilde paths
+	loadedConfig, err := Load()
+	if err != nil {
+		t.Fatalf("Failed to load config: %v", err)
+	}
+
+	// Verify jj path was expanded
+	jjTool, exists := loadedConfig.GetTool("jj")
+	if !exists {
+		t.Fatal("Loaded config should contain jj tool")
+	}
+
+	expectedJJPath := tempDir + "/.config/jj/config.toml"
+	if jjTool.ConfigPath != expectedJJPath {
+		t.Errorf("jj ConfigPath should be expanded to '%s', got '%s'", expectedJJPath, jjTool.ConfigPath)
+	}
+
+	// Verify mise path was expanded
+	miseTool, exists := loadedConfig.GetTool("mise")
+	if !exists {
+		t.Fatal("Loaded config should contain mise tool")
+	}
+
+	expectedMisePath := tempDir + "/.config/mise/config.toml"
+	if miseTool.ConfigPath != expectedMisePath {
+		t.Errorf("mise ConfigPath should be expanded to '%s', got '%s'", expectedMisePath, miseTool.ConfigPath)
+	}
+}