Skip to content

Tool calls look pretty #9344

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jul 11, 2025
Merged

Tool calls look pretty #9344

merged 2 commits into from
Jul 11, 2025

Conversation

estib-vega
Copy link
Contributor

  • Added a ToolCall parsing helper with reusable utilities for type-checking, icons, and descriptions in tool.ts.
  • Improved FeedItem visuals for a more appealing and readable UI.
  • Displayed human-friendly tool call descriptions for easier understanding.
  • Simplified FeedItem prop passing and removed FeedToolCall component.
  • Unified styling of all tool calls using FeedItemKind.
  • Added additional helpers and utilities to support future enhancements.

Extract the ToolCall parsing logic and helpers into a new file, apps/desktop/src/lib/ai/tool.ts. This provides reusable utilities for dealing with tool call props, type-checking, icons, and human-friendly descriptions.
This commit upgrades FeedItem and tool call handling to provide a more visually appealing and readable UI for tool calls and message items. Human-friendly descriptions are shown, making each tool call much easier to understand at a glance. FeedItem prop passing is simplified, FeedToolCall is removed, all tool calls use FeedItemKind for style, and code maintains unified, clear patterns. Additional helpers and utilities are added for future enhancements.
@vercel vercel bot temporarily deployed to Preview – gitbutler-web July 11, 2025 10:09 Inactive
Copy link

vercel bot commented Jul 11, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
gitbutler-components ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 11, 2025 10:10am
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
gitbutler-web ⬜️ Skipped (Inspect) Jul 11, 2025 10:10am

@estib-vega estib-vega enabled auto-merge July 11, 2025 10:09
@estib-vega estib-vega merged commit b457ebf into master Jul 11, 2025
20 checks passed
@estib-vega estib-vega deleted the feeditem-refactor-props branch July 11, 2025 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant