Skip to content

Removing SemanticKernel Connectors dependency and replacing it - #7584

Merged
adamsitnik merged 1 commit into
mainfrom
copilot/remove-semantic-kernel-dependency
Jun 26, 2026
Merged

adamsitnik merged 1 commit into
mainfrom
copilot/remove-semantic-kernel-dependency

Conversation

Copilot AI commented Jun 25, 2026 •

Copy link
Copy Markdown
Contributor

Pull request created by AI Agent

Microsoft Reviewers: Open in CodeFlow

โ€ฆwith CommunityToolkit.VectorData equivalents
@dotnet-comment-bot

Copy link
Copy Markdown
Collaborator

โ€ผ๏ธ Found issues โ€ผ๏ธ

Project Coverage Type Expected Actual
Microsoft.Extensions.Diagnostics.Testing Line 99 98.65 ๐Ÿ”ป
Microsoft.Extensions.Telemetry Line 93 91.95 ๐Ÿ”ป
Microsoft.Extensions.AI Line 89 88.64 ๐Ÿ”ป
Microsoft.Extensions.AI Branch 89 88.57 ๐Ÿ”ป
Microsoft.Extensions.AI.OpenAI Line 75 62.89 ๐Ÿ”ป
Microsoft.Extensions.AI.OpenAI Branch 75 50.41 ๐Ÿ”ป
Microsoft.Extensions.DataIngestion.MarkItDown Line 75 4.46 ๐Ÿ”ป
Microsoft.Extensions.DataIngestion.MarkItDown Branch 75 0 ๐Ÿ”ป
Microsoft.Extensions.Diagnostics.ResourceMonitoring Line 99 96.03 ๐Ÿ”ป
Microsoft.Extensions.Diagnostics.ResourceMonitoring Branch 99 94.39 ๐Ÿ”ป
Microsoft.Extensions.Diagnostics.ResourceMonitoring.Kubernetes Line 99 97.73 ๐Ÿ”ป
Microsoft.Extensions.ServiceDiscovery.Dns Line 75 69.93 ๐Ÿ”ป
Microsoft.Extensions.ServiceDiscovery.Abstractions Line 75 42.11 ๐Ÿ”ป
Microsoft.Extensions.ServiceDiscovery.Abstractions Branch 75 42.86 ๐Ÿ”ป
Microsoft.Extensions.ServiceDiscovery Line 75 67.21 ๐Ÿ”ป
Microsoft.Extensions.ServiceDiscovery Branch 75 71.43 ๐Ÿ”ป
Microsoft.Extensions.ServiceDiscovery.Yarp Line 75 73.85 ๐Ÿ”ป
Microsoft.Extensions.ServiceDiscovery.Yarp Branch 75 70 ๐Ÿ”ป
Microsoft.Extensions.VectorData.Abstractions Line 75 37.39 ๐Ÿ”ป
Microsoft.Extensions.VectorData.Abstractions Branch 75 22.73 ๐Ÿ”ป

๐ŸŽ‰ Good job! The coverage increased ๐ŸŽ‰
Update MinCodeCoverage in the project files.

Project Expected Actual
Microsoft.Gen.BuildMetadata 97 100
Microsoft.Gen.MetadataExtractor 57 73
Microsoft.Gen.MetricsReports 67 69
Microsoft.Extensions.AI.Abstractions 82 85
Microsoft.Extensions.AI.Evaluation.NLP 0 78
Microsoft.Extensions.Caching.Hybrid 82 89
Microsoft.Extensions.DataIngestion 75 89
Microsoft.Extensions.DataIngestion.Markdig 75 90
Microsoft.Extensions.Http.Resilience 97 100

Full code coverage report: https://dev.azure.com/dnceng-public/public/_build/results?buildId=1481818&view=codecoverage-tab

@adamsitnik
adamsitnik marked this pull request as ready for review June 26, 2026 08:01
Copilot AI review requested due to automatic review settings June 26, 2026 08:01
@adamsitnik
adamsitnik requested review from a team as code owners June 26, 2026 08:01

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.

Pull request overview

Updates the AIChatWeb project template and its integration-test snapshots to remove the Microsoft.SemanticKernel.Connectors.* package dependencies and use the CommunityToolkit.VectorData.* equivalents instead, aligning the templateโ€™s vector-store dependencies with the CommunityToolkit VectorData ecosystem.

Changes:

  • Replaced Microsoft.SemanticKernel.Connectors.AzureAISearch with CommunityToolkit.VectorData.AzureAISearch in the AIChatWeb template and verified snapshots.
  • Replaced Microsoft.SemanticKernel.Connectors.Qdrant with CommunityToolkit.VectorData.Qdrant in the AIChatWeb template and verified snapshots.
  • Updated template package version pinning to include the new CommunityToolkit VectorData packages.

Reviewed changes

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

Show a summary per file
File Description
test/ProjectTemplates/Microsoft.Extensions.AI.Templates.IntegrationTests/Snapshots/aichatweb/aichatweb.oai_aais.verified/aichatweb/aichatweb.csproj Updates verified snapshot to reference CommunityToolkit.VectorData.AzureAISearch instead of SemanticKernel connector.
test/ProjectTemplates/Microsoft.Extensions.AI.Templates.IntegrationTests/Snapshots/aichatweb/aichatweb.o_q.verified/aichatweb/aichatweb.Web/aichatweb.Web.csproj Updates verified snapshot to reference CommunityToolkit.VectorData.Qdrant instead of SemanticKernel connector.
test/ProjectTemplates/Microsoft.Extensions.AI.Templates.IntegrationTests/Snapshots/aichatweb/aichatweb.A_aoai_aais.verified/aichatweb/aichatweb.Web/aichatweb.Web.csproj Updates verified snapshot to reference CommunityToolkit.VectorData.AzureAISearch instead of SemanticKernel connector.
src/ProjectTemplates/Microsoft.Extensions.AI.Templates/templates/AIChatWeb-CSharp/AIChatWeb-CSharp.Web/AIChatWeb-CSharp.Web.csproj-in Updates the actual template project file to use CommunityToolkit VectorData packages for Azure AI Search and Qdrant configurations.
eng/packages/ProjectTemplates.props Replaces SemanticKernel connector version entries with version pins for the CommunityToolkit VectorData Azure AI Search and Qdrant packages.

@dotnet-comment-bot

Copy link
Copy Markdown
Collaborator

โ€ผ๏ธ Found issues โ€ผ๏ธ

Project Coverage Type Expected Actual
Microsoft.Extensions.Diagnostics.Testing Line 99 98.65 ๐Ÿ”ป
Microsoft.Extensions.Telemetry Line 93 91.95 ๐Ÿ”ป
Microsoft.Extensions.AI Line 89 88.56 ๐Ÿ”ป
Microsoft.Extensions.AI Branch 89 88.57 ๐Ÿ”ป
Microsoft.Extensions.AI.OpenAI Line 75 62.89 ๐Ÿ”ป
Microsoft.Extensions.AI.OpenAI Branch 75 50.41 ๐Ÿ”ป
Microsoft.Extensions.DataIngestion.MarkItDown Line 75 4.46 ๐Ÿ”ป
Microsoft.Extensions.DataIngestion.MarkItDown Branch 75 0 ๐Ÿ”ป
Microsoft.Extensions.Diagnostics.ResourceMonitoring Line 99 96.03 ๐Ÿ”ป
Microsoft.Extensions.Diagnostics.ResourceMonitoring Branch 99 94.39 ๐Ÿ”ป
Microsoft.Extensions.Diagnostics.ResourceMonitoring.Kubernetes Line 99 97.73 ๐Ÿ”ป
Microsoft.Extensions.ServiceDiscovery.Dns Line 75 71.61 ๐Ÿ”ป
Microsoft.Extensions.ServiceDiscovery.Abstractions Line 75 42.11 ๐Ÿ”ป
Microsoft.Extensions.ServiceDiscovery.Abstractions Branch 75 42.86 ๐Ÿ”ป
Microsoft.Extensions.ServiceDiscovery Line 75 67.81 ๐Ÿ”ป
Microsoft.Extensions.ServiceDiscovery Branch 75 71.43 ๐Ÿ”ป
Microsoft.Extensions.ServiceDiscovery.Yarp Line 75 73.85 ๐Ÿ”ป
Microsoft.Extensions.ServiceDiscovery.Yarp Branch 75 70 ๐Ÿ”ป
Microsoft.Extensions.VectorData.Abstractions Line 75 37.39 ๐Ÿ”ป
Microsoft.Extensions.VectorData.Abstractions Branch 75 22.73 ๐Ÿ”ป

๐ŸŽ‰ Good job! The coverage increased ๐ŸŽ‰
Update MinCodeCoverage in the project files.

Project Expected Actual
Microsoft.Gen.BuildMetadata 97 100
Microsoft.Gen.MetadataExtractor 57 73
Microsoft.Gen.MetricsReports 67 69
Microsoft.Extensions.AI.Abstractions 82 85
Microsoft.Extensions.AI.Evaluation.NLP 0 78
Microsoft.Extensions.Caching.Hybrid 82 89
Microsoft.Extensions.DataIngestion 75 89
Microsoft.Extensions.DataIngestion.Markdig 75 90
Microsoft.Extensions.Http.Resilience 97 100

Full code coverage report: https://dev.azure.com/dnceng-public/public/_build/results?buildId=1482563&view=codecoverage-tab

@adamsitnik
adamsitnik enabled auto-merge (squash) June 26, 2026 11:41
@adamsitnik
adamsitnik requested a review from roji June 26, 2026 11:42
@adamsitnik
adamsitnik merged commit d825ca2 into main Jun 26, 2026
10 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 27, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants