feat: allow all three --from/--to/--duration flags in edit when consistent

Previously, specifying --duration, --from, and --to together was rejected
outright. Now all three are accepted as long as they are consistent
(to - from = duration), giving users explicit control without ambiguity.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: Flyrell

README.md

@@ -184,7 +184,7 @@ hourgit edit <hash> [--duration <dur>] [--from <time>] [--to <time>] [--date <da
 | `-m`, `--message` | — | New message |
 | `-y`, `--yes` | `false` | Skip confirmation prompt |
 
-> Any two of `--duration`, `--from`, and `--to` can be combined (the third is computed), but all three cannot be specified together. `--from` only: keeps duration, shifts the time window. `--to` only: keeps start, recalculates duration. `--duration` only: keeps start, shifts end. Entry ID and creation timestamp are preserved. If the entry is not found in the current repo's project, all projects are searched.
+> Any combination of `--duration`, `--from`, and `--to` is allowed. With two flags, the third is computed. With all three, they must be consistent (to - from = duration). `--from` only: keeps duration, shifts the time window. `--to` only: keeps start, recalculates duration. `--duration` only: keeps start, shifts end. Entry ID and creation timestamp are preserved. If the entry is not found in the current repo's project, all projects are searched.
 
 **Examples**
 

internal/cli/edit.go

@@ -212,11 +212,6 @@ func applyFlagEdits(
 	hasTo := flagsChanged["to"]
 	hasDate := flagsChanged["date"]
 
-	// Over-specified: all three time flags
-	if hasDuration && hasFrom && hasTo {
-		return e, fmt.Errorf("--duration, --from, and --to cannot all be specified together")
-	}
-
 	// Handle date shift
 	if hasDate {
 		newDate, err := resolveBaseDate(dateFlag, nowFn())
@@ -230,6 +225,34 @@ func applyFlagEdits(
 
 	// Handle time changes — interdependent from/to/duration
 	switch {
+	case hasDuration && hasFrom && hasTo:
+		// All three specified — validate consistency: to - from must equal duration
+		fromTOD, err := schedule.ParseTimeOfDay(fromFlag)
+		if err != nil {
+			return e, fmt.Errorf("invalid --from time: %w", err)
+		}
+		toTOD, err := schedule.ParseTimeOfDay(toFlag)
+		if err != nil {
+			return e, fmt.Errorf("invalid --to time: %w", err)
+		}
+		minutes, err := entry.ParseDuration(durationFlag)
+		if err != nil {
+			return e, err
+		}
+		fromMins := fromTOD.Hour*60 + fromTOD.Minute
+		toMins := toTOD.Hour*60 + toTOD.Minute
+		if toMins <= fromMins {
+			return e, fmt.Errorf("--to (%s) must be after --from (%s)", toFlag, fromFlag)
+		}
+		computed := toMins - fromMins
+		if computed != minutes {
+			return e, fmt.Errorf("--duration (%s) does not match --from/%s to --to/%s (%s)",
+				durationFlag, fromFlag, toFlag, entry.FormatMinutes(computed))
+		}
+		y, m, d := e.Start.Date()
+		e.Start = time.Date(y, m, d, fromTOD.Hour, fromTOD.Minute, 0, 0, e.Start.Location())
+		e.Minutes = minutes
+
 	case hasDuration && hasFrom:
 		// --duration --from → set from, set minutes (to = from + duration)
 		fromTOD, err := schedule.ParseTimeOfDay(fromFlag)

internal/cli/edit_test.go

@@ -243,14 +243,31 @@ func TestEditDurationAndTo(t *testing.T) {
 	assert.Equal(t, 120, e.Minutes)
 }
 
-func TestEditDurationFromToOverSpecified(t *testing.T) {
+func TestEditAllThreeConsistent(t *testing.T) {
+	homeDir, repoDir, proj, _ := setupEditTest(t)
+
+	// --from 10am --to 12pm --duration 2h → consistent, should succeed
+	flags := map[string]bool{"duration": true, "from": true, "to": true}
+	stdout, err := execEdit(homeDir, repoDir, "", "ed01234", flags, "2h", "10am", "12pm", "", "", "")
+
+	require.NoError(t, err)
+	assert.Contains(t, stdout, "updated entry")
+
+	e, err := entry.ReadEntry(homeDir, proj.Slug, "ed01234")
+	require.NoError(t, err)
+	assert.Equal(t, 10, e.Start.Hour())
+	assert.Equal(t, 120, e.Minutes)
+}
+
+func TestEditAllThreeInconsistent(t *testing.T) {
 	homeDir, repoDir, _, _ := setupEditTest(t)
 
+	// --from 10am --to 12pm --duration 3h → inconsistent (2h ≠ 3h)
 	flags := map[string]bool{"duration": true, "from": true, "to": true}
-	_, err := execEdit(homeDir, repoDir, "", "ed01234", flags, "2h", "9am", "11am", "", "", "")
+	_, err := execEdit(homeDir, repoDir, "", "ed01234", flags, "3h", "10am", "12pm", "", "", "")
 
 	assert.Error(t, err)
-	assert.Contains(t, err.Error(), "cannot all be specified")
+	assert.Contains(t, err.Error(), "does not match")
 }
 
 func TestEditToBeforeStart(t *testing.T) {

web/docs/commands/time-tracking.md

@@ -73,7 +73,7 @@ hourgit edit <hash> [--duration <dur>] [--from <time>] [--to <time>] [--date <da
 | `-m`, `--message` | — | New message |
 | `-y`, `--yes` | `false` | Skip confirmation prompt |
 
-> Any two of `--duration`, `--from`, and `--to` can be combined (the third is computed), but all three cannot be specified together. `--from` only: keeps duration, shifts the time window. `--to` only: keeps start, recalculates duration. `--duration` only: keeps start, shifts end. Entry ID and creation timestamp are preserved.
+> Any combination of `--duration`, `--from`, and `--to` is allowed. With two flags, the third is computed. With all three, they must be consistent (to - from = duration). `--from` only: keeps duration, shifts the time window. `--to` only: keeps start, recalculates duration. `--duration` only: keeps start, shifts end. Entry ID and creation timestamp are preserved.
 
 **Examples:**