Extract timer implementations into self-registering sub-packages

- timer/localtimer/ and timer/webtimer/ are now independent packages
  that register themselves via init() using timer.Register(Factory)
- timer package no longer imports its implementations; dependency arrows
  are reversed: implementations depend on the timer package, not vice versa
- timer.go (main) blank-imports both packages to trigger registration
- mob.go uses localtimer.ExecuteCommandsInBackgroundProcess/VoiceCommand
  directly for the moo feature

https://claude.ai/code/session_01DTB35xCmhRgW5SQUsYAtAq

Author: claude

mob.go

@@ -21,7 +21,7 @@ import (
 	"github.com/remotemobprogramming/mob/v5/help"
 	"github.com/remotemobprogramming/mob/v5/open"
 	"github.com/remotemobprogramming/mob/v5/say"
-	timerpkg "github.com/remotemobprogramming/mob/v5/timer"
+	"github.com/remotemobprogramming/mob/v5/timer/localtimer"
 	"github.com/remotemobprogramming/mob/v5/workdir"
 )
 
@@ -453,7 +453,7 @@ func currentTime() string {
 
 func moo(configuration config.Configuration) {
 	voiceMessage := "moo"
-	err := timerpkg.ExecuteCommandsInBackgroundProcess(timerpkg.VoiceCommand(voiceMessage, configuration.VoiceCommand))
+	err := localtimer.ExecuteCommandsInBackgroundProcess(localtimer.VoiceCommand(voiceMessage, configuration.VoiceCommand))
 
 	if err != nil {
 		say.Warning(fmt.Sprintf("can't run voice command on your system (%s)", runtime.GOOS))

timer.go

@@ -4,6 +4,8 @@ import (
 	config "github.com/remotemobprogramming/mob/v5/configuration"
 	"github.com/remotemobprogramming/mob/v5/exit"
 	timerpkg "github.com/remotemobprogramming/mob/v5/timer"
+	_ "github.com/remotemobprogramming/mob/v5/timer/localtimer"
+	_ "github.com/remotemobprogramming/mob/v5/timer/webtimer"
 )
 
 func StartTimer(timerInMinutes string, configuration config.Configuration) {

timer/localtimer.go timer/localtimer/localtimer.gotimer/localtimer.go renamed to timer/localtimer/localtimer.go

@@ -1,4 +1,4 @@
-package timer
+package localtimer
 
 import (
 	"fmt"
@@ -9,9 +9,16 @@ import (
 	config "github.com/remotemobprogramming/mob/v5/configuration"
 	"github.com/remotemobprogramming/mob/v5/exit"
 	"github.com/remotemobprogramming/mob/v5/say"
+	"github.com/remotemobprogramming/mob/v5/timer"
 	"github.com/remotemobprogramming/mob/v5/workdir"
 )
 
+func init() {
+	timer.Register(func(configuration config.Configuration) timer.Timer {
+		return NewProcessLocalTimer(configuration)
+	})
+}
+
 // ProcessLocalTimer is a Timer implementation that uses background OS processes.
 type ProcessLocalTimer struct {
 	configuration config.Configuration

timer/timer.go

@@ -18,14 +18,22 @@ type Timer interface {
 	StartBreakTimer(minutes int) error
 }
 
-// GetTimers returns all timers that report themselves as active.
+// Factory creates a Timer for the given configuration.
+type Factory func(configuration config.Configuration) Timer
+
+var factories []Factory
+
+// Register adds a Timer factory to the registry.
+// Implementation packages call this in their init() function.
+func Register(f Factory) {
+	factories = append(factories, f)
+}
+
+// GetTimers returns all registered timers that report themselves as active.
 func GetTimers(configuration config.Configuration) []Timer {
-	all := []Timer{
-		NewWebTimer(configuration),
-		NewProcessLocalTimer(configuration),
-	}
 	var active []Timer
-	for _, t := range all {
+	for _, f := range factories {
+		t := f(configuration)
 		if t.IsActive() {
 			active = append(active, t)
 		}

timer/webtimer.go timer/webtimer/webtimer.gotimer/webtimer.go renamed to timer/webtimer/webtimer.go

@@ -1,19 +1,26 @@
-package timer
+package webtimer
 
 import (
 	"encoding/json"
 	"fmt"
 
 	config "github.com/remotemobprogramming/mob/v5/configuration"
 	"github.com/remotemobprogramming/mob/v5/httpclient"
+	"github.com/remotemobprogramming/mob/v5/timer"
 )
 
+func init() {
+	timer.Register(func(configuration config.Configuration) timer.Timer {
+		return NewWebTimer(configuration)
+	})
+}
+
 // WebTimer is a Timer implementation that notifies a remote timer service via HTTP.
 type WebTimer struct {
-	room           string
-	timerUser      string
-	timerUrl       string
-	timerInsecure  bool
+	room          string
+	timerUser     string
+	timerUrl      string
+	timerInsecure bool
 }
 
 func NewWebTimer(configuration config.Configuration) WebTimer {