Repository navigation
Torrent v2 1/8: Contracts, verification infrastructure and resource limits - #231
Merged
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
10 of 38 tasks
Contributor
Test Results 481 files +145 481 suites +145 3m 54s ⏱️ +55s Results for commit 2fdddd3. ± Comparison against base commit 06e8e2c. This pull request removes 4 and adds 74 tests. Note that renamed tests count towards both. |
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.
Establishes capability/revision contracts and conformance gates, then bounds retained metadata, session state and payload file operations. Existing v1 behavior remains covered; this does not enable the public v2 runtime.
Part 1/8 of the consolidated stack for #162. Review and merge bottom-up. This PR targets
main; its diff contains only this capability group.Preservation and validation
Consolidates #163–#169. The consolidation snapshot matches original boundary
f1a85687exactly. 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.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
24702a3cb1eadef504bbde68f957d6251c7f7b257d271036f1a85687Review carry-forward
These original discussions and their responses remain linked for review. Their unresolved status is preserved; consolidation does not imply reviewer approval.
KotlinTorrentSession.kt; latest response.Consolidated stack
The listener regression failed before the fix and passed afterward on JVM and Android host. The foundation JVM suite also passed locally.