fix: 🐞 Bug: Model is deleted when renaming, then pressing Ctrl-C (#228)

* fix: prevent model deletion when cancelling rename with Ctrl-C or Esc

When a user pressed Ctrl-C during a rename operation, the model would
be deleted. This happened because Ctrl-C was treated the same as Enter,
causing the (partial) rename to proceed and delete the original model.

This fix:
- Adds a 'cancelled' flag to textInputModel to track cancellation
- Returns the old name when Ctrl-C or Esc is pressed
- Handles cancellation in handleRenameModelKey and handleCopyModelKey
- Updates the UI hint to show both cancel options

Fixes #227

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

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Signed-off-by: majiayu000 <1835304752@qq.com>

* fix: return explicit cancel status from promptForNewName

- Change promptForNewName to return (string, bool) to explicitly signal cancellation
- Fix ambiguity where newName == oldName couldn't distinguish between:
  - User cancelled (Ctrl-C/Esc)
  - User intentionally kept the same name (Tab + Enter)
- Remove unreachable empty name check (was dead code)
- For copy: require different name, show error if same name
- For rename: allow same name gracefully with "Name unchanged" message

Addresses review feedback from GitHub Copilot on PR #228

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

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

---------

Signed-off-by: majiayu000 <1835304752@qq.com>
Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>

Author: majiayu000

app_model.go

@@ -685,9 +685,13 @@ func (m *AppModel) handleCopyModelKey() (tea.Model, tea.Cmd) {
 	}()
 	logging.DebugLogger.Println("CopyModel key matched")
 	if item, ok := m.list.SelectedItem().(Model); ok {
-		newName := promptForNewName(item.Name) // Pass the selected item as the model
-		if newName == "" {
-			m.message = "Error: name can't be empty"
+		newName, cancelled := promptForNewName(item.Name)
+		if cancelled {
+			m.message = styles.InfoStyle().Render("Copy cancelled")
+		} else if newName == "" {
+			m.message = styles.ErrorStyle().Render("Error: name can't be empty")
+		} else if newName == item.Name {
+			m.message = styles.ErrorStyle().Render("Error: new name must be different from the original name")
 		} else {
 			copyModel(m, m.client, item.Name, newName)
 			m.message = fmt.Sprintf("Model %s copied to %s", item.Name, newName)
@@ -790,9 +794,14 @@ func (m *AppModel) handleInspectModelKey() (tea.Model, tea.Cmd) {
 func (m *AppModel) handleRenameModelKey() (tea.Model, tea.Cmd) {
 	logging.DebugLogger.Println("RenameModel key matched")
 	if item, ok := m.list.SelectedItem().(Model); ok {
-		newName := promptForNewName(item.Name)
-		if newName == "" {
+		newName, cancelled := promptForNewName(item.Name)
+		if cancelled {
+			m.message = styles.InfoStyle().Render("Rename cancelled")
+		} else if newName == "" {
 			m.message = styles.ErrorStyle().Render("Error: name can't be empty")
+		} else if newName == item.Name {
+			// User explicitly confirmed the same name, no action needed
+			m.message = styles.InfoStyle().Render("Name unchanged")
 		} else {
 			err := renameModel(m, item.Name, newName)
 			if err != nil {

text_input.go

@@ -15,10 +15,12 @@ type textInputModel struct {
 	textInput textinput.Model
 	oldName   string
 	quitting  bool
+	cancelled bool
 }
 
 // promptForNewName displays a text input prompt for renaming a model.
-func promptForNewName(oldName string) string {
+// Returns the new name and a boolean indicating if the operation was cancelled.
+func promptForNewName(oldName string) (string, bool) {
 	ti := textinput.New()
 	// print 'renaming oldName' to the console with the oldName in purple
 	ti.Prompt = oldName + "\n" + "Name for new model: "
@@ -46,23 +48,33 @@ func promptForNewName(oldName string) string {
 		logging.ErrorLogger.Printf("Error starting text input program: %v\n", err)
 	}
 
+	// If the user cancelled (Ctrl-C or Esc), return empty string and cancelled=true
+	if m.cancelled {
+		logging.InfoLogger.Println("Rename cancelled by user")
+		return "", true
+	}
+
 	newName := m.textInput.Value()
 
+	// If user pressed enter without typing anything, return empty string
 	if newName == "" {
-		// error handling
-		logging.ErrorLogger.Println("No new name entered, returning old name")
-		return oldName
+		logging.DebugLogger.Println("No new name entered, returning empty string")
+		return "", false
 	}
 
-	return newName
+	return newName, false
 }
 
 func (m *textInputModel) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 	var cmd tea.Cmd
 	switch msg := msg.(type) {
 	case tea.KeyMsg:
 		switch msg.String() {
-		case "ctrl+c", "enter":
+		case "ctrl+c", "esc":
+			m.quitting = true
+			m.cancelled = true
+			return m, tea.Quit
+		case "enter":
 			m.quitting = true
 			return m, tea.Quit
 		}
@@ -83,6 +95,6 @@ func (m textInputModel) View() string {
 	return fmt.Sprintf(
 		"\n%s\n\n%s",
 		m.textInput.View(),
-		"(ctrl+c to cancel)",
+		"(esc or ctrl+c to cancel)",
 	)
 }