Skip to content

.Net: Remove migrated .NET Chroma connector - #14579

Merged
Adam Sitnik (adamsitnik) merged 4 commits into
mainfrom
copilot/remove-chroma-provider-code
Oct 13, 2026
Merged

Adam Sitnik (adamsitnik) merged 4 commits into
mainfrom
copilot/remove-chroma-provider-code

Conversation

Copilot AI commented Oct 12, 2026

Copy link
Copy Markdown
Contributor

Motivation and Context

The Chroma provider has moved to CommunityToolkit/AI. Remove the obsolete Semantic Kernel implementation and direct users to its maintained location.

Description

  • Provider cleanup: Remove the .NET Chroma implementation and its unit tests; remove their entries from MEVD.slnx.
  • Migration notice: Update the Chroma README with a link to CommunityToolkit/AI.

Contribution Checklist

Copilot AI and others added 2 commits October 12, 2026 08:31
Co-authored-by: adamsitnik <6011991+adamsitnik@users.noreply.github.com>
Co-authored-by: adamsitnik <6011991+adamsitnik@users.noreply.github.com>
Copilot AI deployed to github-app-auth October 12, 2026 08:39 Active
@adamsitnik
Adam Sitnik (adamsitnik) marked this pull request as ready for review October 12, 2026 08:39
@adamsitnik
Adam Sitnik (adamsitnik) requested a review from a team as a code owner October 12, 2026 08:39
Copilot AI balanced review requested due to automatic review settings October 12, 2026 08:39
Copilot AI deployed to integration October 12, 2026 08:39 Active
@semantic-kernel-automation semantic-kernel-automation Bot added .NET Issue or Pull requests regarding .NET code documentation labels Oct 12, 2026
@github-actions github-actions Bot changed the title Remove migrated .NET Chroma connector .Net: Remove migrated .NET Chroma connector Oct 12, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

The experimental-feature documentation remains stale and an orphaned test configuration file prevents complete cleanup.

1 open finding
What changed in this PR

Removes the migrated .NET Chroma connector while directing users to its CommunityToolkit/AI replacement.

Changes:

  • Deletes the Chroma implementation, HTTP models, project, and tests.
  • Removes Chroma projects from MEVD.slnx.
  • Replaces the README with a verified migration link.
File Description
dotnet/​test/​VectorData/​Chroma.UnitTests/​ChromaMemoryStoreTests.cs Removes connector tests.
dotnet/​test/​VectorData/​Chroma.UnitTests/​Chroma.UnitTests.csproj Removes the test project.
dotnet/​src/​VectorData/​Chroma/​README.md Adds the migration notice.
dotnet/​src/​VectorData/​Chroma/​IChromaClient.cs Removes the client interface.
dotnet/​src/​VectorData/​Chroma/​Http/​ApiSchema/​Internal/​UpsertEmbeddingsRequest.cs Removes the upsert request.
dotnet/​src/​VectorData/​Chroma/​Http/​ApiSchema/​Internal/​QueryEmbeddingsRequest.cs Removes the query request.
dotnet/​src/​VectorData/​Chroma/​Http/​ApiSchema/​Internal/​ListCollectionsRequest.cs Removes the list request.
dotnet/​src/​VectorData/​Chroma/​Http/​ApiSchema/​Internal/​GetEmbeddingsRequest.cs Removes the embeddings request.
dotnet/​src/​VectorData/​Chroma/​Http/​ApiSchema/​Internal/​GetCollectionRequest.cs Removes the collection request.
dotnet/​src/​VectorData/​Chroma/​Http/​ApiSchema/​Internal/​DeleteEmbeddingsRequest.cs Removes the delete-embeddings request.
dotnet/​src/​VectorData/​Chroma/​Http/​ApiSchema/​Internal/​DeleteCollectionRequest.cs Removes the delete-collection request.
dotnet/​src/​VectorData/​Chroma/​Http/​ApiSchema/​Internal/​CreateCollectionRequest.cs Removes the create request.
dotnet/​src/​VectorData/​Chroma/​Http/​ApiSchema/​ChromaQueryResultModel.cs Removes the query response model.
dotnet/​src/​VectorData/​Chroma/​Http/​ApiSchema/​ChromaEmbeddingsModel.cs Removes the embeddings model.
dotnet/​src/​VectorData/​Chroma/​Http/​ApiSchema/​ChromaCollectionModel.cs Removes the collection model.
dotnet/​src/​VectorData/​Chroma/​ChromaMemoryStore.cs Removes the memory-store implementation.
dotnet/​src/​VectorData/​Chroma/​ChromaMemoryBuilderExtensions.cs Removes registration extensions.
dotnet/​src/​VectorData/​Chroma/​ChromaClient.cs Removes the HTTP client.
dotnet/​src/​VectorData/​Chroma/​Chroma.csproj Removes the connector project.
dotnet/​src/​VectorData/​Chroma/​AssemblyInfo.cs Removes experimental metadata.
dotnet/​MEVD.slnx Removes Chroma project references.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread dotnet/src/VectorData/Chroma/README.md Outdated
Co-authored-by: adamsitnik <6011991+adamsitnik@users.noreply.github.com>

@adamsitnik Adam Sitnik (adamsitnik) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM!

@adamsitnik

Copy link
Copy Markdown
Member

westey (@westey-m) SergeyMenshykh We have received a brand new Chroma provider contribution in Community Toolkit AI repo, so this code can be removed now.

Context: CommunityToolkit/AI#58

Comment thread dotnet/src/VectorData/Chroma/README.md Outdated
Co-authored-by: westey <164392973+westey-m@users.noreply.github.com>
@adamsitnik
Adam Sitnik (adamsitnik) added this pull request to the merge queue Oct 13, 2026
Merged via the queue into main with commit 5b4a326 Oct 13, 2026
18 checks passed
@adamsitnik
Adam Sitnik (adamsitnik) deleted the copilot/remove-chroma-provider-code branch October 13, 2026 15:20
@Laurianti

Copy link
Copy Markdown
Contributor

Adam Sitnik (@adamsitnik) thanks for this. On NuGet, the Qdrant, Redis, PgVector, Weaviate and SqliteVec connectors are deprecated with their CommunityToolkit.VectorData package as the alternative. Could Microsoft.SemanticKernel.Connectors.Chroma get the same, with CommunityToolkit.VectorData.Chroma?

@adamsitnik

Copy link
Copy Markdown
Member

Adam Sitnik (Adam Sitnik (@adamsitnik)) thanks for this. On NuGet, the Qdrant, Redis, PgVector, Weaviate and SqliteVec connectors are deprecated with their CommunityToolkit.VectorData package as the alternative. Could Microsoft.SemanticKernel.Connectors.Chroma get the same, with CommunityToolkit.VectorData.Chroma?

Great point, I've asked westey (@westey-m) offline to take a look.

This branch was successfully deployed

2 active deployments
github-app-auth — 64c9921f Deployed Oct 13, 2026 by adamsitnik via add_label #29328
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation .NET Issue or Pull requests regarding .NET code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants