stability: cap text previews and uniquify imports

Author: yhs0602

app/src/main/java/com/kyhsgeekcode/disassembler/ui/tabs/TextTab.kt

@@ -20,18 +20,44 @@ import kotlinx.coroutines.flow.MutableStateFlow
 import kotlinx.coroutines.flow.StateFlow
 import timber.log.Timber
 
+internal const val MAX_RENDERED_TEXT_BYTES = 256 * 1024
+
+data class TextContentPreview(
+    val bytes: ByteArray,
+    val originalSize: Int,
+    val isTruncated: Boolean
+)
+
+fun buildTextContentPreview(bytes: ByteArray, maxBytes: Int = MAX_RENDERED_TEXT_BYTES): TextContentPreview {
+    if (bytes.size <= maxBytes) {
+        return TextContentPreview(bytes = bytes, originalSize = bytes.size, isTruncated = false)
+    }
+    return TextContentPreview(
+        bytes = bytes.copyOf(maxBytes),
+        originalSize = bytes.size,
+        isTruncated = true
+    )
+}
+
 class TextTabData(val data: TabKind.Text) : PreparedTabData() {
     val relPath = data.key
-    val fileContent = ProjectDataStorage.getFileContent(relPath)
 
     private val _highlighted = MutableStateFlow(AnnotatedString(""))
     val highlighted = _highlighted as StateFlow<AnnotatedString>
+    private val _notice = MutableStateFlow<String?>(null)
+    val notice = _notice as StateFlow<String?>
 
     override suspend fun prepare() {
-        highlightContents()
+        val preview = buildTextContentPreview(ProjectDataStorage.getFileContent(relPath))
+        _notice.value = if (preview.isTruncated) {
+            "Showing first ${preview.bytes.size} bytes of ${preview.originalSize} bytes"
+        } else {
+            null
+        }
+        highlightContents(preview.bytes)
     }
 
-    private suspend fun highlightContents() {
+    private suspend fun highlightContents(fileContent: ByteArray) {
         val ext = ProjectDataStorage.getExtension(relPath) // File(relPath).extension.toLowerCase()
         var highlightedBuilder = AnnotatedString("")
         var strContent: String?
@@ -66,14 +92,18 @@ class TextTabData(val data: TabKind.Text) : PreparedTabData() {
 fun TextTab(data: TabData, viewModel: MainViewModel) {
     val preparedTabData: TextTabData = viewModel.getTabData(data)
     val highlighted = preparedTabData.highlighted.collectAsState()
-    Text(
-        text = highlighted.value,
-        Modifier
-            .background(Color.Black)
-            .verticalScroll(
-                rememberScrollState()
-            )
-    )
+    val notice = preparedTabData.notice.collectAsState()
+    androidx.compose.foundation.layout.Column {
+        notice.value?.let { Text(it) }
+        Text(
+            text = highlighted.value,
+            Modifier
+                .background(Color.Black)
+                .verticalScroll(
+                    rememberScrollState()
+                )
+        )
+    }
 
 //    binding.textFragmentTextView.setBackgroundColor(Color.BLACK)
 }

app/src/main/java/com/kyhsgeekcode/disassembler/viewmodel/MainViewModel.kt

@@ -163,7 +163,10 @@ class MainViewModel(application: Application) : AndroidViewModel(application) {
                 app.contentResolver.openInputStream(uri).use { inStream ->
                     requireNotNull(inStream) { "Failed to open content URI: $uri" }
                     val fileName = resolveImportedFileName(app, uri, displayName)
-                    val file = app.filesDir.resolve("imports").resolve(fileName)
+                    val file = resolveImportedDestinationFile(
+                        app.filesDir.resolve("imports"),
+                        fileName
+                    )
                     file.parentFile?.mkdirs()
                     file.outputStream().use { fileOut ->
                         inStream.copyTo(fileOut)
@@ -375,6 +378,23 @@ internal fun sanitizeImportedFileName(displayName: String?): String {
     return normalized ?: "openDirect"
 }
 
+internal fun resolveImportedDestinationFile(importsDir: File, displayName: String?): File {
+    val sanitizedName = sanitizeImportedFileName(displayName)
+    var candidate = importsDir.resolve(sanitizedName)
+    if (!candidate.exists()) {
+        return candidate
+    }
+
+    val baseName = candidate.nameWithoutExtension.ifBlank { candidate.name }
+    val extensionSuffix = candidate.extension.takeIf { it.isNotBlank() }?.let { ".$it" } ?: ""
+    var index = 1
+    while (candidate.exists()) {
+        candidate = importsDir.resolve("${baseName}_$index$extensionSuffix")
+        index++
+    }
+    return candidate
+}
+
 private fun resolveImportedFileName(
     app: Application,
     uri: Uri,

app/src/test/java/com/kyhsgeekcode/disassembler/ui/tabs/TextPreviewTest.kt

@@ -0,0 +1,26 @@
+package com.kyhsgeekcode.disassembler.ui.tabs
+
+import kotlin.test.Test
+import kotlin.test.assertEquals
+import kotlin.test.assertFalse
+import kotlin.test.assertTrue
+
+class TextPreviewTest {
+    @Test
+    fun `buildTextContentPreview keeps short content unchanged`() {
+        val preview = buildTextContentPreview("hello".encodeToByteArray(), maxBytes = 8)
+
+        assertEquals("hello", preview.bytes.decodeToString())
+        assertFalse(preview.isTruncated)
+        assertEquals(5, preview.originalSize)
+    }
+
+    @Test
+    fun `buildTextContentPreview truncates large content`() {
+        val preview = buildTextContentPreview("abcdefghij".encodeToByteArray(), maxBytes = 4)
+
+        assertEquals("abcd", preview.bytes.decodeToString())
+        assertTrue(preview.isTruncated)
+        assertEquals(10, preview.originalSize)
+    }
+}

app/src/test/java/com/kyhsgeekcode/disassembler/viewmodel/ImportedFileNameTest.kt

@@ -1,6 +1,9 @@
 package com.kyhsgeekcode.disassembler.viewmodel
 
+import java.io.File
+import kotlin.io.path.createTempDirectory
 import org.junit.jupiter.api.Assertions.assertEquals
+import org.junit.jupiter.api.Assertions.assertTrue
 import org.junit.jupiter.api.Test
 
 class ImportedFileNameTest {
@@ -20,4 +23,24 @@ class ImportedFileNameTest {
         assertEquals("openDirect", sanitizeImportedFileName(null))
         assertEquals("openDirect", sanitizeImportedFileName("   "))
     }
+
+    @Test
+    fun `resolveImportedDestinationFile keeps original file name when unused`() {
+        val importsDir = createTempDirectory("imports-test").toFile()
+
+        val destination = resolveImportedDestinationFile(importsDir, "sample.apk")
+
+        assertEquals(File(importsDir, "sample.apk"), destination)
+    }
+
+    @Test
+    fun `resolveImportedDestinationFile appends suffix when file name already exists`() {
+        val importsDir = createTempDirectory("imports-test").toFile()
+        File(importsDir, "sample.apk").writeText("existing")
+
+        val destination = resolveImportedDestinationFile(importsDir, "sample.apk")
+
+        assertEquals("sample_1.apk", destination.name)
+        assertTrue(destination.parentFile == importsDir)
+    }
 }

docs/maintenance/implementation-log.ko.md

@@ -37,6 +37,8 @@
 | 이슈 `#219` 대용량 파일 detail 렌더링 OOM 완화 | binary detail 화면이 전체 문자열을 그대로 레이아웃하지 않도록 preview helper를 추가하고, 화면에는 잘린 preview만 유지하며 전체 내용은 export 시점에만 다시 생성하도록 바꿨다 | 큰 ELF/PE detail 문자열이 화면 레이아웃 단계에서 메모리를 과도하게 잡는 경로를 줄이고, 저장 기능은 유지한 채 UI 메모리 압박을 낮췄다 | `app/src/main/java/com/kyhsgeekcode/disassembler/ui/tabs/BinaryDetailTab.kt`, `app/src/test/java/com/kyhsgeekcode/disassembler/ui/tabs/BinaryDetailsPreviewTest.kt` | 완료 |
 | 이슈 `#219`, `#523` 대용량 파일 확장 cache key OOM 완화 | archive/dex/app expansion cache 경로가 더 이상 파일 전체를 `readBytes()`로 읽지 않고, streaming SHA-256으로 cache key를 계산하도록 바꿨다 | 큰 파일을 확장하기 전에 해시 계산만으로 메모리를 크게 잡아먹던 경계를 제거해서 대용량 import의 선행 OOM 위험을 줄였다 | `app/src/main/java/com/kyhsgeekcode/filechooser/model/ExpandedFileCache.kt`, `app/src/test/java/com/kyhsgeekcode/filechooser/model/ExpandedFileCacheTest.kt` | 완료 |
 | 이슈 `#523` 대용량 Hex 탭 렌더링 완화 | Hex 탭이 매우 큰 파일을 열 때 전체 바이트 배열을 그대로 Compose에 전달하지 않고, 상한이 있는 preview만 렌더링하며 원본 크기 안내 문구를 보여주도록 바꿨다 | 큰 파일을 Hex 화면에 띄우는 순간 UI 레이어가 과도한 메모리와 레이아웃 비용을 쓰는 경로를 줄여서 large-file cluster를 더 좁혔다 | `app/src/main/java/com/kyhsgeekcode/disassembler/ui/components/HexView.kt`, `app/src/main/java/com/kyhsgeekcode/disassembler/ui/tabs/HexTab.kt`, `app/src/test/java/com/kyhsgeekcode/disassembler/ui/components/HexViewLayoutTest.kt` | 완료 |
+| 이슈 `#523` 대용량 Text 탭 렌더링 완화 | Text 탭이 매우 큰 파일을 열 때 전체 바이트를 그대로 highlight하지 않고, 상한이 있는 preview만 decode/highlight하며 원본 크기 안내 문구를 보여주도록 바꿨다 | 큰 텍스트/스몰리/XML 파일을 Text 화면에 띄우는 순간 AnnotatedString과 syntax highlighter가 과도한 메모리를 잡는 경로를 줄였다 | `app/src/main/java/com/kyhsgeekcode/disassembler/ui/tabs/TextTab.kt`, `app/src/test/java/com/kyhsgeekcode/disassembler/ui/tabs/TextPreviewTest.kt` | 완료 |
+| 이슈 `#95` content URI import 파일 충돌 완화 | content URI를 app-private `imports/` 아래로 복사할 때 같은 display name이 이미 있으면 `_1`, `_2` suffix를 붙인 새 파일로 저장하도록 바꿨다 | 같은 이름의 문서를 여러 번 가져오거나 provider가 동일한 표시 이름을 줄 때 기존 import를 조용히 덮어쓰는 문제를 막아 SAF 기반 import 경계를 더 안전하게 만들었다 | `app/src/main/java/com/kyhsgeekcode/disassembler/viewmodel/MainViewModel.kt`, `app/src/test/java/com/kyhsgeekcode/disassembler/viewmodel/ImportedFileNameTest.kt` | 완료 |
 | 이슈 `#129` `.ar` archive 지원 | 공용 archive extractor를 추가하고, Compose/legacy drawer의 archive 확장 경로를 ZIP 전용 구현에서 generic archive extractor로 교체했다 | archive 판정은 되는데 실제 확장은 ZIP만 되던 불일치를 제거해서 `.ar` 같은 지원 가능한 archive도 실제로 탐색 가능하게 만들었다 | `app/src/main/java/com/kyhsgeekcode/Util.kt`, `app/src/main/java/com/kyhsgeekcode/disassembler/ui/FileDrawerTree.kt`, `app/src/main/java/com/kyhsgeekcode/disassembler/FileDrawerListItem.kt`, `app/src/main/java/com/kyhsgeekcode/disassembler/FileDrawerListAdapter.kt`, `app/src/test/java/com/kyhsgeekcode/disassembler/ArchiveExtractionTest.kt` | 완료 |
 | 프로젝트 경로/파일명 회귀 방지 | 프로젝트 상대경로 계산과 import 파일명 정규화를 pure helper로 분리 | 단위 테스트가 가능하도록 로직을 분리하고 경계 케이스를 줄였다 | `app/src/main/java/com/kyhsgeekcode/disassembler/project/ProjectManager.kt`, `app/src/main/java/com/kyhsgeekcode/disassembler/viewmodel/MainViewModel.kt` | 완료 |
 | 회귀 테스트 부재 | `ProjectManager`, 저장소 권한, Hex 레이아웃, import 파일명 테스트 추가 | 최소한의 유지보수 안전망을 확보했다 | `app/src/test/java/com/kyhsgeekcode/disassembler/ProjectManagerTest.kt`, `app/src/test/java/com/kyhsgeekcode/disassembler/PermissionUtilsTest.kt`, `app/src/test/java/com/kyhsgeekcode/disassembler/ui/components/HexViewLayoutTest.kt`, `app/src/test/java/com/kyhsgeekcode/disassembler/viewmodel/ImportedFileNameTest.kt` | 완료 |
@@ -63,6 +65,8 @@
 | binary detail preview 테스트 | 통과 | 긴 detail 문자열은 preview만 렌더링하고 notice를 붙이는 규칙을 고정 |
 | expanded file cache key 테스트 | 통과 | 같은 파일 내용은 같은 streaming hash를 만들고, 전체 `readBytes()` 없이 key를 계산하는 규칙을 고정 |
 | hex preview 테스트 | 통과 | Hex 탭이 큰 파일 바이트 배열을 preview 상한으로 잘라 렌더링하는 규칙을 고정 |
+| text preview 테스트 | 통과 | Text 탭이 큰 파일 바이트 배열을 preview 상한으로 잘라 highlight하는 규칙을 고정 |
+| import destination 파일명 테스트 | 통과 | 같은 display name으로 여러 번 import해도 app-private 파일이 덮어써지지 않는 규칙을 고정 |
 | architecture mapping 테스트 | 통과 | `x86_64`, `PPC64`가 64-bit mode로 매핑되는 규칙을 고정 |
 | binary manual setup reload 테스트 | 통과 | override autosetup 변경 시 disassembly 재로드 필요 여부를 고정 |
 | project export archive 테스트 | 통과 | ZIP 엔트리가 절대경로를 포함하지 않고 `sourceFilePath`, `baseFolder/...`로 묶이는지 확인 |