[code-simplifier] refactor: simplify duplicate catch blocks and attribute-loop consistency

[Amaury Levé (Evangelink)]

Code Simplification — 2026-06-29

This PR simplifies two small pieces of recently modified code to improve clarity and consistency, without any behavioral change.

Files Simplified

• src/Platform/Microsoft.Testing.Extensions.PackagedApp/PackagedAppTestHostHandle.cs — combine two identical catch blocks
• src/Adapter/MSTestAdapter.PlatformServices/SourceGeneration/SourceGeneratedReflectionOperations.cs — align loop pattern with adjacent methods

Improvements Made

1. PackagedAppTestHostHandle — eliminate duplicate catch bodies (from #9454)

The Dispose method had two catch blocks with identical handler bodies:

// Beforecatch(IOExceptionex){Debug.WriteLine($"Best-effort cleanup of deployment directory '{_deploymentDirectory}' failed: {ex}");}catch(UnauthorizedAccessExceptionex){Debug.WriteLine($"Best-effort cleanup of deployment directory '{_deploymentDirectory}' failed: {ex}");}

Combined into a single handler using a C# exception filter, eliminating the duplication:

// Aftercatch(Exceptionex)when(exisIOException or UnauthorizedAccessException){Debug.WriteLine($"Best-effort cleanup of deployment directory '{_deploymentDirectory}' failed: {ex}");}

2. SourceGeneratedReflectionOperations.GetSingleAttributeOrDefault — loop consistency (from #9479)

The GetSingleAttributeOrDefault method used a is not T / continue guard, while the two adjacent methods (IsAttributeDefined and GetFirstAttributeOrDefault) that were introduced in the same PR use a direct positive is T match. Aligned the style:

// Before (uses negated guard + continue)if(attris not TAttributematch){continue;}if(firstis not null){throw ...;}first=match;// After (positive match, consistent with adjacent methods)if(attrisTAttributematch){if(firstis not null){throw ...;}first=match;}

Changes Based On

Recent changes from:

• Add generic ITestHostLauncher extension point + PackagedApp reference extension #9454 — Add generic ITestHostLauncher extension point + PackagedApp reference extension
• [efficiency-improver] perf: eliminate LINQ iterator allocations in SourceGeneratedReflectionOperations attribute lookup #9479 — perf: eliminate LINQ iterator allocations in SourceGeneratedReflectionOperations attribute lookup

Testing

• ✅ No functional changes — behavior is identical
• ✅ No tests need updating; the changes are purely structural

🤖 Automated content by GitHub Copilot. Posted via a maintainer's GitHub token, so it appears under their account — the account owner did not write or approve this content personally. Generated by the Code Simplifier workflow. · 217.9 AIC · ⌖ 18.3 AIC · ⊞ 9.3K · [◷]( · ◷)

Add this agentic workflows to your repo

To install this agentic workflow, run

gh aw add githubnext/agentics/workflows/code-simplifier.md@main

• expires on Jun 30, 2026, 2:46 PM UTC

[Copilot]

Pull request overview

This PR applies small, targeted refactors in Microsoft.Testing.Platform and MSTestAdapter.PlatformServices to reduce duplication and align local coding style, without changing observable behavior.

Changes:

• Consolidate two identical catch blocks in PackagedAppTestHostHandle.Dispose() via an exception filter.
• Adjust GetSingleAttributeOrDefault<TAttribute>’s loop to use a positive type match pattern consistent with adjacent attribute helpers.

Show a summary per file

File	Description
src/Platform/Microsoft.Testing.Extensions.PackagedApp/PackagedAppTestHostHandle.cs	Replaces duplicated IOException/UnauthorizedAccessException handlers with a single filtered catch block.
src/Adapter/MSTestAdapter.PlatformServices/SourceGeneration/SourceGeneratedReflectionOperations.cs	Refactors the attribute scan loop in GetSingleAttributeOrDefault for consistency with neighboring methods.

Review details

• Files reviewed: 2/2 changed files
• Comments generated: 0
• Review effort level: Low

[Amaury Levé (Evangelink)]

Note

🤖 Automated review by GitHub Copilot. Posted via a maintainer's GitHub token, so it appears under their account — the account owner did not write or approve this content personally. Generated by the Expert Code Review workflow. To request a follow-up action, reply by tagging @copilot directly.

✅ 22/22 dimensions clean — no findings.

Scope: 2 files, purely structural refactoring — no behavioral change.

• SourceGeneratedReflectionOperations.cs: early-continue → nested if (attr is TAttribute match) is semantically identical; logic (duplicate-check, then assign) is preserved in the same order.
• PackagedAppTestHostHandle.cs: catch (Exception ex) when (ex is IOException or UnauthorizedAccessException) captures exactly the same exception types as the two preceding separate catch blocks, with the same handler body.

No public API surface changed, no threading concerns, no localization regressions, no test-isolation issues. The or pattern in the when clause is C# 9, consistent with the net8.0+-only target of Microsoft.Testing.Extensions.PackagedApp.

[Amaury Levé (Evangelink)]

Code Simplification — 2026-06-29

This PR simplifies two small pieces of recently modified code to improve clarity and consistency, without any behavioral change.

Files Simplified

• src/Platform/Microsoft.Testing.Extensions.PackagedApp/PackagedAppTestHostHandle.cs — combine two identical catch blocks
• src/Adapter/MSTestAdapter.PlatformServices/SourceGeneration/SourceGeneratedReflectionOperations.cs — align loop pattern with adjacent methods

Improvements Made

1. PackagedAppTestHostHandle — eliminate duplicate catch bodies (from #9454)

The Dispose method had two catch blocks with identical handler bodies:

// Beforecatch(IOExceptionex){Debug.WriteLine($"Best-effort cleanup of deployment directory '{_deploymentDirectory}' failed: {ex}");}catch(UnauthorizedAccessExceptionex){Debug.WriteLine($"Best-effort cleanup of deployment directory '{_deploymentDirectory}' failed: {ex}");}

Combined into a single handler using a C# exception filter, eliminating the duplication:

// Aftercatch(Exceptionex)when(exisIOException or UnauthorizedAccessException){Debug.WriteLine($"Best-effort cleanup of deployment directory '{_deploymentDirectory}' failed: {ex}");}

2. SourceGeneratedReflectionOperations.GetSingleAttributeOrDefault — loop consistency (from #9479)

The GetSingleAttributeOrDefault method used a is not T / continue guard, while the two adjacent methods (IsAttributeDefined and GetFirstAttributeOrDefault) that were introduced in the same PR use a direct positive is T match. Aligned the style:

// Before (uses negated guard + continue)if(attris not TAttributematch){continue;}if(firstis not null){throw ...;}first=match;// After (positive match, consistent with adjacent methods)if(attrisTAttributematch){if(firstis not null){throw ...;}first=match;}

Changes Based On

Recent changes from:

• Add generic ITestHostLauncher extension point + PackagedApp reference extension #9454 — Add generic ITestHostLauncher extension point + PackagedApp reference extension
• [efficiency-improver] perf: eliminate LINQ iterator allocations in SourceGeneratedReflectionOperations attribute lookup #9479 — perf: eliminate LINQ iterator allocations in SourceGeneratedReflectionOperations attribute lookup

Testing

• ✅ No functional changes — behavior is identical
• ✅ No tests need updating; the changes are purely structural

🤖 Automated content by GitHub Copilot. Posted via a maintainer's GitHub token, so it appears under their account — the account owner did not write or approve this content personally. Generated by the Code Simplifier workflow. · 217.9 AIC · ⌖ 18.3 AIC · ⊞ 9.3K · [◷]( · ◷)

Add this agentic workflows to your repo

To install this agentic workflow, run

gh aw add githubnext/agentics/workflows/code-simplifier.md@main

• expires on Jun 30, 2026, 2:46 PM UTC

[Copilot]

Pull request overview

This PR applies small, targeted refactors in Microsoft.Testing.Platform and MSTestAdapter.PlatformServices to reduce duplication and align local coding style, without changing observable behavior.

Changes:

• Consolidate two identical catch blocks in PackagedAppTestHostHandle.Dispose() via an exception filter.
• Adjust GetSingleAttributeOrDefault<TAttribute>’s loop to use a positive type match pattern consistent with adjacent attribute helpers.

Show a summary per file

File	Description
src/Platform/Microsoft.Testing.Extensions.PackagedApp/PackagedAppTestHostHandle.cs	Replaces duplicated IOException/UnauthorizedAccessException handlers with a single filtered catch block.
src/Adapter/MSTestAdapter.PlatformServices/SourceGeneration/SourceGeneratedReflectionOperations.cs	Refactors the attribute scan loop in GetSingleAttributeOrDefault for consistency with neighboring methods.

Review details

• Files reviewed: 2/2 changed files
• Comments generated: 0
• Review effort level: Low

[Amaury Levé (Evangelink)]

Note

🤖 Automated review by GitHub Copilot. Posted via a maintainer's GitHub token, so it appears under their account — the account owner did not write or approve this content personally. Generated by the Expert Code Review workflow. To request a follow-up action, reply by tagging @copilot directly.

✅ 22/22 dimensions clean — no findings.

Scope: 2 files, purely structural refactoring — no behavioral change.

• SourceGeneratedReflectionOperations.cs: early-continue → nested if (attr is TAttribute match) is semantically identical; logic (duplicate-check, then assign) is preserved in the same order.
• PackagedAppTestHostHandle.cs: catch (Exception ex) when (ex is IOException or UnauthorizedAccessException) captures exactly the same exception types as the two preceding separate catch blocks, with the same handler body.

No public API surface changed, no threading concerns, no localization regressions, no test-isolation issues. The or pattern in the when clause is C# 9, consistent with the net8.0+-only target of Microsoft.Testing.Extensions.PackagedApp.

[Amaury Levé (Evangelink)]

Code Simplification — 2026-06-29

This PR simplifies two small pieces of recently modified code to improve clarity and consistency, without any behavioral change.

Files Simplified

• src/Platform/Microsoft.Testing.Extensions.PackagedApp/PackagedAppTestHostHandle.cs — combine two identical catch blocks
• src/Adapter/MSTestAdapter.PlatformServices/SourceGeneration/SourceGeneratedReflectionOperations.cs — align loop pattern with adjacent methods

Improvements Made

1. PackagedAppTestHostHandle — eliminate duplicate catch bodies (from #9454)

The Dispose method had two catch blocks with identical handler bodies:

// Beforecatch(IOExceptionex){Debug.WriteLine($"Best-effort cleanup of deployment directory '{_deploymentDirectory}' failed: {ex}");}catch(UnauthorizedAccessExceptionex){Debug.WriteLine($"Best-effort cleanup of deployment directory '{_deploymentDirectory}' failed: {ex}");}

Combined into a single handler using a C# exception filter, eliminating the duplication:

// Aftercatch(Exceptionex)when(exisIOException or UnauthorizedAccessException){Debug.WriteLine($"Best-effort cleanup of deployment directory '{_deploymentDirectory}' failed: {ex}");}

2. SourceGeneratedReflectionOperations.GetSingleAttributeOrDefault — loop consistency (from #9479)

The GetSingleAttributeOrDefault method used a is not T / continue guard, while the two adjacent methods (IsAttributeDefined and GetFirstAttributeOrDefault) that were introduced in the same PR use a direct positive is T match. Aligned the style:

// Before (uses negated guard + continue)if(attris not TAttributematch){continue;}if(firstis not null){throw ...;}first=match;// After (positive match, consistent with adjacent methods)if(attrisTAttributematch){if(firstis not null){throw ...;}first=match;}

Changes Based On

Recent changes from:

• Add generic ITestHostLauncher extension point + PackagedApp reference extension #9454 — Add generic ITestHostLauncher extension point + PackagedApp reference extension
• [efficiency-improver] perf: eliminate LINQ iterator allocations in SourceGeneratedReflectionOperations attribute lookup #9479 — perf: eliminate LINQ iterator allocations in SourceGeneratedReflectionOperations attribute lookup

Testing

• ✅ No functional changes — behavior is identical
• ✅ No tests need updating; the changes are purely structural

🤖 Automated content by GitHub Copilot. Posted via a maintainer's GitHub token, so it appears under their account — the account owner did not write or approve this content personally. Generated by the Code Simplifier workflow. · 217.9 AIC · ⌖ 18.3 AIC · ⊞ 9.3K · [◷]( · ◷)

Add this agentic workflows to your repo

To install this agentic workflow, run

gh aw add githubnext/agentics/workflows/code-simplifier.md@main

• expires on Jun 30, 2026, 2:46 PM UTC

[Copilot]

Pull request overview

This PR applies small, targeted refactors in Microsoft.Testing.Platform and MSTestAdapter.PlatformServices to reduce duplication and align local coding style, without changing observable behavior.

Changes:

• Consolidate two identical catch blocks in PackagedAppTestHostHandle.Dispose() via an exception filter.
• Adjust GetSingleAttributeOrDefault<TAttribute>’s loop to use a positive type match pattern consistent with adjacent attribute helpers.

Show a summary per file

File	Description
src/Platform/Microsoft.Testing.Extensions.PackagedApp/PackagedAppTestHostHandle.cs	Replaces duplicated IOException/UnauthorizedAccessException handlers with a single filtered catch block.
src/Adapter/MSTestAdapter.PlatformServices/SourceGeneration/SourceGeneratedReflectionOperations.cs	Refactors the attribute scan loop in GetSingleAttributeOrDefault for consistency with neighboring methods.

Review details

• Files reviewed: 2/2 changed files
• Comments generated: 0
• Review effort level: Low

[Amaury Levé (Evangelink)]

Note

🤖 Automated review by GitHub Copilot. Posted via a maintainer's GitHub token, so it appears under their account — the account owner did not write or approve this content personally. Generated by the Expert Code Review workflow. To request a follow-up action, reply by tagging @copilot directly.

✅ 22/22 dimensions clean — no findings.

Scope: 2 files, purely structural refactoring — no behavioral change.

• SourceGeneratedReflectionOperations.cs: early-continue → nested if (attr is TAttribute match) is semantically identical; logic (duplicate-check, then assign) is preserved in the same order.
• PackagedAppTestHostHandle.cs: catch (Exception ex) when (ex is IOException or UnauthorizedAccessException) captures exactly the same exception types as the two preceding separate catch blocks, with the same handler body.

No public API surface changed, no threading concerns, no localization regressions, no test-isolation issues. The or pattern in the when clause is C# 9, consistent with the net8.0+-only target of Microsoft.Testing.Extensions.PackagedApp.

[Amaury Levé (Evangelink)]

Code Simplification — 2026-06-29

This PR simplifies two small pieces of recently modified code to improve clarity and consistency, without any behavioral change.

Files Simplified

• src/Platform/Microsoft.Testing.Extensions.PackagedApp/PackagedAppTestHostHandle.cs — combine two identical catch blocks
• src/Adapter/MSTestAdapter.PlatformServices/SourceGeneration/SourceGeneratedReflectionOperations.cs — align loop pattern with adjacent methods

Improvements Made

1. PackagedAppTestHostHandle — eliminate duplicate catch bodies (from #9454)

The Dispose method had two catch blocks with identical handler bodies:

// Beforecatch(IOExceptionex){Debug.WriteLine($"Best-effort cleanup of deployment directory '{_deploymentDirectory}' failed: {ex}");}catch(UnauthorizedAccessExceptionex){Debug.WriteLine($"Best-effort cleanup of deployment directory '{_deploymentDirectory}' failed: {ex}");}

Combined into a single handler using a C# exception filter, eliminating the duplication:

// Aftercatch(Exceptionex)when(exisIOException or UnauthorizedAccessException){Debug.WriteLine($"Best-effort cleanup of deployment directory '{_deploymentDirectory}' failed: {ex}");}

2. SourceGeneratedReflectionOperations.GetSingleAttributeOrDefault — loop consistency (from #9479)

The GetSingleAttributeOrDefault method used a is not T / continue guard, while the two adjacent methods (IsAttributeDefined and GetFirstAttributeOrDefault) that were introduced in the same PR use a direct positive is T match. Aligned the style:

// Before (uses negated guard + continue)if(attris not TAttributematch){continue;}if(firstis not null){throw ...;}first=match;// After (positive match, consistent with adjacent methods)if(attrisTAttributematch){if(firstis not null){throw ...;}first=match;}

Changes Based On

Recent changes from:

• Add generic ITestHostLauncher extension point + PackagedApp reference extension #9454 — Add generic ITestHostLauncher extension point + PackagedApp reference extension
• [efficiency-improver] perf: eliminate LINQ iterator allocations in SourceGeneratedReflectionOperations attribute lookup #9479 — perf: eliminate LINQ iterator allocations in SourceGeneratedReflectionOperations attribute lookup

Testing

• ✅ No functional changes — behavior is identical
• ✅ No tests need updating; the changes are purely structural

🤖 Automated content by GitHub Copilot. Posted via a maintainer's GitHub token, so it appears under their account — the account owner did not write or approve this content personally. Generated by the Code Simplifier workflow. · 217.9 AIC · ⌖ 18.3 AIC · ⊞ 9.3K · [◷]( · ◷)

Add this agentic workflows to your repo

To install this agentic workflow, run

gh aw add githubnext/agentics/workflows/code-simplifier.md@main

• expires on Jun 30, 2026, 2:46 PM UTC

[Copilot]

Pull request overview

This PR applies small, targeted refactors in Microsoft.Testing.Platform and MSTestAdapter.PlatformServices to reduce duplication and align local coding style, without changing observable behavior.

Changes:

• Consolidate two identical catch blocks in PackagedAppTestHostHandle.Dispose() via an exception filter.
• Adjust GetSingleAttributeOrDefault<TAttribute>’s loop to use a positive type match pattern consistent with adjacent attribute helpers.

Show a summary per file

File	Description
src/Platform/Microsoft.Testing.Extensions.PackagedApp/PackagedAppTestHostHandle.cs	Replaces duplicated IOException/UnauthorizedAccessException handlers with a single filtered catch block.
src/Adapter/MSTestAdapter.PlatformServices/SourceGeneration/SourceGeneratedReflectionOperations.cs	Refactors the attribute scan loop in GetSingleAttributeOrDefault for consistency with neighboring methods.

Review details

• Files reviewed: 2/2 changed files
• Comments generated: 0
• Review effort level: Low

[Amaury Levé (Evangelink)]

Note

🤖 Automated review by GitHub Copilot. Posted via a maintainer's GitHub token, so it appears under their account — the account owner did not write or approve this content personally. Generated by the Expert Code Review workflow. To request a follow-up action, reply by tagging @copilot directly.

✅ 22/22 dimensions clean — no findings.

Scope: 2 files, purely structural refactoring — no behavioral change.

• SourceGeneratedReflectionOperations.cs: early-continue → nested if (attr is TAttribute match) is semantically identical; logic (duplicate-check, then assign) is preserved in the same order.
• PackagedAppTestHostHandle.cs: catch (Exception ex) when (ex is IOException or UnauthorizedAccessException) captures exactly the same exception types as the two preceding separate catch blocks, with the same handler body.

No public API surface changed, no threading concerns, no localization regressions, no test-isolation issues. The or pattern in the when clause is C# 9, consistent with the net8.0+-only target of Microsoft.Testing.Extensions.PackagedApp.

[Amaury Levé (Evangelink)]

Code Simplification — 2026-06-29

This PR simplifies two small pieces of recently modified code to improve clarity and consistency, without any behavioral change.

Files Simplified

• src/Platform/Microsoft.Testing.Extensions.PackagedApp/PackagedAppTestHostHandle.cs — combine two identical catch blocks
• src/Adapter/MSTestAdapter.PlatformServices/SourceGeneration/SourceGeneratedReflectionOperations.cs — align loop pattern with adjacent methods

Improvements Made

1. PackagedAppTestHostHandle — eliminate duplicate catch bodies (from #9454)

The Dispose method had two catch blocks with identical handler bodies:

// Beforecatch(IOExceptionex){Debug.WriteLine($"Best-effort cleanup of deployment directory '{_deploymentDirectory}' failed: {ex}");}catch(UnauthorizedAccessExceptionex){Debug.WriteLine($"Best-effort cleanup of deployment directory '{_deploymentDirectory}' failed: {ex}");}

Combined into a single handler using a C# exception filter, eliminating the duplication:

// Aftercatch(Exceptionex)when(exisIOException or UnauthorizedAccessException){Debug.WriteLine($"Best-effort cleanup of deployment directory '{_deploymentDirectory}' failed: {ex}");}

2. SourceGeneratedReflectionOperations.GetSingleAttributeOrDefault — loop consistency (from #9479)

The GetSingleAttributeOrDefault method used a is not T / continue guard, while the two adjacent methods (IsAttributeDefined and GetFirstAttributeOrDefault) that were introduced in the same PR use a direct positive is T match. Aligned the style:

// Before (uses negated guard + continue)if(attris not TAttributematch){continue;}if(firstis not null){throw ...;}first=match;// After (positive match, consistent with adjacent methods)if(attrisTAttributematch){if(firstis not null){throw ...;}first=match;}

Changes Based On

Recent changes from:

• Add generic ITestHostLauncher extension point + PackagedApp reference extension #9454 — Add generic ITestHostLauncher extension point + PackagedApp reference extension
• [efficiency-improver] perf: eliminate LINQ iterator allocations in SourceGeneratedReflectionOperations attribute lookup #9479 — perf: eliminate LINQ iterator allocations in SourceGeneratedReflectionOperations attribute lookup

Testing

• ✅ No functional changes — behavior is identical
• ✅ No tests need updating; the changes are purely structural

🤖 Automated content by GitHub Copilot. Posted via a maintainer's GitHub token, so it appears under their account — the account owner did not write or approve this content personally. Generated by the Code Simplifier workflow. · 217.9 AIC · ⌖ 18.3 AIC · ⊞ 9.3K · [◷]( · ◷)

Add this agentic workflows to your repo

To install this agentic workflow, run

gh aw add githubnext/agentics/workflows/code-simplifier.md@main

• expires on Jun 30, 2026, 2:46 PM UTC

[Copilot]

Pull request overview

This PR applies small, targeted refactors in Microsoft.Testing.Platform and MSTestAdapter.PlatformServices to reduce duplication and align local coding style, without changing observable behavior.

Changes:

• Consolidate two identical catch blocks in PackagedAppTestHostHandle.Dispose() via an exception filter.
• Adjust GetSingleAttributeOrDefault<TAttribute>’s loop to use a positive type match pattern consistent with adjacent attribute helpers.

Show a summary per file

File	Description
src/Platform/Microsoft.Testing.Extensions.PackagedApp/PackagedAppTestHostHandle.cs	Replaces duplicated IOException/UnauthorizedAccessException handlers with a single filtered catch block.
src/Adapter/MSTestAdapter.PlatformServices/SourceGeneration/SourceGeneratedReflectionOperations.cs	Refactors the attribute scan loop in GetSingleAttributeOrDefault for consistency with neighboring methods.

Review details

• Files reviewed: 2/2 changed files
• Comments generated: 0
• Review effort level: Low

[Amaury Levé (Evangelink)]

Note

🤖 Automated review by GitHub Copilot. Posted via a maintainer's GitHub token, so it appears under their account — the account owner did not write or approve this content personally. Generated by the Expert Code Review workflow. To request a follow-up action, reply by tagging @copilot directly.

✅ 22/22 dimensions clean — no findings.

Scope: 2 files, purely structural refactoring — no behavioral change.

• SourceGeneratedReflectionOperations.cs: early-continue → nested if (attr is TAttribute match) is semantically identical; logic (duplicate-check, then assign) is preserved in the same order.
• PackagedAppTestHostHandle.cs: catch (Exception ex) when (ex is IOException or UnauthorizedAccessException) captures exactly the same exception types as the two preceding separate catch blocks, with the same handler body.

No public API surface changed, no threading concerns, no localization regressions, no test-isolation issues. The or pattern in the when clause is C# 9, consistent with the net8.0+-only target of Microsoft.Testing.Extensions.PackagedApp.

[Amaury Levé (Evangelink)]

Code Simplification — 2026-06-29

This PR simplifies two small pieces of recently modified code to improve clarity and consistency, without any behavioral change.

Files Simplified

• src/Platform/Microsoft.Testing.Extensions.PackagedApp/PackagedAppTestHostHandle.cs — combine two identical catch blocks
• src/Adapter/MSTestAdapter.PlatformServices/SourceGeneration/SourceGeneratedReflectionOperations.cs — align loop pattern with adjacent methods

Improvements Made

1. PackagedAppTestHostHandle — eliminate duplicate catch bodies (from #9454)

The Dispose method had two catch blocks with identical handler bodies:

// Beforecatch(IOExceptionex){Debug.WriteLine($"Best-effort cleanup of deployment directory '{_deploymentDirectory}' failed: {ex}");}catch(UnauthorizedAccessExceptionex){Debug.WriteLine($"Best-effort cleanup of deployment directory '{_deploymentDirectory}' failed: {ex}");}

Combined into a single handler using a C# exception filter, eliminating the duplication:

// Aftercatch(Exceptionex)when(exisIOException or UnauthorizedAccessException){Debug.WriteLine($"Best-effort cleanup of deployment directory '{_deploymentDirectory}' failed: {ex}");}

2. SourceGeneratedReflectionOperations.GetSingleAttributeOrDefault — loop consistency (from #9479)

The GetSingleAttributeOrDefault method used a is not T / continue guard, while the two adjacent methods (IsAttributeDefined and GetFirstAttributeOrDefault) that were introduced in the same PR use a direct positive is T match. Aligned the style:

// Before (uses negated guard + continue)if(attris not TAttributematch){continue;}if(firstis not null){throw ...;}first=match;// After (positive match, consistent with adjacent methods)if(attrisTAttributematch){if(firstis not null){throw ...;}first=match;}

Changes Based On

Recent changes from:

• Add generic ITestHostLauncher extension point + PackagedApp reference extension #9454 — Add generic ITestHostLauncher extension point + PackagedApp reference extension
• [efficiency-improver] perf: eliminate LINQ iterator allocations in SourceGeneratedReflectionOperations attribute lookup #9479 — perf: eliminate LINQ iterator allocations in SourceGeneratedReflectionOperations attribute lookup

Testing

• ✅ No functional changes — behavior is identical
• ✅ No tests need updating; the changes are purely structural

🤖 Automated content by GitHub Copilot. Posted via a maintainer's GitHub token, so it appears under their account — the account owner did not write or approve this content personally. Generated by the Code Simplifier workflow. · 217.9 AIC · ⌖ 18.3 AIC · ⊞ 9.3K · [◷]( · ◷)

Add this agentic workflows to your repo

To install this agentic workflow, run

gh aw add githubnext/agentics/workflows/code-simplifier.md@main

• expires on Jun 30, 2026, 2:46 PM UTC

[Copilot]

Pull request overview

This PR applies small, targeted refactors in Microsoft.Testing.Platform and MSTestAdapter.PlatformServices to reduce duplication and align local coding style, without changing observable behavior.

Changes:

• Consolidate two identical catch blocks in PackagedAppTestHostHandle.Dispose() via an exception filter.
• Adjust GetSingleAttributeOrDefault<TAttribute>’s loop to use a positive type match pattern consistent with adjacent attribute helpers.

Show a summary per file

File	Description
src/Platform/Microsoft.Testing.Extensions.PackagedApp/PackagedAppTestHostHandle.cs	Replaces duplicated IOException/UnauthorizedAccessException handlers with a single filtered catch block.
src/Adapter/MSTestAdapter.PlatformServices/SourceGeneration/SourceGeneratedReflectionOperations.cs	Refactors the attribute scan loop in GetSingleAttributeOrDefault for consistency with neighboring methods.

Review details

• Files reviewed: 2/2 changed files
• Comments generated: 0
• Review effort level: Low

[Amaury Levé (Evangelink)]

Note

🤖 Automated review by GitHub Copilot. Posted via a maintainer's GitHub token, so it appears under their account — the account owner did not write or approve this content personally. Generated by the Expert Code Review workflow. To request a follow-up action, reply by tagging @copilot directly.

✅ 22/22 dimensions clean — no findings.

Scope: 2 files, purely structural refactoring — no behavioral change.

• SourceGeneratedReflectionOperations.cs: early-continue → nested if (attr is TAttribute match) is semantically identical; logic (duplicate-check, then assign) is preserved in the same order.
• PackagedAppTestHostHandle.cs: catch (Exception ex) when (ex is IOException or UnauthorizedAccessException) captures exactly the same exception types as the two preceding separate catch blocks, with the same handler body.

No public API surface changed, no threading concerns, no localization regressions, no test-isolation issues. The or pattern in the when clause is C# 9, consistent with the net8.0+-only target of Microsoft.Testing.Extensions.PackagedApp.

[Amaury Levé (Evangelink)]

Code Simplification — 2026-06-29

This PR simplifies two small pieces of recently modified code to improve clarity and consistency, without any behavioral change.

Files Simplified

• src/Platform/Microsoft.Testing.Extensions.PackagedApp/PackagedAppTestHostHandle.cs — combine two identical catch blocks
• src/Adapter/MSTestAdapter.PlatformServices/SourceGeneration/SourceGeneratedReflectionOperations.cs — align loop pattern with adjacent methods

Improvements Made

1. PackagedAppTestHostHandle — eliminate duplicate catch bodies (from #9454)

The Dispose method had two catch blocks with identical handler bodies:

// Beforecatch(IOExceptionex){Debug.WriteLine($"Best-effort cleanup of deployment directory '{_deploymentDirectory}' failed: {ex}");}catch(UnauthorizedAccessExceptionex){Debug.WriteLine($"Best-effort cleanup of deployment directory '{_deploymentDirectory}' failed: {ex}");}

Combined into a single handler using a C# exception filter, eliminating the duplication:

// Aftercatch(Exceptionex)when(exisIOException or UnauthorizedAccessException){Debug.WriteLine($"Best-effort cleanup of deployment directory '{_deploymentDirectory}' failed: {ex}");}

2. SourceGeneratedReflectionOperations.GetSingleAttributeOrDefault — loop consistency (from #9479)

The GetSingleAttributeOrDefault method used a is not T / continue guard, while the two adjacent methods (IsAttributeDefined and GetFirstAttributeOrDefault) that were introduced in the same PR use a direct positive is T match. Aligned the style:

// Before (uses negated guard + continue)if(attris not TAttributematch){continue;}if(firstis not null){throw ...;}first=match;// After (positive match, consistent with adjacent methods)if(attrisTAttributematch){if(firstis not null){throw ...;}first=match;}

Changes Based On

Recent changes from:

• Add generic ITestHostLauncher extension point + PackagedApp reference extension #9454 — Add generic ITestHostLauncher extension point + PackagedApp reference extension
• [efficiency-improver] perf: eliminate LINQ iterator allocations in SourceGeneratedReflectionOperations attribute lookup #9479 — perf: eliminate LINQ iterator allocations in SourceGeneratedReflectionOperations attribute lookup

Testing

• ✅ No functional changes — behavior is identical
• ✅ No tests need updating; the changes are purely structural

🤖 Automated content by GitHub Copilot. Posted via a maintainer's GitHub token, so it appears under their account — the account owner did not write or approve this content personally. Generated by the Code Simplifier workflow. · 217.9 AIC · ⌖ 18.3 AIC · ⊞ 9.3K · [◷]( · ◷)

Add this agentic workflows to your repo

To install this agentic workflow, run

gh aw add githubnext/agentics/workflows/code-simplifier.md@main

• expires on Jun 30, 2026, 2:46 PM UTC

[Copilot]

Pull request overview

This PR applies small, targeted refactors in Microsoft.Testing.Platform and MSTestAdapter.PlatformServices to reduce duplication and align local coding style, without changing observable behavior.

Changes:

• Consolidate two identical catch blocks in PackagedAppTestHostHandle.Dispose() via an exception filter.
• Adjust GetSingleAttributeOrDefault<TAttribute>’s loop to use a positive type match pattern consistent with adjacent attribute helpers.

Show a summary per file

File	Description
src/Platform/Microsoft.Testing.Extensions.PackagedApp/PackagedAppTestHostHandle.cs	Replaces duplicated IOException/UnauthorizedAccessException handlers with a single filtered catch block.
src/Adapter/MSTestAdapter.PlatformServices/SourceGeneration/SourceGeneratedReflectionOperations.cs	Refactors the attribute scan loop in GetSingleAttributeOrDefault for consistency with neighboring methods.

Review details

• Files reviewed: 2/2 changed files
• Comments generated: 0
• Review effort level: Low

[Amaury Levé (Evangelink)]

Note

🤖 Automated review by GitHub Copilot. Posted via a maintainer's GitHub token, so it appears under their account — the account owner did not write or approve this content personally. Generated by the Expert Code Review workflow. To request a follow-up action, reply by tagging @copilot directly.

✅ 22/22 dimensions clean — no findings.

Scope: 2 files, purely structural refactoring — no behavioral change.

• SourceGeneratedReflectionOperations.cs: early-continue → nested if (attr is TAttribute match) is semantically identical; logic (duplicate-check, then assign) is preserved in the same order.
• PackagedAppTestHostHandle.cs: catch (Exception ex) when (ex is IOException or UnauthorizedAccessException) captures exactly the same exception types as the two preceding separate catch blocks, with the same handler body.

No public API surface changed, no threading concerns, no localization regressions, no test-isolation issues. The or pattern in the when clause is C# 9, consistent with the net8.0+-only target of Microsoft.Testing.Extensions.PackagedApp.

[Amaury Levé (Evangelink)]

Code Simplification — 2026-06-29

This PR simplifies two small pieces of recently modified code to improve clarity and consistency, without any behavioral change.

Files Simplified

• src/Platform/Microsoft.Testing.Extensions.PackagedApp/PackagedAppTestHostHandle.cs — combine two identical catch blocks
• src/Adapter/MSTestAdapter.PlatformServices/SourceGeneration/SourceGeneratedReflectionOperations.cs — align loop pattern with adjacent methods

Improvements Made

1. PackagedAppTestHostHandle — eliminate duplicate catch bodies (from #9454)

The Dispose method had two catch blocks with identical handler bodies:

// Beforecatch(IOExceptionex){Debug.WriteLine($"Best-effort cleanup of deployment directory '{_deploymentDirectory}' failed: {ex}");}catch(UnauthorizedAccessExceptionex){Debug.WriteLine($"Best-effort cleanup of deployment directory '{_deploymentDirectory}' failed: {ex}");}

Combined into a single handler using a C# exception filter, eliminating the duplication:

// Aftercatch(Exceptionex)when(exisIOException or UnauthorizedAccessException){Debug.WriteLine($"Best-effort cleanup of deployment directory '{_deploymentDirectory}' failed: {ex}");}

2. SourceGeneratedReflectionOperations.GetSingleAttributeOrDefault — loop consistency (from #9479)

The GetSingleAttributeOrDefault method used a is not T / continue guard, while the two adjacent methods (IsAttributeDefined and GetFirstAttributeOrDefault) that were introduced in the same PR use a direct positive is T match. Aligned the style:

// Before (uses negated guard + continue)if(attris not TAttributematch){continue;}if(firstis not null){throw ...;}first=match;// After (positive match, consistent with adjacent methods)if(attrisTAttributematch){if(firstis not null){throw ...;}first=match;}

Changes Based On

Recent changes from:

• Add generic ITestHostLauncher extension point + PackagedApp reference extension #9454 — Add generic ITestHostLauncher extension point + PackagedApp reference extension
• [efficiency-improver] perf: eliminate LINQ iterator allocations in SourceGeneratedReflectionOperations attribute lookup #9479 — perf: eliminate LINQ iterator allocations in SourceGeneratedReflectionOperations attribute lookup

Testing

• ✅ No functional changes — behavior is identical
• ✅ No tests need updating; the changes are purely structural

🤖 Automated content by GitHub Copilot. Posted via a maintainer's GitHub token, so it appears under their account — the account owner did not write or approve this content personally. Generated by the Code Simplifier workflow. · 217.9 AIC · ⌖ 18.3 AIC · ⊞ 9.3K · [◷]( · ◷)

Add this agentic workflows to your repo

To install this agentic workflow, run

gh aw add githubnext/agentics/workflows/code-simplifier.md@main

• expires on Jun 30, 2026, 2:46 PM UTC

[Copilot]

Pull request overview

This PR applies small, targeted refactors in Microsoft.Testing.Platform and MSTestAdapter.PlatformServices to reduce duplication and align local coding style, without changing observable behavior.

Changes:

• Consolidate two identical catch blocks in PackagedAppTestHostHandle.Dispose() via an exception filter.
• Adjust GetSingleAttributeOrDefault<TAttribute>’s loop to use a positive type match pattern consistent with adjacent attribute helpers.

Show a summary per file

File	Description
src/Platform/Microsoft.Testing.Extensions.PackagedApp/PackagedAppTestHostHandle.cs	Replaces duplicated IOException/UnauthorizedAccessException handlers with a single filtered catch block.
src/Adapter/MSTestAdapter.PlatformServices/SourceGeneration/SourceGeneratedReflectionOperations.cs	Refactors the attribute scan loop in GetSingleAttributeOrDefault for consistency with neighboring methods.

Review details

• Files reviewed: 2/2 changed files
• Comments generated: 0
• Review effort level: Low

[Amaury Levé (Evangelink)]

Note

🤖 Automated review by GitHub Copilot. Posted via a maintainer's GitHub token, so it appears under their account — the account owner did not write or approve this content personally. Generated by the Expert Code Review workflow. To request a follow-up action, reply by tagging @copilot directly.

✅ 22/22 dimensions clean — no findings.

Scope: 2 files, purely structural refactoring — no behavioral change.

• SourceGeneratedReflectionOperations.cs: early-continue → nested if (attr is TAttribute match) is semantically identical; logic (duplicate-check, then assign) is preserved in the same order.
• PackagedAppTestHostHandle.cs: catch (Exception ex) when (ex is IOException or UnauthorizedAccessException) captures exactly the same exception types as the two preceding separate catch blocks, with the same handler body.

No public API surface changed, no threading concerns, no localization regressions, no test-isolation issues. The or pattern in the when clause is C# 9, consistent with the net8.0+-only target of Microsoft.Testing.Extensions.PackagedApp.