chore: move m.search.Update out of case

DanielleMaywood · DanielleMaywood · commit 579c73b9ef90 · 2024-09-04T10:18:49.000Z
diff --git a/cli/cliui/select.go b/cli/cliui/select.go
@@ -368,19 +368,18 @@ func (m multiSelectModel) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 			for _, option := range options {
 				option.chosen = false
 			}
+		}
+	}
 
-		default:
-			oldSearch := m.search.Value()
-			m.search, cmd = m.search.Update(msg)
-
-			// If the search query has changed then we need to ensure
-			// the cursor is still pointing at a valid option.
-			if m.search.Value() != oldSearch {
-				options := m.filteredOptions()
-				if m.cursor > len(options)-1 {
-					m.cursor = max(0, len(options)-1)
-				}
-			}
+	oldSearch := m.search.Value()
+	m.search, cmd = m.search.Update(msg)
+
+	// If the search query has changed then we need to ensure
+	// the cursor is still pointing at a valid option.
+	if m.search.Value() != oldSearch {
+		options := m.filteredOptions()
+		if m.cursor > len(options)-1 {
+			m.cursor = max(0, len(options)-1)
 		}
 	}
 
