Update config command with show and edit subcommands

Author: manyfacedqod

cmd/config.go

@@ -4,94 +4,96 @@ import (
 	"fmt"
 	"os"
 	"os/exec"
+	"strings"
 
 	"smartcommit/config"
 
 	"github.com/spf13/cobra"
 )
 
-var ConfigCmd = &cobra.Command{
+var configCmd = &cobra.Command{
 	Use:   "config",
-	Short: "View or change smartcommit configuration",
-	Long: `Manage smartcommit configuration
+	Short: "View or manage SmartCommit configuration",
+}
 
-Examples:
-  smartcommit config edit       # Edit the system prompt (tone/style)
-  smartcommit config show       # View current configuration`,
+var showCmd = &cobra.Command{
+	Use:   "show",
+	Short: "Show current configuration",
+	Run: func(cmd *cobra.Command, args []string) {
+		cfg := config.LoadOrDefault()
+		fmt.Println("📁 Current Config:")
+		fmt.Println("  Provider:     ", cfg.Provider)
+		fmt.Println("  Model:        ", cfg.Model)
+		fmt.Println("  Base URL:     ", cfg.BaseURL)
+		if cfg.APIKey != "" {
+			fmt.Println("  API Key:      ", mask(cfg.APIKey))
+		}
+		fmt.Println("  SystemPrompt: ", cfg.SystemPrompt)
+	},
 }
 
-var EditConfigCmd = &cobra.Command{
+var editCmd = &cobra.Command{
 	Use:   "edit",
-	Short: "Edit system prompt in your preferred editor (e.g., vim, nano, code)",
+	Short: "Edit the system prompt used for generation (in Vim)",
 	Run: func(cmd *cobra.Command, args []string) {
 		cfg := config.LoadOrDefault()
 
 		tmpFile, err := os.CreateTemp("", "smartcommit-prompt-*.txt")
 		if err != nil {
-			fmt.Println("❌ Failed to create temp file:", err)
+			fmt.Println("❌ Could not create temp file:", err)
 			return
 		}
 		defer os.Remove(tmpFile.Name())
 
-		// Write current prompt to temp file
+		// Write current prompt
 		_, _ = tmpFile.WriteString(cfg.SystemPrompt)
 		tmpFile.Close()
 
-		// Open in default editor
 		editor := os.Getenv("EDITOR")
 		if editor == "" {
 			editor = "vim"
 		}
-		cmdEdit := exec.Command(editor, tmpFile.Name())
-		cmdEdit.Stdin = os.Stdin
-		cmdEdit.Stdout = os.Stdout
-		cmdEdit.Stderr = os.Stderr
 
-		if err := cmdEdit.Run(); err != nil {
-			fmt.Println("❌ Failed to open editor:", err)
+		editCmd := exec.Command(editor, tmpFile.Name())
+		editCmd.Stdin = os.Stdin
+		editCmd.Stdout = os.Stdout
+		editCmd.Stderr = os.Stderr
+		if err := editCmd.Run(); err != nil {
+			fmt.Println("❌ Editor closed with error:", err)
 			return
 		}
 
-		// Read edited prompt
-		editedBytes, err := os.ReadFile(tmpFile.Name())
+		// Read new prompt
+		newPromptBytes, err := os.ReadFile(tmpFile.Name())
 		if err != nil {
-			fmt.Println("❌ Failed to read edited prompt:", err)
+			fmt.Println("❌ Could not read edited file:", err)
 			return
 		}
-
-		edited := string(editedBytes)
-		if edited == "" {
-			fmt.Println("⚠️ Prompt left empty. Aborting.")
+		newPrompt := strings.TrimSpace(string(newPromptBytes))
+		if newPrompt == "" {
+			fmt.Println("⚠️  Prompt is empty — edit cancelled.")
 			return
 		}
 
-		cfg.SystemPrompt = edited
+		cfg.SystemPrompt = newPrompt
 		if err := config.Save(cfg); err != nil {
 			fmt.Println("❌ Failed to save config:", err)
-		} else {
-			fmt.Println("✅ System prompt updated successfully.")
+			return
 		}
+		fmt.Println("✅ System prompt updated.")
 	},
 }
 
 
-var ShowConfigCmd = &cobra.Command{
-	Use:   "show",
-	Short: "Show current config",
-	Run: func(cmd *cobra.Command, args []string) {
-		cfg := config.LoadOrDefault()
-		fmt.Printf("\nSystem Prompt: %s\n", cfg.SystemPrompt)
-		fmt.Printf("Provider: %s\n", cfg.Provider)
-		fmt.Printf("Model: %s\n", cfg.Model)
-		if cfg.APIKey != "" {
-			fmt.Println("API Key: [set]")
-		} else {
-			fmt.Println("API Key: [not set]")
-		}
-	},
+func init() {
+	configCmd.AddCommand(showCmd)
+	configCmd.AddCommand(editCmd)
+	rootCmd.AddCommand(configCmd)
 }
 
-func init() {
-	ConfigCmd.AddCommand(EditConfigCmd)
-	ConfigCmd.AddCommand(ShowConfigCmd)
-} 
+func mask(s string) string {
+	if len(s) <= 6 {
+		return "******"
+	}
+	return s[:3] + "..." + s[len(s)-3:]
+}

config/config.go

@@ -56,3 +56,7 @@ func Save(cfg *Config) error {
     }
     return os.WriteFile(path, out, 0644)
 }
+func ConfigPath() string {
+	dir, _ := os.UserConfigDir()
+	return filepath.Join(dir, "smartcommit", "config.yaml")
+}