Skip to content

Add graceful TUI fallback for terminals that do not support Textual#597

Merged
chernistry merged 1 commit into
mainfrom
bernstein/task-79b51fdcec15
Apr 8, 2026
Merged

Add graceful TUI fallback for terminals that do not support Textual#597
chernistry merged 1 commit into
mainfrom
bernstein/task-79b51fdcec15

Conversation

@chernistry

Copy link
Copy Markdown
Collaborator

Add graceful TUI fallback for terminals that do not support Textual

Add graceful TUI fallback for terminals that do not support Textual

Description

When the terminal does not support the features Textual requires (e.g., basic SSH terminals, screen sessions, CI logs), the TUI crashes or displays garbage. Detect terminal capabilities on startup and fall back to a Rich-based simple status display.

Role: frontend
Model: sonnet


Generated by Bernstein — task 79b51fdcec15

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@chernistry chernistry added bernstein Bernstein auto-generated PR auto-generated Created by Bernstein agents labels Apr 8, 2026
@chernistry chernistry merged commit 6e7d588 into main Apr 8, 2026
@github-actions

github-actions Bot commented Apr 8, 2026

Copy link
Copy Markdown
Contributor

AI Review (Gemini)

Gemini review could not be generated (HTTP 503).

@github-actions

github-actions Bot commented Apr 8, 2026

Copy link
Copy Markdown
Contributor

AI Review (GitHub Models)

Summary

The proposed changes in PR #597 aim to implement a fallback Text User Interface (TUI) for terminals that lack support for the Textual library. This enhancement is expected to increase the usability of Bernstein across various terminal environments, accommodating users with different setups.

Risk Assessment

Medium: While the fallback functionality improves accessibility, it introduces additional complexity to the codebase. The implementation needs to ensure that it does not inadvertently disrupt existing functionalities, particularly in how the TUI is rendered across different environments.

Actionable Recommendations

  • Ensure thorough unit and integration tests are added to cover the new fallback implementation.
  • Review error handling to avoid potential crashes in environments where both the primary and fallback TUIs may fail.
  • Document the changes comprehensively, including the criteria for assessing TUI support in different terminals.
  • Perform cross-platform tests to validate the functionality works consistently on various OS and terminal configurations.
  • Consider adding a configuration option that allows users to toggle between Textual and the fallback

@sonarqubecloud

sonarqubecloud Bot commented Apr 8, 2026

Copy link
Copy Markdown

@github-actions

github-actions Bot commented Apr 8, 2026

Copy link
Copy Markdown
Contributor

CI Summary

Check Result
Repo hygiene ✅ success
Lint ✅ success
Type check ✅ success
Tests ❌ failure
Spelling ✅ success
Dead code ✅ success
Workflow lint ✅ success
Dist size ✅ success
SonarCloud ⏭️ skipped

Coverage and detailed reports are available via Codecov and the Checks tab.

@codecov

codecov Bot commented Apr 8, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@chernistry chernistry deleted the bernstein/task-79b51fdcec15 branch April 8, 2026 17:55
chernistry added a commit that referenced this pull request Apr 30, 2026
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-generated Created by Bernstein agents bernstein Bernstein auto-generated PR size/xs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant