fixes #29193 | handles case for no documents

[prady00]

Thank you for contributing to LangChain!

• fixes DOC: <A long term memory agent 'DOC: ' prefix> #29193 | handles case for no documents: "package: docs"

• PR message: Delete this entire checklist and replace with

  • Description: handles case for no documents
  • Issue: DOC: <A long term memory agent 'DOC: ' prefix> #29193
  • Dependencies:
  • Twitter handle:prady00 if your PR gets announced, and you'd like a mention, we'll gladly shout you out!

• Add tests and docs: If you're adding a new integration, please include

  1. a test for the integration, preferably unit tests that do not rely on network access,
  2. an example notebook showing its use. It lives in docs/docs/integrations directory.

• 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, ccurme, vbarda, hwchase17.

[vercel]

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

Name	Status	Preview	Comments	Updated (UTC)
langchain	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 14, 2025 11:32am

[ccurme]

I'm confused by this. documents is of type list[Document].

If not documents, then we should have documents==[].

If documents==[], then the list comprehension will be [] as well. This conditional does not appear necessary.

Also confusing is that the error described in the issue is 429 / insufficient_quota.