Skip to content

.NET: Fixes for bug bash.#3927

Merged
westey-m merged 2 commits into
microsoft:mainfrom
westey-m:2026-02-14-bugbash-fixes
Feb 16, 2026
Merged

.NET: Fixes for bug bash.#3927
westey-m merged 2 commits into
microsoft:mainfrom
westey-m:2026-02-14-bugbash-fixes

Conversation

@westey-m

Copy link
Copy Markdown
Contributor

Motivation and Context

Description

  • A few small bug bash fixes.

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.

Copilot AI review requested due to automatic review settings February 13, 2026 17:36
@markwallace-microsoft markwallace-microsoft added documentation Usage: [Issues, PRs], Target: documentation in the code base and learn docs .NET Usage: [Issues, PRs], Target: .Net labels Feb 13, 2026
@github-actions github-actions Bot changed the title Fixes for bug bash. .NET: Fixes for bug bash. Feb 13, 2026
@westey-m westey-m enabled auto-merge February 13, 2026 17:37

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 a few .NET GettingStarted samples to address bug-bash feedback by clarifying expected runtime behavior and adjusting sample instructions/output.

Changes:

  • Clarified chat-reduction sample behavior by adding additional chat history inspections and updating the follow-up prompt.
  • Adjusted persisted-conversation sample to avoid temp-file roundtrip and instead display serialized session data, then resume the session.
  • Updated MCP Inspector README command to explicitly run the sample under net10.0.

Reviewed changes

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

File Description
dotnet/samples/GettingStarted/Agents/Agent_Step16_ChatReduction/Program.cs Adds explanatory comments and extra history checks to illustrate when reduction occurs.
dotnet/samples/GettingStarted/Agents/Agent_Step10_AsMcpTool/README.md Updates inspector command to include --framework net10.0 to match the project’s target.
dotnet/samples/GettingStarted/Agents/Agent_Step06_PersistedConversations/Program.cs Switches from temp-file persistence demo to console output + immediate session resume; adds CA1869 suppression.

@westey-m westey-m added this pull request to the merge queue Feb 16, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Feb 16, 2026
@westey-m westey-m added this pull request to the merge queue Feb 16, 2026
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Feb 16, 2026
@westey-m westey-m added this pull request to the merge queue Feb 16, 2026
Merged via the queue into microsoft:main with commit ed113f9 Feb 16, 2026
15 checks passed
@westey-m westey-m deleted the 2026-02-14-bugbash-fixes branch February 25, 2026 13:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Usage: [Issues, PRs], Target: documentation in the code base and learn docs .NET Usage: [Issues, PRs], Target: .Net

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants