Skip to content

Conversation

@rogerbarreto
Copy link
Member

This pull request updates the Azure Foundry Agents samples and documentation to clarify the differences between the classic and new Foundry Agents experiences, and to improve code clarity and consistency in the agent creation samples. The changes also update variable names for better readability and remove unnecessary constants.

Documentation improvements:

  • Added sections in the README.md files for both classic and new Foundry Agents samples, including a comparison and a link to the official migration guide. [1] [2] [3]
  • Updated the main Foundry Agents README.md to highlight the new experience and direct users to the appropriate samples and migration documentation.

Sample code consistency and clarity:

  • In both Agent_With_AzureAIProject/Program.cs and FoundryAgents_Step01.1_Basics/Program.cs, removed unnecessary instruction string constants, updated variable names (e.g., jokerAgentV1existingJokerAgent, jokerAgentV2newJokerAgent, latestVersionlatestAgentVersion), and inlined instruction strings for improved clarity. [1] [2]
  • Updated agent deletion code to use the new variable names for consistency. [1] [2]

@rogerbarreto rogerbarreto self-assigned this Nov 25, 2025
Copilot AI review requested due to automatic review settings November 25, 2025 17:07
Copilot finished reviewing on behalf of rogerbarreto November 25, 2025 17:09
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 clarifies the distinction between classic and new Azure Foundry Agents experiences by updating documentation and improving code clarity in sample files. The changes help developers understand which agent type they're working with and provide guidance for migration.

Key changes:

  • Added documentation sections explaining classic vs new Foundry Agents with migration guide links
  • Improved variable naming (e.g., jokerAgentV1existingJokerAgent, latestVersionlatestAgentVersion)
  • Removed unnecessary instruction constants and inlined literal strings for improved clarity

Reviewed changes

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

Show a summary per file
File Description
dotnet/samples/GettingStarted/FoundryAgents/README.md Added section explaining classic vs new Foundry Agents with migration documentation link
dotnet/samples/GettingStarted/AgentProviders/Agent_With_AzureAIProject/README.md Added header and comparison section identifying this as a new Foundry Agents sample
dotnet/samples/GettingStarted/AgentProviders/Agent_With_AzureAIProject/Program.cs Updated variable names, removed constant, and inlined instruction strings for clarity
dotnet/samples/GettingStarted/AgentProviders/Agent_With_AzureAIAgentsPersistent/README.md Added header and comparison section identifying this as a classic Foundry Agents sample
dotnet/samples/GettingStarted/FoundryAgents/FoundryAgents_Step01.1_Basics/Program.cs Updated variable names, removed constants, and inlined instruction strings for clarity

@rogerbarreto rogerbarreto added this pull request to the merge queue Nov 26, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 26, 2025
@rogerbarreto rogerbarreto added this pull request to the merge queue Nov 26, 2025
Merged via the queue into microsoft:main with commit 0723f1d Nov 26, 2025
15 checks passed
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