Move timer room logic to WebTimer constructor and enrich config with branch qualifier

Refactors getMobTimerRoom() into two responsibilities:
1. enrichConfigurationWithBranchQualifier() in mob.go extracts the qualifier from the current WIP branch and writes it to the configuration
2. WebTimer constructor determines the effective timer room based on TimerRoomUseWipBranchQualifier setting

This fixes the bug where the extracted qualifier was not written to the configuration, making it unavailable for other commands.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: hollesse

mob.go

@@ -435,6 +435,24 @@ func determineBranches(currentBranch Branch, localBranches []string, configurati
 	return
 }
 
+func enrichConfigurationWithBranchQualifier(configuration config.Configuration) config.Configuration {
+	if !isGit() {
+		return configuration
+	}
+
+	if configuration.WipBranchQualifier == "" {
+		currentBranch := gitCurrentBranch()
+		currentBaseBranch, _ := determineBranches(currentBranch, gitBranches(), configuration)
+
+		if currentBranch.IsWipBranch(configuration) {
+			wipBranchWithoutWipPrefix := currentBranch.removeWipPrefix(configuration).Name
+			configuration.WipBranchQualifier = removePrefix(removePrefix(wipBranchWithoutWipPrefix, currentBaseBranch.Name), configuration.WipBranchQualifierSeparator)
+		}
+	}
+
+	return configuration
+}
+
 func injectCommandWithMessage(command string, message string) string {
 	placeHolders := strings.Count(command, "%s")
 	if placeHolders > 1 {

timer.go

@@ -15,7 +15,7 @@ func StartTimer(timerInMinutes string, configuration config.Configuration) {
 }
 
 func startTimer(timerInMinutes string, configuration config.Configuration) error {
-	configuration.TimerRoom = getMobTimerRoom(configuration)
+	configuration = enrichConfigurationWithBranchQualifier(configuration)
 	return timerpkg.RunTimer(timerInMinutes, configuration)
 }
 
@@ -26,29 +26,6 @@ func StartBreakTimer(timerInMinutes string, configuration config.Configuration)
 }
 
 func startBreakTimer(timerInMinutes string, configuration config.Configuration) error {
-	configuration.TimerRoom = getMobTimerRoom(configuration)
+	configuration = enrichConfigurationWithBranchQualifier(configuration)
 	return timerpkg.RunBreakTimer(timerInMinutes, configuration)
 }
-
-func getMobTimerRoom(configuration config.Configuration) string {
-	if !isGit() {
-		return configuration.TimerRoom
-	}
-
-	currentWipBranchQualifier := configuration.WipBranchQualifier
-	if currentWipBranchQualifier == "" {
-		currentBranch := gitCurrentBranch()
-		currentBaseBranch, _ := determineBranches(currentBranch, gitBranches(), configuration)
-
-		if currentBranch.IsWipBranch(configuration) {
-			wipBranchWithoutWipPrefix := currentBranch.removeWipPrefix(configuration).Name
-			currentWipBranchQualifier = removePrefix(removePrefix(wipBranchWithoutWipPrefix, currentBaseBranch.Name), configuration.WipBranchQualifierSeparator)
-		}
-	}
-
-	if configuration.TimerRoomUseWipBranchQualifier && currentWipBranchQualifier != "" {
-		return currentWipBranchQualifier
-	}
-
-	return configuration.TimerRoom
-}

timer/webtimer/webtimer.go

@@ -28,8 +28,14 @@ type WebTimer struct {
 }
 
 func NewWebTimer(configuration config.Configuration) WebTimer {
+	// Determine the effective timer room
+	room := configuration.TimerRoom
+	if configuration.TimerRoomUseWipBranchQualifier && configuration.WipBranchQualifier != "" {
+		room = configuration.WipBranchQualifier
+	}
+
 	return WebTimer{
-		room:          configuration.TimerRoom,
+		room:          room,
 		timerUser:     getUserForMobTimer(configuration.TimerUser),
 		timerUrl:      configuration.TimerUrl,
 		timerInsecure: configuration.TimerInsecure,