fix: update Dockerfile to include git installation, enhance pyproject.toml with license and dependency organization and refine CI workflow by removing unused validation step

Author: hashkode

.github/workflows/ci.yml

@@ -41,9 +41,6 @@ jobs:
       - name: "Python lint"
         run: uvx ruff check .
 
-      - name: "Validate project metadata"
-        run: uvx --from 'validate-pyproject[all,store]' validate-pyproject  pyproject.toml
-
   build-and-publish:
     runs-on: ubuntu-latest
 

Dockerfile

@@ -7,7 +7,7 @@ ENV UV_COMPILE_BYTECODE=1 \
 
 # Install build dependencies and clean up in the same layer
 RUN apt-get update -y && \
-    apt-get install --no-install-recommends -y clang && \
+    apt-get install --no-install-recommends -y clang git && \
     rm -rf /var/lib/apt/lists/*
 
 # Install Python before the project for caching

pyproject.toml

@@ -1,13 +1,10 @@
-[build-system]
-requires = ["hatchling", "uv-dynamic-versioning"]
-build-backend = "hatchling.build"
-
 [project]
 name = "browser-use-mcp-server"
 dynamic = ["version"]
 description = "MCP browser-use server library"
 readme = "README.md"
 requires-python = ">=3.11,<4.0"
+license = {text = "MIT"}
 authors = [
     {name = "Cobrowser Team"}
 ]
@@ -32,17 +29,19 @@ dependencies = [
 ]
 
 [project.optional-dependencies]
-dev = [
+# Dependencies for running tests
+test = [
     "pytest>=7.0.0",
     "pytest-asyncio>=0.21.0",
+    "pytest-cov>=4.1.0",
+]
+# Dependencies for development (includes test dependencies)
+dev = [
+    "browser-use-mcp-server[test]",
     "black>=23.0.0",
     "isort>=5.12.0",
     "mypy>=1.0.0",
-]
-test = [
-    "pytest>=7.0.0",
-    "pytest-asyncio>=0.21.0",
-    "pytest-cov>=4.1.0",
+    "ruff>=0.5.5",
 ]
 
 [project.urls]
@@ -72,6 +71,10 @@ disallow_incomplete_defs = true
 [project.scripts]
 browser-use-mcp-server = "browser_use_mcp_server.cli:cli"
 
+[build-system]
+requires = ["hatchling", "uv-dynamic-versioning"]
+build-backend = "hatchling.build"
+
 [tool.hatch.build]
 packages = ["src", "server"]
 include = ["server"]
@@ -86,3 +89,16 @@ source = "uv-dynamic-versioning"
 vcs = "git"
 style = "pep440"
 bump = true
+
+# Added Ruff configuration
+[tool.ruff]
+line-length = 88
+target-version = "py311"
+
+[tool.ruff.lint]
+# Enable common Pyflakes, pycodestyle, and isort rules
+select = ["E", "F", "W", "I"]
+
+[tool.ruff.format]
+# Use black-compatible formatting
+quote-style = "double"

server/server.py

@@ -10,34 +10,34 @@
 """
 
 # Standard library imports
-import os
 import asyncio
 import json
 import logging
+import os
+import sys
+import time
 import traceback
 import uuid
 from datetime import datetime
 from typing import Any, Dict, Optional, Tuple, Union
-import time
-import sys
 
 # Third-party imports
 import click
-from dotenv import load_dotenv
-from pythonjsonlogger import jsonlogger
+import mcp.types as types
 
 # Browser-use library imports
 from browser_use import Agent
 from browser_use.browser.browser import Browser, BrowserConfig
 from browser_use.browser.context import BrowserContext, BrowserContextConfig
-
-# MCP server components
-from mcp.server import Server
-import mcp.types as types
+from dotenv import load_dotenv
+from langchain_core.language_models import BaseLanguageModel
 
 # LLM provider
 from langchain_openai import ChatOpenAI
-from langchain_core.language_models import BaseLanguageModel
+
+# MCP server components
+from mcp.server import Server
+from pythonjsonlogger import jsonlogger
 
 # Configure logging
 logger = logging.getLogger()
@@ -806,12 +806,13 @@ def main(
     )
 
     # Set up SSE transport
+    import asyncio
+    import threading
+
+    import uvicorn
     from mcp.server.sse import SseServerTransport
     from starlette.applications import Starlette
     from starlette.routing import Mount, Route
-    import uvicorn
-    import asyncio
-    import threading
 
     sse = SseServerTransport("/messages/")
 

src/browser_use_mcp_server/__init__.py

@@ -5,4 +5,4 @@
 using the browser_use library.
 """
 
-__version__ = "0.1.3"
+__version__ = "0.0.0"

src/browser_use_mcp_server/cli.py

@@ -5,12 +5,13 @@
 It wraps the existing server functionality with a CLI.
 """
 
-import os
-import sys
+import importlib.util
 import json
 import logging
+import os
+import sys
+
 import click
-import importlib.util
 from pythonjsonlogger import jsonlogger
 
 # Configure logging for CLI

src/browser_use_mcp_server/server.py

@@ -6,15 +6,16 @@
 
 import os
 import sys
+
 from server.server import (
+    CONFIG,
     Server,
-    main,
-    create_browser_context_for_task,
-    run_browser_task_async,
     cleanup_old_tasks,
+    create_browser_context_for_task,
     create_mcp_server,
     init_configuration,
-    CONFIG,
+    main,
+    run_browser_task_async,
     task_store,
 )