Skip to content

refactor: consolidate auto-start settings and fix stale appcast#582

Merged
rygel merged 2 commits intodevelopfrom
refactor/consolidate-autostart
Apr 26, 2026
Merged

refactor: consolidate auto-start settings and fix stale appcast#582
rygel merged 2 commits intodevelopfrom
refactor/consolidate-autostart

Conversation

@rygel
Copy link
Copy Markdown
Owner

@rygel rygel commented Apr 26, 2026

Summary

  • Remove redundant Monitor auto-start checkbox from Monitor tab (was dead code, never wired)
  • Remove "Start Monitor with Windows" from Layout tab; simplify to single "Start with Windows" checkbox
  • Simplify WindowsStartupService to manage only the UI registry entry (Monitor always auto-starts with UI)
  • Remove StartMonitorWithWindows from AppPreferences
  • Remove Monitor startup task from installer (setup.iss)
  • Update beta appcast files from stale v2.3.4-beta.11 to current v2.3.4-beta.33

Test plan

  • Build succeeds (dotnet build)
  • All 1307 tests pass (4 previously failing appcast tests now pass)
  • Verify Settings > Layout tab shows single "Start with Windows" checkbox
  • Verify Monitor tab no longer shows dead "Auto-start Monitor" checkbox

Remove redundant Monitor auto-start controls from Monitor tab and Layout tab.
The UI always auto-starts the Monitor, so only a single 'Start with Windows'
checkbox is needed to control the HKCU Run registry entry.
Appcast files were stale, still referencing v2.3.4-beta.11. Updated to
point to the latest beta release with correct asset sizes.
@rygel rygel merged commit d23d8d5 into develop Apr 26, 2026
18 of 19 checks passed
@rygel rygel deleted the refactor/consolidate-autostart branch April 26, 2026 15:21
@rygel rygel mentioned this pull request Apr 26, 2026
4 tasks
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