fix(tui): dedupe FileWritesPanel rows — one row per path with edit count

[kelsonpw]

User report

"oh god look how bad this is"

The Files panel shows duplicate rows whenever the inner agent edits the same file more than once during a run:

Files · 14 written
✓ MODIFY excalidraw-app/App.tsx · edited 4ms
✓ MODIFY excalidraw-app/share/ShareDialog.tsx · edited 3ms
✓ MODIFY excalidraw-app/share/ShareDialog.tsx · edited 3ms       ← duplicate
✓ MODIFY excalidraw-app/collab/Portal.tsx · edited 10ms
✓ MODIFY excalidraw-app/collab/Portal.tsx · edited 4ms           ← duplicate
✓ MODIFY excalidraw-app/components/ExportToExcalidrawPlus.tsx · edited 6ms
✓ MODIFY excalidraw-app/components/ExportToExcalidrawPlus.tsx · edited 7ms ← duplicate
✓ MODIFY packages/excalidraw/actions/actionAddToLibrary.ts · edited 5ms

WizardStore.recordFileChangePlanned / recordFileChangeApplied append one FileWriteEntry per emission — they only collapse a back-to-back planned duplicate. As soon as the first edit completes (applied), every subsequent edit of the same file produces another row.

After

Files · 8 written
✓ MODIFY excalidraw-app/App.tsx · edited · 4ms
✓ MODIFY excalidraw-app/share/ShareDialog.tsx · edited 2× · 3ms
✓ MODIFY excalidraw-app/collab/Portal.tsx · edited 2× · 4ms
✓ MODIFY excalidraw-app/components/ExportToExcalidrawPlus.tsx · edited 2× · 7ms
✓ MODIFY packages/excalidraw/actions/actionAddToLibrary.ts · edited · 5ms

One row per file. The latest emission's editTime / bytes win. Repeats are annotated with × N.

Implementation

Render-time dedupe in FileWritesPanel (smaller blast radius — store, agent UI, file-change-ledger, and rollback paths all keep their append-only shape).

• New dedupeByPath(entries) helper builds a Map<path, { entry: latest, editCount }>, then sorts by latest startedAt so "most recent activity at the bottom" still holds.
• Dedupe runs before the maxVisible slice — otherwise we'd drop distinct files to make room for duplicates of the same one.
• Header counters (X/Y written, N written) read off the deduped view too — the original "14 written" was misleading.
• × N annotation appears in every status branch (applied, failed, planned) when editCount > 1. Single-edit rows render unchanged so we don't add noise to the common case.
• React key switches from ${startedAt}-${path} to path — paths are unique within the deduped list and stable across rerenders.

Dedupe key

Plain string equality on entry.path. The inner agent normalizes to absolute paths before emitting (per the comment on FileWriteEntry), so two emissions for the same file always carry the same string. No additional path normalization needed at the render layer.

Tests

Added 4 cases in FileWritesPanel.test.tsx:

• 3 emissions of the same path → 1 row, count = 3, header 1 written
• 2 distinct paths + 1 doubled → 2 rows, only the doubled annotated 2×
• Latest emission's bytes / completedAt win on the collapsed row
• Single-edit rows do not pick up a 1× annotation

The 10 pre-existing snapshot/contract tests still pass — no frame regressions.

Verification

• node_modules/.bin/tsc -p tsconfig.json — clean
• node_modules/.bin/vitest run --pool=forks --maxWorkers=1 src/ui/tui/components/__tests__/FileWritesPanel.test.tsx — 14/14 pass
• node_modules/.bin/vitest run --pool=forks --maxWorkers=1 — 3575/3575 pass (243 files)
• pnpm lint — only pre-existing unrelated warning in EventPlanFullScreen.test.tsx

Coordination

• No conflict with feat(tui): live discovery feed during cold start (and gentle logo pulse) #672 (DiscoveryFeed) or feat(tui): live substep narration under active task — show what the wizard is doing now #674 (ActiveTaskSubsteps) — both add new components, this only touches FileWritesPanel.tsx + its test file.
• Width-aware behavior from fix(tui): responsive layout — consistent file list + width-aware KeyHintBar + tab bar #667 unchanged.
• No new context provider; no store/router changes.

🤖 Generated with Claude Code

---

Note

Low Risk
Low risk: render-only behavior change in the TUI that collapses duplicate rows; main risk is subtle ordering/count display regressions, covered by new tests.

Overview
Fixes the TUI FileWritesPanel to collapse multiple write emissions for the same file path into a single row, showing the latest status/bytes/duration and annotating repeats with × N.

Dedupe is applied before maxVisible slicing, header counters now reflect the deduped view, and row keys switch to path for stability. Adds targeted tests to lock down dedupe behavior, latest-entry selection, and × N rendering.

^{Reviewed by Cursor Bugbot for commit c4cf07f. Bugbot is set up for automated code reviews on this repo. Configure here.}