fix(cleanup-task): ruff/mypy strict-Blocker beheben

- asyncio.TimeoutError → TimeoutError (built-in, py312) in stop() und _loop()
- logger.warning-Zeile auf 4 Zeilen aufgeteilt (> 100 Zeichen)
- unbenutztes `import asyncio` aus test_cleanup_task.py entfernt
- -> None Return-Type auf alle async def test_*-Funktionen ergänzt
- Single-Use-Hinweis in CleanupTask-Docstring ergänzt

Refs #93

Author: strausmann

backend/app/services/cleanup_task.py

@@ -25,6 +25,9 @@ class CleanupTask:
 
     Der Loop führt beim Start sofort einen ersten Run durch, danach in ``interval``-Abständen.
     Exceptions in _run_once werden geloggt und der Loop läuft weiter (fail-soft).
+
+    Single-Use: nach stop() kann die Instanz nicht restartet werden.
+    Lifespan startet eine neue Instanz beim App-Startup.
     """
 
     def __init__(
@@ -56,9 +59,12 @@ async def stop(self, timeout_s: float = 5.0) -> None:
         if self._task is not None:
             try:
                 await asyncio.wait_for(self._task, timeout=timeout_s)
-            except asyncio.TimeoutError:
+            except TimeoutError:
                 self._task.cancel()
-                logger.warning("CleanupTask hat sich nicht in %ss beendet, Task gecancelled", timeout_s)
+                logger.warning(
+                    "CleanupTask hat sich nicht in %ss beendet, Task gecancelled",
+                    timeout_s,
+                )
             self._task = None
 
     async def _loop(self) -> None:
@@ -70,7 +76,7 @@ async def _loop(self) -> None:
                     self._stopping.wait(),
                     timeout=self._interval.total_seconds(),
                 )
-            except asyncio.TimeoutError:
+            except TimeoutError:
                 await self._run_once()
 
     async def _run_once(self) -> None:

backend/tests/unit/services/test_cleanup_task.py

@@ -2,7 +2,6 @@
 
 from __future__ import annotations
 
-import asyncio
 from datetime import timedelta
 from unittest.mock import AsyncMock
 
@@ -11,14 +10,14 @@
 
 
 @pytest.mark.asyncio
-async def test_cleanup_validates_retention_days():
+async def test_cleanup_validates_retention_days() -> None:
     store = AsyncMock()
     with pytest.raises(ValueError, match="retention_days must be >= 1"):
         CleanupTask(store=store, retention_days=0)
 
 
 @pytest.mark.asyncio
-async def test_cleanup_initial_run_on_start():
+async def test_cleanup_initial_run_on_start() -> None:
     store = AsyncMock()
     store.evict_terminal_older_than.return_value = 3
     task = CleanupTask(store=store, retention_days=30, interval=timedelta(seconds=99))
@@ -33,7 +32,7 @@ async def test_cleanup_initial_run_on_start():
 
 
 @pytest.mark.asyncio
-async def test_cleanup_fail_soft_on_exception():
+async def test_cleanup_fail_soft_on_exception() -> None:
     store = AsyncMock()
     store.evict_terminal_older_than.side_effect = RuntimeError("boom")
     task = CleanupTask(store=store, retention_days=30, interval=timedelta(seconds=99))