[Juliusolsson05]

Fixes #1347.

Plan: docs/plans/2026-09-26-undo-close-derived-identity.md (first commit).

Problem

Undo Close respawned a closed agent with tldrIdentity: meta.tldrIdentity, the explicit field only. An agent whose identity is DERIVED has no explicit field: its reporting domains are on, but main created it, or its metadata arrived after it acquired TLDR or Goal. Its identity is its session id (tldrIdentityForSession). Undo Close gave it undefined, so spawn minted a fresh UUID. The restored agent then had a new TLDR/Goal identity, so its TLDR and Goal records were no longer reachable from the restored pane. Reload Agents already used the derived rule. Found by review a of #1342.

What merges

• Undo Close passes tldrIdentityForSession(closedSessionId, meta), the rule Reload Agents uses.
• respawn now takes the closed id; both callers (a single pane, a project member) have it.
• An explicit identity passes through unchanged.

Scope

This PR fixes the TLDR/Goal identity (#1347, narrowed after review). Agent Analytics continuity across the new session id is #1342, which can land before or after this one.

Tests

• undoCloseFailure.renderer.test.tsx:
  • a closed agent with a reporting domain and no explicit identity respawns with its closed id as tldrIdentity, and an explicit one passes through unchanged; red on main (undefined);
  • a project restore gives each member its own closed id; this case was added after review, and a wrong member id now fails it.
• Gates: on ff32bce5 (main, including fix(goal-loop): Undo Close hands the restored pane its goal loop #1331, merged), npx tsc -b is clean and the renderer actions suites pass (252 tests).

🤖 Generated with Claude Code

[Juliusolsson05]

Review disposition. Round 1: a MERGE-READY; b and c FIX-BEFORE-MERGE. Verification: b MERGE-READY (round 2). c FIX-BEFORE-MERGE in round 2 (a base conflict, plus a project test that needed two derived members), then MERGE-READY in round 3 at ff32bce5.

Fixed

• a1, b, c3: the project path's per-member identity was pinned by nothing. The project test restores two derived members and one explicit member. c mutated every member to the first member's id, and the test goes red.
• c, round 2: base conflict. Resolved by hand twice, after fix(workflows): a replaced pane keeps its workflow runs, across restarts too #1325 and then after fix(goal-loop): Undo Close hands the restored pane its goal loop #1331. Each time the only conflict was the import block of undoClose.ts: this PR's tldrIdentityForSession is kept alongside main's successorCarry imports. c confirmed both merge commits and that respawn(meta, closedSessionId) survives on both paths.

Out of scope (the PR and #1347 are narrowed to the TLDR/Goal identity)

• c1: Agent Analytics continuity across Undo Close is fix(analytics): count a replaced agent that keeps its conversation as one agent #1342.
• c2: Goal Loop carry on Undo Close was bug(goal-loop): Undo Close respawns a pane under a new id without carrying its goal loop #1320, now fixed on main by fix(goal-loop): Undo Close hands the restored pane its goal loop #1331.

Info (a2): the real-data count shows that the derived state is a transient window, not the steady state. No change needed.

🤖 Generated with Claude Code