fix: 🐛 Fix Bedrock Converse's tool use blocks, when there are multipl…

…e consecutive function calls (#14386)

llama-index-integrations/llms/llama-index-llms-bedrock-converse/llama_index/llms/bedrock_converse/utils.py

@@ -120,28 +120,39 @@ def messages_to_converse_messages(
             status = message.additional_kwargs.get("status")
             if status:
                 content["toolResult"]["status"] = status
-            converse_message = {
-                "role": "user",
-                "content": [content],
-            }
-            converse_messages.append(converse_message)
+            converse_messages.append(
+                {
+                    "role": "user",
+                    "content": [content],
+                }
+            )
         else:
-            content = []
             if message.content:
                 # get the text of the message
-                content.append({"text": message.content})
-            # convert tool calls to the AWS Bedrock Converse format
-            tool_calls = message.additional_kwargs.get("tool_calls", [])
-            for tool_call in tool_calls:
-                assert "toolUseId" in tool_call, f"`toolUseId` not found in {tool_call}"
-                assert "input" in tool_call, f"`input` not found in {tool_call}"
-                assert "name" in tool_call, f"`name` not found in {tool_call}"
-                content.append({"toolUse": tool_call})
-            converse_message = {
-                "role": message.role.value,
-                "content": content,
-            }
-            converse_messages.append(converse_message)
+                converse_messages.append(
+                    {
+                        "role": message.role.value,
+                        "content": [{"text": message.content}],
+                    }
+                )
+        # convert tool calls to the AWS Bedrock Converse format
+        # NOTE tool calls might show up within any message,
+        # e.g. within assistant message or in consecutive tool calls,
+        # thus this tool call check is done for all messages
+        tool_calls = message.additional_kwargs.get("tool_calls", [])
+        content = []
+        for tool_call in tool_calls:
+            assert "toolUseId" in tool_call, f"`toolUseId` not found in {tool_call}"
+            assert "input" in tool_call, f"`input` not found in {tool_call}"
+            assert "name" in tool_call, f"`name` not found in {tool_call}"
+            content.append({"toolUse": tool_call})
+        if len(content) > 0:
+            converse_messages.append(
+                {
+                    "role": "assistant",  # tool calls are always from the assistant
+                    "content": content,
+                }
+            )
 
     return __merge_common_role_msgs(converse_messages), system_prompt.strip()