Skip to content

feat(tui): restore tool detail visibility controls - #49474

Closed
parzival1l wants to merge 1 commit into
anomalyco:v2from
parzival1l:tool-visibility
Closed

parzival1l wants to merge 1 commit into
anomalyco:v2from
parzival1l:tool-visibility

Conversation

@parzival1l

@parzival1l parzival1l commented Sep 17, 2026 •

Copy link
Copy Markdown

Issue for this PR

Closes #45872

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

Large completed tool outputs can make long TUI sessions difficult to read. This restores the V1-style ability to hide completed tool details while keeping running and failed tools visible.

It adds a persistent session.tool_details preference plus a bindable session.toggle.tool_details command. The command is intentionally unbound by default so it does not conflict with the existing <leader>i image shortcut.

PR #48300 overlaps with this change and provides broader show / minimal / hide modes. This PR is the narrower option focused on restoring the show/hide control and keyboard binding requested in #45872; I am happy for maintainers to fold these pieces into #48300 if that direction is preferred.

How did you verify your code works?

  • bun test test/config-v2.test.tsx test/keybind.test.ts: 20 passed, 0 failed.
  • bun run lint: 0 warnings and 0 errors.
  • Full pre-push bun run typecheck: 35 tasks passed.
  • The side-by-side patched client was connected to the user's existing V2 server and sessions without changing server or session data.
  • The user verified the problem and reviewed the resulting behavior and proposed control.

To verify manually, set session.tool_details to hide, or bind and invoke session.toggle.tool_details, in a session containing completed, running, and failed tool calls. Completed calls should disappear; running and failed calls should remain visible.

Screenshots / recordings

Not attached yet. The change restores the previous Hide tool details behavior without otherwise changing the TUI layout. A recording can be added if maintainers want one.

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

AI disclosure: This PR was created with OpenAI gpt-5.6-sol (medium variant) through the OpenCode V2 harness. The user directed the implementation and reviewed and verified the result.

@github-actions github-actions Bot added the needs:compliance This means the issue will auto-close after 2 hours. label Sep 17, 2026
@github-actions

Copy link
Copy Markdown
Contributor

The following comment was made by an LLM, it may be inaccurate:

Based on my search results, I found one potentially related PR:

Related PR:

@github-actions github-actions Bot removed the needs:compliance This means the issue will auto-close after 2 hours. label Sep 17, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Thanks for updating your PR! It now meets our contributing guidelines. 👍

@parzival1l

parzival1l commented Sep 17, 2026 •

Copy link
Copy Markdown
Author

I saw the maintainer's comment on the referenced PR #48300. Closing this PR to avoid duplicate work.

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