mirror of
https://github.com/CopilotKit/CopilotKit.git
synced 2026-09-14 16:26:20 +08:00
Jordan Ritter
31a7d23d1e
The Deployed Starters smoke test in integration-smoke.spec.ts uses checkHealth() with its default paths list ["/api/health", "/health"]. All deployed starters serve their health endpoint at /api/health (standard Next.js convention); /health returns a Next.js 404 HTML page. When /api/health returns a legitimate 5xx (e.g. 503 "agent degraded"), the fallback silently probes /health and the resulting 404 becomes the reported failure — so the Slack alert reads "status=404 path=/health" and implies a route-missing bug, hiding the real upstream 503. Shape A fix: change the default to ["/api/health"] only. No fallback. The reported status and body are now the actual /api/health response. starter-smoke.spec.ts passes an explicit paths argument and is unaffected. The integration backend L1 health check also targets /api/health on all deployed backends (verified). Evidence runs (all same failure mode): - https://github.com/CopilotKit/CopilotKit/actions/runs/24619147643 - https://github.com/CopilotKit/CopilotKit/actions/runs/24618944101 - https://github.com/CopilotKit/CopilotKit/actions/runs/24617315522