Skip to content
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

Fix ToolResult display for complex values #80

Merged
merged 1 commit into from
Aug 28, 2024

Conversation

AtlantisPleb
Copy link
Contributor

This PR addresses issue #78: ToolResult shows malformed data.

Changes made:

  • Updated the getToolParams function in panes/chat/ToolResult.tsx to handle different types of values more appropriately.
  • Instead of showing '[complex value]' for non-string values, it now displays:
    • Actual values for strings, numbers, and booleans
    • '[Array]' for arrays
    • '{Object}' for objects
    • 'null' for null values
    • The type of the value for any other cases

This change will provide more informative and accurate displays of tool parameters in the UI.

Copy link

vercel bot commented Aug 28, 2024

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

Name Status Preview Comments Updated (UTC)
v2 ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 28, 2024 1:39am

@AtlantisPleb AtlantisPleb merged commit a7caa3b into main Aug 28, 2024
2 of 3 checks passed
@AtlantisPleb AtlantisPleb deleted the issue-78-fix-toolresult-display branch August 28, 2024 01:41
Copy link

The Vercel build for this PR has failed. Please check the Vercel deployment logs for more information.

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.

1 participant