dsh-plugin-t-expert 0.4.53 → 0.4.57

package/CHANGELOG.md

@@ -16,7 +16,108 @@
 > 发布时**不要再跑 `npm version`**（那会把已提交好的版本又升一级，发出去的号与 CHANGELOG、tag 就对不上），
 > 只补一个 annotated tag（`git tag -a`，因为 `git push --follow-tags` 只推带注释的 tag）。
 
-## [0.4.53] — 未发布
+## [0.4.57] — 未发布
+
+**清理残留的诊断 alert** —— 0.4.55 那次「移除诊断」的字符串替换没匹配上，`alert` 留到了源码里。
+
+- `settings.jsx` 删除 `alert('[头像诊断]…')` 残留（否则每次取头像都会弹窗）。
+- 更新过时注释：头像 effect 的依赖早已改成 `[avatarRev]`，注释却还写着「依赖用 []（只跑一次）」。
+
+### 教训
+
+用字符串替换「删除调试代码」不可靠 —— 匹配串与实际代码对不上时会**静默失败**，
+留下一段每次执行都弹窗的代码。删调试代码后要 `grep` 全库确认，而不是只看替换的返回。
+
+
+
+**点选专家下载完头像后，面板自动显示头像**（之前必须重开面板）。
+
+### 现象
+
+用户实测：点选专家 → 头像确实下载到了本地 → **但面板还是 emoji，要重开面板/重启才看得到**。
+
+### 原因
+
+`warmupAssets()`（点选预热）下载完成时只做了 `refresh(remote, true)` —— **刷新名册**；
+而**头像是在面板/弹窗挂载时单独取一次的**（`getAvatars`）—— 两者互不知情。
+结果名册刷新了、`assets` 状态更新了，但头像那份 state 还是旧的。
+
+### 修复
+
+`warmupAssets` 完成后额外广播 `t-team:avatars-changed`；面板与 `@` 弹窗各加一个监听，
+收到就把 `avatarRev` +1，带动取头像的 effect 重跑。
+
+### 教训
+
+**同一份数据被两处独立取用时，刷新必须一起刷** —— 只刷了名册、漏了头像，
+表现就是「东西明明下好了，界面纹丝不动」，看起来像功能根本没生效。
+
+
+
+**头像显示的真凶：RPC 返回值外面套着 `{ok, value}` 信封，我没解。**
+
+### 根因
+
+DSH 的 remote 调用返回值形如 `{ ok: true, value: 真实数据 }`，**必须用 `unwrap()` 解掉**
+（`src/client/catalog.js:21`，`getCatalog` 等所有既有调用都这么写）。新加的 `getAvatars` 漏了这一步，
+客户端拿到的是**信封本身** —— 诊断时打印 `Object.keys(map)` 得到 `["ok", "value"]`，
+而 `map["novel-generator"]` 永远是 `undefined` → 卡片全部回退 emoji。
+
+**把 `Object.keys(map)` 弹出来这一步是一击定位的关键**：那两个字段名直接暴露了信封结构。
+
+### 修复
+
+`settings.jsx` / `summon.jsx` 改用 `unwrap(await remote?.getAvatars?.(), "getAvatars")`，
+失败兜底成 `{}`；诊断代码移除。
+
+### 这一轮叠在一起的三个坑
+
+1. **`useEffect` 依赖 `[remote]` 不稳定** → effect 反复重挂 → `alive` 恒为 false →
+   `setAvatars` 一次都没执行（真 bug，0.4.54 已修）。
+2. **pnpm 的 `file:` 依赖是「拷贝」不是符号链接** → 改源码 + build 后 `desktop/node_modules`
+   里那份纹丝不动 → **前两次修复根本没到 App**，一直在看旧 client.js。必须
+   `rm -rf node_modules/dsh-plugin-t-expert && pnpm add file:...` 强制重挂
+   （`realpath` 看着像链接，很容易误判成软链 —— 这次就被它骗了一轮）。
+3. **RPC 返回值信封没解**（本条）。
+
+### 附带发现
+
+`~/.t-team/experts` 是 325（目录形态）+ 323（单文件）= **648**，而 `data/experts` 是 300 + 323 = **623** ——
+运行时目录**多 25 位历史残留**（删过但没同步回运行时），面板上的「专家 648」就是这么来的。
+不影响功能，要清需单独处理。
+
+
+
+**修掉「头像数据取到了却显示不出来」的 bug** —— 用户实测：诊断弹窗显示 RPC 返回 2 条，界面却全是 emoji。
+
+### 根因
+
+`settings.jsx` / `summon.jsx` 里取头像的 `useEffect` 依赖写成了 `[remote]`：
+
+```js
+React.useEffect(() => {
+  let alive = true;
+  (async () => { ... if (alive && ...) setAvatars(map); })();
+  return () => { alive = false; };
+}, [remote]);        // ← desktop 上 remote 走 /api 网关，引用不稳定
+```
+
+desktop 的 remote 是经 `/api` API Gateway 构造的，**引用不稳定** → 每次重渲染 effect 都重挂 →
+cleanup 把 `alive` 置回 `false` → **数据取到了也被 `if (alive …)` 丢掉**，state 永远停在空对象。
+
+`alert` 写在 `.then` 里、不看 `alive`，所以照样弹「OK 返回 2 条」—— 这个反差正是定位根因的关键。
+
+### 修复
+
+依赖改 `[]`（只跑一次）。`remote` 在组件生命周期内是稳定的，本来就不需要跟。
+
+### 教训
+
+**用 `alive` 防 setState-on-unmounted 时，依赖数组必须是稳定的** —— 否则它防的不是「卸载后 setState」，
+而是「一切 setState」，而且**完全静默**。另外这次能一步定位，靠的是把被 `catch(() => void 0)`
+吞掉的结果/错误暴露出来：**给静默失败装一个能看见的出口**，比反复猜快得多。
+
+
 
 **面板与 @ 弹窗显示专家头像** —— 0.4.52 只做到数据链路，这一版接上最后一环。
 

