Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 4, 2025

Motivation and Context

Relocates TextSearchProvider and TextSearchProviderOptions classes from Microsoft.Agents.AI.Data namespace to Microsoft.Agents.AI namespace for better API organization.

Description

  • Namespace change: Both classes now use namespace Microsoft.Agents.AI; instead of namespace Microsoft.Agents.AI.Data;
  • File relocation: Moved files from Data/ subdirectory to parent Microsoft.Agents.AI/ directory
  • Updated consumers: Removed using Microsoft.Agents.AI.Data; from:
    • AgentJsonUtilities.cs
    • TextSearchProviderTests.cs

Note: Sample files retain the using Microsoft.Agents.AI.Data; statement and will be updated separately.

Before:

using Microsoft.Agents.AI.Data;

var provider = new TextSearchProvider(...);

After:

using Microsoft.Agents.AI;

var provider = new TextSearchProvider(...);

Contribution Checklist

  • The code builds clean without any errors or warnings
  • The PR follows the Contribution Guidelines
  • All unit tests pass, and I have added new tests where possible
  • Is this a breaking change? If yes, add "[BREAKING]" prefix to the title of the PR.
Original prompt

Move TextSearchProvider and TextSearchProviderOptions classes to Microsoft.Agents.AI namespace


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Move TextSearchProvider and TextSearchProviderOptions to Microsoft.Agents.AI namespace Move TextSearchProvider and TextSearchProviderOptions to Microsoft.Agents.AI namespace Dec 4, 2025
Copilot AI requested a review from SergeyMenshykh December 4, 2025 18:14
@github-actions github-actions bot changed the title Move TextSearchProvider and TextSearchProviderOptions to Microsoft.Agents.AI namespace .NET: Move TextSearchProvider and TextSearchProviderOptions to Microsoft.Agents.AI namespace Dec 4, 2025
@SergeyMenshykh
Copy link
Member

@copilot Please revert the exclusion of the AgentWithTextSearchRag project from the solution. The sample will be updated later.

Copy link
Contributor Author

Copilot AI commented Dec 4, 2025

@copilot Please revert the exclusion of the AgentWithTextSearchRag project from the solution. The sample will be updated later.

Reverted the exclusion of AgentWithTextSearchRag from the solution in commit f2cf48b.

…om AgentWithRAG_Step01_BasicTextRAG

Co-authored-by: SergeyMenshykh <[email protected]>
@SergeyMenshykh SergeyMenshykh marked this pull request as ready for review December 4, 2025 20:43
Copilot AI review requested due to automatic review settings December 4, 2025 20:43
@SergeyMenshykh SergeyMenshykh moved this to In Review in Agent Framework Dec 4, 2025
Copy link
Contributor

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 relocates TextSearchProvider and TextSearchProviderOptions from the Microsoft.Agents.AI.Data namespace to the parent Microsoft.Agents.AI namespace to improve API organization.

  • Moved namespace declaration from Microsoft.Agents.AI.Data to Microsoft.Agents.AI for both classes
  • Removed obsolete using Microsoft.Agents.AI.Data; statements from all affected files

Reviewed changes

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

Show a summary per file
File Description
dotnet/src/Microsoft.Agents.AI/TextSearchProvider.cs Updated namespace declaration from Microsoft.Agents.AI.Data to Microsoft.Agents.AI
dotnet/src/Microsoft.Agents.AI/TextSearchProviderOptions.cs Updated namespace declaration from Microsoft.Agents.AI.Data to Microsoft.Agents.AI
dotnet/src/Microsoft.Agents.AI/AgentJsonUtilities.cs Removed now-unnecessary using Microsoft.Agents.AI.Data; statement
dotnet/tests/Microsoft.Agents.AI.UnitTests/Data/TextSearchProviderTests.cs Removed now-unnecessary using Microsoft.Agents.AI.Data; statement
dotnet/samples/GettingStarted/AgentWithRAG/AgentWithRAG_Step01_BasicTextRAG/Program.cs Removed now-unnecessary using Microsoft.Agents.AI.Data; statement
dotnet/samples/GettingStarted/AgentWithRAG/AgentWithRAG_Step02_CustomVectorStoreRAG/Program.cs Removed now-unnecessary using Microsoft.Agents.AI.Data; statement
dotnet/samples/GettingStarted/AgentWithRAG/AgentWithRAG_Step03_CustomRAGDataSource/Program.cs Removed now-unnecessary using Microsoft.Agents.AI.Data; statement

@rogerbarreto
Copy link
Member

Add [Breaking] to the title.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants