Merge pull request #487 from christianlouis/copilot/fix-mypy-error-templates

Fix mypy TemplateResponse annotation in db_wizard and harden owner fallback in process_document

app/tasks/process_document.py

@@ -56,8 +56,9 @@ def process_document(
       3. If force_cloud_ocr is True, skip local text extraction and use cloud OCR
     """
     # Fall back to the configured default_owner_id when no explicit owner was provided
-    if owner_id is None and settings.default_owner_id:
+    default_owner_id = settings.default_owner_id
-        owner_id = settings.default_owner_id
+    if owner_id is None and isinstance(default_owner_id, str) and default_owner_id.strip():
+        owner_id = default_owner_id
 
     task_id = self.request.id
     logger.info(f"[{task_id}] Starting document processing: {original_local_file}")

app/views/db_wizard.py

@@ -8,6 +8,7 @@ and migrating data from one database to another.
 import logging
 
 from fastapi import Request
+from fastapi.responses import Response
 
 from app.config import settings
 from app.views.base import APIRouter, templates
@@ -17,7 +18,7 @@ router = APIRouter()
 
 
 @router.get("/database-wizard")
-async def database_wizard(request: Request) -> templates.TemplateResponse:
+async def database_wizard(request: Request) -> Response:
     """Render the database configuration wizard page."""
     return templates.TemplateResponse(
         "db_wizard.html",