Skip to content

Queue overlapping command syncs (#622) - #645

Merged
iamfatness merged 1 commit into
mainfrom
codex/622-sync-scheduler
Sep 26, 2026
Merged

iamfatness merged 1 commit into
mainfrom
codex/622-sync-scheduler

Conversation

@iamfatness

Copy link
Copy Markdown
Owner

Fixes #622. MediaCoreBridgeService now uses one scheduler for production commands and periodic empty polls. Commands queue with cancellation and keep a copy of the submitted payload; redundant polls skip when busy or when a command is waiting. The supervisor's single-slot guard remains as a safety boundary. A regression constructs two overlapping commands and proves both send in order while an empty poll is coalesced. Local validation: 2,235 MediaCore tests passed in Release.

@iamfatness

Copy link
Copy Markdown
Owner Author

Live acceptance on combined local candidate 9e237e37: app joined the authorized test meeting, Engine On received a Zoom source on Program, and two overlapping Preview scene selects returned success with nativePreviewSceneId ending on the second requested scene (panel). Program frames continued advancing. Deterministic scheduler overlap/cancellation tests and all 2,235 MediaCore tests passed locally. CI TSan failure is the pre-existing intermittent #584 TakeRecord media-reopen assertion; rerun pending.

@iamfatness
iamfatness merged commit 1dadf2f into main Sep 26, 2026
34 of 35 checks passed
@iamfatness
iamfatness deleted the codex/622-sync-scheduler branch September 26, 2026 04:17
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.

Single-slot sync plus UI-thread snapshot apply can drop a fire-and-forget command

1 participant