Skip to content

Refactor large services into focused partial classes - #155

Merged
Widthdom merged 1 commit into
mainfrom
codex/split-large-methods-20260416
Apr 15, 2026
Merged

Refactor large services into focused partial classes#155
Widthdom merged 1 commit into
mainfrom
codex/split-large-methods-20260416

Conversation

@Widthdom

Copy link
Copy Markdown
Owner

Summary

Split several oversized service files into focused partial classes without changing behavior.

This refactor:

  • extracts ProgramRunner constructor/constants into Runner/ProgramRunner.Core.cs
  • moves IL comparison/filtering helpers into Services/ILOutputService.Comparison.cs
  • moves Markdown report helper logic into Services/ReportGenerateService.Helpers.cs
  • extracts HTML semantic change rendering into Services/HtmlReport/HtmlReportGenerateService.SemanticChanges.cs
  • simplifies shared HTML detail row rendering across inline diffs, semantic changes, and dependency changes
  • updates doc/DEVELOPER_GUIDE.md and CHANGELOG.md to match the new file layout

Why

Some core services had grown beyond the repository's preferred file-size limits, which made navigation and maintenance harder.

This change keeps the existing call paths and behavior intact while making responsibilities easier to locate:

  • orchestration stays in the main partial
  • comparison/rendering helpers move into dedicated partials
  • shared HTML detail rendering becomes easier to reuse consistently

Validation

  • dotnet test FolderDiffIL4DotNet.Tests/FolderDiffIL4DotNet.Tests.csproj --configuration Release --nologo
  • Result: 1919 passed, 1 skipped

@Widthdom
Widthdom merged commit b0f7fd1 into main Apr 15, 2026
8 checks passed
@Widthdom
Widthdom deleted the codex/split-large-methods-20260416 branch April 25, 2026 02:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant