Add --break option to finish subcommand (#36)

## Summary
• Added a `--break` flag to the `finish` command that automatically
starts a break after finishing a pomodoro
• The break duration can be specified in minutes (e.g., `--break 5` or
`--break 5m`)
• Implemented in two separate commits: first working implementation,
then refactored to eliminate code duplication

## Test plan
- [x] Build successfully compiles
- [x] `pomodoro finish --help` shows the new `--break` flag
- [ ] Test `pomodoro finish --break 5` starts a 5-minute break after
finishing a pomodoro
- [ ] Test default break duration works when just using `--break`

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

Author: justincampbell

cmd/break.go

@@ -1,12 +1,6 @@
 package cmd
 
 import (
-	"fmt"
-	"strconv"
-	"time"
-
-	"github.com/justincampbell/go-countdown"
-	"github.com/justincampbell/go-countdown/format"
 	"github.com/open-pomodoro/openpomodoro-cli/hook"
 	"github.com/spf13/cobra"
 )
@@ -42,21 +36,3 @@ func breakCmd(cmd *cobra.Command, args []string) error {
 
 	return hook.Run(client, "stop")
 }
-
-func wait(d time.Duration) error {
-	err := countdown.For(d, time.Second).Do(func(c *countdown.Countdown) error {
-		fmt.Printf("\r%s", format.MinSec(c.Remaining()))
-		return nil
-	})
-
-	fmt.Println()
-
-	return err
-}
-
-func parseDurationMinutes(s string) (time.Duration, error) {
-	if _, err := strconv.Atoi(s); err == nil {
-		s = fmt.Sprintf("%sm", s)
-	}
-	return time.ParseDuration(s)
-}

cmd/finish.go

@@ -2,20 +2,26 @@ package cmd
 
 import (
 	"fmt"
+	"strings"
 	"time"
 
 	"github.com/open-pomodoro/openpomodoro-cli/format"
 	"github.com/open-pomodoro/openpomodoro-cli/hook"
 	"github.com/spf13/cobra"
 )
 
+var breakFlag string
+
 func init() {
 	command := &cobra.Command{
 		Use:   "finish",
 		Short: "Finish the current Pomodoro",
 		RunE:  finishCmd,
 	}
 
+	command.Flags().StringVarP(&breakFlag, "break", "b", "", "take a break after finishing (duration in minutes)")
+	command.Flags().Lookup("break").NoOptDefVal = " "
+
 	RootCmd.AddCommand(command)
 }
 
@@ -32,5 +38,32 @@ func finishCmd(cmd *cobra.Command, args []string) error {
 		return err
 	}
 
-	return client.Finish()
+	if err := client.Finish(); err != nil {
+		return err
+	}
+
+	if cmd.Flags().Changed("break") {
+		breakDuration := settings.DefaultBreakDuration
+
+		trimmedFlag := strings.TrimSpace(breakFlag)
+		if trimmedFlag != "" {
+			var err error
+			breakDuration, err = parseDurationMinutes(trimmedFlag)
+			if err != nil {
+				return err
+			}
+		}
+
+		if err := hook.Run(client, "break"); err != nil {
+			return err
+		}
+
+		if err := wait(breakDuration); err != nil {
+			return err
+		}
+
+		return hook.Run(client, "stop")
+	}
+
+	return nil
 }

cmd/util.go

@@ -0,0 +1,30 @@
+package cmd
+
+import (
+	"fmt"
+	"strconv"
+	"time"
+
+	"github.com/justincampbell/go-countdown"
+	"github.com/justincampbell/go-countdown/format"
+)
+
+// wait displays a countdown timer for the specified duration
+func wait(d time.Duration) error {
+	err := countdown.For(d, time.Second).Do(func(c *countdown.Countdown) error {
+		fmt.Printf("\r%s", format.MinSec(c.Remaining()))
+		return nil
+	})
+
+	fmt.Println()
+
+	return err
+}
+
+// parseDurationMinutes parses a duration string, defaulting to minutes if no unit is specified
+func parseDurationMinutes(s string) (time.Duration, error) {
+	if _, err := strconv.Atoi(s); err == nil {
+		s = fmt.Sprintf("%sm", s)
+	}
+	return time.ParseDuration(s)
+}

cmd/util_test.go

@@ -0,0 +1,66 @@
+package cmd
+
+import (
+	"testing"
+	"time"
+
+	"github.com/stretchr/testify/assert"
+	"github.com/stretchr/testify/require"
+)
+
+func TestParseDurationMinutes(t *testing.T) {
+	tests := []struct {
+		name     string
+		input    string
+		expected time.Duration
+		wantErr  bool
+	}{
+		{
+			name:     "integer defaults to minutes",
+			input:    "5",
+			expected: 5 * time.Minute,
+		},
+		{
+			name:     "explicit minutes",
+			input:    "10m",
+			expected: 10 * time.Minute,
+		},
+		{
+			name:     "seconds",
+			input:    "30s",
+			expected: 30 * time.Second,
+		},
+		{
+			name:     "hours",
+			input:    "1h",
+			expected: 1 * time.Hour,
+		},
+		{
+			name:     "complex duration",
+			input:    "1h30m",
+			expected: 90 * time.Minute,
+		},
+		{
+			name:    "invalid duration",
+			input:   "invalid",
+			wantErr: true,
+		},
+		{
+			name:     "zero",
+			input:    "0",
+			expected: 0,
+		},
+	}
+
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			got, err := parseDurationMinutes(tt.input)
+			if tt.wantErr {
+				require.Error(t, err)
+			} else {
+				require.NoError(t, err)
+				assert.Equal(t, tt.expected, got)
+			}
+		})
+	}
+}

test/finish.bats

@@ -43,3 +43,19 @@ load test_helper
     [ "$status" -eq 0 ]
     assert_file_empty "current"
 }
+
+@test "finish --break flag is accepted" {
+    create_hook "break" 'exit 1'
+
+    pomodoro start "Task" --ago 1m
+    run pomodoro finish --break
+    [ "$status" -ne 0 ]
+}
+
+@test "finish --break with custom duration is accepted" {
+    create_hook "break" 'exit 1'
+
+    pomodoro start "Task" --ago 1m
+    run pomodoro finish --break 5
+    [ "$status" -ne 0 ]
+}