dsh-plugin-t-expert 0.4.57 → 0.4.63

package/CHANGELOG.md

@@ -16,7 +16,111 @@
 > 发布时**不要再跑 `npm version`**（那会把已提交好的版本又升一级，发出去的号与 CHANGELOG、tag 就对不上），
 > 只补一个 annotated tag（`git tag -a`，因为 `git push --follow-tags` 只推带注释的 tag）。
 
-## [0.4.57] — 未发布
+## [0.4.63] — 未发布
+
+**补 0.4.62 漏写的 `avatarsCache` 声明** —— 上一版只写了「失效」，漏了 `let avatarsCache` 声明与缓存判断，
+导致 `tsc` 报 `Cannot find name 'avatarsCache'`、`npm publish` 的 typecheck 失败。
+
+把 `let avatarsCache` + 命中判断 + 写回补齐，`tsc` 通过。
+
+
+
+**三处性能优化**（审查报告里挑出的高收益项）。
+
+- `settings.jsx`：专家过滤与分组套 `React.useMemo`（600+ 项不再随任意 state 变更全量重算，对齐 summon.jsx）。
+- `lib/index.js` 的 `avatars()`：结果进程内缓存，只有 `fetchAssets` / `fetch_expert_assets` 拉来新头像才失效 ——
+  避免每次打开设置页/浮层都读 281 张 webp 并拼 base64（约 1.3 MB RPC 传输）。
+- `summon.jsx` 的 `toggle()`：改 stale-while-revalidate —— 打开弹窗先用缓存的 snap 立即显示、后台再刷，
+  不再 await 最慢的那次 RPC。
+
+
+
+**修两个代码审查发现的 bug**（逐条读码验证属实后修）。
+
+1. **`readPrefs()` 漏读 `enabledSeedRoster`**（数据正确性）。entry 模式（0.1.7+，无 register 只有
+   mutate）下 `seedDefaultEnabled` 拿到的 `seen` 恒为 `[]`，基线退化成当前 enabled —— 用户停用过、
+   但仍在名册里的专家会被当成「新增」重新写回 enabled。补上 `enabledSeedRoster: unwrapVolatile(...)`。
+2. **`deleteExpert` 未 guard `settingsService`**。settings 未就绪时硬调 `settingsService.mutate` 会抛错
+   打断删除流程（文件已删、enabled 留 ghost 反而无害）。加上 `&& settingsService !== undefined`，
+   与 `pruneStaleEnabled` / `seedDefaultEnabled` 对齐。
+
+verify 补一条源码级回归断言（entry 模式走不到 scope mock，用源码断言防回归），546 → 547。
+
+
+
+**清理 0.4.59 遗留的两处死代码**（代码检查时发现）。
+
+`avatars` 进了 `KEEP_LOCAL` 之后就不再出现在 `dirs` 里，于是：
+
+- `collect()` 里「只装 avatars」那个分支永远不会触发（`dirs.includes("avatars")` 恒为 false）；
+- `isRemote` 里的 `|| p.dirs.includes("avatars")` 同样恒为 false。
+
+两处都删掉，`isRemote` 回到单纯的体积判断。**死分支留着会误导后来人** —— 看着像「头像要上云」，
+实际早就不是了。
+
+复核过的两点：
+
+- `prune()` 按 `p.dirs` 删，而 `avatars` 已被排除在 `dirs` 之外 —— **不会被误删** ✓
+- roster 条目里没有 `avatarsOnly` 这类临时字段残留 ✓
+
+
+
+**头像改为随包发布** —— 放弃云端按需下载那条链路。
+
+### 为什么改
+
+用户实测云端链路始终不稳定：点选/点 ☁️ 不触发下载、下载完要重开面板才刷新，几轮修下来
+仍会「有的能显示、有的不能」。而 **281 张 128px WEBP 加起来总共才 0.94 MB** ——
+直接进包最省事，也永远不会「显示不出来」。**能用简单办法解决的复杂度，就不该留着。**
+
+### 改动
+
+- `pack-assets.mjs`：`avatars` 加入 `KEEP_LOCAL`（与 `.codebuddy-plugin` 同级 —— 绝不上云、随包发布）。
+- `lib/index.js` 的 `assetsOf`：roster 里那些「只装 avatars」的条目**视为本地** ——
+  否则面板会给 105 位专家挂一个点了也没必要的 ☁️。
+- `tools/import-avatars.py` 的收尾提示同步更新。
+
+### 保留的部分
+
+客户端与 host 的那一层**全部保留**（`getAvatars` / `unwrap` / 卡片渲染 / 下载后广播）：
+逻辑不变，只是头像的来源从「云端下载」换成了「包内自带」。云端资源包里的 avatars 存量
+不再维护，下次同步重打包时会自然消失。
+
+代价：npm 包 +0.94 MB。
+
+
+
+**面板的 ☁️ 标识改成可点的下载入口** —— 之前它只是个 `<span>`，注释却写着「点选与召唤都会自动下载」。
+
+### 现象
+
+用户反馈：「部分专家有头像、部分怎么点都没有」。
+
+### 根因
+
+`warmupAssets`（下载预热）只挂在两处：
+
+- `@` 弹窗的 `pick()`（`summon.jsx`）
+- 输入框 `@` 菜单的 `onPick()`（`index.jsx`）
+
+**面板列表（`settings.jsx`）完全没有触发点**。于是规律很清楚：
+
+- **召唤过 / 在 @ 弹窗选过**的专家 → 附件已下载 → 有头像 ✓
+- **只在面板里看过**的专家 → 永远不触发下载 → 永远 emoji ✗
+
+而面板那个 ☁️ 标识本身是个 `<span>`（注释写着「只是个标识、不是按钮」），点它同样没用。
+
+### 修复
+
+把 ☁️ 改成 `<button>`，点击即 `warmupAssets(remote, expert)`；点完立刻收起标识
+（下载在后台跑，不给即时反馈会让人以为没生效）。CSS 加 `.t-team-cloud` 抹掉按钮默认外观。
+
+### 教训
+
+**同一个动作（下载预热）在三个入口里只挂了两处** —— 注释写的是「点选与召唤都会自动下载」，
+但面板那条路径压根没实现。写「A 和 B 都会做 X」这类注释时，得真的去核对 B。
+
+
 
 **清理残留的诊断 alert** —— 0.4.55 那次「移除诊断」的字符串替换没匹配上，`alert` 留到了源码里。