openhands-tools 1.29.2__tar.gz → 1.29.3__tar.gz

{openhands_tools-1.29.2 → openhands_tools-1.29.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: openhands-tools
-Version: 1.29.2
+Version: 1.29.3
 Summary: OpenHands Tools - Runtime tools for AI agents
 Project-URL: Source, https://github.com/OpenHands/software-agent-sdk
 Project-URL: Homepage, https://github.com/OpenHands/software-agent-sdk

{openhands_tools-1.29.2 → openhands_tools-1.29.3}/openhands/tools/apply_patch/core.py

@@ -59,7 +59,9 @@ def assemble_changes(
                     old_content=old_content,
                 )
             else:
-                assert False
+                raise DiffError(
+                    f"Unexpected state: path '{path}' exists in neither orig nor dest"
+                )
     return commit
 
 
@@ -95,13 +97,15 @@ class Parser(BaseModel):
         return False
 
     def startswith(self, prefix: str | tuple[str, ...]) -> bool:
-        assert self.index < len(self.lines), f"Index: {self.index} >= {len(self.lines)}"
+        if self.index >= len(self.lines):
+            raise DiffError(f"Unexpected end of patch at index {self.index}")
         if self.lines[self.index].startswith(prefix):
             return True
         return False
 
     def read_str(self, prefix: str = "", return_everything: bool = False) -> str:
-        assert self.index < len(self.lines), f"Index: {self.index} >= {len(self.lines)}"
+        if self.index >= len(self.lines):
+            raise DiffError(f"Unexpected end of patch at index {self.index}")
         line = self.lines[self.index]
         if line.startswith(prefix):
             text = line if return_everything else line[len(prefix) :]
@@ -116,11 +120,15 @@ class Parser(BaseModel):
                 if path in self.patch.actions:
                     raise DiffError(f"Update File Error: Duplicate Path: {path}")
                 move_to = self.read_str("*** Move to: ")
+                if move_to and (".." in move_to.split("/") or move_to.startswith("/")):
+                    raise DiffError(
+                        f"Update File Error: Invalid move path '{move_to}': "
+                        "must be a relative path without '..' components"
+                    )
                 if path not in self.current_files:
                     raise DiffError(f"Update File Error: Missing File: {path}")
                 text = self.current_files[path]
                 action = self.parse_update_file(text)
-                # TODO: Check move_to is valid
                 action.move_path = move_to
                 self.patch.actions[path] = action
                 continue
@@ -359,7 +367,10 @@ def identify_files_needed(text: str) -> list[str]:
 
 
 def _get_updated_file(text: str, action: PatchAction, path: str) -> str:
-    assert action.type == ActionType.UPDATE
+    if action.type != ActionType.UPDATE:
+        raise DiffError(
+            f"_get_updated_file: expected UPDATE action for '{path}', got {action.type}"
+        )
     orig_lines = text.split("\n")
     dest_lines = []
     orig_index = 0
@@ -375,7 +386,6 @@ def _get_updated_file(text: str, action: PatchAction, path: str) -> str:
                 f"_get_updated_file: {path}: orig_index {orig_index} > "
                 f"chunk.orig_index {chunk.orig_index}"
             )
-        assert orig_index <= chunk.orig_index
         dest_lines.extend(orig_lines[orig_index : chunk.orig_index])
         delta = chunk.orig_index - orig_index
         orig_index += delta
@@ -389,8 +399,16 @@ def _get_updated_file(text: str, action: PatchAction, path: str) -> str:
     delta = len(orig_lines) - orig_index
     orig_index += delta
     dest_index += delta
-    assert orig_index == len(orig_lines)
-    assert dest_index == len(dest_lines)
+    if orig_index != len(orig_lines):
+        raise DiffError(
+            f"_get_updated_file: {path}: did not consume all original lines "
+            f"(orig_index={orig_index}, len={len(orig_lines)})"
+        )
+    if dest_index != len(dest_lines):
+        raise DiffError(
+            f"_get_updated_file: {path}: dest line count mismatch "
+            f"(dest_index={dest_index}, len={len(dest_lines)})"
+        )
     return "\n".join(dest_lines)
 
 
