.NET: Fix CopySessionConfig to respect SessionConfig.Streaming

[Protocol-zero-0]

Description

CopySessionConfig() and CopyResumeSessionConfig() in GitHubCopilotAgent.cs hardcode Streaming = true, ignoring the value from SessionConfig.Streaming. This makes it impossible to disable streaming output.

Changes

• CopySessionConfig: Changed Streaming = true to Streaming = source.Streaming
• CopyResumeSessionConfig: Changed Streaming = true to Streaming = source?.Streaming ?? true (preserves default behavior when source is null)

Fixes #4732

Made with Cursor

[Copilot]

Pull request overview

Fixes a bug in the .NET GitHub Copilot agent adapter where session config copying forced streaming on, preventing callers from disabling streaming output.

Changes:

• Update CopySessionConfig() to copy Streaming from the source SessionConfig.
• Update CopyResumeSessionConfig() to copy Streaming when provided, defaulting to true when source is null.
• Update XML docs to reflect the new streaming behavior.

[Copilot]

This file appears to have been saved without the UTF-8 BOM (many repo C# files start with //, and dotnet/.editorconfig specifies charset = utf-8-bom). Please re-save/format this file to restore the BOM to match repo conventions and avoid formatting churn/tooling mismatches.

Suggested change

	// Copyright (c) Microsoft. All rights reserved.
	// Copyright (c) Microsoft. All rights reserved.

[Copilot]

The unit tests for CopySessionConfig currently only assert the default Streaming value; they don’t cover the bug fix scenario where source.Streaming is explicitly set to false. Please add/adjust tests to verify Streaming is preserved from the source (including the false case).

[Copilot]

The unit tests for CopyResumeSessionConfig currently assert Streaming is true but don’t validate that an explicitly disabled source.Streaming = false is carried over. Please add/adjust tests to cover both source.Streaming = false and the source is null => Streaming defaults to true behavior.

[Protocol-zero-0]

Updated. I restored the file BOM to match the repository encoding convention, and I expanded the unit coverage so both copy helpers now verify that an explicit Streaming = false value is preserved from the source. The existing null-source test still covers the source == null => Streaming defaults to true behavior.

[Protocol-zero-0]

@microsoft-github-policy-service agree