fixed some issues

Author: GauravJangra9988

cmd/cli/llmSetup.go

@@ -1,7 +1,7 @@
 package cmd
 
 import (
-	
+	"errors"
 	"fmt"
 
 	"github.com/dfanso/commit-msg/cmd/cli/store"
@@ -24,12 +24,13 @@ func SetupLLM() error {
 
 	apiKeyPrompt := promptui.Prompt{
 		Label: "Enter API Key",
+		Mask: '*',
 
 	}
 
 	apiKey, err := apiKeyPrompt.Run()
 	if err != nil {
-		return fmt.Errorf("invalid API Key")
+		return fmt.Errorf("failed to read API Key: %w", err)
 	}
 
 	LLMConfig := store.LLMProvider{
@@ -55,9 +56,9 @@ func UpdateLLM() error {
 		return err
 	}
 
-	if len(SavedModels.LLMProviders) == 0{
-		 fmt.Println("No model exists, Please add atleast one model")
-		 return nil
+	if len(SavedModels.LLMProviders) == 0 {
+		 return errors.New("no model exists, Please add atleast one model Run: 'commit llm setup'")
+		 
 	}
 
 	models := []string{}

cmd/cli/store/store.go

@@ -44,8 +44,7 @@ func Save(LLMConfig LLMProvider) error {
 	data, err := os.ReadFile(configPath)
 	if errors.Is(err, os.ErrNotExist){
 		data = []byte("{}")
-	} else if
-	 err != nil {
+	} else if err != nil {
 		return err
 	}
 
@@ -166,16 +165,18 @@ func DefaultLLMKey() (*LLMProvider, error) {
 		return nil, err
 	}
 
-
-	if len(data) > 0 {
+	fmt.Println(len(data))
+	if len(data) > 2 {
 		err = json.Unmarshal(data, &cfg)
 		if err != nil {
 			return nil, err
 		}
 	} else {
-		return nil, errors.New("config file is empty")
+		return nil, errors.New("config file is empty, Please add atlead one LLM Key")
 	}
 
+	
+
 	defaultLLM := cfg.Default
 
 	for i, p := range cfg.LLMProviders {
@@ -289,8 +290,7 @@ func DeleteModel(Model string) error {
 
 	if Model == cfg.Default {
 		if len(cfg.LLMProviders) > 1 {
-			fmt.Println("Please set other model as default, Cant delete default model")
-			return nil
+			return fmt.Errorf("cannot delete %s while it is default, set other model default first", Model)
 		} else {
 			return os.WriteFile(configPath, []byte("{}"), 0600)
 		}