mirror of
https://github.com/eigent-ai/eigent.git
synced 2026-05-04 22:50:18 +00:00
camel support 0.2.76a1
This commit is contained in:
luo
parent
c0dbb5f35d
commit
fc323b6670
6 changed files with 926 additions and 935 deletions
|
|
@ -12,8 +12,7 @@ from camel.models import BaseModelBackend
|
|||
from camel.toolkits.hybrid_browser_toolkit.hybrid_browser_toolkit_ts import (
|
||||
HybridBrowserToolkit as BaseHybridBrowserToolkit,
|
||||
)
|
||||
from camel.toolkits.hybrid_browser_toolkit.ws_wrapper import \
|
||||
WebSocketBrowserWrapper as BaseWebSocketBrowserWrapper
|
||||
from camel.toolkits.hybrid_browser_toolkit.ws_wrapper import WebSocketBrowserWrapper as BaseWebSocketBrowserWrapper
|
||||
from app.component.command import bun, uv
|
||||
from app.component.environment import env
|
||||
from app.service.task import Agents
|
||||
|
|
@ -38,18 +37,16 @@ class WebSocketBrowserWrapper(BaseWebSocketBrowserWrapper):
|
|||
response_data = await self.websocket.recv()
|
||||
response = json.loads(response_data)
|
||||
|
||||
message_id = response.get('id')
|
||||
message_id = response.get("id")
|
||||
if message_id and message_id in self._pending_responses:
|
||||
# Set the result for the waiting coroutine
|
||||
future = self._pending_responses.pop(message_id)
|
||||
if not future.done():
|
||||
future.set_result(response)
|
||||
logger.debug(
|
||||
f"Processed response for message {message_id}")
|
||||
logger.debug(f"Processed response for message {message_id}")
|
||||
else:
|
||||
# Log unexpected messages
|
||||
logger.warning(
|
||||
f"Received unexpected message: {response}")
|
||||
logger.warning(f"Received unexpected message: {response}")
|
||||
|
||||
except asyncio.CancelledError:
|
||||
disconnect_reason = "Receive loop cancelled"
|
||||
|
|
@ -57,22 +54,18 @@ class WebSocketBrowserWrapper(BaseWebSocketBrowserWrapper):
|
|||
break
|
||||
except websockets.exceptions.ConnectionClosed as e:
|
||||
disconnect_reason = f"WebSocket closed: code={e.code}, reason={e.reason}"
|
||||
logger.warning(
|
||||
f"WebSocket disconnect: {disconnect_reason}")
|
||||
logger.warning(f"WebSocket disconnect: {disconnect_reason}")
|
||||
break
|
||||
except websockets.exceptions.WebSocketException as e:
|
||||
disconnect_reason = f"WebSocket error: {type(e).__name__}: {e}"
|
||||
logger.error(
|
||||
f"WebSocket disconnect: {disconnect_reason}")
|
||||
logger.error(f"WebSocket disconnect: {disconnect_reason}")
|
||||
break
|
||||
except json.JSONDecodeError as e:
|
||||
logger.error(f"Failed to decode WebSocket message: {e}")
|
||||
continue # Try to continue on JSON errors
|
||||
except Exception as e:
|
||||
disconnect_reason = f"Unexpected error: {type(e).__name__}: {e}"
|
||||
logger.error(
|
||||
f"WebSocket disconnect: {disconnect_reason}",
|
||||
exc_info=True)
|
||||
logger.error(f"WebSocket disconnect: {disconnect_reason}", exc_info=True)
|
||||
# Notify all pending futures of the error
|
||||
for future in self._pending_responses.values():
|
||||
if not future.done():
|
||||
|
|
@ -80,8 +73,7 @@ class WebSocketBrowserWrapper(BaseWebSocketBrowserWrapper):
|
|||
self._pending_responses.clear()
|
||||
break
|
||||
finally:
|
||||
logger.info(
|
||||
f"WebSocket receive loop terminated. Reason: {disconnect_reason or 'Normal shutdown'}")
|
||||
logger.info(f"WebSocket receive loop terminated. Reason: {disconnect_reason or 'Normal shutdown'}")
|
||||
# Mark the websocket as None to indicate disconnection
|
||||
self.websocket = None
|
||||
|
||||
|
|
@ -113,13 +105,11 @@ class WebSocketBrowserWrapper(BaseWebSocketBrowserWrapper):
|
|||
)
|
||||
if build_result.returncode != 0:
|
||||
logger.error(f"TypeScript build failed: {build_result.stderr}")
|
||||
raise RuntimeError(
|
||||
f"TypeScript build failed: {build_result.stderr}")
|
||||
raise RuntimeError(f"TypeScript build failed: {build_result.stderr}")
|
||||
else:
|
||||
# Log warnings but don't fail on them
|
||||
if build_result.stderr:
|
||||
logger.warning(
|
||||
f"TypeScript build warnings: {build_result.stderr}")
|
||||
logger.warning(f"TypeScript build warnings: {build_result.stderr}")
|
||||
logger.info("TypeScript build completed successfully")
|
||||
|
||||
# Start the WebSocket server
|
||||
|
|
@ -140,8 +130,7 @@ class WebSocketBrowserWrapper(BaseWebSocketBrowserWrapper):
|
|||
if self.process.poll() is not None:
|
||||
# Process died
|
||||
stderr = self.process.stderr.read() # type: ignore
|
||||
raise RuntimeError(
|
||||
f"WebSocket server failed to start: {stderr}")
|
||||
raise RuntimeError(f"WebSocket server failed to start: {stderr}")
|
||||
|
||||
try:
|
||||
line = self.process.stdout.readline() # type: ignore
|
||||
|
|
@ -149,15 +138,13 @@ class WebSocketBrowserWrapper(BaseWebSocketBrowserWrapper):
|
|||
if line.startswith("SERVER_READY:"):
|
||||
self.server_port = int(line.split(":")[1].strip())
|
||||
server_ready = True
|
||||
logger.info(
|
||||
f"WebSocket server ready on port {self.server_port}")
|
||||
logger.info(f"WebSocket server ready on port {self.server_port}")
|
||||
except (ValueError, IndexError):
|
||||
continue
|
||||
|
||||
if not server_ready:
|
||||
self.process.kill()
|
||||
raise RuntimeError(
|
||||
"WebSocket server failed to start within timeout")
|
||||
raise RuntimeError("WebSocket server failed to start within timeout")
|
||||
|
||||
# Connect to the WebSocket server
|
||||
try:
|
||||
|
|
@ -170,8 +157,7 @@ class WebSocketBrowserWrapper(BaseWebSocketBrowserWrapper):
|
|||
logger.info("Connected to WebSocket server")
|
||||
except Exception as e:
|
||||
self.process.kill()
|
||||
raise RuntimeError(
|
||||
f"Failed to connect to WebSocket server: {e}") from e
|
||||
raise RuntimeError(f"Failed to connect to WebSocket server: {e}") from e
|
||||
|
||||
# Start the background receiver task - THIS WAS MISSING!
|
||||
self._receive_task = asyncio.create_task(self._receive_loop())
|
||||
|
|
@ -184,14 +170,12 @@ class WebSocketBrowserWrapper(BaseWebSocketBrowserWrapper):
|
|||
logger.debug("WebSocket server initialized successfully")
|
||||
except RuntimeError as e:
|
||||
if "Timeout waiting for response to command: init" in str(e):
|
||||
logger.warning(
|
||||
"Init timeout - continuing anyway (CDP connection may be slow)")
|
||||
logger.warning("Init timeout - continuing anyway (CDP connection may be slow)")
|
||||
# Continue without error - the WebSocket server is likely still initializing
|
||||
else:
|
||||
raise
|
||||
|
||||
async def _send_command(self, command: str, params: Dict[str, Any]) -> \
|
||||
Dict[str, Any]:
|
||||
async def _send_command(self, command: str, params: Dict[str, Any]) -> Dict[str, Any]:
|
||||
"""Send a command to the WebSocket server with enhanced error handling."""
|
||||
try:
|
||||
# First ensure we have a valid connection
|
||||
|
|
@ -199,14 +183,13 @@ class WebSocketBrowserWrapper(BaseWebSocketBrowserWrapper):
|
|||
raise RuntimeError("WebSocket connection not established")
|
||||
|
||||
# Check connection state before sending
|
||||
if hasattr(self.websocket, 'state'):
|
||||
if hasattr(self.websocket, "state"):
|
||||
import websockets.protocol
|
||||
if self.websocket.state != websockets.protocol.State.OPEN:
|
||||
raise RuntimeError(
|
||||
f"WebSocket is in {self.websocket.state} state, not OPEN")
|
||||
|
||||
logger.debug(
|
||||
f"Sending command '{command}' with params: {params}")
|
||||
if self.websocket.state != websockets.protocol.State.OPEN:
|
||||
raise RuntimeError(f"WebSocket is in {self.websocket.state} state, not OPEN")
|
||||
|
||||
logger.debug(f"Sending command '{command}' with params: {params}")
|
||||
|
||||
# Call parent's _send_command
|
||||
result = await super()._send_command(command, params)
|
||||
|
|
@ -222,8 +205,7 @@ class WebSocketBrowserWrapper(BaseWebSocketBrowserWrapper):
|
|||
self.websocket = None
|
||||
raise
|
||||
except Exception as e:
|
||||
logger.error(
|
||||
f"Unexpected error sending command '{command}': {type(e).__name__}: {e}")
|
||||
logger.error(f"Unexpected error sending command '{command}': {type(e).__name__}: {e}")
|
||||
raise
|
||||
|
||||
|
||||
|
|
@ -235,8 +217,7 @@ class WebSocketConnectionPool:
|
|||
self._connections: Dict[str, WebSocketBrowserWrapper] = {}
|
||||
self._lock = asyncio.Lock()
|
||||
|
||||
async def get_connection(self, session_id: str, config: Dict[
|
||||
str, Any]) -> WebSocketBrowserWrapper:
|
||||
async def get_connection(self, session_id: str, config: Dict[str, Any]) -> WebSocketBrowserWrapper:
|
||||
"""Get or create a connection for the given session ID."""
|
||||
async with self._lock:
|
||||
# Check if we have an existing connection for this session
|
||||
|
|
@ -248,50 +229,43 @@ class WebSocketConnectionPool:
|
|||
if wrapper.websocket:
|
||||
try:
|
||||
# Check WebSocket state based on available attributes
|
||||
if hasattr(wrapper.websocket, 'state'):
|
||||
if hasattr(wrapper.websocket, "state"):
|
||||
import websockets.protocol
|
||||
|
||||
is_healthy = wrapper.websocket.state == websockets.protocol.State.OPEN
|
||||
if not is_healthy:
|
||||
logger.debug(
|
||||
f"Session {session_id} WebSocket state: {wrapper.websocket.state}")
|
||||
elif hasattr(wrapper.websocket, 'open'):
|
||||
logger.debug(f"Session {session_id} WebSocket state: {wrapper.websocket.state}")
|
||||
elif hasattr(wrapper.websocket, "open"):
|
||||
is_healthy = wrapper.websocket.open
|
||||
else:
|
||||
# Try ping as last resort
|
||||
try:
|
||||
await asyncio.wait_for(
|
||||
wrapper.websocket.ping(), timeout=1.0)
|
||||
await asyncio.wait_for(wrapper.websocket.ping(), timeout=1.0)
|
||||
is_healthy = True
|
||||
except:
|
||||
is_healthy = False
|
||||
except Exception as e:
|
||||
logger.debug(
|
||||
f"Health check failed for session {session_id}: {e}")
|
||||
logger.debug(f"Health check failed for session {session_id}: {e}")
|
||||
is_healthy = False
|
||||
|
||||
if is_healthy:
|
||||
logger.debug(
|
||||
f"Reusing healthy WebSocket connection for session {session_id}")
|
||||
logger.debug(f"Reusing healthy WebSocket connection for session {session_id}")
|
||||
return wrapper
|
||||
else:
|
||||
# Connection is unhealthy, clean it up
|
||||
logger.info(
|
||||
f"Removing unhealthy WebSocket connection for session {session_id}")
|
||||
logger.info(f"Removing unhealthy WebSocket connection for session {session_id}")
|
||||
try:
|
||||
await wrapper.stop()
|
||||
except Exception as e:
|
||||
logger.debug(
|
||||
f"Error stopping unhealthy wrapper: {e}")
|
||||
logger.debug(f"Error stopping unhealthy wrapper: {e}")
|
||||
del self._connections[session_id]
|
||||
|
||||
# Create a new connection
|
||||
logger.info(
|
||||
f"Creating new WebSocket connection for session {session_id}")
|
||||
logger.info(f"Creating new WebSocket connection for session {session_id}")
|
||||
wrapper = WebSocketBrowserWrapper(config)
|
||||
await wrapper.start()
|
||||
self._connections[session_id] = wrapper
|
||||
logger.info(
|
||||
f"Successfully created WebSocket connection for session {session_id}")
|
||||
logger.info(f"Successfully created WebSocket connection for session {session_id}")
|
||||
return wrapper
|
||||
|
||||
async def close_connection(self, session_id: str):
|
||||
|
|
@ -302,11 +276,9 @@ class WebSocketConnectionPool:
|
|||
try:
|
||||
await wrapper.stop()
|
||||
except Exception as e:
|
||||
logger.error(
|
||||
f"Error closing WebSocket connection for session {session_id}: {e}")
|
||||
logger.error(f"Error closing WebSocket connection for session {session_id}: {e}")
|
||||
del self._connections[session_id]
|
||||
logger.info(
|
||||
f"Closed WebSocket connection for session {session_id}")
|
||||
logger.info(f"Closed WebSocket connection for session {session_id}")
|
||||
|
||||
async def _close_connection_unlocked(self, session_id: str):
|
||||
"""Close connection without acquiring lock (for internal use)."""
|
||||
|
|
@ -315,11 +287,9 @@ class WebSocketConnectionPool:
|
|||
try:
|
||||
await wrapper.stop()
|
||||
except Exception as e:
|
||||
logger.error(
|
||||
f"Error closing WebSocket connection for session {session_id}: {e}")
|
||||
logger.error(f"Error closing WebSocket connection for session {session_id}: {e}")
|
||||
del self._connections[session_id]
|
||||
logger.info(
|
||||
f"Closed WebSocket connection for session {session_id}")
|
||||
logger.info(f"Closed WebSocket connection for session {session_id}")
|
||||
|
||||
async def close_all(self):
|
||||
"""Close all connections in the pool."""
|
||||
|
|
@ -337,28 +307,31 @@ class HybridBrowserToolkit(BaseHybridBrowserToolkit, AbstractToolkit):
|
|||
agent_name: str = Agents.search_agent
|
||||
|
||||
def __init__(
|
||||
self,
|
||||
api_task_id: str,
|
||||
*,
|
||||
headless: bool = False,
|
||||
user_data_dir: str | None = None,
|
||||
stealth: bool = True,
|
||||
web_agent_model: BaseModelBackend | None = None,
|
||||
cache_dir: str = "tmp/",
|
||||
enabled_tools: List[str] | None = None,
|
||||
browser_log_to_file: bool = False,
|
||||
session_id: str | None = None,
|
||||
default_start_url: str = "https://google.com/",
|
||||
default_timeout: int | None = None,
|
||||
short_timeout: int | None = None,
|
||||
navigation_timeout: int | None = None,
|
||||
network_idle_timeout: int | None = None,
|
||||
screenshot_timeout: int | None = None,
|
||||
page_stability_timeout: int | None = None,
|
||||
dom_content_loaded_timeout: int | None = None,
|
||||
viewport_limit: bool = False,
|
||||
connect_over_cdp: bool = True,
|
||||
cdp_url: str | None = "http://localhost:9222",
|
||||
self,
|
||||
api_task_id: str,
|
||||
*,
|
||||
headless: bool = False,
|
||||
user_data_dir: str | None = None,
|
||||
stealth: bool = True,
|
||||
web_agent_model: BaseModelBackend | None = None,
|
||||
cache_dir: Optional[str] = None,
|
||||
enabled_tools: List[str] | None = None,
|
||||
browser_log_to_file: bool = False,
|
||||
log_dir: Optional[str] = None,
|
||||
session_id: str | None = None,
|
||||
default_start_url: Optional[str] = None,
|
||||
default_timeout: int | None = None,
|
||||
short_timeout: int | None = None,
|
||||
navigation_timeout: int | None = None,
|
||||
network_idle_timeout: int | None = None,
|
||||
screenshot_timeout: int | None = None,
|
||||
page_stability_timeout: int | None = None,
|
||||
dom_content_loaded_timeout: int | None = None,
|
||||
viewport_limit: bool = False,
|
||||
connect_over_cdp: bool = True,
|
||||
cdp_url: str | None = "http://localhost:9222",
|
||||
cdp_keep_current_page: bool = False,
|
||||
full_visual_mode: bool = False,
|
||||
) -> None:
|
||||
self.api_task_id = api_task_id
|
||||
super().__init__(
|
||||
|
|
@ -381,6 +354,8 @@ class HybridBrowserToolkit(BaseHybridBrowserToolkit, AbstractToolkit):
|
|||
viewport_limit=viewport_limit,
|
||||
connect_over_cdp=connect_over_cdp,
|
||||
cdp_url=cdp_url,
|
||||
cdp_keep_current_page=cdp_keep_current_page,
|
||||
full_visual_mode=full_visual_mode,
|
||||
)
|
||||
|
||||
async def _ensure_ws_wrapper(self):
|
||||
|
|
@ -388,22 +363,18 @@ class HybridBrowserToolkit(BaseHybridBrowserToolkit, AbstractToolkit):
|
|||
global websocket_connection_pool
|
||||
|
||||
# Get session ID from config or use default
|
||||
session_id = self._ws_config.get('session_id', 'default')
|
||||
session_id = self._ws_config.get("session_id", "default")
|
||||
|
||||
# Get or create connection from pool
|
||||
self._ws_wrapper = await websocket_connection_pool.get_connection(
|
||||
session_id, self._ws_config)
|
||||
self._ws_wrapper = await websocket_connection_pool.get_connection(session_id, self._ws_config)
|
||||
|
||||
# Additional health check
|
||||
if self._ws_wrapper.websocket is None:
|
||||
logger.warning(
|
||||
f"WebSocket connection for session {session_id} is None after pool retrieval, recreating...")
|
||||
logger.warning(f"WebSocket connection for session {session_id} is None after pool retrieval, recreating...")
|
||||
await websocket_connection_pool.close_connection(session_id)
|
||||
self._ws_wrapper = await websocket_connection_pool.get_connection(
|
||||
session_id, self._ws_config)
|
||||
self._ws_wrapper = await websocket_connection_pool.get_connection(session_id, self._ws_config)
|
||||
|
||||
def clone_for_new_session(self,
|
||||
new_session_id: str | None = None) -> "HybridBrowserToolkit":
|
||||
def clone_for_new_session(self, new_session_id: str | None = None) -> "HybridBrowserToolkit":
|
||||
import uuid
|
||||
|
||||
if new_session_id is None:
|
||||
|
|
@ -418,6 +389,7 @@ class HybridBrowserToolkit(BaseHybridBrowserToolkit, AbstractToolkit):
|
|||
cache_dir=f"{self._cache_dir.rstrip('/')}/_clone_{new_session_id}/",
|
||||
enabled_tools=self.enabled_tools.copy(),
|
||||
browser_log_to_file=self._browser_log_to_file,
|
||||
log_dir=self.config_loader.get_toolkit_config().log_dir,
|
||||
session_id=new_session_id,
|
||||
default_start_url=self._default_start_url,
|
||||
default_timeout=self._default_timeout,
|
||||
|
|
@ -430,6 +402,8 @@ class HybridBrowserToolkit(BaseHybridBrowserToolkit, AbstractToolkit):
|
|||
viewport_limit=self._viewport_limit,
|
||||
connect_over_cdp=self.config_loader.get_browser_config().connect_over_cdp,
|
||||
cdp_url=f"http://localhost:{env('browser_port', '9222')}",
|
||||
cdp_keep_current_page=self.config_loader.get_browser_config().cdp_keep_current_page,
|
||||
full_visual_mode=self._full_visual_mode,
|
||||
)
|
||||
|
||||
@classmethod
|
||||
|
|
@ -446,17 +420,15 @@ class HybridBrowserToolkit(BaseHybridBrowserToolkit, AbstractToolkit):
|
|||
logger.error(f"Error closing browser: {e}")
|
||||
|
||||
# Release connection from pool
|
||||
session_id = self._ws_config.get('session_id', 'default')
|
||||
session_id = self._ws_config.get("session_id", "default")
|
||||
await websocket_connection_pool.close_connection(session_id)
|
||||
logger.info(
|
||||
f"Released WebSocket connection for session {session_id}")
|
||||
logger.info(f"Released WebSocket connection for session {session_id}")
|
||||
|
||||
def __del__(self):
|
||||
"""Cleanup when object is garbage collected."""
|
||||
if hasattr(self, '_ws_wrapper') and self._ws_wrapper:
|
||||
session_id = self._ws_config.get('session_id', 'default')
|
||||
logger.debug(
|
||||
f"HybridBrowserToolkit for session {session_id} is being garbage collected")
|
||||
if hasattr(self, "_ws_wrapper") and self._ws_wrapper:
|
||||
session_id = self._ws_config.get("session_id", "default")
|
||||
logger.debug(f"HybridBrowserToolkit for session {session_id} is being garbage collected")
|
||||
|
||||
@listen_toolkit(BaseHybridBrowserToolkit.browser_open)
|
||||
async def browser_open(self) -> Dict[str, Any]:
|
||||
|
|
@ -474,8 +446,7 @@ class HybridBrowserToolkit(BaseHybridBrowserToolkit, AbstractToolkit):
|
|||
logger.debug(f"browser_visit_page succeeded for URL: {url}")
|
||||
return result
|
||||
except Exception as e:
|
||||
logger.error(
|
||||
f"browser_visit_page failed for URL {url}: {type(e).__name__}: {e}")
|
||||
logger.error(f"browser_visit_page failed for URL {url}: {type(e).__name__}: {e}")
|
||||
raise
|
||||
|
||||
@listen_toolkit(BaseHybridBrowserToolkit.browser_back)
|
||||
|
|
@ -491,10 +462,8 @@ class HybridBrowserToolkit(BaseHybridBrowserToolkit, AbstractToolkit):
|
|||
return await super().browser_get_page_snapshot()
|
||||
|
||||
@listen_toolkit(BaseHybridBrowserToolkit.browser_get_som_screenshot)
|
||||
async def browser_get_som_screenshot(self, read_image: bool = False,
|
||||
instruction: str | None = None) -> str:
|
||||
return await super().browser_get_som_screenshot(read_image,
|
||||
instruction)
|
||||
async def browser_get_som_screenshot(self, read_image: bool = False, instruction: str | None = None) -> str:
|
||||
return await super().browser_get_som_screenshot(read_image, instruction)
|
||||
|
||||
@listen_toolkit(BaseHybridBrowserToolkit.browser_click)
|
||||
async def browser_click(self, *, ref: str) -> Dict[str, Any]:
|
||||
|
|
@ -505,23 +474,19 @@ class HybridBrowserToolkit(BaseHybridBrowserToolkit, AbstractToolkit):
|
|||
return await super().browser_type(ref=ref, text=text)
|
||||
|
||||
@listen_toolkit(BaseHybridBrowserToolkit.browser_select)
|
||||
async def browser_select(self, *, ref: str, value: str) -> Dict[
|
||||
str, Any]:
|
||||
async def browser_select(self, *, ref: str, value: str) -> Dict[str, Any]:
|
||||
return await super().browser_select(ref=ref, value=value)
|
||||
|
||||
@listen_toolkit(BaseHybridBrowserToolkit.browser_scroll)
|
||||
async def browser_scroll(self, *, direction: str, amount: int = 500) -> \
|
||||
Dict[str, Any]:
|
||||
return await super().browser_scroll(direction=direction,
|
||||
amount=amount)
|
||||
async def browser_scroll(self, *, direction: str, amount: int = 500) -> Dict[str, Any]:
|
||||
return await super().browser_scroll(direction=direction, amount=amount)
|
||||
|
||||
@listen_toolkit(BaseHybridBrowserToolkit.browser_enter)
|
||||
async def browser_enter(self) -> Dict[str, Any]:
|
||||
return await super().browser_enter()
|
||||
|
||||
@listen_toolkit(BaseHybridBrowserToolkit.browser_wait_user)
|
||||
async def browser_wait_user(self, timeout_sec: float | None = None) -> \
|
||||
Dict[str, Any]:
|
||||
async def browser_wait_user(self, timeout_sec: float | None = None) -> Dict[str, Any]:
|
||||
return await super().browser_wait_user(timeout_sec)
|
||||
|
||||
@listen_toolkit(BaseHybridBrowserToolkit.browser_switch_tab)
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue