Housekeeping, surfaced during the #279 review.
The Rust tree carries pre-existing clippy warnings unrelated to any current work. They
are noise in every review of src/, which is exactly how a real warning gets scrolled
past.
Clean them in one pass, then consider whether clippy -D warnings belongs in CI so the
debt cannot rebuild. Keep the cleanup commit separate from any behavioural change so it
stays trivially reviewable.
Note for whoever picks this up: format only the files you change. A whole-tree
cargo fmt reflows ~14 unrelated files because of the same pre-existing debt, which
turns a one-line fix into an unreviewable diff.
🤖 Generated with Claude Code
https://claude.ai/code/session_01EBwY8GiRjf1zJBaEZ2CiEa
Housekeeping, surfaced during the #279 review.
The Rust tree carries pre-existing clippy warnings unrelated to any current work. They
are noise in every review of
src/, which is exactly how a real warning gets scrolledpast.
Clean them in one pass, then consider whether
clippy -D warningsbelongs in CI so thedebt cannot rebuild. Keep the cleanup commit separate from any behavioural change so it
stays trivially reviewable.
Note for whoever picks this up: format only the files you change. A whole-tree
cargo fmtreflows ~14 unrelated files because of the same pre-existing debt, whichturns a one-line fix into an unreviewable diff.
🤖 Generated with Claude Code
https://claude.ai/code/session_01EBwY8GiRjf1zJBaEZ2CiEa