src/main.py

@@ -22,10 +22,6 @@ FRONTEND_DIR = Path(__file__).resolve().parent.parent / "frontend"
 class ContextPayloadError(ValueError):
     """Raised when Gitea returns a structurally invalid context payload."""
 
-
-class ReadinessPayloadError(ValueError):
-    """Raised when Gitea returns a structurally invalid readiness payload."""
-
 app.add_middleware(
     CORSMiddleware,
     allow_origins=["*"],
@@ -60,19 +56,13 @@ async def readiness():
             current_user(), timeout=READINESS_TIMEOUT_SECONDS
         )
         if not isinstance(user, dict) or not user.get("login"):
-            raise ReadinessPayloadError(
+            raise ValueError("Gitea current-user response did not include a login")
-                "Gitea current-user response did not include a login"
-            )
     except Exception as exc:
         timed_out = isinstance(exc, TimeoutError)
         error_message = (
             f"Gitea readiness check timed out after {READINESS_TIMEOUT_SECONDS:g}s"
             if timed_out
-            else (
+            else str(exc)
-                str(exc)
-                if isinstance(exc, ReadinessPayloadError)
-                else "Gitea readiness check is temporarily unavailable"
-            )
         )
         return JSONResponse(
             {

tests/test_health.py

@@ -29,21 +29,17 @@ async def test_readiness_endpoint_reports_connected_gitea_user(monkeypatch):
 
 
 @pytest.mark.anyio
-async def test_readiness_endpoint_hides_unexpected_upstream_error_details(monkeypatch):
+async def test_readiness_endpoint_returns_503_when_gitea_is_unavailable(monkeypatch):
     async def unavailable_user():
-        raise RuntimeError("secret internal upstream detail")
+        raise RuntimeError("connection refused")
 
     monkeypatch.setattr(main, "current_user", unavailable_user)
 
     response = await main.readiness()
 
     assert response.status_code == 503
-    assert json.loads(response.body) == {
+    assert b'"status":"not_ready"' in response.body
-        "status": "not_ready",
+    assert b'"error":"connection refused"' in response.body
-        "service": "stackchain-dashboard",
-        "error": "Gitea readiness check is temporarily unavailable",
-    }
-    assert b"secret internal upstream detail" not in response.body
 
 
 @pytest.mark.anyio