chore: review and clean up CLI code

internal/cmd/root.go

@@ -1,3 +1,6 @@
+// Package cmd implements the Headjack CLI commands using Cobra.
+// It provides commands for managing isolated LLM agent instances,
+// including create, attach, stop, remove, and session management.
 package cmd
 
 import (

internal/config/config.go

@@ -100,36 +100,6 @@ func (c *Config) Validate() error {
 	return nil
 }
 
-// IsValidAgent returns true if the agent name is valid.
-func (c *Config) IsValidAgent(name string) bool {
-	return validAgents[name]
-}
-
-// ValidAgentNames returns the list of valid agent names.
-func (c *Config) ValidAgentNames() []string {
-	return []string{"claude", "gemini", "codex"}
-}
-
-// IsValidMultiplexer returns true if the multiplexer name is valid.
-func (c *Config) IsValidMultiplexer(name string) bool {
-	return validMultiplexers[name]
-}
-
-// ValidMultiplexerNames returns the list of valid multiplexer names.
-func (c *Config) ValidMultiplexerNames() []string {
-	return []string{"tmux", "zellij"}
-}
-
-// IsValidRuntime returns true if the runtime name is valid.
-func (c *Config) IsValidRuntime(name string) bool {
-	return validRuntimes[name]
-}
-
-// ValidRuntimeNames returns the list of valid runtime names.
-func (c *Config) ValidRuntimeNames() []string {
-	return []string{"podman", "apple"}
-}
-
 // Loader provides configuration loading and saving.
 type Loader struct {
 	v       *viper.Viper

internal/config/config_test.go

@@ -210,19 +210,16 @@ func TestConfig_Validate(t *testing.T) {
 	})
 }
 
-func TestConfig_IsValidAgent(t *testing.T) {
-	cfg := &Config{}
-
-	assert.True(t, cfg.IsValidAgent("claude"))
-	assert.True(t, cfg.IsValidAgent("gemini"))
-	assert.True(t, cfg.IsValidAgent("codex"))
-	assert.False(t, cfg.IsValidAgent("invalid"))
-	assert.False(t, cfg.IsValidAgent(""))
+func TestIsValidAgent(t *testing.T) {
+	assert.True(t, IsValidAgent("claude"))
+	assert.True(t, IsValidAgent("gemini"))
+	assert.True(t, IsValidAgent("codex"))
+	assert.False(t, IsValidAgent("invalid"))
+	assert.False(t, IsValidAgent(""))
 }
 
-func TestConfig_ValidAgentNames(t *testing.T) {
-	cfg := &Config{}
-	names := cfg.ValidAgentNames()
+func TestValidAgentNames(t *testing.T) {
+	names := ValidAgentNames()
 
 	assert.Contains(t, names, "claude")
 	assert.Contains(t, names, "gemini")