Repository navigation
test(qt): verify retained hidden-frame body through Qt/ConPTY - #315
Merged
JavaZeroo merged 3 commits intoNov 4, 2026
Merged
Conversation
Add a formal Qt / fixed-ConPTY seven-phase fixture: a real child writes the fixed 91x20 primary body, the public widget narrows to 61 columns, the child enters the alternate screen, only the height changes, and 1049l returns with the width restored. The test checks the retained source body, row identity, display page, display caret and hit mapping against literal expectations; the child's native rows are recorded separately and are not the retained-body oracle. Wire the new qt_hidden_retention target into the Qt workflow. This closes the fixed formal-path scenario for the 2026-11-04 confirmed full-body decision; it does not claim physical presentation, WindTerm equivalence, complete resize or #27/#29 closure. Co-authored-by: JavaZeroo <[email protected]>
Publish the fixed seven-phase Qt / fixed-ConPTY result for the confirmed hidden-frame full-body decision: stage table, source/artifact/dependency hashes, display and hit observations, and the offscreen-only limits. Update FEATURES, EVIDENCE, ADR-0010 and the Qt entry so the formal-path identity stays separate from the original WindTerm observation and the core engineering regression. #27/#29/#30/#7 stay open; physical presentation, DPI, other sizes, Unicode and complete reflow remain uncovered. Co-authored-by: JavaZeroo <[email protected]>
Record that the fixed formal-path scenario was already retained by the existing resize/display transaction; the initial test failures were the new test's own display-row expectations and native-row fixture coordinate, not a product RED. #27/#29 stay open. Co-authored-by: JavaZeroo <[email protected]>
5 tasks
Open
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.
背景
2026-11-04 用户已确认:隐藏帧返回时 TermForge 保留完整正文,接受 WindTerm 2.7.0 返回主屏后裁剪隐藏行的可观察差异(决定评论)。ADR-0010 已归档,但当时“正式路径验收”仍是未完成项。本 PR 补上该固定场景的正式 Qt / 固定 ConPTY 路径实现与验证。
改动
qt_hidden_retention(src/qt/hidden_retention_test.cpp、src/qt/hidden_retention_fixture.hpp):真实 child 写入固定 75 B、91×20 主屏正文;公开 Qt 控件缩窄到 61 列;child1049h进入备用屏;仅高度 20→14→20;1049l返回主屏;恢复宽度 91。ViewMapping::hit命中。CMakeLists.txt注册qt_hidden_retention;.github/workflows/qt-cpp.yml构建、运行并记录 runtime manifest。src/qt/README.md,把“正式路径验收”从待补改为固定七阶段离屏通过,并保留其余限制。resize_preserving+ 显示投影已满足该固定场景;新测试初次失败来自测试自身的显示行预期与夹具原生行读取坐标,不是产品 RED,也未发现需要修复的正文丢失。验证(本地实测,Windows 11 x64 / MSVC 19.42 / Qt 6.10.3 offscreen / 固定 ConPTY 1.25.260930003)
child_pid外输出一致。0e683ff0…:缩窄、隐藏高度变化与1049l返回均未丢失已确认的完整正文;保留行 id12、91-Cell 源行与 5 行历史前缀保持;显示页/光标/命中映射同一源身份。边界
presented_done=null)、非真实 DPI/焦点、非原生 IME、非性能验收。semantic_acceptance=false不改写;不为匹配原版丢失结果而裁剪正文。