stability: bound large hex previews

Author: yhs0602

app/src/main/java/com/kyhsgeekcode/disassembler/ui/components/HexView.kt

@@ -28,6 +28,24 @@ internal const val BYTES_PER_ROW = 16
 private val OFFSET_WIDTH = 90.dp
 private val HEX_CELL_WIDTH = 25.dp
 private val ASCII_CELL_WIDTH = 12.dp
+internal const val MAX_RENDERED_HEX_BYTES = 256 * 1024
+
+data class HexPreview(
+    val bytes: ByteArray,
+    val originalSize: Int,
+    val isTruncated: Boolean
+)
+
+fun buildHexPreview(bytes: ByteArray, maxBytes: Int = MAX_RENDERED_HEX_BYTES): HexPreview {
+    if (bytes.size <= maxBytes) {
+        return HexPreview(bytes = bytes, originalSize = bytes.size, isTruncated = false)
+    }
+    return HexPreview(
+        bytes = bytes.copyOf(maxBytes),
+        originalSize = bytes.size,
+        isTruncated = true
+    )
+}
 
 data class HexRow(
     val offset: Int,

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

@@ -1,23 +1,26 @@
 package com.kyhsgeekcode.disassembler.ui.tabs
 
 import androidx.compose.foundation.ExperimentalFoundationApi
+import androidx.compose.foundation.layout.Column
 import androidx.compose.runtime.Composable
 import androidx.compose.runtime.collectAsState
 import com.kyhsgeekcode.disassembler.project.ProjectDataStorage
 import com.kyhsgeekcode.disassembler.ui.TabData
 import com.kyhsgeekcode.disassembler.ui.TabKind
 import com.kyhsgeekcode.disassembler.ui.components.HexView
+import com.kyhsgeekcode.disassembler.ui.components.buildHexPreview
 import com.kyhsgeekcode.disassembler.viewmodel.MainViewModel
+import androidx.compose.material3.Text
 import kotlinx.coroutines.flow.MutableStateFlow
 import kotlinx.coroutines.flow.StateFlow
 
 
 class HexTabData(val data: TabKind.Hex) : PreparedTabData() {
-    private val _bytes = MutableStateFlow(byteArrayOf())
-    val bytes = _bytes as StateFlow<ByteArray>
+    private val _preview = MutableStateFlow(buildHexPreview(byteArrayOf()))
+    val preview = _preview as StateFlow<com.kyhsgeekcode.disassembler.ui.components.HexPreview>
 
     override suspend fun prepare() {
-        _bytes.value = ProjectDataStorage.getFileContent(data.relPath)
+        _preview.value = buildHexPreview(ProjectDataStorage.getFileContent(data.relPath))
     }
 }
 
@@ -26,6 +29,11 @@ class HexTabData(val data: TabKind.Hex) : PreparedTabData() {
 @Composable
 fun HexTab(data: TabData, viewModel: MainViewModel) {
     val preparedTabData: HexTabData = viewModel.getTabData(data)
-    val bytes = preparedTabData.bytes.collectAsState()
-    HexView(bytes = bytes.value)
-}
+    val preview = preparedTabData.preview.collectAsState()
+    Column {
+        if (preview.value.isTruncated) {
+            Text("Showing first ${preview.value.bytes.size} bytes of ${preview.value.originalSize} bytes")
+        }
+        HexView(bytes = preview.value.bytes)
+    }
+}

app/src/test/java/com/kyhsgeekcode/disassembler/ui/components/HexViewLayoutTest.kt

@@ -6,6 +6,23 @@ import org.junit.jupiter.api.Assertions.assertTrue
 import org.junit.jupiter.api.Test
 
 class HexViewLayoutTest {
+    @Test
+    fun `buildHexPreview keeps short byte arrays unchanged`() {
+        val preview = buildHexPreview(byteArrayOf(0x01, 0x02, 0x03), maxBytes = 4)
+
+        assertEquals(byteArrayOf(0x01, 0x02, 0x03).toList(), preview.bytes.toList())
+        assertFalse(preview.isTruncated)
+    }
+
+    @Test
+    fun `buildHexPreview truncates long byte arrays`() {
+        val preview = buildHexPreview(ByteArray(6) { it.toByte() }, maxBytes = 4)
+
+        assertEquals(byteArrayOf(0x00, 0x01, 0x02, 0x03).toList(), preview.bytes.toList())
+        assertTrue(preview.isTruncated)
+        assertEquals(6, preview.originalSize)
+    }
+
     @Test
     fun `buildHexRows splits bytes into 16-byte rows with offsets`() {
         val input = ByteArray(18) { it.toByte() }

docs/maintenance/implementation-log.ko.md

@@ -36,6 +36,7 @@
 | 이슈 `#123`, `#720` 프로젝트 export와 details 저장 부재 | ZIP 생성기를 재귀 상대경로 방식으로 고치고, 프로젝트 overview에 SAF `CreateDocument` 기반 `Export project` 버튼을 추가했으며, binary detail 탭에는 `Save Details to file` 버튼을 구현했다 | 깨져 있던 project ZIP export 경로를 실제로 복구하고, 최신 Android 저장소 모델에 맞게 상세 정보 텍스트를 `.txt` 문서로 저장할 수 있게 했다 | `app/src/main/java/com/kyhsgeekcode/Util.kt`, `app/src/main/java/com/kyhsgeekcode/disassembler/exporting/ExportDocuments.kt`, `app/src/main/java/com/kyhsgeekcode/disassembler/project/ProjectManager.kt`, `app/src/main/java/com/kyhsgeekcode/disassembler/viewmodel/MainViewModel.kt`, `app/src/main/java/com/kyhsgeekcode/disassembler/ui/MainTab.kt`, `app/src/main/java/com/kyhsgeekcode/disassembler/ui/tabs/BinaryDetailTab.kt`, `app/src/test/java/com/kyhsgeekcode/disassembler/ProjectExportArchiveTest.kt`, `app/src/test/java/com/kyhsgeekcode/disassembler/exporting/BinaryDetailsExportTest.kt` | 완료 |
 | 이슈 `#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` | 완료 |
 | 이슈 `#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` | 완료 |
@@ -61,6 +62,7 @@
 | string search accumulator 테스트 | 통과 | 문자열 결과 상한과 truncation 동작을 고정 |
 | binary detail preview 테스트 | 통과 | 긴 detail 문자열은 preview만 렌더링하고 notice를 붙이는 규칙을 고정 |
 | expanded file cache key 테스트 | 통과 | 같은 파일 내용은 같은 streaming hash를 만들고, 전체 `readBytes()` 없이 key를 계산하는 규칙을 고정 |
+| hex preview 테스트 | 통과 | Hex 탭이 큰 파일 바이트 배열을 preview 상한으로 잘라 렌더링하는 규칙을 고정 |
 | architecture mapping 테스트 | 통과 | `x86_64`, `PPC64`가 64-bit mode로 매핑되는 규칙을 고정 |
 | binary manual setup reload 테스트 | 통과 | override autosetup 변경 시 disassembly 재로드 필요 여부를 고정 |
 | project export archive 테스트 | 통과 | ZIP 엔트리가 절대경로를 포함하지 않고 `sourceFilePath`, `baseFolder/...`로 묶이는지 확인 |