Skip to content

fix(web): sidebar rows show the branch again, not a truncated plan step - #5776

Merged
t3dotgg merged 1 commit into
mainfrom
t3code/fix-thread-subtitle-branch-names
Aug 9, 2026
Merged

fix(web): sidebar rows show the branch again, not a truncated plan step#5776
t3dotgg merged 1 commit into
mainfrom
t3code/fix-thread-subtitle-branch-names

Conversation

@t3dotgg

@t3dotgg t3dotgg commented Aug 9, 2026

Copy link
Copy Markdown
Member

Some sidebar rows stopped showing the branch name and showed a random-looking sentence fragment instead. Gross.

That text was the agent's current plan step. #5558 gave it the branch's slot on any working row, and #5672 made that sidebar the default, so it showed up everywhere at once. Two problems with it:

  • The step text almost always overflowed and truncated to a half-sentence.
  • The n/m progress counter sat inside the same truncating span, so it got cut off too.

The row gave up its most stable identifier and got nothing readable back. Now the subtitle is always the branch.

thread.planProgress is still on the wire and the chat view still derives its own step label from activities, so the in-chat working row is unchanged. The sidebar was the only reader of the server field, so getThreadPlanProgress is now dead weight in the projection snapshot query. Left it alone here since removing it touches the contract.


Made with Claude Opus 5 (1M context) on Claude Code.


Note

Low Risk
Single conditional UI change in the sidebar with no auth, data, or API contract changes in this diff.

Overview
Sidebar card rows no longer swap the subtitle for the agent’s current plan step while a thread is Working. That slot always shows thread.branch again (or stays empty when there is no branch).

Previously, working rows prioritized thread.planProgress (step text plus completedSteps/totalSteps), which often truncated mid-sentence and hid the branch—the row’s most stable identifier. In-chat working UI still derives its step label from activities; only the sidebar read of server planProgress is removed here.

Reviewed by Cursor Bugbot for commit 57a0580. Bugbot is set up for automated code reviews on this repo. Configure here.

Note

Fix sidebar thread rows to show branch instead of truncated plan step

In Sidebar.tsx, the secondary metadata line in SidebarThreadRow previously swapped to showing the current plan step and completed/total counters while a thread was in working status. It now unconditionally shows thread.branch when available, removing the plan progress display entirely.

Macroscope summarized 57a0580.

The plan step replaced the branch on every working row. It almost always
truncated to a half-sentence, and the n/m counter it carried lived inside
the truncating span, so it got cut too. The row lost its branch and gained
nothing readable.

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

coderabbitai Bot commented Aug 9, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 47f9209f-ae90-4070-844d-6cee51c55afa

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:S 10-29 changed lines (additions + deletions). labels Aug 9, 2026
@macroscopeapp

macroscopeapp Bot commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved 57a0580

This is a simple UI fix that removes conditional rendering logic to always show the branch name in sidebar rows instead of conditionally showing plan step progress. The change is self-contained, affects only presentation, and the author is the primary editor of this code.

You can customize Macroscope's approvability policy. Learn more.

@github-actions

github-actions Bot commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider Metric Main baseline This PR Impact PR ceiling
Codex Total thread wire 11.3 KiB 11.3 KiB 0 B (0.0%) 15.1 KiB
Codex Thread snapshot wire 5.5 KiB 5.5 KiB +4 B (+0.1%) 7.3 KiB
Codex Live turn WebSocket wire 5.9 KiB 5.9 KiB −4 B (−0.1%) 7.8 KiB
Codex Live turn WebSocket decoded 49.7 KiB 49.7 KiB 0 B (0.0%) 66.4 KiB
Codex Live turn messages 16 16 0 (0.0%) 21
Claude Total thread wire 11.3 KiB 11.3 KiB +33 B (+0.3%) 15.1 KiB
Claude Thread snapshot wire 5.5 KiB 5.5 KiB +6 B (+0.1%) 7.3 KiB
Claude Live turn WebSocket wire 5.8 KiB 5.9 KiB +27 B (+0.5%) 7.8 KiB
Claude Live turn WebSocket decoded 50.6 KiB 50.6 KiB 0 B (0.0%) 66.4 KiB
Claude Live turn messages 16 16 0 (0.0%) 21

Baseline: 886195e · PR result: 57a0580 · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

  • Codex decoded thread snapshot: 94.6 KiB
  • Claude decoded thread snapshot: 95.4 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

@t3dotgg
t3dotgg merged commit 6f69b44 into main Aug 9, 2026
17 checks passed
@t3dotgg
t3dotgg deleted the t3code/fix-thread-subtitle-branch-names branch August 9, 2026 02:45
github-actions Bot added a commit to omarcresp/t3code-flake that referenced this pull request Aug 9, 2026
## What's Changed
* chore: vouch gfsaaser24 by @t3dotgg in pingdotgg/t3code#5761
* chore: vouch saphid by @t3dotgg in pingdotgg/t3code#5763
* fix(server): stop Codex threads with queued follow-ups by @t3dotgg in pingdotgg/t3code#5762
* fix(web): usage page loses the cost quality panel, gains a back button by @t3dotgg in pingdotgg/t3code#5756
* feat(server): agents can now open the images you paste into chat by @t3dotgg in pingdotgg/t3code#5757
* fix(web): pinned reorder no longer reshuffles while writes land by @t3dotgg in pingdotgg/t3code#5767
* feat(web): overhaul project settings into a real settings page by @t3dotgg in pingdotgg/t3code#5768
* fix(web): usage totals no longer jump while devices report in by @t3dotgg in pingdotgg/t3code#5772
* fix(server): settle no longer leaves monitors and dev servers running by @t3dotgg in pingdotgg/t3code#5774
* feat: pick worktree or current checkout per project by @t3dotgg in pingdotgg/t3code#5766
* fix(web): sidebar rows show the branch again, not a truncated plan step by @t3dotgg in pingdotgg/t3code#5776
* feat(server): vp run migrate-dev-db seeds worktree dev dbs with real data by @t3dotgg in pingdotgg/t3code#5773
* feat(web): keep unsent drafts one click away in the sidebar by @t3dotgg in pingdotgg/t3code#5777


**Full Changelog**: pingdotgg/t3code@v0.0.33-nightly.20260809.1039...v0.0.33-nightly.20260809.1040

Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.33-nightly.20260809.1040
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S 10-29 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.

1 participant