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

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: openhands-sdk
-Version: 1.7.1
+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.1 → 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.1 → openhands_sdk-1.7.3}/openhands/sdk/agent/base.py

@@ -194,7 +194,10 @@ class AgentBase(DiscriminatedUnionMixin, ABC):
             **template_kwargs,
         )
         if self.agent_context:
-            _system_message_suffix = self.agent_context.get_system_message_suffix()
+            _system_message_suffix = self.agent_context.get_system_message_suffix(
+                llm_model=self.llm.model,
+                llm_model_canonical=self.llm.model_canonical_name,
+            )
             if _system_message_suffix:
                 system_message += "\n\n" + _system_message_suffix
         return system_message

{openhands_sdk-1.7.1 → openhands_sdk-1.7.3}/openhands/sdk/context/agent_context.py

@@ -13,6 +13,7 @@ from openhands.sdk.context.skills import (
     load_user_skills,
 )
 from openhands.sdk.llm import Message, TextContent
+from openhands.sdk.llm.utils.model_prompt_spec import get_model_prompt_spec
 from openhands.sdk.logger import get_logger
 from openhands.sdk.secret import SecretSource, SecretValue
 
@@ -154,7 +155,11 @@ class AgentContext(BaseModel):
             secret_infos.append({"name": name, "description": description})
         return secret_infos
 
-    def get_system_message_suffix(self) -> str | None:
+    def get_system_message_suffix(
+        self,
+        llm_model: str | None = None,
+        llm_model_canonical: str | None = None,
+    ) -> str | None:
         """Get the system message with repo skill content and custom suffix.
 
         Custom suffix can typically includes:
@@ -164,11 +169,30 @@ class AgentContext(BaseModel):
         - Repository-specific instructions (collected from repo skills)
         """
         repo_skills = [s for s in self.skills if s.trigger is None]
+
+        # Gate vendor-specific repo skills based on model family.
+        if llm_model or llm_model_canonical:
+            spec = get_model_prompt_spec(llm_model or "", llm_model_canonical)
+            family = (spec.family or "").lower()
+            if family:
+                filtered: list[Skill] = []
+                for s in repo_skills:
+                    n = (s.name or "").lower()
+                    if n == "claude" and not (
+                        "anthropic" in family or "claude" in family
+                    ):
+                        continue
+                    if n == "gemini" and not (
+                        "gemini" in family or "google_gemini" in family
+                    ):
+                        continue
+                    filtered.append(s)
+                repo_skills = filtered
+
         logger.debug(f"Triggered {len(repo_skills)} repository skills: {repo_skills}")
         # Build the workspace context information
         secret_infos = self.get_secret_infos()
         if repo_skills or self.system_message_suffix or secret_infos:
-            # TODO(test): add a test for this rendering to make sure they work
             formatted_text = render_template(
                 prompt_dir=str(PROMPT_DIR),
                 template_name="system_message_suffix.j2",

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

@@ -36,9 +36,9 @@ class LLMSummarizingCondenser(RollingCondenser):
     """
 
     llm: LLM
-    max_size: int = Field(default=120, gt=0)
+    max_size: int = Field(default=240, gt=0)
     max_tokens: int | None = None
-    keep_first: int = Field(default=4, ge=0)
+    keep_first: int = Field(default=2, ge=0)
 
     @model_validator(mode="after")
     def validate_keep_first_vs_max_size(self):
@@ -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.1 → openhands_sdk-1.7.3}/openhands/sdk/context/skills/skill.py

@@ -411,7 +411,10 @@ def load_skills_from_dir(
     # Process all files in one loop
     for file in chain(special_files, md_files):
         try:
-            skill = Skill.load(file, skill_dir)
+            skill = Skill.load(
+                file,
+                skill_dir,
+            )
             if skill.trigger is None:
                 repo_skills[skill.name] = skill
             else:
@@ -700,6 +703,8 @@ def load_public_skills(
                     path=skill_file,
                     skill_dir=repo_path,
                 )
+                if skill is None:
+                    continue
                 all_skills.append(skill)
                 logger.debug(f"Loaded public skill: {skill.name}")
             except Exception as e:

{openhands_sdk-1.7.1 → openhands_sdk-1.7.3}/openhands/sdk/conversation/impl/local_conversation.py

@@ -377,9 +377,24 @@ class LocalConversation(BaseConversation):
                     if (
                         self.state.execution_status
                         == ConversationExecutionStatus.WAITING_FOR_CONFIRMATION
-                        or iteration >= self.max_iteration_per_run
                     ):
                         break
+
+                    if iteration >= self.max_iteration_per_run:
+                        error_msg = (
+                            f"Agent reached maximum iterations limit "
+                            f"({self.max_iteration_per_run})."
+                        )
+                        logger.error(error_msg)
+                        self._state.execution_status = ConversationExecutionStatus.ERROR
+                        self._on_event(
+                            ConversationErrorEvent(
+                                source="environment",
+                                code="MaxIterationsReached",
+                                detail=error_msg,
+                            )
+                        )
+                        break
         except Exception as e:
             self._state.execution_status = ConversationExecutionStatus.ERROR
 

{openhands_sdk-1.7.1 → 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.1 → 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.1 → openhands_sdk-1.7.3}/openhands/sdk/llm/utils/verified_models.py

@@ -1,6 +1,5 @@
 VERIFIED_OPENAI_MODELS = [
     "gpt-5.2",
-    "gpt-5.2-codex",
     "gpt-5.1",
     "gpt-5.1-codex-max",
     "gpt-5.1-codex",

{openhands_sdk-1.7.1 → 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.1 → 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.1
+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.1 → openhands_sdk-1.7.3}/pyproject.toml

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