feat(web): improve terminal workspace controls

[maria-rcks]

What changed

• Reworks the terminal drawer controls and multi-session sidebar while retaining split groups and resize behavior.
• Persists custom terminal names by scoped thread, prunes stale labels, and clears labels when terminals or threads disappear.
• Adds an explicit drawer hide path and keeps single-terminal sessions free of unnecessary sidebar chrome.

Screenshots

Direct parent on the left; this PR on the right. Same viewport and copied application state.

Before	After

## Why

Terminal state and interaction changes need a focused review. This layer depends only on the shared chat header from #7153.

Validation

• Terminal drawer tests
• Terminal UI store migration, naming, cleanup, reconciliation, and sidebar tests
• Affected-package typechecks
• Changed-file lint and formatting checks

Stack order

1. feat(web): unify workspace navigation #7153 workspace and navigation
2. feat(web): redesign usage insights #7147 Usage
3. feat(web): refresh pull request details #7148 Pull Requests
4. feat(web): improve terminal workspace controls #7149 terminal, this PR
5. feat(web): attach composer state drawers #7150 composer
6. fix(server): preserve tool lifecycle identity #7151 tool lifecycle projection
7. feat(web): collapse tool activity into one line #7152 tool activity UI

Built with GPT-5.6-sol in the Codex harness.

---

Note

Medium Risk
Touches persisted terminal UI state (migration v5) and server/client terminal ID reconciliation; behavior is covered by tests but wrong label pruning could affect multi-surface terminal UX.

Overview
Reworks the thread terminal drawer with a resizable session sidebar, persisted per-terminal names, and a toolbar action to hide the drawer without closing sessions.

The multi-terminal sidebar moves to the right of the viewport with a drag handle (useResizableWidth + separate localStorage keys for drawer vs panel). Width is capped so the terminal area keeps a minimum usable width. Toolbar actions switch from popover hover buttons to compact icon tooltips; drawer mode adds Hide (wired from ChatView via onHide).

Custom terminal labels are stored in terminalUiStateStore (persist v5): rename via double-click or F2, with logic so unchanged automatic titles are not saved as custom names. Labels apply in the drawer list and in ChatView composer terminal chips. Closing a terminal or pruning during reconcileTerminalIds drops stale labels; reconciliation can pass panel terminal ids so right-panel sessions keep names when drawer state syncs from the server.

Single-terminal layout keeps a floating compact toolbar only (no sidebar chrome).

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

Note

Improve terminal workspace controls with inline renaming, custom labels, and resizable sidebar

• Adds inline terminal renaming (double-click or F2) in ThreadTerminalDrawer, persisting custom labels up to 80 chars in terminalUiStateStore (version bumped 4→5 with migration).
• Introduces a resizable terminal sidebar with dynamic max-width clamping and per-context persisted widths; sidebar hides when only one terminal is open.
• Adds onHide callback to ThreadTerminalDrawer and wires it through ChatViewContent so the drawer can request to be hidden.
• Reworks PullRequestDetailPanel to prioritize conflict resolution in the primary action slot and adds a stacked-PR detection via isStackedPullRequestBase and a new listRefs query.
• Adds rightPanelUnavailableLabel prop to PanelLayoutControls and ensures tooltip triggers remain interactive when toggles are disabled.
• Removes chromeVariant prop from PullRequestDetailPanel and its callers; condensed state is now derived solely from scroll position.
• Risk: terminalUiStateStore persistence version increment triggers migration for all existing users; label data not matching the new schema is silently dropped.

^{Macroscope summarized 69bbbf1.}

[coderabbitai]

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: dcb2d16b-de0c-4b21-b181-d99a892ed553

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ No successful main baseline artifact is available yet. This run establishes the initial measurement.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	—	12.6 KiB	—	15.1 KiB	✅
Codex	Thread snapshot wire	—	6.3 KiB	—	7.3 KiB	✅
Codex	Live turn WebSocket wire	—	6.3 KiB	—	7.8 KiB	✅
Codex	Live turn WebSocket decoded	—	51.8 KiB	—	66.4 KiB	✅
Codex	Live turn messages	—	16	—	21	✅
Claude	Total thread wire	—	12.6 KiB	—	15.1 KiB	✅
Claude	Thread snapshot wire	—	6.3 KiB	—	7.3 KiB	✅
Claude	Live turn WebSocket wire	—	6.3 KiB	—	7.8 KiB	✅
Claude	Live turn WebSocket decoded	—	52.7 KiB	—	66.4 KiB	✅
Claude	Live turn messages	—	16	—	21	✅

Baseline: unavailable · PR result: 69bbbf1 · 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: 99.9 KiB
• Claude decoded thread snapshot: 100.6 KiB

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

[macroscopeapp]

UI consistency review of the terminal drawer refresh. The migration to Button size="icon-xs" for the toolbar, the RightPanelResizeHandle/useResizableWidth reuse for the sidebar, and the tab-row treatment copied from RightPanelTabs all line up with the shared component system. Two tooltip-related consistency notes below; nothing else looked like a regression.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Approvability

Verdict: Not approved

Macroscope's review found this PR not approvable — This PR introduces new user-facing features including terminal renaming, a resizable sidebar, and stacked pull request detection. The scope and complexity of these new capabilities, combined with unresolved findings about stale label pruning and resize persistence behavior, warrant human review.

Not approved because:

• 2 blocking correctness issues found at or above your repo's Minimum Blocking Severity

^{Adjust the Minimum Blocking Severity for this repo — including turning it Off — in Settings. You can add or adjust custom eligibility rules. Learn more.}

[macroscopeapp]

Two new findings on the terminal sidebar chrome. The two tooltip findings from the previous run (native title where the sibling tab bar / compact toolbars use the shared Tooltip, and the unreachable split-limit wording on the disabled Button) are still open and unchanged.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

UI consistency review of the terminal drawer refresh. Two new findings on the added controls; previously flagged items (sidebar toolbar overflow at the 144px min width, per-instance useResizableWidth state sharing one storage key, native title instead of the shared Tooltip) still apply and are not repeated here.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

🟡 Medium

t3code/apps/web/src/components/ChatView.tsx

Line 790 in 69bbbf1

if(terminalIdListsEqual(serverOrderedTerminalIds,terminalUiState.terminalIds)){

Stale custom labels remain persisted when the server and drawer terminal IDs are equal, so a label for a deleted terminal is never pruned and can be reused when that ID is allocated again. The early return skips reconcileTerminalIds, which performs the label pruning; allow reconciliation to run in this case.

🚀 Reply "fix it for me" or copy this AI Prompt for your agent:

In file @apps/web/src/components/ChatView.tsx around line 790:
Stale custom labels remain persisted when the server and drawer terminal IDs are equal, so a label for a deleted terminal is never pruned and can be reused when that ID is allocated again. The early return skips `reconcileTerminalIds`, which performs the label pruning; allow reconciliation to run in this case.

[cursor]

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 69bbbf1. Configure here.}

[cursor]

Resize reverts if persist fails

Medium Severity

After a resize ends, dragWidth is cleared whenever the pointer is not down. Display then falls back to storedWidth. If setStoredWidth fails (quota or write error, which useLocalStorage swallows), the panel snaps back to the old width instead of keeping the drag result. Previously in-memory width still updated after a failed persist.

Additional Locations (1)

• apps/web/src/hooks/useResizableWidth.ts#L135-L143

^{Reviewed by Cursor Bugbot for commit 69bbbf1. Configure here.}

[macroscopeapp]

🟡 Mediumlib/openPullRequestLink.ts:126

changeRequestRepositoryUrl returns null for valid Azure URLs such as https://dev.azure.com/pull/123/_git/repo/pullrequest/42, and can return a truncated repository URL for similar paths. Its non-greedy fallback stops at the first /pull/<number>-like segment instead of the final PR suffix; extract the repository path using the last recognized provider suffix.

- const repositoryPath =- /^(.*?)\/\/-\/merge_requests\/\d+(?:\/|$)/iu.exec(url.pathname)?.[1] ??- /^(.*?)(?:\/pull\/\d+|\/-\/merge_requests\/\d+|\/pull-requests\/\d+|\/pullrequest\/\d+)(?:\/|$)/iu.exec(- url.pathname,- )?.[1];+ const repositoryPath = /^(.*)(?:\/pull\/\d+|\/-\/merge_requests\/\d+|\/pull-requests\/\d+|\/pullrequest\/\d+)(?:\/|$)/iu.exec(+ url.pathname,+ )?.[1];

🚀 Reply "fix it for me" or copy this AI Prompt for your agent:

In file @apps/web/src/lib/openPullRequestLink.ts around lines 126-130:
`changeRequestRepositoryUrl` returns `null` for valid Azure URLs such as `https://dev.azure.com/pull/123/_git/repo/pullrequest/42`, and can return a truncated repository URL for similar paths. Its non-greedy fallback stops at the first `/pull/<number>`-like segment instead of the final PR suffix; extract the repository path using the last recognized provider suffix.

