Skip to content

feat(streaming): add connection health monitoring#38

Merged
AKKI0511 merged 4 commits intomainfrom
implement-streaming-connection-health-monitoring
Aug 29, 2025
Merged

feat(streaming): add connection health monitoring#38
AKKI0511 merged 4 commits intomainfrom
implement-streaming-connection-health-monitoring

Conversation

@AKKI0511
Copy link
Copy Markdown
Owner

Summary

  • implement StreamingHealthMonitor with latency checks, throughput metrics and reconnection handling
  • add alert, metrics collector and recovery manager utilities
  • integrate health monitoring into streaming gateway and configuration
  • add tests for alerting and reconnection logic

Testing

  • poetry run pre-commit run --files config/streaming.yaml quanttradeai/streaming/__init__.py quanttradeai/streaming/gateway.py quanttradeai/streaming/monitoring/__init__.py quanttradeai/streaming/monitoring/health_monitor.py quanttradeai/streaming/monitoring/alerts.py quanttradeai/streaming/monitoring/metrics_collector.py quanttradeai/streaming/monitoring/recovery_manager.py tests/streaming/test_health_monitor.py
  • PYTHONPATH=. pytest tests/streaming/test_health_monitor.py

https://chatgpt.com/codex/tasks/task_e_68b22602cfb4832aafd8ff80439bbcf6

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Codex Review: Here are some suggestions.

Reply with @codex fix comments to fix any unresolved comments.

About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you open a pull request for review, mark a draft as ready, or comment "@codex review". If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex fix this CI failure" or "@codex address that feedback".

@cursor
Copy link
Copy Markdown

cursor bot commented Aug 29, 2025

Bugbot found 1 bug

To see it, activate your membership in the Cursor dashboard.

@AKKI0511
Copy link
Copy Markdown
Owner Author

@codex fix this CI failure

@chatgpt-codex-connector
Copy link
Copy Markdown

Summary

Testing

  • SKIP=test poetry run pre-commit run --files config/streaming.yaml quanttradeai/streaming/__init__.py quanttradeai/streaming/gateway.py quanttradeai/streaming/monitoring/__init__.py quanttradeai/streaming/monitoring/health_monitor.py quanttradeai/streaming/monitoring/alerts.py quanttradeai/streaming/monitoring/metrics_collector.py quanttradeai/streaming/monitoring/recovery_manager.py tests/streaming/test_health_monitor.py
  • poetry run pytest tests/streaming/test_health_monitor.py

View task →

@AKKI0511 AKKI0511 merged commit c1fa667 into main Aug 29, 2025
1 check passed
@AKKI0511 AKKI0511 deleted the implement-streaming-connection-health-monitoring branch August 29, 2025 23:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant