Torrent v2 8/8: V2 engine lifecycle, rate controls and restart recovery - #238
Merged
linroid merged 1 commit intoSep 20, 2026
Merged
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
This was referenced Sep 20, 2026
linroid
force-pushed
the
torrent-v2-stack-08-runtime
branch
from
September 20, 2026 07:50
6c77951 to
74bd4bb
Compare
10 of 38 tasks
Contributor
Test Results 782 files 782 suites 4m 51s ⏱️ Results for commit 74bd4bb. |
This was referenced Sep 20, 2026
linroid
added this pull request to stack #239
September 20, 2026 08:46
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Owns v2 sessions under shared engine registration, admission and joined shutdown. Includes previously unpublished global/task request rate controls and ownership-validated checkpoint restoration; restored hints never bypass payload rechecking. Incoming v2 routes, automatic checkpoint saving, bounded checkpoint decoding, public source integration and seeding remain incomplete.
Part 8/8 of the consolidated stack for #162. Review and merge bottom-up. This PR targets
torrent-v2-stack-07-trackers; its diff contains only this capability group.Preservation and validation
Consolidates #227–#228. The consolidation snapshot matches original boundary
b55c4816exactly. Fresh CI then exposed a listener shutdown race: canceledaccept()could escape as a socket exception. Foundation commit2fdddd39fixes cancellation classification and adds a deterministic regression test. This inherited fix and its test are the only file differences from that original boundary; all other contents are preserved. Original branches and discussions are retained.Includes local commits
86f59382andb55c4816beyond #228. That exact source tree passed 619 JVM tests, 599 executed iOS simulator tests and the native-runtime guard. Independent-client conformance was not rerun locally for this recovery addition.Fresh CI runs on this replacement PR; historical checks and approvals are not transferred. The remaining production roadmap and release gates are still open.
Original PRs and frozen heads
d72fce518b6b1c1fReview carry-forward
These original discussions and their responses remain linked for review. Their unresolved status is preserved; consolidation does not imply reviewer approval.
TorrentV2DownloadSession.kt; latest response.KotlinTorrentEngine.kt; latest response.Consolidated stack
Final consolidated head
74bd4bb0, including the listener shutdown fix, passed 620 JVM tests, 600 executed iOS simulator tests andverifyNoNativeTorrentRuntimelocally. Fresh GitHub CI covers the remaining configured platforms and independent-client conformance.