feat: improve search TUI usability and detail view styling#856
Merged
alishakawaguchi merged 4 commits intomainfrom Apr 4, 2026
Merged
feat: improve search TUI usability and detail view styling#856alishakawaguchi merged 4 commits intomainfrom
alishakawaguchi merged 4 commits intomainfrom
Conversation
Remove mouse capture (tea.WithMouseCellMotion) so terminal-native text selection and copy works. Add sectioned detail view with OVERVIEW, SOURCE, FILES headers in full detail mode, and a cleaner card preview without section headers. Fix card content width calculation for correct text wrapping inside lipgloss borders. Reduce repo column proportion to give more space to prompts. Collapse whitespace in prompt display. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Entire-Checkpoint: 83cc168b7177
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Entire-Checkpoint: 96df0f045049
… TUI Merge sectionHeader into detailTitle (identical orange 214, bold). Remove formatAuthor which became dead code after inline styled version replaced it. Remove redundant comment restating parameter names. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Entire-Checkpoint: 4cc9dfc39742
Contributor
There was a problem hiding this comment.
Pull request overview
This PR improves the entire search Bubble Tea TUI usability and formatting, focusing on better terminal-native selection/copy behavior and more readable detail layouts.
Changes:
- Removed mouse capture support to allow normal terminal text selection/copy.
- Added sectioned full-detail view (OVERVIEW/SOURCE/SNIPPET/FILES) and simplified inline detail card formatting.
- Adjusted column width proportions and prompt rendering (whitespace collapsing) to improve wrapping and space usage.
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
cmd/entire/cli/search_tui.go |
Adds sectioned detail rendering, prompt whitespace collapsing, and revises detail card width/wrapping calculations. |
cmd/entire/cli/search_tui_test.go |
Updates tests to match new column sizing/truncation behavior and removes tests for deleted helpers. |
cmd/entire/cli/search_cmd.go |
Removes tea.WithMouseCellMotion() when launching the TUI. |
.gitignore |
Ignores .superpowers/ directory. |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
evisdren
previously approved these changes
Apr 4, 2026
Fix author field blank when AuthorUsername is non-nil empty string by falling back to Author instead of using derefStr. Remove minimum border width (44) that could exceed terminal on narrow screens. Compute content width correctly in NO_COLOR mode (no border/padding). Add tests for renderDetailContent sections, author edge case, and prompt wrapping. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> Entire-Checkpoint: 6419b86ac741
Contributor
Author
|
bugbot run |
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit 43473cc. Configure here.
pjbgf
approved these changes
Apr 4, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
tea.WithMouseCellMotion) so terminal-native text selection and copy workssectionHeader/detailTitlestyles, remove deadformatAuthorTest plan
mise run fmt && mise run lint && mise run test:cientire search <query>and verify text can be selected/copied with mouseenter, verify sectioned layout with headersj/k,pgup/pgdn🤖 Generated with Claude Code
Note
Low Risk
Low risk: changes are limited to search TUI rendering/layout and Bubble Tea program options, with updated tests; no auth, networking, or data mutation logic is altered.
Overview
Improves the
entire searchinteractive TUI by disabling mouse motion capture so terminal text selection/copy works again.Refactors the detail view rendering to support sectioned full-detail output (e.g. OVERVIEW/SOURCE/SNIPPET/FILES) while keeping the inline card preview more compact, fixes wrapping/width calculations inside lipgloss borders, collapses prompt whitespace for cleaner display, and tweaks table column proportions to give more space to prompts.
Adds/updates unit tests around detail rendering, prompt collapsing, and column sizing, and updates
.gitignoreto exclude.superpowers/.Reviewed by Cursor Bugbot for commit 43473cc. Configure here.