Repository navigation
Ignore agent worktree directories - #3
Merged
Merged
Conversation
iamfatness
marked this pull request as ready for review
June 14, 2026 02:47
iamfatness
added a commit
that referenced
this pull request
Jun 19, 2026
…133) Redesign the engine/capture/compositor subsystem so the media-core / compositor is always running while in a meeting, and the top-right toggle now subscribes/unsubscribes the raw Zoom CAPTURE spine instead of starting/stopping the engine. Fixes Zoom status never updating, the dead toggle, surfaces stuck on "waiting for compositor output", and the no-op Take button. Status (#1): always apply meeting/ZoomStatus/roster fields from each snapshot before any early-return, so meeting state keeps updating even when capture is unsubscribed. Toggle (#2 + redesign #4): rename EngineRunning -> ZoomCaptureSubscribed throughout StudioViewModel and VideoSurfaceCoordinator. ToggleEngineAsync now restarts the media core if it died (rather than dead-ending) and subscribes/unsubscribes ConfigureZoomSpineSync. Capture can be toggled whenever in a meeting; recording rights only require being in a meeting (not capture), supporting breakout record-rights. Label stays "Capture On/Off". Compositor (#3): the C++ compositor already emits a synthetic black slate program frame on every tick (verified). Stop gating program / compositor frame acceptance in VideoSurfaceCoordinator on the old flag so Program/Preview always show live compositor output; per-participant Zoom frames remain gated on the capture subscription. Surfaces start on a "Compositor warming up" slate and show "Capture paused" (not "waiting for compositor output") when capture is off. Take (#7): add CanTake (PreviewSceneId != ActiveSceneId) as the command's CanExecute, add IsEnabled to the Take button in StudioWorkspace.xaml, refresh program composition, and sync to native whenever the core is running regardless of capture state. Native: add a regression test asserting an empty applyCommands tick with no Zoom input still emits a synthetic program frame and preview, and that the program frame number advances on each tick (always-on compositor). Claude-Session: https://claude.ai/code/session_01HoCa1n85KjP7TBDQgMdJ3g Co-authored-by: Claude <noreply@anthropic.com>
This was referenced Sep 18, 2026
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
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
.claude/worktrees/to.gitignoreto avoid tracking ephemeral agent worktree checkoutsTest plan
Generated by Claude Code