Skip to content

fix: bound replay matching cost and drain replies safely on close (#53) - #77

Merged
JavaZeroo merged 6 commits into
fix/ui-paste-fence-r53from
fix/candidate-replay-cost-r53
Oct 26, 2026
Merged

JavaZeroo merged 6 commits into
fix/ui-paste-fence-r53from
fix/candidate-replay-cost-r53

Conversation

@JavaZeroo

@JavaZeroo JavaZeroo commented Oct 26, 2026 •

Copy link
Copy Markdown
Owner

八会话原场景中的 resize 重放反复为同一行生成匹配键,消耗大量 UI 处理时间;关闭阶段还会为剩余输出里的查询向已停止的输入队列发送回复。此变更复用单次匹配中的行键,并在关闭输入后继续解析输出、停止发送终端回复,正常会话仍严格检查回复错误。叠加 #78 的限定完成 oracle 修正,处理空记录 CR 导致的保存测试误拒绝。

验证源eae4b6a的原单窗口两轮取消/关闭、真实 EOF/缺GO负例及两轮资源采样的原八会话 data-only 均通过;真实关闭查询回归修前红、修后绿。1,000,000汉字/3MB、768B checkpoint、实际writer持块与全部原期限保持。#53机制与原件身份。Standards/Spec代码及证据均通过审查。

#78叠加后源码4daf992的三配置保存/恢复共18次通过;完整候选/官方54/56、449.38s,Vim59、resize43、main7、ui52仍由#51/#52/#63承接。最终分配置证据。当前HEAD f199481的两条CI均成功(40364334201、40364334390),已覆盖此前#72导致的默认CI红例。

关联#53,目标为#76的fix/ui-paste-fence-r53,最终进入实验后端分支。该通过不选择产品后端,不包含同次真实CF/IME、实际呈现、长期稳定或WindTerm性能等价。#72独立主线PR79仍受另一真实四shell退出失败#80阻塞,不把实验栈CI借给主线。

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 26, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-26T17:12:33.458333Z f199481 Draft marked ready
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@JavaZeroo
JavaZeroo marked this pull request as draft October 26, 2026 16:26
@JavaZeroo
JavaZeroo marked this pull request as ready for review October 26, 2026 17:11
@JavaZeroo
JavaZeroo merged commit 97ac142 into fix/ui-paste-fence-r53 Oct 26, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant