Skip to content

feat(eot): add eot/inference protos#637

Merged
cloudwebrtc merged 4 commits intomainfrom
chenghao/feat/add-eot-protos
Apr 27, 2026
Merged

feat(eot): add eot/inference protos#637
cloudwebrtc merged 4 commits intomainfrom
chenghao/feat/add-eot-protos

Conversation

@chenghao-mou
Copy link
Copy Markdown
Member

  • Updated protocol version/commit
  • Add inference protos in the generation script

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

🐛 New agent_inference module missing from agent_pb/__init__.py (livekit-protocol/livekit/protocol/agent_pb/__init__.py:1)

The agent_pb/__init__.py (livekit-protocol/livekit/protocol/agent_pb/__init__.py:1) imports agent_dev, agent_session, agent_text but does not include the newly added agent_inference module. All other modules in the agent_pb directory are listed in __init__.py, so this breaks the convention. As a result, from livekit.protocol import agent_pb will not auto-import agent_inference, and users doing agent_pb.agent_inference will get an AttributeError unless they explicitly import it.

View 1 additional finding in Devin Review.

Open in Devin Review

@cloudwebrtc cloudwebrtc requested review from xianshijing-lk and removed request for xianshijing-lk April 24, 2026 08:55
@chenghao-mou chenghao-mou requested a review from a team April 24, 2026 09:05
@cloudwebrtc cloudwebrtc merged commit aa23f11 into main Apr 27, 2026
14 checks passed
@cloudwebrtc cloudwebrtc deleted the chenghao/feat/add-eot-protos branch April 27, 2026 06:54
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