Merge pull request #3 from fixture-dev/claude/implement-label-view-012EiC2CYpfS8VTEqafQrkxQ

Implement label view in TUI navigation pane

Author: fohara

crates/lash-tui/src/app.rs

@@ -6,7 +6,7 @@ use std::io;
 use std::path::Path;
 use std::time::Duration;
 
-use lash_db::repository::{FileRepository, TaskRepository};
+use lash_db::repository::{FileRepository, LabelRepository, TaskRepository};
 
 use crate::error::{TuiError, TuiResult};
 use crate::event::{poll_event, AppEvent};
@@ -73,6 +73,12 @@ impl TuiApp {
                 .map_err(|e| TuiError::App(format!("Failed to load tasks: {e}")))?;
         }
 
+        // Load labels with stats
+        let label_repo = LabelRepository::new(&conn);
+        state.labels = label_repo
+            .get_label_stats()
+            .map_err(|e| TuiError::App(format!("Failed to load labels: {e}")))?;
+
         Ok(Self {
             terminal,
             conn,
@@ -138,12 +144,13 @@ impl TuiApp {
                     AppEvent::ExpandAll => self.handle_expand_all(),
                     AppEvent::CollapseAll => self.handle_collapse_all(),
 
+                    AppEvent::LabelFilter => self.handle_label_toggle()?,
+                    AppEvent::ClearFilters => self.state.clear_label_filter(),
+
                     // TODO: implement these features
                     AppEvent::ExpandNode
                     | AppEvent::CollapseNode
-                    | AppEvent::ClearFilters
                     | AppEvent::Search
-                    | AppEvent::LabelFilter
                     | AppEvent::DependencyGraph
                     | AppEvent::PrevTask
                     | AppEvent::NextTask
@@ -163,35 +170,47 @@ impl TuiApp {
 
     /// Handle selection/enter in current pane
     fn handle_select(&mut self) -> TuiResult<()> {
+        use crate::state::{FocusedPane, NavMode};
+
         match self.state.focused_pane {
-            crate::state::FocusedPane::Navigation => {
-                // Check if tree view is active and get selected node info
-                if let Some(selected) = self.state.selected_tree_node() {
-                    if selected.is_directory {
-                        // Toggle expand/collapse for directories
-                        self.state.toggle_selected_node();
-                    } else if let Some(file) = selected.file_record {
-                        // Load tasks for file and switch to detail pane
-                        let task_repo = TaskRepository::new(&self.conn);
-                        self.state.tasks = task_repo
-                            .get_by_file(file.id)
-                            .map_err(|e| TuiError::App(format!("Failed to load tasks: {e}")))?;
-                        self.state.selected_task_index = 0;
-                        self.state.switch_pane();
+            FocusedPane::Navigation => {
+                match self.state.nav_mode {
+                    NavMode::Files | NavMode::SearchResults => {
+                        // Check if tree view is active and get selected node info
+                        if let Some(selected) = self.state.selected_tree_node() {
+                            if selected.is_directory {
+                                // Toggle expand/collapse for directories
+                                self.state.toggle_selected_node();
+                            } else if let Some(file) = selected.file_record {
+                                // Load tasks for file and switch to detail pane
+                                let task_repo = TaskRepository::new(&self.conn);
+                                self.state.tasks = task_repo.get_by_file(file.id).map_err(|e| {
+                                    TuiError::App(format!("Failed to load tasks: {e}"))
+                                })?;
+                                self.state.selected_task_index = 0;
+                                self.state.build_task_tree();
+                                self.state.switch_pane();
+                            }
+                        } else {
+                            // Fallback for flat view: load tasks for selected file
+                            if let Some(file) = self.state.selected_file() {
+                                let task_repo = TaskRepository::new(&self.conn);
+                                self.state.tasks = task_repo.get_by_file(file.id).map_err(|e| {
+                                    TuiError::App(format!("Failed to load tasks: {e}"))
+                                })?;
+                                self.state.selected_task_index = 0;
+                                self.state.build_task_tree();
+                                self.state.switch_pane();
+                            }
+                        }
                     }
-                } else {
-                    // Fallback for flat view: load tasks for selected file
-                    if let Some(file) = self.state.selected_file() {
-                        let task_repo = TaskRepository::new(&self.conn);
-                        self.state.tasks = task_repo
-                            .get_by_file(file.id)
-                            .map_err(|e| TuiError::App(format!("Failed to load tasks: {e}")))?;
-                        self.state.selected_task_index = 0;
-                        self.state.switch_pane();
+                    NavMode::Labels => {
+                        // Select a label and filter tasks
+                        self.handle_label_select()?;
                     }
                 }
             }
-            crate::state::FocusedPane::Detail => {
+            FocusedPane::Detail => {
                 // Enter on detail pane shows task details
                 // TODO: Implement task detail view
             }
@@ -329,6 +348,43 @@ impl TuiApp {
             }
         }
     }
+
+    /// Handle toggling between files and labels view
+    fn handle_label_toggle(&mut self) -> TuiResult<()> {
+        // Reload labels to ensure we have latest stats
+        let label_repo = LabelRepository::new(&self.conn);
+        self.state.labels = label_repo
+            .get_label_stats()
+            .map_err(|e| TuiError::App(format!("Failed to load labels: {e}")))?;
+
+        self.state.toggle_nav_mode();
+        Ok(())
+    }
+
+    /// Handle selecting a label to filter tasks
+    fn handle_label_select(&mut self) -> TuiResult<()> {
+        let Some(label) = self.state.selected_label() else {
+            return Ok(());
+        };
+
+        let label_name = label.name.clone();
+
+        // Load tasks with this label
+        let task_repo = TaskRepository::new(&self.conn);
+        self.state.tasks = task_repo
+            .find_by_label(&label_name)
+            .map_err(|e| TuiError::App(format!("Failed to load tasks for label: {e}")))?;
+
+        // Set current filter and reset selection
+        self.state.current_label_filter = Some(label_name);
+        self.state.selected_task_index = 0;
+        self.state.build_task_tree();
+
+        // Switch to detail pane to show filtered tasks
+        self.state.focused_pane = crate::state::FocusedPane::Detail;
+
+        Ok(())
+    }
 }
 
 impl Drop for TuiApp {

crates/lash-tui/src/event.rs

@@ -114,6 +114,7 @@ fn handle_key_event(key: KeyEvent) -> AppEvent {
         (KeyCode::Char(' '), KeyModifiers::NONE) => AppEvent::Select,
         (KeyCode::Char('e'), KeyModifiers::NONE) => AppEvent::OpenEditor,
         (KeyCode::Char('/'), KeyModifiers::NONE) => AppEvent::Search,
+        (KeyCode::Char('F'), KeyModifiers::SHIFT) => AppEvent::LabelFilter,
         (KeyCode::Char('c'), KeyModifiers::NONE) => AppEvent::ClearFilters,
         (KeyCode::Char('g'), KeyModifiers::CONTROL) => AppEvent::DependencyGraph,
         (KeyCode::Char('?'), KeyModifiers::SHIFT) => AppEvent::Help,

crates/lash-tui/src/state.rs

@@ -4,6 +4,7 @@
 
 use crate::colors::Theme;
 use lash_db::repository::files::FileRecord;
+use lash_db::repository::labels::LabelStats;
 use lash_db::repository::tasks::TaskRecord;
 use lash_types::tree::{TreeChars, TreeNode};
 use std::path::PathBuf;
@@ -88,6 +89,15 @@ pub struct AppState {
 
     /// Tree rendering characters (Unicode or ASCII)
     pub tree_chars: TreeChars,
+
+    /// Available labels with task counts (for label view)
+    pub labels: Vec<LabelStats>,
+
+    /// Currently selected label (index into labels vec)
+    pub selected_label_index: usize,
+
+    /// Current label filter (if any)
+    pub current_label_filter: Option<String>,
 }
 
 /// State for the theme selector modal
@@ -150,6 +160,9 @@ impl AppState {
             file_tree: None,
             task_tree: None,
             tree_chars: TreeChars::detect(),
+            labels: Vec::new(),
+            selected_label_index: 0,
+            current_label_filter: None,
         }
     }
 
@@ -215,11 +228,18 @@ impl AppState {
     /// Move selection up
     pub fn move_up(&mut self) {
         match self.focused_pane {
-            FocusedPane::Navigation => {
-                if self.selected_file_index > 0 {
-                    self.selected_file_index -= 1;
+            FocusedPane::Navigation => match self.nav_mode {
+                NavMode::Files | NavMode::SearchResults => {
+                    if self.selected_file_index > 0 {
+                        self.selected_file_index -= 1;
+                    }
                 }
-            }
+                NavMode::Labels => {
+                    if self.selected_label_index > 0 {
+                        self.selected_label_index -= 1;
+                    }
+                }
+            },
             FocusedPane::Detail => {
                 if self.selected_task_index > 0 {
                     self.selected_task_index -= 1;
@@ -231,12 +251,19 @@ impl AppState {
     /// Move selection down
     pub fn move_down(&mut self) {
         match self.focused_pane {
-            FocusedPane::Navigation => {
-                let max_index = self.visible_tree_node_count();
-                if self.selected_file_index + 1 < max_index {
-                    self.selected_file_index += 1;
+            FocusedPane::Navigation => match self.nav_mode {
+                NavMode::Files | NavMode::SearchResults => {
+                    let max_index = self.visible_tree_node_count();
+                    if self.selected_file_index + 1 < max_index {
+                        self.selected_file_index += 1;
+                    }
                 }
-            }
+                NavMode::Labels => {
+                    if self.selected_label_index + 1 < self.labels.len() {
+                        self.selected_label_index += 1;
+                    }
+                }
+            },
             FocusedPane::Detail => {
                 if self.selected_task_index + 1 < self.tasks.len() {
                     self.selected_task_index += 1;
@@ -248,20 +275,30 @@ impl AppState {
     /// Go to top of current list
     pub fn go_top(&mut self) {
         match self.focused_pane {
-            FocusedPane::Navigation => self.selected_file_index = 0,
+            FocusedPane::Navigation => match self.nav_mode {
+                NavMode::Files | NavMode::SearchResults => self.selected_file_index = 0,
+                NavMode::Labels => self.selected_label_index = 0,
+            },
             FocusedPane::Detail => self.selected_task_index = 0,
         }
     }
 
     /// Go to bottom of current list
     pub fn go_bottom(&mut self) {
         match self.focused_pane {
-            FocusedPane::Navigation => {
-                let max_index = self.visible_tree_node_count();
-                if max_index > 0 {
-                    self.selected_file_index = max_index - 1;
+            FocusedPane::Navigation => match self.nav_mode {
+                NavMode::Files | NavMode::SearchResults => {
+                    let max_index = self.visible_tree_node_count();
+                    if max_index > 0 {
+                        self.selected_file_index = max_index - 1;
+                    }
                 }
-            }
+                NavMode::Labels => {
+                    if !self.labels.is_empty() {
+                        self.selected_label_index = self.labels.len() - 1;
+                    }
+                }
+            },
             FocusedPane::Detail => {
                 if !self.tasks.is_empty() {
                     self.selected_task_index = self.tasks.len() - 1;
@@ -486,6 +523,38 @@ impl AppState {
         self.tasks.get(self.selected_task_index)
     }
 
+    /// Get currently selected label
+    #[must_use]
+    pub fn selected_label(&self) -> Option<&LabelStats> {
+        self.labels.get(self.selected_label_index)
+    }
+
+    /// Switch to labels view mode
+    pub fn switch_to_labels(&mut self) {
+        self.nav_mode = NavMode::Labels;
+        self.selected_label_index = 0;
+    }
+
+    /// Switch to files view mode
+    pub fn switch_to_files(&mut self) {
+        self.nav_mode = NavMode::Files;
+        self.current_label_filter = None;
+    }
+
+    /// Toggle between files and labels view
+    pub fn toggle_nav_mode(&mut self) {
+        match self.nav_mode {
+            NavMode::Files => self.switch_to_labels(),
+            NavMode::Labels | NavMode::SearchResults => self.switch_to_files(),
+        }
+    }
+
+    /// Clear label filter and return to file view
+    pub fn clear_label_filter(&mut self) {
+        self.current_label_filter = None;
+        self.nav_mode = NavMode::Files;
+    }
+
     /// Build file tree from flat file list
     ///
     /// Converts the flat `self.files` list into a hierarchical directory tree.

crates/lash-tui/src/ui/detail_pane.rs

@@ -23,10 +23,14 @@ pub fn render(frame: &mut Frame, area: Rect, state: &AppState) {
         themes::unfocused_border_style(theme)
     };
 
-    // Get title from selected file
-    let title = state
-        .selected_file()
-        .map_or_else(|| " Tasks ".to_string(), |f| format!(" {} ", f.title));
+    // Get title - show label filter if active, otherwise selected file
+    let title = if let Some(label) = &state.current_label_filter {
+        format!(" Tasks [#{label}] ")
+    } else {
+        state
+            .selected_file()
+            .map_or_else(|| " Tasks ".to_string(), |f| format!(" {} ", f.title))
+    };
 
     let block = Block::default()
         .borders(Borders::ALL)

crates/lash-tui/src/ui/help.rs

@@ -56,7 +56,8 @@ pub fn render(frame: &mut Frame, area: Rect, _state: &AppState) {
         Line::from("  Space          Toggle task status"),
         Line::from("  e              Open file in $EDITOR"),
         Line::from("  /              Search"),
-        Line::from("  c              Clear filters"),
+        Line::from("  F              Toggle Files/Labels view"),
+        Line::from("  c              Clear label filter"),
         Line::from("  Ctrl-g         Show dependency graph"),
         Line::from(""),
         Line::from(vec![Span::styled(