Skip to content

Conversation

marckohlbrugge
Copy link
Contributor

@marckohlbrugge marckohlbrugge commented Sep 15, 2025

Summary

  • Added visualization of tool calls in the chat UI message partial
  • Tool calls are displayed with function name and arguments in JSON format
  • Styled with monospace font and gray background for better readability

Test plan

  • Generate a chat UI with the updated template
  • Verify tool calls are displayed correctly in messages
  • Check that messages without tool calls render normally
CleanShot 2025-09-21 at 22 21 23@2x

🤖 Generated with Claude Code

Added visualization of tool calls in the chat UI message partial. Tool calls are now displayed in a monospace font with a gray background, showing the function name and arguments in JSON format.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
marckohlbrugge and others added 3 commits September 21, 2025 22:04
- Created _tool_calls.html.erb.tt partial for reusable tool call rendering
- Updated arguments display to use Ruby method call syntax (key: value) instead of JSON
- Aligned render statement with codebase conventions (double quotes)

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
- Add tool_call_model_name method to generator for dynamic model naming
- Generate tool_calls partial with dynamic filename based on model configuration
- Update message template to use dynamic partial path for tool calls

This allows the generator to properly handle tool call displays when
generating the chat UI, supporting both default and custom model names.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
- Kept tool_calls display functionality from our branch
- Integrated content div ID from upstream changes
- Both _tool_calls and _content partials are now included
Copy link

codecov bot commented Sep 22, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.58%. Comparing base (46ac613) to head (700987a).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #416   +/-   ##
=======================================
  Coverage   84.58%   84.58%           
=======================================
  Files          37       37           
  Lines        1933     1933           
  Branches      500      500           
=======================================
  Hits         1635     1635           
  Misses        298      298           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@crmne crmne merged commit a74233f into crmne:main Sep 22, 2025
15 of 27 checks passed
@marckohlbrugge marckohlbrugge deleted the marc/show-tool-calls-in-messages branch September 22, 2025 13:27
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.

2 participants