Skip to content

research(ui): 1049 切换前后的 SGR 保持性 (#126/#123) - #183

Merged
JavaZeroo merged 2 commits into
mainfrom
research/1049-sgr-carry
Oct 29, 2026
Merged

JavaZeroo merged 2 commits into
mainfrom
research/1049-sgr-carry

Conversation

@JavaZeroo

Copy link
Copy Markdown
Owner

内容

在已合入的 1049 颜色结果上追加SGR 属性保持性观察:

  • 会话内原生命令先设 SGR 31;44(红字蓝底),切换前不再 reset;
  • 阶段 M(主屏)→ 1049h → A(备用屏)→ 1049l → R(回主屏),每阶段画"当前背景空格 + 反显空格";
  • 逐阶段截图、终端矩形逐行取色:
阶段 背景(空格行) 前景(反显行) 裁切 SHA-16
M #3465A4 #C4265E 675e5d7b24dbdcf9
A #3465A4 #C4265E 734a58fe0224fcfb
R #3465A4 #C4265E 31a070a2c1706709

结论:在这条 conhost + ConPTY + WindTerm 管线里,1049h/1049l 既不重置也不恢复当前 SGR;裁切哈希不同说明阶段确实推进。

同时更新 validation.json 与 EVIDENCE。

边界

Refs #126 #123

Sets 31;44 before the switch and keeps it; main, alternate and restored phases render the same blue background and red foreground with distinct crop hashes, so the switch neither resets nor restores the attributes in this pipeline.
@chatgpt-codex-connector

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 ⚠️ Failed 2026-10-29T13:15:18.656047Z
We couldn’t complete this review. Comment @codex review to try again.
e13be0c PR opened
ℹ️ 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.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

…tion refs #123 #126

With 31;44 on main and 32;45 inside the alternate screen, the frames show magenta/green in the alternate buffer and the default colours back on main after 1049l, matching the earlier carry run where the entry-time attributes came back.
@JavaZeroo
JavaZeroo merged commit a0a0023 into main Oct 29, 2026
6 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