修复任务栏歌词评审问题

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: imsyy

.github/copilot-instructions.md

@@ -0,0 +1,66 @@
+# SPlayer-Next Copilot 指引
+
+本仓库所有 Copilot 会话、Code Review、建议，**一律使用中文作答**（代码、标识符、注释保持原项目语言）。
+
+## 项目概述
+
+SPlayer-Next 是基于 **Electron + Vue 3 + TypeScript** 的桌面音乐播放器，后继 SPlayer，搭配 **Rust NAPI-RS** 原生模块做音频解码、系统媒体集成、任务栏集成。
+
+### 进程架构
+
+- **主进程** `electron/main/`：窗口管理、IPC、原生模块调度
+- **Preload** `electron/preload/`：通过 contextBridge 暴露 `window.api`
+- **渲染进程** `src/`：Vue 3 SPA；另有三个独立窗口 `windows/desktop-lyric`、`windows/dynamic-island`、`windows/taskbar-lyric`
+
+### 原生模块（`native/`）
+
+- `audio-engine`：FFmpeg 解码 + rodio 播放 + FFT + 封面提取
+- `media-ctrl`：跨平台系统媒体控件（Windows SMTC / Linux MPRIS / macOS MPNowPlaying）+ Discord RPC
+- `taskbar-lyric`：Windows 专属，把窗口嵌入任务栏 + RegistryWatcher / UiaWatcher / TrayWatcher 四路监听
+
+所有原生模块通过 `loadNativeModule()`（`electron/main/utils/nativeLoader.ts`）懒加载。
+
+### 路径别名
+
+| 别名 | 指向 | 使用范围 |
+|---|---|---|
+| `@/*` | `src/*` | 渲染进程 |
+| `@shared/*` | `shared/*` | 主进程 + 渲染进程 + 原生类型 |
+| `@main/*` | `electron/main/*` | 主进程 |
+| `@server/*` | `electron/server/*` | 主进程 |
+| `@windows/*` | `windows/*` | 三个独立歌词窗口（复用 `windows/shared/`） |
+| `@splayer/audio-engine` | `native/audio-engine` | 主进程，类型从自动生成的 `index.d.ts` 导入 |
+| `@splayer/media-ctrl` | `native/media-ctrl` | 同上 |
+| `@splayer/taskbar-lyric` | `native/taskbar-lyric` | 同上 |
+
+## 代码约定
+
+- **语言**：注释、Git commit、PR 描述一律中文
+- **时间单位**：渲染端所有时间值是**毫秒**；Rust 引擎内部用秒，转换在 `electron/main/ipc/player.ts` 里做（`toMs()`）
+- **Prettier**：双引号、分号、100 字符宽、末尾逗号
+- **Auto-imports**（无需手写 `import`）：`vue`、`pinia`、`vue-router`、`@vueuse/core`、`vue-i18n`；UI 组件自动从 `src/components/` 注入
+- **原生模块类型**：**禁止手写**，必须从 `@splayer/<module>` 的自动生成 `index.d.ts` 导入
+- **状态管理双层**：`src/stores/status.ts` 走 Pinia 响应式（5Hz 更新）；`src/services/playback.ts` 存非响应式时间源，供 60fps 渲染（歌词 / 频谱）用 RAF 采样
+- **Reactivity**：`Track[]` 等大集合用 `shallowRef`，避免深层代理；响应式代理对象存 IndexedDB 会 `DataCloneError`
+- **Store 持久化**：只把轻量数据写 sessionStorage；`TrackDetail`（含歌词大字符串）**禁止**持久化
+- **IPC 监听器清理**：preload 的 `onEvent` 必须先 `removeAllListeners` 再 `on`，防 HMR 累积；渲染端 composable 在 `onBeforeUnmount` 调用 preload 返回的 `unsubscribe`
+- **自动生成文件**（不要编辑）：`auto-imports.d.ts`、`components.d.ts`、`native/*/index.d.ts`
+- **Logger**：主进程统一用 `@main/utils/logger` 的 scoped logger（`coreLog` / `playerLog` / `mediaLog` / `trayLog` / `taskbarLog` / `nativeLog` 等），**不要**直接 `import log from "electron-log"`
+
+## 窗口侧（`windows/`）
+
+三个歌词窗口都复用：
+
+- `@windows/shared/composables/useNowPlayingSync` 提供播放同步、歌词索引、锚点插值
+- `getNowPlayingCurrentMs()` 非响应式读当前时间，供逐字高亮 RAF 循环用
+- 歌词行选择算法按窗口语义选：桌面歌词用 `pickPrimaryIndex`（考虑 overlap）、灵动岛用 `pickLatestStartedIndex`（立即切换）
+
+不要在窗口内重新实现上述逻辑，直接调用共享 composable。
+
+## 设置 schema
+
+- 结构定义在 `src/settings/schema.ts`
+- 类型在 `src/types/settings-schema.ts`：`SettingCategory` → `SettingSection` → `SettingItem`
+- `SettingSection.tag` / `SettingItem.tag` 支持 `SettingTag = { text: string; type?: "primary" | "warning" | ... }`，用于标题旁的徽标（Beta / 实验等）
+- i18n key 约定：`settings.section.{id}` / `settings.{itemKey}.label` / `settings.{itemKey}.description`
+- UI 原子组件：`STag` / `SButton` / `SSelect` / `SSlider` / `SSwitch` / `SColor` / `SDialog` 等，统一 7 色 theme token（default / primary / cover / info / success / warning / error）

