Skip to content

Consolidate stragglers: document cache crate, bump MAX_PEERS to 200 - #3

Merged
jspwrd merged 2 commits into
mainfrom
consolidate/stragglers
Apr 22, 2026
Merged

Consolidate stragglers: document cache crate, bump MAX_PEERS to 200#3
jspwrd merged 2 commits into
mainfrom
consolidate/stragglers

Conversation

@jspwrd

@jspwrd jspwrd commented Apr 21, 2026

Copy link
Copy Markdown
Owner

Summary

Brings in the only outstanding straggler: the previously-uncommitted work from the local build branch.

Changes

  • Document the oxidebt-cache crate in CLAUDE.md's crate tree
  • Refresh frontend description in CLAUDE.md ("Minimal" → current state)
  • Bump MAX_PEERS_PER_TORRENT from 100 → 200 in oxidebt-constants
  • Add resume.md to .gitignore
  • Minor formatting of reqwest feature list in src-tauri/Cargo.toml

Branches analyzed but not included (no unmerged work)

Test plan

  • cargo check --workspace passes in src-tauri
  • Sanity-check the peer cap bump doesn't regress memory/connection behavior under load

🤖 Generated with Claude Code

jasper and others added 2 commits April 21, 2026 16:49
Update CLAUDE.md to reflect oxidebt-cache layer and current frontend
state. Bump peer cap from 100 to 200 and ignore resume.md.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
The clippy::unnecessary_sort_by lint (stabilized in 1.95) flagged two
pre-existing call sites. Local Rust 1.93 doesn't emit this, but CI
runs a newer toolchain and rejects warnings.

- crates/oxidebt-dht/src/routing.rs: sort_by(|a, b| a.1.cmp(&b.1))
  -> sort_by_key(|a| a.1)
- src/engine/mod.rs: sort_by(|a, b| b.1.cmp(&a.1))
  -> sort_by_key(|a| Reverse(a.1))

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@jspwrd
jspwrd merged commit 1660b21 into main Apr 22, 2026
4 checks passed
@jspwrd
jspwrd deleted the consolidate/stragglers branch April 22, 2026 00:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant