openai[patch]: Fix tool content list (#5756)

* openai[patch]: Fix tool content list * same for func calls * chore: lint files * drop console log
langchain-ai · Jun 13, 2024 · b1cef30 · b1cef30
1 parent 308e5b2
commit b1cef30
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 6 deletions.
diff --git a/libs/langchain-openai/src/chat_models.ts b/libs/langchain-openai/src/chat_models.ts
@@ -228,11 +228,13 @@ function convertMessagesToOpenAIParams(messages: BaseMessage[]) {
     }
     if (message.additional_kwargs.function_call != null) {
       completionParam.function_call = message.additional_kwargs.function_call;
+      completionParam.content = null;
     }
     if (isAIMessage(message) && !!message.tool_calls?.length) {
       completionParam.tool_calls = message.tool_calls.map(
         convertLangChainToolCallToOpenAI
       );
+      completionParam.content = null;
     } else {
       if (message.additional_kwargs.tool_calls != null) {
         completionParam.tool_calls = message.additional_kwargs.tool_calls;

diff --git a/libs/langchain-openai/src/tests/chat_models.standard.int.test.ts b/libs/langchain-openai/src/tests/chat_models.standard.int.test.ts
@@ -24,12 +24,6 @@ class ChatOpenAIStandardIntegrationTests extends ChatModelIntegrationTests<
     });
   }
 
-  async testToolMessageHistoriesListContent() {
-    console.warn(
-      "ChatOpenAI testToolMessageHistoriesListContent test known failure. Skipping..."
-    );
-  }
-
   async testUsageMetadataStreaming() {
     // ChatOpenAI does not support streaming tokens by
     // default, so we must pass in a call option to