Standardize error handling: HTTPException everywhere

backend/api/routes/control.py

@@ -4,8 +4,7 @@ from __future__ import annotations
 
 import logging
 
-from fastapi import APIRouter
-from fastapi.responses import JSONResponse
+from fastapi import APIRouter, HTTPException
 
 from backend.api.deps import get_datahub, get_redis_client
 
@@ -21,9 +20,9 @@ async def control_status():
         data = redis.get_json("control:last_tick")
         if data:
             return data
-    return JSONResponse(
+    raise HTTPException(
         status_code=503,
-        content={"error": "No tick result available (worker may not have run yet)"},
+        detail="No tick result available (worker may not have run yet)",
     )
 
 
@@ -42,10 +41,10 @@ async def control_plan():
         with open(DAILY_PLAN_PATH) as f:
             return json.load(f)
     except FileNotFoundError:
-        return JSONResponse(status_code=404, content={"error": "No plan available"})
+        raise HTTPException(status_code=404, detail="No plan available")
     except Exception as exc:
         log.error("Failed to load plan from file: %s", exc)
-        return JSONResponse(status_code=500, content={"error": "Plan loading failed"})
+        raise HTTPException(status_code=500, detail="Plan loading failed")
 
 
 @router.get("/budget")
@@ -56,9 +55,9 @@ async def control_budget():
         data = redis.get_json("control:budget")
         if data:
             return data
-    return JSONResponse(
+    raise HTTPException(
         status_code=503,
-        content={"error": "No budget data available"},
+        detail="No budget data available",
     )
 
 
@@ -71,7 +70,4 @@ async def control_trackers():
         return {"trackers": snapshot.get("trackers", {}), "source": "ThingsBoard"}
     except Exception as exc:
         log.error("Tracker fetch failed: %s", exc)
-        return JSONResponse(
-            status_code=502,
-            content={"error": "Tracker fetch failed"},
-        )
+        raise HTTPException(status_code=502, detail="Tracker fetch failed")