Skip to content
This repository was archived by the owner on May 11, 2026. It is now read-only.

fix(ws): break reconnect-handler Arc cycle that leaked WebSocket clients (#325) - #334

Open
skyc1e wants to merge 3 commits into
Polymarket:mainfrom
skyc1e:fix/ws-reconnect-handler-arc-cycle
Open

fix(ws): break reconnect-handler Arc cycle that leaked WebSocket clients (#325)#334
skyc1e wants to merge 3 commits into
Polymarket:mainfrom
skyc1e:fix/ws-reconnect-handler-arc-cycle

style(rtds): satisfy rustfmt on collapsed Client::new expect lines

d42a0d4
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Apr 14, 2026 in 0s

86.89% (+1.34%) compared to 77264a4

View this Pull Request on Codecov

86.89% (+1.34%) compared to 77264a4

Details

Codecov Report

❌ Patch coverage is 90.85366% with 15 lines in your changes missing coverage. Please review.
✅ Project coverage is 86.89%. Comparing base (77264a4) to head (d42a0d4).

Files with missing lines Patch % Lines
src/rtds/client.rs 92.20% 6 Missing ⚠️
src/clob/ws/subscription.rs 90.00% 3 Missing ⚠️
src/rtds/subscription.rs 89.65% 3 Missing ⚠️
src/ws/task.rs 88.46% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #334      +/-   ##
==========================================
+ Coverage   85.54%   86.89%   +1.34%     
==========================================
  Files          32       33       +1     
  Lines        5167     5325     +158     
==========================================
+ Hits         4420     4627     +207     
+ Misses        747      698      -49     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.