openhands-sdk 1.7.2__tar.gz → 1.7.3__tar.gz

{openhands_sdk-1.7.2 → openhands_sdk-1.7.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: openhands-sdk
-Version: 1.7.2
+Version: 1.7.3
 Summary: OpenHands SDK - Core functionality for building AI agents
 Requires-Python: >=3.12
 Requires-Dist: deprecation>=2.1.0

{openhands_sdk-1.7.2 → openhands_sdk-1.7.3}/openhands/sdk/agent/agent.py

@@ -502,16 +502,29 @@ class Agent(AgentBase):
             )
 
         # Execute actions!
-        if should_enable_observability():
-            tool_name = extract_action_name(action_event)
-            observation: Observation = observe(name=tool_name, span_type="TOOL")(tool)(
-                action_event.action, conversation
+        try:
+            if should_enable_observability():
+                tool_name = extract_action_name(action_event)
+                observation: Observation = observe(name=tool_name, span_type="TOOL")(
+                    tool
+                )(action_event.action, conversation)
+            else:
+                observation = tool(action_event.action, conversation)
+            assert isinstance(observation, Observation), (
+                f"Tool '{tool.name}' executor must return an Observation"
             )
-        else:
-            observation = tool(action_event.action, conversation)
-        assert isinstance(observation, Observation), (
-            f"Tool '{tool.name}' executor must return an Observation"
-        )
+        except ValueError as e:
+            # Tool execution raised a ValueError (e.g., invalid argument combination)
+            # Convert to AgentErrorEvent so the agent can correct itself
+            err = f"Error executing tool '{tool.name}': {e}"
+            logger.warning(err)
+            error_event = AgentErrorEvent(
+                error=err,
+                tool_name=tool.name,
+                tool_call_id=action_event.tool_call.id,
+            )
+            on_event(error_event)
+            return error_event
 
         obs_event = ObservationEvent(
             observation=observation,

{openhands_sdk-1.7.2 → openhands_sdk-1.7.3}/openhands/sdk/context/condenser/llm_summarizing_condenser.py

@@ -120,7 +120,17 @@ class LLMSummarizingCondenser(RollingCondenser):
 
         Returns:
             Condensation: The generated condensation object.
+
+        Raises:
+            ValueError: If forgotten_events is empty (0 events to condense).
         """
+        if len(forgotten_events) == 0:
+            raise ValueError(
+                "Cannot condense 0 events. This typically occurs when a tool loop "
+                "spans almost the entire view, leaving no valid range for forgetting "
+                "events. Consider adjusting keep_first or max_size parameters."
+            )
+
         # Convert events to strings for the template
         event_strings = [str(forgotten_event) for forgotten_event in forgotten_events]
 

{openhands_sdk-1.7.2 → openhands_sdk-1.7.3}/openhands/sdk/conversation/impl/remote_conversation.py

@@ -28,6 +28,7 @@ from openhands.sdk.conversation.visualizer import (
     DefaultConversationVisualizer,
 )
 from openhands.sdk.event.base import Event
+from openhands.sdk.event.conversation_error import ConversationErrorEvent
 from openhands.sdk.event.conversation_state import (
     FULL_STATE_KEY,
     ConversationStateUpdateEvent,
@@ -489,6 +490,11 @@ class RemoteConversation(BaseConversation):
         self._cleanup_initiated = False
 
         if conversation_id is None:
+            # Import here to avoid circular imports
+            from openhands.sdk.tool.registry import get_tool_module_qualnames
+
+            tool_qualnames = get_tool_module_qualnames()
+            logger.debug(f"Sending tool_module_qualnames to server: {tool_qualnames}")
             payload = {
                 "agent": agent.model_dump(
                     mode="json", context={"expose_secrets": True}
@@ -500,6 +506,8 @@ class RemoteConversation(BaseConversation):
                 "workspace": LocalWorkspace(
                     working_dir=self.workspace.working_dir
                 ).model_dump(),
+                # Include tool module qualnames for dynamic registration on server
+                "tool_module_qualnames": tool_qualnames,
             }
             if stuck_detection_thresholds is not None:
                 # Convert to StuckDetectionThresholds if dict, then serialize
@@ -759,6 +767,19 @@ class RemoteConversation(BaseConversation):
                 status = info.get("execution_status")
 
                 if status != ConversationExecutionStatus.RUNNING.value:
+                    if status == ConversationExecutionStatus.ERROR.value:
+                        detail = self._get_last_error_detail()
+                        raise ConversationRunError(
+                            self._id,
+                            RuntimeError(
+                                detail or "Remote conversation ended with error"
+                            ),
+                        )
+                    if status == ConversationExecutionStatus.STUCK.value:
+                        raise ConversationRunError(
+                            self._id,
+                            RuntimeError("Remote conversation got stuck"),
+                        )
                     logger.info(
                         f"Run completed with status: {status} (elapsed: {elapsed:.1f}s)"
                     )
@@ -771,6 +792,22 @@ class RemoteConversation(BaseConversation):
 
             time.sleep(poll_interval)
 
+    def _get_last_error_detail(self) -> str | None:
+        """Return the most recent ConversationErrorEvent detail, if available."""
+        try:
+            events = self._state.events
+            for idx in range(len(events) - 1, -1, -1):
+                event = events[idx]
+                if isinstance(event, ConversationErrorEvent):
+                    detail = event.detail.strip()
+                    code = event.code.strip()
+                    if detail and code:
+                        return f"{code}: {detail}"
+                    return detail or code or None
+        except Exception as exc:
+            logger.debug("Failed to read conversation error detail: %s", exc)
+        return None
+
     def set_confirmation_policy(self, policy: ConfirmationPolicyBase) -> None:
         payload = {"policy": policy.model_dump()}
         _send_request(

{openhands_sdk-1.7.2 → openhands_sdk-1.7.3}/openhands/sdk/llm/utils/model_features.py

@@ -152,6 +152,7 @@ FORCE_STRING_SERIALIZER_MODELS: list[str] = [
 # in the message input
 SEND_REASONING_CONTENT_MODELS: list[str] = [
     "kimi-k2-thinking",
+    "openrouter/minimax-m2",  # MiniMax-M2 via OpenRouter (interleaved thinking)
     "deepseek/deepseek-reasoner",
 ]
 

{openhands_sdk-1.7.2 → openhands_sdk-1.7.3}/openhands/sdk/tool/registry.py

@@ -29,6 +29,7 @@ Returns: A sequence of ToolDefinition instances. Most of the time this will be a
 
 _LOCK = RLock()
 _REG: dict[str, Resolver] = {}
+_MODULE_QUALNAMES: dict[str, str] = {}  # Maps tool name to module qualname
 
 
 def _resolver_from_instance(name: str, tool: ToolDefinition) -> Resolver:
@@ -137,11 +138,22 @@ def register_tool(
             "(3) a callable factory returning a Sequence[ToolDefinition]"
         )
 
+    # Track the module qualname for this tool
+    module_qualname = None
+    if isinstance(factory, type):
+        module_qualname = factory.__module__
+    elif callable(factory):
+        module_qualname = getattr(factory, "__module__", None)
+    elif isinstance(factory, ToolDefinition):
+        module_qualname = factory.__class__.__module__
+
     with _LOCK:
         # TODO: throw exception when registering duplicate name tools
         if name in _REG:
             logger.warning(f"Duplicate tool name registerd {name}")
         _REG[name] = resolver
+        if module_qualname:
+            _MODULE_QUALNAMES[name] = module_qualname
 
 
 def resolve_tool(
@@ -159,3 +171,14 @@ def resolve_tool(
 def list_registered_tools() -> list[str]:
     with _LOCK:
         return list(_REG.keys())
+
+
+def get_tool_module_qualnames() -> dict[str, str]:
+    """Get a mapping of tool names to their module qualnames.
+
+    Returns:
+        A dictionary mapping tool names to module qualnames (e.g.,
+        {"glob": "openhands.tools.glob.definition"}).
+    """
+    with _LOCK:
+        return dict(_MODULE_QUALNAMES)

{openhands_sdk-1.7.2 → openhands_sdk-1.7.3}/openhands_sdk.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: openhands-sdk
-Version: 1.7.2
+Version: 1.7.3
 Summary: OpenHands SDK - Core functionality for building AI agents
 Requires-Python: >=3.12
 Requires-Dist: deprecation>=2.1.0

{openhands_sdk-1.7.2 → openhands_sdk-1.7.3}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "openhands-sdk"
-version = "1.7.2"
+version = "1.7.3"
 description = "OpenHands SDK - Core functionality for building AI agents"
 
 requires-python = ">=3.12"