agentle 0.9.34__py3-none-any.whl → 0.9.36__py3-none-any.whl

agentle/agents/whatsapp/whatsapp_bot.py

@@ -2679,11 +2679,25 @@ class WhatsAppBot[T_Schema: WhatsAppResponseBase = WhatsAppResponseBase](BaseMod
             return "mp3"  # default
 
     def _split_message_by_line_breaks(self, text: str) -> Sequence[str]:
-        """Split message by line breaks first, then by length if needed with enhanced validation."""
+        """Split message by line breaks first, then by length if needed with enhanced validation.
+
+        CRITICAL: This method must preserve line breaks within messages for proper WhatsApp formatting.
+        """
         if not text or not text.strip():
             return ["[Mensagem vazia]"]  # Portuguese: "Empty message"
 
         try:
+            # Check if entire text fits in one message - if so, return it as-is
+            if len(text) <= self.config.max_message_length:
+                logger.debug(
+                    f"[SPLIT_MESSAGE] Message fits in single message ({len(text)} chars), returning as-is"
+                )
+                return [text]
+
+            logger.info(
+                f"[SPLIT_MESSAGE] Message too long ({len(text)} chars), splitting into multiple messages"
+            )
+
             # First split by double line breaks (paragraphs)
             paragraphs = text.split("\n\n")
             messages: MutableSequence[str] = []
@@ -2692,38 +2706,54 @@ class WhatsAppBot[T_Schema: WhatsAppResponseBase = WhatsAppResponseBase](BaseMod
                 if not paragraph.strip():
                     continue
 
+                # If paragraph fits, keep it intact with all line breaks
+                if len(paragraph) <= self.config.max_message_length:
+                    messages.append(paragraph)
+                    continue
+
+                # Paragraph is too long - need to split it
                 # Check if paragraph is a list (has list markers)
                 lines = paragraph.split("\n")
                 is_list_paragraph = self._is_list_content(lines)
 
                 if is_list_paragraph:
-                    # Group list items together instead of splitting each line
-                    # IMPORTANT: Keep line breaks intact for list formatting
+                    # Group list items together, preserving line breaks
                     grouped_list = self._group_list_items(lines)
                     messages.extend(grouped_list)
                 else:
-                    # For non-list paragraphs, keep the whole paragraph together if possible
-                    paragraph_text = paragraph.strip()
-                    if not paragraph_text:
-                        continue
-
-                    # Check if paragraph fits within message length limits
-                    if len(paragraph_text) <= self.config.max_message_length:
-                        messages.append(paragraph_text)
-                    else:
-                        # Split by individual lines only if paragraph is too long
-                        for line in lines:
-                            line = line.strip()
-                            if not line:
-                                continue
+                    # For non-list paragraphs, try to keep lines together
+                    current_chunk = ""
+                    for line in lines:
+                        if not line.strip():
+                            # Preserve empty lines for spacing
+                            if current_chunk:
+                                current_chunk += "\n"
+                            continue
+
+                        # Try adding this line to current chunk
+                        test_chunk = (
+                            current_chunk + ("\n" if current_chunk else "") + line
+                        )
 
-                            # Check if this line fits within message length limits
-                            if len(line) <= self.config.max_message_length:
-                                messages.append(line)
-                            else:
-                                # Split long lines by length
+                        if len(test_chunk) <= self.config.max_message_length:
+                            current_chunk = test_chunk
+                        else:
+                            # Current chunk is full, save it
+                            if current_chunk:
+                                messages.append(current_chunk)
+
+                            # Check if single line is too long
+                            if len(line) > self.config.max_message_length:
+                                # Split long line by length
                                 split_lines = self._split_long_line(line)
                                 messages.extend(split_lines)
+                                current_chunk = ""
+                            else:
+                                current_chunk = line
+
+                    # Add remaining chunk
+                    if current_chunk:
+                        messages.append(current_chunk)
 
             # Filter out empty messages and validate
             final_messages = []

agentle/generations/providers/openrouter/_adapters/agentle_message_to_openrouter_message_adapter.py

@@ -29,14 +29,17 @@ from agentle.generations.providers.openrouter._types import (
     OpenRouterMessage,
     OpenRouterSystemMessage,
     OpenRouterToolCall,
+    OpenRouterToolMessage,
     OpenRouterUserMessage,
 )
+from agentle.generations.tools.tool_execution_result import ToolExecutionResult
+import json
 
 
 class AgentleMessageToOpenRouterMessageAdapter(
     Adapter[
         AssistantMessage | DeveloperMessage | UserMessage,
-        OpenRouterMessage,
+        OpenRouterMessage | list[OpenRouterMessage],
     ]
 ):
     """
@@ -44,15 +47,18 @@ class AgentleMessageToOpenRouterMessageAdapter(
 
     Handles conversion of:
     - DeveloperMessage -> OpenRouterSystemMessage
-    - UserMessage -> OpenRouterUserMessage
+    - UserMessage -> OpenRouterUserMessage (or OpenRouterToolMessage if contains tool results)
     - AssistantMessage -> OpenRouterAssistantMessage (with tool calls)
+
+    Note: When a message contains ToolExecutionResult parts, they are extracted
+    and returned as separate OpenRouterToolMessage objects.
     """
 
     @override
     def adapt(
         self,
         _f: AssistantMessage | DeveloperMessage | UserMessage,
-    ) -> OpenRouterMessage:
+    ) -> OpenRouterMessage | list[OpenRouterMessage]:
         """
         Convert an Agentle message to OpenRouter format.
 
@@ -60,7 +66,9 @@ class AgentleMessageToOpenRouterMessageAdapter(
             _f: The Agentle message to convert.
 
         Returns:
-            The corresponding OpenRouter message.
+            The corresponding OpenRouter message(s). Returns a list when the message
+            contains ToolExecutionResult parts that need to be split into separate
+            tool messages.
         """
         message = _f
         part_adapter = AgentlePartToOpenRouterPartAdapter()
@@ -76,12 +84,28 @@ class AgentleMessageToOpenRouterMessageAdapter(
                 )
 
             case UserMessage():
+                # Check if this message contains tool execution results
+                tool_results = [
+                    p for p in message.parts if isinstance(p, ToolExecutionResult)
+                ]
+
+                if tool_results:
+                    # Convert each tool result to a separate tool message
+                    return [
+                        OpenRouterToolMessage(
+                            role="tool",
+                            tool_call_id=result.suggestion.id,
+                            content=self._serialize_tool_result(result.result),
+                        )
+                        for result in tool_results
+                    ]
+
                 # User messages can have multimodal content
-                # Filter out non-content parts (like tool execution suggestions)
+                # Filter out non-content parts (like tool execution suggestions and results)
                 content_parts = [
                     p
                     for p in message.parts
-                    if not isinstance(p, ToolExecutionSuggestion)
+                    if not isinstance(p, (ToolExecutionSuggestion, ToolExecutionResult))
                 ]
 
                 # If only text parts, concatenate into a string
@@ -102,6 +126,69 @@ class AgentleMessageToOpenRouterMessageAdapter(
                 )
 
             case AssistantMessage():
+                # Check if this message contains tool execution results
+                tool_results = [
+                    p for p in message.parts if isinstance(p, ToolExecutionResult)
+                ]
+
+                if tool_results:
+                    # If assistant message has tool results, we need to split it
+                    # First, create the assistant message with tool calls (if any)
+                    messages: list[OpenRouterMessage] = []
+
+                    # Separate text content from tool calls
+                    text_parts = [p for p in message.parts if isinstance(p, TextPart)]
+                    tool_suggestions = [
+                        p
+                        for p in message.parts
+                        if isinstance(p, ToolExecutionSuggestion)
+                    ]
+
+                    # Only create assistant message if there's content or tool calls
+                    if text_parts or tool_suggestions:
+                        content = (
+                            "".join(str(p) for p in text_parts) if text_parts else None
+                        )
+
+                        tool_calls: list[OpenRouterToolCall] = [
+                            OpenRouterToolCall(
+                                id=suggestion.id,
+                                type="function",
+                                function={
+                                    "name": suggestion.tool_name,
+                                    "arguments": self._serialize_tool_arguments(
+                                        suggestion.args
+                                    ),
+                                },
+                            )
+                            for suggestion in tool_suggestions
+                        ]
+
+                        assistant_msg = OpenRouterAssistantMessage(
+                            role="assistant",
+                            content=content,
+                        )
+
+                        if tool_calls:
+                            assistant_msg["tool_calls"] = tool_calls
+
+                        if hasattr(message, "reasoning") and message.reasoning:
+                            assistant_msg["reasoning"] = message.reasoning
+
+                        messages.append(assistant_msg)
+
+                    # Add tool result messages
+                    for result in tool_results:
+                        messages.append(
+                            OpenRouterToolMessage(
+                                role="tool",
+                                tool_call_id=result.suggestion.id,
+                                content=self._serialize_tool_result(result.result),
+                            )
+                        )
+
+                    return messages
+
                 # Separate text content from tool calls
                 text_parts = [p for p in message.parts if isinstance(p, TextPart)]
                 tool_suggestions = [
@@ -118,7 +205,9 @@ class AgentleMessageToOpenRouterMessageAdapter(
                         type="function",
                         function={
                             "name": suggestion.tool_name,
-                            "arguments": str(suggestion.args),  # Should be JSON string
+                            "arguments": self._serialize_tool_arguments(
+                                suggestion.args
+                            ),
                         },
                     )
                     for suggestion in tool_suggestions
@@ -137,3 +226,34 @@ class AgentleMessageToOpenRouterMessageAdapter(
                     result["reasoning"] = message.reasoning
 
                 return result
+
+    def _serialize_tool_arguments(self, args: object) -> str:
+        """
+        Serialize tool arguments to JSON string.
+
+        Args:
+            args: The arguments to serialize.
+
+        Returns:
+            JSON string representation of the arguments.
+        """
+        if isinstance(args, str):
+            return args
+        return json.dumps(args)
+
+    def _serialize_tool_result(self, result: object) -> str:
+        """
+        Serialize tool execution result to string.
+
+        Args:
+            result: The result to serialize.
+
+        returns:
+            String representation of the result.
+        """
+        if isinstance(result, str):
+            return result
+        try:
+            return json.dumps(result)
+        except (TypeError, ValueError):
+            return str(result)

agentle/generations/providers/openrouter/openrouter_generation_provider.py

@@ -71,6 +71,7 @@ from agentle.generations.providers.openrouter._types import (
     OpenRouterFileParserPlugin,
     OpenRouterModelsResponse,
     OpenRouterModel,
+    OpenRouterMessage,
 )
 from agentle.generations.providers.openrouter.error_handler import (
     parse_and_raise_openrouter_error,
@@ -1247,10 +1248,14 @@ class OpenRouterGenerationProvider(GenerationProvider):
                     ),
                 )
 
-        # Convert messages
-        openrouter_messages = [
-            self.message_adapter.adapt(message) for message in messages_list
-        ]
+        # Convert messages - adapter may return single message or list of messages
+        openrouter_messages: list[OpenRouterMessage] = []
+        for message in messages_list:
+            adapted = self.message_adapter.adapt(message)
+            if isinstance(adapted, list):
+                openrouter_messages.extend(adapted)
+            else:
+                openrouter_messages.append(adapted)
 
         # Convert tools if provided
         openrouter_tools = (
@@ -1404,10 +1409,14 @@ class OpenRouterGenerationProvider(GenerationProvider):
         """
         _generation_config = self._normalize_generation_config(generation_config)
 
-        # Convert messages
-        openrouter_messages = [
-            self.message_adapter.adapt(message) for message in messages
-        ]
+        # Convert messages - adapter may return single message or list of messages
+        openrouter_messages: list[OpenRouterMessage] = []
+        for message in messages:
+            adapted = self.message_adapter.adapt(message)
+            if isinstance(adapted, list):
+                openrouter_messages.extend(adapted)
+            else:
+                openrouter_messages.append(adapted)
 
         # Convert tools if provided
         openrouter_tools = (

{agentle-0.9.34.dist-info → agentle-0.9.36.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: agentle
-Version: 0.9.34
+Version: 0.9.36
 Summary: ...
 Author-email: Arthur Brenno <64020210+arthurbrenno@users.noreply.github.com>
 License-File: LICENSE

{agentle-0.9.34.dist-info → agentle-0.9.36.dist-info}/RECORD

@@ -137,7 +137,7 @@ agentle/agents/ui/__init__.py,sha256=IjHRV0k2DNwvFrEHebmsXiBvmITE8nQUnsR07h9tVkU
 agentle/agents/ui/streamlit.py,sha256=9afICL0cxtG1o2pWh6vH39-NdKiVfADKiXo405F2aB0,42829
 agentle/agents/whatsapp/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 agentle/agents/whatsapp/human_delay_calculator.py,sha256=BGCDeoNTPsMn4d_QYmG0BWGCG8SiUJC6Fk295ulAsAk,18268
-agentle/agents/whatsapp/whatsapp_bot.py,sha256=RH4WDpaqD4yTazKkjyYYPlhb-3BA-sX8h4pWZWHWznU,163775
+agentle/agents/whatsapp/whatsapp_bot.py,sha256=tF35s2c4G9Fo0bLVmePYXWSnSNEgg-Rpi9V0MrrRCCA,164948
 agentle/agents/whatsapp/models/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 agentle/agents/whatsapp/models/audio_message.py,sha256=kUqG1HdNW6DCYD-CqscJ6WHlAyv9ufmTSKMdjio9XWk,2705
 agentle/agents/whatsapp/models/context_info.py,sha256=sk80KuNE36S6VRnLh7n6UXmzZCXIB4E4lNxnRyVizg8,563
@@ -355,9 +355,9 @@ agentle/generations/providers/openrouter/__init__.py,sha256=47DEQpj8HBSa-_TImW-5
 agentle/generations/providers/openrouter/_types.py,sha256=VSAf1auxopTqhYwLHPWl_Q0-okWUfLKJa0JmWlAFuGg,10557
 agentle/generations/providers/openrouter/error_handler.py,sha256=4qm8v_cjdrB-59UXyCJLnIkOxzIfmsltZY8Q137-8Qg,8075
 agentle/generations/providers/openrouter/exceptions.py,sha256=o3_-tuyhewc0v5L2cAXH0f9ixHyyDgZbHq0KX5cUyPE,23179
-agentle/generations/providers/openrouter/openrouter_generation_provider.py,sha256=c2fQMdm-xFzIR2Z3ClAXiu6BzKfrnQQTpzX2jJQSNlI,66596
+agentle/generations/providers/openrouter/openrouter_generation_provider.py,sha256=XhXU_Ix10jmWl2SzRUHwsVSPRIWq6zHnia7IMaY6Yy4,67129
 agentle/generations/providers/openrouter/_adapters/__init__.py,sha256=orgZeEBqH4X_cpyOMiClvfZHY5cLwLNqhAYLqNjGIH4,1826
-agentle/generations/providers/openrouter/_adapters/agentle_message_to_openrouter_message_adapter.py,sha256=xqLumYTs__1UHD1_WwvHsHhzpEUKOAwMAj4pvo4BWMU,4957
+agentle/generations/providers/openrouter/_adapters/agentle_message_to_openrouter_message_adapter.py,sha256=IvaovVP0ChYEreysiDyqF8SXdXxnD_Y9MjA2raEoncw,9790
 agentle/generations/providers/openrouter/_adapters/agentle_part_to_openrouter_part_adapter.py,sha256=eIfwQQ9BokHy3FQ90GJ4i_J3L46XCiSBd1RWnzu-gAo,5967
 agentle/generations/providers/openrouter/_adapters/agentle_tool_to_openrouter_tool_adapter.py,sha256=41i3B6awaTNZsdQ46Oi1e10cuNhQqWL-KBJ5V_sHkiI,9547
 agentle/generations/providers/openrouter/_adapters/openrouter_message_to_generated_assistant_message_adapter.py,sha256=uB4TYc0fuwsoYdRnEnLhbwQISyaZ2Z2RWkPFGQXUc80,5295
@@ -1018,7 +1018,7 @@ agentle/web/actions/scroll.py,sha256=WqVVAORNDK3BL1oASZBPmXJYeSVkPgAOmWA8ibYO82I
 agentle/web/actions/viewport.py,sha256=KCwm88Pri19Qc6GLHC69HsRxmdJz1gEEAODfggC_fHo,287
 agentle/web/actions/wait.py,sha256=IKEywjf-KC4ni9Gkkv4wgc7bY-hk7HwD4F-OFWlyf2w,571
 agentle/web/actions/write_text.py,sha256=9mxfHcpKs_L7BsDnJvOYHQwG8M0GWe61SRJAsKk3xQ8,748
-agentle-0.9.34.dist-info/METADATA,sha256=RGHCMaogcrdNvsf-g95zBnU_RdGssPcmfltQ4440Q9I,86849
-agentle-0.9.34.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
-agentle-0.9.34.dist-info/licenses/LICENSE,sha256=T90S9vqRS6qP-voULxAcvwEs558wRRo6dHuZrjgcOUI,1085
-agentle-0.9.34.dist-info/RECORD,,
+agentle-0.9.36.dist-info/METADATA,sha256=hSUSyQ9vk4kdTs91_EW7Yb1tEi41flD2JGVlqugY4YA,86849
+agentle-0.9.36.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
+agentle-0.9.36.dist-info/licenses/LICENSE,sha256=T90S9vqRS6qP-voULxAcvwEs558wRRo6dHuZrjgcOUI,1085
+agentle-0.9.36.dist-info/RECORD,,