fix(ui): unlisten Tauri events when unmounted before listen() resolves

[Felix201209]

User description

摘要

修复 3 处 useEffect 里 Tauri 事件监听的清理竞态：组件在异步 listen() 的 promise resolve 之前就卸载时，会泄漏一个永不解绑的监听器。

问题

这几处用的是：

}).then(fn => { if (!cancelled) unlisten = fn; }).catch(() => {});

如果组件在 listen() resolve 前就卸载，cancelled 已经是 true，于是 unlisten 永远不会被赋值——但底层 Tauri 订阅其实已经创建。cleanup 里的 unlisten?.() 是空操作，监听器就此泄漏，并继续在已卸载的组件上触发回调（setMaximized / setState 等），造成无谓开销和 React 警告。

WindowChrome 的 LinuxTitlebar（tauri://resize）最容易复现：在那个动态 import(...) + w.listen() 还在飞行途中时快速重挂载窗口 chrome，就会泄漏一个 resize 监听器。

改动

对齐到仓库自己已有的正确约定（Capsule.tsx / Vocab.tsx / LessComputerPanel.tsx / LessComputerGlow.tsx）：

}).then(fn => { if (cancelled) fn(); else unlisten = fn; }).catch(() => {});

已卸载就立即解绑。涉及：

• components/AudioCue.tsx：prefs:changed 与 capsule:state 两个监听
• components/WindowChrome.tsx：LinuxTitlebar 的 tauri://resize 监听

共 +4/-3，纯前端、行为对齐既有约定、低风险。

验证

• tsc --noEmit 通过。

🤖 Generated with Claude Code

---

PR Type

Bug fix

---

Description

• Prevent Tauri event listener leaks on unmounted components

• Fix race condition: unlisten handle dropped if component unmounts before async listen() resolves

• Align AudioCue.tsx and WindowChrome.tsx with established cleanup pattern

• Low-risk, pure front-end changes

---

Diagram Walkthrough

flowchart LR
    A["Component mounts"] --> B["listen() async"]
    B --> C{"Unmounted before resolve?"}
    C -- Yes --> D["Call unlisten fn immediately"]
    C -- No --> E["Store unlisten handle"]
    D --> F["Cleanup on unmount"]
    E --> F

Loading

File Walkthrough

Relevant files

Bug fix

AudioCue.tsx Fix Tauri listener cleanup in AudioCue                                      openless-all/app/src/components/AudioCue.tsx Fixed two Tauri event listeners (prefs:changed and capsule:state) to immediately unsubscribe if component unmounted before listen() resolves. Changed from if (!cancelled) unlisten = fn; to if (cancelled) fn(); else unlisten = fn;. +2/-2      WindowChrome.tsx Fix Tauri listener cleanup in LinuxTitlebar                            openless-all/app/src/components/WindowChrome.tsx Fixed tauri://resize listener in LinuxTitlebar to unsubscribe immediately if component unmounts before listen() resolves. Changed from if (!cancelled) unlisten = fn; to if (cancelled) fn(); else unlisten = fn;. +2/-1

---

[Copilot]

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

[github-actions]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪

🧪 No relevant tests

🔒 No security concerns identified

⚡ No major issues detected