Skip to content

Remove realtime initial ack startup gate#4

Open
AdirAmsalem wants to merge 1 commit into
mainfrom
rene/realtime-startup-without-initial-ack
Open

Remove realtime initial ack startup gate#4
AdirAmsalem wants to merge 1 commit into
mainfrom
rene/realtime-startup-without-initial-ack

Conversation

@AdirAmsalem

@AdirAmsalem AdirAmsalem commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Bundle the initial realtime prompt/image (including the null-image bootstrap) into livekit_join.initial_state.
  • Stop blocking LiveKit room connect / track publishing on the initial prompt/image ack.
  • Report failed or timed-out initial-state acks through the existing async onError callback while keeping normal post-connect setPrompt/setImage calls blocking.

Context

Verification

  • ./scripts/format.sh
  • git diff --check
  • cmake -B build -S . -G Ninja -DCMAKE_BUILD_TYPE=Release -DDECART_BUILD_EXAMPLES=ON -DDECART_BUILD_TESTS=ON && cmake --build build && ctest --test-dir build --output-on-failure -C Release
  • codex exec --dangerously-bypass-approvals-and-sandbox "Review the uncommitted diff..."
  • GitHub CI: ubuntu, macOS, Windows, and clang-format all pass.

@AdirAmsalem AdirAmsalem force-pushed the rene/realtime-startup-without-initial-ack branch 2 times, most recently from 2625f03 to e5d8cf4 Compare June 18, 2026 20:43
@AdirAmsalem AdirAmsalem force-pushed the rene/realtime-startup-without-initial-ack branch from e5d8cf4 to ef1f692 Compare June 18, 2026 20:57
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.

1 participant