stability: reset workspace state when opening projects

Author: yhs0602

app/src/main/java/com/kyhsgeekcode/disassembler/project/ProjectDataStorage.kt

@@ -146,6 +146,10 @@ object ProjectDataStorage {
         val key = Pair(keykey, DataType.FileContent)
         data[key] = datadata
     }
+
+    fun clear() {
+        data.clear()
+    }
 }
 
 internal const val MAX_CACHED_FILE_CONTENT_BYTES = 8L * 1024 * 1024

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

@@ -48,6 +48,27 @@ import kotlin.collections.HashMap
 import kotlin.math.max
 import kotlin.math.min
 
+internal data class OpenedProjectWorkspaceState(
+    val openedTabs: List<TabData>,
+    val currentTabIndex: Int
+)
+
+internal fun openedProjectWorkspaceState(
+    previousTabs: List<TabData>,
+    previousCurrentTabIndex: Int
+): OpenedProjectWorkspaceState {
+    val defaultTabs = listOf(TabData("Overview", TabKind.ProjectOverview))
+    if (previousCurrentTabIndex == 0 && previousTabs == defaultTabs) {
+        return OpenedProjectWorkspaceState(
+            openedTabs = previousTabs,
+            currentTabIndex = previousCurrentTabIndex
+        )
+    }
+    return OpenedProjectWorkspaceState(
+        openedTabs = defaultTabs,
+        currentTabIndex = 0
+    )
+}
 
 sealed class ShowSearchForStringsDialog {
     object NotShown : ShowSearchForStringsDialog()
@@ -161,8 +182,7 @@ class MainViewModel(application: Application) : AndroidViewModel(application) {
                 val project = withContext(Dispatchers.IO) {
                     ProjectManager.openProject(projectInfoFile.absolutePath)
                 }
-                _selectedFilePath.value = project.sourceFilePath
-                _currentProject.value = project
+                applyOpenedProject(project)
             } catch (e: Exception) {
                 eventChannel.send(Event.AlertError("Failed to open project"))
             }
@@ -177,8 +197,7 @@ class MainViewModel(application: Application) : AndroidViewModel(application) {
                 val project = withContext(Dispatchers.IO) {
                     ProjectManager.import(archiveFile)
                 }
-                _selectedFilePath.value = project.sourceFilePath
-                _currentProject.value = project
+                applyOpenedProject(project)
             } catch (e: Exception) {
                 eventChannel.send(Event.AlertError("Failed to import project"))
             }
@@ -214,8 +233,7 @@ class MainViewModel(application: Application) : AndroidViewModel(application) {
 
                         is ImportedUriAction.ImportProjectArchive -> ProjectManager.import(action.archiveFile)
                     }
-                    _selectedFilePath.value = project.sourceFilePath
-                    _currentProject.value = project
+                    applyOpenedProject(project)
                 }
             } catch (e: Exception) {
                 viewModelScope.launch {
@@ -234,8 +252,7 @@ class MainViewModel(application: Application) : AndroidViewModel(application) {
                 val project = withContext(Dispatchers.IO) {
                     onClickCopyDialog(copy)
                 }
-                _selectedFilePath.value = project.sourceFilePath
-                _currentProject.value = project
+                applyOpenedProject(project)
             } catch (e: Exception) {
                 eventChannel.send(Event.AlertError("Failed to create project"))
             }
@@ -397,6 +414,20 @@ class MainViewModel(application: Application) : AndroidViewModel(application) {
         return openedTabs.value[currentTabIndex.value].title.replaceAfter("as ", with)
     }
 
+    private fun applyOpenedProject(project: ProjectModel) {
+        val workspaceState = openedProjectWorkspaceState(
+            previousTabs = _openedTabs.value,
+            previousCurrentTabIndex = _currentTabIndex.value
+        )
+        ProjectDataStorage.clear()
+        tabDataMap.clear()
+        _openedTabs.value = workspaceState.openedTabs
+        _currentTabIndex.value = workspaceState.currentTabIndex
+        _showSearchForStrings.value = ShowSearchForStringsDialog.NotShown
+        _selectedFilePath.value = project.sourceFilePath
+        _currentProject.value = project
+    }
+
     private fun openNewTab(tabData: TabData) {
         prepareTabData(tabData)
         val newList = ArrayList<TabData>()

app/src/test/java/com/kyhsgeekcode/disassembler/project/ProjectDataStorageCachePolicyTest.kt

@@ -37,4 +37,13 @@ class ProjectDataStorageCachePolicyTest {
 
         assertContentEquals(byteArrayOf(1, 2, 3, 4), readPreviewBytes(file, maxBytes = 4))
     }
+
+    @Test
+    fun `clear removes cached file content`() {
+        ProjectDataStorage.data["sample" to DataType.FileContent] = byteArrayOf(1, 2, 3)
+
+        ProjectDataStorage.clear()
+
+        assertTrue(ProjectDataStorage.data.isEmpty())
+    }
 }

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

@@ -0,0 +1,21 @@
+package com.kyhsgeekcode.disassembler.viewmodel
+
+import com.kyhsgeekcode.disassembler.ui.TabData
+import com.kyhsgeekcode.disassembler.ui.TabKind
+import kotlin.test.Test
+import kotlin.test.assertEquals
+
+class OpenedProjectWorkspaceStateTest {
+    @Test
+    fun `openedProjectWorkspaceState resets tabs to overview`() {
+        val previousTabs = listOf(
+            TabData("Overview", TabKind.ProjectOverview),
+            TabData("classes.dex as Binary", TabKind.Binary("classes.dex"))
+        )
+
+        val state = openedProjectWorkspaceState(previousTabs, previousCurrentTabIndex = 1)
+
+        assertEquals(0, state.currentTabIndex)
+        assertEquals(listOf(TabData("Overview", TabKind.ProjectOverview)), state.openedTabs)
+    }
+}

docs/maintenance/backlog-triage.ko.md

@@ -35,7 +35,7 @@
 
 | 작업 묶음 | 관련 이슈 | 제안 상태 | 판단 | 다음 액션 |
 | --- | --- | --- | --- | --- |
-| 대용량/메모리/RecyclerView 크래시 | `#219`, `#235`, `#442`, `#523` | `planned-fast-follow` | `#728`에서 큰 파일 byte cache 제한과 문자열 검색 결과 상한/stable key를 먼저 넣었다 | `#728` 병합 후 실제 150MB 파일과 긴 문자열 리스트로 재검증하고 나머지 OOM 경로를 분리 |
+| 대용량/메모리/RecyclerView 크래시 | `#219`, `#235`, `#442`, `#523` | `planned-fast-follow` | `#728`에서 큰 파일 byte cache 제한과 문자열 검색 결과 상한/stable key를 먼저 넣었고, `#741`에서 문자열 검색 입력 상한과 프로젝트 전환 시 탭/전역 캐시 리셋을 추가하고 있다 | 실제 150MB 파일과 긴 문자열 리스트로 재검증하고, 살아 있는 경로와 legacy stack trace를 더 분리 |
 | `.so`/ELF/autosetup | `#514`, `#543`, `#576`, `#137` | `planned-fast-follow` | `#728`에서 64-bit ELF machine type 매핑과 override autosetup 재적용 경로를 먼저 수정했다 | 실제 `.so` 샘플로 재검증하고 남는 parser 문제만 분리 |
 | 구형 Android archive 감지 크래시 | `#507`, `#508` | `planned-fast-follow` | `#728`에서 archive 확장자 fast path와 `NoClassDefFoundError` 방어를 넣어 구형 Android의 Commons Compress 감지 크래시 경로를 우회했다 | Android 6~7 계열에서 archive chooser와 open 경로를 재확인하고 정리 |
 | project-relative path assertion 크래시 | `#512` | `planned-fast-follow` | `#728`에서 drawer/tab 경로가 project 바깥 항목을 열려다 `getRelPath()` 예외로 죽지 않도록 null-safe guard를 넣었다 | stale project/drawer entry에서 실패 메시지로만 처리되는지 확인하고 정리 |

docs/maintenance/implementation-log.ko.md

@@ -78,6 +78,8 @@
 | expanded file cache key 테스트 | 통과 | 같은 파일 내용은 같은 streaming hash를 만들고, 전체 `readBytes()` 없이 key를 계산하는 규칙을 고정 |
 | hex preview 테스트 | 통과 | Hex 탭이 큰 파일 바이트 배열을 preview 상한으로 잘라 렌더링하는 규칙을 고정 |
 | text preview 테스트 | 통과 | Text 탭이 큰 파일 바이트 배열을 preview 상한으로 잘라 highlight하는 규칙을 고정 |
+| project workspace reset 테스트 | 통과 | 새 프로젝트를 열면 열린 탭과 현재 탭 인덱스를 `Overview` 하나로 리셋하는 규칙을 고정 |
+| project data cache clear 테스트 | 통과 | 프로젝트 전환 시 전역 파일 content cache를 비우는 규칙을 고정 |
 | import destination 파일명 테스트 | 통과 | 같은 display name으로 여러 번 import해도 app-private 파일이 덮어써지지 않는 규칙을 고정 |
 | imported project relocation 테스트 | 통과 | project archive import 후 `sourceFilePath`, `generatedFolder`, `project_info.json` 경로를 새 프로젝트 위치로 다시 맞추는 규칙을 고정 |
 | project open action 테스트 | 통과 | `Open as project`가 기존 project 디렉터리와 exported project ZIP에 대해 올바른 reopen/import 동작을 고르는 규칙을 고정 |