Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@peterj
Copy link
Collaborator

@peterj peterj commented Oct 27, 2025

No description provided.

@peterj peterj marked this pull request as ready for review October 27, 2025 14:29
Copilot AI review requested due to automatic review settings October 27, 2025 14:29
@peterj peterj requested a review from yuval-k as a code owner October 27, 2025 14:29
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 functionality to display tool calls and tool results in the TUI chat interface. The changes enhance the chat display by showing when the agent invokes tools and their results, rather than only showing text messages.

Key Changes:

  • Added new styling functions for tool calls and tool results with color-coded output
  • Refactored message processing to handle multiple message part types (text, tool calls, tool results)
  • Replaced direct text extraction with a unified message parts handler

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
go/cli/internal/tui/theme/colors.go Added style functions for tool calls, tool results, and dimmed text with emerald and sky color schemes
go/cli/internal/tui/chat.go Refactored message display logic into handleMessageParts to process and display tool calls, tool results, and text content

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@peterj peterj marked this pull request as draft October 27, 2025 14:30
@peterj peterj force-pushed the peterj/displaytoolcalls branch from f637feb to f54bd18 Compare October 27, 2025 19:21
Signed-off-by: Peter Jausovec <[email protected]>
@peterj peterj force-pushed the peterj/displaytoolcalls branch from f54bd18 to afb0797 Compare October 27, 2025 19:23
@peterj peterj marked this pull request as ready for review October 27, 2025 19:23
@peterj
Copy link
Collaborator Author

peterj commented Oct 27, 2025

fixes #1048

@peterj peterj merged commit 0c69c8b into main Oct 28, 2025
27 of 28 checks passed
@peterj peterj deleted the peterj/displaytoolcalls branch October 28, 2025 13:05
killjoycircuit pushed a commit to killjoycircuit/kagent that referenced this pull request Nov 1, 2025
Signed-off-by: Peter Jausovec <[email protected]>
Signed-off-by: killjoycircuit <[email protected]>
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