Skip to content

Conversation

@youssmox
Copy link

Summary

  • Added support for propagating the Groq-specific disable_tool_validation flag through livekit.plugins.openai.LLM, storing it in _LLMOptions.
  • Forward the flag in LLM.chat only when the client’s base URL points at Groq, raising a clear error otherwise to avoid misconfiguration.
  • This ensures the Groq plugin can opt out of tool validation without breaking other OpenAI-compatible providers.

Testing

  • Not run (feature is configuration-only; recommend running Groq integration test/smoke before merge).

@CLAassistant
Copy link

CLAassistant commented Nov 10, 2025

CLA assistant check
All committers have signed the CLA.

@chenghao-mou
Copy link
Contributor

chenghao-mou commented Nov 14, 2025

Thanks for creating the PR!

This flag should be an extra body parameter instead of a function parameter in extra/extra_kwargs. Otherwise, it will throw errors like:

TypeError: AsyncCompletions.create() got an unexpected keyword argument 'disable_tool_validation'

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