Merge pull request #1899 from faraz152/fix/telegram-cache-timsort-contract

fix: snapshot lastModified before sort in TelegramCacheManager

Author: theovilardo

app/src/main/java/com/theveloper/pixelplay/data/telegram/TelegramCacheManager.kt

@@ -189,8 +189,13 @@ class TelegramCacheManager @Inject constructor(
                     return@launch // Within limits
                 }
 
-                // Sort by last modified (oldest first) for LRU eviction
-                embeddedArtFiles.sortBy { it.lastModified() }
+                // Sort by last modified (oldest first) for LRU eviction.
+                // Snapshot timestamps before sorting — reading lastModified() inside the
+                // comparator on every pairwise call lets another coroutine's setLastModified()
+                // change the value mid-sort, violating TimSort's comparator contract and
+                // throwing "Comparison method violates its general contract!".
+                val snapshots = embeddedArtFiles.associateWith { it.lastModified() }
+                embeddedArtFiles.sortWith(compareBy { snapshots[it] })
 
                 var deletedCount = 0
                 for (file in embeddedArtFiles) {