Document dependency consolidation candidates - #423
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Team Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
Thermo-nuclear review: APPROVE AS A PROPOSED ADRNo code-quality blocker. This is documentation only and, importantly, it does not authorize dependency accumulation: every candidate is gated on deleting/centralizing measured code, Tier 2 requires a concrete problem/prototype, and the ADR remains Process note: the PR targets the older No runtime code, 1,000-line threshold, or CUA issue applies. |
Summary
strum,ts-rs,serde_with,url,parking_lot,itertools,bon,serde_path_to_error)tokio-util,secrecy,moka, HTTP middleware/retry,rust_decimal,phf)Audit evidence
ProviderId: 70 variants;rust/src/core/provider.rs: 1,131 LOCFetchContext: 26 manual construction siteslock().unwrap*sites, including 8 explicit poison-recovery pathsValidation
port/upstream-0.56.0before commitgit diff --checkpassedThis ADR stays Proposed so review can reject or reorder individual candidates before any implementation PR lands.