Skip to content

#34: God-class refactor, CI workflow, and quality rules#35

Merged
Unthred merged 1 commit into
mainfrom
feature/34-god-class-refactor
Jun 22, 2026
Merged

#34: God-class refactor, CI workflow, and quality rules#35
Unthred merged 1 commit into
mainfrom
feature/34-god-class-refactor

Conversation

@Unthred

@Unthred Unthred commented Jun 22, 2026

Copy link
Copy Markdown
Owner

Closes #34
Closes #32

Summary

  • Split ProjectRuntime into ProjectRuntime.cs; ProjectOrchestrator.cs is now ~393 lines (was ~2,700)
  • Extract TrayTooltipFormatter and BuildLifecycleFormatting (Core) plus BuildLifecycleToastNotifier; App.xaml.cs reduced ~155 lines
  • Add ci.yml build+test on windows-latest for every PR
  • Add commit-msg hook, PR issue-link workflow, board sync script (from Ops: Project board backlog sync and issue-linked commit enforcement #32)
  • Add code-structure.mdc and strengthen testing.mdc (Tier 2 orchestration tests mandatory)
  • InternalsVisibleTo for tests; fix BuildLogParserTests warning-count fixture

Refactor follow-ups

  • ProjectRuntime.cs is still ~2,300 lines — next splits: build, run/watch, and test pipelines
  • App.xaml.cs ~1,450 lines — tray menu builder is the next extraction candidate

Test plan

  • dotnet build BuildMonitor.slnx -c Release
  • dotnet test (111 passed)
  • CI green on PR
  • install-githooks.ps1 on other clones

Move ProjectRuntime to its own file; extract tray tooltip and build-lifecycle toast helpers.
Add CI build/test workflow, commit-msg hook, board sync script, and code-structure/testing rules.
@Unthred Unthred merged commit f3a92a9 into main Jun 22, 2026
3 checks passed
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.

Refactor god classes; CI workflow and orchestration test rules Ops: Project board backlog sync and issue-linked commit enforcement

1 participant