From 6c73a06539f793659a88439cfc78169cf23fb85c Mon Sep 17 00:00:00 2001 From: gilesb Date: Sun, 11 Jan 2026 18:02:24 +0000 Subject: [PATCH] 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 --- app/__init__.py | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/app/__init__.py b/app/__init__.py index 7a2b65c..96aea6a 100644 --- a/app/__init__.py +++ b/app/__init__.py @@ -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