Python: Introducing the Chroma Connector with the new vector store design

[eavanvalkenburg]

Motivation and Context

Adds ChromaStore and ChromaConnector

Closes: #9926

Description

Contribution Checklist

• The code builds clean without any errors or warnings
• The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
• All unit tests pass, and I have added new tests where possible
• I didn't break anyone 😄

[markwallace-microsoft]

Python Test Coverage Report •

File	Stmts	Miss	Cover	Missing
TOTAL	18865	2366	87%

report-only-changed-files is enabled. No files were changed during this commit :)

Python Unit Test Overview

Tests	Skipped	Failures	Errors	Time
3185	5 💤	0 ❌	0 🔥	1m 28s ⏱️

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (2)

python/semantic_kernel/connectors/memory/chroma/chroma.py:301

• [nitpick] The variable name 'filter' shadows Python’s built-in function. Consider renaming it (e.g., to 'filter_clause') to improve clarity.

for filter in options.filter.filters:

python/semantic_kernel/connectors/memory/chroma/chroma.py:205

• Using a try-except block to catch an IndexError for an empty 'ids' list may hide other issues. Consider explicitly checking if the 'ids' list is empty before accessing its first element.

if isinstance(results["ids"][0], str):

[Copilot]

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

python/semantic_kernel/connectors/memory/chroma/chroma.py:219

• The conversion logic in _unpack_results that wraps each value in a list may lead to nested lists if the results are already in list form. Consider refining the conversion check to handle both single and multiple result scenarios explicitly.

if isinstance(results["ids"][0], str):