Problem
_resolve_source_or_404 is copy-pasted into all 4 analytics modules introduced in #3436:
api/analytics_quality.pyapi/analytics_evolution.pyapi/analytics_code_review.pyapi/analytics_code_generation.py
The copies are also inconsistent: two lazy-import HTTPException inside the function body while two rely on a module-level import — the docstring says "lazy import" in all four which is misleading.
Fix
Extract to a single shared location, e.g. a new autobot-backend/api/analytics_shared.py:
```python
async def resolve_source_or_404(source_id: Optional[str]) -> None:
"""Raise HTTP 404 if source_id is provided but not found."""
if source_id is None:
return
from api.codebase_analytics.endpoints.shared import resolve_source_root
source_root = await resolve_source_root(source_id)
if not source_root:
raise HTTPException(status_code=404, detail=f"Source '{source_id}' not found")
```
Import from there in each of the 4 modules and remove the local copies.
Problem
_resolve_source_or_404is copy-pasted into all 4 analytics modules introduced in #3436:api/analytics_quality.pyapi/analytics_evolution.pyapi/analytics_code_review.pyapi/analytics_code_generation.pyThe copies are also inconsistent: two lazy-import
HTTPExceptioninside the function body while two rely on a module-level import — the docstring says "lazy import" in all four which is misleading.Fix
Extract to a single shared location, e.g. a new
autobot-backend/api/analytics_shared.py:```python
async def resolve_source_or_404(source_id: Optional[str]) -> None:
"""Raise HTTP 404 if source_id is provided but not found."""
if source_id is None:
return
from api.codebase_analytics.endpoints.shared import resolve_source_root
source_root = await resolve_source_root(source_id)
if not source_root:
raise HTTPException(status_code=404, detail=f"Source '{source_id}' not found")
```
Import from there in each of the 4 modules and remove the local copies.