also add pop thinking_config to handle_genai_tools

[oegedijk]

adds to #1751 to put the fix also in handle_genai_tools function (now was only in handle_genai_structured_outputs)

---

Important

Remove thinking_config from new_kwargs in handle_genai_tools to match handle_genai_structured_outputs.

• Behavior:
  • In handle_genai_tools, remove thinking_config from new_kwargs to match behavior in handle_genai_structured_outputs.

^{This description was created by for e72ac07. You can customize this summary. It will automatically update as commits are pushed.}

[ellipsis-dev]

Important

Looks good to me! 👍

Reviewed everything up to e72ac07 in 47 seconds. Click for details.

• Reviewed 12 lines of code in 1 files
• Skipped 0 files when reviewing.
• Skipped posting 1 draft comments. View those below.
• Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.

1. instructor/providers/gemini/utils.py:897

• Draft comment:
  Good fix: popping 'thinking_config' in handle_genai_tools now aligns with its removal in handle_genai_structured_outputs. This prevents unintended propagation.
• Reason this comment was not posted:
  Comment did not seem useful. Confidence is useful = 0% <= threshold 85% This comment is purely informative, acknowledging a good fix without providing any actionable feedback or suggestions. It doesn't align with the rules for useful comments.

Workflow ID: wflow_RGf3RWLVocxB9Qaw

^{You can customize by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.}