Skip to content

Conversation

@ammar-agent
Copy link
Collaborator

Adds JSON syntax highlighting to the GenericToolCall component, making tool arguments and results easier to read.

Changes

  • Add JsonHighlight component using the existing highlightCode worker API for Shiki syntax highlighting
  • Update GenericToolCall to use JsonHighlight for displaying arguments and results
  • Add createGenericTool factory function for storybook testing
  • Add GenericToolCallWithJsonHighlighting story to demonstrate the feature

Testing

  • Run make static-check - all checks pass
  • Open Storybook and navigate to App/Chat/GenericToolCallWithJsonHighlighting, click the tool header to expand and see highlighted JSON

Generated with mux

@chatgpt-codex-connector
Copy link

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Repo admins can enable using credits for code reviews in their settings.

@ammar-agent ammar-agent force-pushed the highlight-json-output branch 29 times, most recently from bfc52fe to 7ac8239 Compare December 8, 2025 16:24
@ammar-agent ammar-agent force-pushed the highlight-json-output branch 7 times, most recently from 41de0cd to 0b23501 Compare December 8, 2025 16:32
- Add HighlightedCode component with optional line numbers
- Add JsonHighlight component for tool call display
- Update GenericToolCall to use JsonHighlight
- Add createGenericTool factory for stories
- Add GenericToolCallWithJsonHighlighting story with play function
- Update AGENTS.md with storybook best practices
@ammar-agent ammar-agent force-pushed the highlight-json-output branch from 0b23501 to b1e80eb Compare December 8, 2025 16:34
@ammario ammario merged commit 92010b0 into main Dec 8, 2025
19 checks passed
@ammario ammario deleted the highlight-json-output branch December 8, 2025 16:52
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