Keep resumed sessions consistent about their task list

Two surfaces disagreed with each other after a resume. The transcript
dropped its manage_tasks rows in favor of one aggregated task block, but
only when the rows were stripped alongside submit_plan's — without that,
every call reappeared as its own row beneath the summary. And the chrome
panel reads the director's in-memory list, which a freshly constructed
director leaves empty until the model happens to call manage_tasks again,
so a resumed session showed tasks in its scrollback and none in its panel.

Restoring the derived list into the director closes both: one source of
task state, painted the same way everywhere.

Author: TheGreatAxios

src/agent/director.ts

@@ -404,6 +404,15 @@ class ChatDirectorImpl extends DefaultDirector {
     return [...this.tasks];
   }
 
+  // A resumed session's task list lives in the transcript, not in the freshly
+  // constructed director. Without this the chrome panel would read an empty
+  // list until the model happened to call manage_tasks again, disagreeing
+  // with the task block already painted in the transcript.
+  restoreTasks(tasks: Task[]): void {
+    this.tasks = [...tasks];
+    this.onTasksChange?.(this.tasks);
+  }
+
   // The status bar's context meter falls back to this when a provider omits
   // or zeroes usage on the latest turn — a local lower-then-corrected bound
   // beats displaying a number the provider never actually reported.
@@ -828,5 +837,6 @@ export interface ChatDirector extends ReactorDirector {
   setGoalGovernor(goal: GoalGovernor | undefined): void;
   getGoalGovernor(): GoalGovernor | undefined;
   getTasks(): Task[];
+  restoreTasks(tasks: Task[]): void;
   getContextEstimate(): { tokens: number; isEstimate: boolean };
 }

src/director.test.ts

@@ -791,6 +791,19 @@ describe("onTasksChange live wiring", () => {
     expect(updates[0]).toEqual([{ id: "t1", title: "work", status: "doing" }]);
   });
 
+  test("restoreTasks seeds a resumed session's task list and notifies the consumer", () => {
+    const updates: Array<Array<{ id: string; title: string; status: string }>> = [];
+    const director = createChatDirector("base", [], {
+      onTasksChange: (tasks) => updates.push(tasks),
+    });
+
+    const restored = [{ id: "t1", title: "from transcript", status: "doing" as const }];
+    director.restoreTasks(restored);
+
+    expect(director.getTasks()).toEqual(restored);
+    expect(updates).toEqual([restored]);
+  });
+
   test("onTasksChange is not invoked for tool calls that are not manage_tasks", async () => {
     const updates: unknown[] = [];
     const director = createChatDirector("base", [], {

src/tui/runner.ts

@@ -2272,7 +2272,10 @@ export async function runTUI(initialConfig: Config): Promise<number> {
     .then((turns) => {
       const blocks = turnsToContentBlocks(turns, { maxBlocks: RESUME_TRANSCRIPT_BLOCK_LIMIT });
       const tasks = hydrateTasksFromTurns(turns);
-      if (tasks.length > 0) blocks.unshift({ type: "tasks", tasks });
+      if (tasks.length > 0) {
+        blocks.unshift({ type: "tasks", tasks });
+        directorHolder.instance?.restoreTasks(tasks);
+      }
       if (blocks.length > 0) emitter.emit("history.hydrate", blocks);
     })
     .catch((err: unknown) => {

src/tui/turns-to-blocks.test.ts

@@ -34,6 +34,20 @@ describe("turnsToContentBlocks no longer derives tasks", () => {
     const blocks = turnsToContentBlocks(turns);
     expect(blocks.some((b) => b.type === "tasks")).toBe(false);
   });
+
+  // The aggregated task block is unshifted separately on resume, so leaving
+  // the raw rows in would show every manage_tasks call twice over.
+  test("manage_tasks call and result rows are stripped from the resumed transcript", () => {
+    const turns = [
+      manageTasksTurn("m1", "todo"),
+      toolResultTurn("m1", false),
+      manageTasksTurn("m2", "doing"),
+      toolResultTurn("m2", false),
+    ];
+    const blocks = turnsToContentBlocks(turns);
+    expect(blocks.some((b) => b.type === "tool_call" && b.name === "manage_tasks")).toBe(false);
+    expect(blocks.some((b) => b.type === "tool_result" && b.name === "manage_tasks")).toBe(false);
+  });
 });
 
 describe("hydrateTasksFromTurns", () => {

src/tui/turns-to-blocks.ts

@@ -90,7 +90,12 @@ function upsertResumeBlock(
   return next;
 }
 
-/** Mirror live-stream tool.done handling for plan (submit_plan) when hydrating a session. */
+/**
+ * Mirror live-stream tool.done handling when hydrating a session: submit_plan
+ * collapses into a single plan block, and manage_tasks rows are dropped
+ * entirely because the resumed task list is rendered as one aggregated block
+ * (see hydrateTasksFromTurns) rather than as one row per call.
+ */
 function finalizeResumeToolBlocks(blocks: ContentBlockData[]): ContentBlockData[] {
   const callIdToCallIndex = new Map<string, number>();
   for (let i = 0; i < blocks.length; i += 1) {
@@ -109,7 +114,14 @@ function finalizeResumeToolBlocks(blocks: ContentBlockData[]): ContentBlockData[
     const callIndex = callIdToCallIndex.get(result.callId);
     if (callIndex === undefined) continue;
     const call = blocks[callIndex];
-    if (call?.type !== "tool_call" || call.name !== "submit_plan") continue;
+    if (call?.type !== "tool_call") continue;
+
+    if (call.name === "manage_tasks") {
+      indicesToRemove.add(callIndex);
+      indicesToRemove.add(i);
+      continue;
+    }
+    if (call.name !== "submit_plan") continue;
 
     indicesToRemove.add(callIndex);
     indicesToRemove.add(i);