Results 35 comments of Thiago dos Santos Hora

> The change looks generally good to me, just a minor recommendation about the configuration. > > Please fill up the description template of this PR. I'm missing information, specially...

Fixed all issues by setting some envious. I will merge it on Monday

HI @dhvll, Thanks for your contribution. Can you add tests for it? You can check the `createAndGetCost` test setup.

Hi guys, The error was trying to manually add `RedisChatMemoryStore` to the context. I didn't realize this was done automatically, so I removed it, but the error is still there....

It's a dependency of the framework I'm using. I changed it, but the result is the same. ![Screenshot 2024-08-28 at 11 55 41](https://github.com/user-attachments/assets/53b57242-c721-4077-80c4-d6187b288e46)

> What's the value of `quarkus.langchain4j.memorystore.redis.client-name` (or perhaps your whole config)? It sounds a bit like the AI service is trying to use a redis client that does not exist......

> LGTM. In addition, we could simply disable these tests. > > They were very useful to reproduce the issue and demonstrate the fix. > > But they aren't functional...