Conversation
Reapply the full remaining diff from #7014 on current main, retaining the shared-compute UX, smoke coverage, and mesh agent E2E updates. Mic Neale's foundation from #6189 is already present in main. Co-authored-by: Michael Neale <michael.neale@gmail.com> Signed-off-by: Alessandro Joabar <sandro@squareup.com>
Update two stale test constructors to match ConnectionState.auth_state, which now uses std::sync::Mutex. This separately fixes the upstream compile blocker discovered while validating the shared-compute PR. Signed-off-by: Alessandro Joabar <sandro@squareup.com>
🔐 Codex Security Review
|
Signed-off-by: Alessandro Joabar <sandro@squareup.com>
sandro-sq
marked this pull request as ready for review
September 16, 2026 22:39
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
1 participant
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.
Summary
Fresh draft of #7014 on current
main, preserving its full remaining 25-file diff.Relationship to prior work
main; this branch retains the current upstream implementation rather than replaying its old commit/merge history.366cfe7fa0c613cca272bd88b04f07003a10a298.Validation
just desktop-check desktop-typecheck).git diff --cached --checkand original-vs-fresh stable patch-ID comparison.just ciand pre-push Rust tests exposed two staleRwLocktest constructors in upstreamcrates/buzz-relay/src/handlers/event.rs.ConnectionState.auth_statenow usesstd::sync::Mutex; separately signed-off commit4b0f7abb2updates those two constructors, with the author's approval.just ci(exit 0), including formatting, Clippy, desktop/web checks and builds, Rust/Tauri/desktop/mobile tests, security-review checks, and file-size checks.Why a fresh run
The old CI run's PostgreSQL rerun failed before testing because the
desktop-e2e-relayartifact could not be downloaded. Its security audit flaggedrustls 0.23.42(RUSTSEC-2026-0285); the current main branch already usesrustls 0.23.45. Neither failure required editing the shared-compute UI diff.mesh-demo-compute-v2.mov