Add LRU cache for content

Author: SimonSchubert

composeApp/src/androidMain/kotlin/com/linuxcommandlibrary/app/platform/AndroidAssetReader.kt

@@ -13,7 +13,7 @@ class AndroidAssetReader(private val context: Context) : AssetReader {
     }
 
     override fun readFile(path: String): String? = try {
-        context.assets.open(path).bufferedReader().readText()
+        context.assets.open(path).bufferedReader().use { it.readText() }
     } catch (e: Exception) {
         null
     }

composeApp/src/commonMain/kotlin/com/linuxcommandlibrary/app/data/BasicsRepository.kt

@@ -7,10 +7,15 @@ import com.linuxcommandlibrary.shared.platform.AssetReader
 
 class BasicsRepository(private val assetReader: AssetReader) {
 
+    private var cachedCategories: List<BasicCategory>? = null
+    private val cachedGroupsAndCommands = mutableMapOf<String, Pair<List<BasicGroup>, Map<Long, List<BasicCommand>>>>()
+
     fun getCategories(): List<BasicCategory> {
+        cachedCategories?.let { return it }
+
         val files = assetReader.listFiles("basics")
 
-        return files
+        val categories = files
             .filter { it.endsWith(".md") }
             .mapNotNull { filename ->
                 val id = filename.removeSuffix(".md")
@@ -22,6 +27,9 @@ class BasicsRepository(private val assetReader: AssetReader) {
                 }
             }
             .sortedBy { basicsSortOrder.indexOf(it.title) }
+
+        cachedCategories = categories
+        return categories
     }
 
     private fun readCategoryTitle(filename: String): String? = try {
@@ -32,6 +40,8 @@ class BasicsRepository(private val assetReader: AssetReader) {
     }
 
     fun getGroupsAndCommands(categoryId: String): Pair<List<BasicGroup>, Map<Long, List<BasicCommand>>> {
+        cachedGroupsAndCommands[categoryId]?.let { return it }
+
         val groups = mutableListOf<BasicGroup>()
         val commandsByGroupId = mutableMapOf<Long, MutableList<BasicCommand>>()
 
@@ -63,7 +73,9 @@ class BasicsRepository(private val assetReader: AssetReader) {
             // Return empty on error
         }
 
-        return Pair(groups, commandsByGroupId)
+        val result = Pair<List<BasicGroup>, Map<Long, List<BasicCommand>>>(groups, commandsByGroupId)
+        cachedGroupsAndCommands[categoryId] = result
+        return result
     }
 
     fun getBasicInfo(categoryId: String): BasicInfo? = try {

composeApp/src/commonMain/kotlin/com/linuxcommandlibrary/app/data/CommandsRepository.kt

@@ -8,6 +8,9 @@ class CommandsRepository(private val assetReader: AssetReader) {
     private var cachedCommands: List<CommandInfo>? = null
     private var cachedCommandNames: Set<String>? = null
 
+    private val sectionsCache = linkedMapOf<String, List<CommandSectionInfo>>()
+    private val sectionsCacheMaxSize = 50
+
     fun getCommands(): List<CommandInfo> {
         cachedCommands?.let { return it }
 
@@ -51,18 +54,31 @@ class CommandsRepository(private val assetReader: AssetReader) {
         return name in names
     }
 
-    fun getSections(commandName: String): List<CommandSectionInfo> = try {
-        val content = assetReader.readFile("commands/$commandName.md") ?: return emptyList()
+    fun getSections(commandName: String): List<CommandSectionInfo> {
+        sectionsCache.remove(commandName)?.let {
+            sectionsCache[commandName] = it
+            return it
+        }
 
-        MarkdownParser.splitByHeaders(content, "# ").map { (title, sectionContent) ->
-            CommandSectionInfo(
-                id = (commandName + title).hashCode().toLong(),
-                title = title,
-                content = sectionContent,
-            )
-        }.filter { it.title.uppercase() != "TAGLINE" }
-            .sortedBy { it.getSortPriority() }
-    } catch (e: Exception) {
-        emptyList()
+        val sections = try {
+            val content = assetReader.readFile("commands/$commandName.md") ?: return emptyList()
+
+            MarkdownParser.splitByHeaders(content, "# ").map { (title, sectionContent) ->
+                CommandSectionInfo(
+                    id = (commandName + title).hashCode().toLong(),
+                    title = title,
+                    content = sectionContent,
+                )
+            }.filter { it.title.uppercase() != "TAGLINE" }
+                .sortedBy { it.getSortPriority() }
+        } catch (e: Exception) {
+            emptyList()
+        }
+
+        sectionsCache[commandName] = sections
+        if (sectionsCache.size > sectionsCacheMaxSize) {
+            sectionsCache.remove(sectionsCache.keys.first())
+        }
+        return sections
     }
 }