Skip to content

Conversation

@JoannaaKL
Copy link
Contributor

Summary

Why

To enable colours we need to send docker output to TTY.

MCP impact

  • No tool or API changes
  • Tool schema or behavior changed
  • New tool added

Security / limits

  • No security or limits impact
  • Auth / permissions considered
  • Data exposure, filtering, or token/size limits considered

Tool renaming

  • I am renaming tools as part of this PR (e.g. a part of a consolidation effort)
    • I have added the new tool aliases in deprecated_tool_aliases.go
  • I am not renaming tools as part of this PR

Note: if you're renaming tools, you must add the tool aliases. For more information on how to do so, please refer to the official docs.

Lint & tests

  • Linted locally with ./script/lint
  • Tested locally with ./script/test

Docs

  • Not needed
  • Updated (README / docs / examples)

@JoannaaKL JoannaaKL requested a review from a team as a code owner January 20, 2026 09:03
Copilot AI review requested due to automatic review settings January 20, 2026 09:03
SamMorrowDrums
SamMorrowDrums previously approved these changes Jan 20, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds the -t flag to the docker run command for the tool-search CLI utility to enable TTY support for color output. The change updates the documentation to guide users on proper Docker usage when they want colored terminal output.

Changes:

  • Added a note explaining that color output requires TTY and users should use docker run -t or -it
  • Updated the docker command example from -i to -it for the tool-search utility

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@SamMorrowDrums SamMorrowDrums merged commit 64752fd into main Jan 20, 2026
15 checks passed
@SamMorrowDrums SamMorrowDrums deleted the update-docs branch January 20, 2026 10:23
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