[maria-rcks]

What changed

• Reworks the terminal drawer controls and multi-session sidebar while retaining split groups and resize behavior.
• Persists custom terminal names by scoped thread, prunes stale labels, and clears labels when terminals or threads disappear.
• Adds an explicit drawer hide path and keeps single-terminal sessions free of unnecessary sidebar chrome.

Screenshots

Direct parent on the left; this PR on the right. Same viewport and copied application state.

Before	After

## Why

Terminal state and interaction changes need a focused review. This layer depends only on the shared chat header from #7153.

Validation

• Terminal drawer tests
• Terminal UI store migration, naming, cleanup, reconciliation, and sidebar tests
• Affected-package typechecks
• Changed-file lint and formatting checks

Stack order

1. feat(web): unify workspace navigation #7153 workspace and navigation
2. feat(web): redesign usage insights #7147 Usage
3. feat(web): refresh pull request details #7148 Pull Requests
4. feat(web): improve terminal workspace controls #7149 terminal, this PR
5. feat(web): attach composer state drawers #7150 composer
6. fix(server): preserve tool lifecycle identity #7151 tool lifecycle projection
7. feat(web): collapse tool activity into one line #7152 tool activity UI

Built with GPT-5.6-sol in the Codex harness.

---

Note

Medium Risk
Touches persisted terminal UI state (migration v5) and server/client terminal ID reconciliation; behavior is covered by tests but wrong label pruning could affect multi-surface terminal UX.

Overview
Reworks the thread terminal drawer with a resizable session sidebar, persisted per-terminal names, and a toolbar action to hide the drawer without closing sessions.

The multi-terminal sidebar moves to the right of the viewport with a drag handle (useResizableWidth + separate localStorage keys for drawer vs panel). Width is capped so the terminal area keeps a minimum usable width. Toolbar actions switch from popover hover buttons to compact icon tooltips; drawer mode adds Hide (wired from ChatView via onHide).

Custom terminal labels are stored in terminalUiStateStore (persist v5): rename via double-click or F2, with logic so unchanged automatic titles are not saved as custom names. Labels apply in the drawer list and in ChatView composer terminal chips. Closing a terminal or pruning during reconcileTerminalIds drops stale labels; reconciliation can pass panel terminal ids so right-panel sessions keep names when drawer state syncs from the server.

Single-terminal layout keeps a floating compact toolbar only (no sidebar chrome).

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

Note

Improve terminal workspace controls with inline renaming, custom labels, and resizable sidebar

• Adds inline terminal renaming (double-click or F2) in ThreadTerminalDrawer, persisting custom labels up to 80 chars in terminalUiStateStore (version bumped 4→5 with migration).
• Introduces a resizable terminal sidebar with dynamic max-width clamping and per-context persisted widths; sidebar hides when only one terminal is open.
• Adds onHide callback to ThreadTerminalDrawer and wires it through ChatViewContent so the drawer can request to be hidden.
• Reworks PullRequestDetailPanel to prioritize conflict resolution in the primary action slot and adds a stacked-PR detection via isStackedPullRequestBase and a new listRefs query.
• Adds rightPanelUnavailableLabel prop to PanelLayoutControls and ensures tooltip triggers remain interactive when toggles are disabled.
• Removes chromeVariant prop from PullRequestDetailPanel and its callers; condensed state is now derived solely from scroll position.
• Risk: terminalUiStateStore persistence version increment triggers migration for all existing users; label data not matching the new schema is silently dropped.

^{Macroscope summarized 69bbbf1.}

[coderabbitai]

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: dcb2d16b-de0c-4b21-b181-d99a892ed553

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ No successful main baseline artifact is available yet. This run establishes the initial measurement.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	—	12.6 KiB	—	15.1 KiB	✅
Codex	Thread snapshot wire	—	6.3 KiB	—	7.3 KiB	✅
Codex	Live turn WebSocket wire	—	6.3 KiB	—	7.8 KiB	✅
Codex	Live turn WebSocket decoded	—	51.8 KiB	—	66.4 KiB	✅
Codex	Live turn messages	—	16	—	21	✅
Claude	Total thread wire	—	12.6 KiB	—	15.1 KiB	✅
Claude	Thread snapshot wire	—	6.3 KiB	—	7.3 KiB	✅
Claude	Live turn WebSocket wire	—	6.3 KiB	—	7.8 KiB	✅
Claude	Live turn WebSocket decoded	—	52.7 KiB	—	66.4 KiB	✅
Claude	Live turn messages	—	16	—	21	✅

Baseline: unavailable · PR result: 69bbbf1 · 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: 99.9 KiB
• Claude decoded thread snapshot: 100.6 KiB

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

[macroscopeapp]

UI consistency review of the terminal drawer refresh. The migration to Button size="icon-xs" for the toolbar, the RightPanelResizeHandle/useResizableWidth reuse for the sidebar, and the tab-row treatment copied from RightPanelTabs all line up with the shared component system. Two tooltip-related consistency notes below; nothing else looked like a regression.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Approvability

Verdict: Not approved

Macroscope's review found this PR not approvable — This PR introduces new user-facing features including terminal renaming, a resizable sidebar, and stacked pull request detection. The scope and complexity of these new capabilities, combined with unresolved findings about stale label pruning and resize persistence behavior, warrant human review.

Not approved because:

• 2 blocking correctness issues found at or above your repo's Minimum Blocking Severity

^{Adjust the Minimum Blocking Severity for this repo — including turning it Off — in Settings. You can add or adjust custom eligibility rules. Learn more.}

[macroscopeapp]

Two new findings on the terminal sidebar chrome. The two tooltip findings from the previous run (native title where the sibling tab bar / compact toolbars use the shared Tooltip, and the unreachable split-limit wording on the disabled Button) are still open and unchanged.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

UI consistency review of the terminal drawer refresh. Two new findings on the added controls; previously flagged items (sidebar toolbar overflow at the 144px min width, per-instance useResizableWidth state sharing one storage key, native title instead of the shared Tooltip) still apply and are not repeated here.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

🟡 Medium

t3code/apps/web/src/components/ChatView.tsx

Line 790 in 69bbbf1

if(terminalIdListsEqual(serverOrderedTerminalIds,terminalUiState.terminalIds)){

Stale custom labels remain persisted when the server and drawer terminal IDs are equal, so a label for a deleted terminal is never pruned and can be reused when that ID is allocated again. The early return skips reconcileTerminalIds, which performs the label pruning; allow reconciliation to run in this case.

🚀 Reply "fix it for me" or copy this AI Prompt for your agent:

In file @apps/web/src/components/ChatView.tsx around line 790:
Stale custom labels remain persisted when the server and drawer terminal IDs are equal, so a label for a deleted terminal is never pruned and can be reused when that ID is allocated again. The early return skips `reconcileTerminalIds`, which performs the label pruning; allow reconciliation to run in this case.

[cursor]

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 69bbbf1. Configure here.}

[cursor]

Resize reverts if persist fails

Medium Severity

After a resize ends, dragWidth is cleared whenever the pointer is not down. Display then falls back to storedWidth. If setStoredWidth fails (quota or write error, which useLocalStorage swallows), the panel snaps back to the old width instead of keeping the drag result. Previously in-memory width still updated after a failed persist.

Additional Locations (1)

• apps/web/src/hooks/useResizableWidth.ts#L135-L143

^{Reviewed by Cursor Bugbot for commit 69bbbf1. Configure here.}

[macroscopeapp]

🟡 Mediumlib/openPullRequestLink.ts:126

changeRequestRepositoryUrl returns null for valid Azure URLs such as https://dev.azure.com/pull/123/_git/repo/pullrequest/42, and can return a truncated repository URL for similar paths. Its non-greedy fallback stops at the first /pull/<number>-like segment instead of the final PR suffix; extract the repository path using the last recognized provider suffix.

- const repositoryPath =- /^(.*?)\/\/-\/merge_requests\/\d+(?:\/|$)/iu.exec(url.pathname)?.[1] ??- /^(.*?)(?:\/pull\/\d+|\/-\/merge_requests\/\d+|\/pull-requests\/\d+|\/pullrequest\/\d+)(?:\/|$)/iu.exec(- url.pathname,- )?.[1];+ const repositoryPath = /^(.*)(?:\/pull\/\d+|\/-\/merge_requests\/\d+|\/pull-requests\/\d+|\/pullrequest\/\d+)(?:\/|$)/iu.exec(+ url.pathname,+ )?.[1];

🚀 Reply "fix it for me" or copy this AI Prompt for your agent:

In file @apps/web/src/lib/openPullRequestLink.ts around lines 126-130:
`changeRequestRepositoryUrl` returns `null` for valid Azure URLs such as `https://dev.azure.com/pull/123/_git/repo/pullrequest/42`, and can return a truncated repository URL for similar paths. Its non-greedy fallback stops at the first `/pull/<number>`-like segment instead of the final PR suffix; extract the repository path using the last recognized provider suffix.

[maria-rcks]

What changed

• Reworks the terminal drawer controls and multi-session sidebar while retaining split groups and resize behavior.
• Persists custom terminal names by scoped thread, prunes stale labels, and clears labels when terminals or threads disappear.
• Adds an explicit drawer hide path and keeps single-terminal sessions free of unnecessary sidebar chrome.

Screenshots

Direct parent on the left; this PR on the right. Same viewport and copied application state.

Before	After

## Why

Terminal state and interaction changes need a focused review. This layer depends only on the shared chat header from #7153.

Validation

• Terminal drawer tests
• Terminal UI store migration, naming, cleanup, reconciliation, and sidebar tests
• Affected-package typechecks
• Changed-file lint and formatting checks

Stack order

1. feat(web): unify workspace navigation #7153 workspace and navigation
2. feat(web): redesign usage insights #7147 Usage
3. feat(web): refresh pull request details #7148 Pull Requests
4. feat(web): improve terminal workspace controls #7149 terminal, this PR
5. feat(web): attach composer state drawers #7150 composer
6. fix(server): preserve tool lifecycle identity #7151 tool lifecycle projection
7. feat(web): collapse tool activity into one line #7152 tool activity UI

Built with GPT-5.6-sol in the Codex harness.

---

Note

Medium Risk
Touches persisted terminal UI state (migration v5) and server/client terminal ID reconciliation; behavior is covered by tests but wrong label pruning could affect multi-surface terminal UX.

Overview
Reworks the thread terminal drawer with a resizable session sidebar, persisted per-terminal names, and a toolbar action to hide the drawer without closing sessions.

The multi-terminal sidebar moves to the right of the viewport with a drag handle (useResizableWidth + separate localStorage keys for drawer vs panel). Width is capped so the terminal area keeps a minimum usable width. Toolbar actions switch from popover hover buttons to compact icon tooltips; drawer mode adds Hide (wired from ChatView via onHide).

Custom terminal labels are stored in terminalUiStateStore (persist v5): rename via double-click or F2, with logic so unchanged automatic titles are not saved as custom names. Labels apply in the drawer list and in ChatView composer terminal chips. Closing a terminal or pruning during reconcileTerminalIds drops stale labels; reconciliation can pass panel terminal ids so right-panel sessions keep names when drawer state syncs from the server.

Single-terminal layout keeps a floating compact toolbar only (no sidebar chrome).

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

Note

Improve terminal workspace controls with inline renaming, custom labels, and resizable sidebar

• Adds inline terminal renaming (double-click or F2) in ThreadTerminalDrawer, persisting custom labels up to 80 chars in terminalUiStateStore (version bumped 4→5 with migration).
• Introduces a resizable terminal sidebar with dynamic max-width clamping and per-context persisted widths; sidebar hides when only one terminal is open.
• Adds onHide callback to ThreadTerminalDrawer and wires it through ChatViewContent so the drawer can request to be hidden.
• Reworks PullRequestDetailPanel to prioritize conflict resolution in the primary action slot and adds a stacked-PR detection via isStackedPullRequestBase and a new listRefs query.
• Adds rightPanelUnavailableLabel prop to PanelLayoutControls and ensures tooltip triggers remain interactive when toggles are disabled.
• Removes chromeVariant prop from PullRequestDetailPanel and its callers; condensed state is now derived solely from scroll position.
• Risk: terminalUiStateStore persistence version increment triggers migration for all existing users; label data not matching the new schema is silently dropped.

^{Macroscope summarized 69bbbf1.}

[coderabbitai]

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: dcb2d16b-de0c-4b21-b181-d99a892ed553

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ No successful main baseline artifact is available yet. This run establishes the initial measurement.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	—	12.6 KiB	—	15.1 KiB	✅
Codex	Thread snapshot wire	—	6.3 KiB	—	7.3 KiB	✅
Codex	Live turn WebSocket wire	—	6.3 KiB	—	7.8 KiB	✅
Codex	Live turn WebSocket decoded	—	51.8 KiB	—	66.4 KiB	✅
Codex	Live turn messages	—	16	—	21	✅
Claude	Total thread wire	—	12.6 KiB	—	15.1 KiB	✅
Claude	Thread snapshot wire	—	6.3 KiB	—	7.3 KiB	✅
Claude	Live turn WebSocket wire	—	6.3 KiB	—	7.8 KiB	✅
Claude	Live turn WebSocket decoded	—	52.7 KiB	—	66.4 KiB	✅
Claude	Live turn messages	—	16	—	21	✅

Baseline: unavailable · PR result: 69bbbf1 · 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: 99.9 KiB
• Claude decoded thread snapshot: 100.6 KiB

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

[macroscopeapp]

UI consistency review of the terminal drawer refresh. The migration to Button size="icon-xs" for the toolbar, the RightPanelResizeHandle/useResizableWidth reuse for the sidebar, and the tab-row treatment copied from RightPanelTabs all line up with the shared component system. Two tooltip-related consistency notes below; nothing else looked like a regression.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Approvability

Verdict: Not approved

Macroscope's review found this PR not approvable — This PR introduces new user-facing features including terminal renaming, a resizable sidebar, and stacked pull request detection. The scope and complexity of these new capabilities, combined with unresolved findings about stale label pruning and resize persistence behavior, warrant human review.

Not approved because:

• 2 blocking correctness issues found at or above your repo's Minimum Blocking Severity

^{Adjust the Minimum Blocking Severity for this repo — including turning it Off — in Settings. You can add or adjust custom eligibility rules. Learn more.}

[macroscopeapp]

Two new findings on the terminal sidebar chrome. The two tooltip findings from the previous run (native title where the sibling tab bar / compact toolbars use the shared Tooltip, and the unreachable split-limit wording on the disabled Button) are still open and unchanged.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

UI consistency review of the terminal drawer refresh. Two new findings on the added controls; previously flagged items (sidebar toolbar overflow at the 144px min width, per-instance useResizableWidth state sharing one storage key, native title instead of the shared Tooltip) still apply and are not repeated here.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

🟡 Medium

t3code/apps/web/src/components/ChatView.tsx

Line 790 in 69bbbf1

if(terminalIdListsEqual(serverOrderedTerminalIds,terminalUiState.terminalIds)){

Stale custom labels remain persisted when the server and drawer terminal IDs are equal, so a label for a deleted terminal is never pruned and can be reused when that ID is allocated again. The early return skips reconcileTerminalIds, which performs the label pruning; allow reconciliation to run in this case.

🚀 Reply "fix it for me" or copy this AI Prompt for your agent:

In file @apps/web/src/components/ChatView.tsx around line 790:
Stale custom labels remain persisted when the server and drawer terminal IDs are equal, so a label for a deleted terminal is never pruned and can be reused when that ID is allocated again. The early return skips `reconcileTerminalIds`, which performs the label pruning; allow reconciliation to run in this case.

[cursor]

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 69bbbf1. Configure here.}

[cursor]

Resize reverts if persist fails

Medium Severity

After a resize ends, dragWidth is cleared whenever the pointer is not down. Display then falls back to storedWidth. If setStoredWidth fails (quota or write error, which useLocalStorage swallows), the panel snaps back to the old width instead of keeping the drag result. Previously in-memory width still updated after a failed persist.

Additional Locations (1)

• apps/web/src/hooks/useResizableWidth.ts#L135-L143

^{Reviewed by Cursor Bugbot for commit 69bbbf1. Configure here.}

[macroscopeapp]

🟡 Mediumlib/openPullRequestLink.ts:126

changeRequestRepositoryUrl returns null for valid Azure URLs such as https://dev.azure.com/pull/123/_git/repo/pullrequest/42, and can return a truncated repository URL for similar paths. Its non-greedy fallback stops at the first /pull/<number>-like segment instead of the final PR suffix; extract the repository path using the last recognized provider suffix.

- const repositoryPath =- /^(.*?)\/\/-\/merge_requests\/\d+(?:\/|$)/iu.exec(url.pathname)?.[1] ??- /^(.*?)(?:\/pull\/\d+|\/-\/merge_requests\/\d+|\/pull-requests\/\d+|\/pullrequest\/\d+)(?:\/|$)/iu.exec(- url.pathname,- )?.[1];+ const repositoryPath = /^(.*)(?:\/pull\/\d+|\/-\/merge_requests\/\d+|\/pull-requests\/\d+|\/pullrequest\/\d+)(?:\/|$)/iu.exec(+ url.pathname,+ )?.[1];

🚀 Reply "fix it for me" or copy this AI Prompt for your agent:

In file @apps/web/src/lib/openPullRequestLink.ts around lines 126-130:
`changeRequestRepositoryUrl` returns `null` for valid Azure URLs such as `https://dev.azure.com/pull/123/_git/repo/pullrequest/42`, and can return a truncated repository URL for similar paths. Its non-greedy fallback stops at the first `/pull/<number>`-like segment instead of the final PR suffix; extract the repository path using the last recognized provider suffix.

