Skip to content

fix(tui): clear dismissed update prompt#30479

Draft
fcoury-oai wants to merge 1 commit into
mainfrom
fcoury/fix-update-prompt-clear
Draft

fix(tui): clear dismissed update prompt#30479
fcoury-oai wants to merge 1 commit into
mainfrom
fcoury/fix-update-prompt-clear

Conversation

@fcoury-oai

Copy link
Copy Markdown
Contributor

Summary

  • clear the full-frame update prompt after every rendered selection path
  • prevent Skip, Escape, Ctrl-C, and Skip until next version from leaving stale prompt contents during slower startup

Fixes #30205

Testing

  • cargo test -p codex-tui update_prompt_ --release -- --skip update_prompt_snapshot
  • git diff --check

Note: the existing update_prompt_snapshot currently differs from its checked-in snapshot on main and generates a .snap.new; this PR leaves that unrelated snapshot unchanged.

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.

Bug: update prompt remains visible after Skip during codex resume startup

1 participant