feat: better health check (#64)

Author: bolinocroustibat

.env.example

@@ -1,7 +1,7 @@
 MCP_HOST=127.0.0.1
 MCP_PORT="8000"
 MCP_ENV="local"
-DATAGOUV_ENV="prod"
+DATAGOUV_API_ENV="prod"
 LOG_LEVEL="INFO"
 
 # Matomo tracking

main.py

@@ -17,6 +17,8 @@
 
 init_sentry()
 
+SERVER_START_TIME = datetime.now(timezone.utc)
+
 # Configure logging
 LOGGER_NAME = "datagouv_mcp"
 
@@ -69,15 +71,20 @@ async def app(scope, receive, send):
 
             # Handle /health endpoint (no tracking)
             if path == "/health":
-                timestamp = datetime.now(timezone.utc).isoformat()
                 # Get version from package metadata (managed by setuptools-scm)
                 try:
                     app_version = version("datagouv-mcp")
                 except PackageNotFoundError:
                     app_version = "unknown"
 
                 body = json.dumps(
-                    {"status": "ok", "timestamp": timestamp, "version": app_version}
+                    {
+                        "status": "ok",
+                        "uptime_since": SERVER_START_TIME.isoformat(),
+                        "version": app_version,
+                        "env": os.getenv("MCP_ENV", "unknown"),
+                        "data_env": os.getenv("DATAGOUV_API_ENV", "unknown"),
+                    }
                 ).encode("utf-8")
                 headers = [
                     (b"content-type", b"application/json"),

tests/test_health_endpoint.py

@@ -13,6 +13,8 @@ async def test_health_endpoint_returns_ok_status():
     assert response.status_code == 200
     payload = response.json()
     assert payload.get("status") == "ok"
-    assert "timestamp" in payload
+    assert "uptime_since" in payload
     assert "version" in payload
     assert isinstance(payload.get("version"), str)
+    assert "env" in payload
+    assert "data_env" in payload