Conversation
Introduces a new /api/health endpoint for simple health checks without external dependencies. Updates the deployment healthcheck path in railway.json to use this new endpoint.
There was a problem hiding this comment.
Pull request overview
This PR introduces a lightweight health check endpoint specifically designed for deployment health checks. The previous health check used the /api/models endpoint which makes external calls to the prompture library for model discovery, making it unsuitable as a health check probe.
Changes:
- Added a new
/api/healthendpoint that returns a simple{"status": "ok"}response without making any external calls - Updated Railway deployment configuration to use the new
/api/healthendpoint instead of/api/modelsfor health checks
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| agentsite/api/app.py | Adds the new /api/health GET endpoint that returns a simple status response |
| railway.json | Updates the healthcheckPath from /api/models to /api/health |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @app.get("/api/health") | ||
| async def health(): | ||
| return {"status": "ok"} |
There was a problem hiding this comment.
The new /api/health endpoint lacks test coverage. Since this repository has comprehensive test coverage for API endpoints in tests/test_api.py (including tests for /api/models, /api/projects, and other endpoints), you should add a test for the new health check endpoint to maintain consistency with the codebase's testing practices.
Introduces a new /api/health endpoint for simple health checks without external dependencies. Updates the deployment healthcheck path in railway.json to use this new endpoint.