fix: tolerate legacy run history without targets

Author: czyt

internal/history/history.go

@@ -81,6 +81,9 @@ func (s *Store) List() ([]Run, error) {
 		if err := json.Unmarshal([]byte(row.Payload), &run); err != nil {
 			return nil, err
 		}
+		if run.Targets == nil {
+			run.Targets = []TargetResult{}
+		}
 		runs = append(runs, run)
 	}
 	return runs, nil

internal/history/history_test.go

@@ -0,0 +1,46 @@
+package history
+
+import (
+	"context"
+	"testing"
+	"time"
+
+	"github.com/ca-x/nowledge-mem-snap/internal/persist"
+)
+
+func TestListNormalizesLegacyNullTargets(t *testing.T) {
+	client, err := persist.OpenClient(t.TempDir())
+	if err != nil {
+		t.Fatalf("OpenClient: %v", err)
+	}
+	t.Cleanup(func() {
+		if err := client.Close(); err != nil {
+			t.Fatalf("close client: %v", err)
+		}
+	})
+
+	startedAt := time.Date(2026, 1, 2, 3, 4, 5, 0, time.UTC)
+	payload := `{"id":"legacy","task_key":"task","task_name":"Task","source_key":"source","status":"success","object_name":"backup.zip","encrypted":false,"size_bytes":42,"started_at":"2026-01-02T03:04:05Z","targets":null}`
+	if err := client.RunRecord.Create().
+		SetTenant("tenant").
+		SetRunID("legacy").
+		SetPayload(payload).
+		SetStartedAt(startedAt).
+		Exec(context.Background()); err != nil {
+		t.Fatalf("create run record: %v", err)
+	}
+
+	runs, err := NewStore(client, "tenant", 100).List()
+	if err != nil {
+		t.Fatalf("List: %v", err)
+	}
+	if len(runs) != 1 {
+		t.Fatalf("len(runs) = %d, want 1", len(runs))
+	}
+	if runs[0].Targets == nil {
+		t.Fatal("Targets is nil, want empty slice")
+	}
+	if len(runs[0].Targets) != 0 {
+		t.Fatalf("len(Targets) = %d, want 0", len(runs[0].Targets))
+	}
+}

web/package-lock.json

@@ -1,7 +1,7 @@
 {
   "name": "nowledge-mem-snap-web",
   "private": true,
-  "version": "0.2.0",
+  "version": "0.2.1",
   "type": "module",
   "scripts": {
     "dev": "vite --host 0.0.0.0",

web/package.json

@@ -11,21 +11,24 @@ export function RunsPage({ runs, locale }: { runs: Run[]; locale: string }) {
   return (
     <Panel title={t('runs')}>
       <div className="runs">
-        {runs.length === 0 ? <Empty text={t('noBackupsYet')} /> : runs.map((run) => (
-          <Card key={run.id} color="purple" pattern="purple" className="run">
-            <div className="item-head"><h3>{run.task_name}</h3><span className={`badge ${run.status}`}>{statusLabel(run.status, t)}</span></div>
-            <p>{new Date(run.started_at).toLocaleString(locale)} · {formatBytes(run.size_bytes)} · {run.encrypted ? t('encrypted') : t('zip')}</p>
-            <code>{run.object_name}</code>
-            <div className="target-results">
-              {run.targets.map((target) => (
-                <span key={target.target_name}>
-                  <ShieldCheck size={14} /> {target.target_name}: {statusLabel(target.status, t)}
-                  {target.retention_deleted ? ` · ${t('deleted')} ${target.retention_deleted}` : ''}
-                </span>
-              ))}
-            </div>
-          </Card>
-        ))}
+        {runs.length === 0 ? <Empty text={t('noBackupsYet')} /> : runs.map((run) => {
+          const targets = run.targets ?? [];
+          return (
+            <Card key={run.id} color="purple" pattern="purple" className="run">
+              <div className="item-head"><h3>{run.task_name}</h3><span className={`badge ${run.status}`}>{statusLabel(run.status, t)}</span></div>
+              <p>{new Date(run.started_at).toLocaleString(locale)} · {formatBytes(run.size_bytes)} · {run.encrypted ? t('encrypted') : t('zip')}</p>
+              <code>{run.object_name}</code>
+              <div className="target-results">
+                {targets.map((target) => (
+                  <span key={target.target_name}>
+                    <ShieldCheck size={14} /> {target.target_name}: {statusLabel(target.status, t)}
+                    {target.retention_deleted ? ` · ${t('deleted')} ${target.retention_deleted}` : ''}
+                  </span>
+                ))}
+              </div>
+            </Card>
+          );
+        })}
       </div>
     </Panel>
   );

web/src/pages/RunsPage.tsx

@@ -155,7 +155,7 @@ export type Run = {
   encrypted: boolean;
   size_bytes: number;
   started_at: string;
-  targets: Array<{ target_name: string; status: string; bytes: number; retention_deleted?: number; error?: string }>;
+  targets?: Array<{ target_name: string; status: string; bytes: number; retention_deleted?: number; error?: string }> | null;
 };
 
 export type SourceRoot = {