openhands-tools 1.58.2__tar.gz → 1.59.0__tar.gz

{openhands_tools-1.58.2 → openhands_tools-1.59.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: openhands-tools
-Version: 1.58.2
+Version: 1.59.0
 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.58.2 → openhands_tools-1.59.0}/openhands/tools/child_conversation/definition.py

@@ -6,6 +6,7 @@ from typing import TYPE_CHECKING, ClassVar, Literal, Self
 from pydantic import Field
 from rich.text import Text
 
+from openhands.sdk.event.types import UnmaskedID
 from openhands.sdk.tool.registry import register_tool
 from openhands.sdk.tool.tool import (
     Action,
@@ -36,7 +37,8 @@ class StartChildConversationAction(Action):
         default="worktree",
         description=(
             "'worktree' gives the child its own git worktree and branch when the "
-            "workspace is a git repository; 'shared' runs it in the same directory."
+            "workspace has a usable git base commit; otherwise it shares the "
+            "directory. 'shared' always runs it in the same directory."
         ),
     )
 
@@ -54,8 +56,8 @@ class StartChildConversationAction(Action):
 class StartChildConversationObservation(Observation):
     """Result of a child conversation launch."""
 
-    conversation_id: str | None = None
-    parent_conversation_id: str | None = None
+    conversation_id: UnmaskedID | None = None
+    parent_conversation_id: UnmaskedID | None = None
     status: str | None = None
     title: str | None = None
     url: str | None = None
@@ -92,7 +94,8 @@ _DESCRIPTION = (
     "conversation's history, so include the goal, relevant paths, constraints "
     "and the expected deliverable. Use `title` to name it. `isolation` controls "
     "the workspace: `worktree` (default) gives the child its own git worktree "
-    "and branch when the workspace is a git repository; the worktree starts "
+    "and branch when the workspace has a usable git base commit; otherwise "
+    "it shares the directory and reports `shared` isolation. The worktree starts "
     "from the repository's default branch, not from this conversation's branch "
     "or uncommitted changes, so use `shared` when the child must see them. "
     "`shared` runs it in the same directory; backends that manage workspaces "

{openhands_tools-1.58.2 → openhands_tools-1.59.0}/openhands/tools/task/definition.py

@@ -17,6 +17,7 @@ from pydantic.json_schema import SkipJsonSchema
 from rich.text import Text
 
 from openhands.sdk import ImageContent, TextContent
+from openhands.sdk.event.types import UnmaskedID
 from openhands.sdk.logger import get_logger
 from openhands.sdk.subagent import (
     AgentDefinition,
@@ -75,7 +76,7 @@ class TaskAction(Action):
 class TaskObservation(Observation):
     """Observation from a task execution."""
 
-    task_id: str = Field(description="The unique identifier of the task.")
+    task_id: UnmaskedID = Field(description="The unique identifier of the task.")
     subagent: str = Field(description="The subagent of the task.")
     status: str = Field(description="The status of the task.")
 

openhands_tools-1.59.0/openhands/tools/terminal/env.py

@@ -0,0 +1,88 @@
+"""Environment helpers for terminal backends."""
+
+import codecs
+import locale
+import os
+import re
+from collections.abc import Mapping
+
+from openhands.sdk.logger import get_logger
+from openhands.sdk.utils import sanitized_env
+
+
+logger = get_logger(__name__)
+
+ENV_VAR_NAME_RE = re.compile(r"^[a-zA-Z_][a-zA-Z0-9_]*$")
+UTF8_FALLBACK_LOCALE = "C.UTF-8"
+
+
+def normalize_terminal_env(
+    extra_env: Mapping[str, str] | None,
+) -> dict[str, str] | None:
+    """Validate and copy client-provided terminal environment variables."""
+    if extra_env is None:
+        return None
+
+    normalized: dict[str, str] = {}
+    for key, value in extra_env.items():
+        if not isinstance(key, str) or not ENV_VAR_NAME_RE.match(key):
+            raise ValueError(f"Invalid terminal environment variable name: {key!r}")
+        if not isinstance(value, str):
+            raise TypeError(
+                "Terminal environment variable values must be strings: "
+                f"{key!r}={value!r}"
+            )
+        normalized[key] = value
+    return normalized
+
+
+def build_terminal_env(extra_env: Mapping[str, str] | None = None) -> dict[str, str]:
+    """Return the sanitized process environment plus client-provided overrides."""
+    env = sanitized_env()
+    normalized = normalize_terminal_env(extra_env)
+    if normalized:
+        env.update(normalized)
+        env = sanitized_env(env)
+    return env
+
+
+def has_utf8_locale(environ: Mapping[str, str]) -> bool:
+    """Apply tmux's check: TMUX is set, or the first non-empty of LC_ALL,
+    LC_CTYPE, LANG names UTF-8."""
+    if "TMUX" in environ:
+        return True
+    for name in ("LC_ALL", "LC_CTYPE", "LANG"):
+        value = environ.get(name)
+        if value:
+            return "utf-8" in value.lower() or "utf8" in value.lower()
+    return False
+
+
+def ensure_utf8_locale() -> tuple[str, str | None] | None:
+    """Fall back to a UTF-8 locale when the process has none.
+
+    Outside a UTF-8 locale, tmux replaces non-ASCII characters in format output
+    with ``_``, which breaks libtmux's parsing of the ``␞`` separator. As in
+    PEP 538, only LC_CTYPE is set unless LC_ALL overrides it.
+
+    Return the variable that was set and its previous value, or None.
+    """
+    if has_utf8_locale(os.environ):
+        return None
+    # libtmux decodes tmux output with the encoding Python picked at startup,
+    # so a locale set now cannot help if that encoding is not UTF-8.
+    encoding = locale.getpreferredencoding(False)
+    if codecs.lookup(encoding).name != "utf-8":
+        logger.warning(
+            "tmux needs a UTF-8 locale, but Python decodes %s; start the "
+            "process with a UTF-8 locale or PYTHONUTF8=1",
+            encoding,
+        )
+        return None
+    name = "LC_ALL" if os.environ.get("LC_ALL") else "LC_CTYPE"
+    previous = os.environ.get(name)
+    os.environ[name] = UTF8_FALLBACK_LOCALE
+    logger.warning(
+        "No UTF-8 locale set; using %s=%s for tmux", name, UTF8_FALLBACK_LOCALE
+    )
+    return name, previous

{openhands_tools-1.58.2 → openhands_tools-1.59.0}/openhands/tools/terminal/terminal/tmux_pane_pool.py

@@ -26,6 +26,7 @@ from openhands.tools.terminal.constants import (
 )
 from openhands.tools.terminal.env import (
     build_terminal_env,
+    ensure_utf8_locale,
     normalize_terminal_env,
 )
 from openhands.tools.terminal.terminal.process_priority import (
@@ -123,6 +124,8 @@ class TmuxPanePool:
         if self._initialized:
             return
 
+        # Before building env, so the shells get the same locale as tmux.
+        ensure_utf8_locale()
         env = build_terminal_env(self.env)
         session_name = f"openhands-pool-{self.username}-{uuid.uuid4()}"
         # Stale handles must not reach another pool after tmux restarts.

{openhands_tools-1.58.2 → openhands_tools-1.59.0}/openhands/tools/terminal/terminal/tmux_terminal.py

@@ -25,6 +25,7 @@ from openhands.tools.terminal.constants import (
 )
 from openhands.tools.terminal.env import (
     build_terminal_env,
+    ensure_utf8_locale,
     normalize_terminal_env,
 )
 from openhands.tools.terminal.metadata import CmdOutputMetadata
@@ -214,6 +215,8 @@ class TmuxTerminal(TerminalInterface):
         if self._initialized:
             return
 
+        # Before building env, so the shell gets the same locale as tmux.
+        ensure_utf8_locale()
         env = build_terminal_env(self._env)
         # Disable interactive pagers (git, man, systemctl, ...) so commands that
         # auto-launch `less` on a TTY don't capture the pane and wedge the session.

{openhands_tools-1.58.2 → openhands_tools-1.59.0}/openhands_tools.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: openhands-tools
-Version: 1.58.2
+Version: 1.59.0
 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.58.2 → openhands_tools-1.59.0}/pyproject.toml

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

openhands_tools-1.58.2/openhands/tools/terminal/env.py

@@ -1,39 +0,0 @@
-"""Environment helpers for terminal backends."""
-
-import re
-from collections.abc import Mapping
-
-from openhands.sdk.utils import sanitized_env
-
-
-ENV_VAR_NAME_RE = re.compile(r"^[a-zA-Z_][a-zA-Z0-9_]*$")
-
-
-def normalize_terminal_env(
-    extra_env: Mapping[str, str] | None,
-) -> dict[str, str] | None:
-    """Validate and copy client-provided terminal environment variables."""
-    if extra_env is None:
-        return None
-
-    normalized: dict[str, str] = {}
-    for key, value in extra_env.items():
-        if not isinstance(key, str) or not ENV_VAR_NAME_RE.match(key):
-            raise ValueError(f"Invalid terminal environment variable name: {key!r}")
-        if not isinstance(value, str):
-            raise TypeError(
-                "Terminal environment variable values must be strings: "
-                f"{key!r}={value!r}"
-            )
-        normalized[key] = value
-    return normalized
-
-
-def build_terminal_env(extra_env: Mapping[str, str] | None = None) -> dict[str, str]:
-    """Return the sanitized process environment plus client-provided overrides."""
-    env = sanitized_env()
-    normalized = normalize_terminal_env(extra_env)
-    if normalized:
-        env.update(normalized)
-        env = sanitized_env(env)
-    return env