From bc80c7cd6799529e173fd4e10a84c2bfb5abed45 Mon Sep 17 00:00:00 2001
From: Bo-Yi Wu <appleboy.tw@gmail.com>
Date: Thu, 13 Feb 2025 09:07:34 +0800
Subject: [PATCH] refactor: refactor commit prompts and update default OpenAI
 model

- Refactor commit preview confirmation prompt logic
- Refactor commit message change prompt logic
- Update default OpenAI model to `gpt-4o` in `config_set.go`
- Update default OpenAI model to `gpt-4o` in `review.go`

Signed-off-by: Bo-Yi Wu <appleboy.tw@gmail.com>
---
 cmd/commit.go     | 20 +++++++-------------
 cmd/config_set.go |  2 +-
 cmd/review.go     |  2 +-
 3 files changed, 9 insertions(+), 15 deletions(-)

diff --git a/cmd/commit.go b/cmd/commit.go
index 6597d0d..a449d6a 100644
--- a/cmd/commit.go
+++ b/cmd/commit.go
@@ -291,36 +291,30 @@ var commitCmd = &cobra.Command{
 			return err
 		}
 
+		// Handle preview: if preview and noConfirm, or preview prompt declined, then exit early
 		if preview {
 			if noConfirm {
 				return nil
-			} else {
-				input := confirmation.New("Commit preview summary?", confirmation.Yes)
-				ready, err := input.RunPrompt()
+			}
+			if ready, err := confirmation.New("Commit preview summary?", confirmation.Yes).RunPrompt(); err != nil || !ready {
 				if err != nil {
 					return err
 				}
-				if !ready {
-					return nil
-				}
+				return nil
 			}
 		}
 
+		// Handle commit message change prompt when confirmation is enabled
 		if !noConfirm {
-			input := confirmation.New("Do you want to change the commit message?", confirmation.No)
-			change, err := input.RunPrompt()
-			if err != nil {
+			if change, err := confirmation.New("Do you want to change the commit message?", confirmation.No).RunPrompt(); err != nil {
 				return err
-			}
-
-			if change {
+			} else if change {
 				m := initialPrompt(commitMessage)
 				p := tea.NewProgram(m, tea.WithContext(cmd.Context()))
 				if _, err := p.Run(); err != nil {
 					return err
 				}
 				p.Wait()
-
 				commitMessage = m.textarea.Value()
 			}
 		}
diff --git a/cmd/config_set.go b/cmd/config_set.go
index 764965f..073506d 100644
--- a/cmd/config_set.go
+++ b/cmd/config_set.go
@@ -13,7 +13,7 @@ func init() {
 	configCmd.AddCommand(configSetCmd)
 	configSetCmd.Flags().StringP("base_url", "b", "", availableKeys["openai.base_url"])
 	configSetCmd.Flags().StringP("api_key", "k", "", availableKeys["openai.api_key"])
-	configSetCmd.Flags().StringP("model", "m", "gpt-3.5-turbo", availableKeys["openai.model"])
+	configSetCmd.Flags().StringP("model", "m", "gpt-4o", availableKeys["openai.model"])
 	configSetCmd.Flags().StringP("lang", "l", "en", availableKeys["openai.lang"])
 	configSetCmd.Flags().StringP("org_id", "o", "", availableKeys["openai.org_id"])
 	configSetCmd.Flags().StringP("proxy", "", "", availableKeys["openai.proxy"])
diff --git a/cmd/review.go b/cmd/review.go
index 34ddb63..c3aa01b 100644
--- a/cmd/review.go
+++ b/cmd/review.go
@@ -23,7 +23,7 @@ func init() {
 		"generate diffs with <n> lines of context, default is 3")
 	reviewCmd.PersistentFlags().IntVar(&maxTokens, "max_tokens", 300,
 		"the maximum number of tokens to generate in the chat completion.")
-	reviewCmd.PersistentFlags().StringVar(&commitModel, "model", "gpt-3.5-turbo", "select openai model")
+	reviewCmd.PersistentFlags().StringVar(&commitModel, "model", "gpt-4o", "select openai model")
 	reviewCmd.PersistentFlags().StringVar(&commitLang, "lang", "en", "summarizing language uses English by default")
 	reviewCmd.PersistentFlags().StringSliceVar(&excludeList, "exclude_list", []string{}, "exclude file from git diff command")
 	reviewCmd.PersistentFlags().BoolVar(&commitAmend, "amend", false,