Skip to content

Fix redundant realtime events #1162

Merged
toubatbrian merged 3 commits intomainfrom
brian/fix-realtime-msg
Mar 23, 2026
Merged

Fix redundant realtime events #1162
toubatbrian merged 3 commits intomainfrom
brian/fix-realtime-msg

Conversation

@toubatbrian
Copy link
Copy Markdown
Contributor

Summary

  • Avoid sending response.cancel when there is no in-flight realtime response, which removes spurious cancellation errors during tool follow-up
  • Avoid sending no-op session.update events when toolChoice is unchanged.
  • Make duplicate conversation.item.added handling idempotent when the same user item was already inserted locally, preventing duplicate-ID failures

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Mar 21, 2026

🦋 Changeset detected

Latest commit: 9776512

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 22 packages
Name Type
@livekit/agents-plugin-openai Patch
@livekit/agents-plugin-anam Patch
@livekit/agents-plugin-cartesia Patch
@livekit/agents-plugin-deepgram Patch
@livekit/agents-plugin-elevenlabs Patch
@livekit/agents-plugin-google Patch
@livekit/agents-plugin-inworld Patch
@livekit/agents-plugin-neuphonic Patch
@livekit/agents-plugin-resemble Patch
@livekit/agents-plugin-rime Patch
@livekit/agents-plugin-sarvam Patch
@livekit/agents-plugin-xai Patch
@livekit/agents Patch
@livekit/agents-plugin-baseten Patch
@livekit/agents-plugin-bey Patch
@livekit/agents-plugin-hedra Patch
@livekit/agents-plugin-lemonslice Patch
@livekit/agents-plugin-livekit Patch
@livekit/agents-plugin-phonic Patch
@livekit/agents-plugin-silero Patch
@livekit/agents-plugin-trugen Patch
@livekit/agents-plugins-test Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

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: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 4 additional findings.

Open in Devin Review

Comment on lines +1260 to +1261
if (existingItem) {
if (serverEventType === 'conversation.item.added' && !pendingCreateFuture) {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

nit: combine this into the same if?

@toubatbrian toubatbrian merged commit 0c683fd into main Mar 23, 2026
8 of 9 checks passed
@toubatbrian toubatbrian deleted the brian/fix-realtime-msg branch March 23, 2026 21:02
@github-actions github-actions bot mentioned this pull request Mar 23, 2026
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.

2 participants