Skip to content

Conversation

@jirispilka
Copy link
Contributor

@jirispilka jirispilka commented Aug 19, 2025

It was supposed to be a quick fix (and it actually was), but the integration and testing ended up taking longer than expected. 🤷🏻‍♂️

close: https://github.com/apify/ai-team/issues/62

FYI @MichalKalita

@jirispilka jirispilka requested a review from Copilot August 19, 2025 07:46
@jirispilka jirispilka self-assigned this Aug 19, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds support for remote Chroma database connections, migrating from local/self-hosted setups to cloud and enterprise deployments. The changes modernize the Chroma integration to work with newer versions and authentication methods.

  • Updated Chroma client configuration to support remote instances with API tokens, tenants, and databases
  • Replaced legacy authentication mechanism with modern API token-based authentication
  • Updated dependencies to newer versions (ChromaDB 1.0.17, langchain-chroma 0.2.5, etc.)

Reviewed Changes

Copilot reviewed 13 out of 14 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
docker-compose.yaml Updates ChromaDB and OpenSearch images to specific newer versions
code/src/vector_stores/chroma.py Refactors client initialization to support remote connections with API tokens, tenants, and databases
code/src/models/chroma_input_model.py Updates input model to replace legacy auth fields with modern API token and tenant/database fields
code/tests/conftest.py Updates test configuration to use new authentication parameters
code/src/utils.py Updates import from langchain_community to langchain_apify
code/pyproject.toml Updates dependencies to newer versions
code/.env.example Updates environment variables for new authentication approach
actors/chroma/README.md Updates documentation to reflect cloud/enterprise usage patterns
actors/chroma/.actor/input_schema.json Updates input schema to match new authentication model
code/src/examples/*.py Updates example scripts to use new configuration approach

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.

@jirispilka jirispilka requested a review from MQ37 August 19, 2025 08:26
Copy link
Contributor

@MQ37 MQ37 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I did not test the integration, just scouted through the code. Just one change and question othewise LGTM 👍 Ready to merge after you address these.

@jirispilka
Copy link
Contributor Author

I did not test the integration, just scouted through the code. Just one change and question othewise LGTM 👍 Ready to merge after you address these.

Fixed, thanks!

@jirispilka jirispilka merged commit 8f00788 into master Aug 19, 2025
2 checks passed
@jirispilka jirispilka deleted the feat/chroma-remote branch August 19, 2025 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants