feat(web): refresh workspace layouts and tool activity - #6275
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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. Comment |
ApprovabilityVerdict: Needs human review Diff is too large for automated approval analysis. A human reviewer should evaluate this PR. You can customize Macroscope's approvability policy. Learn more. |
6d872d8 to
5a1d64b
Compare
5a1d64b to
11b64b9
Compare
There was a problem hiding this comment.
UI consistency review found three regressions in changed code: a reverted container-query fix in the changed-files card, a terminal sidebar whose minimum width is narrower than the toolbar it now hosts, and an expandable tool row that lost its hover affordance. Details inline.
Posted via Macroscope — UI Consistency
d1ddd16 to
2a7d0e7
Compare
There was a problem hiding this comment.
UI consistency review of the latest head (2a7d0e7). One new finding from the refactor(web): trim workspace refresh scope commit; the three findings from the previous run are still open and unchanged, so they are not re-posted here.
Posted via Macroscope — UI Consistency
There was a problem hiding this comment.
One accessibility finding on the new live tool-call row. Earlier findings from previous runs on this PR (changed-files container query, terminal sidebar minimum width, plain tool row hover affordance, deleted primitive tests) still stand and are not repeated here.
Posted via Macroscope — UI Consistency
There was a problem hiding this comment.
Reviewed the changed web UI for shared-primitive, Tailwind-ownership, and theming consistency. Four new findings below (earlier findings on this PR are still open and not repeated).
Posted via Macroscope — UI Consistency
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit dbe6252. Configure here.
There was a problem hiding this comment.
Two consistency findings in the workspace-shell refactor. Details inline.
Posted via Macroscope — UI Consistency
There was a problem hiding this comment.
Two issues found in the changed web UI. Everything else in this revision (shared WorkspacePageHeader/WorkspacePageContainer adoption, SegmentedTab*, sidebar Button migrations, terminal sidebar, ghost/loading shapes) looks consistent with the primitive contracts.
Posted via Macroscope — UI Consistency
There was a problem hiding this comment.
One finding this round: the redesigned subagent CTA loses its accessible name on narrow widths. Everything flagged in earlier runs (settings scroll-fade band, compact refresh glyph size, provider total color, snooze tooltip/title, panel-right glyph size, live-row aria-label, changed-files container query, terminal sidebar minimum width) is resolved at this head.
Posted via Macroscope — UI Consistency
There was a problem hiding this comment.
Three findings, all in apps/web. The shared-primitive migrations (WorkspacePageHeader, SegmentedTab, Button/Badge adoption in the sidebar, terminal toolbar and PR chrome) otherwise look consistent with the component system, and the earlier notes on the snooze tooltip, PanelRightIcon sizing, the live-work row's accessible name, the changed-files container query and the terminal sidebar minimum width are addressed at this head.
Posted via Macroscope — UI Consistency
## What's Changed * feat(web): open remote environments in your local editor over SSH by @t3dotgg in pingdotgg/t3code#6572 * feat(web): refresh workspace layouts and tool activity by @maria-rcks in pingdotgg/t3code#6275 * revert: refresh workspace layouts and tool activity by @maria-rcks in pingdotgg/t3code#6657 * feat(web): older chat timestamps show the date, not just the time by @t3dotgg in pingdotgg/t3code#6654 * fix(web): align pull request action menu rows by @Bil0000 in pingdotgg/t3code#6534 **Full Changelog**: pingdotgg/t3code@v0.0.34-nightly.20260815.1097...v0.0.34-nightly.20260815.1098 Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.34-nightly.20260815.1098

Changes
Workspace, navigation, and Settings
Usage
Pull Requests
Terminal
Timeline, tool calls, and changed files
Validation
Built with GPT-5.6 Sol via Codex in T3 Code.
Note
Medium Risk
Changes span server activity projection/ingestion and client timeline folding logic, which can affect how tool rows merge and display; terminal UI state persistence may migrate stored label data.
Overview
This PR threads tool lifecycle identity through ingestion and projection: runtime tool events now stamp top-level
toolCallIdfromitemId, activity slimming normalizes command text from Claude/OpenCode-shaped payloads, and lifecycle collapse keys prefer payloadtoolCallIdover legacy nested ids.On the chat timeline, active turns get a single Working header (with optional Thinking until content appears), in-progress tool batches render as expandable live rows with concise labels (e.g. Running psql), and completed batches collapse into action-aware summaries (files read/changed, commands, etc.) with failure styling. Worked for… folds anchor before the terminal assistant message after steers, and changed-files cards use tighter inline previews instead of scope chips and “show all” affordances.
Workspace UI consolidates top-bar geometry via
WorkspacePageHeader/WorkspacePageContainer, aligns panel toggles and composer chip metrics, and makes New thread create directly in a scoped project. The terminal drawer gains hide, a resizable multi-session sidebar, split-group labeling, and per-thread custom names (double-click rename, persisted in terminal UI state). Sidebar lifecycle controls use consistent compact icon slots; Electron context menus honorseparatorBeforealongside destructive separators.Reviewed by Cursor Bugbot for commit 9969027. Bugbot is set up for automated code reviews on this repo. Configure here.
Note
Refresh workspace layouts, tool activity timeline, and terminal drawer UI
WorkspacePageHeader,WorkspacePageContainer, andWorkspacePageHeaderEdgeControlcomponents in WorkspacePageContainer.tsx, replacing bespoke header markup across Settings, ChatView, Usage, Pull Requests, and onboarding routes.work-livekind) to the messages timeline: in-progress tool batches are collapsed into a single interactive row with a derived label (e.g. "Running psql"), and tool-only groups show a summarized toggle with failure indicators.terminalUiStateStore).tool.startedactivity handling in session-logic.ts so running commands appear immediately; tool lifecycle entries with the sametoolCallIdnow collapse across non-adjacent rows.separatorBeforeon items (honored in both Electron and web fallback), a nested 'Copy' submenu, and icons for many thread actions.terminalUiStateStorepersistence bumps from version 4 to 5; existing stored state is migrated and normalized on first load.Macroscope summarized 9969027.