fix(desktop): diagnose and recover Multiverse receiver initialization - #7364
fix(desktop): diagnose and recover Multiverse receiver initialization#7364loganj wants to merge 4 commits into
Conversation
Signed-off-by: Logan Johnson <loganj@squareup.com>
Signed-off-by: Logan Johnson <loganj@squareup.com>
|
Updated receiver repair at Independent source review found the original new warning used a one-shot readiness callback: production cleared it on EOSE, so subsequent CLOSED was invisible. Fixed at the subscription owner with opt-in persistent state and explicit receiver retirement. Default subscribers retain shared retry/reconnect. New regression tests drive real RelayClient inbound EOSE/CLOSED/reset, queued lifecycle work fencing and deliberate fresh retry, rather than inventing a persistent fake readiness callback. Validation: full 6,279 frontend tests, typecheck/check and production build passed on the exact patch; mounted desktop-stop smoke passed after local integration. Committed diff matches the tested patch byte-for-byte. Prior Still not approved: the separate runtime-authority collision finding in parent #7363 is being repaired. The original native initialization exception remains unproven; instrumented native onboarding continuation is active. Destination-local keyless provisioning remains unavailable. No merge/deployment. |
|
Native diagnostic update (2026-09-05): the isolated full-feature instrumented 110374b demo reached legitimate GUI onboarding and captured a real initialization error at subscribeLive: "Failed to connect to relay." The general relay connection subsequently authenticated/recovered, while that original receiver remained unavailable. This establishes the failure before history projection/admission, not the underlying non-Error plugin rejection (which was discarded), and not that the lost original A/B exception was identical. Current 7fdaa65 already offers deliberate Retry for initialization failures; native acceptance of that repair remains pending. C also stalled before receiver mount on a later relaunch; no healthy-restart claim. All private fixture processes/services were stopped; installed Desktop/production state and frozen A/B artifacts were untouched. No successful native remote Stop/retry/no-resurrection claim is made by the 6,279-test frontend pass. |
|
Follow-up review of exact The availability finding is also actionable: retryable subscription CLOSED on an otherwise healthy socket has no connection-reconnect notification, and the native C diagnosis separately proved failed first connect followed by first success can leave the original one-shot initializer stranded. A bounded receiver-owned restart is being implemented. Each attempt retires/fences the old receiver, creates a fresh live-only subscription, and performs placement projection before admission; it must never replay a command or revive an old subscription. Terminal errors/budget exhaustion retain deliberate Retry. Rate-limit backoff and cancellation are explicit acceptance boundaries. The review's CI on this exact head remains running with no failure at 03:23 UTC. No successful native remote Stop/recovery or destination-host keyless provisioning is claimed. The composed repair remains a draft, not approved/ready to merge. |
Signed-off-by: Logan Johnson <loganj@squareup.com>
|
Published receiver-owned recovery at
Independent delta review is running; current-head CI has been refreshed. The separate authority-receipt repair is still under correction and not published. Real destination-host keyless provisioning remains unavailable. No merge/deployment/readiness claim. |
Signed-off-by: Logan Johnson <loganj@squareup.com>
|
Receiver review follow-through: published |
|
Composed runtime-authority follow-up is drafted in #7365 at |
|
Current-head CI completed successfully at https://github.com/block/buzz/actions/runs/33943460385 This closes the fresh CI gate, not native acceptance or technical approval. The repaired native Stop/retry/recovery run is currently inaccessible because its isolated remote host lost its control connection; no pass/fail verdict is inferred. The original native failure remains recorded. Keyless session issuance remains unimplemented and requires the designated trusted host/interface. Nothing merged or deployed. |
Dependent repair atop #7363. Origin: Buzz channel
f45d3304-dcf0-44e8-a46d-bcd63b235fbc, thread16211fefcee85904f49802ce5e1709bf24b4895401a944f0585e70233c4e4d39.Behavior
limit: 0) and completes projection sync before command admission. Cancellation/generation fences retire late work. Explicit subscription retirement cannot reissue an orphan REQ after reconnect.Evidence and review
Current head:
8f2dd70f9633181c8603a37a837addecec19704e. Full local frontend suite: 6,291 passed; TypeScript and static checks passed. The preceding bounded-owner heada2380832passed four mounted Stop/chrome checks; the final delta changes only terminal-init classification/tests/docs, not geometry. Independent source review found the prior receiver defects repaired and one P3 terminal-init classification gap, now fixed with a production owner regression. No GitHub technical approval or current-head CI completion is claimed here; see current checks/comments.Native and launch limits
The completed parent
110374bdnative run failed remote Stop before admission. Subsequent isolated diagnostic C capturedsubscribeLive → Failed to connect to relay; general connection recovered while that original receiver remained unavailable. The plugin discarded the lower-level rejection, so neither its precise cause nor identity with the original A/B failure is established. C was cleaned up; its separate relaunch splash stall is not repaired here. Successful native remote Stop/result, stored-result retry and no-resurrection acceptance remain outstanding. Local Stop auto-start policy is unchanged. Actual destination-host keyless provisioning remains unavailable; no successful new Start/Restart/Move is claimed. Nothing merged or deployed.