兼容性问题：Safari/WebKit 上已完成的会话无法渲染

[neverth]

现象

在 iOS Safari 和 macOS Safari 上，会话正在流式输出时显示正常，但回答结束后再打开这个会话就渲染不出来，刷新也没用。Chrome 完全正常。

根因

packages/util/values/src/index.ts 的 hasIntrinsicConstructor 用精确字符串比较来判断一个构造器是不是原生内建：

Function.prototype.toString.call(constructor) === `function ${name}() { [native code] }`

V8（Chrome / Node）输出的正是这个单行格式，但 JavaScriptCore（Safari）输出的是带换行和缩进的：

"function Object() {\n    [native code]\n}"

两种写法都符合 ECMAScript 的 NativeFunction 语法 —— 该语法固定了 function、名字、参数列表和 [native code] 这些 token，但把空白的排布留给引擎自行实现。因此把某一种排版写死，等于假定了 V8。

后果是在 Safari 上每一个普通对象都过不了「内部原型」检查，连锁反应：

snapshotJsonValue()  →  返回 undefined
  ↑ snapshotChunk()             (packages/llm/llm/src/assistant-stream.ts:88)
  ↑ expandAssistantStream()     抛 TypeError:
      "Assistant stream raw chunk must be a lossless JSON object"   (同文件 :492)
  ↑ session-controller 的 event feed 订阅者失败
  ↑ 整个会话渲染不出来

流式输出走的是另一条路径，不回放持久化的 chunk，所以只有「已完成的会话」才复现 —— 这也解释了为什么边输出边看是好的。

复现步骤

1. 用 Safari（iOS 或 macOS）打开 Web UI；
2. 发一条消息，等回答完整结束；
3. 切到别的会话再切回来，或直接刷新页面；
4. 该会话空白，渲染不出来。控制台可见 Assistant stream raw chunk must be a lossless JSON object。

不用真机也能在 Node 里复现，只要把 JavaScriptCore 的排版模拟出来：

const nativeToString = Function.prototype.toString
Function.prototype.toString = function patched(this: unknown): string {
  if (this === Object) return 'function Object() {\n    [native code]\n}'
  if (this === Array) return 'function Array() {\n    [native code]\n}'
  return nativeToString.call(this)
}
expandAssistantStream([
  { type: 'chunk', time: 0, chunk: { choices: [{ delta: { content: 'hi' } }] } },
])
// TypeError: Assistant stream raw chunk must be a lossless JSON object

引入版本

由 f99b06eaed (feat(session)!: embed assistant streams in format v2) 引入，该 commit 已在当前 master 中。

建议的修复

改为按 NativeFunction 语法匹配，而不是钉死某一种排版 —— 语法固定 token，空白交给引擎：

const NATIVE_FUNCTION_SOURCE = /^function\s+([A-Za-z$_][\w$]*)\s*\(\s*\)\s*\{\s*\[native code\]\s*\}$/u

安全性没有放松：函数体必须恰好是 [native code]，所以伪造的 toString 依然会被拒。以下这些都仍然判为 false：

• function Object() { return "[native code]" }
• function Object() { /* [native code] */ }
• function Object() { doEvil(); [native code] }
• 名字不匹配、带参数、前后有多余字符的各种情况

环境

• 版本：0.1.3-alpha.2（master b0a7d2ce3b）
• 受影响浏览器：iOS Safari、macOS Safari（所有 JavaScriptCore 内核，含 iOS 上的 Chrome/Edge）
• 不受影响：Chrome、Edge、Firefox（桌面端）
• Node v22.22.3 / pnpm 11.7.0

[CNyaotian-Lunar]

确认并致谢：这条根因分析是对的，我们独立复现了 / Confirmed — and thank you: this root-cause analysis is correct, and we reproduced it independently

你 9/8 给出的定位，我们这边逐条核对成立，并且独立复现了整条链：

• 你点名的 hasIntrinsicConstructor（packages/util/values/src/index.ts）确实是症结所在；
• 补两份引擎源码出处（实测级）：WebKit 的 Source/JavaScriptCore/runtime/FunctionPrototype.cpp → functionProtoFuncToString 里硬编码了 "() {\n [native code]\n}"；Gecko 的 js/src/vm/JSFunction.cpp:969（及 :1005）追加的是同一个多行形式；V8 与 Hermes 则是单行 ⇒ 受影响的是 Gecko + WebKit，V8 / Hermes 免疫。也就是说：不止 Safari —— Firefox 系一样中（本楼之外的 [Bug] Web UI history never loads on Firefox-engine browsers (infinite "Loading history…") — Firefox-only failure in lossless-JSON validation #5677 从 Firefox 侧报到的是同一件事）。
• 真机验证：Gecko 上 Function.prototype.toString.call(Object) 是 3 行，判据恒假 ⇒ snapshotJsonValue 对一切普通对象返回 undefined ⇒ 会话打不开。

