Skip to content

[arbiter-authentication/3.1.1] Shared bounded carrier and streaming peer - #84

Draft
omry wants to merge 1 commit into
pr83from
pr84
Draft

omry wants to merge 1 commit into
pr83from
pr84

Conversation

@omry

@omry omry commented Nov 3, 2026 •

Copy link
Copy Markdown
Owner

Move pairing and control TLS I/O onto shared bounded carrier credits and a serialized memory-BIO pump. Add a private bidirectional Python/Go streaming peer to exercise fragmentation, coalescing, backpressure, partial writes, and actual-worker ownership during cancellation.

Validation: 1,665 unit tests passed (43 skipped), 61 real native-peer tests passed, Go test/vet passed, Black checked 143 files, and pyrefly reported zero errors. The optional full Python suite retains one Docker-access failure.


Move pairing and control TLS I/O onto shared bounded carrier credits and a serialized memory-BIO pump. Add a private bidirectional Python/Go streaming peer to exercise fragmentation, coalescing, backpressure, partial writes, and actual-worker ownership during cancellation.

Validation: 1,665 unit tests passed (43 skipped), 61 real native-peer tests passed, Go test/vet passed, Black checked 143 files, and pyrefly reported zero errors. The optional full Python suite retains one Docker-access failure.
@gitar-bot

gitar-bot Bot commented Nov 5, 2026

Copy link
Copy Markdown
Code Review ✅ Approved

🟡 Medium risk · Refactoring control TLS into a shared streaming pump changes carrier buffering and I/O ownership, which could cause stalled or prematurely rejected exchanges.

Moves pairing and control TLS I/O onto shared bounded carrier credits and a serialized memory-BIO pump, and adds a private bidirectional Python/Go streaming peer to exercise fragmentation, coalescing, backpressure, partial writes, and worker ownership during cancellation. No issues found.

Verification

Rules No rules evaluated

Functional validation Not enabled

Set up

For more verification details, comment gitar display:verbose.

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing compact verification results.

Comment with these commands to change the behavior for this request:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Powered by Gitar — free for open source

This branch has not been deployed

No deployments
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