refactor: simplify function calls and improve readability

- Consolidate parameters in get_or_set_cache calls in libraries.py, specs.py, and stats.py
- Clean up function definition in cache.py for _background_refresh

Author: MarkusNeusinger

api/cache.py

@@ -245,9 +245,7 @@ def _schedule_refresh(key: str, factory: Callable[[], Awaitable[Any]]) -> None:
     asyncio.create_task(_background_refresh(key, factory, lock))
 
 
-async def _background_refresh(
-    key: str, factory: Callable[[], Awaitable[Any]], lock: asyncio.Lock
-) -> None:
+async def _background_refresh(key: str, factory: Callable[[], Awaitable[Any]], lock: asyncio.Lock) -> None:
     """Run factory in background and update cache. Errors are logged, not raised."""
     async with lock:
         try:

api/routers/libraries.py

@@ -62,10 +62,7 @@ async def _fetch() -> dict:
         }
 
     return await get_or_set_cache(
-        cache_key("libraries"),
-        _fetch,
-        refresh_after=settings.cache_refresh_after,
-        refresh_factory=_refresh_libraries,
+        cache_key("libraries"), _fetch, refresh_after=settings.cache_refresh_after, refresh_factory=_refresh_libraries
     )
 
 

api/routers/specs.py

@@ -23,7 +23,11 @@ async def _refresh_specs_list() -> list[SpecListItem]:
         specs = await repo.get_all()
         return [
             SpecListItem(
-                id=spec.id, title=spec.title, description=spec.description, tags=spec.tags, library_count=len(spec.impls)
+                id=spec.id,
+                title=spec.title,
+                description=spec.description,
+                tags=spec.tags,
+                library_count=len(spec.impls),
             )
             for spec in specs
             if spec.impls
@@ -43,17 +47,18 @@ async def _fetch() -> list[SpecListItem]:
         specs = await repo.get_all()
         return [
             SpecListItem(
-                id=spec.id, title=spec.title, description=spec.description, tags=spec.tags, library_count=len(spec.impls)
+                id=spec.id,
+                title=spec.title,
+                description=spec.description,
+                tags=spec.tags,
+                library_count=len(spec.impls),
             )
             for spec in specs
             if spec.impls
         ]
 
     return await get_or_set_cache(
-        cache_key("specs_list"),
-        _fetch,
-        refresh_after=settings.cache_refresh_after,
-        refresh_factory=_refresh_specs_list,
+        cache_key("specs_list"), _fetch, refresh_after=settings.cache_refresh_after, refresh_factory=_refresh_specs_list
     )
 
 

api/routers/stats.py

@@ -72,8 +72,5 @@ async def _fetch() -> StatsResponse:
         return StatsResponse(specs=len(specs_with_impls), plots=total_impls, libraries=len(libraries))
 
     return await get_or_set_cache(
-        cache_key("stats"),
-        _fetch,
-        refresh_after=settings.cache_refresh_after,
-        refresh_factory=_refresh_stats,
+        cache_key("stats"), _fetch, refresh_after=settings.cache_refresh_after, refresh_factory=_refresh_stats
     )