Skip to content

Comments

feat: configurable capture settings in main window#33

Merged
pungys97 merged 1 commit intomainfrom
issue-28-configurable-capture-settings
Feb 19, 2026
Merged

feat: configurable capture settings in main window#33
pungys97 merged 1 commit intomainfrom
issue-28-configurable-capture-settings

Conversation

@pungys97
Copy link
Contributor

Closes #28

Summary

  • CaptureSettingsManager — persists 7 tunable capture settings to userData/capture-settings.json; changes apply immediately via applyToConstants() (no restart needed)
  • Advanced Settings page — accessible from a button in the main window; uses simple useState page navigation (no router dependency)
  • LLM Config section — ManageKeySection and CustomEndpointSection consolidated under one section with an "or" separator to make the two options mutually exclusive and clear
  • Sliders — Visual threshold, typing/scroll/click debounce, min/max activity duration, max screenshots per activity; explicit Save button (disabled until dirty) + Reset to defaults
  • 13 unit tests for CaptureSettingsManager covering defaults, persist/load, reset, and applyToConstants

Test plan

  • Open app → click "Advanced Settings" (bottom-right of main window)
  • Adjust a slider → Save → confirm toast; reopen settings to verify value persisted
  • Reset to defaults → confirm sliders return to original values
  • LLM Config section shows key badge + Manage Key, with "or" / Custom endpoint below
  • npm run test passes

🤖 Generated with Claude Code

- Add CaptureSettingsManager — persists settings to userData/capture-settings.json,
  applies changes immediately via applyToConstants() (no restart required)
- Expose 7 tunable settings: visual threshold, typing/scroll/click debounce,
  min/max activity duration, max screenshots per activity
- Add Advanced Settings page inside main window (React useState navigation),
  with sliders, explicit Save/Reset buttons, and toast feedback
- Move ManageKeySection and CustomEndpointSection into Advanced Settings under
  a unified "LLM Config" section with "or" separator to communicate mutual exclusivity
- Add 13 unit tests for CaptureSettingsManager (defaults, save/load, reset, applyToConstants)
- Improve SemanticClassifier error logging with structured describeError() helper

Co-Authored-By: Claude Sonnet 4.6 <[email protected]>
@pungys97 pungys97 merged commit a8eac8f into main Feb 19, 2026
1 check failed
@pungys97 pungys97 deleted the issue-28-configurable-capture-settings branch February 19, 2026 12:25
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.

Configurable capture settings?

1 participant