Skip to content

Cycle 19: Carry-Forward Clearance#644

Merged
rygel merged 5 commits into
developfrom
feat/cycle-19-core
Jun 12, 2026
Merged

Cycle 19: Carry-Forward Clearance#644
rygel merged 5 commits into
developfrom
feat/cycle-19-core

Conversation

@rygel

@rygel rygel commented Jun 12, 2026

Copy link
Copy Markdown
Owner

Changes

task-88: Fix BetaUpdateCheck version comparison test

  • Fixed ParseAppVersion to strip -develop suffix from beta numbers
  • int.TryParse on "5-develop" was failing, defaulting preRelease to 0
  • Added unit tests for -develop suffix variants

task-89: Delete dead scaffolded types

  • Removed PercentageValueSemantic.cs, BudgetPolicy.cs, ProviderResponseException.cs
  • 59 lines of dead scaffolded code deleted

task-61: Collapse single-impl interfaces

  • Removed IPreferencesStore, IUsageAnalyticsService, IMonitorLauncher
  • All 3 had single implementations, never mocked in tests
  • IMonitorLauncherClient preserved per task-57 resolution
  • 16 files updated across Core, Web, UI.Slim, CLI, Tests

task-75: Move MonitorClient process/HTTP code to Infrastructure

  • Moved MonitorLauncher, MonitorLauncherProcessController, MonitorLifecycleService, MonitorService
  • From Core.MonitorClient to Infrastructure.MonitorClient namespace
  • DTO/snapshot types remain in Core.MonitorClient
  • 23 files updated (namespace changes + using statements)

Test Results

  • 1297 passed, 0 failed, 2 skipped (non-integration)
  • Build: 0 errors

@rygel
rygel merged commit a0fd59a into develop Jun 12, 2026
8 of 9 checks passed
@rygel
rygel deleted the feat/cycle-19-core branch June 12, 2026 15:10
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.

2 participants