fix(web): show live names in thread-read activity

[Bil0000]

What Changed

“Read a T3 thread” activity now names the thread it read, on one line: Read thread “Review auth flow”, in the tool’s tense (Reading / Read / Failed to read). On finished rows the title links to that thread; clicking anywhere else on the row still expands the tool call. The title comes from client shell state, so a rename shows up without a DB join. If the thread is unknown, archived or deleted, the row keeps the generic “Read a T3 thread” label and no link.

Why

The tool label alone does not identify the target thread.

UI Changes

Finished row. The title links to the thread; an unknown thread keeps the generic label:

After renaming the target thread, the same row shows the new name:

While the call runs, the live row reads in the present tense:

Scope and approval

This is a small, focused display fix for an existing thread-read activity. The generic label does not identify the thread that was read. It uses the thread ID already present in the tool input and the existing thread route. The change stays in the web timeline and its tests. Web and desktop share this timeline. Mobile rendering is outside this web-only change. The single-line layout preserves the rewrite by Julius.

Verification

All 237 focused MessagesTimeline component and logic tests pass under Node 24.13.1. The new regression failed before the fix, then passed after it. Web typecheck, scoped lint, format, and diff checks passed. In the isolated real app, a test tool row opened its target, expanded when clicked outside the title, and picked up a rename through the normal app command. Archiving and deleting the target removed the title link; unarchiving restored it. Test projection data was used for the tool row. All current CI checks passed, and CodeRabbit and Macroscope approved this head.

Before, on current main:

After, with the renamed title and expanded details:

The 36-second recording shows the title opening the target, the rest of the row expanding, and a rename updating that same row. Playback is sped up to remove tool wait time.

pr13140-interaction.mp4

Checklist

• This PR is small and focused
• I explained what changed and why
• I included screenshots for UI changes
• I included a video for animation/interaction changes

Model: GPT-6 Codex (native Codex harness). Single-line redesign by Claude Opus 5.5 (Claude Code). Review fixes and verification: GPT-6.1-Sol (Codex harness).

[macroscopeapp]

Approvability

Verdict: Approved at ebf999f

Macroscope's review found this PR approvable — This is a small, self-contained web timeline fix that derives live thread names from existing client state and adds a settled-row link while preserving generic fallbacks. Its runtime impact is limited to presentation and navigation for thread-read activity, with focused helper tests and no schema, deployment, or sensitive-area changes.

^{You can add or adjust custom eligibility rules. Learn more.}

Dismissing prior approval to re-evaluate 2332b93

Dismissing prior approval to re-evaluate 48b64e2

Dismissing prior approval to re-evaluate b15da98

[juliusmarminge]

Note

This comment is posted by Julius' dot

The supplied screenshots show the renamed labels, but not the title-link and row-expansion interaction. Could you add a short recording showing the target opening, the rest of the row expanding, and a rename reaching the same row, plus a before screenshot? This would complete the UI verification evidence while preserving Julius' inline-label rewrite.

Dismissing prior approval to re-evaluate 58d86fe

[macroscopeapp]

Macroscope has since reviewed this pull request. An earlier review was skipped by a cost limit; a review has now completed, so that notice no longer applies.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: Path: .coderabbit.config.ts
• Review profile: CHILL
• Plan: Advanced
• Run ID: 11f8bd85-2e11-40e7-a79e-e162b74d1cbd

📥 Commits

Reviewing files that changed from the base of the PR and between 0678e4e and ebf999f.

📒 Files selected for processing (3)

• apps/web/src/components/chat/MessagesTimeline.logic.test.ts
• apps/web/src/components/chat/MessagesTimeline.logic.ts
• apps/web/src/components/chat/MessagesTimeline.tsx

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 7 remain after this review.

---

📝 Walkthrough

Walkthrough

The message timeline now identifies thread-read work entries, resolves eligible thread titles, and displays them in live and settled rows. Live titles are plain text. Settled titles link to the thread.

Changes

Thread-read presentation

Layer / File(s)	Summary
Resolve thread-read targets and labels apps/web/src/components/chat/MessagesTimeline.logic.ts, apps/web/src/components/chat/MessagesTimeline.logic.test.ts	Helpers extract and normalize thread IDs, eligible shell titles, and thread-read label prefixes. Tests cover target validation, title eligibility, and lifecycle labels.
Display thread titles in live rows apps/web/src/components/chat/MessagesTimeline.tsx	Live work rows display eligible thread titles as plain text. Other work labels retain their existing rendering.
Link thread titles in settled rows apps/web/src/components/chat/MessagesTimeline.tsx	Settled work rows include eligible thread titles in labels and accessible text. Thread titles link to the thread route.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~15 minutes

Change: Feature

Suggested reviewers: juliusmarminge

Merge Risk: ⚪ Minimal · up to ebf99

Eligible thread names appear in activity labels, and settled titles link to their threads. No actionable merge risk is established.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description explains the problem, change, UI behavior, and verification. It does not include the required Scope and approval section, and it does not provide the requested recording and before scr…	Add a Scope and approval section with the linked issue or explicit maintainer approval, or explain why this focused fix qualifies for an exemption. Add a before screenshot and a short recording that demonstrates opening the target thread, e…

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and concisely describes the main change: showing live thread names in thread-read activity.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Approvability	✅ Passed	PASS. The pull request changes only three existing apps/web timeline component and test files. It adds thread-read title resolution, labels, and an internal link for an existing activity row. The di…

Full details: Description check

Explanation

The description explains the problem, change, UI behavior, and verification. It does not include the required Scope and approval section, and it does not provide the requested recording and before screenshot for the link, expansion, and rename interactions.

Resolution

Add a Scope and approval section with the linked issue or explicit maintainer approval, or explain why this focused fix qualifies for an exemption. Add a before screenshot and a short recording that demonstrates opening the target thread, expanding the row outside the title, and seeing a rename update the same row.

• Fix all pre-merge checks with AI

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create a new PR

• Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

---

_{Comment @coderabbitai help to get the list of available commands.}