.github/instructions/review.instructions.md

@@ -0,0 +1,34 @@
+---
+applyTo: "**"
+---
+
+# 审查指引
+
+所有审查评论、建议、总结一律**用中文**输出。
+
+## 审查关注点
+
+**真实问题优先，风格偏好最后**。优先揪出：
+
+- 运行时 bug、竞态、内存 / 资源泄漏（IPC 监听器、RAF、ResizeObserver、Windows 事件钩子未清理）
+- 类型与实现不一致（如 preload API 类型声明与实际 resolve 值不匹配）
+- 主进程资源生命周期：窗口 `closed` 事件之外的重复清理会引发双 `stop()` / 双 broadcast
+- 硬编码值掩盖用户设置：检查配置项是否真的生效到底层模块
+- 原生模块 COM 生命周期（`CoInitializeEx` 的返回值必须判断，`CoUninitialize` 只在真正初始化成功时调用；参考 `native/taskbar-lyric/src/uia.rs` 的 `should_uninitialize` 模式）
+- 锁的持有时长：回调执行前应先 clone `Arc` 放开 `Mutex`，避免阻塞其它访问与重入死锁
+- 渲染端持久化：`TrackDetail` / 大歌词字符串不应进 sessionStorage
+- 不同进程对 `@shared/*` 类型的使用是否一致
+
+## 避免的建议
+
+- 不要建议改 Prettier 风格（双引号 / 分号 / 100 列 / 末尾逗号已锁定）
+- 不要建议手写原生模块类型，必须从 `@splayer/<module>` 导入
+- 不要建议在渲染端直接 `import log from "electron-log"` 或在主进程绕过 `@main/utils/logger`
+- 不要建议为"防御性编程"添加内部代码的运行时校验（只在系统边界校验）
+- 不要建议拆分三行的类似代码为抽象，除非有三个以上真实用例
+
+## 评论风格
+
+- 直接指出问题所在文件 / 行号 / 变量名，配可执行的修法
+- 区分「必修」「建议」「风格」，便于作者决定优先级
+- 若和项目现有模式有冲突，**先查 `CLAUDE.md` 和 `.github/copilot-instructions.md`** 确认约定再提

electron/main/window/taskbarLyric.ts

@@ -38,8 +38,11 @@ let themeRegWatcher: RegistryWatcher | null = null;
 let uiaWatcher: UiaWatcher | null = null;
 let trayWatcher: TrayWatcher | null = null;
 
-/** 传给 Rust 的 lyric_width：Win10 据此从 tasklist 划空间，Win11 忽略 */
-const currentWidth = 300;
+/** 从设置读取当前歌词宽度（Win10 据此从 tasklist 划空间，Win11 忽略） */
+const resolveLyricWidth = (): number => {
+  const width = store.get("taskbarLyric.maxWidth");
+  return typeof width === "number" && width > 0 ? width : 400;
+};
 
 /**
  * 初始窗口尺寸——故意设大，覆盖任何可能的任务栏宽度/高度。
@@ -120,7 +123,7 @@ const applyLayout = (layout: JsTaskbarLayout): void => {
 
 /** Watcher 回调——任何任务栏相关变化都回到这里重算布局 */
 const onLayoutChange = (): void => {
-  service?.update(currentWidth);
+  service?.update(resolveLyricWidth());
 };
 
 /** 安全创建原生 watcher，失败只 warn 不中断启动 */
@@ -198,7 +201,7 @@ export const createTaskbarLyricWindow = (): BrowserWindow | null => {
     const hwndPtr = Number(win.getNativeWindowHandle().readBigUInt64LE(0));
     taskbarLog.info(`嵌入窗口 hwnd=${hwndPtr}`);
     svc.embedWindowByPtr(hwndPtr);
-    svc.update(currentWidth);
+    svc.update(resolveLyricWidth());
 
     advancedRegWatcher = tryStart(
       "RegistryWatcher(Advanced)",
@@ -239,27 +242,24 @@ const cleanupWatchers = (): void => {
   service = null;
 };
 
-/** 关闭任务栏歌词窗口并清理所有资源 */
+/** 请求关闭任务栏歌词窗口，实际清理由 "closed" 事件统一处理 */
 export const closeTaskbarLyricWindow = (): void => {
   if (taskbarLyricWindow && !taskbarLyricWindow.isDestroyed()) {
     taskbarLyricWindow.close();
   }
-  taskbarLyricWindow = null;
-  cleanupWatchers();
-  setTrayTaskbarLyric(false);
-  broadcast("taskbarLyric:visibilityChange", false);
 };
 
-/** 切换任务栏歌词窗口显隐 */
-export const toggleTaskbarLyricWindow = (): void => {
+/** 切换任务栏歌词窗口显隐，返回切换后是否打开 */
+export const toggleTaskbarLyricWindow = (): boolean => {
   if (taskbarLyricWindow && !taskbarLyricWindow.isDestroyed()) {
     closeTaskbarLyricWindow();
-  } else {
-    createTaskbarLyricWindow();
+    return false;
   }
+  createTaskbarLyricWindow();
+  return true;
 };
 
-/** 触发一次布局重算 */
+/** 触发一次布局重算（配置变更后调用） */
 export const applyTaskbarLyricLayout = (): void => {
-  service?.update(currentWidth);
+  service?.update(resolveLyricWidth());
 };

native/taskbar-lyric/src/tray_watcher.rs

@@ -37,11 +37,15 @@ unsafe extern "system" fn win_event_proc(
         let len = unsafe { GetClassNameW(hwnd, &mut buffer) };
         if len > 0 {
             let name = String::from_utf16_lossy(&buffer[..len as usize]);
-            if name == "TrayNotifyWnd"
-                && let Ok(guard) = GLOBAL_CALLBACK.lock()
-                && let Some(cb) = guard.as_ref()
-            {
-                cb();
+            if name == "TrayNotifyWnd" {
+                // 先 clone Arc 再释放锁，避免回调执行期间阻塞其它 GLOBAL_CALLBACK 访问
+                let callback = GLOBAL_CALLBACK
+                    .lock()
+                    .ok()
+                    .and_then(|guard| guard.as_ref().cloned());
+                if let Some(cb) = callback {
+                    cb();
+                }
             }
         }
     }

native/taskbar-lyric/src/uia_watcher.rs

@@ -96,7 +96,9 @@ impl UiaWatcher {
         let callback_arc = Arc::new(callback);
 
         thread::spawn(move || unsafe {
-            let _ = CoInitializeEx(None, COINIT_MULTITHREADED);
+            let hr = CoInitializeEx(None, COINIT_MULTITHREADED);
+            // S_OK / S_FALSE 需要配对 CoUninitialize；RPC_E_CHANGED_MODE 或其它错误则不应调用
+            let should_uninitialize = hr.is_ok();
 
             let thread_id = GetCurrentThreadId();
             let _ = tx.send(thread_id);
@@ -145,7 +147,9 @@ impl UiaWatcher {
             }
 
             drop(_handlers_guard);
-            CoUninitialize();
+            if should_uninitialize {
+                CoUninitialize();
+            }
         });
 
         let thread_id = rx.recv().map_err(|e| anyhow!("获取线程 ID 失败: {e}"))?;