From c4ab603176064933ff69bd233337c55d62729f61 Mon Sep 17 00:00:00 2001
From: Ignacio Perez <ignacio@feuer.me>
Date: Fri, 26 Dec 2025 10:42:23 -0300
Subject: [PATCH] refactor: Remove excessive logging from app initialization

---
 src/app.rs | 58 ++++--------------------------------------------------
 1 file changed, 4 insertions(+), 54 deletions(-)

diff --git a/src/app.rs b/src/app.rs
index 49cb846..a54e7d9 100644
--- a/src/app.rs
+++ b/src/app.rs
@@ -62,21 +62,11 @@ impl App {
                     app.new(|cx: &mut gpui::Context<'_, App>| {
                         let filter_state = cx.new(|_cx| FilterState::new());
 
-                        log::info!("Initializing TaskService");
-                        let mut task_service = match TaskService::new() {
-                            Ok(service) => {
-                                log::info!("TaskService initialized successfully");
-                                service
-                            }
-                            Err(e) => {
-                                log::error!("Failed to initialize TaskService: {:?}", e);
-                                panic!("Cannot continue without TaskService");
-                            }
-                        };
+                        let mut task_service = TaskService::new()
+                            .unwrap_or_else(|e| panic!("Failed to initialize TaskService: {}", e));
 
-                        log::info!("Loading overview (tasks, projects, tags)");
                         let overview = task_service.get_overview().unwrap_or_else(|e| {
-                            log::error!("Failed to get overview: {:?}", e);
+                            log::error!("Failed to load tasks: {}", e);
                             TaskOverview {
                                 tasks: vec![],
                                 projects: vec![],
@@ -87,59 +77,19 @@ impl App {
                             }
                         });
 
-                        log::info!(
-                            "Loaded {} tasks, {} projects, {} tags",
-                            overview.total_tasks,
-                            overview.projects.len(),
-                            overview.tags.len()
-                        );
-
-                        for (i, task) in overview.tasks.iter().take(5).enumerate() {
-                            log::debug!(
-                                "Task {}: {} (project: {:?}, tags: {:?})",
-                                i + 1,
-                                task.description,
-                                task.project,
-                                task.tags
-                            );
-                        }
-                        if overview.tasks.len() > 5 {
-                            log::debug!("... and {} more tasks", overview.tasks.len() - 5);
-                        }
-
                         let mut project_tree = ProjectTree::new();
                         project_tree.build_from_projects(&overview.projects);
 
                         let tags: Vec<TagItem> = overview
                             .tags
                             .into_iter()
-                            .map(|(name, task_count)| {
-                                log::debug!("Tag: {} ({} tasks)", name, task_count);
-                                TagItem { name, task_count }
-                            })
+                            .map(|(name, task_count)| TagItem { name, task_count })
                             .collect();
 
-                        log::info!("Projects and tags loaded successfully");
-
                         let status_bar = cx.new(|cx| StatusBar::new(cx));
 
                         let sidebar = cx.new(|cx| {
                             Sidebar::new(project_tree, tags, filter_state.clone(), cx)
-                                .on_filter_change(|filter, _window, _cx| {
-                                    log::info!("Filter changed");
-                                    if let Some(ref project) = filter.selected_project {
-                                        log::debug!("Selected project: {}", project);
-                                    } else {
-                                        log::debug!("Selected project: All");
-                                    }
-                                    if filter.active_tags.is_empty() {
-                                        log::debug!("Active tags: None");
-                                    } else {
-                                        let tags_list: Vec<&str> =
-                                            filter.active_tags.iter().map(|s| s.as_str()).collect();
-                                        log::debug!("Active tags: {}", tags_list.join(", "));
-                                    }
-                                })
                         });
 
                         App {