fix: patch undo/redo to accurately work for editing tasks.

- closes #50

Author: programmersd21

CHANGELOG.md

@@ -5,6 +5,13 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.6.3] (2026-05-10)
+
+### Fixed
+
+* Fixed undo/redo for task field edits
+* Undo/redo now correctly restores edited task fields including title, due date, tags, parent, project, and description
+
 ## [1.6.2] (2026-05-09)
 
 ### Fixed

VERSION.txt

@@ -1 +1 @@
-1.6.2
+1.6.3

internal/app/model.go

@@ -470,6 +470,11 @@ func (m *Model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 		if x.Refresh {
 			m.rebuildComponentSizes()
 			cmds = append(cmds, m.refreshCmd())
+
+			// If in detail view, also reload the current task to reflect changes (undo/redo)
+			if m.mode == ModeDetail {
+				cmds = append(cmds, m.fetchOpenTaskCmd(m.det.Task().ID))
+			}
 		}
 		return m, tea.Batch(cmds...)
 
@@ -2399,7 +2404,7 @@ func (m *Model) createTaskCmd(t core.Task) tea.Cmd {
 		if err != nil {
 			return errMsg{Err: err}
 		}
-		m.hist.Record(history.CreateOperation(history.OpCreate, "", []string{created.ID}, nil, []core.Task{created}))
+		m.hist.Record(history.CreateOperation(history.OpCreate, "", []string{created.ID}, nil, []core.Task{created.DeepCopy()}))
 		return taskCreatedMsg{Task: created}
 	}
 }
@@ -2419,7 +2424,7 @@ func (m *Model) updateTaskCmd(id string, p core.TaskPatch) tea.Cmd {
 		if p.Status != nil {
 			opType = history.OpToggleStatus
 		}
-		m.hist.Record(history.CreateOperation(opType, "", []string{id}, []core.Task{before}, []core.Task{updated}))
+		m.hist.Record(history.CreateOperation(opType, "", []string{id}, []core.Task{before.DeepCopy()}, []core.Task{updated.DeepCopy()}))
 		return taskUpdatedMsg{Task: updated}
 	}
 }
@@ -2442,7 +2447,7 @@ func (m *Model) deleteTaskCmd(id string) tea.Cmd {
 		if err := m.svc.Delete(m.ctx, id); err != nil {
 			return errMsg{Err: err}
 		}
-		m.hist.Record(history.CreateOperation(history.OpDelete, "", []string{id}, []core.Task{before}, nil))
+		m.hist.Record(history.CreateOperation(history.OpDelete, "", []string{id}, []core.Task{before.DeepCopy()}, nil))
 		return taskDeletedMsg{ID: id}
 	}
 }
@@ -2451,13 +2456,15 @@ func (m *Model) deleteAllTasksCmd() tea.Cmd {
 	return func() tea.Msg {
 		before, _ := m.svc.ListAll(m.ctx)
 		var ids []string
+		var beforeCopy []core.Task
 		for _, t := range before {
 			ids = append(ids, t.ID)
+			beforeCopy = append(beforeCopy, t.DeepCopy())
 		}
 		if err := m.svc.DeleteAll(m.ctx); err != nil {
 			return errMsg{Err: err}
 		}
-		m.hist.Record(history.CreateOperation(history.OpBulkDelete, "Delete All", ids, before, nil))
+		m.hist.Record(history.CreateOperation(history.OpBulkDelete, "Delete All", ids, beforeCopy, nil))
 		return taskUpdatedMsg{} // Trigger reload
 	}
 }
@@ -2522,6 +2529,7 @@ func (m *Model) applyOperation(op *history.Operation, undo bool) error {
 			tasks = op.Before
 		}
 		for _, t := range tasks {
+			t.UpdatedAt = time.Now()
 			if err := m.svc.UpsertTask(m.ctx, t); err != nil {
 				return err
 			}

internal/core/patch_test.go

@@ -0,0 +1,43 @@
+package core
+
+import (
+	"reflect"
+	"testing"
+)
+
+func TestTaskPatch_ApplyTo(t *testing.T) {
+	task := Task{
+		ID:          "1",
+		Title:       "Original",
+		Description: "Desc",
+		Project:     "Proj",
+		Tags:        []string{"a"},
+		Priority:    P1,
+		Status:      StatusTodo,
+	}
+
+	newTitle := "Updated"
+	newTags := []string{"b", "c"}
+	newStatus := StatusDone
+	patch := TaskPatch{
+		Title:  &newTitle,
+		Tags:   &newTags,
+		Status: &newStatus,
+	}
+
+	updated := patch.ApplyTo(task)
+
+	if updated.Title != "Updated" {
+		t.Errorf("expected title Updated, got %s", updated.Title)
+	}
+	if !reflect.DeepEqual(updated.Tags, []string{"b", "c"}) {
+		t.Errorf("expected tags [b c], got %v", updated.Tags)
+	}
+	if updated.Status != StatusDone {
+		t.Errorf("expected status Done, got %s", updated.Status)
+	}
+	// Verify immutability of original
+	if task.Title != "Original" {
+		t.Error("original task title was mutated")
+	}
+}

internal/core/task.go

@@ -85,6 +85,33 @@ type Task struct {
 
 // ... update MarshalJSON/UnmarshalJSON accordingly ...
 
+func (t Task) DeepCopy() Task {
+	clone := t
+	if t.Deadline != nil {
+		d := *t.Deadline
+		clone.Deadline = &d
+	}
+	if t.WaitUntil != nil {
+		w := *t.WaitUntil
+		clone.WaitUntil = &w
+	}
+	if t.Until != nil {
+		u := *t.Until
+		clone.Until = &u
+	}
+	if t.CompletedAt != nil {
+		c := *t.CompletedAt
+		clone.CompletedAt = &c
+	}
+	if t.Tags != nil {
+		clone.Tags = append([]string(nil), t.Tags...)
+	}
+	if t.RecurrenceWeekly != nil {
+		clone.RecurrenceWeekly = append([]string(nil), t.RecurrenceWeekly...)
+	}
+	return clone
+}
+
 func (t Task) NormalizedTags() []string {
 	m := make(map[string]struct{}, len(t.Tags))
 	for _, tag := range t.Tags {

internal/lua/engine.go

@@ -61,7 +61,7 @@ func (e *Engine) SetupKairoAPI(L *lua.LState) {
 	L.SetField(kairo, "notify", L.NewFunction(e.luaNotify))
 
 	// Meta
-	L.SetField(kairo, "version", lua.LString("1.6.2"))
+	L.SetField(kairo, "version", lua.LString("1.6.3"))
 
 	// Set as global
 	L.SetGlobal("kairo", kairo)