Skip to content

fix: harden R2 terminal handling and Windows regression coverage - #43

Merged
JavaZeroo merged 33 commits into
mainfrom
fix/conpty-ctrl-c-r36
Oct 26, 2026
Merged

JavaZeroo merged 33 commits into
mainfrom
fix/conpty-ctrl-c-r36

Conversation

@JavaZeroo

@JavaZeroo JavaZeroo commented Oct 26, 2026 •

Copy link
Copy Markdown
Owner

Windows 回归原来会因短客户区、候选宽字尾格、输入回显时序和 OSC 标题重复标记而误判。修正独立观测和夹具,补上默认核心的 ECH 与 GUI 的合法空标题处理,并让 More 捕获等待完整提示、正常退出及真实 reader EOF。

  • 选区仍使用原内容、锚点和淘汰断言;681/860 两种高度均覆盖。Copy 错误帧按捕获时实际客户区与 DIB 几何严格校验,旧 v1 固定尺寸合同保留。
  • 原生文字观测统一使用标准投影;选择前在原五秒期限内等待实际回显。工作区恢复以独立 transport 判定排除标题/命令回显;完整 CSI 只处理语法封装,不裁决其终端执行语义。
  • ECH 修复保留 360/366 字节原输入和候选 pending-wrap 差异;真实空标题经 console→ConPTY→候选 GUI 验证,非法非空 UTF-8 仍拒绝。新增持续 CI,并覆盖 renderer 路径变更。
  • 拒绝负例允许已完整证明的首读前停止;正常 receiver 门槛不变。真实记录形成旧判定失败、新判定通过,22 个字段反例及非空/正常场景仍拒绝。
  • 增加受监督的原始 Ctrl-C 回归和四 shell 诊断。Ubuntu 前置只在指定一次性 hosted VM 缺失时导入已核 SHA 的官方 rootfs;自有夹具 Job 的强制收尾按显式策略验收,产品四 shell 仍要求正常退出。私有运输记录不上传。

验证包括原场景 red→green、旧二进制/同字节反事实、12 项 receiver 纯测试与 14/14 原工作流、32 条恢复判定断言、真实空标题两轮和原 32 组渲染场景。最终固定 eba9cf0 的 主 PR CI 为 Windows 43/43、Ubuntu 27/27;候选 PR CI 双平台成功,Windows 新 GUI 两项实际通过。主/候选 push 运行也自然成功,合计 8/8 job checks;4 个 PR ZIP 的实际 SHA 与 API digest 一致,均无 _private 或异常路径。云端成功 CTest 的内部 skip 数未知,候选 workflow 不是完整候选 CTest。

本机候选完整 CTest 原件为 45/46,唯一错误是测试子进程未找到现有 Vim;保留该失败。任务子进程 PATH 加入现有 Git Vim 后,仅重跑原 TUI 与差分两项,6+28 子测试全部通过、无内部 skip。它不是把原完整结果改写为 46/46。

已同步 TRACKING/EVIDENCE/PLAN,保留 Kimi 已提交的 margin 实现与验证。#36 的历史 Ctrl-C、#40 的原超时、#44 的 controller 退出以及 #38 的剩余独立回归仍 OPEN;原 normal-1 的异常调用点和载荷未知,不能由新空标题复现回填。完整 R2、实际呈现、长期稳定和 WindTerm 性能等价尚未验收。

Fixes #42
Fixes #45
Fixes #46
Refs #36, #38, #40, #44

@JavaZeroo JavaZeroo changed the title fix: restore strict Windows shell regression coverage fix: harden R2 terminal handling and Windows regression coverage Oct 26, 2026
@JavaZeroo
JavaZeroo marked this pull request as ready for review October 26, 2026 05:20
@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-26T05:22:30.672143Z eba9cf0 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.

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.

test: accept a proven pre-read stop in the rejected-order receiver oracle [R2][Bug] 合法空标题事件导致候选 GUI 抛出 invalid text [R2][回归] 核对 more.com 尾部 ECH 的捕获与 Recovery 差异

1 participant