Skip to content

test(relay): verify encrypted peer admission and cleanup - #2493

Merged
kunickiaj merged 1 commit into
mainfrom
test/relay-encrypted-client
Nov 4, 2026
Merged

kunickiaj merged 1 commit into
mainfrom
test/relay-encrypted-client

Conversation

@kunickiaj

@kunickiaj kunickiaj commented Nov 3, 2026 •

Copy link
Copy Markdown
Owner

Description

Exercise the peer client against actual local WebSocket, Noise, coordinator, and quota components using disposable fixtures. Cover independent admission, registration renewal ownership, paced opens, offline drop, cancellation, interrupted writes, and route cleanup.

Regression tests cover forwarded DATA crossing local cancellation, late ACCEPTED, finite session history, idle recycling, reconnect, and rejection of genuinely unknown DATA.

A regression cancels one of two distinct-peer opens during blocked shared registration and verifies that the survivor completes Noise and HTTP on the same socket.

Type of Change

  • 🧪 Testing (test-only changes)

Testing

  • Relevant checks pass locally: root typechecks, Biome, and all 20 client tests.
  • Added/updated tests for changes

Checklist

  • Code follows project style
  • Self-review completed
  • No new warnings introduced

@kunickiaj kunickiaj changed the title feat(sync): complete encrypted relay sync and readiness test(relay): verify encrypted peer admission and cleanup Nov 3, 2026
@kunickiaj
kunickiaj marked this pull request as ready for review November 3, 2026 22:00
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Nov 3, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-11-04T00:08:05.229349Z 572e35b New commits
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@kunickiaj
kunickiaj force-pushed the test/relay-encrypted-client branch from 1932453 to ad4d4bc Compare November 3, 2026 22:25
@kunickiaj
kunickiaj force-pushed the feat/relay-encrypted-client branch from df9d529 to 5b0ef13 Compare November 3, 2026 22:25
@kunickiaj
kunickiaj force-pushed the test/relay-encrypted-client branch from ad4d4bc to a2cde87 Compare November 3, 2026 22:33
@kunickiaj
kunickiaj force-pushed the feat/relay-encrypted-client branch 2 times, most recently from 7263ff5 to 9fd54fd Compare November 3, 2026 22:45
@kunickiaj
kunickiaj force-pushed the test/relay-encrypted-client branch from a2cde87 to 6d83516 Compare November 3, 2026 22:45
@kunickiaj
kunickiaj force-pushed the feat/relay-encrypted-client branch from 9fd54fd to 30f0aba Compare November 3, 2026 22:51
@kunickiaj
kunickiaj force-pushed the test/relay-encrypted-client branch from 6d83516 to 22f78a7 Compare November 3, 2026 22:51
@kunickiaj
kunickiaj force-pushed the feat/relay-encrypted-client branch from 30f0aba to f28a7c7 Compare November 3, 2026 23:18
@kunickiaj
kunickiaj force-pushed the test/relay-encrypted-client branch from 22f78a7 to 532a331 Compare November 3, 2026 23:18
@kunickiaj
kunickiaj force-pushed the feat/relay-encrypted-client branch from f28a7c7 to f21b017 Compare November 3, 2026 23:39
@kunickiaj
kunickiaj force-pushed the test/relay-encrypted-client branch from 532a331 to cab588e Compare November 3, 2026 23:39
@kunickiaj
kunickiaj force-pushed the feat/relay-encrypted-client branch 2 times, most recently from e1883df to a33dc2d Compare November 3, 2026 23:42
@kunickiaj
kunickiaj force-pushed the test/relay-encrypted-client branch 2 times, most recently from 99f1175 to bb2f8f6 Compare November 3, 2026 23:57
@kunickiaj
kunickiaj force-pushed the feat/relay-encrypted-client branch from a33dc2d to a8699e9 Compare November 3, 2026 23:57
@kunickiaj
kunickiaj changed the base branch from feat/relay-encrypted-client to graphite-base/2493 November 4, 2026 00:06
@kunickiaj
kunickiaj force-pushed the test/relay-encrypted-client branch from bb2f8f6 to 5c76b58 Compare November 4, 2026 00:06
@graphite-app
graphite-app Bot changed the base branch from graphite-base/2493 to main November 4, 2026 00:07
@kunickiaj
kunickiaj force-pushed the test/relay-encrypted-client branch from 5c76b58 to 572e35b Compare November 4, 2026 00:07
@kunickiaj
kunickiaj merged commit 1517a4d into main Nov 4, 2026
18 checks passed

Copy link
Copy Markdown
Owner Author

Merge activity

@kunickiaj
kunickiaj deleted the test/relay-encrypted-client branch November 4, 2026 00:12
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