LLPhant functionResult messages missing 'name' parameter with OpenAI

Adding message of type "functionResult" did not work in OpenAI chat because the parameter 'name' is required. I could not test the fix with other models.

Aug 05 '24 21:08 kaeldric

Hi @kaeldric , can you provide any integration and unit tests?

Aug 06 '24 08:08 f-lombardo

hey @kaeldric, thanks for your contribution! If you can add test that would be amazing or maybe some example code in the doc to show the usage?

Aug 06 '24 09:08 MaximeThoonsen

Hi all! I wrote some tests based on my current usage, I don't know if those work for you.

About the documentation, what do you think about the Integration/Chat/OpenAIChatTest.php function? I can try to elaborate on that with the idea of providing additional context to the assistant with the result function, maybe checking the result function value.

Aug 08 '24 09:08 kaeldric

@kaeldric maybe we can use CalledFunction now it has been introduced?

Sep 04 '24 11:09 MaximeThoonsen