Skip to content

Commit 03bd864

Browse files
committed
fix(tui): require Enter to open /connect provider picker
Typing exactly '/connect' previously auto-opened the interactive provider login picker, conflicting with the documented behavior where '/connect' (no provider) only opens the picker on submit. Drop the keystroke-triggered show_interactive_login() call in handle_text_input; the submit path (handle_auth_command) still opens the picker when the user presses Enter on bare '/connect'.
1 parent 71e5683 commit 03bd864

1 file changed

Lines changed: 0 additions & 8 deletions

File tree

crates/jcode-tui/src/tui/app/input.rs

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -782,14 +782,6 @@ pub(super) fn handle_text_input(app: &mut App, text: &str) -> bool {
782782
}
783783

784784
insert_input_text(app, text);
785-
// Convenience: typing '/connect' (exactly 8 chars) auto-opens the
786-
// interactive provider login picker so the user does not have to
787-
// press Enter to summon it. The input is left untouched so the
788-
// user can keep typing a provider name to refine ('/connect
789-
// openai' will start the openai flow on Enter).
790-
if app.input == "/connect" {
791-
app.show_interactive_login();
792-
}
793785
true
794786
}
795787

0 commit comments

Comments
 (0)