Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@mrubens
Copy link
Collaborator

@mrubens mrubens commented Dec 2, 2025

Important

Assume all LiteLLM models support native tools by default in getLiteLLMModels and update tests accordingly.

  • Behavior:
    • getLiteLLMModels in litellm.ts now assumes all models support native tools by setting supportsNativeTools: true.
  • Tests:
    • Updated litellm.spec.ts to reflect the assumption that all models support native tools by setting supportsNativeTools: true in multiple test cases.

This description was created by Ellipsis for a94ca5d. You can customize this summary. It will automatically update as commits are pushed.

@mrubens mrubens requested review from cte and jr as code owners December 2, 2025 05:17
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Dec 2, 2025
@roomote
Copy link
Contributor

roomote bot commented Dec 2, 2025

Rooviewer Clock   See task on Roo Cloud

Review completed. No issues found.

The changes appropriately assume all LiteLLM models support native tools and update all tests accordingly. The implementation is consistent and doesn't introduce any provable bugs.

Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues.

@mrubens mrubens merged commit 6f0addf into main Dec 2, 2025
13 of 15 checks passed
@mrubens mrubens deleted the assume_all_litellm_models_support_native_tools branch December 2, 2025 05:22
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Dec 2, 2025
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Dec 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M This PR changes 30-99 lines, ignoring generated files.

Projects

No open projects
Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants