fix: Bug OpenAI AssistantMessage (#511)

* Creation of the ete test as well as fixing the empty tools array bug.

* Editing the release notes as well as index.html file of fix: Bug OpenAI AssistantMessage #511 pull request.

* Adding the messagesHistory method in the OpenAiController.

* Adding the messagesHistory method in the OpenAiController.

* Formatting

* Moving created methods and tests to v2.

* Formatting

* Moving created methods and tests to v2.

---------

Co-authored-by: SAP Cloud SDK Bot <[email protected]>

Author: n-o-u-r-h-a-n

docs/release_notes.md

@@ -24,4 +24,4 @@
 
 ### 🐛 Fixed Issues
 
--
+- OpenAi: Fix AssistantMessage Bug by now being able to send Assistant Messages using our API Client. 

foundation-models/openai/src/main/java/com/sap/ai/sdk/foundationmodels/openai/OpenAiAssistantMessage.java

@@ -72,7 +72,8 @@ public class OpenAiAssistantMessage implements OpenAiMessage {
   ChatCompletionRequestAssistantMessage createChatCompletionRequestMessage() {
     final var message =
         new ChatCompletionRequestAssistantMessage()
-            .role(ChatCompletionRequestAssistantMessage.RoleEnum.fromValue(role()));
+            .role(ChatCompletionRequestAssistantMessage.RoleEnum.fromValue(role()))
+            .toolCalls(null);
 
     final var items = content().items();
     if (!items.isEmpty() && items.get(0) instanceof OpenAiTextItem textItem) {

sample-code/spring-app/src/main/java/com/sap/ai/sdk/app/services/OpenAiServiceV2.java

@@ -19,6 +19,7 @@
 import java.util.stream.Stream;
 import javax.annotation.Nonnull;
 import lombok.extern.slf4j.Slf4j;
+import lombok.val;
 import org.springframework.stereotype.Service;
 
 /** Service class for OpenAI service using latest convenience api */
@@ -37,6 +38,28 @@ public OpenAiChatCompletionResponse chatCompletion(@Nonnull final String prompt)
         .chatCompletion(new OpenAiChatCompletionRequest(prompt));
   }
 
+  /**
+   * Chat requests to OpenAI and updating the messages history
+   *
+   * @param previousMessage The request to send to the assistant
+   * @return the assistant message response
+   */
+  @Nonnull
+  public OpenAiChatCompletionResponse messagesHistory(@Nonnull final String previousMessage) {
+    val messagesList = new ArrayList<OpenAiMessage>();
+    messagesList.add(OpenAiMessage.user(previousMessage));
+
+    final OpenAiChatCompletionResponse result =
+        OpenAiClient.forModel(GPT_4O_MINI)
+            .chatCompletion(new OpenAiChatCompletionRequest(messagesList));
+
+    messagesList.add(result.getMessage());
+    messagesList.add(OpenAiMessage.user("What is the typical food there?"));
+
+    return OpenAiClient.forModel(GPT_4O_MINI)
+        .chatCompletion(new OpenAiChatCompletionRequest(messagesList));
+  }
+
   /**
    * Asynchronous stream of an OpenAI chat request
    *

sample-code/spring-app/src/test/java/com/sap/ai/sdk/app/controllers/OpenAiV2Test.java

@@ -33,6 +33,11 @@ void chatCompletion() {
     assertThat(completion.getContent()).isNotEmpty();
   }
 
+  @Test
+  void testMessagesHistory() {
+    assertThat(service.messagesHistory("What is the capital of France?").getContent()).isNotEmpty();
+  }
+
   @Test
   void chatCompletionImage() {
     final var completion =