Skip to content

Add /tools command to show the available tools#2209

Merged
dgageot merged 5 commits intodocker:mainfrom
dgageot:slash-tools
Mar 22, 2026
Merged

Add /tools command to show the available tools#2209
dgageot merged 5 commits intodocker:mainfrom
dgageot:slash-tools

Conversation

@dgageot
Copy link
Member

@dgageot dgageot commented Mar 21, 2026

  • Also stop hiding /permissions command when no permission is set
  • And add the category mcp to mcp tools

dgageot added 5 commits March 21, 2026 18:46
Extract common scrollable read-only dialog boilerplate (Init, Update, Position,
View, scrolling) into readOnlyScrollDialog. Both permissionsDialog and toolsDialog
now embed it and only provide content rendering logic.

Also unhide /permissions command so it always appears in the command palette.

Assisted-By: docker-agent
Size dialog to content (capped by max), pad scrollview output to exact
viewport height, and pin with Height+MaxHeight to prevent layout shifts.

Assisted-By: docker-agent
Use only the first line of tool descriptions to prevent newlines from
rendering as blank lines in the tool list.

Assisted-By: docker-agent
@dgageot dgageot requested a review from a team as a code owner March 21, 2026 18:28
@dgageot dgageot changed the title Add /slash command to show the available tools Add /tools command to show the available tools Mar 21, 2026
Copy link

@docker-agent docker-agent bot left a comment

Choose a reason for hiding this comment

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

Assessment: 🟢 APPROVE

No bugs found in the changed code.

This PR adds a new /tools slash command to display available tools in a TUI dialog. The implementation:

  • Introduces a reusable ReadonlyScrollDialog base component
  • Refactors the permissions dialog to use the shared base
  • Adds tool categorization support
  • Implements clean separation of concerns

The code follows established patterns in the codebase and has no detectable issues.

@dgageot dgageot merged commit 6878e0b into docker:main Mar 22, 2026
8 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.

3 participants