Skip to content

Add eager token construction option for faster connections#381

Draft
Monkatraz wants to merge 1 commit into
brn-add-rehandshakes-via-expiry-fnfrom
brn-eager-token-construction
Draft

Add eager token construction option for faster connections#381
Monkatraz wants to merge 1 commit into
brn-add-rehandshakes-via-expiry-fnfrom
brn-eager-token-construction

Conversation

@Monkatraz
Copy link
Copy Markdown
Contributor

Why

What changed

Versioning

  • Breaking protocol change
  • Breaking ts/js API change

Copy link
Copy Markdown
Contributor Author

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@Monkatraz Monkatraz mentioned this pull request Jun 1, 2026
2 tasks
Comment thread transport/client.ts
* awaited later even if that attempt is abandoned before the handshake is sent.
*/
type ConstructedHandshakeMetadata =
| { ok: true; metadata: unknown }
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we should know the metadata type here? also err should be narrowed probably

@Monkatraz Monkatraz force-pushed the brn-eager-token-construction branch from 7d583df to 4f74629 Compare June 1, 2026 21:45
@Monkatraz Monkatraz force-pushed the brn-add-rehandshakes-via-expiry-fn branch from 327c7fe to 16f814c Compare June 1, 2026 21:45
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.

2 participants