Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion packages/narada/pyproject.toml
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
[project]
name = "narada"
version = "0.1.53a3"
version = "0.1.53a4"
description = "Python client SDK for Narada"
license = "Apache-2.0"
readme = "README.md"
Expand Down
8 changes: 6 additions & 2 deletions packages/narada/src/narada/client.py
Original file line number Diff line number Diff line change
Expand Up @@ -279,7 +279,9 @@ async def _initialize_cloud_browser_window(
# Navigate to login URL (provided by backend with custom token)
context = browser.contexts[0]
initialization_page = context.pages[0]
await initialization_page.goto(login_url, timeout=15_000)
await initialization_page.goto(
login_url, timeout=15_000, wait_until="domcontentloaded"
)

# Wait for browser window ID. The extension can take a bit to be installed, so we retry a
# few times.
Expand All @@ -302,7 +304,9 @@ async def _initialize_cloud_browser_window(
raise
# If browser window ID is not found, reload the page and try again
# try to go to the login URL again (with customToken query param)
await initialization_page.goto(login_url, timeout=15_000)
await initialization_page.goto(
login_url, timeout=15_000, wait_until="domcontentloaded"
)

cloud_window = CloudBrowserWindow(
browser_window_id=browser_window_id,
Expand Down
91 changes: 91 additions & 0 deletions packages/narada/tests/test_cloud_browser.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,91 @@
from types import SimpleNamespace
from unittest.mock import AsyncMock, call

import pytest
from narada.client import Narada
from narada.config import BrowserConfig
from narada_core.errors import NaradaTimeoutError


def _build_client_with_cloud_page(page: AsyncMock) -> Narada:
client = Narada(auth_headers={"x-api-key": "test-key"})
browser = SimpleNamespace(contexts=[SimpleNamespace(pages=[page])])
client._playwright = SimpleNamespace(
chromium=SimpleNamespace(connect_over_cdp=AsyncMock(return_value=browser))
)
return client


@pytest.mark.asyncio
async def test_initialize_cloud_browser_window_uses_domcontentloaded_for_login_navigation(
monkeypatch: pytest.MonkeyPatch,
) -> None:
page = AsyncMock()
client = _build_client_with_cloud_page(page)

wait_for_browser_window_id = AsyncMock(return_value="browser-window-123")
monkeypatch.setattr(
client, "_wait_for_browser_window_id", wait_for_browser_window_id
)

window = await client._initialize_cloud_browser_window(
config=BrowserConfig(interactive=False),
cdp_websocket_url="wss://agentcore.example.test/session-123",
session_id="session-123",
login_url="https://app.narada.ai/chat?customToken=test-token",
cdp_auth_headers={"Authorization": "signed-cdp"},
)

page.goto.assert_awaited_once_with(
"https://app.narada.ai/chat?customToken=test-token",
timeout=15_000,
wait_until="domcontentloaded",
)
wait_for_browser_window_id.assert_awaited_once_with(
page,
BrowserConfig(interactive=False),
timeout=30_000,
)
assert window.browser_window_id == "browser-window-123"
assert window.cloud_browser_session_id == "session-123"


@pytest.mark.asyncio
async def test_initialize_cloud_browser_window_uses_domcontentloaded_for_retry_navigation(
monkeypatch: pytest.MonkeyPatch,
) -> None:
page = AsyncMock()
client = _build_client_with_cloud_page(page)

wait_for_browser_window_id = AsyncMock(
side_effect=[
NaradaTimeoutError("Timed out waiting for browser window ID"),
"browser-window-123",
]
)
monkeypatch.setattr(
client, "_wait_for_browser_window_id", wait_for_browser_window_id
)

window = await client._initialize_cloud_browser_window(
config=BrowserConfig(interactive=False),
cdp_websocket_url="wss://agentcore.example.test/session-123",
session_id="session-123",
login_url="https://app.narada.ai/chat?customToken=test-token",
cdp_auth_headers={"Authorization": "signed-cdp"},
)

assert page.goto.await_args_list == [
call(
"https://app.narada.ai/chat?customToken=test-token",
timeout=15_000,
wait_until="domcontentloaded",
),
call(
"https://app.narada.ai/chat?customToken=test-token",
timeout=15_000,
wait_until="domcontentloaded",
),
]
assert wait_for_browser_window_id.await_count == 2
assert window.browser_window_id == "browser-window-123"
2 changes: 1 addition & 1 deletion uv.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading