a permanently deleted file's steps do not keep a run partly undone

1 files changed, 31 insertions, 0 deletions

diff --git a/internal/journal/read_test.go b/internal/journal/read_test.go
index efb5a15..4f1c902 100644
--- a/internal/journal/read_test.go
+++ b/internal/journal/read_test.go
@@ -779,3 +779,34 @@ func TestRunsMarksAPartlyUndoneRun(t *testing.T) {
 		t.Errorf("after both reversals: %+v, want undone in full", a)
 	}
 }
+
+// TestRunsIgnoresAPermanentlyDeletedFilesSteps: a file whose chain ended in
+// a permanent delete can never be undone, so its earlier steps do not keep
+// the run partly undone forever (plan 11 review L8).
+func TestRunsIgnoresAPermanentlyDeletedFilesSteps(t *testing.T) {
+	path := filepath.Join(t.TempDir(), "krino.log")
+	w, _ := Open(path)
+	t0 := time.Date(2026, 9, 11, 9, 0, 0, 0, time.UTC)
+	for _, e := range []Entry{
+		{Time: t0, Run: "A", Action: "run-start", Status: "ok"},
+		{Time: t0, Run: "A", Dir: "dl", File: "x.pdf", Step: 1, Action: "move", Status: "ok", Src: "/a/x.pdf", Dst: "/b/x.pdf"},
+		{Time: t0, Run: "A", Dir: "dl", File: "y.pdf", Step: 1, Action: "rename", Status: "ok", Src: "/a/y.pdf", Dst: "/a/z.pdf"},
+		{Time: t0, Run: "A", Dir: "dl", File: "y.pdf", Step: 2, Action: "delete", Status: "ok", Src: "/a/z.pdf"},
+		{Time: t0, Run: "A", Action: "run-end", Status: "ok"},
+		{Time: t0.Add(time.Hour), Run: "B", Action: "run-start", Status: "ok", Detail: "undo of A"},
+		{Time: t0.Add(time.Hour), Run: "B", Dir: "dl", File: "x.pdf", Step: 1, Action: "undo-move", Status: "ok", Src: "/b/x.pdf", Dst: "/a/x.pdf"},
+		{Time: t0.Add(time.Hour), Run: "B", Action: "run-end", Status: "ok"},
+	} {
+		w.Append(e)
+	}
+	w.Close()
+	runs, err := Runs(path, 0)
+	if err != nil {
+		t.Fatal(err)
+	}
+	for _, r := range runs {
+		if r.ID == "A" && (!r.Undone || r.PartlyUndone) {
+			t.Errorf("run A = %+v; want undone in full: y.pdf could never be reversed", r)
+		}
+	}
+}