fix: ruff errors and add ruff to pyproject dev dependancies to avoid running pre-commit for ruff checking

Author: ix-56h

pyproject.toml

@@ -110,3 +110,8 @@ asyncio_mode = "auto"
 asyncio_default_fixture_loop_scope = "function"
 python_classes = "Test*"
 python_functions = "test_*"
+
+[dependency-groups]
+dev = [
+    "ruff>=0.12.1",
+]

src/server/main.py

@@ -6,15 +6,14 @@
 from pathlib import Path
 
 from dotenv import load_dotenv
-from fastapi import FastAPI, Request
+from fastapi import FastAPI
 from fastapi.responses import FileResponse, HTMLResponse
 from fastapi.staticfiles import StaticFiles
 from slowapi.errors import RateLimitExceeded
 from starlette.middleware.trustedhost import TrustedHostMiddleware
 
 from server.routers import dynamic, index
 from server.routers.ingest import router as ingest
-from server.server_config import templates
 from server.server_utils import lifespan, limiter, rate_limit_exception_handler
 
 # Load environment variables from .env file
@@ -73,6 +72,7 @@ async def head_root() -> HTMLResponse:
     """
     return HTMLResponse(content=None, headers={"content-type": "text/html; charset=utf-8"})
 
+
 @app.get("/robots.txt", include_in_schema=False)
 async def robots() -> FileResponse:
     """Serve the ``robots.txt`` file to guide search engine crawlers.

src/server/routers/dynamic.py

@@ -3,9 +3,7 @@
 from fastapi import APIRouter, Request
 from fastapi.responses import HTMLResponse
 
-from gitingest.utils.compat_typing import Annotated
 from server.server_config import templates
-from server.server_utils import limiter
 
 router = APIRouter()
 
@@ -38,4 +36,4 @@ async def catch_all(request: Request, full_path: str) -> HTMLResponse:
             "repo_url": full_path,
             "default_file_size": 243,
         },
-    )
+    )

src/server/routers/index.py

@@ -1,13 +1,9 @@
 """Module defining the FastAPI router for the home page of the application."""
 
-from fastapi import APIRouter, Depends, Request
+from fastapi import APIRouter, Request
 from fastapi.responses import HTMLResponse
 
-from gitingest.utils.compat_typing import Annotated
-from server.models import QueryForm
-from server.query_processor import process_query
 from server.server_config import EXAMPLE_REPOS, templates
-from server.server_utils import limiter
 
 router = APIRouter()
 
@@ -38,4 +34,4 @@ async def home(request: Request) -> HTMLResponse:
             "examples": EXAMPLE_REPOS,
             "default_file_size": 243,
         },
-    )
+    )

tests/test_flow_integration.py

@@ -32,9 +32,6 @@ def mock_static_files(mocker: MockerFixture) -> None:
     return mock_static
 
 
-
-
-
 @pytest.fixture(scope="module", autouse=True)
 def cleanup_tmp_dir() -> Generator[None, None, None]:
     """Remove ``/tmp/gitingest`` after this test-module is done."""
@@ -61,7 +58,7 @@ async def test_remote_repository_analysis(request: pytest.FixtureRequest) -> Non
 
     response = client.post("/api/ingest", data=form_data)
     assert response.status_code == status.HTTP_200_OK, f"Form submission failed: {response.text}"
-    
+
     # Check that response is JSON
     response_data = response.json()
     assert "result" in response_data
@@ -87,7 +84,7 @@ async def test_invalid_repository_url(request: pytest.FixtureRequest) -> None:
     response = client.post("/api/ingest", data=form_data)
     # Should return 400 for invalid repository
     assert response.status_code == status.HTTP_400_BAD_REQUEST, f"Request failed: {response.text}"
-    
+
     # Check that response is JSON error
     response_data = response.json()
     assert "error" in response_data
@@ -108,8 +105,9 @@ async def test_large_repository(request: pytest.FixtureRequest) -> None:
 
     response = client.post("/api/ingest", data=form_data)
     # This might fail with 400 if repo doesn't exist, or succeed with 200
-    assert response.status_code in [status.HTTP_200_OK, status.HTTP_400_BAD_REQUEST], f"Request failed: {response.text}"
-    
+    _valid = [status.HTTP_200_OK, status.HTTP_400_BAD_REQUEST]
+    assert response.status_code in _valid, f"Request failed: {response.text}"
+
     response_data = response.json()
     if response.status_code == status.HTTP_200_OK:
         assert "result" in response_data
@@ -132,8 +130,9 @@ def make_request() -> None:
             "token": "",
         }
         response = client.post("/api/ingest", data=form_data)
-        assert response.status_code in [status.HTTP_200_OK, status.HTTP_400_BAD_REQUEST], f"Request failed: {response.text}"
-        
+        _valid = [status.HTTP_200_OK, status.HTTP_400_BAD_REQUEST]
+        assert response.status_code in _valid, f"Request failed: {response.text}"
+
         response_data = response.json()
         if response.status_code == status.HTTP_200_OK:
             assert "result" in response_data
@@ -160,8 +159,9 @@ async def test_large_file_handling(request: pytest.FixtureRequest) -> None:
     }
 
     response = client.post("/api/ingest", data=form_data)
-    assert response.status_code in [status.HTTP_200_OK, status.HTTP_400_BAD_REQUEST], f"Request failed: {response.text}"
-    
+    _valid = [status.HTTP_200_OK, status.HTTP_400_BAD_REQUEST]
+    assert response.status_code in _valid, f"Request failed: {response.text}"
+
     response_data = response.json()
     if response.status_code == status.HTTP_200_OK:
         assert "result" in response_data
@@ -183,8 +183,9 @@ async def test_repository_with_patterns(request: pytest.FixtureRequest) -> None:
     }
 
     response = client.post("/api/ingest", data=form_data)
-    assert response.status_code in [status.HTTP_200_OK, status.HTTP_400_BAD_REQUEST], f"Request failed: {response.text}"
-    
+    _valid = [status.HTTP_200_OK, status.HTTP_400_BAD_REQUEST]
+    assert response.status_code in _valid, f"Request failed: {response.text}"
+
     response_data = response.json()
     if response.status_code == status.HTTP_200_OK:
         assert "result" in response_data