English: Your 9/8 localization checks out line by line on our side, and we reproduced the whole chain independently. Two source-level additions: WebKit's FunctionPrototype.cpp (functionProtoFuncToString) hard-codes the multi-line form "() {\n [native code]\n}", and Gecko's JSFunction.cpp:969/:1005 appends the same; V8 and Hermes emit the single-line form ⇒ Gecko and WebKit are affected, V8 and Hermes are not — i.e. this is not Safari-only; Firefox-family browsers hit it too (#5677 reports the same thing from the Firefox side). On a real machine under Gecko, Function.prototype.toString.call(Object) is 3 lines, the predicate is always false, snapshotJsonValue returns undefined for every plain object, and the session fails to open.

我们把它做成了可安装的补丁 / We turned it into an installable patch（MIT）：
🔗 https://github.com/CNyaotian-Lunar/dsh-waitforsocket-timeout

• 覆盖如实说明：客户端那 2 个产物（dsh-util-values + dsh-api-session-controller 的客户端 bundle）已修；上游另有 6 个 latent 副本（dsh-tools ×2 / dsh-cordis-host-runner ×2 / dsh-ptc-runtime-node ×2）未覆盖 —— 它们跑在宿主 Node / V8 上、判据正常，当前不会触发（[Bug] Web UI history never loads on Firefox-engine browsers (infinite "Loading history…") — Firefox-only failure in lossless-JSON validation #5677 里 argszero 把这几处标为 "latent"，khanecho 另附了参考 diff）。
• 脚本自带幂等 / 自动备份 / 锚点不匹配即拒绝（exit 2）/ 语法检查失败自动回滚；锚点对官方 0.1.7-rc.2 逐字节核对。
• 修法与你写的一致：比较前把连续空白规范化 —— .replace(/\s+/g, " ").trim()。

English: Coverage stated honestly: the two client-side artifacts are patched; the 6 further latent copies upstream are not covered because they run on host Node/V8 where the predicate holds (marked "latent" by argszero in #5677). The script is idempotent, auto-backups, refuses on anchor mismatch (exit 2) and auto-rolls-back on syntax failure; anchors are byte-verified against the official 0.1.7-rc.2. The fix matches yours: normalize whitespace before comparing.

（顺带一句：这类"精确字符串比对引擎输出"的判据，我们建议上游改成 constructor.prototype === prototype，彻底不依赖 toString 的排版。）

English: One aside: for predicates that compare engine output exactly, we would suggest upstream switch to constructor.prototype === prototype, so the check no longer depends on toString's formatting at all.

本评论由 DSH agent 整理、经人工复核后提交 / Written by a DSH agent, reviewed by a human before posting.

[CNyaotian-Lunar]

更正一句（我们昨天评论末尾那条建议）/ One correction to our previous comment

@neverth 我们昨天在这条评论末尾写了「建议上游改用 constructor.prototype === prototype」—— 那句是错的，撤回。

constructor.prototype === prototype 本来就在判据里，所以"换成它"实际等于去掉 toString 检查，会让 name / prototype 都伪造得出来的构造器通过。derSato 在 #5677 用实测指出了这一点（with source check → false / without → true）。

正确的说法是：保留 toString 检查，只在比较前规范化空白 —— 我们补丁里的修法就是后者（且没有动 toString 检查）。

English: @neverth — one correction to the remark at the end of our comment yesterday: we suggested upstream "switch to constructor.prototype === prototype". That was wrong; withdrawn. constructor.prototype === prototype is already part of the predicate, so "switching to it" amounts to dropping the toString check, which would let a constructor with a forged name/prototype through — derSato demonstrated exactly this in #5677 (with source check → false / without → true). The correct statement is: keep the toString check, and only normalize whitespace before comparing — which is what our patch does.

本评论由 DSH agent 整理、经人工复核后提交 / Written by a DSH agent, reviewed by a human before posting.

[ttbj2023]

补充两点 Firefox/SpiderMonkey 侧的增量信息（0.1.7-rc.2 上独立定位到同一处后看到此帖，结论与此楼一致）：

1. Firefox 同样命中，实测格式串：

• Firefox 151（SpiderMonkey）：Function.prototype.toString.call(Object) → "function Object() {\n [native code]\n}"（多行，同 JSC 排版，不匹配模板）
• Chromium/Node（V8）：→ "function Object() { [native code] }"（单行，匹配）

即 Firefox 与 Safari 全中、Chromium 必然免疫——也闭环了 #7677 的 Firefox 症状报告。

2. 触发条件可精确为：follow 快照窗口内存在未结算 attempt。in-flight 流携带 block-start 等原生 chunk 记录（必然含普通对象 chunk，从第一个块起就存在）；attempt 结算后被压缩为 text-chunks 等带类型记录，校验不再经过对象原型检查。因此症状间歇出现：模型流式输出相位必卡、工具执行间隙/回合结束后自愈；持久化日志全量重放校验为 0 坏记录（坏值只存在于活流，从未落盘）。最小触发样本是一个完全合法的记录：{"type":"chunk","time":...,"chunk":{"type":"block-start","index":0,"blockType":"reasoning"}}。

本地在安装包上做了验证性补丁：对该比较做空白归一化（保留 constructor.prototype === prototype 等其余判据不变），Firefox 上症状消失。