From 67f99f4a0abe72bdf03385cfe3a39e983efb758d Mon Sep 17 00:00:00 2001
From: ddidderr <ddidderr@paul.network>
Date: Fri, 14 Nov 2025 10:28:55 +0100
Subject: [PATCH] dont do heavy size calc

---
 .../src-tauri/src/lib.rs                      | 44 +++++--------------
 1 file changed, 11 insertions(+), 33 deletions(-)

diff --git a/crates/lanspread-tauri-deno-ts/src-tauri/src/lib.rs b/crates/lanspread-tauri-deno-ts/src-tauri/src/lib.rs
index 9d96d90..eacf9b8 100644
--- a/crates/lanspread-tauri-deno-ts/src-tauri/src/lib.rs
+++ b/crates/lanspread-tauri-deno-ts/src-tauri/src/lib.rs
@@ -21,7 +21,7 @@ struct LanSpreadState {
     games: Arc<RwLock<GameDB>>,
     games_in_download: Arc<RwLock<HashSet<String>>>,
     games_folder: Arc<RwLock<String>>,
-    // Add access to peer game database for size calculations
+    #[allow(dead_code)]
     peer_game_db: Arc<RwLock<PeerGameDB>>,
 }
 
@@ -444,23 +444,7 @@ fn update_game_installation_state(game: &mut Game, games_root: &Path) {
     let installed = downloaded && local_install_is_ready(&game_path);
     game.installed = installed;
 
-    // Calculate size from local files if available (highest priority)
-    if downloaded || installed {
-        match calculate_directory_size_sync(&game_path) {
-            Ok(local_size) => {
-                game.size = local_size;
-                log::debug!(
-                    "Updated size for game {} from local files: {} bytes",
-                    game.id,
-                    local_size
-                );
-            }
-            Err(e) => {
-                log::warn!("Failed to calculate local size for game {}: {e}", game.id);
-                // Keep the existing size (will fallback to peer/game.db later)
-            }
-        }
-    }
+    // Size stays anchored to bundled game.db; skip expensive recalculation.
 
     if installed {
         log::debug!("Set {game} to installed");
@@ -487,7 +471,8 @@ fn update_game_installation_state(game: &mut Game, games_root: &Path) {
     }
 }
 
-/// Synchronous version of `calculate_directory_size` for use in non-async contexts
+/// Left in place for potential re-enablement. Currently not invoked to avoid expensive IO.
+#[allow(dead_code)]
 fn calculate_directory_size_sync(dir: &Path) -> eyre::Result<u64> {
     let mut total_size = 0u64;
 
@@ -504,19 +489,20 @@ fn calculate_directory_size_sync(dir: &Path) -> eyre::Result<u64> {
     Ok(total_size)
 }
 
-/// Calculate total size from a list of file descriptions (used for peer majority calculation)
+/// Used for peer-majority calculations but currently disabled.
+#[allow(dead_code)]
 fn calculate_size_from_file_descriptions(
     file_descriptions: &[lanspread_db::db::GameFileDescription],
 ) -> u64 {
     file_descriptions
         .iter()
-        .filter(|desc| !desc.is_dir) // Only count files, not directories
+        .filter(|desc| !desc.is_dir)
         .map(|desc| desc.size)
         .sum()
 }
 
-/// Update game sizes from peer majority files when local files are not available
-/// This implements the second priority: "we don't have the game but 1-n peers have. Use the majority files for that game, calculate a total size"
+/// Future hook for reintroducing peer-driven size updates.
+#[allow(dead_code)]
 async fn update_game_sizes_from_peers(
     games: &mut std::collections::HashMap<String, Game>,
     peer_game_db: &Arc<RwLock<PeerGameDB>>,
@@ -526,9 +512,7 @@ async fn update_game_sizes_from_peers(
     let peer_db = peer_game_db.read().await;
 
     for game in games.values_mut() {
-        // Only update sizes for games that don't have local files
         if !game.downloaded && !game.installed {
-            // Check if any peers have this game
             let peer_files_for_game = peer_db.aggregated_game_files(&game.id);
 
             if peer_files_for_game.is_empty() {
@@ -536,7 +520,7 @@ async fn update_game_sizes_from_peers(
                     if peer_size > 0 {
                         game.size = peer_size;
                         log::debug!(
-                            "Updated size for game {} from peer-reported totals: {} bytes",
+                            "Updated size for game {} from peer totals: {} bytes",
                             game.id,
                             peer_size
                         );
@@ -548,16 +532,14 @@ async fn update_game_sizes_from_peers(
                     }
                 } else {
                     log::debug!("No peer size data available for game {}", game.id);
-                    // Keep existing size (will fallback to game.db sizes)
                 }
             } else {
-                // Calculate size from peer majority files
                 let peer_size = calculate_size_from_file_descriptions(&peer_files_for_game);
 
                 if peer_size > 0 {
                     game.size = peer_size;
                     log::debug!(
-                        "Updated size for game {} from peer majority files: {} bytes (from {} files)",
+                        "Updated size for game {} from peer files: {} bytes ({} files)",
                         game.id,
                         peer_size,
                         peer_files_for_game.len()
@@ -702,10 +684,6 @@ async fn update_game_db(games: Vec<Game>, app: AppHandle) {
                 );
             }
         }
-
-        // Update game sizes from peer data (second priority)
-        // This will update sizes for games that don't have local files but are available from peers
-        update_game_sizes_from_peers(&mut game_db.games, &state.peer_game_db).await;
     }
 
     refresh_games_list(&app).await;