perf(gui): optimize GUI add_dirs allocations and cloning (#191)

* ⚡ Optimize `add_dirs` method by removing redundant allocations

Replaced the `iter().unique().cloned().collect()` pipeline with a simple
contains check before pushing to `app_state.dirs`. This prevents entire
vector allocations and item clones per user folder pick. Furthermore,
saving the configuration and cloning `dirs` is now only performed if the
app state actually changed (e.g. if the user picked a new folder).

Cleaned up the unused `itertools::Itertools` import.

Co-authored-by: arabianq <55220741+arabianq@users.noreply.github.com>

* ⚡ Optimize GUI `add_dirs` allocations and cloning

Replaced the `iter().unique().cloned().collect()` pipeline with a simple
contains check before pushing to `app_state.dirs`. This prevents entire
vector allocations and item clones per user folder pick. Furthermore,
saving the configuration and cloning `dirs` is now only performed if the
app state actually changed (e.g. if the user picked a new folder).

Cleaned up the unused `itertools::Itertools` import. Cargo.lock was
not touched.

Co-authored-by: arabianq <55220741+arabianq@users.noreply.github.com>

---------

Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>

pwsp-gui/src/gui/mod.rs

@@ -5,7 +5,6 @@ mod views;
 use anyhow::{Result, anyhow};
 use eframe::{NativeOptions, icon_data::from_png_bytes, run_native};
 use egui::{Context, FontData, FontDefinitions, FontFamily, FontTweak, Vec2, ViewportBuilder};
-use itertools::Itertools;
 use pwsp_lib::{
     types::{
         audio_player::PlayerState,
@@ -110,12 +109,17 @@ impl SoundpadGui {
     pub fn add_dirs(&mut self) {
         let file_dialog = FileDialog::new();
         if let Some(paths) = file_dialog.pick_folders() {
+            let mut changed = false;
             for path in paths {
-                self.app_state.dirs.push(path);
+                if !self.app_state.dirs.contains(&path) {
+                    self.app_state.dirs.push(path);
+                    changed = true;
+                }
+            }
+            if changed {
+                self.config.dirs = self.app_state.dirs.clone();
+                self.config.save_to_file().ok();
             }
-            self.app_state.dirs = self.app_state.dirs.iter().unique().cloned().collect();
-            self.config.dirs = self.app_state.dirs.clone();
-            self.config.save_to_file().ok();
         }
     }
 
@@ -196,18 +200,27 @@ impl SoundpadGui {
                     if let Ok(entries) = fs::read_dir(&dir) {
                         let mut children = Vec::new();
                         for entry in entries.filter_map(|e| e.ok()) {
-                            let p = entry.path();
+                            if let Ok(file_type) = entry.file_type() {
-                            if p.is_dir() {
+                                if file_type.is_dir() {
-                                dirs_to_visit.push(p.clone());
+                                    let p = entry.path();
-                                children.push(p);
+                                    dirs_to_visit.push(p.clone());
-                            } else if crate::gui::SUPPORTED_EXTENSIONS.contains(
+                                    children.push(p);
-                                &p.extension()
+                                } else {
-                                    .unwrap_or_default()
+                                    let file_name = entry.file_name();
-                                    .to_str()
+                                    let is_supported = Path::new(&file_name)
-                                    .unwrap_or_default(),
+                                        .extension()
-                            ) {
+                                        .map(|e| {
-                                all_files.push(p.clone());
+                                            crate::gui::SUPPORTED_EXTENSIONS
-                                children.push(p);
+                                                .contains(&e.to_str().unwrap_or_default())
+                                        })
+                                        .unwrap_or(false);
+
+                                    if is_supported {
+                                        let p = entry.path();
+                                        all_files.push(p.clone());
+                                        children.push(p);
+                                    }
+                                }
                             }
                         }
                         dir_updates.insert(dir, children);