From ff8be525a8de81bd5631cb4cff1326dede3b8d53 Mon Sep 17 00:00:00 2001
From: youdie006 <xncb135@korea.ac.kr>
Date: Fri, 19 Jun 2026 15:03:43 +0900
Subject: [PATCH] fix(conversations): let g/G be typed into the session search
 box

The session-filter search input bound g/G to goto-top/bottom list navigation,
so they were swallowed before reaching the query (typing "go" yielded "o").
Drop those bindings so g/G fall through to the default typing branch, mirroring
the arrow-key-only navigation precedent already applied to up/down/h here.

Closes #166
---
 .../plugins/conversations/plugin_input.go     | 21 ++--------
 internal/plugins/conversations/plugin_test.go | 40 +++++++++++++++++++
 2 files changed, 44 insertions(+), 17 deletions(-)

diff --git a/internal/plugins/conversations/plugin_input.go b/internal/plugins/conversations/plugin_input.go
index 98213c96..c952b33e 100644
--- a/internal/plugins/conversations/plugin_input.go
+++ b/internal/plugins/conversations/plugin_input.go
@@ -351,23 +351,10 @@ func (p *Plugin) updateSearch(msg tea.KeyPressMsg) (plugin.Plugin, tea.Cmd) {
 			return p, p.schedulePreviewLoad(p.selectedSession)
 		}
 
-	case "g":
-		p.cursor = 0
-		p.scrollOff = 0
-		sessions := p.visibleSessions()
-		if len(sessions) > 0 {
-			p.setSelectedSession(sessions[0].ID)
-			return p, p.schedulePreviewLoad(p.selectedSession)
-		}
-
-	case "G":
-		sessions := p.visibleSessions()
-		if len(sessions) > 0 {
-			p.cursor = len(sessions) - 1
-			p.ensureCursorVisible()
-			p.setSelectedSession(sessions[p.cursor].ID)
-			return p, p.schedulePreviewLoad(p.selectedSession)
-		}
+	// Note: 'g'/'G' goto-top/bottom are intentionally NOT bound here so they
+	// fall through to the default branch and are typed into the search query
+	// (issue #166). This mirrors the arrow-key-only navigation precedent
+	// (td-2467e8) already applied to up/down/h in the search input.
 
 	default:
 		// Add character to search query
diff --git a/internal/plugins/conversations/plugin_test.go b/internal/plugins/conversations/plugin_test.go
index ed9716d4..d527ea8a 100644
--- a/internal/plugins/conversations/plugin_test.go
+++ b/internal/plugins/conversations/plugin_test.go
@@ -804,6 +804,46 @@ func TestUpdateSearchBackspace(t *testing.T) {
 	}
 }
 
+// TestUpdateSearchTypingGoto verifies that 'g' and 'G' are typed into the
+// search query instead of being swallowed by list goto-top/bottom navigation
+// while the search input is active (issue #166). Mirrors the arrow-key-only
+// navigation precedent (td-2467e8) already applied to up/down/h in this view.
+func TestUpdateSearchTypingGoto(t *testing.T) {
+	p := New()
+	p.adapters = map[string]adapter.Adapter{"mock": &mockAdapter{}}
+	p.sessions = []adapter.Session{
+		{ID: "test-1", Name: "go-build"},
+		{ID: "test-2", Name: "Golang"},
+		{ID: "test-3", Name: "other"},
+	}
+	p.searchMode = true
+
+	// Type 'g' — must land in the query, not trigger goto-top.
+	msg := tea.KeyPressMsg{Code: 'g', Text: "g"}
+	_, _ = p.Update(msg)
+
+	if p.searchQuery != "g" {
+		t.Errorf("expected 'g' to be typed into searchQuery, got %q (likely swallowed by goto-top navigation)", p.searchQuery)
+	}
+
+	// Type 'o' — completing "go", the exact scenario from issue #166.
+	msg = tea.KeyPressMsg{Code: 'o', Text: "o"}
+	_, _ = p.Update(msg)
+
+	if p.searchQuery != "go" {
+		t.Errorf("expected searchQuery 'go', got %q", p.searchQuery)
+	}
+
+	// 'G' (shift+g) must also be typed, not trigger goto-bottom.
+	p.searchQuery = ""
+	msg = tea.KeyPressMsg{Code: 'G', Text: "G"}
+	_, _ = p.Update(msg)
+
+	if p.searchQuery != "G" {
+		t.Errorf("expected 'G' to be typed into searchQuery, got %q (likely swallowed by goto-bottom navigation)", p.searchQuery)
+	}
+}
+
 // TestUpdateSearchNavigationDown tests down navigation in search results.
 func TestUpdateSearchNavigationDown(t *testing.T) {
 	p := New()