From bbfcfa20821ff8f0b1860d9cf0bb3376e8e64d0a Mon Sep 17 00:00:00 2001
From: teernisse <teernisse@visiostack.com>
Date: Wed, 18 Feb 2026 23:59:04 -0500
Subject: [PATCH] fix(tui): bounds-check scope picker selected index

Replace direct indexing (self.projects[self.selected_index - 1]) with
.get() to prevent panic if selected_index is somehow out of bounds.
Falls back to "All Projects" scope when the index is invalid instead
of panicking.
---
 crates/lore-tui/src/state/scope_picker.rs | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/crates/lore-tui/src/state/scope_picker.rs b/crates/lore-tui/src/state/scope_picker.rs
index 33af80d..9c3274b 100644
--- a/crates/lore-tui/src/state/scope_picker.rs
+++ b/crates/lore-tui/src/state/scope_picker.rs
@@ -76,12 +76,17 @@ impl ScopePickerState {
                 project_id: None,
                 project_name: None,
             }
-        } else {
-            let project = &self.projects[self.selected_index - 1];
+        } else if let Some(project) = self.projects.get(self.selected_index - 1) {
             ScopeContext {
                 project_id: Some(project.id),
                 project_name: Some(project.path.clone()),
             }
+        } else {
+            // Out-of-bounds — fall back to "All Projects".
+            ScopeContext {
+                project_id: None,
+                project_name: None,
+            }
         }
     }