Skip to content

fix: honor short voice splitter chunks#3426

Closed
he-yufeng wants to merge 1 commit into
openai:mainfrom
he-yufeng:fix/voice-short-splitter-chunks
Closed

fix: honor short voice splitter chunks#3426
he-yufeng wants to merge 1 commit into
openai:mainfrom
he-yufeng:fix/voice-short-splitter-chunks

Conversation

@he-yufeng
Copy link
Copy Markdown

Summary

  • send non-empty custom splitter chunks to TTS even when they are shorter than 20 characters
  • preserve turn_ended when a custom splitter consumes the whole buffer before _turn_done()
  • add a regression test for short custom chunks

Fixes #3363.

To verify

  • uv run pytest tests\voice\test_pipeline.py -q --basetemp .tmp\pytest-voice
  • uv run pyright src\agents\voice\result.py tests\voice\test_pipeline.py
  • uv run ruff format --check src\agents\voice\result.py tests\voice\test_pipeline.py
  • uv run ruff check src\agents\voice\result.py tests\voice\test_pipeline.py
  • uv run python -m py_compile src\agents\voice\result.py tests\voice\test_pipeline.py
  • git diff --check

@seratch seratch added the duplicate This issue or pull request already exists label May 16, 2026
@seratch
Copy link
Copy Markdown
Member

seratch commented May 16, 2026

Resolved by #3364

@seratch seratch closed this May 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

duplicate This issue or pull request already exists

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Voice text splitter drops short custom chunks

2 participants