Release v1.19.0 (#2974)

neubig · openhands-agent · web-flow · commit 4278a8f653f7 · 2026-04-27T12:27:13.000-04:00
Co-authored-by: openhands &lt;openhands@all-hands.dev&gt;
diff --git a/.github/workflows/run-eval.yml b/.github/workflows/run-eval.yml
@@ -24,18 +24,7 @@ on:
             sdk_ref:
                 description: SDK commit/ref to evaluate (must be a semantic version like v1.0.0 unless 'Allow unreleased branches' is checked)
                 required: true
-                default: v1.18.1
-
-
-
-
-
-
-
-
-
-
-
+                default: v1.19.0
             allow_unreleased_branches:
                 description: Allow unreleased branches (bypasses semantic version requirement)
                 required: false
diff --git a/AGENTS.md b/AGENTS.md
@@ -116,6 +116,8 @@ When reviewing code, provide constructive feedback:
 - `pr-review-by-openhands` delegates to `OpenHands/extensions/plugins/pr-review@main`. Repo-specific reviewer instructions live in `.agents/skills/custom-codereview-guide.md`, and because task-trigger matching is substring-based, that `/codereview` skill is also auto-injected for the workflow's `/codereview-roasted` prompt.
 - The duplicate-issue automation scripts should validate `owner/repo` arguments before interpolating GitHub API paths, handle per-issue auto-close failures without aborting the whole batch, and keep `app_conversation_id` paths unquoted because OpenHands conversation IDs are already canonicalized for those endpoints.
 - Auto-title generation should not re-read `ConversationState.events` from a background task triggered by a freshly received `MessageEvent`; extract message text synchronously from the incoming event and then reuse shared title helpers (`extract_message_text`, `generate_title_from_message`) to avoid persistence-order races.
+- `RemoteConversation.generate_title()` now reconciles remote events and reuses the shared local `generate_conversation_title(...)` helper instead of calling the removed deprecated agent-server `/generate_title` REST route, so explicit remote title generation still works without a transport-only compatibility endpoint.
+
 
 
 ## Package-specific guidance
diff --git a/openhands-agent-server/openhands/agent_server/conversation_router.py b/openhands-agent-server/openhands/agent_server/conversation_router.py
@@ -19,8 +19,6 @@
     ConversationPage,
     ConversationSortOrder,
     ForkConversationRequest,
-    GenerateTitleRequest,
-    GenerateTitleResponse,
     SendMessageRequest,
     SetConfirmationPolicyRequest,
     SetSecurityAnalyzerRequest,
@@ -339,31 +337,6 @@ async def update_conversation(
     return Success()
 
 
-@conversation_router.post(
-    "/{conversation_id}/generate_title",
-    responses={404: {"description": "Item not found"}},
-    deprecated=True,
-)
-async def generate_conversation_title(
-    conversation_id: UUID,
-    request: GenerateTitleRequest,
-    conversation_service: ConversationService = Depends(get_conversation_service),
-) -> GenerateTitleResponse:
-    """Generate a title for the conversation using LLM.
-
-    Deprecated since v1.11.5 and scheduled for removal in v1.19.0.
-
-    Prefer enabling `autotitle` in `StartConversationRequest` to have the server
-    generate and persist the title automatically from the first user message.
-    """
-    title = await conversation_service.generate_conversation_title(
-        conversation_id, request.max_length, request.llm
-    )
-    if title is None:
-        raise HTTPException(status.HTTP_500_INTERNAL_SERVER_ERROR)
-    return GenerateTitleResponse(title=title)
-
-
 @conversation_router.post(
     "/{conversation_id}/ask_agent",
     responses={404: {"description": "Item not found"}},
diff --git a/openhands-agent-server/pyproject.toml b/openhands-agent-server/pyproject.toml
@@ -1,6 +1,6 @@
 [project]
 name = "openhands-agent-server"
-version = "1.18.1"
+version = "1.19.0"
 description = "OpenHands Agent Server - REST/WebSocket interface for OpenHands AI Agent"
 
 requires-python = ">=3.12"
diff --git a/openhands-sdk/openhands/sdk/conversation/impl/remote_conversation.py b/openhands-sdk/openhands/sdk/conversation/impl/remote_conversation.py
@@ -27,6 +27,7 @@
 )
 from openhands.sdk.conversation.secret_registry import SecretValue
 from openhands.sdk.conversation.state import ConversationExecutionStatus
+from openhands.sdk.conversation.title_utils import generate_conversation_title
 from openhands.sdk.conversation.types import (
     ConversationCallbackType,
     ConversationID,
@@ -1263,29 +1264,21 @@ def generate_title(self, llm: LLM | None = None, max_length: int = 50) -> str:
         """Generate a title for the conversation based on the first user message.
 
         Args:
-            llm: Optional LLM to use for title generation. If provided, its usage_id
-                 will be sent to the server. If not provided, uses the agent's LLM.
+            llm: Optional LLM to use for title generation. If not provided,
+                 uses the agent's LLM.
             max_length: Maximum length of the generated title.
 
         Returns:
             A generated title for the conversation.
         """
-        # For remote conversations, delegate to the server endpoint
-        payload = {
-            "max_length": max_length,
-            "llm": llm.model_dump(mode="json", context={"expose_secrets": True})
-            if llm
-            else None,
-        }
+        # Reconcile before reading state so recently posted user messages are
+        # visible even if they arrived between the last sync and this call.
+        self._state.events.reconcile()
 
-        resp = _send_request(
-            self._client,
-            "POST",
-            f"{self._conversation_action_base_path}/{self._id}/generate_title",
-            json=payload,
+        effective_llm = llm if llm is not None else self.agent.llm
+        return generate_conversation_title(
+            events=self._state.events, llm=effective_llm, max_length=max_length
         )
-        data = resp.json()
-        return data["title"]
 
     def condense(self) -> None:
         """Force condensation of the conversation history.
diff --git a/openhands-sdk/openhands/sdk/plugin/__init__.py b/openhands-sdk/openhands/sdk/plugin/__init__.py
@@ -80,7 +80,7 @@ def __getattr__(name: str) -> Any:
         warn_deprecated(
             f"Importing {name} from openhands.sdk.plugin",
             deprecated_in="1.16.0",
-            removed_in="1.19.0",
+            removed_in="1.21.0",
             details="Import from openhands.sdk.marketplace instead.",
             stacklevel=3,
         )
diff --git a/openhands-sdk/openhands/sdk/plugin/types.py b/openhands-sdk/openhands/sdk/plugin/types.py
@@ -389,7 +389,7 @@ def __getattr__(name: str) -> Any:
         warn_deprecated(
             f"Importing {name} from openhands.sdk.plugin.types",
             deprecated_in="1.16.0",
-            removed_in="1.19.0",
+            removed_in="1.21.0",
             details="Import from openhands.sdk.marketplace instead.",
             stacklevel=3,
         )
diff --git a/openhands-sdk/pyproject.toml b/openhands-sdk/pyproject.toml
@@ -1,6 +1,6 @@
 [project]
 name = "openhands-sdk"
-version = "1.18.1"
+version = "1.19.0"
 description = "OpenHands SDK - Core functionality for building AI agents"
 
 requires-python = ">=3.12"
diff --git a/openhands-tools/openhands/tools/browser_use/logging_fix.py b/openhands-tools/openhands/tools/browser_use/logging_fix.py
@@ -15,14 +15,14 @@
 
 warn_cleanup(
     "Monkey patching to prevent browser_use logging interference",
-    cleanup_by="1.19.0",
+    cleanup_by="1.21.0",
     details=(
         "This workaround should be removed once browser_use fixes the "
         "problematic logging configuration code. The upstream PR #3717 "
         "(https://github.com/browser-use/browser-use/pull/3717) was closed "
-        "without merge. As of browser_use 0.11.9 the module-level "
-        "basicConfig(force=True) and _ensure_all_loggers_use_stderr() "
-        "are still present. Re-evaluate on each release."
+        "without merge. As of browser_use 0.11.9, the server still calls "
+        "_ensure_all_loggers_use_stderr() during import and initialization. "
+        "Re-evaluate when browser_use changes that behavior."
     ),
 )
 
diff --git a/openhands-tools/pyproject.toml b/openhands-tools/pyproject.toml
@@ -1,6 +1,6 @@
 [project]
 name = "openhands-tools"
-version = "1.18.1"
+version = "1.19.0"
 description = "OpenHands Tools - Runtime tools for AI agents"
 
 requires-python = ">=3.12"
diff --git a/openhands-workspace/pyproject.toml b/openhands-workspace/pyproject.toml
@@ -1,6 +1,6 @@
 [project]
 name = "openhands-workspace"
-version = "1.18.1"
+version = "1.19.0"
 description = "OpenHands Workspace - Docker and container-based workspace implementations"
 
 requires-python = ">=3.12"
diff --git a/tests/agent_server/test_conversation_router.py b/tests/agent_server/test_conversation_router.py
@@ -1120,144 +1120,15 @@ def test_update_conversation_invalid_title(
         client.app.dependency_overrides.clear()
 
 
-def test_generate_conversation_title_success(
-    client, mock_conversation_service, sample_conversation_id
-):
-    """Test generate_conversation_title endpoint with successful generation."""
-
-    mock_conversation_service.generate_conversation_title.return_value = (
-        "Generated Title"
-    )
-
-    client.app.dependency_overrides[get_conversation_service] = (
-        lambda: mock_conversation_service
-    )
-
-    try:
-        request_data = {"max_length": 30}
-
-        response = client.post(
-            f"/api/conversations/{sample_conversation_id}/generate_title",
-            json=request_data,
-        )
-
-        assert response.status_code == 200
-        data = response.json()
-        assert data["title"] == "Generated Title"
-
-        mock_conversation_service.generate_conversation_title.assert_called_once()
-        call_args = mock_conversation_service.generate_conversation_title.call_args
-        assert call_args[0][0] == sample_conversation_id
-        assert call_args[0][1] == 30
-        assert call_args[0][2] is None
-    finally:
-        client.app.dependency_overrides.clear()
-
-
-def test_generate_conversation_title_with_llm(
-    client, mock_conversation_service, sample_conversation_id
-):
-    """Test generate_conversation_title endpoint with custom LLM."""
-
-    mock_conversation_service.generate_conversation_title.return_value = (
-        "Custom LLM Title"
-    )
-
-    client.app.dependency_overrides[get_conversation_service] = (
-        lambda: mock_conversation_service
-    )
-
-    try:
-        request_data = {
-            "max_length": 40,
-            "llm": {
-                "model": "gpt-3.5-turbo",
-                "api_key": "custom-key",
-                "usage_id": "custom-llm",
-            },
-        }
-
-        response = client.post(
-            f"/api/conversations/{sample_conversation_id}/generate_title",
-            json=request_data,
-        )
-
-        assert response.status_code == 200
-        data = response.json()
-        assert data["title"] == "Custom LLM Title"
-
-        mock_conversation_service.generate_conversation_title.assert_called_once()
-        call_args = mock_conversation_service.generate_conversation_title.call_args
-        assert call_args[0][0] == sample_conversation_id
-        assert call_args[0][1] == 40
-        assert call_args[0][2] is not None
-    finally:
-        client.app.dependency_overrides.clear()
-
-
-def test_generate_conversation_title_failure(
-    client, mock_conversation_service, sample_conversation_id
-):
-    """Test generate_conversation_title endpoint with generation failure."""
-
-    mock_conversation_service.generate_conversation_title.return_value = None
-
-    client.app.dependency_overrides[get_conversation_service] = (
-        lambda: mock_conversation_service
-    )
-
-    try:
-        request_data = {"max_length": 50}
-
-        response = client.post(
-            f"/api/conversations/{sample_conversation_id}/generate_title",
-            json=request_data,
-        )
-
-        assert response.status_code == 500
-        mock_conversation_service.generate_conversation_title.assert_called_once()
-    finally:
-        client.app.dependency_overrides.clear()
-
-
-def test_generate_conversation_title_invalid_params(
-    client, mock_conversation_service, sample_conversation_id
-):
-    """Test generate_conversation_title endpoint with invalid parameters."""
-
-    client.app.dependency_overrides[get_conversation_service] = (
-        lambda: mock_conversation_service
-    )
-
-    try:
-        request_data = {"max_length": 0}
-        response = client.post(
-            f"/api/conversations/{sample_conversation_id}/generate_title",
-            json=request_data,
-        )
-        assert response.status_code == 422
-
-        request_data = {"max_length": 201}
-        response = client.post(
-            f"/api/conversations/{sample_conversation_id}/generate_title",
-            json=request_data,
-        )
-        assert response.status_code == 422
-    finally:
-        client.app.dependency_overrides.clear()
-
-
-def test_generate_title_endpoint_is_deprecated_in_openapi(client):
+def test_generate_title_endpoint_removed_from_openapi(client):
     response = client.get("/openapi.json")
     assert response.status_code == 200
 
     openapi_schema = response.json()
-    operation = openapi_schema["paths"][
+    assert (
         "/api/conversations/{conversation_id}/generate_title"
-    ]["post"]
-
-    assert operation.get("deprecated") is True
-    assert "scheduled for removal" in operation["description"]
+        not in openapi_schema["paths"]
+    )
 
 
 def test_start_conversation_with_tool_module_qualnames(
diff --git a/tests/sdk/conversation/remote/test_remote_conversation.py b/tests/sdk/conversation/remote/test_remote_conversation.py
diff --git a/uv.lock b/uv.lock

Original file line number	Diff line number	Diff line change
`@@ -80,7 +80,7 @@ def __getattr__(name: str) -> Any:`
`80`	`80`	`warn_deprecated(`
`81`	`81`	`f"Importing {name} from openhands.sdk.plugin",`
`82`	`82`	`deprecated_in="1.16.0",`
`83`		`- removed_in="1.19.0",`
	`83`	`+ removed_in="1.21.0",`
`84`	`84`	`details="Import from openhands.sdk.marketplace instead.",`
`85`	`85`	`stacklevel=3,`
`86`	`86`	`)`
Original file line number	Diff line number	Diff line change
`@@ -389,7 +389,7 @@ def __getattr__(name: str) -> Any:`
`389`	`389`	`warn_deprecated(`
`390`	`390`	`f"Importing {name} from openhands.sdk.plugin.types",`
`391`	`391`	`deprecated_in="1.16.0",`
`392`		`- removed_in="1.19.0",`
	`392`	`+ removed_in="1.21.0",`
`393`	`393`	`details="Import from openhands.sdk.marketplace instead.",`
`394`	`394`	`stacklevel=3,`
`395`	`395`	`)`