Skip to content

Update livekit-ffi to 0.12.57#668

Open
xianshijing-lk wants to merge 3 commits into
mainfrom
sxian/CLT-2919/update-livekit-ffi-0.12.57
Open

Update livekit-ffi to 0.12.57#668
xianshijing-lk wants to merge 3 commits into
mainfrom
sxian/CLT-2919/update-livekit-ffi-0.12.57

Conversation

@xianshijing-lk
Copy link
Copy Markdown
Contributor

  • Update rust-sdks submodule to latest main (497527d4)
  • Regenerate protobuf files
  • Includes: Fix LocalTrackPublished handle leak, WebRTC ADM integration, and other improvements

- Update rust-sdks submodule to latest main (497527d4)
- Regenerate protobuf files
- Includes: Fix LocalTrackPublished handle leak, WebRTC ADM integration, and other improvements

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Devin Review found 1 potential issue.

🐛 1 issue in files not directly in the diff

🐛 Protobuf field ready_for_room_event removed from FfiRequest but room.py still uses it, causing AttributeError on every connection (livekit-rtc/livekit/rtc/_proto/ffi_pb2.pyi:357-358)

The protobuf definitions were updated to replace ready_for_room_event (field 77) on FfiRequest with new_platform_audio, and ReadyForRoomEventRequest/ReadyForRoomEventResponse message types were removed from room_pb2. However, livekit-rtc/livekit/rtc/room.py:560 still does ready_req.ready_for_room_event.room_handle = self._ffi_handle.handle, which will now raise AttributeError: ready_for_room_event at runtime. This line executes during every room connection (in the connect() method), so all room connections will crash after the initial handshake succeeds. I confirmed this by running the protobuf module: FfiRequest no longer has the ready_for_room_event field and room_pb2 no longer has ReadyForRoomEventRequest.

View 2 additional findings in Devin Review.

Open in Devin Review

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.

3 participants