Skip to content

Add hide chat option to the TUI - #150

Merged
destari merged 3 commits into
control-theory:mainfrom
ItsJooL:hide-chat-pane
Sep 11, 2026
Merged

destari merged 3 commits into
control-theory:mainfrom
ItsJooL:hide-chat-pane

Conversation

@ItsJooL

@ItsJooL ItsJooL commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

I sometimes (particularly when using in K9s) would rather have the full view for the log details. It's personal preference and taste, but for me with long messages even when I have wrap enabled I find it easier to digest with the 30% extra space.

The change is opt-in so no issues for existing users.

@destari
destari requested a lite review from Copilot September 10, 2026 21:04
@destari

destari commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Ahh makes sense - will take a look!

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

The log detail modal height calculation doesn’t adjust when the tabs row is omitted (chat hidden), reducing available vertical space and likely leaving unused space in the modal.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Pull request overview

Adds an opt-in way to hide the AI chat pane in the TUI log detail modal to maximize space for log details, with a runtime toggle.

Changes:

  • Introduces --hide-chat-pane (plus config/env support) to start with the chat pane hidden.
  • Adds x keybinding (and status bar hints) to toggle chat pane visibility during a log detail modal session.
  • Updates split-modal rendering/navigation/mouse handling and adds unit tests for the new behavior.
File summaries
File Description
USAGE_GUIDE.md Documents the new hide-chat-pane option, including CLI/config/env and runtime toggle.
internal/tui/update.go Prevents modal click-based pane switching when the chat pane is hidden.
internal/tui/navigation.go Adds x toggle and disables Tab pane switching when chat is hidden.
internal/tui/navigation_test.go Adds tests for x toggle behavior and Tab behavior when chat is hidden/visible.
internal/tui/model.go Adds chatPaneVisible state (default true) and a setter used by app config.
internal/tui/modal_log_details.go Updates log detail modal rendering to support full-width details when chat is hidden.
internal/tui/modal_help.go Updates help modal with the new x shortcut for log detail modal.
internal/tui/modal_base.go Updates status bar hints to reflect chat visibility and toggling.
internal/tui/modal_base_test.go Adds tests for updated status bar behavior with chat shown/hidden.
cmd/gonzo/main.go Adds hide-chat-pane config/flag binding and env/config support via viper.
cmd/gonzo/app.go Applies HideChatPane configuration to the dashboard model at startup.
Review details
  • Files reviewed: 11/11 changed files
  • Comments generated: 2
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread internal/tui/modal_log_details.go
Comment thread internal/tui/modal_help.go Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

@destari destari left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Take a look at the one comment about the sizing

@destari destari added the enhancement New feature or request label Sep 10, 2026

@destari destari left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good! Thank you!

@destari
destari merged commit ea3a124 into control-theory:main Sep 11, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants