Fix redirect handlers to pass dependencies explicitly

The /run/{id} and /recipe/{id} redirects were calling route handlers
directly without passing the required service dependencies.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
gilesb
2026-01-11 18:02:24 +00:00
parent 970faa3fa0
commit 6c73a06539

View File

@@ -69,11 +69,12 @@ def create_app() -> FastAPI:
app.include_router(runs.router, prefix="/runs", tags=["runs"])
# Also mount at /run for single-run detail URLs
from fastapi import APIRouter
from .dependencies import get_run_service, get_recipe_service
run_detail_router = APIRouter()
@run_detail_router.get("/{run_id}")
async def run_detail_redirect(run_id: str, request: Request):
from .routers.runs import run_detail
return await run_detail(run_id, request)
return await run_detail(run_id, request, run_service=get_run_service())
app.include_router(run_detail_router, prefix="/run", tags=["runs"])
# Recipes router - handles both /recipes and /recipe/{id} patterns
@@ -82,7 +83,7 @@ def create_app() -> FastAPI:
@recipe_detail_router.get("/{recipe_id}")
async def recipe_detail_redirect(recipe_id: str, request: Request):
from .routers.recipes import get_recipe
return await get_recipe(recipe_id, request)
return await get_recipe(recipe_id, request, recipe_service=get_recipe_service())
app.include_router(recipe_detail_router, prefix="/recipe", tags=["recipes"])
# Cache router - handles /cache and /media