Skip to content

Simplify seeder upload tracking#11

Open
diegomrsantos wants to merge 2 commits intocommonwarexyz:mainfrom
diegomrsantos:codex/seeder-upload-tracking
Open

Simplify seeder upload tracking#11
diegomrsantos wants to merge 2 commits intocommonwarexyz:mainfrom
diegomrsantos:codex/seeder-upload-tracking

Conversation

@diegomrsantos
Copy link
Copy Markdown

@diegomrsantos diegomrsantos commented Apr 17, 2026

Summary

  • replace the seeder upload bookkeeping with explicit scheduling and confirmation state
  • clear out-of-order completions once the missing upload arrives and the frontier advances
  • add a focused regression test for the out-of-order completion case

Root Cause

The previous seeder upload tracker modeled progress through an inclusive boundary plus a range map of completed uploads. That made the state hard to reason about, caused the seeder to restart from the last confirmed upload, and left stale completed views accumulating behind the confirmed frontier.

Validation

  • cargo test -p battleware-node upload_tracker_advances_past_out_of_order_completions_once_gap_closes
  • cargo test -p battleware-node test_good_links
  • cargo fmt --all -- --check

Closes #6.

@diegomrsantos diegomrsantos deleted the codex/seeder-upload-tracking branch April 17, 2026 13:58
@diegomrsantos diegomrsantos changed the title [codex] Simplify seeder upload tracking Simplify seeder upload tracking Apr 17, 2026
@diegomrsantos diegomrsantos restored the codex/seeder-upload-tracking branch April 17, 2026 14:02
@diegomrsantos diegomrsantos reopened this Apr 17, 2026
@diegomrsantos diegomrsantos marked this pull request as ready for review April 17, 2026 14:47
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.

[seeder] Prune Tracked Uploads

1 participant