nautechsystems / nautilus_trader
Code health
This codebase scores 5.4 out of 10 for code health, which we rate needs work. It also scores maintainability 6.1 and static performance 9.9 out of 10. The three are scored separately and never blended into one number. Risk is concentrated, as it usually is: 766 of 5,357 files are git hotspots, and they average 3.5 — which is where the fixes pay off most.
Do next
4 things worth doing this quarter.
Worth planning
Break up submit_order_list_command (CCN 24, 219 lines)
crates/adapters/polymarket/src/execution/orders.rs:554
submit_order_list_command nests its control flow deeply, and part of it is duplicated, so splitting it also removes a copy; 2 files import it, changed 10 times in 90 days.
- gain
- up to +1.4 health on this file(inferred)
- finding
- Nested complexity
- size
- CCN 24, 219 lines, nests 6 deep
- severity
- high
Medium effort, worth a checkBreak up dispatch_ws_message (CCN 23, 107 lines)
crates/adapters/bitmex/src/websocket/dispatch.rs:270
dispatch_ws_message nests its control flow deeply, and part of it is duplicated, so splitting it also removes a copy; 1 file imports it, changed 2 times in 90 days.
- gain
- up to +1.2 health on this file(inferred)
- finding
- Nested complexity
- size
- CCN 23, 107 lines, nests 10 deep
- severity
- critical
Medium effort, worth a checkDelete 1,123 unused symbols and files (16,452 lines)
Nothing in the graph reaches them, across 78 files; every reader and agent pays to skip them.
- symbols
- 1123
- lines
- 16,452
- safe to delete
- Yes(inferred)
Medium effortBreak up py_connect (CCN 41, 296 lines)
crates/adapters/kraken/src/python/websocket_spot.rs:440
py_connect nests its control flow deeply, and part of it is duplicated, so splitting it also removes a copy; 6 files import it, changed 9 times in 90 days.
- gain
- up to +1.0 health on this file(inferred)
- finding
- Nested complexity
- size
- CCN 41, 296 lines, nests 8 deep
- severity
- critical
Medium effort, worth a check
Improve what Repowise can see
Add a test coverage report
Without one, Repowise cannot tell tested code from untested code, so every test-related action says “unknown”.
- line coverage
- Unknown, no report
Small effort
- Lines of code
- 1.7M
Recent activity
All commitsCommits
- 13b24ca5Fix Binance Futures private user stream URLs2mo ago
- a6242e32Fix v2 matching engine last-price triggers2mo ago
- f507c331Add LiveNode constructed strategy regressions2mo ago
- 79a06923Support add_strategy(instance) on LiveNode (#4487)2mo ago
- c12c58cfFix Binance Futures shared REST rate limits2mo ago
- ea8d0758Fix Miri toolchain drift2mo ago
Decisions
- Fix Polymarket Gamma instrument loading by chunking and matching silent capsproposed · 2mo ago
- Refactor DataEngine instrument subscribers to message bus patternproposed · 2mo ago
- Refactor Polymarket adapter: WS dispatch, instrument handling, and market discoveryproposed · 2mo ago
- Cache Kraken Spot symbol to handle delta-only exec framesproposed · 2mo ago
- Fix Interactive Brokers crypto order sizing for SELL ordersproposed · 2mo ago
- Fix IB v2 fills dropped on non-UTC execution timestampsproposed · 2mo ago
Everything we found
25,830 open
Every detector's open count by area, each with its worst example. An inventory, not a to-do list: the actions above are the part worth doing.
- worst item is critical severity. Code health17,860Untested hotspot · crates/data/src/engine/mod.rs68 findings · 10.0 deducted
- worst item is medium severity. Refactoring1,849PortfolioAnalyzer+1.7 health, XL effort · crates/analysis/src/python/analyzer.rs
- worst item is medium severity. Decisions810crates/data/tests/engine.rs44 awaiting review · 766 hotspots ungoverned
- worst item is medium severity. Ownership3,936crates/execution/src/matching_engine/engine.rs88% single-owner concentration, 57 commits in 90d
- worst item is low severity. Dead code1,375futures_contractUnreachable function (16 lines)
Composition
Open the graphExplore this codebase
- Docs0 pages across 11 modules
- ChatAsk this codebase a question and get an answer with its sources.
- Files5,357 files with per-file docs, health and history
- ArchitectureDependency graph, layers, and 61 entry points
- Code healthPer-file scores, 21,903 open findings, coverage and refactoring targets
- Knowledge graphEntities, communities, and the paths between them
- Change couplingFiles that keep changing together, mined from commit history
- CommitsChange-risk ranked history with agent provenance
- ContributorsBus factor, per-file maintainers, and the human/agent split
- StatsSize class, origin, lifetime churn, rhythm and records
- CostsWhat indexing this snapshot cost, by model and by run.