- differ.rs: build HashMap<u64, RevisionSummary> once in packets_for_revisions
and diff_revisions_merkle instead of O(N) list_revisions() scan per revision
- merger.rs: sort index Vec in verify_packet_hash instead of cloning MB-scale
page data
- backend.rs (Tcp + Quic): eliminate std::fs::read(local_path) — file_blake3
is never validated by server; set to [0u8;32] directly; call list_revisions()
once and reuse; add W=16 pipelined send via Semaphore + into_split() for TCP,
Arc<QuicConnection> clone for QUIC
Co-Authored-By: Claude Sonnet 4.6 <[email protected]>
Runs cargo fmt --all; all 573 tests still passing, clippy still clean.
No logic changes — formatting only.
Co-Authored-By: Claude Sonnet 4.6 <[email protected]>
Clippy fixes:
- clawhdf5-onion/reader.rs: introduce RawPage type alias to resolve
type_complexity lint on revision_pages_raw return type
- clawsync-onion/iblt.rs: remove stray #[forbid(unsafe_code)] module
attribute (crate-level forbid already in lib.rs; the module attr was
useless and triggered the empty-line-after-attribute lint)
- clawsync-onion/differ.rs: replace ok_or_else(|| …) with ok_or(…)
for non-lazily-constructed error values
- clawsync-cli/main.rs: accept &Path instead of &PathBuf in
handle_hdf5_client to avoid unnecessary indirection
CI (.gitea/workflows/ci.yml):
- test job: cargo test --workspace (default + simd-cdc)
- lint job: cargo clippy -D warnings + cargo fmt --check
- msrv job: build + test on Rust 1.85 (workspace rust-version)
- publish-dry-run job: cargo publish --dry-run for clawsync-core
All 573 tests still passing after fixes.
Co-Authored-By: Claude Sonnet 4.6 <[email protected]>
Omar Sobh