fix(riff-backend): 修复 web 终端 stdout 日志糊成墙(SSE 逐行日志丢尾换行)

[deepcoldy]

问题

codex_app_server 路径下,飞书 web 终端的日志正文"糊成墙"——多条 stdout 日志首尾相接、零分隔,不可读(申晗截图复现,截图里能看到 thread.started/turn.started,确认走的是 main 上的 app_server 新路径,不是老 exec)。

根因:SSE 逐行日志丢尾换行

riff 端逐行、无尾换行地存/发 stdout 日志:

1. appServer.ts emitNormalized:logger.info(JSON.stringify(event), {group:'stdout'}) 存的是裸行(带 \n 的那份喂给另一个 sink,不进日志流)。codex_app_server 下这条 message 就是一条 JSON.stringify(event)(thread.started / turn.started / item.* …)。
2. runner logger 存 message 时只做 mask/truncate,不加换行。
3. SSE event: log 的 text = log.message 原样。

到本仓 relay 这端,emitText 只把已有 \r?\n 归一化(\n→\r\n),从不给每条日志补分隔。于是 case 'log' 的 stdout 分支 emitText(text) 把逐条裸行首尾相接 → 糊成墙。

修法

src/adapters/backend/riff-backend.ts 的 case 'log',group === 'stdout' 分支:

- this.emitText(text);
+ this.emitText(`${text}\n`);

因 riff 逐行、无尾换行存,补 \n 精确不重复。

两个刻意的边界

• 没用 emitLine:emitLine 会包 ANSI 色 + 前后 \r\n 空行(状态行语义),会把每条 stdout 撑成双倍行距。stdout 原始流只需要"补一个分隔",所以走 emitText(text+'\n')。
• output/chunk 路径故意不动:event: output 的 chunk 是字节流、可能半行语义,给它补 \n 会把一行劈两段。只有 log 事件是逐行语义,补换行才安全。

影响面

只影响 web 终端实时日志流的渲染(把"糊成墙"降为"每行一个规整 JSON 事件"可读)。不改任何数据、不动终报路径、不动 chunk 流。

注:补 \n 后仍是裸 JSON-per-line,不是人类可读时间线——那是可选的后续打磨(relay 里 parse thread/turn/item 渲成时间线),不在本 PR 范围。

测试

新增 3 个回归测试(test/riff-backend.test.ts):

• 两条连续 stdout log 分行不糊(断言不含 }{,各自 …\r\n)
• 单条不双空行(hello → hello\r\n)
• chunk 路径保持原样(par+tial → partial,无注入换行)

riff-backend.test.ts 50/50 绿,tsc --noEmit 对改动文件干净。

[deepcoldy]

Review 结论：未发现阻塞问题，代码层面 LGTM。当前账号与 PR 作者相同，GitHub 不允许 self-approve，因此以 review comment 记录结论。

核验结论：

• codex_app_server 上游确实按逐行、无尾换行落库：emitNormalized 先生成 line = JSON.stringify(event)，logger.info(line, { group: 'stdout' }) 持久化裸行；带 \n 的 ${line}\n 只送 onOutput，不进入日志表。
• Logger.createRootLog 仅 mask/truncate，不补行尾；runnerLogToTaskLogEvent 将 text 直接设为 log.message，RunnerTaskStreamPoller 再原样序列化进 SSE。因此到 botmux 的 event:log stdout text 没有行尾，补一个 \n 不会重复。
• 普通 exec 路径同样由 readline 的 line 事件去掉行尾后，以裸 line + { group } 落库，和上述契约一致。
• 使用 emitText 而非 emitLine 是正确的：前者只做 \n -> \r\n 归一化；后者会增加 ANSI 及前后 CRLF，确实会造成额外空行。
• event:output 是 chunk 语义，保持原样是必要边界；新增 chunk 拼接测试覆盖到了这一点。

独立验证：

• pnpm exec vitest run --project unit test/riff-backend.test.ts：50/50 通过。
• pnpm exec tsc --noEmit：通过。
• git diff --check：通过。

当前 GitHub build 红灯来自 codex-app-runner.integration 超时和 worker-argv-reaction-status.integration 时序断言；base master 的 7cd22db CI 已以相同两组用例失败，且本 PR 单跑 codex-app-runner.integration 42/42 通过，判断与本 diff 无关。worker-argv-reaction-status.integration 在本地 base/PR 环境均可复现同样失败。

[github-actions]

🚀 Released in v3.12.0