Skip to content

fix: Use async TUI event handling to fix Send issue#686

Merged
braedonsaunders merged 1 commit intoclaude/build-openstudio-app-kgia5from
claude/remove-unused-import-KfbWA
Jan 3, 2026
Merged

fix: Use async TUI event handling to fix Send issue#686
braedonsaunders merged 1 commit intoclaude/build-openstudio-app-kgia5from
claude/remove-unused-import-KfbWA

Conversation

@braedonsaunders
Copy link
Owner

  • Replace blocking crossterm event polling with async EventStream
  • Use LocalSet + spawn_local for non-Send futures (cpal::Stream)
  • TUI now cooperates with tokio runtime, allowing WebSocket server to run
  • Added event-stream feature to crossterm dependency

This fixes the WebSocket server being starved when TUI was blocking, and avoids the Send trait requirement that prevented thread spawning.

- Replace blocking crossterm event polling with async EventStream
- Use LocalSet + spawn_local for non-Send futures (cpal::Stream)
- TUI now cooperates with tokio runtime, allowing WebSocket server to run
- Added event-stream feature to crossterm dependency

This fixes the WebSocket server being starved when TUI was blocking,
and avoids the Send trait requirement that prevented thread spawning.
@vercel
Copy link

vercel bot commented Jan 3, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
openstudio Ready Ready Preview, Comment Jan 3, 2026 6:03am

@braedonsaunders braedonsaunders merged commit ed7f8b9 into claude/build-openstudio-app-kgia5 Jan 3, 2026
2 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