历史记录加载不出来的bug，可以这么改

[krudiy]

问题描述

打开带历史记录的会话时，聊天视图永远停留在"加载中"。浏览器控制台显示：插件 buildLocationData hook 抛出的未捕获 TypeError，穿透 ConversationNodeAssembler.flush → Notifier.rebuild，导致整个会话重建被中断。单个插件 hook 出错会拖垮整条历史渲染链路，且没有任何降级、错误提示或重试机制。

复现步骤

1. 启动 dsh web，加载任意一个未对 context.state === void 0 做守卫的客户端插件（即注册了 turn 作用域定义的插件，见下方最小示例）。
2. 打开一个会话，其已加载的历史窗口内存在"turn/start 事件不在窗口内"的轮次（历史分页加载 / 启动时窗口截断）。
3. 观察结果：历史记录永久卡在"加载中"，控制台输出下方堆栈。

出问题的插件 hook 最小示例（与复现者插件行为一致）：

buildLocationData(context, scope) {
  if (scope !== "turn") return null;
  return {
    kind: "turn",
    turn: context.state.turn, // 当 context.state === void 0 时抛错
    key: "my-definition",
    value: context.state,
  };
}

预期行为

• 插件 hook 抛错时应局部降级（该插件的数据位缺失、控制台记录错误即可），而不是中断整个 Notifier flush / 历史重建。
• 按现有 hook 契约，buildLocationData 在 context.state === void 0 时必须返回 null；这一约束应由引擎侧强制或文档化。

实际行为

client.js?rev=<plugin-rev>:186 Uncaught TypeError: Cannot read properties of undefined (reading 'turn')
    at Object.buildLocationData (client.js?rev=<plugin-rev>:186:31)
    at ConversationNodeAssembler.buildLocationData (client.js?rev=<runtime-rev>:6812:37)
    at ConversationNodeAssembler.replaceLocationData (client.js?rev=<runtime-rev>:6824:25)
    at ConversationNodeAssembler.flush (client.js?rev=<runtime-rev>:6444:11)
    at Notifier.rebuild (client.js?rev=<runtime-rev>:7169:24)
    at Notifier.flush (client.js?rev=<runtime-rev>:5715:11)
    at publish (client.js?rev=<runtime-rev>:5700:11)

UI 后果：聊天视图永不渲染，无限停留在"加载中"。

环境

• dsh 版本：0.1.0-rc.7
• Node：v24.18.0
• 操作系统：Windows
• 浏览器：Chromium 系
• Profile：web

根因分析

问题分两层：

1. 插件 hook 违反契约——replaceLocationData() 会遍历所有已匹配的 context（this.contexts.values()），其中包括"turn/start 尚未进入已加载窗口"的 context；运行时对这些 context 会显式把 state 置为 void 0（replayContexts 分支：if (context.start === void 0) context.state = void 0）。插件若不做守卫直接读 context.state.turn 就会崩溃。内置 dsh-client-ui-deliverables 定义展示了正确的契约写法：

buildLocationData: (context, scope) => scope !== "turn" || context.state === void 0 ? null : {...}

2. 运行时缺少异常隔离——ConversationNodeAssembler.buildLocationData（以及 buildNode）调用 definition hook 时没有 try/catch。抛错会穿透 flush() → Notifier.rebuild() → Notifier.flush() → publish()，直接杀死整个会话重建。同时 UI 没有任何错误/重试状态，于是永久卡死。

建议修复方案（按优先级）

1. 运行时防御加固：将 definition hook 调用（buildLocationData、buildViewNode）包进 try/catch；抛错时 console.error 并返回 null，让插件降级为"数据位缺失"，而不是拖垮整个重建。
2. 引擎层守卫：对 context.state === void 0 的 context 直接跳过 buildLocationData 调用（引擎本来就知道这是合法状态）；或在 ConversationNodeDefinition 的类型/文档中强制约定"无 state 时返回 null"。
3. 失败可见性：Notifier.rebuild 失败目前让 UI 无限停留在"加载中"；应改为呈现错误/重试状态。

临时规避方案（本地）

• 修复出问题的插件：context.state === void 0 时返回 null（与内置契约一致）。
• 更稳妥：对 dsh-client-runtime/lib/client.js 的 buildLocationData / buildNode 做同样的 try/catch 加固，然后强制刷新 GUI。