[maria-rcks]

What changed

• Reworks the terminal drawer controls and multi-session sidebar while retaining split groups and resize behavior.
• Persists custom terminal names by scoped thread, prunes stale labels, and clears labels when terminals or threads disappear.
• Adds an explicit drawer hide path and keeps single-terminal sessions free of unnecessary sidebar chrome.

Screenshots

Direct parent on the left; this PR on the right. Same viewport and copied application state.

Before	After

## Why

Terminal state and interaction changes need a focused review. This layer depends only on the shared chat header from #7153.

Validation

• Terminal drawer tests
• Terminal UI store migration, naming, cleanup, reconciliation, and sidebar tests
• Affected-package typechecks
• Changed-file lint and formatting checks

Stack order

1. feat(web): unify workspace navigation #7153 workspace and navigation
2. feat(web): redesign usage insights #7147 Usage
3. feat(web): refresh pull request details #7148 Pull Requests
4. feat(web): improve terminal workspace controls #7149 terminal, this PR
5. feat(web): attach composer state drawers #7150 composer
6. fix(server): preserve tool lifecycle identity #7151 tool lifecycle projection
7. feat(web): collapse tool activity into one line #7152 tool activity UI

Built with GPT-5.6-sol in the Codex harness.

---

Note

Medium Risk
Touches persisted terminal UI state (migration v5) and server/client terminal ID reconciliation; behavior is covered by tests but wrong label pruning could affect multi-surface terminal UX.

Overview
Reworks the thread terminal drawer with a resizable session sidebar, persisted per-terminal names, and a toolbar action to hide the drawer without closing sessions.

The multi-terminal sidebar moves to the right of the viewport with a drag handle (useResizableWidth + separate localStorage keys for drawer vs panel). Width is capped so the terminal area keeps a minimum usable width. Toolbar actions switch from popover hover buttons to compact icon tooltips; drawer mode adds Hide (wired from ChatView via onHide).

Custom terminal labels are stored in terminalUiStateStore (persist v5): rename via double-click or F2, with logic so unchanged automatic titles are not saved as custom names. Labels apply in the drawer list and in ChatView composer terminal chips. Closing a terminal or pruning during reconcileTerminalIds drops stale labels; reconciliation can pass panel terminal ids so right-panel sessions keep names when drawer state syncs from the server.

Single-terminal layout keeps a floating compact toolbar only (no sidebar chrome).

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

Note

Improve terminal workspace controls with inline renaming, custom labels, and resizable sidebar

• Adds inline terminal renaming (double-click or F2) in ThreadTerminalDrawer, persisting custom labels up to 80 chars in terminalUiStateStore (version bumped 4→5 with migration).
• Introduces a resizable terminal sidebar with dynamic max-width clamping and per-context persisted widths; sidebar hides when only one terminal is open.
• Adds onHide callback to ThreadTerminalDrawer and wires it through ChatViewContent so the drawer can request to be hidden.
• Reworks PullRequestDetailPanel to prioritize conflict resolution in the primary action slot and adds a stacked-PR detection via isStackedPullRequestBase and a new listRefs query.
• Adds rightPanelUnavailableLabel prop to PanelLayoutControls and ensures tooltip triggers remain interactive when toggles are disabled.
• Removes chromeVariant prop from PullRequestDetailPanel and its callers; condensed state is now derived solely from scroll position.
• Risk: terminalUiStateStore persistence version increment triggers migration for all existing users; label data not matching the new schema is silently dropped.

^{Macroscope summarized 69bbbf1.}

[coderabbitai]

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: dcb2d16b-de0c-4b21-b181-d99a892ed553

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ No successful main baseline artifact is available yet. This run establishes the initial measurement.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	—	12.6 KiB	—	15.1 KiB	✅
Codex	Thread snapshot wire	—	6.3 KiB	—	7.3 KiB	✅
Codex	Live turn WebSocket wire	—	6.3 KiB	—	7.8 KiB	✅
Codex	Live turn WebSocket decoded	—	51.8 KiB	—	66.4 KiB	✅
Codex	Live turn messages	—	16	—	21	✅
Claude	Total thread wire	—	12.6 KiB	—	15.1 KiB	✅
Claude	Thread snapshot wire	—	6.3 KiB	—	7.3 KiB	✅
Claude	Live turn WebSocket wire	—	6.3 KiB	—	7.8 KiB	✅
Claude	Live turn WebSocket decoded	—	52.7 KiB	—	66.4 KiB	✅
Claude	Live turn messages	—	16	—	21	✅

Baseline: unavailable · PR result: 69bbbf1 · 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: 99.9 KiB
• Claude decoded thread snapshot: 100.6 KiB

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

[macroscopeapp]

UI consistency review of the terminal drawer refresh. The migration to Button size="icon-xs" for the toolbar, the RightPanelResizeHandle/useResizableWidth reuse for the sidebar, and the tab-row treatment copied from RightPanelTabs all line up with the shared component system. Two tooltip-related consistency notes below; nothing else looked like a regression.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Approvability

Verdict: Not approved

Macroscope's review found this PR not approvable — This PR introduces new user-facing features including terminal renaming, a resizable sidebar, and stacked pull request detection. The scope and complexity of these new capabilities, combined with unresolved findings about stale label pruning and resize persistence behavior, warrant human review.

Not approved because:

• 2 blocking correctness issues found at or above your repo's Minimum Blocking Severity

^{Adjust the Minimum Blocking Severity for this repo — including turning it Off — in Settings. You can add or adjust custom eligibility rules. Learn more.}

[macroscopeapp]

Two new findings on the terminal sidebar chrome. The two tooltip findings from the previous run (native title where the sibling tab bar / compact toolbars use the shared Tooltip, and the unreachable split-limit wording on the disabled Button) are still open and unchanged.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

UI consistency review of the terminal drawer refresh. Two new findings on the added controls; previously flagged items (sidebar toolbar overflow at the 144px min width, per-instance useResizableWidth state sharing one storage key, native title instead of the shared Tooltip) still apply and are not repeated here.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

🟡 Medium

t3code/apps/web/src/components/ChatView.tsx

Line 790 in 69bbbf1

if(terminalIdListsEqual(serverOrderedTerminalIds,terminalUiState.terminalIds)){

Stale custom labels remain persisted when the server and drawer terminal IDs are equal, so a label for a deleted terminal is never pruned and can be reused when that ID is allocated again. The early return skips reconcileTerminalIds, which performs the label pruning; allow reconciliation to run in this case.

🚀 Reply "fix it for me" or copy this AI Prompt for your agent:

In file @apps/web/src/components/ChatView.tsx around line 790:
Stale custom labels remain persisted when the server and drawer terminal IDs are equal, so a label for a deleted terminal is never pruned and can be reused when that ID is allocated again. The early return skips `reconcileTerminalIds`, which performs the label pruning; allow reconciliation to run in this case.

[cursor]

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 69bbbf1. Configure here.}

[cursor]

Resize reverts if persist fails

Medium Severity

After a resize ends, dragWidth is cleared whenever the pointer is not down. Display then falls back to storedWidth. If setStoredWidth fails (quota or write error, which useLocalStorage swallows), the panel snaps back to the old width instead of keeping the drag result. Previously in-memory width still updated after a failed persist.

Additional Locations (1)

• apps/web/src/hooks/useResizableWidth.ts#L135-L143

^{Reviewed by Cursor Bugbot for commit 69bbbf1. Configure here.}

[macroscopeapp]

🟡 Mediumlib/openPullRequestLink.ts:126

changeRequestRepositoryUrl returns null for valid Azure URLs such as https://dev.azure.com/pull/123/_git/repo/pullrequest/42, and can return a truncated repository URL for similar paths. Its non-greedy fallback stops at the first /pull/<number>-like segment instead of the final PR suffix; extract the repository path using the last recognized provider suffix.

- const repositoryPath =- /^(.*?)\/\/-\/merge_requests\/\d+(?:\/|$)/iu.exec(url.pathname)?.[1] ??- /^(.*?)(?:\/pull\/\d+|\/-\/merge_requests\/\d+|\/pull-requests\/\d+|\/pullrequest\/\d+)(?:\/|$)/iu.exec(- url.pathname,- )?.[1];+ const repositoryPath = /^(.*)(?:\/pull\/\d+|\/-\/merge_requests\/\d+|\/pull-requests\/\d+|\/pullrequest\/\d+)(?:\/|$)/iu.exec(+ url.pathname,+ )?.[1];

🚀 Reply "fix it for me" or copy this AI Prompt for your agent:

In file @apps/web/src/lib/openPullRequestLink.ts around lines 126-130:
`changeRequestRepositoryUrl` returns `null` for valid Azure URLs such as `https://dev.azure.com/pull/123/_git/repo/pullrequest/42`, and can return a truncated repository URL for similar paths. Its non-greedy fallback stops at the first `/pull/<number>`-like segment instead of the final PR suffix; extract the repository path using the last recognized provider suffix.

[maria-rcks]

What changed

• Reworks the terminal drawer controls and multi-session sidebar while retaining split groups and resize behavior.
• Persists custom terminal names by scoped thread, prunes stale labels, and clears labels when terminals or threads disappear.
• Adds an explicit drawer hide path and keeps single-terminal sessions free of unnecessary sidebar chrome.

Screenshots

Direct parent on the left; this PR on the right. Same viewport and copied application state.

Before	After

## Why

Terminal state and interaction changes need a focused review. This layer depends only on the shared chat header from #7153.

Validation

• Terminal drawer tests
• Terminal UI store migration, naming, cleanup, reconciliation, and sidebar tests
• Affected-package typechecks
• Changed-file lint and formatting checks

Stack order

1. feat(web): unify workspace navigation #7153 workspace and navigation
2. feat(web): redesign usage insights #7147 Usage
3. feat(web): refresh pull request details #7148 Pull Requests
4. feat(web): improve terminal workspace controls #7149 terminal, this PR
5. feat(web): attach composer state drawers #7150 composer
6. fix(server): preserve tool lifecycle identity #7151 tool lifecycle projection
7. feat(web): collapse tool activity into one line #7152 tool activity UI

Built with GPT-5.6-sol in the Codex harness.

---

Note

Medium Risk
Touches persisted terminal UI state (migration v5) and server/client terminal ID reconciliation; behavior is covered by tests but wrong label pruning could affect multi-surface terminal UX.

Overview
Reworks the thread terminal drawer with a resizable session sidebar, persisted per-terminal names, and a toolbar action to hide the drawer without closing sessions.

The multi-terminal sidebar moves to the right of the viewport with a drag handle (useResizableWidth + separate localStorage keys for drawer vs panel). Width is capped so the terminal area keeps a minimum usable width. Toolbar actions switch from popover hover buttons to compact icon tooltips; drawer mode adds Hide (wired from ChatView via onHide).

Custom terminal labels are stored in terminalUiStateStore (persist v5): rename via double-click or F2, with logic so unchanged automatic titles are not saved as custom names. Labels apply in the drawer list and in ChatView composer terminal chips. Closing a terminal or pruning during reconcileTerminalIds drops stale labels; reconciliation can pass panel terminal ids so right-panel sessions keep names when drawer state syncs from the server.

Single-terminal layout keeps a floating compact toolbar only (no sidebar chrome).

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

Note

Improve terminal workspace controls with inline renaming, custom labels, and resizable sidebar

• Adds inline terminal renaming (double-click or F2) in ThreadTerminalDrawer, persisting custom labels up to 80 chars in terminalUiStateStore (version bumped 4→5 with migration).
• Introduces a resizable terminal sidebar with dynamic max-width clamping and per-context persisted widths; sidebar hides when only one terminal is open.
• Adds onHide callback to ThreadTerminalDrawer and wires it through ChatViewContent so the drawer can request to be hidden.
• Reworks PullRequestDetailPanel to prioritize conflict resolution in the primary action slot and adds a stacked-PR detection via isStackedPullRequestBase and a new listRefs query.
• Adds rightPanelUnavailableLabel prop to PanelLayoutControls and ensures tooltip triggers remain interactive when toggles are disabled.
• Removes chromeVariant prop from PullRequestDetailPanel and its callers; condensed state is now derived solely from scroll position.
• Risk: terminalUiStateStore persistence version increment triggers migration for all existing users; label data not matching the new schema is silently dropped.

^{Macroscope summarized 69bbbf1.}

[coderabbitai]

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: dcb2d16b-de0c-4b21-b181-d99a892ed553

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ No successful main baseline artifact is available yet. This run establishes the initial measurement.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	—	12.6 KiB	—	15.1 KiB	✅
Codex	Thread snapshot wire	—	6.3 KiB	—	7.3 KiB	✅
Codex	Live turn WebSocket wire	—	6.3 KiB	—	7.8 KiB	✅
Codex	Live turn WebSocket decoded	—	51.8 KiB	—	66.4 KiB	✅
Codex	Live turn messages	—	16	—	21	✅
Claude	Total thread wire	—	12.6 KiB	—	15.1 KiB	✅
Claude	Thread snapshot wire	—	6.3 KiB	—	7.3 KiB	✅
Claude	Live turn WebSocket wire	—	6.3 KiB	—	7.8 KiB	✅
Claude	Live turn WebSocket decoded	—	52.7 KiB	—	66.4 KiB	✅
Claude	Live turn messages	—	16	—	21	✅

Baseline: unavailable · PR result: 69bbbf1 · 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: 99.9 KiB
• Claude decoded thread snapshot: 100.6 KiB

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

[macroscopeapp]

UI consistency review of the terminal drawer refresh. The migration to Button size="icon-xs" for the toolbar, the RightPanelResizeHandle/useResizableWidth reuse for the sidebar, and the tab-row treatment copied from RightPanelTabs all line up with the shared component system. Two tooltip-related consistency notes below; nothing else looked like a regression.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Approvability

Verdict: Not approved

Macroscope's review found this PR not approvable — This PR introduces new user-facing features including terminal renaming, a resizable sidebar, and stacked pull request detection. The scope and complexity of these new capabilities, combined with unresolved findings about stale label pruning and resize persistence behavior, warrant human review.

Not approved because:

• 2 blocking correctness issues found at or above your repo's Minimum Blocking Severity

^{Adjust the Minimum Blocking Severity for this repo — including turning it Off — in Settings. You can add or adjust custom eligibility rules. Learn more.}

[macroscopeapp]

Two new findings on the terminal sidebar chrome. The two tooltip findings from the previous run (native title where the sibling tab bar / compact toolbars use the shared Tooltip, and the unreachable split-limit wording on the disabled Button) are still open and unchanged.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

UI consistency review of the terminal drawer refresh. Two new findings on the added controls; previously flagged items (sidebar toolbar overflow at the 144px min width, per-instance useResizableWidth state sharing one storage key, native title instead of the shared Tooltip) still apply and are not repeated here.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

🟡 Medium

t3code/apps/web/src/components/ChatView.tsx

Line 790 in 69bbbf1

if(terminalIdListsEqual(serverOrderedTerminalIds,terminalUiState.terminalIds)){

Stale custom labels remain persisted when the server and drawer terminal IDs are equal, so a label for a deleted terminal is never pruned and can be reused when that ID is allocated again. The early return skips reconcileTerminalIds, which performs the label pruning; allow reconciliation to run in this case.

🚀 Reply "fix it for me" or copy this AI Prompt for your agent:

In file @apps/web/src/components/ChatView.tsx around line 790:
Stale custom labels remain persisted when the server and drawer terminal IDs are equal, so a label for a deleted terminal is never pruned and can be reused when that ID is allocated again. The early return skips `reconcileTerminalIds`, which performs the label pruning; allow reconciliation to run in this case.

[cursor]

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 69bbbf1. Configure here.}

[cursor]

Resize reverts if persist fails

Medium Severity

After a resize ends, dragWidth is cleared whenever the pointer is not down. Display then falls back to storedWidth. If setStoredWidth fails (quota or write error, which useLocalStorage swallows), the panel snaps back to the old width instead of keeping the drag result. Previously in-memory width still updated after a failed persist.

Additional Locations (1)

• apps/web/src/hooks/useResizableWidth.ts#L135-L143

^{Reviewed by Cursor Bugbot for commit 69bbbf1. Configure here.}

[macroscopeapp]

🟡 Mediumlib/openPullRequestLink.ts:126

changeRequestRepositoryUrl returns null for valid Azure URLs such as https://dev.azure.com/pull/123/_git/repo/pullrequest/42, and can return a truncated repository URL for similar paths. Its non-greedy fallback stops at the first /pull/<number>-like segment instead of the final PR suffix; extract the repository path using the last recognized provider suffix.

- const repositoryPath =- /^(.*?)\/\/-\/merge_requests\/\d+(?:\/|$)/iu.exec(url.pathname)?.[1] ??- /^(.*?)(?:\/pull\/\d+|\/-\/merge_requests\/\d+|\/pull-requests\/\d+|\/pullrequest\/\d+)(?:\/|$)/iu.exec(- url.pathname,- )?.[1];+ const repositoryPath = /^(.*)(?:\/pull\/\d+|\/-\/merge_requests\/\d+|\/pull-requests\/\d+|\/pullrequest\/\d+)(?:\/|$)/iu.exec(+ url.pathname,+ )?.[1];

🚀 Reply "fix it for me" or copy this AI Prompt for your agent:

In file @apps/web/src/lib/openPullRequestLink.ts around lines 126-130:
`changeRequestRepositoryUrl` returns `null` for valid Azure URLs such as `https://dev.azure.com/pull/123/_git/repo/pullrequest/42`, and can return a truncated repository URL for similar paths. Its non-greedy fallback stops at the first `/pull/<number>`-like segment instead of the final PR suffix; extract the repository path using the last recognized provider suffix.

[maria-rcks]

What changed

• Reworks the terminal drawer controls and multi-session sidebar while retaining split groups and resize behavior.
• Persists custom terminal names by scoped thread, prunes stale labels, and clears labels when terminals or threads disappear.
• Adds an explicit drawer hide path and keeps single-terminal sessions free of unnecessary sidebar chrome.

Screenshots

Direct parent on the left; this PR on the right. Same viewport and copied application state.

Before	After

## Why

Terminal state and interaction changes need a focused review. This layer depends only on the shared chat header from #7153.

Validation

• Terminal drawer tests
• Terminal UI store migration, naming, cleanup, reconciliation, and sidebar tests
• Affected-package typechecks
• Changed-file lint and formatting checks

Stack order

1. feat(web): unify workspace navigation #7153 workspace and navigation
2. feat(web): redesign usage insights #7147 Usage
3. feat(web): refresh pull request details #7148 Pull Requests
4. feat(web): improve terminal workspace controls #7149 terminal, this PR
5. feat(web): attach composer state drawers #7150 composer
6. fix(server): preserve tool lifecycle identity #7151 tool lifecycle projection
7. feat(web): collapse tool activity into one line #7152 tool activity UI

Built with GPT-5.6-sol in the Codex harness.

---

Note

Medium Risk
Touches persisted terminal UI state (migration v5) and server/client terminal ID reconciliation; behavior is covered by tests but wrong label pruning could affect multi-surface terminal UX.

Overview
Reworks the thread terminal drawer with a resizable session sidebar, persisted per-terminal names, and a toolbar action to hide the drawer without closing sessions.

The multi-terminal sidebar moves to the right of the viewport with a drag handle (useResizableWidth + separate localStorage keys for drawer vs panel). Width is capped so the terminal area keeps a minimum usable width. Toolbar actions switch from popover hover buttons to compact icon tooltips; drawer mode adds Hide (wired from ChatView via onHide).

Custom terminal labels are stored in terminalUiStateStore (persist v5): rename via double-click or F2, with logic so unchanged automatic titles are not saved as custom names. Labels apply in the drawer list and in ChatView composer terminal chips. Closing a terminal or pruning during reconcileTerminalIds drops stale labels; reconciliation can pass panel terminal ids so right-panel sessions keep names when drawer state syncs from the server.

Single-terminal layout keeps a floating compact toolbar only (no sidebar chrome).

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

Note

Improve terminal workspace controls with inline renaming, custom labels, and resizable sidebar

• Adds inline terminal renaming (double-click or F2) in ThreadTerminalDrawer, persisting custom labels up to 80 chars in terminalUiStateStore (version bumped 4→5 with migration).
• Introduces a resizable terminal sidebar with dynamic max-width clamping and per-context persisted widths; sidebar hides when only one terminal is open.
• Adds onHide callback to ThreadTerminalDrawer and wires it through ChatViewContent so the drawer can request to be hidden.
• Reworks PullRequestDetailPanel to prioritize conflict resolution in the primary action slot and adds a stacked-PR detection via isStackedPullRequestBase and a new listRefs query.
• Adds rightPanelUnavailableLabel prop to PanelLayoutControls and ensures tooltip triggers remain interactive when toggles are disabled.
• Removes chromeVariant prop from PullRequestDetailPanel and its callers; condensed state is now derived solely from scroll position.
• Risk: terminalUiStateStore persistence version increment triggers migration for all existing users; label data not matching the new schema is silently dropped.

^{Macroscope summarized 69bbbf1.}

[coderabbitai]

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: dcb2d16b-de0c-4b21-b181-d99a892ed553

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ No successful main baseline artifact is available yet. This run establishes the initial measurement.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	—	12.6 KiB	—	15.1 KiB	✅
Codex	Thread snapshot wire	—	6.3 KiB	—	7.3 KiB	✅
Codex	Live turn WebSocket wire	—	6.3 KiB	—	7.8 KiB	✅
Codex	Live turn WebSocket decoded	—	51.8 KiB	—	66.4 KiB	✅
Codex	Live turn messages	—	16	—	21	✅
Claude	Total thread wire	—	12.6 KiB	—	15.1 KiB	✅
Claude	Thread snapshot wire	—	6.3 KiB	—	7.3 KiB	✅
Claude	Live turn WebSocket wire	—	6.3 KiB	—	7.8 KiB	✅
Claude	Live turn WebSocket decoded	—	52.7 KiB	—	66.4 KiB	✅
Claude	Live turn messages	—	16	—	21	✅

Baseline: unavailable · PR result: 69bbbf1 · 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: 99.9 KiB
• Claude decoded thread snapshot: 100.6 KiB

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

[macroscopeapp]

UI consistency review of the terminal drawer refresh. The migration to Button size="icon-xs" for the toolbar, the RightPanelResizeHandle/useResizableWidth reuse for the sidebar, and the tab-row treatment copied from RightPanelTabs all line up with the shared component system. Two tooltip-related consistency notes below; nothing else looked like a regression.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Approvability

Verdict: Not approved

Macroscope's review found this PR not approvable — This PR introduces new user-facing features including terminal renaming, a resizable sidebar, and stacked pull request detection. The scope and complexity of these new capabilities, combined with unresolved findings about stale label pruning and resize persistence behavior, warrant human review.

Not approved because:

• 2 blocking correctness issues found at or above your repo's Minimum Blocking Severity

^{Adjust the Minimum Blocking Severity for this repo — including turning it Off — in Settings. You can add or adjust custom eligibility rules. Learn more.}

[macroscopeapp]

Two new findings on the terminal sidebar chrome. The two tooltip findings from the previous run (native title where the sibling tab bar / compact toolbars use the shared Tooltip, and the unreachable split-limit wording on the disabled Button) are still open and unchanged.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

UI consistency review of the terminal drawer refresh. Two new findings on the added controls; previously flagged items (sidebar toolbar overflow at the 144px min width, per-instance useResizableWidth state sharing one storage key, native title instead of the shared Tooltip) still apply and are not repeated here.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

🟡 Medium

t3code/apps/web/src/components/ChatView.tsx

Line 790 in 69bbbf1

if(terminalIdListsEqual(serverOrderedTerminalIds,terminalUiState.terminalIds)){

Stale custom labels remain persisted when the server and drawer terminal IDs are equal, so a label for a deleted terminal is never pruned and can be reused when that ID is allocated again. The early return skips reconcileTerminalIds, which performs the label pruning; allow reconciliation to run in this case.

🚀 Reply "fix it for me" or copy this AI Prompt for your agent:

In file @apps/web/src/components/ChatView.tsx around line 790:
Stale custom labels remain persisted when the server and drawer terminal IDs are equal, so a label for a deleted terminal is never pruned and can be reused when that ID is allocated again. The early return skips `reconcileTerminalIds`, which performs the label pruning; allow reconciliation to run in this case.

[cursor]

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 69bbbf1. Configure here.}

[cursor]

Resize reverts if persist fails

Medium Severity

After a resize ends, dragWidth is cleared whenever the pointer is not down. Display then falls back to storedWidth. If setStoredWidth fails (quota or write error, which useLocalStorage swallows), the panel snaps back to the old width instead of keeping the drag result. Previously in-memory width still updated after a failed persist.

Additional Locations (1)

• apps/web/src/hooks/useResizableWidth.ts#L135-L143

^{Reviewed by Cursor Bugbot for commit 69bbbf1. Configure here.}

[macroscopeapp]

🟡 Mediumlib/openPullRequestLink.ts:126

changeRequestRepositoryUrl returns null for valid Azure URLs such as https://dev.azure.com/pull/123/_git/repo/pullrequest/42, and can return a truncated repository URL for similar paths. Its non-greedy fallback stops at the first /pull/<number>-like segment instead of the final PR suffix; extract the repository path using the last recognized provider suffix.

- const repositoryPath =- /^(.*?)\/\/-\/merge_requests\/\d+(?:\/|$)/iu.exec(url.pathname)?.[1] ??- /^(.*?)(?:\/pull\/\d+|\/-\/merge_requests\/\d+|\/pull-requests\/\d+|\/pullrequest\/\d+)(?:\/|$)/iu.exec(- url.pathname,- )?.[1];+ const repositoryPath = /^(.*)(?:\/pull\/\d+|\/-\/merge_requests\/\d+|\/pull-requests\/\d+|\/pullrequest\/\d+)(?:\/|$)/iu.exec(+ url.pathname,+ )?.[1];

🚀 Reply "fix it for me" or copy this AI Prompt for your agent:

In file @apps/web/src/lib/openPullRequestLink.ts around lines 126-130:
`changeRequestRepositoryUrl` returns `null` for valid Azure URLs such as `https://dev.azure.com/pull/123/_git/repo/pullrequest/42`, and can return a truncated repository URL for similar paths. Its non-greedy fallback stops at the first `/pull/<number>`-like segment instead of the final PR suffix; extract the repository path using the last recognized provider suffix.

[maria-rcks]

What changed

