-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Mistral adapter error handling: Improve object validation and error messages for nested content #3784
Comments
This was a separate issue with our error logging that I believe has been resolved in latest pre-release. Thanks for the +1 on it though! |
I encountered this problem again today, and it happened on the computer where the plugin was installed for the first time. When I used Run Continue to run the plugin locally, it worked well. It was packaged into an installation package and ran well after installation in my conputer. This may be related to the session history that already exists locally. But I sent the installation package to my colleague, who installed it in IntelliJ for the first time. When he sent the problem, the plug-in window flashed for a while, and no content was displayed. It looks like the streaming request was stopped and not completed.,If sent text multiple times, an error message that the content could not be read would show. Strangely, when I cloned the latest main code test again in another directory and built it as an installation package, the stream request call was not invalid (the UI flickered and nothing was displayed), and it runs normally now. But I don’t think it’s because the problem was fixed in the latest code. The previous fix for this problem was #3725 When I pulled the code, the code was #3790 That is to say, my current code already includes the changes of #3725 |
@Patrick-Erichsen After the first request stream was invalid for unknown reasons (nothing was displayed, the window flickered), and then I sent text again, the reading content error message appeared. ( I'll try to record a video later ) It seems that something strange happened when I asked the question for the first time. After that, plugin want to update this history record, but it could not be updated : In the plugin window after installation, I found the dev tools tool and saw the following error message when the reading content error: According to the log information, I found the code: |
In fact, I had this problem in the earlier 0.83 version: the stream request seemed to be stopped instantly (the UI flickered and nothing was displayed) Then I changed the directory to clone the code again, and ran it again, and there was no problem. I think this may be related to the running/packaging environment, and it seems not to be a problem with the code itself. I think I should create a new issue to record this problem detail , because the root cause of the problem may not be that the content cannot be read, but it is just an incidental issue |
I tested and the issue no longer occurs. I get a very nice error message now 😃 |
@Patrick-Erichsen Thanks for your support! @AfterStories Can you please open your comments in another issue, so we keep things nice and clean? |
Before submitting your bug report
Relevant environment info
Description
The Mistral adapter for Codestral (And properly other models as well) is not displaying a proper error.
Instead the not very helpful "Cannot read properties of undefined (reading 'content')" is displayed. IMHO the code should handle deep nested objects more gracefully and e.g. display a JSON stringify of the object if the object nesting is different than expected.
To reproduce
Log output
The text was updated successfully, but these errors were encountered: