fix(tui): per-file polish round 4 — context/

Author: edy

MERGE.md

@@ -199,6 +199,7 @@ fork 与上游改了同一处（常见于 TUI 视觉/UX、core 加固逻辑）
 | 逐文件打磨轮1：`ui/{spinner,dialog-select,dialog-prompt,dialog-export-options,link,icon,glyphs}.ts(x)` + `{app,app-commands,clipboard,editor,logo,audio}.ts(x)` + `util/{collapse-tool-output,filetype,presentation}.ts` + `config/keybind.ts` | 中 | 保留 bug 修复：spinner 共享 RGBA 常量原地突变改克隆；dialog-select `selectedForeground` 惰值改响应式 + setTimeout 补 onCleanup；collapse-tool-output 负 hiddenCount 防护；app-commands KV 快照改响应式读 + heap snapshot undefined 提示 + isVersionGreater 多连字符预发布；clipboard GNU screen 用平 DCS 透传；editor $EDITOR 引号感知拆分；filetype 支持无扩展名/复合后缀；presentation 复用 logo.ts + 缺 sessionID 略去 Continue 行；删除死代码（icon.tsx 未用组件/映射表、glyphs 死导出、logo.marks、audio.stopVoice、Keybinds 壳、死 onCancel prop、app.tsx console.log）（上游若已做可取上游版本） |
 | 逐文件打磨轮2：`component/{dialog-workspace-create,dialog-console-org,dialog-mcp,dialog-retry-action,dialog-session-rename,dialog-workspace-unavailable,dialog-status,spinner,bg-pulse-render,error-component,todo-item,workspace-label}.ts(x)` | 中 | 保留 bug 修复：dialog-workspace-create 最近工作区按连接状态过滤 + 空 adapters 用 `<Show>` 包裹；dialog-console-org 切换失败弹 error toast；dialog-session-rename 重命名失败弹 error toast；dialog-workspace-unavailable 恢复成功后关闭对话框；dialog-retry-action `selectedForeground` 惰值改响应式；error-component 复制失败重置 Copied 态 + ✖ 改 `GLYPH.cross`；todo-item icon/color/attrs 改派生函数（响应式）；删除死代码（dialog-mcp 未用 setRef、dialog-status 空 Props 类型、spinner SPINNER_FRAMES 别名、workspace-label 未用组件、bg-pulse-render 死导出与未用 cache:false 分支）（上游若已做可取上游版本） |
 | 逐文件打磨轮3：`component/prompt/{autocomplete,index,move,workspace}.tsx` + 删除 `component/prompt/creating-dots.ts`、`component/workspace-label.tsx` | 中 | 保留 bug 修复：autocomplete `<Index>` 行内 8 个 setup 期常量改派生函数（行复用时 label/图标/颜色冻结）+ files resource 源补 `store.visible`（弹窗打开且查询未变时不刷新）；index shell/斜杠命令失败补 error toast（原静默丢输入）+ extmark 样式 ID 改访问器（主题切换后指向已销毁 SyntaxStyle）+ interrupt 双击计数器补 clearTimeout/onCleanup（陈旧定时器清零新计数）+ 内联 reduce 换用 `expandPastedTextPlaceholders` + `basename ?? "image"` 改 `\|\|`；workspace 死 notice 改为成功 warp 弹 success toast（原 showNotice 从未渲染）；删除死代码（Autocomplete sessionID prop、AutocompleteOption.disabled、move creatingDots/pendingNew、workspace label memo、creating-dots.ts 与 workspace-label.tsx 整文件）（上游若已做可取上游版本） |
