refactor: extract profile functionality into its own module

src-tauri/src/auto_updater.rs

@@ -1,6 +1,6 @@
 use crate::api_client::is_browser_version_nightly;
-use crate::browser_runner::{BrowserProfile, BrowserRunner};
 use crate::browser_version_service::{BrowserVersionInfo, BrowserVersionService};
+use crate::profile::BrowserProfile;
 use crate::settings_manager::SettingsManager;
 use serde::{Deserialize, Serialize};
 use std::collections::{HashMap, HashSet};
@@ -30,7 +30,6 @@ pub struct AutoUpdateState {
 
 pub struct AutoUpdater {
   version_service: BrowserVersionService,
-  browser_runner: BrowserRunner,
   settings_manager: SettingsManager,
 }
 
@@ -38,7 +37,6 @@ impl AutoUpdater {
   pub fn new() -> Self {
     Self {
       version_service: BrowserVersionService::new(),
-      browser_runner: BrowserRunner::new(),
       settings_manager: SettingsManager::new(),
     }
   }
@@ -51,8 +49,8 @@ impl AutoUpdater {
     let mut browser_versions: HashMap<String, Vec<BrowserVersionInfo>> = HashMap::new();
 
     // Group profiles by browser
-    let profiles = self
-      .browser_runner
+    let profile_manager = crate::profile::ProfileManager::new();
+    let profiles = profile_manager
       .list_profiles()
       .map_err(|e| format!("Failed to list profiles: {e}"))?;
     let mut browser_profiles: HashMap<String, Vec<BrowserProfile>> = HashMap::new();
@@ -294,8 +292,8 @@ impl AutoUpdater {
     browser: &str,
     new_version: &str,
   ) -> Result<Vec<String>, Box<dyn std::error::Error + Send + Sync>> {
-    let profiles = self
-      .browser_runner
+    let profile_manager = crate::profile::ProfileManager::new();
+    let profiles = profile_manager
       .list_profiles()
       .map_err(|e| format!("Failed to list profiles: {e}"))?;
 
@@ -312,10 +310,7 @@ impl AutoUpdater {
         // Check if this is an update (newer version)
         if self.is_version_newer(new_version, &profile.version) {
           // Update the profile version
-          match self
-            .browser_runner
-            .update_profile_version(&profile.name, new_version)
-          {
+          match profile_manager.update_profile_version(&profile.name, new_version) {
             Ok(_) => {
               updated_profiles.push(profile.name);
             }
@@ -361,8 +356,8 @@ impl AutoUpdater {
     &self,
   ) -> Result<Vec<String>, Box<dyn std::error::Error + Send + Sync>> {
     // Load current profiles
-    let profiles = self
-      .browser_runner
+    let profile_manager = crate::profile::ProfileManager::new();
+    let profiles = profile_manager
       .list_profiles()
       .map_err(|e| format!("Failed to load profiles: {e}"))?;
 

src-tauri/src/camoufox.rs

@@ -1,4 +1,4 @@
-use crate::browser_runner::BrowserProfile;
+use crate::profile::BrowserProfile;
 use serde::{Deserialize, Serialize};
 use std::collections::HashMap;
 use std::sync::Arc;

src-tauri/src/downloaded_browsers.rs

@@ -201,7 +201,7 @@ impl DownloadedBrowsersRegistry {
   /// Get all browsers and versions referenced by active profiles
   pub fn get_active_browser_versions(
     &self,
-    profiles: &[crate::browser_runner::BrowserProfile],
+    profiles: &[crate::profile::BrowserProfile],
   ) -> Vec<(String, String)> {
     profiles
       .iter()
@@ -251,7 +251,7 @@ impl DownloadedBrowsersRegistry {
   /// Get all browsers and versions that are currently running
   pub fn get_running_browser_versions(
     &self,
-    profiles: &[crate::browser_runner::BrowserProfile],
+    profiles: &[crate::profile::BrowserProfile],
   ) -> Vec<(String, String)> {
     profiles
       .iter()

src-tauri/src/group_manager.rs

@@ -140,7 +140,7 @@ impl GroupManager {
 
   pub fn get_groups_with_profile_counts(
     &self,
-    profiles: &[crate::browser_runner::BrowserProfile],
+    profiles: &[crate::profile::BrowserProfile],
   ) -> Result<Vec<GroupWithCount>, Box<dyn std::error::Error>> {
     let groups = self.get_all_groups()?;
     let mut group_counts = HashMap::new();
@@ -186,9 +186,7 @@ lazy_static::lazy_static! {
 }
 
 // Helper function to get groups with counts
-pub fn get_groups_with_counts(
-  profiles: &[crate::browser_runner::BrowserProfile],
-) -> Vec<GroupWithCount> {
+pub fn get_groups_with_counts(profiles: &[crate::profile::BrowserProfile]) -> Vec<GroupWithCount> {
   let group_manager = GROUP_MANAGER.lock().unwrap();
   group_manager
     .get_groups_with_profile_counts(profiles)
@@ -206,8 +204,8 @@ pub async fn get_profile_groups() -> Result<Vec<ProfileGroup>, String> {
 
 #[tauri::command]
 pub async fn get_groups_with_profile_counts() -> Result<Vec<GroupWithCount>, String> {
-  let browser_runner = crate::browser_runner::BrowserRunner::new();
-  let profiles = browser_runner
+  let profile_manager = crate::profile::ProfileManager::new();
+  let profiles = profile_manager
     .list_profiles()
     .map_err(|e| format!("Failed to list profiles: {e}"))?;
   Ok(get_groups_with_counts(&profiles))
@@ -242,16 +240,16 @@ pub async fn assign_profiles_to_group(
   profile_names: Vec<String>,
   group_id: Option<String>,
 ) -> Result<(), String> {
-  let browser_runner = crate::browser_runner::BrowserRunner::new();
-  browser_runner
+  let profile_manager = crate::profile::ProfileManager::new();
+  profile_manager
     .assign_profiles_to_group(profile_names, group_id)
     .map_err(|e| format!("Failed to assign profiles to group: {e}"))
 }
 
 #[tauri::command]
 pub async fn delete_selected_profiles(profile_names: Vec<String>) -> Result<(), String> {
-  let browser_runner = crate::browser_runner::BrowserRunner::new();
-  browser_runner
+  let profile_manager = crate::profile::ProfileManager::new();
+  profile_manager
     .delete_multiple_profiles(profile_names)
     .map_err(|e| format!("Failed to delete profiles: {e}"))
 }

src-tauri/src/lib.rs

@@ -20,7 +20,8 @@ mod downloaded_browsers;
 mod extraction;
 mod geoip_downloader;
 mod group_manager;
-
+mod platform_browser;
+mod profile;
 mod profile_importer;
 mod proxy_manager;
 mod settings_manager;
@@ -36,7 +37,8 @@ use browser_runner::{
   fetch_browser_versions_with_count, fetch_browser_versions_with_count_cached_first,
   get_browser_release_types, get_downloaded_browser_versions, get_supported_browsers,
   is_browser_supported_on_platform, kill_browser_profile, launch_browser_profile,
-  list_browser_profiles, rename_profile, update_profile_proxy, update_profile_version,
+  list_browser_profiles, rename_profile, update_camoufox_config, update_profile_proxy,
+  update_profile_version,
 };
 
 use settings_manager::{
@@ -174,17 +176,6 @@ async fn delete_stored_proxy(proxy_id: String) -> Result<(), String> {
     .map_err(|e| format!("Failed to delete stored proxy: {e}"))
 }
 
-#[tauri::command]
-async fn update_camoufox_config(
-  profile_name: String,
-  config: crate::camoufox::CamoufoxConfig,
-) -> Result<(), String> {
-  let browser_runner = browser_runner::BrowserRunner::new();
-  browser_runner
-    .update_camoufox_config(&profile_name, config)
-    .map_err(|e| format!("Failed to update Camoufox config: {e}"))
-}
-
 #[cfg_attr(mobile, tauri::mobile_entry_point)]
 pub fn run() {
   let args: Vec<String> = env::args().collect();

src-tauri/src/profile/mod.rs

@@ -0,0 +1,5 @@
+pub mod manager;
+pub mod types;
+
+pub use manager::ProfileManager;
+pub use types::BrowserProfile;

src-tauri/src/profile/types.rs

@@ -0,0 +1,34 @@
+use crate::camoufox::CamoufoxConfig;
+use serde::{Deserialize, Serialize};
+use std::path::{Path, PathBuf};
+
+#[derive(Debug, Serialize, Deserialize, Clone)]
+pub struct BrowserProfile {
+  pub id: uuid::Uuid,
+  pub name: String,
+  pub browser: String,
+  pub version: String,
+  #[serde(default)]
+  pub proxy_id: Option<String>, // Reference to stored proxy
+  #[serde(default)]
+  pub process_id: Option<u32>,
+  #[serde(default)]
+  pub last_launch: Option<u64>,
+  #[serde(default = "default_release_type")]
+  pub release_type: String, // "stable" or "nightly"
+  #[serde(default)]
+  pub camoufox_config: Option<CamoufoxConfig>, // Camoufox configuration
+  #[serde(default)]
+  pub group_id: Option<String>, // Reference to profile group
+}
+
+pub fn default_release_type() -> String {
+  "stable".to_string()
+}
+
+impl BrowserProfile {
+  /// Get the path to the profile data directory (profiles/{uuid}/profile)
+  pub fn get_profile_data_path(&self, profiles_dir: &Path) -> PathBuf {
+    profiles_dir.join(self.id.to_string()).join("profile")
+  }
+}

src-tauri/src/profile_importer.rs

@@ -680,7 +680,7 @@ impl ProfileImporter {
     // We need to find a suitable version for this browser type
     let available_versions = self.get_default_version_for_browser(browser_type)?;
 
-    let profile = crate::browser_runner::BrowserProfile {
+    let profile = crate::profile::BrowserProfile {
       id: profile_id,
       name: new_profile_name.to_string(),
       browser: browser_type.to_string(),