diff --git a/app/src/main/kotlin/top/yeij/cyrene/ui/sessions/SessionsViewModel.kt b/app/src/main/kotlin/top/yeij/cyrene/ui/sessions/SessionsViewModel.kt index 08c91a0..038f877 100644 --- a/app/src/main/kotlin/top/yeij/cyrene/ui/sessions/SessionsViewModel.kt +++ b/app/src/main/kotlin/top/yeij/cyrene/ui/sessions/SessionsViewModel.kt @@ -89,9 +89,14 @@ class SessionsViewModel(private val container: AppContainer) : ViewModel() { .onSuccess { list -> prefs.saveSnapshot(list, fingerprint()) val cached = prefs.data.first() + // 会话可能在**别处**被删掉(Web 面板 / 别的设备 / 直接调 API),而别处删除无法感知—— + // 不校验 `activeSessionId` 就会一直指着一个已删会话,且这个陈旧 id 会被持久化下去。 + val active = resolveActiveSession(_state.value.activeSessionId, list.map { it.id }) + if (active != _state.value.activeSessionId) prefs.setActive(active) _state.update { it.copy( sessions = list.map { dto -> dto.toItem() }, + activeSessionId = active, loading = false, fromCache = false, syncedAt = cached.snapshotAt, @@ -191,6 +196,23 @@ internal fun SessionSummaryDto.toItem(): SessionItem = SessionItem( lastMessage = lastMessage?.content?.toPreviewText()?.takeIf { it.isNotBlank() }, ) +/** + * 刷新成功后校验「当前会话」是否还在列表里;不在就回落到主会话。 + * + * 什么时候会不在:会话在**别处**被删掉(Web 面板 / 别的设备 / 直接调 API)。 + * 本端自己的删除已经处理过(`deleteSession` 会主动切回主会话),但**别处删除无法感知**—— + * 不校验的话 `activeSessionId` 会一直指着一个已删会话:聊天页对着空会话, + * 而且这个陈旧 id 会被 `prefs.setActive` 持久化下去,重启后依然是它。 + * + * 列表为空时**不回落**:那是异常响应,不该拿它去改用户的选择。 + */ +internal fun resolveActiveSession(activeSessionId: String, fetchedIds: List): String = + if (fetchedIds.isEmpty() || activeSessionId in fetchedIds) { + activeSessionId + } else { + SessionSummaryDto.MAIN_SESSION_ID + } + /** * 列表行标题:优先用服务端的自动标题(首条用户消息生成), * 会话还没有用户消息时 `title` 键整个缺席(或为空白),此时才兜底。 diff --git a/app/src/test/kotlin/top/yeij/cyrene/ui/sessions/SessionItemTest.kt b/app/src/test/kotlin/top/yeij/cyrene/ui/sessions/SessionItemTest.kt index 4369f23..9eb8820 100644 --- a/app/src/test/kotlin/top/yeij/cyrene/ui/sessions/SessionItemTest.kt +++ b/app/src/test/kotlin/top/yeij/cyrene/ui/sessions/SessionItemTest.kt @@ -106,4 +106,22 @@ class SessionItemTest { assertEquals("", " ".toPreviewText()) assertNull(dto(lastMessage = SessionLastMessageDto("assistant", "", 0L)).toItem().lastMessage) } + + @Test + fun `an active session that vanished from the list falls back to main`() { + // 会话在**别处**被删掉(Web 面板 / 别的设备 / 直接调 API)时客户端无法感知—— + // 不回落的话 `activeSessionId` 会一直指着一个已删会话,聊天页对着空会话。 + assertEquals("web", resolveActiveSession("web:gone", listOf("web", "web:alive"))) + } + + @Test + fun `an active session still present is kept`() { + assertEquals("web:alive", resolveActiveSession("web:alive", listOf("web", "web:alive"))) + } + + @Test + fun `an empty list does not override the user's choice`() { + // 空列表是异常响应,不该拿它去改用户的选择。 + assertEquals("web:alive", resolveActiveSession("web:alive", emptyList())) + } }