@@ -449,10 +467,14 @@ def apply_commit(
         if change.type == ActionType.DELETE:
             remove_fn(path)
         elif change.type == ActionType.ADD:
-            assert change.new_content is not None
+            if change.new_content is None:
+                raise DiffError(f"apply_commit: ADD change for '{path}' has no content")
             write_fn(path, change.new_content)
         elif change.type == ActionType.UPDATE:
-            assert change.new_content is not None
+            if change.new_content is None:
+                raise DiffError(
+                    f"apply_commit: UPDATE change for '{path}' has no content"
+                )
             if change.move_path:
                 write_fn(change.move_path, change.new_content)
                 remove_fn(path)
@@ -470,7 +492,8 @@ def process_patch(
 
     Returns (message, fuzz, commit)
     """
-    assert text.startswith("*** Begin Patch")
+    if not text.startswith("*** Begin Patch"):
+        raise DiffError("Invalid patch: must start with '*** Begin Patch'")
     paths = identify_files_needed(text)
     orig = load_files(paths, open_fn)
     patch, fuzz = text_to_patch(text, orig)

{openhands_tools-1.29.2 → openhands_tools-1.29.3}/openhands/tools/apply_patch/definition.py

@@ -77,7 +77,7 @@ class ApplyPatchExecutor(ToolExecutor[ApplyPatchAction, ApplyPatchObservation]):
             if not p.startswith("/")
             else Path(p).resolve()
         )
-        if not str(pth).startswith(str(self.workspace_root)):
+        if not pth.is_relative_to(self.workspace_root):
             raise DiffError("Absolute or escaping paths are not allowed")
         return pth
 

{openhands_tools-1.29.2 → openhands_tools-1.29.3}/openhands/tools/gemini/edit/impl.py

@@ -1,6 +1,5 @@
 """Edit tool executor implementation."""
 
-import os
 from pathlib import Path
 from typing import TYPE_CHECKING
 
@@ -43,11 +42,7 @@ class EditExecutor(ToolExecutor[EditAction, EditObservation]):
         new_string = action.new_string
         expected_replacements = action.expected_replacements
 
-        # Resolve path relative to workspace
-        if not os.path.isabs(file_path):
-            resolved_path = self.workspace_root / file_path
-        else:
-            resolved_path = Path(file_path)
+        resolved_path = (self.workspace_root / file_path).resolve()
 
         # Handle file creation (old_string is empty)
         if old_string == "":

{openhands_tools-1.29.2 → openhands_tools-1.29.3}/openhands_tools.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: openhands-tools
-Version: 1.29.2
+Version: 1.29.3
 Summary: OpenHands Tools - Runtime tools for AI agents
 Project-URL: Source, https://github.com/OpenHands/software-agent-sdk
 Project-URL: Homepage, https://github.com/OpenHands/software-agent-sdk

{openhands_tools-1.29.2 → openhands_tools-1.29.3}/openhands_tools.egg-info/SOURCES.txt

@@ -15,7 +15,6 @@ pyproject.toml
 ./openhands/tools/delegate/definition.py
 ./openhands/tools/delegate/impl.py
 ./openhands/tools/delegate/visualizer.py
-./openhands/tools/delegate/templates/delegate_tool_description.j2
 ./openhands/tools/file_editor/__init__.py
 ./openhands/tools/file_editor/definition.py
 ./openhands/tools/file_editor/editor.py
@@ -108,7 +107,6 @@ openhands/tools/delegate/__init__.py
 openhands/tools/delegate/definition.py
 openhands/tools/delegate/impl.py
 openhands/tools/delegate/visualizer.py
-openhands/tools/delegate/templates/delegate_tool_description.j2
 openhands/tools/file_editor/__init__.py
 openhands/tools/file_editor/definition.py
 openhands/tools/file_editor/editor.py

{openhands_tools-1.29.2 → openhands_tools-1.29.3}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "openhands-tools"
-version = "1.29.2"
+version = "1.29.3"
 description = "OpenHands Tools - Runtime tools for AI agents"
 
 requires-python = ">=3.12"

openhands_tools-1.29.2/openhands/tools/delegate/templates/delegate_tool_description.j2

@@ -1,27 +0,0 @@
-Delegation tool for spawning sub-agents and delegating tasks to them.
-
-This tool provides two commands:
-
-**spawn**: Initialize sub-agents with meaningful identifiers and optional types
-- Use descriptive identifiers that make sense for your use case (e.g., 'refactoring', 'run_tests', 'research')
-- Optionally specify agent types for specialized capabilities
-- Each identifier creates a separate sub-agent conversation
-- Examples:
-{% raw %}  - Default agents: {"command": "spawn", "ids": ["research", "implementation"]}
-  - Specialized agents: {"command": "spawn", "ids": ["research", "code"], "agent_types": ["researcher", "programmer"]}
-  - Mixed types: {"command": "spawn", "ids": ["research", "generic"], "agent_types": ["researcher"]}  # unspecified entries fall back to the default agent{% endraw %}
-
-**delegate**: Send tasks to specific sub-agents and wait for results
-- Use a dictionary mapping sub-agent identifiers to task descriptions
-- This is a blocking operation - waits for all sub-agents to complete
-- Returns a single observation containing results from all sub-agents
-- Example: {% raw %}{"command": "delegate", "tasks": {"research": "Find best practices for async code", "implementation": "Refactor the MyClass class"}}{% endraw %}
-
-**Available agent types:**
-{{ agent_types_info }}
-
-**Important Notes:**
-- Identifiers used in delegate must match those used in spawn
-- All operations are blocking and return comprehensive results
-- Sub-agents work in the same workspace as the main agent: {{ workspace_path }}
-- If you omit an agent type for an ID, a default general-purpose agent is used