Ticket/AI-013

ai-backend/.pre-commit-config.yaml

@@ -28,7 +28,7 @@ repos:
     rev: 5.12.0
     hooks:
       - id: isort
-        args: ["--profile", "black"]
+        args: ['--profile', 'black']
 
   - repo: https://github.com/pre-commit/mirrors-mypy
     rev: v1.7.1

ai-backend/agents/editor.py

@@ -34,9 +34,7 @@ async def review_article(
         logger.info("Reviewing article content")
         return article_content, {}
 
-    async def fact_check(
-        self, article_content: str, source_data: dict[str, Any]
-    ) -> dict[str, Any]:
+    async def fact_check(self, article_content: str, source_data: dict[str, Any]) -> dict[str, Any]:
         """Fact-check article content against source data.
 
         Args:

ai-backend/agents/researcher.py

@@ -21,9 +21,7 @@ def __init__(self, config: dict[str, Any]):
         self.config = config
         logger.info("Research Agent initialized")
 
-    async def research_team_history(
-        self, team_id: str, opponent_id: str
-    ) -> dict[str, Any]:
+    async def research_team_history(self, team_id: str, opponent_id: str) -> dict[str, Any]:
         """Research historical matchups between teams.
 
         Args:
@@ -35,9 +33,7 @@ async def research_team_history(
         """
         # TODO: Implement team history research
         team_safe, opponent_safe = sanitize_multiple_log_inputs(team_id, opponent_id)
-        logger.info(
-            "Researching history between teams: %s vs %s", team_safe, opponent_safe
-        )
+        logger.info("Researching history between teams: %s vs %s", team_safe, opponent_safe)
         return {}
 
     async def research_player_performance(

ai-backend/config/settings.py

@@ -8,8 +8,8 @@
 from typing import Any
 
 from dotenv import load_dotenv
-from pydantic import Field, validator
-from pydantic_settings import BaseSettings
+from pydantic import Field, field_validator
+from pydantic_settings import BaseSettings, SettingsConfigDict
 
 # Load environment variables from .env file
 load_dotenv()
@@ -21,14 +21,12 @@ class Settings(BaseSettings):
     """Application settings loaded from environment variables with validation."""
 
     # Required settings
-    openai_api_key: str = Field(..., min_length=20, description="OpenAI API key")
-    supabase_url: str = Field(..., description="Supabase project URL")
-    supabase_service_role_key: str = Field(
+    OPENAI_API_KEY: str = Field(..., min_length=20, description="OpenAI API key")
+    SUPABASE_URL: str = Field(..., description="Supabase project URL")
+    SUPABASE_SERVICE_ROLE_KEY: str = Field(
         ..., min_length=20, description="Supabase service role key"
     )
-    rapidapi_key: str = Field(
-        ..., min_length=10, description="RapidAPI key for API-Football"
-    )
+    RAPIDAPI_KEY: str = Field(..., min_length=10, description="RapidAPI key for API-Football")
 
     # OpenAI Configuration
     openai_model: str = Field(default="gpt-4-turbo", description="OpenAI model to use")
@@ -40,9 +38,7 @@ class Settings(BaseSettings):
 
     # Chainlit Configuration
     chainlit_host: str = Field(default="127.0.0.1", description="Chainlit host")
-    chainlit_port: int = Field(
-        default=8001, ge=1, le=65535, description="Chainlit port"
-    )
+    chainlit_port: int = Field(default=8001, ge=1, le=65535, description="Chainlit port")
 
     # Environment Configuration
     environment: str = Field(default="development", description="Environment name")
@@ -56,39 +52,44 @@ class Settings(BaseSettings):
         description="API-Football base URL",
     )
 
-    @validator("openai_api_key")
-    def validate_openai_key(cls, v: str) -> str:  # noqa: N805
+    @field_validator("OPENAI_API_KEY")
+    @classmethod
+    def validate_openai_key(cls, v: str) -> str:
         if not v or v == "your-openai-api-key" or v == "sk-...":
             raise ValueError("Valid OpenAI API key is required")
         if not v.startswith("sk-"):
             raise ValueError('OpenAI API key must start with "sk-"')
         return v
 
-    @validator("supabase_url")
-    def validate_supabase_url(cls, v: str) -> str:  # noqa: N805
+    @field_validator("SUPABASE_URL")
+    @classmethod
+    def validate_supabase_url(cls, v: str) -> str:
         if not v.startswith("https://"):
             raise ValueError("Supabase URL must be a valid HTTPS URL")
         if not v.endswith(".supabase.co"):
             raise ValueError("Supabase URL must end with .supabase.co")
         return v
 
-    @validator("environment")
-    def validate_environment(cls, v: str) -> str:  # noqa: N805
+    @field_validator("environment")
+    @classmethod
+    def validate_environment(cls, v: str) -> str:
         allowed = ["development", "staging", "production"]
         if v not in allowed:
             raise ValueError(f"Environment must be one of: {allowed}")
         return v
 
-    @validator("log_level")
-    def validate_log_level(cls, v: str) -> str:  # noqa: N805
+    @field_validator("log_level")
+    @classmethod
+    def validate_log_level(cls, v: str) -> str:
         allowed = ["DEBUG", "INFO", "WARNING", "ERROR", "CRITICAL"]
         v_upper = v.upper()
         if v_upper not in allowed:
             raise ValueError(f"Log level must be one of: {allowed}")
         return v_upper
 
-    @validator("log_format")
-    def validate_log_format(cls, v: str) -> str:  # noqa: N805
+    @field_validator("log_format")
+    @classmethod
+    def validate_log_format(cls, v: str) -> str:
         allowed = ["json", "text"]
         if v not in allowed:
             raise ValueError(f"Log format must be one of: {allowed}")
@@ -110,13 +111,7 @@ def to_dict(self) -> dict[str, Any]:
             "api_football_base_url": self.api_football_base_url,
         }
 
-    class Config:
-        """Pydantic configuration for Settings model."""
-
-        env_file = ".env"
-        case_sensitive = True
-        # Allow extra fields for forward compatibility
-        extra = "ignore"
+    model_config = SettingsConfigDict(env_file=".env", case_sensitive=True, extra="ignore")
 
 
 # Global settings instance

ai-backend/main.py

@@ -113,9 +113,7 @@ async def generate_article(self, request: ArticleRequest) -> ArticleResponse:
 
         except Exception as e:
             logger.error(f"Error generating article: {e!s}")
-            raise HTTPException(
-                status_code=500, detail=f"Failed to generate article: {e!s}"
-            ) from e
+            raise HTTPException(status_code=500, detail=f"Failed to generate article: {e!s}") from e
 
     async def _collect_game_data(
         self, collector: DataCollectorAgent, game_id: str
@@ -149,15 +147,11 @@ async def _generate_content(
             },
         }
 
-    async def _edit_content(
-        self, editor: EditorAgent, content: dict[str, Any]
-    ) -> dict[str, Any]:
+    async def _edit_content(self, editor: EditorAgent, content: dict[str, Any]) -> dict[str, Any]:
         """Edit and finalize content."""
         article_content = content.get("content", "")
         metadata = content.get("metadata", {})
-        edited_content, review_feedback = await editor.review_article(
-            article_content, metadata
-        )
+        edited_content, review_feedback = await editor.review_article(article_content, metadata)
         return {
             "content": edited_content,
             "metadata": {**metadata, "review_feedback": review_feedback},
@@ -275,9 +269,7 @@ async def root() -> dict[str, str]:
 if __name__ == "__main__":
     import uvicorn
 
-    logger.info(
-        "Starting server on %s:%s", settings.fastapi_host, settings.fastapi_port
-    )
+    logger.info("Starting server on %s:%s", settings.fastapi_host, settings.fastapi_port)
     uvicorn.run(
         "main:app",
         host=settings.fastapi_host,

ai-backend/pytest.ini

@@ -4,8 +4,16 @@ python_files = test_*.py
 python_classes = Test*
 python_functions = test_*
 addopts =
+    --verbose
+    --tb=short
     --strict-markers
     --disable-warnings
-    --verbose
-asyncio_default_fixture_loop_scope = function
-asyncio_mode = auto
+    --cov=.
+    --cov-report=term-missing
+    --cov-report=html:htmlcov
+    --cov-fail-under=80
+markers =
+    slow: marks tests as slow (deselect with '-m "not slow"')
+    integration: marks tests as integration tests
+    unit: marks tests as unit tests
+    asyncio: marks tests as asyncio tests

ai-backend/tests/test_environment.py

@@ -0,0 +1,48 @@
+"""Test script to verify all dependencies are properly installed."""
+
+import sys
+
+print(f"Python version: {sys.version}")
+
+# Test core dependencies
+try:
+    print("✅ OpenAI package imported successfully")
+except ImportError as e:
+    print(f"❌ OpenAI import failed: {e}")
+
+try:
+    print("✅ OpenAI Agents package imported successfully")
+except ImportError as e:
+    print(f"❌ OpenAI Agents import failed: {e}")
+
+try:
+    print("✅ FastAPI package imported successfully")
+except ImportError as e:
+    print(f"❌ FastAPI import failed: {e}")
+
+try:
+    print("✅ Pydantic package imported successfully")
+except ImportError as e:
+    print(f"❌ Pydantic import failed: {e}")
+
+try:
+    print("✅ Supabase package imported successfully")
+except ImportError as e:
+    print(f"❌ Supabase import failed: {e}")
+
+try:
+    print("✅ Aiohttp package imported successfully")
+except ImportError as e:
+    print(f"❌ Aiohttp import failed: {e}")
+
+try:
+    print("✅ Python-dotenv package imported successfully")
+except ImportError as e:
+    print(f"❌ Python-dotenv import failed: {e}")
+
+try:
+    print("✅ Structlog package imported successfully")
+except ImportError as e:
+    print(f"❌ Structlog import failed: {e}")
+
+print("\n🎉 Environment test completed!")