Skip to content

feat: add webhook retry monitoring summary#386

Open
yuliuyi717-ux wants to merge 2 commits intorohitdash08:mainfrom
yuliuyi717-ux:codex/issue-130-0313
Open

feat: add webhook retry monitoring summary#386
yuliuyi717-ux wants to merge 2 commits intorohitdash08:mainfrom
yuliuyi717-ux:codex/issue-130-0313

Conversation

@yuliuyi717-ux
Copy link

/claim #130

Summary

  • add a delivery monitoring summary endpoint for webhook retries
  • expose webhook delivery lifecycle metrics for retry/success/failure visibility
  • document the new monitoring surface in the README and OpenAPI spec

What changed

  • added GET /webhooks/deliveries/summary with backlog and failure timestamps
  • tracked webhook delivery events in Prometheus via finmind_webhook_delivery_events_total
  • added focused backend tests for summary aggregation and retry metrics

Validation

  • REDIS_URL=redis://localhost:6379/15 PYTHONPATH=. ../../.venv/bin/pytest tests/test_webhooks.py::test_webhook_delivery_summary_reports_retry_backlog tests/test_observability.py::test_metrics_endpoint_exposes_webhook_retry_metrics -q
  • python3 -m py_compile packages/backend/app/observability.py packages/backend/app/routes/webhooks.py packages/backend/app/services/webhooks.py

Demo

  • recorded a short sanitized terminal demo covering the new endpoint, metrics, and passing targeted tests

@yuliuyi717-ux
Copy link
Author

finmind_130_demo.mp4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant