openhands-agent-server 1.22.0__tar.gz → 1.22.1__tar.gz

{openhands_agent_server-1.22.0 → openhands_agent_server-1.22.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: openhands-agent-server
-Version: 1.22.0
+Version: 1.22.1
 Summary: OpenHands Agent Server - REST/WebSocket interface for OpenHands AI Agent
 Project-URL: Source, https://github.com/OpenHands/software-agent-sdk
 Project-URL: Homepage, https://github.com/OpenHands/software-agent-sdk

{openhands_agent_server-1.22.0 → openhands_agent_server-1.22.1}/openhands/agent_server/__main__.py

@@ -18,6 +18,7 @@ logger = get_logger(__name__)
 
 
 _INTERNAL_SERVER_URL_ENV = "OH_INTERNAL_SERVER_URL"
+_EXTRA_PYTHON_PATH_ENV = "OH_EXTRA_PYTHON_PATH"
 
 
 def _get_internal_server_url(host: str, port: int) -> str:
@@ -43,6 +44,46 @@ def _get_internal_server_url(host: str, port: int) -> str:
     return f"http://{resolved_host}:{port}"
 
 
+def extend_python_path(extra_paths: str | None) -> None:
+    """Add directories to ``sys.path`` so ``importlib.import_module`` can find
+    external custom-tool modules — even when running from a PyInstaller binary.
+
+    Paths are read from *extra_paths* (``--extra-python-path`` CLI arg) **and**
+    the ``OH_EXTRA_PYTHON_PATH`` environment variable.  Both use the
+    platform path separator (``':'`` on POSIX, ``';'`` on Windows).
+
+    Non-existent directories are skipped with a warning; duplicates and paths
+    already on ``sys.path`` are silently ignored.
+    """
+    raw_parts: list[str] = []
+    for source in (extra_paths, os.environ.get(_EXTRA_PYTHON_PATH_ENV)):
+        if source:
+            raw_parts.extend(source.split(os.pathsep))
+
+    added = 0
+    for part in raw_parts:
+        part = part.strip()
+        if not part:
+            continue
+        resolved = os.path.abspath(part)
+        if not os.path.isdir(resolved):
+            logger.warning(
+                "Ignoring non-existent --extra-python-path entry: %s", resolved
+            )
+            continue
+        if resolved not in sys.path:
+            sys.path.insert(0, resolved)
+            logger.info("Added to sys.path: %s", resolved)
+            added += 1
+
+    if added:
+        logger.info(
+            "Extended sys.path with %d director%s for custom tool imports",
+            added,
+            "y" if added == 1 else "ies",
+        )
+
+
 def preload_modules(modules_arg: str | None) -> None:
     """Import user-specified modules so their top-level side effects run.
 
@@ -171,6 +212,16 @@ def main() -> None:
             "(e.g. 'myapp.tools,myapp.plugins')"
         ),
     )
+    parser.add_argument(
+        "--extra-python-path",
+        type=str,
+        default=None,
+        help=(
+            "Additional directories to add to sys.path for custom tool imports "
+            f"('{os.pathsep}'-separated).  Also reads from the "
+            f"{_EXTRA_PYTHON_PATH_ENV} environment variable."
+        ),
+    )
 
     args = parser.parse_args()
 
@@ -181,6 +232,10 @@ def main() -> None:
         else:
             sys.exit(1)
 
+    # Extend sys.path before importing user modules so external .py files
+    # are reachable — critical for PyInstaller binary builds.
+    extend_python_path(args.extra_python_path)
+
     # Import user modules after early-exit checks
     preload_modules(args.import_modules)
 

{openhands_agent_server-1.22.0 → openhands_agent_server-1.22.1}/openhands/agent_server/bash_service.py

@@ -32,7 +32,8 @@ class BashEventService:
 
     bash_events_dir: Path = field()
     _pub_sub: PubSub[BashEventBase] = field(
-        default_factory=lambda: PubSub[BashEventBase](), init=False
+        default_factory=lambda: PubSub[BashEventBase](max_subscribers=50),
+        init=False,
     )
 
     def _ensure_bash_events_dir(self) -> None:

{openhands_agent_server-1.22.0 → openhands_agent_server-1.22.1}/openhands/agent_server/config.py

@@ -174,6 +174,15 @@ class Config(BaseModel):
         default=True,
         description="Whether to preload tools",
     )
+    max_concurrent_runs: int = Field(
+        default=10,
+        ge=1,
+        description=(
+            "Maximum number of conversations that can execute agent steps "
+            "concurrently.  Controls the size of the dedicated thread pool "
+            "used for conversation.run() calls."
+        ),
+    )
     secret_key: SecretStr | None = Field(
         default_factory=_default_secret_key,
         description=(

{openhands_agent_server-1.22.0 → openhands_agent_server-1.22.1}/openhands/agent_server/conversation_router.py

@@ -19,10 +19,7 @@ from openhands.agent_server._secrets_exposure import (
     decrypt_incoming_llm_secrets,
     get_cipher,
 )
-from openhands.agent_server.conversation_service import (
-    ConversationContractMismatchError,
-    ConversationService,
-)
+from openhands.agent_server.conversation_service import ConversationService
 from openhands.agent_server.dependencies import get_conversation_service
 from openhands.agent_server.models import (
     AgentResponseResult,
@@ -162,10 +159,7 @@ async def batch_get_conversations(
 # Write Methods
 
 
-@conversation_router.post(
-    "",
-    responses={409: {"description": "Conversation contract mismatch"}},
-)
+@conversation_router.post("")
 async def start_conversation(
     request: Annotated[
         StartConversationRequest, Body(examples=START_CONVERSATION_EXAMPLES)
@@ -174,13 +168,7 @@ async def start_conversation(
     conversation_service: ConversationService = Depends(get_conversation_service),
 ) -> ConversationInfo:
     """Start a conversation in the local environment."""
-    try:
-        info, is_new = await conversation_service.start_conversation(request)
-    except ConversationContractMismatchError as e:
-        raise HTTPException(
-            status_code=status.HTTP_409_CONFLICT,
-            detail=str(e),
-        ) from e
+    info, is_new = await conversation_service.start_conversation(request)
     response.status_code = status.HTTP_201_CREATED if is_new else status.HTTP_200_OK
     return info
 

{openhands_agent_server-1.22.0 → openhands_agent_server-1.22.1}/openhands/agent_server/conversation_router_acp.py

@@ -1,5 +1,10 @@
 """ACP-capable conversation routes for the schema-sensitive endpoints."""
 
+# Deprecated REST contract: all /api/acp/conversations routes were deprecated
+# in v1.22.0 and are scheduled for removal in v1.27.0. The standard
+# FastAPI/OpenAPI deprecation marker for routes is ``deprecated=True`` on each
+# route decorator; keep matching docstring notices for CI deprecation checks.
+
 from typing import Annotated
 from uuid import UUID
 
@@ -53,7 +58,7 @@ START_ACP_CONVERSATION_EXAMPLES = [
 ]
 
 
-@conversation_router_acp.get("/search")
+@conversation_router_acp.get("/search", deprecated=True)
 async def search_acp_conversations(
     page_id: Annotated[
         str | None,
@@ -73,7 +78,11 @@ async def search_acp_conversations(
     ] = ConversationSortOrder.CREATED_AT_DESC,
     conversation_service: ConversationService = Depends(get_conversation_service),
 ) -> ACPConversationPage:
-    """Search conversations using the ACP-capable contract."""
+    """Search conversations using the ACP-capable contract.
+
+    Deprecated since v1.22.0 and scheduled for removal in v1.27.0.
+    Use ``/api/conversations/search`` instead.
+    """
     assert limit > 0
     assert limit <= 100
     return await conversation_service.search_acp_conversations(
@@ -81,7 +90,7 @@ async def search_acp_conversations(
     )
 
 
-@conversation_router_acp.get("/count")
+@conversation_router_acp.get("/count", deprecated=True)
 async def count_acp_conversations(
     status: Annotated[
         ConversationExecutionStatus | None,
@@ -89,36 +98,49 @@ async def count_acp_conversations(
     ] = None,
     conversation_service: ConversationService = Depends(get_conversation_service),
 ) -> int:
-    """Count conversations using the ACP-capable contract."""
-    return await conversation_service.count_acp_conversations(status)
+    """Count conversations using the ACP-capable contract.
+
+    Deprecated since v1.22.0 and scheduled for removal in v1.27.0.
+    Use ``/api/conversations/count`` instead.
+    """
+    return await conversation_service.count_conversations(status)
 
 
 @conversation_router_acp.get(
     "/{conversation_id}",
     responses={404: {"description": "Item not found"}},
+    deprecated=True,
 )
 async def get_acp_conversation(
     conversation_id: UUID,
     conversation_service: ConversationService = Depends(get_conversation_service),
 ) -> ACPConversationInfo:
-    """Get a conversation using the ACP-capable contract."""
+    """Get a conversation using the ACP-capable contract.
+
+    Deprecated since v1.22.0 and scheduled for removal in v1.27.0.
+    Use ``/api/conversations/{conversation_id}`` instead.
+    """
     conversation = await conversation_service.get_acp_conversation(conversation_id)
     if conversation is None:
         raise HTTPException(status.HTTP_404_NOT_FOUND)
     return conversation
 
 
-@conversation_router_acp.get("")
+@conversation_router_acp.get("", deprecated=True)
 async def batch_get_acp_conversations(
     ids: Annotated[list[UUID], Query()],
     conversation_service: ConversationService = Depends(get_conversation_service),
 ) -> list[ACPConversationInfo | None]:
-    """Batch get conversations using the ACP-capable contract."""
+    """Batch get conversations using the ACP-capable contract.
+
+    Deprecated since v1.22.0 and scheduled for removal in v1.27.0.
+    Use ``/api/conversations`` instead.
+    """
     assert len(ids) < 100
     return await conversation_service.batch_get_acp_conversations(ids)
 
 
-@conversation_router_acp.post("")
+@conversation_router_acp.post("", deprecated=True)
 async def start_acp_conversation(
     request: Annotated[
         StartACPConversationRequest,
@@ -127,7 +149,12 @@ async def start_acp_conversation(
     response: Response,
     conversation_service: ConversationService = Depends(get_conversation_service),
 ) -> ACPConversationInfo:
-    """Start a conversation using the ACP-capable contract."""
+    """Start a conversation using the ACP-capable contract.
+
+    Deprecated since v1.22.0 and scheduled for removal in v1.27.0.
+    Use ``/api/conversations`` instead; it now accepts ACP agents and
+    ``agent_settings`` payloads.
+    """
     info, is_new = await conversation_service.start_acp_conversation(request)
     response.status_code = status.HTTP_201_CREATED if is_new else status.HTTP_200_OK
     return info