Support for Request Body containing Reference : OpenAPI functions

[anujmehta]

Thank you for contributing to LangChain!

• PR title: "package: description"

  • Where "package" is whichever of langchain, community, core, experimental, etc. is being modified. Use "docs: ..." for purely docs changes, "templates: ..." for template changes, "infra: ..." for CI changes.
  • Example: "community: add foobar LLM"

• PR message: Delete this entire checklist and replace with

  • Description: In OpenAPI spec for a POST method if the request body contains a Reference to another object then the schema of the Reference object is not sent to LLM and it leads to LLM creating incorrect payload. Additionally for a complex payload the schema was not sent to LLM. The PR tries resolves such issues.
    -In below screenshot we have a complex payload and this PR tries to handle schema of "address" and "hobbies" attributes as shown below

• Issue: 19750
• Dependencies: None
• Twitter handle: https://twitter.com/anujmehta

.

• Lint and test: Run make format, make lint and make test from the root of the package(s) you've modified. See contribution guidelines for more: https://python.langchain.com/docs/contributing/

Additional guidelines:

• Make sure optional dependencies are imported within a function.
• Please do not add dependencies to pyproject.toml files (even optional ones) unless they are required for unit tests.
• Most PRs should not touch more than one package.
• Changes should be backwards compatible.
• If you are adding something to community, do not re-import it in langchain.

If no one reviews your PR within a few days, please @-mention one of baskaryan, efriis, eyurtsev, hwchase17.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment

Name	Status	Preview	Comments	Updated (UTC)
langchain	⬜️ Ignored (Inspect)	Visit Preview		Apr 30, 2024 9:09am

[anujmehta]

@baskaryan @efriis Can you please review this PR?

[baskaryan]

could we add a unit test for this change?

[anujmehta]

@baskaryan I had updated the PR with unit tests cases. Also I have tried to add more description here in PR, will add details in linked issue also. Request you to kindly review the PR.

[anujmehta]

@baskaryan @efriis Please review this PR

[anujmehta]

@baskaryan @dosubot I need small help with this PR. Looks like build was failing due to lint and test failure. I had pushed changes for lint but I am unable to identify solution for test failure which is failing due to module missing. Please let me know whom can I reach out for help?

Looks like my build is failing due to module not found "openapi-pydantic" in following test class https://github.com/anujmehta/langchain/blob/request-body-reference/libs/community/tests/unit_tests/utilities/test_openapi.py. I am unable to find solution for it as the dependency is already present at https://github.com/anujmehta/langchain/blob/request-body-reference/libs/community/pyproject.toml#L22 and is added to extended_testing https://github.com/anujmehta/langchain/blob/request-body-reference/libs/community/pyproject.toml#L244.

[anujmehta]

@baskaryan @efriis @hwchase17 Gentle reminder. Can you please help me with above query?

[anujmehta]

@baskaryan @dosubot I need small help with this PR. Looks like build was failing due to lint and test failure. I had pushed changes for lint but I am unable to identify solution for test failure which is failing due to module missing. Please let me know whom can I reach out for help?

Looks like my build is failing due to module not found "openapi-pydantic" in following test class https://github.com/anujmehta/langchain/blob/request-body-reference/libs/community/tests/unit_tests/utilities/test_openapi.py. I am unable to find solution for it as the dependency is already present at https://github.com/anujmehta/langchain/blob/request-body-reference/libs/community/pyproject.toml#L22 and is added to extended_testing https://github.com/anujmehta/langchain/blob/request-body-reference/libs/community/pyproject.toml#L244.

@olgavrou @agola11 @3coins @sjwhitmore I see your valuable contribution to Langchain. Can you please help me in resolving this minor issue so that I can merge the PR?