• Reworks the terminal drawer controls and multi-session sidebar while retaining split groups and resize behavior.
• Persists custom terminal names by scoped thread, prunes stale labels, and clears labels when terminals or threads disappear.
• Adds an explicit drawer hide path and keeps single-terminal sessions free of unnecessary sidebar chrome.

Screenshots

Direct parent on the left; this PR on the right. Same viewport and copied application state.

Before	After

## Why

Terminal state and interaction changes need a focused review. This layer depends only on the shared chat header from #7153.

Validation

• Terminal drawer tests
• Terminal UI store migration, naming, cleanup, reconciliation, and sidebar tests
• Affected-package typechecks
• Changed-file lint and formatting checks

Stack order

1. feat(web): unify workspace navigation #7153 workspace and navigation
2. feat(web): redesign usage insights #7147 Usage
3. feat(web): refresh pull request details #7148 Pull Requests
4. feat(web): improve terminal workspace controls #7149 terminal, this PR
5. feat(web): attach composer state drawers #7150 composer
6. fix(server): preserve tool lifecycle identity #7151 tool lifecycle projection
7. feat(web): collapse tool activity into one line #7152 tool activity UI

Built with GPT-5.6-sol in the Codex harness.

---

Note

Medium Risk
Touches persisted terminal UI state (migration v5) and server/client terminal ID reconciliation; behavior is covered by tests but wrong label pruning could affect multi-surface terminal UX.

Overview
Reworks the thread terminal drawer with a resizable session sidebar, persisted per-terminal names, and a toolbar action to hide the drawer without closing sessions.

The multi-terminal sidebar moves to the right of the viewport with a drag handle (useResizableWidth + separate localStorage keys for drawer vs panel). Width is capped so the terminal area keeps a minimum usable width. Toolbar actions switch from popover hover buttons to compact icon tooltips; drawer mode adds Hide (wired from ChatView via onHide).

Custom terminal labels are stored in terminalUiStateStore (persist v5): rename via double-click or F2, with logic so unchanged automatic titles are not saved as custom names. Labels apply in the drawer list and in ChatView composer terminal chips. Closing a terminal or pruning during reconcileTerminalIds drops stale labels; reconciliation can pass panel terminal ids so right-panel sessions keep names when drawer state syncs from the server.

Single-terminal layout keeps a floating compact toolbar only (no sidebar chrome).

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

Note

Improve terminal workspace controls with inline renaming, custom labels, and resizable sidebar

• Adds inline terminal renaming (double-click or F2) in ThreadTerminalDrawer, persisting custom labels up to 80 chars in terminalUiStateStore (version bumped 4→5 with migration).
• Introduces a resizable terminal sidebar with dynamic max-width clamping and per-context persisted widths; sidebar hides when only one terminal is open.
• Adds onHide callback to ThreadTerminalDrawer and wires it through ChatViewContent so the drawer can request to be hidden.
• Reworks PullRequestDetailPanel to prioritize conflict resolution in the primary action slot and adds a stacked-PR detection via isStackedPullRequestBase and a new listRefs query.
• Adds rightPanelUnavailableLabel prop to PanelLayoutControls and ensures tooltip triggers remain interactive when toggles are disabled.
• Removes chromeVariant prop from PullRequestDetailPanel and its callers; condensed state is now derived solely from scroll position.
• Risk: terminalUiStateStore persistence version increment triggers migration for all existing users; label data not matching the new schema is silently dropped.

^{Macroscope summarized 69bbbf1.}

[coderabbitai]

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: dcb2d16b-de0c-4b21-b181-d99a892ed553

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ No successful main baseline artifact is available yet. This run establishes the initial measurement.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	—	12.6 KiB	—	15.1 KiB	✅
Codex	Thread snapshot wire	—	6.3 KiB	—	7.3 KiB	✅
Codex	Live turn WebSocket wire	—	6.3 KiB	—	7.8 KiB	✅
Codex	Live turn WebSocket decoded	—	51.8 KiB	—	66.4 KiB	✅
Codex	Live turn messages	—	16	—	21	✅
Claude	Total thread wire	—	12.6 KiB	—	15.1 KiB	✅
Claude	Thread snapshot wire	—	6.3 KiB	—	7.3 KiB	✅
Claude	Live turn WebSocket wire	—	6.3 KiB	—	7.8 KiB	✅
Claude	Live turn WebSocket decoded	—	52.7 KiB	—	66.4 KiB	✅
Claude	Live turn messages	—	16	—	21	✅

Baseline: unavailable · PR result: 69bbbf1 · 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: 99.9 KiB
• Claude decoded thread snapshot: 100.6 KiB

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

[macroscopeapp]

UI consistency review of the terminal drawer refresh. The migration to Button size="icon-xs" for the toolbar, the RightPanelResizeHandle/useResizableWidth reuse for the sidebar, and the tab-row treatment copied from RightPanelTabs all line up with the shared component system. Two tooltip-related consistency notes below; nothing else looked like a regression.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Approvability

Verdict: Not approved

Macroscope's review found this PR not approvable — This PR introduces new user-facing features including terminal renaming, a resizable sidebar, and stacked pull request detection. The scope and complexity of these new capabilities, combined with unresolved findings about stale label pruning and resize persistence behavior, warrant human review.

Not approved because:

• 2 blocking correctness issues found at or above your repo's Minimum Blocking Severity

^{Adjust the Minimum Blocking Severity for this repo — including turning it Off — in Settings. You can add or adjust custom eligibility rules. Learn more.}

[macroscopeapp]

Two new findings on the terminal sidebar chrome. The two tooltip findings from the previous run (native title where the sibling tab bar / compact toolbars use the shared Tooltip, and the unreachable split-limit wording on the disabled Button) are still open and unchanged.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

UI consistency review of the terminal drawer refresh. Two new findings on the added controls; previously flagged items (sidebar toolbar overflow at the 144px min width, per-instance useResizableWidth state sharing one storage key, native title instead of the shared Tooltip) still apply and are not repeated here.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

🟡 Medium

t3code/apps/web/src/components/ChatView.tsx

Line 790 in 69bbbf1

if(terminalIdListsEqual(serverOrderedTerminalIds,terminalUiState.terminalIds)){

Stale custom labels remain persisted when the server and drawer terminal IDs are equal, so a label for a deleted terminal is never pruned and can be reused when that ID is allocated again. The early return skips reconcileTerminalIds, which performs the label pruning; allow reconciliation to run in this case.

🚀 Reply "fix it for me" or copy this AI Prompt for your agent:

In file @apps/web/src/components/ChatView.tsx around line 790:
Stale custom labels remain persisted when the server and drawer terminal IDs are equal, so a label for a deleted terminal is never pruned and can be reused when that ID is allocated again. The early return skips `reconcileTerminalIds`, which performs the label pruning; allow reconciliation to run in this case.

[cursor]

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 69bbbf1. Configure here.}

[cursor]

Resize reverts if persist fails

Medium Severity

After a resize ends, dragWidth is cleared whenever the pointer is not down. Display then falls back to storedWidth. If setStoredWidth fails (quota or write error, which useLocalStorage swallows), the panel snaps back to the old width instead of keeping the drag result. Previously in-memory width still updated after a failed persist.

Additional Locations (1)

• apps/web/src/hooks/useResizableWidth.ts#L135-L143

^{Reviewed by Cursor Bugbot for commit 69bbbf1. Configure here.}

[macroscopeapp]

🟡 Mediumlib/openPullRequestLink.ts:126

changeRequestRepositoryUrl returns null for valid Azure URLs such as https://dev.azure.com/pull/123/_git/repo/pullrequest/42, and can return a truncated repository URL for similar paths. Its non-greedy fallback stops at the first /pull/<number>-like segment instead of the final PR suffix; extract the repository path using the last recognized provider suffix.

- const repositoryPath =- /^(.*?)\/\/-\/merge_requests\/\d+(?:\/|$)/iu.exec(url.pathname)?.[1] ??- /^(.*?)(?:\/pull\/\d+|\/-\/merge_requests\/\d+|\/pull-requests\/\d+|\/pullrequest\/\d+)(?:\/|$)/iu.exec(- url.pathname,- )?.[1];+ const repositoryPath = /^(.*)(?:\/pull\/\d+|\/-\/merge_requests\/\d+|\/pull-requests\/\d+|\/pullrequest\/\d+)(?:\/|$)/iu.exec(+ url.pathname,+ )?.[1];

🚀 Reply "fix it for me" or copy this AI Prompt for your agent:

In file @apps/web/src/lib/openPullRequestLink.ts around lines 126-130:
`changeRequestRepositoryUrl` returns `null` for valid Azure URLs such as `https://dev.azure.com/pull/123/_git/repo/pullrequest/42`, and can return a truncated repository URL for similar paths. Its non-greedy fallback stops at the first `/pull/<number>`-like segment instead of the final PR suffix; extract the repository path using the last recognized provider suffix.

[maria-rcks]

What changed

