Merge pull request #20 from nandatheguntupalli/main

Author: neverSettles

.github/workflows/ruff.yml

@@ -0,0 +1,17 @@
+name: Ruff Lint
+
+on:
+  push:
+    branches: ["main", "master"]
+  pull_request:
+    branches: ["main", "master"]
+
+jobs:
+  ruff:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - name: Run Ruff
+        uses: astral-sh/ruff-action@v3 

.pre-commit-config.yaml

@@ -0,0 +1,10 @@
+repos:
+- repo: https://github.com/astral-sh/ruff-pre-commit
+  # Ruff version.
+  rev: v0.11.9
+  hooks:
+    # Run the linter.
+    - id: ruff
+      args: [ --fix, --exit-zero ]
+    # Run the formatter.
+    - id: ruff-format

pyproject.toml

@@ -27,4 +27,4 @@ requires = ["setuptools"]
 build-backend = "setuptools.build_meta"
 
 [tool.setuptools.package-data]
-webEvalAgent = ["templates/**", "src/agent_overlay.js", "src/**"]
+webEvalAgent = ["templates/**", "src/agent_overlay.js", "src/**"]

webEvalAgent/src/browser_manager.py

@@ -196,9 +196,11 @@ async def open_url(self, url: str) -> str:
             send_log(f"Failed to start CDP screencast: {e}", "❌", log_type='status')
             self.screencast_task_running = False
             if self.cdp_session:
-                 try: await self.cdp_session.detach()
-                 except: pass
-                 self.cdp_session = None
+                try:
+                    await self.cdp_session.detach()
+                except Exception:
+                    pass
+                self.cdp_session = None
             return f"Opened {url}, but failed to start screen streaming."
 
         return f"Opened {url} successfully in headless mode. Streaming view to dashboard."
@@ -289,8 +291,10 @@ async def _handle_screencast_frame(self, params: Dict) -> None:
                 if "Target closed" in str(e) or "Session closed" in str(e) or "Connection closed" in str(e):
                     self.screencast_task_running = False # Mark as stopped
                     if self.cdp_session:
-                        try: await self.cdp_session.detach()
-                        except: pass
+                        try:
+                            await self.cdp_session.detach()
+                        except Exception:
+                            pass
                         self.cdp_session = None
 
 
@@ -433,17 +437,21 @@ async def handle_browser_input(self, event_type: str, details: Dict) -> None:
                 send_log("CDP session closed, stopping input handling", "⚠️", log_type='status')
                 self.screencast_task_running = False # Mark as stopped
                 if self.cdp_session:
-                    try: 
+                    try:
                         await self.cdp_session.detach()
-                    except: 
+                    except Exception:
                         pass
                     self.cdp_session = None
 
     def _map_modifiers(self, details: Dict) -> int:
         """Maps modifier keys from frontend details to CDP modifier bitmask."""
         modifiers = 0
-        if details.get('altKey'): modifiers |= 1
-        if details.get('ctrlKey'): modifiers |= 2
-        if details.get('metaKey'): modifiers |= 4 # Command key on Mac
-        if details.get('shiftKey'): modifiers |= 8
+        if details.get('altKey'):
+            modifiers |= 1
+        if details.get('ctrlKey'):
+            modifiers |= 2
+        if details.get('metaKey'):
+            modifiers |= 4  # Command key on Mac
+        if details.get('shiftKey'):
+            modifiers |= 8
         return modifiers