Required properties not being set on litellm tool call #2202
Description
Describe the bug
I've been testing some MCP tools that have required attributes, as well as some internal tools that I'm configuring and I've noticed that the required attribute is not being passed when using litellm.
It looks like the source of the problem is at _function_declaration_to_tool_param inside lite_llm.py.
My question is: Was it on purpose? If not, I'm happy to send a PR fixing this. This is causing several issues for us.
To Reproduce
Steps to reproduce the behavior:
- Configure an agent using litellm
- Setup a tool with required properties
- Inspect the call - the required field won't be there.
Expected behavior
It should pass the required field when doing the API calls.