search

quarkiverse / quarkus-langchain4j

Quarkus Langchain4j extension
https://docs.quarkiverse.io/quarkus-langchain4j/dev/index.html
Apache License 2.0
121 stars 66 forks source link

fix: 🐛 fix NPE if chunk is null for Mistral AI #510

Closed philippart-s closed 2 months ago

philippart-s commented 2 months ago

fix #489

@geoand as discuss, the PR to fix the NPE in case of no data in content field in Mistral response.

I didn't find tests in Mistral package but I tested on my project and it fixes the bug.

Let me know if I need to write a test somewhere 😉

geoand commented 2 months ago

Thanks!

Unfortunately SSE is really hard to test in unit tests, so for now let's not worry about it