Skip to content

Fix WebSocket spacing and Ctrl-C shutdown behavior#745

Merged
ryanfowler merged 1 commit into
mainfrom
fix-websocket-interactive-spacing
Jun 10, 2026
Merged

Fix WebSocket spacing and Ctrl-C shutdown behavior#745
ryanfowler merged 1 commit into
mainfrom
fix-websocket-interactive-spacing

Conversation

@ryanfowler

Copy link
Copy Markdown
Owner

Summary

  • Remove the extra spacer row after interactive WebSocket messages so wrapping and scrolling stay consistent.
  • Reserve a blank line above the status/input area for clearer separation in interactive mode.
  • Treat Ctrl-C as a successful exit instead of a runtime error, including WebSocket non-interactive mode.
  • Add regression coverage for the updated interactive layout and Ctrl-C behavior.

Testing

  • cargo fmt --check
  • cargo clippy --locked --all-targets --all-features -- -D warnings
  • cargo test --locked --all-features --lib --bins
  • cargo test --locked --all-features --test websocket -- --test-threads=1
  • Focused unit and integration tests for WebSocket interactive rendering and Ctrl-C handling passed.

@ryanfowler ryanfowler enabled auto-merge June 10, 2026 04:25
@ryanfowler ryanfowler merged commit 0d9c2ea into main Jun 10, 2026
4 checks passed
@ryanfowler ryanfowler deleted the fix-websocket-interactive-spacing branch June 10, 2026 04:27
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