Skip to content

Conversation

jbrown9513
Copy link
Member

This PR is for fixing the following issue - #42

To verify the fix works correctly, you can test it by:

  • Comment out the fix in libs/oci/langchain_oci/chat_models/oci_generative_ai.py line 716:

  • Run the new unit tests:
    cd libs/oci poetry run pytest tests/unit_tests/chat_models/test_oci_generative_ai.py::test_ai_message_tool_calls_direct_field -v poetry run pytest tests/unit_tests/chat_models/test_oci_generative_ai.py::test_ai_message_tool_calls_additional_kwargs -v

  • Confirm failure
    =================================== FAILURES =================================== ___________________ test_ai_message_tool_calls_direct_field ____________________

  • Restore the fix by uncommenting the line:

  • Run the tests again to confirm they pass with the fix in place.

@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Oct 10, 2025
@YouNeedCryDear YouNeedCryDear self-requested a review October 14, 2025 23:29
@YouNeedCryDear
Copy link
Member

@jbrown9513 Thank you for the fix.

@YouNeedCryDear YouNeedCryDear merged commit 5ce1280 into oracle:main Oct 14, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

OCA Verified All contributors have signed the Oracle Contributor Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants