Split GitHub Actions summary reporter into focused partials - #10859
Conversation
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Note
🤖 Automated review by GitHub Copilot. Generated by the Expert Code Review workflow. To request a follow-up action, reply by tagging @copilot directly.
✅ 22/22 dimensions clean — no findings.
This is a pure mechanical refactor that splits GitHubActionsSummaryReporter.cs into three focused partial class files (FailureTracking, Snapshot, SummaryWriting). The code is moved verbatim — no logic, API surface, or behavioral changes. The using Microsoft.Testing.Platform; directive removed from the main file is correctly placed in the partials that reference its types. Clean split with no issues.
There was a problem hiding this comment.
Copilot review overview
Review tier: Balanced
Findings: None
What changed in this PR
Refactors the GitHub Actions summary reporter into focused partial classes without changing behavior or API surface.
Changes:
- Separates failure tracking, snapshot construction, and summary writing.
- Reduces every reporter file below 300 lines.
- Preserves existing lifecycle and aggregation logic.
| File | Description |
|---|---|
GitHubActionsSummaryReporter.cs |
Retains core wiring and event ingestion. |
GitHubActionsSummaryReporter.FailureTracking.cs |
Contains failure retention and diagnostics. |
GitHubActionsSummaryReporter.Snapshot.cs |
Contains snapshot and module construction. |
GitHubActionsSummaryReporter.SummaryWriting.cs |
Contains session completion and summary writing. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
GitHubActionsSummaryReporter.cshad grown difficult to navigate because it combined event ingestion, failure retention, snapshot construction, and step-summary writing in one file. This refactor separates those responsibilities while preserving behavior and API surface.Changes
Validation
GitHubActionsSummaryReporterTests.Fixes #10848