[saddlepaddle]

The right pane's "+" now opens a full New Tab pane instead of a dropdown, following Satya's right-pane audit (#general, 10-27). It has three sections, titled like the cloud sidebar's hover card (layout "A2" from the mocks):

• Tools: Terminal, Browser, Changes, Files, Review, Pages (plus Desktop on a sandbox and Mobile behind its flag), with their hotkeys. These are the center empty state's rows (EmptyTabActionButton).
• Pages: this workspace's 4 newest pages, as the right pane's Pages view shows them (PagesListCard).
• Pull requests: every PR linked to this workspace, newest first, in any state, as one-line hover-card rows (CloudPullRequestRow), from the host's PR history (pullRequests.historyByWorkspaces), so it works for local and cloud workspaces.

Screenshots: https://app.superset.sh/page/right-pane-new-tab-screen-zupovt

What changed

• "+" adds a New Tab, and an overflowing tab bar scrolls it into view. Splits in the right pane (header button, Split Horizontally/Vertically) put a New Tab in the new split; the center still splits to a terminal.
• Opening an empty right pane adds a New Tab. This only reacts to the pane opening or its layout loading, so closing the last tab still closes the pane.
• Picking anything (tool, page, PR) replaces the New Tab in place and focuses the new pane. If that page, or the workspace's one PR pane, is already open, it is retargeted and focused instead, and the New Tab stays.
• Hotkeys: when a New Tab is the focused pane, ⌘T, ⌘⇧B and ⌘⇧G replace it with a terminal, browser or Changes pane. Anywhere else they behave as before.
• No duplicate header: a New Tab alone in its tab hides its pane header, since the tab already says "New Tab". In a split it keeps the header, so each pane can be told apart and closed. Splitting a lone New Tab is from the right-click menu. (@superset/panes gets an optional hideHeader(context) on pane definitions.)
• Lists stay loaded: useNewTabLists owns the pages and PR queries. The right pane area (mounted while the workspace is open) keeps them cached and refreshes them every minute, so a New Tab renders them on its first frame with no request.
• One helper, replaceNewTabPane, serves picks, page/PR clicks and hotkeys. It unpins first, because a header click pins a pane and replacePane skips pinned panes.
• @superset/panes: Workspace takes onAddTab so "+" can be a plain button. New new-tab pane kind. PagesListCard moves up to be shared. EmptyTabActionButton takes a className.
• Removed RightPaneAddMenu, RightPaneEmptyState and page.tsx's addRightPane.

Choices to check

• PR +/− counts: the host's PR history doesn't store them, so each row reads them from the PR detail query the pull-request pane uses (local and cloud). The right pane area keeps those queries warm, so counts are there when a New Tab opens. A PR linked twice (for example under the old URL after the repo rename) shows once, keeping the newest link.
• Which pages: this workspace's pages, newest first. Sections with nothing to show are hidden.
• Terminal from the New Tab opens a blank terminal and skips "apply on new tab" presets, which build their own tab layout.
• Files, Review and Pages have no hotkeys. Split hotkeys still act on the center pane only.

Test cases

Case	Expected	Checked
Open an empty right pane	One New Tab	App + hook test
Close the only tab / reopen	Pane closes, not refilled / fresh New Tab	App + hook test
"Move to Right Side" while closed	Only the moved pane	App
"+"	Adds a New Tab	App
"+" with the tab bar overflowing (2 → 15 tabs)	New tab fully visible every time	App + unit test
Split from header or context menu	New split is a New Tab	App + unit test
Lone New Tab / New Tab in a split	No header / header shown	App
Pick a tool (Browser, Files, Changes, Terminal)	Replaces in place, takes focus	App
Pick after a header click (pinned)	Still replaces	App + unit test
⌘T / ⌘⇧B / ⌘⇧G with a New Tab focused	Replaces it in place, no extra tab	App
⌘⇧B with another pane focused	Adds a browser tab as before	App
New Tab after 60 s with none on screen	Lists in the first render, 0 requests; background refresh ran once at 60 s	App
Click a page card / PR row	Page / PR pane in place of the New Tab	App
PR pane already open elsewhere	Retargets it, New Tab stays	Unit test
Desktop/Mobile tiles	Only on a sandbox / with mobile-pane	Unit test

Known gap, not changed: "Reveal in sidebar" still targets the old sidebar's file tree, which the right pane area doesn't render.

Verification

• Dev app over CDP with right-pane-area on: every "App" row above, no console.error. A page opened from the New Tab gets the right page pane, but its body is blank locally because bun dev doesn't run the page content server (port 6794). That applies to every page pane in this setup.
• bun test: new tests for replaceNewTabPane, withNewTabSplits, the open-empty hook and createRightPane pass, plus packages/panes. A full run of the v2-workspace folder shows 1 unrelated failure: a Lexical import-order error in the chat composer's useLinkChips.test.tsx, which passes alone.
• Biome clean. typecheck has no errors in changed files; the existing better-auth type errors in packages/auth/packages/trpc are unrelated. check:i18n clean, no new strings.

[coderabbitai]

Caution

Review failed

The pull request is closed.

Finishing Touches

📝 Generate docstrings

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

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

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

[expo]

No installed development build can load this pull request as an update: its native fingerprint is 9d26f603f51c and no development build has it. Add the mobile-build label to build one.

[github-actions]

🧹 Preview Cleanup Complete

The following preview resources have been cleaned up:

• ✅ Neon database branch

Thank you for your contribution! 🎉

[greptile-apps]

Confidence Score: 4/5

[Medium impact] The PR changes how people start and fill panes in the desktop workspace.

The PR appears safe to merge, with non-blocking improvements to PR-count fetching and add-menu behavior.

Findings

1.  Hidden lists fetch PR activity ▶
2.  Opening menus adds tabs ▶

Fix with agent prompt

### Issue 1
apps/desktop/src/renderer/routes/_authenticated/_dashboard/v2-workspace/$workspaceId/components/RightPaneArea/RightPaneArea.tsx:75-77
`RightPaneArea` fetches full details for every historical PR, even when no New Tab is visible. `usePullRequestCounts` calls `usePullRequestDetail`, which requests activity as well as counts. Since workspace history has no limit, opening a long-lived workspace can start many expensive GitHub reads just to display two numbers. Use a lightweight counts query, or limit which PRs get full-detail prefetches.

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

### Issue 2
packages/panes/src/react/components/Workspace/components/TabBar/TabBar.tsx:72
`onAddTab` still runs when `renderAddTabMenu` is provided, despite the new API saying it is ignored in that case. The same button keeps its `onClick` when wrapped by `DropdownMenuTrigger`, so a caller supplying both props adds a tab while opening the menu. Only attach `onAddTab` when no menu renderer is supplied.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Summary

The right pane now opens a New Tab screen with tools, recent pages, and linked PRs.

• The right pane’s New Tab screen brings tools, recent pages, and pull requests together.
• The right pane opens a New Tab when people add, split, or reopen it empty.
• Picks and shortcuts replace a focused New Tab with the chosen pane.
• The shared pane workspace can add tabs directly and keep new tabs visible.
• Page thumbnails keep their placeholder visible until the image loads.

Acknowledged limits:

• saddlepaddle explicitly identified the sidebar reveal gap as unchanged behavior.
• saddlepaddle attributed blank local page bodies to the development content server not running, not to this change.
• saddlepaddle explicitly chose to skip terminal presets when opening a terminal from New Tab and to retain split actions in the context menu when the header is hidden.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart TD
  A["Right pane opens empty or user clicks +"] --> B["New Tab"]
  C["Right-pane split"] --> B
  B --> D["Choose tool, page, or PR"]
  D --> E{"Matching keyed pane exists?"}
  E -->|Yes| F["Update and focus existing pane"]
  E -->|No| G["Unpin and replace New Tab"]
  G --> H["Focus new pane"]

Loading

_{Reviews (1) · Last reviewed commit: "fix(desktop): New Tab PR rows right-alig..." · Reviewed by Greptile}

[greptile-apps]

Hidden lists fetch PR activity

RightPaneArea fetches full details for every historical PR, even when no New Tab is visible. usePullRequestCounts calls usePullRequestDetail, which requests activity as well as counts. Since workspace history has no limit, opening a long-lived workspace can start many expensive GitHub reads just to display two numbers. Use a lightweight counts query, or limit which PRs get full-detail prefetches.

Prompt To Fix With AI

This is a comment left during a code review.
Path: apps/desktop/src/renderer/routes/_authenticated/_dashboard/v2-workspace/$workspaceId/components/RightPaneArea/RightPaneArea.tsx
Line: 75-77

Comment:
**Hidden lists fetch PR activity**

`RightPaneArea` fetches full details for every historical PR, even when no New Tab is visible. `usePullRequestCounts` calls `usePullRequestDetail`, which requests activity as well as counts. Since workspace history has no limit, opening a long-lived workspace can start many expensive GitHub reads just to display two numbers. Use a lightweight counts query, or limit which PRs get full-detail prefetches.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

[greptile-apps]

Opening menus adds tabs

onAddTab still runs when renderAddTabMenu is provided, despite the new API saying it is ignored in that case. The same button keeps its onClick when wrapped by DropdownMenuTrigger, so a caller supplying both props adds a tab while opening the menu. Only attach onAddTab when no menu renderer is supplied.

Prompt To Fix With AI

This is a comment left during a code review.
Path: packages/panes/src/react/components/Workspace/components/TabBar/TabBar.tsx
Line: 72

Comment:
**Opening menus adds tabs**

`onAddTab` still runs when `renderAddTabMenu` is provided, despite the new API saying it is ignored in that case. The same button keeps its `onClick` when wrapped by `DropdownMenuTrigger`, so a caller supplying both props adds a tab while opening the menu. Only attach `onAddTab` when no menu renderer is supplied.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.