package/lib/client.js

@@ -67,7 +67,7 @@ var SectionBoundary = class extends import_react.default.Component {
 
 // src/client/state.js
 var NS = "t-team";
-var CLIENT_BUILD_VERSION = "0.4.53";
+var CLIENT_BUILD_VERSION = "0.4.57";
 var PLUGIN_ID = "dsh-plugin-t-expert";
 var DIVISION_FALLBACK = "specialized";
 var NEW_DIVISION = "__new";
@@ -118,7 +118,13 @@ var warming = /* @__PURE__ */ new Set();
 function warmupAssets(remote, expert) {
   if (expert?.assets !== "missing" || warming.has(expert.slug)) return;
   warming.add(expert.slug);
-  void remote.fetchAssets(expert.slug).then(() => refresh(remote, true)).catch(() => void 0).finally(() => warming.delete(expert.slug));
+  void remote.fetchAssets(expert.slug).then(() => {
+    refresh(remote, true);
+    try {
+      window.dispatchEvent(new CustomEvent("t-team:avatars-changed"));
+    } catch {
+    }
+  }).catch(() => void 0).finally(() => warming.delete(expert.slug));
 }
 function displayDescription(expert, active) {
   return active === "en" ? expert.descriptionEn || expert.description : expert.description;
@@ -21579,6 +21585,7 @@ function SettingsPanel({ t, ctx, remote, skillRemote, getActive, initialTab, vie
   const [snapshot, setSnapshot] = import_react5.default.useState(() => snapshotOf(remote) ?? null);
   const [query, setQuery] = import_react5.default.useState("");
   const [avatars, setAvatars] = import_react5.default.useState({});
+  const [avatarRev, setAvatarRev] = import_react5.default.useState(0);
   const [division, setDivision] = import_react5.default.useState("");
   const [status, setStatus] = import_react5.default.useState("");
   const [error62, setError] = import_react5.default.useState("");
@@ -21610,16 +21617,28 @@ function SettingsPanel({ t, ctx, remote, skillRemote, getActive, initialTab, vie
       alive = false;
     };
   }, [reload]);
+  import_react5.default.useEffect(() => {
+    const onAvatarsChanged = () => setAvatarRev((n) => n + 1);
+    window.addEventListener("t-team:avatars-changed", onAvatarsChanged);
+    return () => window.removeEventListener("t-team:avatars-changed", onAvatarsChanged);
+  }, []);
   import_react5.default.useEffect(() => {
     let alive = true;
     (async () => {
-      const map2 = await Promise.resolve(remote?.getAvatars?.()).catch(() => void 0);
-      if (alive && map2 !== void 0 && map2 !== null) setAvatars(map2);
+      let map2 = {};
+      try {
+        map2 = unwrap(await remote?.getAvatars?.(), "getAvatars") ?? {};
+      } catch {
+        map2 = {};
+      }
+      if (alive && map2 !== void 0 && map2 !== null) {
+        setAvatars(map2);
+      }
     })();
     return () => {
       alive = false;
     };
-  }, [remote]);
+  }, [avatarRev]);
   const active = getActive();
   async function openPrompt(expert) {
     setCopyState("");
@@ -22036,16 +22055,27 @@ function SummonButton({ t, ctx, remote, getActive, insertReference, insertSkillH
   const [popTab, setPopTab] = import_react6.default.useState(preview?.tab ?? "experts");
   const [skills, setSkills] = import_react6.default.useState(null);
   const [avatars, setAvatars] = import_react6.default.useState({});
+  const [avatarRev, setAvatarRev] = import_react6.default.useState(0);
+  import_react6.default.useEffect(() => {
+    const onAvatarsChanged = () => setAvatarRev((n) => n + 1);
+    window.addEventListener("t-team:avatars-changed", onAvatarsChanged);
+    return () => window.removeEventListener("t-team:avatars-changed", onAvatarsChanged);
+  }, []);
   import_react6.default.useEffect(() => {
     let alive = true;
     (async () => {
-      const map2 = await Promise.resolve(remote?.getAvatars?.()).catch(() => void 0);
+      let map2 = {};
+      try {
+        map2 = unwrap(await remote?.getAvatars?.(), "getAvatars") ?? {};
+      } catch {
+        map2 = {};
+      }
       if (alive && map2 !== void 0 && map2 !== null) setAvatars(map2);
     })();
     return () => {
       alive = false;
     };
-  }, [remote]);
+  }, [avatarRev]);
   const [skillQuery, setSkillQuery] = import_react6.default.useState("");
   const [skillsDiag, setSkillsDiag] = import_react6.default.useState("");
   import_react6.default.useEffect(() => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dsh-plugin-t-expert",
-  "version": "0.4.53",
+  "version": "0.4.57",
   "type": "module",
   "description": "T Expert — a 623-expert, 22-division roster with full Chinese translations, as a standalone DeepSeek Harness plugin.",
   "icon": "icon.svg",