Skip to content

fix(tui): harden terminal resume and runtime context#950

Merged
Hmbown merged 1 commit intomainfrom
hotfix/terminal-resume-version
May 7, 2026
Merged

fix(tui): harden terminal resume and runtime context#950
Hmbown merged 1 commit intomainfrom
hotfix/terminal-resume-version

Conversation

@Hmbown
Copy link
Copy Markdown
Owner

@Hmbown Hmbown commented May 7, 2026

Summary

  • keep alternate-screen enabled in default auto mode so transcript scrolling stays inside the TUI instead of falling back to terminal scrollback; inline scrollback now requires explicit --no-alt-screen or tui.alternate_screen = "never"
  • queue ResumeEvents if the terminal guard drops while the engine event channel is full, so mouse capture/raw terminal state is restored after interactive tool cancellation or bursts
  • scope crash-checkpoint recovery to the resolved launch workspace instead of the shell cwd, so --workspace cannot recover an old tree by accident
  • add deepseek_version to the system prompt Environment block and tell agents to distinguish runtime version from a stale workspace Cargo.toml

Test plan

  • cargo test -p deepseek-tui --bin deepseek-tui terminal_mode_tests --locked
  • cargo test -p deepseek-tui --bin deepseek-tui terminal_guard_queues_resume_when_event_channel_is_full --locked
  • cargo test -p deepseek-tui --bin deepseek-tui render_environment_block_lists_supplied_locale_and_workspace --locked
  • cargo test -p deepseek-tui --bin deepseek-tui workspace_scope_ --locked
  • cargo fmt --all -- --check
  • git diff --check
  • cargo clippy -p deepseek-tui --bin deepseek-tui --all-targets --locked -- -D warnings
  • cargo build --release --locked -p deepseek-tui-cli -p deepseek-tui

Local smoke

Installed rebuilt target/release/deepseek and target/release/deepseek-tui into ~/.cargo/bin for maintainer terminal testing before merge/release.

Copilot AI review requested due to automatic review settings May 7, 2026 02:42
@gemini-code-assist
Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@Hmbown Hmbown force-pushed the hotfix/terminal-resume-version branch from 5a5d8b7 to 6526970 Compare May 7, 2026 02:48
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@Hmbown Hmbown merged commit c270ef8 into main May 7, 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.

2 participants