From ddfd62b8ce3240d954460e918fc0d351519a22cb Mon Sep 17 00:00:00 2001 From: Claude Date: Thu, 2 Apr 2026 09:56:40 +0100 Subject: [PATCH] ax(mining): replace prose comments with usage examples in settings_manager Three comments restated the function/type name in prose (AX principle 2 violation). Replaced with concrete call-site examples showing real usage. Co-Authored-By: Charon --- pkg/mining/settings_manager.go | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/pkg/mining/settings_manager.go b/pkg/mining/settings_manager.go index 98c38d4..fc9fd64 100644 --- a/pkg/mining/settings_manager.go +++ b/pkg/mining/settings_manager.go @@ -58,7 +58,8 @@ type AppSettings struct { Theme string `json:"theme"` // "light", "dark", "system" } -// DefaultSettings returns sensible defaults for app settings +// settings := DefaultSettings() +// settings.Theme = "dark" func DefaultSettings() *AppSettings { return &AppSettings{ Window: WindowState{ @@ -85,14 +86,16 @@ func DefaultSettings() *AppSettings { } } -// SettingsManager handles loading and saving app settings +// sm, err := NewSettingsManager() +// settings := sm.Get() type SettingsManager struct { mu sync.RWMutex settings *AppSettings settingsPath string } -// NewSettingsManager creates a new settings manager +// sm, err := NewSettingsManager() +// if err != nil { return err } func NewSettingsManager() (*SettingsManager, error) { settingsPath, err := xdg.ConfigFile(filepath.Join("lethean-desktop", settingsFileName)) if err != nil {