Merge pull request #554 from taro1025/fix-readme

docs(readme): Move 'messages << message' outside of loop to prevent duplicatentries

Author: alexrudall

README.md

@@ -682,6 +682,10 @@ response =
 message = response.dig("choices", 0, "message")
 
 if message["role"] == "assistant" && message["tool_calls"]
+
+  # For a subsequent message with the role "tool", OpenAI requires the preceding message to have a tool_calls argument.
+  messages << message
+
   message["tool_calls"].each do |tool_call|
     tool_call_id = tool_call.dig("id")
     function_name = tool_call.dig("function", "name")
@@ -697,9 +701,6 @@ if message["role"] == "assistant" && message["tool_calls"]
         # decide how to handle
       end
 
-    # For a subsequent message with the role "tool", OpenAI requires the preceding message to have a tool_calls argument.
-    messages << message
-
     messages << {
       tool_call_id: tool_call_id,
       role: "tool",