• Reworks the terminal drawer controls and multi-session sidebar while retaining split groups and resize behavior.
• Persists custom terminal names by scoped thread, prunes stale labels, and clears labels when terminals or threads disappear.
• Adds an explicit drawer hide path and keeps single-terminal sessions free of unnecessary sidebar chrome.

Screenshots

Direct parent on the left; this PR on the right. Same viewport and copied application state.

Before	After

## Why

Terminal state and interaction changes need a focused review. This layer depends only on the shared chat header from #7153.

Validation

• Terminal drawer tests
• Terminal UI store migration, naming, cleanup, reconciliation, and sidebar tests
• Affected-package typechecks
• Changed-file lint and formatting checks

Stack order

1. feat(web): unify workspace navigation #7153 workspace and navigation
2. feat(web): redesign usage insights #7147 Usage
3. feat(web): refresh pull request details #7148 Pull Requests
4. feat(web): improve terminal workspace controls #7149 terminal, this PR
5. feat(web): attach composer state drawers #7150 composer
6. fix(server): preserve tool lifecycle identity #7151 tool lifecycle projection
7. feat(web): collapse tool activity into one line #7152 tool activity UI

Built with GPT-5.6-sol in the Codex harness.

---

Note

Medium Risk
Touches persisted terminal UI state (migration v5) and server/client terminal ID reconciliation; behavior is covered by tests but wrong label pruning could affect multi-surface terminal UX.

Overview
Reworks the thread terminal drawer with a resizable session sidebar, persisted per-terminal names, and a toolbar action to hide the drawer without closing sessions.

The multi-terminal sidebar moves to the right of the viewport with a drag handle (useResizableWidth + separate localStorage keys for drawer vs panel). Width is capped so the terminal area keeps a minimum usable width. Toolbar actions switch from popover hover buttons to compact icon tooltips; drawer mode adds Hide (wired from ChatView via onHide).

Custom terminal labels are stored in terminalUiStateStore (persist v5): rename via double-click or F2, with logic so unchanged automatic titles are not saved as custom names. Labels apply in the drawer list and in ChatView composer terminal chips. Closing a terminal or pruning during reconcileTerminalIds drops stale labels; reconciliation can pass panel terminal ids so right-panel sessions keep names when drawer state syncs from the server.

Single-terminal layout keeps a floating compact toolbar only (no sidebar chrome).

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

Note

Improve terminal workspace controls with inline renaming, custom labels, and resizable sidebar

• Adds inline terminal renaming (double-click or F2) in ThreadTerminalDrawer, persisting custom labels up to 80 chars in terminalUiStateStore (version bumped 4→5 with migration).
• Introduces a resizable terminal sidebar with dynamic max-width clamping and per-context persisted widths; sidebar hides when only one terminal is open.
• Adds onHide callback to ThreadTerminalDrawer and wires it through ChatViewContent so the drawer can request to be hidden.
• Reworks PullRequestDetailPanel to prioritize conflict resolution in the primary action slot and adds a stacked-PR detection via isStackedPullRequestBase and a new listRefs query.
• Adds rightPanelUnavailableLabel prop to PanelLayoutControls and ensures tooltip triggers remain interactive when toggles are disabled.
• Removes chromeVariant prop from PullRequestDetailPanel and its callers; condensed state is now derived solely from scroll position.
• Risk: terminalUiStateStore persistence version increment triggers migration for all existing users; label data not matching the new schema is silently dropped.

^{Macroscope summarized 69bbbf1.}

[coderabbitai]

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: dcb2d16b-de0c-4b21-b181-d99a892ed553

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ No successful main baseline artifact is available yet. This run establishes the initial measurement.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	—	12.6 KiB	—	15.1 KiB	✅
Codex	Thread snapshot wire	—	6.3 KiB	—	7.3 KiB	✅
Codex	Live turn WebSocket wire	—	6.3 KiB	—	7.8 KiB	✅
Codex	Live turn WebSocket decoded	—	51.8 KiB	—	66.4 KiB	✅
Codex	Live turn messages	—	16	—	21	✅
Claude	Total thread wire	—	12.6 KiB	—	15.1 KiB	✅
Claude	Thread snapshot wire	—	6.3 KiB	—	7.3 KiB	✅
Claude	Live turn WebSocket wire	—	6.3 KiB	—	7.8 KiB	✅
Claude	Live turn WebSocket decoded	—	52.7 KiB	—	66.4 KiB	✅
Claude	Live turn messages	—	16	—	21	✅

Baseline: unavailable · PR result: 69bbbf1 · 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: 99.9 KiB
• Claude decoded thread snapshot: 100.6 KiB

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

[macroscopeapp]

UI consistency review of the terminal drawer refresh. The migration to Button size="icon-xs" for the toolbar, the RightPanelResizeHandle/useResizableWidth reuse for the sidebar, and the tab-row treatment copied from RightPanelTabs all line up with the shared component system. Two tooltip-related consistency notes below; nothing else looked like a regression.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

Approvability

Verdict: Not approved

Macroscope's review found this PR not approvable — This PR introduces new user-facing features including terminal renaming, a resizable sidebar, and stacked pull request detection. The scope and complexity of these new capabilities, combined with unresolved findings about stale label pruning and resize persistence behavior, warrant human review.

Not approved because:

• 2 blocking correctness issues found at or above your repo's Minimum Blocking Severity

^{Adjust the Minimum Blocking Severity for this repo — including turning it Off — in Settings. You can add or adjust custom eligibility rules. Learn more.}

[macroscopeapp]

Two new findings on the terminal sidebar chrome. The two tooltip findings from the previous run (native title where the sibling tab bar / compact toolbars use the shared Tooltip, and the unreachable split-limit wording on the disabled Button) are still open and unchanged.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

UI consistency review of the terminal drawer refresh. Two new findings on the added controls; previously flagged items (sidebar toolbar overflow at the 144px min width, per-instance useResizableWidth state sharing one storage key, native title instead of the shared Tooltip) still apply and are not repeated here.

^{Posted via Macroscope — UI Consistency}

[macroscopeapp]

🟡 Medium

t3code/apps/web/src/components/ChatView.tsx

Line 790 in 69bbbf1

if(terminalIdListsEqual(serverOrderedTerminalIds,terminalUiState.terminalIds)){

Stale custom labels remain persisted when the server and drawer terminal IDs are equal, so a label for a deleted terminal is never pruned and can be reused when that ID is allocated again. The early return skips reconcileTerminalIds, which performs the label pruning; allow reconciliation to run in this case.

🚀 Reply "fix it for me" or copy this AI Prompt for your agent:

In file @apps/web/src/components/ChatView.tsx around line 790:
Stale custom labels remain persisted when the server and drawer terminal IDs are equal, so a label for a deleted terminal is never pruned and can be reused when that ID is allocated again. The early return skips `reconcileTerminalIds`, which performs the label pruning; allow reconciliation to run in this case.

[cursor]

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 69bbbf1. Configure here.}

[cursor]

Resize reverts if persist fails

Medium Severity

After a resize ends, dragWidth is cleared whenever the pointer is not down. Display then falls back to storedWidth. If setStoredWidth fails (quota or write error, which useLocalStorage swallows), the panel snaps back to the old width instead of keeping the drag result. Previously in-memory width still updated after a failed persist.

Additional Locations (1)

• apps/web/src/hooks/useResizableWidth.ts#L135-L143

^{Reviewed by Cursor Bugbot for commit 69bbbf1. Configure here.}

[macroscopeapp]

🟡 Mediumlib/openPullRequestLink.ts:126

changeRequestRepositoryUrl returns null for valid Azure URLs such as https://dev.azure.com/pull/123/_git/repo/pullrequest/42, and can return a truncated repository URL for similar paths. Its non-greedy fallback stops at the first /pull/<number>-like segment instead of the final PR suffix; extract the repository path using the last recognized provider suffix.

- const repositoryPath =- /^(.*?)\/\/-\/merge_requests\/\d+(?:\/|$)/iu.exec(url.pathname)?.[1] ??- /^(.*?)(?:\/pull\/\d+|\/-\/merge_requests\/\d+|\/pull-requests\/\d+|\/pullrequest\/\d+)(?:\/|$)/iu.exec(- url.pathname,- )?.[1];+ const repositoryPath = /^(.*)(?:\/pull\/\d+|\/-\/merge_requests\/\d+|\/pull-requests\/\d+|\/pullrequest\/\d+)(?:\/|$)/iu.exec(+ url.pathname,+ )?.[1];

🚀 Reply "fix it for me" or copy this AI Prompt for your agent:

In file @apps/web/src/lib/openPullRequestLink.ts around lines 126-130:
`changeRequestRepositoryUrl` returns `null` for valid Azure URLs such as `https://dev.azure.com/pull/123/_git/repo/pullrequest/42`, and can return a truncated repository URL for similar paths. Its non-greedy fallback stops at the first `/pull/<number>`-like segment instead of the final PR suffix; extract the repository path using the last recognized provider suffix.