Skip to content

fix(api): require heartbeat SigNoz rate-limit checks#23

Merged
samuelclay merged 2 commits into
mainfrom
fix/heartbeat-signoz-checklist
Jul 7, 2026
Merged

fix(api): require heartbeat SigNoz rate-limit checks#23
samuelclay merged 2 commits into
mainfrom
fix/heartbeat-signoz-checklist

Conversation

@samuelclay

Copy link
Copy Markdown
Collaborator

Summary

  • require heartbeat investigations to run a realtime-replica SigNoz checklist before final answers
  • include targeted searches for Too Many Requests, 429, ConnectError, HTTP error, TransportStep boot failures, and livekit_ffi
  • add a unit test that locks the checklist into the generated Watch Agent prompt

Validation

  • uv run ruff format api/workflows/heartbeat_investigation.py tests/test_heartbeat_investigation_workflow.py
  • uv run ruff check api/workflows/heartbeat_investigation.py tests/test_heartbeat_investigation_workflow.py
  • uv run pytest tests/test_heartbeat_investigation_workflow.py
  • just build-one api
  • docker run --rm -i --entrypoint /app/.venv/bin/python centaur-api:latest - prompt checklist assertion

Local deploy smoke

Blocked on this workstation: there is no local Kubernetes context and the required local secret env vars are absent (OP_SERVICE_ACCOUNT_TOKEN, OP_VAULT, SLACK_BOT_TOKEN, SLACK_SIGNING_SECRET, SLACKBOT_API_KEY).

Fork tracking

Pending follow-up commit after PR URL is known.

@samuelclay samuelclay merged commit db877da into main Jul 7, 2026
7 checks passed
@samuelclay samuelclay deleted the fix/heartbeat-signoz-checklist branch July 7, 2026 17:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant