refactor: clean up unused dependencies, simplify imports, and update menu bar implementation

Author: sguzman

src/app/capture.rs

@@ -1,6 +1,4 @@
 use super::*;
-use crate::engine::{RenderSurfaceMetadata};
-use eframe::egui::ColorImage;
 
 impl BrazenApp {
     pub(super) fn update_render_frame(&mut self, ctx: &eframe::egui::Context) {

src/app/input.rs

@@ -1,5 +1,5 @@
 use super::*;
-use crate::engine::{InputEvent, AlphaMode};
+use crate::engine::InputEvent;
 
 impl BrazenApp {
     pub(super) fn forward_input_events(&mut self, ctx: &eframe::egui::Context) {

src/app/mod.rs

@@ -11,32 +11,27 @@ pub mod recovery;
 pub mod workspace;
 pub use state::*;
 
-
 use chrono::Utc;
-use serde::{Deserialize, Serialize};
-use eframe::egui::Color32;
 use std::collections::{HashMap, VecDeque};
-use std::path::PathBuf;
 use std::sync::{Arc, RwLock};
 use std::time::{Duration, Instant};
 
 use crate::automation::{
     AutomationCapabilityEvent, AutomationCommand, AutomationHandle, AutomationNavigationEvent,
-    AutomationActivityStatus, drain_automation_commands,
+    drain_automation_commands,
 };
-use crate::cache::{AssetQuery, AssetStore};
+use crate::cache::AssetStore;
 use crate::commands::{AppCommand, dispatch_command};
 use crate::config::BrazenConfig;
 use crate::engine::{
-    AlphaMode, BrowserEngine, BrowserTab, DialogKind, EngineEvent, EngineFactory, EngineLoadStatus,
-    EngineStatus, FocusState, InputEvent, RenderSurfaceHandle, RenderSurfaceMetadata,
-    SecurityWarningKind, WindowDisposition,
+    AlphaMode, BrowserEngine, EngineFactory, EngineLoadStatus, FocusState,
+    RenderSurfaceHandle, RenderSurfaceMetadata, WindowDisposition,
 };
 use crate::navigation::{normalize_url_input, resolve_startup_url};
 use crate::permissions::Capability;
 use crate::platform_paths::RuntimePaths;
 use crate::rendering::{normalize_pixels, probe_frame_stats};
-use crate::session::{NavigationEntry, SessionSnapshot, load_session};
+use crate::session::{SessionSnapshot, load_session};
 use crate::profile_db::ProfileDb;
 use tokio::sync::mpsc;
 
@@ -724,7 +719,3 @@ impl BrazenApp {
 #[cfg(test)]
 mod tests;
 
-
-fn extract_entities(html: &str) -> Vec<ExtractedEntity> {
-    crate::extraction::extract_entities(html)
-}

src/app/secondary_ui.rs

@@ -1,4 +1,4 @@
-use super::state::*;
+use super::*;
 use crate::cache::AssetQuery;
 
 impl super::BrazenApp {

src/app/state.rs

@@ -1,18 +1,15 @@
-use crate::engine::{BrowserEngine, EngineEvent, EngineStatus, FocusState, RenderSurfaceMetadata, SecurityWarningKind, WindowDisposition, BrowserTab, DialogKind};
+use crate::engine::{BrowserEngine, EngineEvent, EngineStatus, SecurityWarningKind, WindowDisposition, BrowserTab, DialogKind};
 use crate::extraction::extract_entities;
-use std::time::Duration;
 use std::collections::VecDeque;
 use crate::platform_paths::RuntimePaths;
 use std::sync::{Arc, RwLock};
 
 use chrono::Utc;
-use crate::session::{NavigationEntry, SessionSnapshot, load_session};
+use crate::session::{NavigationEntry, SessionSnapshot};
 use crate::engine::EngineLoadStatus;
 use crate::profile_db::ProfileDb;
-use std::time::Instant;
 use serde::{Deserialize, Serialize};
 use std::collections::HashMap;
-use std::path::PathBuf;
 
 
 #[derive(Debug, Clone, Serialize, Deserialize)]

src/app/ui_components.rs

@@ -304,7 +304,7 @@ impl super::BrazenApp {
 
     pub fn render_top_menu(&mut self, ctx: &eframe::egui::Context) {
         eframe::egui::TopBottomPanel::top("top_menu_bar").show(ctx, |ui| {
-            eframe::egui::menu::bar(ui, |ui| {
+            eframe::egui::MenuBar::new().ui(ui, |ui| {
                 ui.menu_button("File", |ui| {
                     if ui.button("New Tab").clicked() {
                         self.apply_palette_command(PaletteCommand::NewTab);

src/app/ui_main.rs

@@ -1,7 +1,5 @@
-use super::state::*;
-use crate::engine::RenderSurfaceMetadata;
+use super::*;
 use crate::navigation::normalize_url_input;
-use std::collections::VecDeque;
 
 impl super::BrazenApp {
 
@@ -31,16 +29,6 @@ impl super::BrazenApp {
 
 
 
-fn empty_to_none(value: &str) -> Option<String> {
-    let trimmed = value.trim();
-    if trimmed.is_empty() {
-        None
-    } else {
-        Some(trimmed.to_string())
-    }
-}
-
-
 
 
 

src/app/workspace.rs

@@ -1,10 +1,6 @@
 use super::*;
 
 impl BrazenApp {
-    pub(super) fn load_workspace_layout(path: &PathBuf) -> Option<WorkspaceLayout> {
-        let data = std::fs::read_to_string(path).ok()?;
-        serde_json::from_str(&data).ok()
-    }
 
     pub(super) fn save_workspace_layout(&self) {
         let Some(db) = &self.profile_db else {