genai: Fix handling of optional arrays in tool input

[piotr-rudnik]

PR Description

This code fixes handling of optional arrays in pydantic models provided as tool inputs like this:

class ExampleToolInput(BaseModel):
        numbers: Optional[List[str]] = Field()

Optional parameters are handled by setting nullable field in gemini API. Current code works for every type, except for arrays, because items field, that contains information about type of values inside the array, is present in anyOf field as one of the possibilities, where it's ignored.

Also added log where user is warned that only only type can be used inside array. Before, last defined type was used.
This is an edge case where user could be trying to use an array with multiple types which is not supported by genai, but can be useful for finding bugs

Type

🐛 Bug Fix
✅ Test

Testing(optional)

Launching pytest with arguments test_function_utils.py::test_tool_input_can_have_optional_arrays --no-header --no-summary -q in /xxx/langchain-google/libs/genai/tests/unit_tests

============================= test session starts ==============================
collecting ... collected 1 item

test_function_utils.py::test_tool_input_can_have_optional_arrays 

========================= 1 passed, 1 warning in 0.61s =========================

[Shahar-Y]

Hi, great catch!
Notice we already solved this bug in #658 a few day ago as a fix for #657, hopefully will be merged into main soon 😄

[piotr-rudnik]

Hey @Shahar-Y, I saw your PR before and it's similar but not the same, tried your code and didn't work in my case. Here, arrays are handled and they have items field setup, not properties in v object (I'm not that invested in the codebase, maybe it's not a problem). But those cases are very similar, maybe could be merged somehow.

[lkuligin]

@piotr-rudnik could you rebase and fix linter errors, please?

[piotr-rudnik]

@lkuligin Hey, branch got rebased, looks like all checks are passing.

[piotr-rudnik]

There still were failing ruff checks, amended files to previous commit and force pushed new version

[lkuligin]

@piotr-rudnik linter is still failing

[krobert]

Hey, im getting the same error for ChatGroq too, i think this issue can be found in all basechatmodels