-
Notifications
You must be signed in to change notification settings - Fork 332
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
feat(provider): initial error handling on claude #221
Conversation
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 think we should have a providers.on_error
function instead of this.
whipped up what you suggested. Was gonna add a default case for |
65abb27
to
7ca4238
Compare
7ca4238
to
d680f95
Compare
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.
let's do it iteratively. I think this is a good start
Wasn't the only one running into this issue and it's more prominent due to how these companies are handling their API usage now. Can see an example in #117. Will likely have to improve this at some point and support more cases, but these seem to be the two most common problems that people run into with no feedback