interrupt the same speech handle

[chenghao-mou]

This makes sure we interrupt the same speech handle, given there is a chance that self._current_speech might be changed during the await point. Potentially fix for #4443

Summary by CodeRabbit

• Refactor
  • Internal code optimization to improve performance and readability of voice agent activity handling. No user-facing changes.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

📝 Walkthrough

Walkthrough

A single file refactoring that introduces a local variable to replace repeated self._current_speech attribute lookups, improving code readability while preserving existing logic and behavior.

Changes

Cohort / File(s)	Summary
Refactoring: attribute lookup optimization livekit-agents/livekit/agents/voice/agent_activity.py	Replaces repeated self._current_speech checks with a local variable reference to reduce attribute access overhead and enhance control flow clarity. Conditional logic paths for interrupt eligibility and execution remain semantically equivalent.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

A rabbit hops through code with glee,
One variable set, lookups now three,
No longer repeating the same old quest,
Clear paths through logic, now at their best! 🐰✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'interrupt the same speech handle' directly reflects the main refactoring change: introducing a local variable to ensure consistent operation on the same speech handle rather than potentially changed references.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

---

---

📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between c639d8b and e252534.

📒 Files selected for processing (1)

• livekit-agents/livekit/agents/voice/agent_activity.py

🧰 Additional context used

📓 Path-based instructions (1)

**/*.py

📄 CodeRabbit inference engine (AGENTS.md)

**/*.py: Format code with ruff
Run ruff linter and auto-fix issues
Run mypy type checker in strict mode
Maintain line length of 100 characters maximum
Ensure Python 3.9+ compatibility
Use Google-style docstrings

Files:

• livekit-agents/livekit/agents/voice/agent_activity.py

🧬 Code graph analysis (1)

livekit-agents/livekit/agents/voice/agent_activity.py (3)

livekit-agents/livekit/agents/voice/agent_session.py (2)

• current_speech (410-411)
• interrupt (959-969)

livekit-agents/livekit/agents/voice/agent.py (1)

• allow_interruptions (573-583)

livekit-agents/livekit/agents/voice/speech_handle.py (3)

• allow_interruptions (87-88)
• allow_interruptions (91-109)
• interrupt (118-131)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (7)

• GitHub Check: livekit-plugins-cartesia
• GitHub Check: livekit-plugins-openai
• GitHub Check: type-check (3.9)
• GitHub Check: livekit-plugins-deepgram
• GitHub Check: livekit-plugins-inworld
• GitHub Check: type-check (3.13)
• GitHub Check: unit-tests

🔇 Additional comments (1)

livekit-agents/livekit/agents/voice/agent_activity.py (1)

1426-1436: LGTM — local handle avoids the await race.

Capturing self._current_speech before the await ensures the interrupt targets the intended handle.

_{✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.}

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}