tui: extract timer work integration helper

3 files changed, 49 insertions, 38 deletions

diff --git a/internal/tui/timer.go b/internal/tui/timer.go
index 2c64ef7..d02a890 100644
--- a/internal/tui/timer.go
+++ b/internal/tui/timer.go
@@ -34,11 +34,15 @@ type TimerModel struct {
 	font        string
 	fontIndex   int
 
-	workEnabled  bool
-	workDBDir    string
-	workHost     string
-	workLoggedIn bool
-	workStatus   string
+	work workIntegration
+}
+
+type workIntegration struct {
+	enabled  bool
+	dbDir    string
+	host     string
+	loggedIn bool
+	status   string
 }
 
 // NewTimerModel builds the timer screen model.
@@ -68,16 +72,21 @@ func NewTimerModel(font string, cfg config.Config) (TimerModel, error) {
 		statusStyle: lipgloss.NewStyle().Italic(true),
 		font:        selectedFont,
 		fontIndex:   fontIndex,
-		workStatus:  "work integration disabled",
+		work: workIntegration{
+			status: "work integration disabled",
+		},
 	}
 
 	if strings.TrimSpace(cfg.WorktimeDBDir) != "" {
 		host, err := cfg.EffectiveHostname()
 		if err == nil {
-			model.workEnabled = true
-			model.workDBDir = cfg.WorktimeDBDir
-			model.workHost = host
-			model.refreshWorkStatus()
+			model.work = workIntegration{
+				enabled: true,
+				dbDir:   cfg.WorktimeDBDir,
+				host:    host,
+				status:  "work integration enabled",
+			}
+			model.work.refreshStatus()
 		}
 	}
 
@@ -127,19 +136,19 @@ func (m TimerModel) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 				m.state.Running = true
 				m.state.StartTime = time.Now()
 				if err := m.state.Save(); err != nil {
-					m.workStatus = "save error: " + err.Error()
+					m.work.status = "save error: " + err.Error()
 				}
 				return m, timerTick()
 			}
 			if err := m.state.Save(); err != nil {
-				m.workStatus = "save error: " + err.Error()
+				m.work.status = "save error: " + err.Error()
 			}
 			return m, nil
 
 		case "r":
 			m.state = timrTimer.State{}
 			if _, err := timrTimer.ResetTimer(); err != nil {
-				m.workStatus = "reset error: " + err.Error()
+				m.work.status = "reset error: " + err.Error()
 			}
 			return m, nil
 
@@ -149,7 +158,7 @@ func (m TimerModel) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 			return m, nil
 
 		case "l":
-			m.toggleWorkLogin()
+			m.work.toggle()
 			return m, nil
 		}
 	}
@@ -181,7 +190,7 @@ func (m TimerModel) View() string {
 		m.statusStyle.Render(status),
 		"",
 		m.helpStyle.Render(fmt.Sprintf("Font: %s", m.font)),
-		m.helpStyle.Render(fmt.Sprintf("Work: %s", m.workStatus)),
+		m.helpStyle.Render(fmt.Sprintf("Work: %s", m.work.status)),
 		m.helpStyle.Render("s/Space: start-stop, r: reset, f: change font, l: work login/logout, q: quit"),
 	}
 
@@ -209,44 +218,44 @@ func (m TimerModel) renderTimer(elapsed time.Duration) string {
 	return m.timerStyle.Render(asciiTime)
 }
 
-func (m *TimerModel) toggleWorkLogin() {
-	if !m.workEnabled {
-		m.workStatus = "work integration disabled"
+func (w *workIntegration) toggle() {
+	if !w.enabled {
+		w.status = "work integration disabled"
 		return
 	}
 
 	now := time.Now()
 	var err error
-	if m.workLoggedIn {
-		_, err = worktime.Logout(m.workDBDir, m.workHost, "work", now, "tui timer toggle")
+	if w.loggedIn {
+		_, err = worktime.Logout(w.dbDir, w.host, "work", now, "tui timer toggle")
 	} else {
-		_, err = worktime.Login(m.workDBDir, m.workHost, "work", now, "tui timer toggle")
+		_, err = worktime.Login(w.dbDir, w.host, "work", now, "tui timer toggle")
 	}
 	if err != nil {
-		m.workStatus = "work toggle error: " + err.Error()
+		w.status = "work toggle error: " + err.Error()
 		return
 	}
 
-	m.refreshWorkStatus()
+	w.refreshStatus()
 }
 
-func (m *TimerModel) refreshWorkStatus() {
-	if !m.workEnabled {
-		m.workStatus = "work integration disabled"
+func (w *workIntegration) refreshStatus() {
+	if !w.enabled {
+		w.status = "work integration disabled"
 		return
 	}
 
-	entries, err := worktime.LoadAll(m.workDBDir)
+	entries, err := worktime.LoadAll(w.dbDir)
 	if err != nil {
-		m.workStatus = "work status error: " + err.Error()
+		w.status = "work status error: " + err.Error()
 		return
 	}
 
-	m.workLoggedIn = workCategoryLoggedIn(entries)
-	if m.workLoggedIn {
-		m.workStatus = "logged in"
+	w.loggedIn = workCategoryLoggedIn(entries)
+	if w.loggedIn {
+		w.status = "logged in"
 	} else {
-		m.workStatus = "logged out"
+		w.status = "logged out"
 	}
 }
 
diff --git a/internal/tui/timer_test.go b/internal/tui/timer_test.go
index 6263c09..3eb39b2 100644
--- a/internal/tui/timer_test.go
+++ b/internal/tui/timer_test.go
@@ -21,19 +21,19 @@ func TestTimerModelToggleWorkLogin(t *testing.T) {
 		t.Fatalf("NewTimerModel() error = %v", err)
 	}
 
-	if !model.workEnabled {
+	if !model.work.enabled {
 		t.Fatal("work integration should be enabled")
 	}
 
 	modelAny, _ := model.Update(tea.KeyMsg{Type: tea.KeyRunes, Runes: []rune{'l'}})
 	model = modelAny.(TimerModel)
-	if !model.workLoggedIn {
+	if !model.work.loggedIn {
 		t.Fatal("work should be logged in after first l")
 	}
 
 	modelAny, _ = model.Update(tea.KeyMsg{Type: tea.KeyRunes, Runes: []rune{'l'}})
 	model = modelAny.(TimerModel)
-	if model.workLoggedIn {
+	if model.work.loggedIn {
 		t.Fatal("work should be logged out after second l")
 	}
 
@@ -75,8 +75,8 @@ func TestTimerModelWorkToggleWhenDisabled(t *testing.T) {
 
 	modelAny, _ := model.Update(tea.KeyMsg{Type: tea.KeyRunes, Runes: []rune{'l'}})
 	model = modelAny.(TimerModel)
-	if model.workStatus != "work integration disabled" {
-		t.Fatalf("workStatus = %q, want work integration disabled", model.workStatus)
+	if model.work.status != "work integration disabled" {
+		t.Fatalf("workStatus = %q, want work integration disabled", model.work.status)
 	}
 }
 
diff --git a/internal/tui/tui.go b/internal/tui/tui.go
index 3c2e6e9..4a3c961 100644
--- a/internal/tui/tui.go
+++ b/internal/tui/tui.go
@@ -216,7 +216,9 @@ func newFallbackTimerModel(status string) TimerModel {
 		timerStyle:  lipgloss.NewStyle().Bold(true).Foreground(lipgloss.Color("#00BFFF")),
 		statusStyle: lipgloss.NewStyle().Italic(true),
 		font:        "doom",
-		workStatus:  status,
+		work: workIntegration{
+			status: status,
+		},
 	}
 }