Skip to content

refactor(binance): enhance websocket architecture and cleanup codebase - #44

Merged
Praying merged 1 commit into
mainfrom
dev
Jan 30, 2026
Merged

Praying merged 1 commit into
mainfrom
dev

Conversation

@Praying

@Praying Praying commented Jan 30, 2026

Copy link
Copy Markdown
Owner
  • Refactor Binance WebSocket implementation to use ConnectionManager for public and private connections
  • Implement SubscriptionManager for better stream tracking and auto-resubscription logic
  • Add ListenKeyManager to handle listenKeyExpired events and auto-regeneration
  • Refactor MessageRouter and handlers to support new architecture
  • Update Bitget exchange capabilities and implementation details
  • Deprecate legacy ccxt-exchanges/src/exchange.rs re-exports in favor of ccxt-core
  • Cleanup unwrap() usage in tests, replacing with expect() for better diagnostics
  • Update benchmarks for all exchanges (Binance, Bybit, Hyperliquid, OKX) to match API changes
  • Suppress clippy warnings in benchmarks and tests

- Refactor Binance WebSocket implementation to use `ConnectionManager` for public and private connections
- Implement `SubscriptionManager` for better stream tracking and auto-resubscription logic
- Add `ListenKeyManager` to handle `listenKeyExpired` events and auto-regeneration
- Refactor `MessageRouter` and `handlers` to support new architecture
- Update `Bitget` exchange capabilities and implementation details
- Deprecate legacy `ccxt-exchanges/src/exchange.rs` re-exports in favor of `ccxt-core`
- Cleanup `unwrap()` usage in tests, replacing with `expect()` for better diagnostics
- Update benchmarks for all exchanges (Binance, Bybit, Hyperliquid, OKX) to match API changes
- Suppress clippy warnings in benchmarks and tests
@Praying
Praying merged commit 8c544ff into main Jan 30, 2026
9 checks passed
@Praying
Praying deleted the dev branch January 30, 2026 11:25
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