Skip to content

fix(tools): raise default command TTS timeout to 300s - #50082

Closed
Nomadcxx wants to merge 1 commit into
NousResearch:mainfrom
Nomadcxx:bump-command-tts-timeout
Closed

Nomadcxx wants to merge 1 commit into
NousResearch:mainfrom
Nomadcxx:bump-command-tts-timeout

Conversation

@Nomadcxx

@Nomadcxx Nomadcxx commented Jun 21, 2026 •

Copy link
Copy Markdown

Mitigation for #50081.

Command providers that fall through multiple TTS engines sequentially can exceed the previous 120s ceiling before a working tier runs, causing the wrapper to kill the subprocess mid-synthesis and leave a truncated audio file.

This buys command-provider scripts more headroom while a proper idle/progress timeout is discussed.

Command providers that fall through multiple TTS engines sequentially
(e.g. paid API -> free API -> local model) can exceed the previous 120s
ceiling before a working tier runs, causing the subprocess to be killed
mid-synthesis and leaving a truncated audio file.

This is a mitigation for NousResearch#50081; a proper idle/progress timeout would be
the longer-term fix.
@alt-glitch alt-glitch added type/bug Something isn't working comp/tools Tool registry, model_tools, toolsets tool/tts Text-to-speech and transcription P2 Medium — degraded but workaround exists labels Jun 21, 2026
@Nomadcxx

Copy link
Copy Markdown
Author

Closing in favor of #50087, which implements the idle/progress timeout approach from #50081 rather than the timeout-bump mitigation. No need for two fixes for the same issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/tools Tool registry, model_tools, toolsets P2 Medium — degraded but workaround exists tool/tts Text-to-speech and transcription type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants