refactor: remove duplicate utils.py, dead license_routes, and pytest.ini

Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com>

app/routes/license_routes.py

@@ -1,19 +0,0 @@
-from pathlib import Path
-
-from fastapi import APIRouter, HTTPException
-from fastapi.responses import PlainTextResponse
-
-router = APIRouter()
-
-
-@router.get("/licenses/lgpl.txt", response_class=PlainTextResponse)
-async def get_lgpl_license():
-    """
-    Serve the LGPL license text file
-    """
-    license_path = Path("frontend/static/licenses/lgpl.txt")
-    if not license_path.exists():
-        raise HTTPException(status_code=404, detail="License file not found")
-
-    with open(license_path, "r") as f:
-        return f.read()

app/utils.py

@@ -1,7 +0,0 @@
-# app/utils.py
-# This file is deprecated. Functions have been moved to the utils package.
-# To avoid breaking existing imports, we'll import and re-export the functions
-from app.utils.file_operations import hash_file  # noqa: F401
-from app.utils.logging import log_task_progress  # noqa: F401
-
-# These functions are now available directly from the app.utils package

pytest.ini

@@ -1,64 +0,0 @@
-[tool:pytest]
-# Pytest configuration
-testpaths = tests
-python_files = test_*.py
-python_classes = Test*
-python_functions = test_*
-
-# Output options
-addopts = 
-    --verbose
-    --strict-markers
-    --strict-config
-    --cov=app
-    --cov-report=term-missing
-    --cov-report=html
-    --cov-report=xml
-    --cov-branch
-    --cov-fail-under=0
-    # Note: Coverage threshold set to 0 initially, should be increased gradually
-    # Target: 80% coverage for production code
-
-# Markers for organizing tests
-markers =
-    unit: Unit tests for individual functions/methods
-    integration: Integration tests for API endpoints and workflows
-    slow: Tests that take significant time to run
-    security: Security-related tests
-    requires_external: Tests requiring external services (OpenAI, Azure, etc.)
-    requires_db: Tests requiring database
-    requires_redis: Tests requiring Redis
-
-# Ignore patterns
-norecursedirs = 
-    .git
-    .tox
-    dist
-    build
-    *.egg
-    __pycache__
-    .venv
-    venv
-    env
-
-# Coverage options
-[coverage:run]
-source = app
-omit = 
-    */tests/*
-    */test_*.py
-    */__pycache__/*
-    */venv/*
-    */env/*
-    */.venv/*
-
-[coverage:report]
-exclude_lines =
-    pragma: no cover
-    def __repr__
-    raise AssertionError
-    raise NotImplementedError
-    if __name__ == .__main__.:
-    if TYPE_CHECKING:
-    @abstractmethod
-    @abc.abstractmethod

tests/test_reexports.py

@@ -1,8 +1,8 @@
 """
-Tests for simple re-export modules (app/utils.py, app/frontend.py, app/utils/config_validator.py)
+Tests for simple re-export modules (app/frontend.py, app/utils/config_validator.py)
 
 These modules are simple re-exports of functions from other modules.
-We test that imports work correctly.
+We test that imports work correctly, including the app.utils package.
 """
 
 import pytest
@@ -26,12 +26,12 @@ class TestUtilsReexports:
         # Function should exist and be callable
         assert callable(log_task_progress)
 
-    def test_utils_module_is_backward_compatible(self):
-        """Test that utils module maintains backward compatibility"""
-        # The module comment says it's deprecated but maintains compatibility
+    def test_utils_package_exports(self):
+        """Test that utils package exports expected functions"""
+        # The utils package should export functions via __init__.py
         import app.utils
 
-        # Module should exist and have expected attributes
+        # Package should exist and have expected attributes
         assert hasattr(app.utils, "hash_file")
         assert hasattr(app.utils, "log_task_progress")