fix: PR #779 review polish — 全量清理 P2/P3 finding

[hrygo]

spec ⑦ Phase 3 PR #779 review polish — 清理三轮 review 的非阻塞 P2/P3 finding（13 项去重：11 项修，1 项开 #782，1 项接受）。

重建自 #783（作者误为 hotplex-ai 致 self-approve 限制）。已整合 #783 review 的 P2/P3 修复（commit 418d977f）。

修复清单

MembersTab

• P3-3 admin disable 自己加 confirm（自锁风险）
• P3-5 busyUserId 全局 disable（防并发 PATCH 基于陈旧 list）
• P3-8 flash setTimeout useRef + unmount cleanup
• P3-5 invitation is_expired 前端 fallback

SettingsModal

• P3-2 activeTab 回落 general（tab 消失时空状态）
• R1-P3-3 AnimatePresence exit 动画

GeneralTab / AIConfigTab

• P3-4 ApiError 替代 (err as any) 反模式
• P3-8 success setTimeout cleanup

ChatContainer

• P3-4 dropdown active 项显示 work_dir（三轮提的 UX 回归）
• P3-7 currentUser 401 区分（settings 红点 + re-login 提示）
• P2（fix: PR #779 review polish — 全量清理 P2/P3 finding #783 review） getMe catch 仅 401/403 setAuthError（网络/5xx 静默，re-login 解决不了）

API 层

• P3-4 errors.ts 新增 ApiError 类（status/code/info 类型安全）+ raw 回退
• P3-4 workspaces.ts updateWorkspace 抛 ApiError
• P3-5 auth.ts Invitation is_expired 接口 + admin* cookie 通道注释
• P3-2（fix: PR #779 review polish — 全量清理 P2/P3 finding #783 review） ApiError.fromResponse 接入 getMe（消除死代码）

后端

• P3-5 ListInvitations 响应加 is_expired（服务器计算，避免客户端时钟漂移）

spec

• P3-1 §3.1 Gap-C1 条件勘误（三者皆空才跳过 mergeContextUsage）
• P3-7 status: draft → analysis

后续 issue

• settings-modal tab 与 admin 页面组件复用（架构重构） #782 settings-modal tab 与 admin 组件复用（架构重构，P2-1）

接受（历史已定）

• spec 混入 PR feat(webchat): Phase 3 Settings Modal + 顶部 workspace bar 布局重构 (P3.1-P3.4) #779（已合并，无法拆分）

验证

• go vet ./internal/admin/... + gofmt + go test ./internal/admin/... ✓
• pnpm exec tsc --noEmit ✓
• pre-commit golangci-lint 0 issues ✓

Refs #772

[hotplex-ai]

Code Review — hotplex-ai

Verdict: Request Changes | P0:0 P1:1 P2:1 P3:0

本 PR 作为 PR #779/#783 review 的 polish,整体质量高:is_expired 服务端计算、ApiError 类型化、abort signal 传播、setTimeout 清理、Hooks 顺序修复均正确。仅 1 个 P1(本 PR 新增路径引入的行为不一致)需处理。

---

🔴 P1 — 自禁用账号后未登出,confirm 文案与实际行为不符(矛盾 UI 状态)

webchat/app/components/chat/settings-modal/members-tab.tsx:77-92

handleToggleUser 的 self-lock guard 对用户承诺:"You will be logged out immediately and unable to sign back in"(line 78),但成功路径(line 86-87)只做了 flash('ok') + load(),既无 logout() 也无 window.location.replace('/login')。

实际因果链(已对照后端 internal/admin/user_handlers.go:52-56 核实):

1. adminUpdateUserStatus(self, 'disabled') 成功 → flash('ok', "→ disabled")
2. 紧接 load() → adminListUsers → GET /api/admin/users
3. requireAdmin 此时当前用户 status 已是 disabled → 返回 403 USER_DISABLED(user_handlers.go:53-54)
4. load() catch → setError(...) → MembersTab 整体渲染为红色 error div(line 118-120),取代整个列表

用户实际看到的是:绿色"成功"flash 紧跟红色"user disabled"错误页,卡死,而非文案承诺的"立即登出"。这是本 PR 新增的 self-lock 路径引入的 UX/行为一致性缺陷。

修复建议(成功分支针对自禁用单独处理):

await adminUpdateUserStatus(user.id, next);
if (currentUser?.id === user.id && next === 'disabled') {
  // 兑现 confirm 文案承诺的 "logged out immediately"
  try { await logout(); } catch {}
  window.location.replace('/login');
  return;
}
flash('ok', `${user.username} → ${next}`);
load();

🟡 P2 — ChatContainer 的 getMe 未传 AbortSignal(与 PR 主题不一致)

webchat/app/components/chat/ChatContainer.assistant-ui.tsx:129

本 PR 正在系统性补 abort signal(MembersTab load()、各 tab 的 setTimeout 清理),但 ChatContainer 此处的 getMe() 调用未传 signal。getMe(signal?) 已支持(auth.ts:59)。mounted 闭包守卫已规避 unmount 后 setState 的实际危害,故降为 P2(非 bug,主题一致性改进)。

---

✅ 已核验通过(无 finding)

• Go invitationView 嵌入 *session.Invitation + IsExpired:无 nil 解引用、无 JSON tag 冲突(SQLite/PG 两种 store 的 scanInvitation 失败即 return err,不追加 nil)
• 前端 inv.is_expired ?? (expires_at*1000 < Date.now()) fallback:与 expires_at 秒级 Unix 时间戳单位一致,向后兼容旧后端
• ApiError extends Error 类型化:所有现存调用点(instanceof Error/空 catch)向前兼容
• settings-modal AnimatePresence + 无条件 useEffect:修复了原 if(!open) return null 的 Hooks 顺序隐患
• MembersTab busyUserId !== null 全局禁用:正确串行化并发 PATCH,避免陈旧列表快照写入
• spec 勘误(Gap-C1 守卫条件):已对照 bridge_forward.go:913-916 核实行号与 MaxTokens||TotalTokens||Model 短路逻辑准确

P1/P2 已在 fc86c22 修复（self-disable logout 兑现文案 + getMe AbortSignal），dismiss 旧 review 等 cron re-review。

[hotplex-ai]

等效 re-review APPROVE：上轮 CHANGES_REQUESTED 的 P1（self-disable 后 logout 兑现 confirm 文案）+ P2（getMe AbortSignal）已在 fc86c22 修复。cron gateway 收到 webhook（202）但未生成自动 review，手动提交通过结论。