-
Notifications
You must be signed in to change notification settings - Fork 3k
Handle JSON Parsing errors #5995
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
Conversation
Your cubic subscription is currently inactive. Please reactivate your subscription to receive AI reviews and use cubic. |
✅ Deploy Preview for continuedev ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm assuming this PR fixes some bugs, could you add relevant tests?
Added tests in |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for adding the test 💪
Handles invalid tool call args parsing (caused by when JSON tool arg streaming is truncated)
Also adds some try/catch handling for other JSON parsing throughout the repo to make it more robust
A followup will be to properly cancel tool call messages by removing invalid args on cancellation
Related discussion here #5662