Skip to content

Split GitHub Actions summary reporter into focused partials - #10859

Merged
Amaury Levé (Evangelink) merged 1 commit into
mainfrom
dev/amauryleve/fix-issue-10848
Aug 31, 2026
Merged

Split GitHub Actions summary reporter into focused partials#10859
Amaury Levé (Evangelink) merged 1 commit into
mainfrom
dev/amauryleve/fix-issue-10848

Conversation

@Evangelink

Copy link
Copy Markdown
Member

GitHubActionsSummaryReporter.cs had 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

  • Move failure retention, ordering, and diagnostic capture into a focused partial.
  • Move immutable snapshot and aggregation-module construction into a focused partial.
  • Move test-session finishing and step-summary write orchestration into a focused partial.
  • Keep the core reporter and every new split file below 300 lines.

Validation

  • Built the GitHub Actions reporter for all target frameworks with zero warnings.
  • Passed all 95 GitHubActionsSummaryReporterTests.
  • Passed the full extensions unit-test suite: 1,490 passed and 7 platform-specific skips.

Fixes #10848

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI balanced review requested due to automatic review settings August 29, 2026 08:13
@Evangelink Amaury Levé (Evangelink) added the state/needs-review Awaiting review from the team. label Aug 29, 2026
@Evangelink
Amaury Levé (Evangelink) enabled auto-merge (squash) August 29, 2026 08:17

@github-actions github-actions Bot 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.

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.

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.

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.

@Evangelink
Amaury Levé (Evangelink) merged commit 8246177 into main Aug 31, 2026
27 checks passed
@Evangelink
Amaury Levé (Evangelink) deleted the dev/amauryleve/fix-issue-10848 branch August 31, 2026 06:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

state/needs-review Awaiting review from the team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[file-diet] Refactor GitHubActionsSummaryReporter.cs (854 lines) into focused modules

3 participants