+| 逐文件打磨轮4：`context/{sync,sdk,project,local,theme,route}.tsx` | 高/中 | 保留 bug 修复：sync `message.removed`/`message.part.removed` 补 `if (!messages/!parts) break`（未加载会话/被驱逐消息触发 `search(undefined)` 抛错杀死事件循环）+ `lsp.updated` 改无条件调用 debounce（默认无 workspace 时 LSP 状态不再刷新的回归）+ 会话列表排序改用与二分查找一致的码位比较（localeCompare 大小写次序不一致）；sdk `onCleanup` 移出 `await` 之后（owner 已失效导致订阅永不清理）+ SSE 重连循环连接/迭代包 try/catch 并在收到事件后重置退避（一次网络错误即永久断连）；project/local 事件订阅补 onCleanup；删除死代码（LocalTheme、RouteContext、SyncContext 解构、theme_mode 只写不读的持久化）（上游若已做可取上游版本） |
 
 ### 其他
 

packages/tui/src/context/local.tsx

@@ -1,6 +1,6 @@
 import { createStore } from "solid-js/store"
 import { createSimpleContext } from "./helper"
-import { batch, createEffect, createMemo } from "solid-js"
+import { batch, createEffect, createMemo, onCleanup } from "solid-js"
 import { useSync } from "./sync"
 import { useEvent } from "./event"
 import path from "path"
@@ -14,16 +14,6 @@ import { useToast } from "../ui/toast"
 import { useRoute } from "./route"
 import { usePermission } from "./permission"
 
-export type LocalTheme = {
-  secondary: RGBA
-  accent: RGBA
-  success: RGBA
-  warning: RGBA
-  primary: RGBA
-  error: RGBA
-  info: RGBA
-}
-
 export function parseModel(model: string) {
   const [providerID, ...rest] = model.split("/")
   return {
@@ -466,9 +456,11 @@ export const { use: useLocal, provider: LocalProvider } = createSimpleContext({
         })
       }
 
-      event.on("session.deleted", (evt) => {
-        prune(evt.properties.info.id)
-      })
+      onCleanup(
+        event.on("session.deleted", (evt) => {
+          prune(evt.properties.info.id)
+        }),
+      )
 
       return {
         get ready() {

packages/tui/src/context/project.tsx

@@ -1,4 +1,4 @@
-import { batch } from "solid-js"
+import { batch, onCleanup } from "solid-js"
 import type { Path, Workspace } from "@opencode-ai/sdk/v2"
 import { createStore, reconcile } from "solid-js/store"
 import { createSimpleContext } from "./helper"
@@ -67,11 +67,13 @@ export const { use: useProject, provider: ProjectProvider } = createSimpleContex
       })
     }
 
-    sdk.event.on("event", (event) => {
-      if (event.payload.type === "workspace.status") {
-        setStore("workspace", "status", event.payload.properties.workspaceID, event.payload.properties.status)
-      }
-    })
+    onCleanup(
+      sdk.event.on("event", (event) => {
+        if (event.payload.type === "workspace.status") {
+          setStore("workspace", "status", event.payload.properties.workspaceID, event.payload.properties.status)
+        }
+      }),
+    )
 
     return {
       data: store,

packages/tui/src/context/route.tsx

@@ -52,8 +52,6 @@ function initialRoute(value: unknown): Route | undefined {
   }
 }
 
-export type RouteContext = ReturnType<typeof useRoute>
-
 export function useRouteData<T extends Route["type"]>(type: T) {
   const route = useRoute()
   return route.data as Extract<Route, { type: typeof type }>

packages/tui/src/context/sdk.tsx

@@ -87,14 +87,19 @@ export const { use: useSDK, provider: SDKProvider } = createSimpleContext({
         while (true) {
           if (abort.signal.aborted || ctrl.signal.aborted) break
 
-          const events = await sdk.global.event({
-            signal: ctrl.signal,
-            sseMaxRetryAttempts: 0,
-          })
-
-          for await (const event of events.stream) {
-            if (ctrl.signal.aborted) break
-            handleEvent(event)
+          try {
+            const events = await sdk.global.event({
+              signal: ctrl.signal,
+              sseMaxRetryAttempts: 0,
+            })
+
+            for await (const event of events.stream) {
+              if (ctrl.signal.aborted) break
+              attempt = 0
+              handleEvent(event)
+            }
+          } catch {
+            // fall through to backoff and reconnect
           }
 
           if (timer) clearTimeout(timer)
@@ -109,13 +114,15 @@ export const { use: useSDK, provider: SDKProvider } = createSimpleContext({
       })().catch(() => {})
     }
 
-    onMount(async () => {
+    onMount(() => {
       if (props.events) {
-        const unsub = await props.events.subscribe(handleEvent)
-        onCleanup(unsub)
-      } else {
-        startSSE()
+        const pending = props.events.subscribe(handleEvent)
+        onCleanup(() => {
+          void pending.then((unsub) => unsub())
+        })
+        return
       }
+      startSSE()
     })
 
     onCleanup(() => {

packages/tui/src/context/sync.tsx

@@ -52,7 +52,6 @@ function search<T>(items: T[], target: string, key: (item: T) => string) {
 }
 
 export const {
-  context: SyncContext,
   use: useSync,
   provider: SyncProvider,
 } = createSimpleContext({
@@ -145,7 +144,7 @@ export const {
     const syncingSessions = new Map<string, Promise<void>>()
     const hydratingSessions = new Map<string, { messages: Set<string>; parts: Set<string> }>()
     let lspStatusTimer: ReturnType<typeof setTimeout> | undefined
-    const debouncedLspStatus = (workspace: string) => {
+    const debouncedLspStatus = (workspace: string | undefined) => {
       if (lspStatusTimer) clearTimeout(lspStatusTimer)
       lspStatusTimer = setTimeout(() => {
         void sdk.client.lsp.status({ workspace }).then(
@@ -177,7 +176,7 @@ export const {
     function listSessions() {
       return sdk.client.session
         .list({ start: Date.now() - 30 * 24 * 60 * 60 * 1000, ...sessionListQuery() })
-        .then((x) => (x.data ?? []).toSorted((a, b) => a.id.localeCompare(b.id)))
+        .then((x) => (x.data ?? []).toSorted((a, b) => (a.id < b.id ? -1 : a.id > b.id ? 1 : 0)))
     }
 
     const unsubscribe = event.subscribe((event, { directory, workspace }) => {
@@ -368,6 +367,7 @@ export const {
         case "message.removed": {
           touchMessage(event.properties.sessionID, event.properties.messageID)
           const messages = store.message[event.properties.sessionID]
+          if (!messages) break
           const result = search(messages, event.properties.messageID, (m) => m.id)
           if (result.found) {
             setStore(
@@ -424,6 +424,7 @@ export const {
         case "message.part.removed": {
           touchPart(event.properties.sessionID, event.properties.partID)
           const parts = store.part[event.properties.messageID]
+          if (!parts) break
           const result = search(parts, event.properties.partID, (p) => p.id)
           if (result.found) {
             setStore(
@@ -438,8 +439,7 @@ export const {
         }
 
         case "lsp.updated": {
-          const workspace = project.workspace.current()
-          if (workspace) debouncedLspStatus(workspace)
+          debouncedLspStatus(project.workspace.current())
           break
         }
 

packages/tui/src/context/theme.tsx

@@ -200,7 +200,6 @@ export const { use: useTheme, provider: ThemeProvider } = createSimpleContext({
     }
 
     function apply(mode: "dark" | "light") {
-      if (store.lock !== undefined) kv.set("theme_mode", mode)
       if (store.mode === mode) return
       setStore("mode", mode)
       refreshSystemTheme(mode)
@@ -215,7 +214,6 @@ export const { use: useTheme, provider: ThemeProvider } = createSimpleContext({
     function free() {
       setStore("lock", undefined)
       kv.set("theme_mode_lock", undefined)
-      kv.set("theme_mode", undefined)
       refreshSystemTheme(renderer.themeMode ?? store.mode)
     }