Skip to content

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

Open
Bil0000 wants to merge 2 commits into
pingdotgg:t3code/codex-turn-mappingfrom
Bil0000:show-live-thread-names
Open

Bil0000 wants to merge 2 commits into
pingdotgg:t3code/codex-turn-mappingfrom
Bil0000:show-live-thread-names

Conversation

@Bil0000

@Bil0000 Bil0000 commented Sep 22, 2026 •

Copy link
Copy Markdown
Contributor

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:

Finished rows: Read thread “Review auth flow” with the title linked, and a failed read of an unknown thread with the generic label

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

After rename: Read thread “Harden session refresh”

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

In progress: Reading thread “Harden session refresh”

Verification

Focused MessagesTimeline tests (204), web typecheck, and scoped lint and format checks passed. In an isolated dev app with real Codex t3_thread_read calls, the title opened the target thread, followed a rename, and clicking the rest of the row still expanded it. All latest CI checks passed.

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).

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:M 30-99 changed lines (additions + deletions). labels Sep 22, 2026
macroscopeapp[bot]
macroscopeapp Bot previously approved these changes Sep 22, 2026
@macroscopeapp

macroscopeapp Bot commented Sep 22, 2026 •

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at b15da98

Macroscope's review found this PR approvable — This is a focused web timeline fix that resolves existing thread-read activity to live shell titles and adds navigation without changing tool execution, stored data, or product defaults. The production change is localized and supported by focused tests.

No code changes detected at 401dc23. Prior analysis still applies.

You can add or adjust custom eligibility rules. Learn more.

@juliusmarminge
juliusmarminge force-pushed the t3code/codex-turn-mapping branch from a419382 to f2bdd9f Compare September 23, 2026 00:50
@Bil0000
Bil0000 force-pushed the show-live-thread-names branch from 1fcdb1c to d9b7330 Compare September 23, 2026 02:45
@macroscopeapp
macroscopeapp Bot dismissed their stale review September 23, 2026 04:09

Dismissing prior approval to re-evaluate 2332b93

@juliusmarminge
juliusmarminge force-pushed the t3code/codex-turn-mapping branch from 060756d to d6dcd10 Compare September 23, 2026 04:10
macroscopeapp[bot]
macroscopeapp Bot previously approved these changes Sep 23, 2026
@Bil0000
Bil0000 force-pushed the show-live-thread-names branch from 2332b93 to 235ede3 Compare September 23, 2026 04:14
@juliusmarminge
juliusmarminge force-pushed the t3code/codex-turn-mapping branch 2 times, most recently from d2675dc to 9cbaf77 Compare September 23, 2026 07:35
@Bil0000
Bil0000 force-pushed the show-live-thread-names branch from 235ede3 to 48b64e2 Compare September 23, 2026 10:17
@macroscopeapp
macroscopeapp Bot dismissed their stale review September 23, 2026 10:18

Dismissing prior approval to re-evaluate 48b64e2

macroscopeapp[bot]
macroscopeapp Bot previously approved these changes Sep 23, 2026
@juliusmarminge
juliusmarminge force-pushed the t3code/codex-turn-mapping branch 2 times, most recently from 6ca6a24 to 3e4ca4c Compare September 23, 2026 23:24
@Bil0000
Bil0000 force-pushed the show-live-thread-names branch from 48b64e2 to 6bdfff1 Compare September 24, 2026 01:37
@juliusmarminge
juliusmarminge force-pushed the t3code/codex-turn-mapping branch from 1bd44f2 to 3b9c885 Compare September 24, 2026 04:06
@macroscopeapp

This comment has been minimized.

@macroscopeapp
macroscopeapp Bot dismissed their stale review September 24, 2026 07:46

Dismissing prior approval to re-evaluate b15da98

@macroscopeapp

This comment has been minimized.

1 similar comment
@macroscopeapp

This comment has been minimized.

@juliusmarminge
juliusmarminge force-pushed the t3code/codex-turn-mapping branch 2 times, most recently from 6448cfe to 1563a64 Compare September 24, 2026 23:21
@Bil0000
Bil0000 force-pushed the show-live-thread-names branch from b15da98 to 51614af Compare September 25, 2026 00:45
@juliusmarminge
juliusmarminge force-pushed the t3code/codex-turn-mapping branch from fe4f6ad to 87c67bd Compare September 25, 2026 05:55
Bil0000 and others added 2 commits September 25, 2026 12:58
Thread-read rows now read `Read thread “Title”` on one line, in the
tool's tense. On settled rows the title links to the thread; the rest
of the row still expands the tool call. The live row stays one button.
Unknown or unavailable threads keep the generic label.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M 30-99 changed lines (additions + deletions). vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants