Skip to content

Add --reset-schedule flag for CLI and GUI installers#98

Merged
erikdarlingdata merged 1 commit intodevfrom
feature/92-reset-schedule
Feb 17, 2026
Merged

Add --reset-schedule flag for CLI and GUI installers#98
erikdarlingdata merged 1 commit intodevfrom
feature/92-reset-schedule

Conversation

@erikdarlingdata
Copy link
Owner

Summary

  • Adds --reset-schedule CLI flag that truncates config.collection_schedule before re-populating with current recommended defaults
  • Adds "Reset collection schedule to recommended defaults" checkbox in GUI installer
  • Both installers prepend TRUNCATE TABLE config.collection_schedule; GO to the 04_create_schedule_table.sql content at runtime when the flag/checkbox is set

Why

When we tune the default schedule (e.g., PR #88 bumped cheap collectors from 5-min to 1-min), existing users keep the old schedule because 04_ uses WHERE NOT EXISTS inserts. This flag lets users opt in to the latest recommended frequencies on re-install.

Test plan

  • CLI: --reset-schedule on sql2022 update install — schedule reset to 33 collectors with current frequencies
  • CLI: Without flag — existing schedule preserved (only new collectors added via WHERE NOT EXISTS)
  • GUI: Checkbox wired to ExecuteInstallationAsync resetSchedule parameter
  • GUI: Visual test of checkbox and install flow

Closes #92

🤖 Generated with Claude Code

CLI: --reset-schedule flag prepends TRUNCATE to 04_create_schedule_table.sql
GUI: "Reset collection schedule to recommended defaults" checkbox
Both pass resetSchedule to installer logic which truncates config.collection_schedule
before the WHERE NOT EXISTS inserts re-populate with current defaults.

Tested on sql2022 with update install — schedule reset to 33 collectors with
current recommended frequencies.

Closes #92

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@erikdarlingdata erikdarlingdata merged commit 22bd9b6 into dev Feb 17, 2026
2 checks passed
@erikdarlingdata erikdarlingdata deleted the feature/92-reset-schedule branch February 20, 2026 13:20
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