From 4b3dc69235342b813b0acfc85853fd8badcc01ec Mon Sep 17 00:00:00 2001
From: Ann Rose <annrose2277@gmail.com>
Date: Wed, 3 Jun 2026 17:12:05 +0530
Subject: [PATCH 1/2] Fix memory leak

---
 .../tv/data/repository/MediaRepository.kt     | 21 ++++++++++++-------
 1 file changed, 13 insertions(+), 8 deletions(-)

diff --git a/app/src/main/kotlin/com/arflix/tv/data/repository/MediaRepository.kt b/app/src/main/kotlin/com/arflix/tv/data/repository/MediaRepository.kt
index 2a5a217f..256d58ed 100644
--- a/app/src/main/kotlin/com/arflix/tv/data/repository/MediaRepository.kt
+++ b/app/src/main/kotlin/com/arflix/tv/data/repository/MediaRepository.kt
@@ -53,6 +53,7 @@ import java.text.SimpleDateFormat
 import java.util.Calendar
 import java.util.Locale
 import java.util.concurrent.ConcurrentHashMap
+import android.util.LruCache
 import com.arflix.tv.util.ParsedCatalogUrl
 import javax.inject.Inject
 import javax.inject.Singleton
@@ -105,21 +106,25 @@ class MediaRepository @Inject constructor(
     private data class CacheEntry<T>(val data: T, val timestamp: Long)
     private val CACHE_TTL_MS = 5 * 60 * 1000L // 5 minutes
 
+    private operator fun <K : Any, V : Any> LruCache<K, V>.set(key: K, value: V) {
+        put(key, value)
+    }
+
     // Home categories cache - survives ViewModel recreation
     @Volatile var cachedHomeCategories: List<Category> = emptyList()
         private set
     @Volatile private var homeCategoriesFetchedAt = 0L
     private val HOME_CATEGORIES_CACHE_MS = 120_000L // 2 minutes
 
-    private val detailsCache = mutableMapOf<String, CacheEntry<MediaItem>>()
+    private val detailsCache = LruCache<String, CacheEntry<MediaItem>>(200)
     private val fullDetailsCacheKeys = mutableSetOf<String>()
-    private val castCache = mutableMapOf<String, CacheEntry<List<CastMember>>>()
-    private val similarCache = mutableMapOf<String, CacheEntry<List<MediaItem>>>()
-    private val logoCache = mutableMapOf<String, CacheEntry<String?>>()
-    private val reviewsCache = mutableMapOf<String, CacheEntry<List<Review>>>()
-    private val watchProvidersCache = mutableMapOf<String, CacheEntry<StreamingServicesResult?>>()
-    private val seasonEpisodesCache = mutableMapOf<String, CacheEntry<List<Episode>>>()
-    private val imdbRatingCache = ConcurrentHashMap<String, CacheEntry<String>>()
+    private val castCache = LruCache<String, CacheEntry<List<Cast>>>(150)
+    private val similarCache = LruCache<String, CacheEntry<List<MediaItem>>>(150)
+    private val logoCache = LruCache<String, CacheEntry<String>>(300)
+    private val reviewsCache = LruCache<String, CacheEntry<List<Review>>>(100)
+    private val watchProvidersCache = LruCache<String, CacheEntry<List<WatchProvider>>>(150)
+    private val seasonEpisodesCache = LruCache<String, CacheEntry<List<Episode>>>(200)
+    private val imdbRatingCache = LruCache<String, CacheEntry<String>>(500)
     private val imdbEpisodeRatingsCache = ConcurrentHashMap<String, CacheEntry<Map<Pair<Int, Int>, String>>>()
     private val imdbRatingsByIdCache = ConcurrentHashMap<String, CacheEntry<String>>()
     private val episodeImdbIdCache = ConcurrentHashMap<String, CacheEntry<String>>()

From e044c9dd0b9b0f09d763839c5c8b0bae63b1c654 Mon Sep 17 00:00:00 2001
From: Ann Rose <annrose2277@gmail.com>
Date: Wed, 3 Jun 2026 18:50:23 +0530
Subject: [PATCH 2/2] fix: replace LruCache with bounded LinkedHashMap to
 resolve build failure

---
 .../tv/data/repository/MediaRepository.kt     | 21 ++++++++++++-------
 1 file changed, 13 insertions(+), 8 deletions(-)

diff --git a/app/src/main/kotlin/com/arflix/tv/data/repository/MediaRepository.kt b/app/src/main/kotlin/com/arflix/tv/data/repository/MediaRepository.kt
index 256d58ed..1d9e45f7 100644
--- a/app/src/main/kotlin/com/arflix/tv/data/repository/MediaRepository.kt
+++ b/app/src/main/kotlin/com/arflix/tv/data/repository/MediaRepository.kt
@@ -116,15 +116,20 @@ class MediaRepository @Inject constructor(
     @Volatile private var homeCategoriesFetchedAt = 0L
     private val HOME_CATEGORIES_CACHE_MS = 120_000L // 2 minutes
 
-    private val detailsCache = LruCache<String, CacheEntry<MediaItem>>(200)
+    private fun <K, V> createBoundedCache(maxSize: Int): MutableMap<K, V> {
+        return object : java.util.LinkedHashMap<K, V>(maxSize, 0.75f, true) {
+            override fun removeEldestEntry(eldest: MutableMap.MutableEntry<K, V>?): Boolean = size > maxSize
+        }
+    }
+    private val detailsCache = createBoundedCache<String, CacheEntry<MediaItem>>(200)
     private val fullDetailsCacheKeys = mutableSetOf<String>()
-    private val castCache = LruCache<String, CacheEntry<List<Cast>>>(150)
-    private val similarCache = LruCache<String, CacheEntry<List<MediaItem>>>(150)
-    private val logoCache = LruCache<String, CacheEntry<String>>(300)
-    private val reviewsCache = LruCache<String, CacheEntry<List<Review>>>(100)
-    private val watchProvidersCache = LruCache<String, CacheEntry<List<WatchProvider>>>(150)
-    private val seasonEpisodesCache = LruCache<String, CacheEntry<List<Episode>>>(200)
-    private val imdbRatingCache = LruCache<String, CacheEntry<String>>(500)
+    private val castCache = createBoundedCache<String, CacheEntry<List<CastMember>>>(150)
+    private val similarCache = createBoundedCache<String, CacheEntry<List<MediaItem>>>(150)
+    private val logoCache = createBoundedCache<String, CacheEntry<String?>>(300)
+    private val reviewsCache = createBoundedCache<String, CacheEntry<List<Review>>>(100)
+    private val watchProvidersCache = createBoundedCache<String, CacheEntry<StreamingServicesResult?>>(150)
+    private val seasonEpisodesCache = createBoundedCache<String, CacheEntry<List<Episode>>>(200)
+    private val imdbRatingCache = java.util.Collections.synchronizedMap(createBoundedCache<String, CacheEntry<String>>(500))
     private val imdbEpisodeRatingsCache = ConcurrentHashMap<String, CacheEntry<Map<Pair<Int, Int>, String>>>()
     private val imdbRatingsByIdCache = ConcurrentHashMap<String, CacheEntry<String>>()
     private val episodeImdbIdCache = ConcurrentHashMap<String, CacheEntry<String>>()