remove unnecessary logging

Author: miguelg719

stagehand/api.py

@@ -110,10 +110,6 @@ async def _execute(self, method: str, payload: dict[str, Any]) -> Any:
     modified_payload = convert_dict_keys_to_camel_case(payload)
 
     client = self.httpx_client or httpx.AsyncClient(timeout=self.timeout_settings)
-    self.logger.debug(f"\n==== EXECUTING {method.upper()} ====")
-    self.logger.debug(f"URL: {self.api_url}/sessions/{self.session_id}/{method}")
-    self.logger.debug(f"Payload: {modified_payload}")
-    self.logger.debug(f"Headers: {headers}")
 
     async with client:
         try:
@@ -133,8 +129,6 @@ async def _execute(self, method: str, payload: dict[str, Any]) -> Any:
                     raise RuntimeError(
                         f"Request failed with status {response.status_code}: {error_message}"
                     )
-
-                self.logger.debug("[STREAM] Processing server response")
                 result = None
 
                 async for line in response.aiter_lines():
@@ -163,9 +157,6 @@ async def _execute(self, method: str, payload: dict[str, Any]) -> Any:
                                 )
                             elif status == "finished":
                                 result = message.get("data", {}).get("result")
-                                self.logger.debug(
-                                    "[SYSTEM] Operation completed successfully"
-                                )
                         elif msg_type == "log":
                             # Process log message using _handle_log
                             await self._handle_log(message)

stagehand/browser.py

@@ -40,7 +40,6 @@ async def connect_browserbase_browser(
     # Connect to remote browser via Browserbase SDK and CDP
     bb = Browserbase(api_key=browserbase_api_key)
     try:
-        logger.debug(f"Retrieving Browserbase session details for {session_id}...")
         session = bb.sessions.retrieve(session_id)
         if session.status != "RUNNING":
             raise RuntimeError(
@@ -54,7 +53,6 @@ async def connect_browserbase_browser(
     logger.debug(f"Connecting to remote browser at: {connect_url}")
     try:
         browser = await playwright.chromium.connect_over_cdp(connect_url)
-        logger.debug(f"Connected to remote browser: {browser}")
     except Exception as e:
         logger.error(f"Failed to connect Playwright via CDP: {str(e)}")
         raise
@@ -233,7 +231,7 @@ async def connect_local_browser(
 
 async def apply_stealth_scripts(context: BrowserContext, logger: StagehandLogger):
     """Applies JavaScript init scripts to make the browser less detectable."""
-    logger.debug("Applying stealth init scripts to the context...")
+    logger.debug("Applying stealth scripts to the context...")
     stealth_script = """
     (() => {
         // Override navigator.webdriver
@@ -280,7 +278,6 @@ async def apply_stealth_scripts(context: BrowserContext, logger: StagehandLogger
     """
     try:
         await context.add_init_script(stealth_script)
-        logger.debug("Stealth init script added successfully.")
     except Exception as e:
         logger.error(f"Failed to add stealth init script: {str(e)}")
 
@@ -308,6 +305,12 @@ async def cleanup_browser_resources(
             await context.close()
         except Exception as e:
             logger.error(f"Error closing context: {str(e)}")
+    if browser:
+        try:
+            logger.debug("Closing browser...")
+            await browser.close()
+        except Exception as e:
+            logger.error(f"Error closing browser: {str(e)}")
 
     # Clean up temporary user data directory if created
     if temp_user_data_dir:

stagehand/client.py

@@ -395,7 +395,6 @@ def _get_lock_for_session(self) -> asyncio.Lock:
         """
         if self.session_id not in self._session_locks:
             self._session_locks[self.session_id] = asyncio.Lock()
-            self.logger.debug(f"Created lock for session {self.session_id}")
         return self._session_locks[self.session_id]
 
     async def __aenter__(self):