fix(herdr): 统一原生终端流与 Web 历史 - #512
Draft
deepcoldy wants to merge 1 commit into
Draft
Conversation
Owner
Author
独立复核结论(relay + codex 交叉验证):先别合方向正确(原生 Bug 1:Herdr + Workflow 可能丢最终输出
Bug 2:Web 慢客户端 resync / 新连接在累计超 1MB 后残屏
与 master 的冲突
验证状态
结论不 Approve、不合码。建议申晗修完上述两个 bug、rebase 最新 master 解 3 个内容冲突、重跑 build + 全量 + Herdr e2e 后再复核。 此次只做审查,没有改源码、提交或合并,也没有触碰 platform 仓。 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
改了什么
terminal session observe原生 NDJSON 流,并保留 full/delta/seq/尺寸语义。agent attach尺寸 ownership、光标、历史合并和/restart重绑定。为什么
PR #484 的原生流方向与已合并 #419 的 Web 终端实现存在强实现冲突。直接按序合并会破坏 scrollback、历史快照与 resize 生命周期。本 PR 在最新 master 上给出经两轮独立审查后的统一实现,作为 #484 的整合替代版本。
影响面
onData路径。验证
pnpm buildTZ=Asia/Shanghai pnpm test:583 files,9324/9324pnpm vitest run test/herdr-backend.e2e.ts test/worker-herdr-web-terminal.e2e.ts:11/11(真实 Herdr 0.7.3)