feat(desktop): add browser tab duplication actions

Author: Sg312

apps/desktop/src/main/browser-agent/driver.ts

@@ -807,6 +807,12 @@ export async function handlePanelAction(action: BrowserPanelAction): Promise<voi
     session.addTab()
     return
   }
+  if (action.action === 'duplicate-tab') {
+    if (typeof action.tabId === 'string') {
+      session.duplicateTab(action.tabId)
+    }
+    return
+  }
   if (action.action === 'switch-tab') {
     if (typeof action.tabId === 'string') {
       session.switchTab(action.tabId)

apps/desktop/src/main/browser-agent/session.ts

@@ -524,6 +524,27 @@ export function reopenClosedTab(): AgentTab | null {
   return tab
 }
 
+/**
+ * Opens a copy of a tab at the same URL. A duplicate is a fresh load rather
+ * than a clone of the original's session history: the history belongs to the
+ * WebContents, and there is no way to fork it.
+ */
+export function duplicateTab(tabId: string): AgentTab | null {
+  restorePinnedTabs()
+  const source = tabs.find((entry) => entry.id === tabId)
+  if (!source || listTabs().length >= MAX_BROWSER_TABS) return null
+
+  const url = sanitizeRestorableUrl(source.view.webContents.getURL())
+  const tab = addTabInternal()
+  if (url && url !== 'about:blank') {
+    // Sanitized to http(s) without embedded credentials above, and the
+    // partition's onBeforeRequest still runs the full SSRF check on the load —
+    // same reasoning as reopenClosedTab, and this is likewise a user action.
+    void tab.view.webContents.loadURL(url).catch(() => {})
+  }
+  return tab
+}
+
 export function switchTab(tabId: string): AgentTab {
   restorePinnedTabs()
   const tab = tabs.find((entry) => entry.id === tabId)

apps/sim/app/workspace/[workspaceId]/home/components/mothership-view/components/resource-content/components/browser-session/browser-session.tsx

@@ -271,6 +271,10 @@ export function BrowserSession() {
     sendBrowserPanelAction('close-tab', { tabId })
   }, [])
 
+  const handleDuplicateTab = useCallback((tabId: string) => {
+    sendBrowserPanelAction('duplicate-tab', { tabId })
+  }, [])
+
   const handleSetTabPinned = useCallback((tabId: string, pinned: boolean) => {
     setBrowserTabPinned(tabId, pinned)
   }, [])
@@ -289,6 +293,7 @@ export function BrowserSession() {
             onNewTab={handleNewTab}
             onSwitchTab={handleSwitchTab}
             onCloseTab={handleCloseTab}
+            onDuplicateTab={handleDuplicateTab}
             onSetTabPinned={handleSetTabPinned}
             onReorderTab={handleReorderTab}
             pinningSupported={tabPinningSupported}

apps/sim/app/workspace/[workspaceId]/home/components/mothership-view/components/resource-content/components/browser-session/browser-tab-strip.tsx

@@ -14,6 +14,7 @@ interface BrowserTabStripProps {
   onNewTab: () => void
   onSwitchTab: (tabId: string) => void
   onCloseTab: (tabId: string) => void
+  onDuplicateTab: (tabId: string) => void
   onSetTabPinned: (tabId: string, pinned: boolean) => void
   onReorderTab: (tabId: string, targetIndex: number) => void
   pinningSupported: boolean
@@ -58,7 +59,8 @@ function BrowserTabIcon({ tab }: { tab: BrowserTabState }) {
 
 /**
  * The browser panel's tab strip: the shared {@link TabStrip} plus the two
- * things only the browser has — favicons, and a pin/unpin context menu. The
+ * things only the browser has — favicons, and a right-click menu carrying
+ * pin/unpin alongside the duplicate and close every tab strip offers. The
  * active Electron view remains the only native view attached over the panel;
  * selecting a tab switches which live view is attached.
  */
@@ -68,6 +70,7 @@ export function BrowserTabStrip({
   onNewTab,
   onSwitchTab,
   onCloseTab,
+  onDuplicateTab,
   onSetTabPinned,
   onReorderTab,
   pinningSupported,
@@ -98,15 +101,10 @@ export function BrowserTabStrip({
   const openTabContextMenu = useCallback(
     (event: ReactMouseEvent<HTMLDivElement>, tabId: string) => {
       window.getSelection()?.removeAllRanges()
-      if (!pinningSupported) {
-        event.preventDefault()
-        event.stopPropagation()
-        return
-      }
       setContextTabId(tabId)
       handleContextMenu(event)
     },
-    [handleContextMenu, pinningSupported]
+    [handleContextMenu]
   )
 
   return (
@@ -120,7 +118,7 @@ export function BrowserTabStrip({
       {...(reorderingSupported ? { onReorder: onReorderTab } : {})}
     >
       <ContextMenu
-        isOpen={isContextMenuOpen && Boolean(contextTab) && pinningSupported}
+        isOpen={isContextMenuOpen && Boolean(contextTab)}
         position={contextMenuPosition}
         menuRef={contextMenuRef}
         onClose={closeContextMenu}
@@ -129,11 +127,17 @@ export function BrowserTabStrip({
             ? () => onSetTabPinned(contextTab.tabId, !contextTab.pinned)
             : undefined
         }
+        onDuplicate={contextTab ? () => onDuplicateTab(contextTab.tabId) : undefined}
+        // A pinned tab has no close affordance in the strip either.
+        {...(contextTab && !contextTab.pinned
+          ? { onCloseTab: () => onCloseTab(contextTab.tabId), showCloseTab: true }
+          : {})}
         onDelete={() => {}}
         showPin={Boolean(contextTab) && pinningSupported}
         isPinned={Boolean(contextTab?.pinned)}
         showRename={false}
-        showDuplicate={false}
+        showDuplicate={Boolean(contextTab)}
+        disableDuplicate={tabs.length >= MAX_BROWSER_TABS}
         showDelete={false}
       />
     </TabStrip>

apps/sim/app/workspace/[workspaceId]/w/components/sidebar/components/workflow-list/components/context-menu/context-menu.tsx

@@ -22,6 +22,7 @@ import {
   SquareArrowUpRight,
   Trash,
   Unlock,
+  X,
 } from '@sim/emcn/icons'
 import { Pin, PinOff } from 'lucide-react'
 
@@ -52,6 +53,12 @@ interface ContextMenuProps {
   onDuplicate?: () => void
   onExport?: () => void
   onDelete: () => void
+  /**
+   * Closes the item rather than deleting it — for tabs, where the destructive
+   * action is "close this one", not "delete it forever". Named for the item so
+   * it cannot be confused with `onClose`, which dismisses this menu.
+   */
+  onCloseTab?: () => void
   showOpenInNewTab?: boolean
   showMarkAsRead?: boolean
   showMarkAsUnread?: boolean
@@ -78,6 +85,7 @@ interface ContextMenuProps {
   disableLock?: boolean
   isLocked?: boolean
   showDelete?: boolean
+  showCloseTab?: boolean
   onUploadLogo?: () => void
   showUploadLogo?: boolean
   disableUploadLogo?: boolean
@@ -103,6 +111,7 @@ export function ContextMenu({
   onDuplicate,
   onExport,
   onDelete,
+  onCloseTab,
   showOpenInNewTab = false,
   showMarkAsRead = false,
   showMarkAsUnread = false,
@@ -129,6 +138,7 @@ export function ContextMenu({
   disableLock = false,
   isLocked = false,
   showDelete = true,
+  showCloseTab = false,
   onUploadLogo,
   showUploadLogo = false,
   disableUploadLogo = false,
@@ -325,7 +335,7 @@ export function ContextMenu({
         )}
 
         {(hasNavigationSection || hasStatusSection || hasEditSection || hasCopySection) &&
-          (showLeave || showDelete) && <DropdownMenuSeparator />}
+          (showLeave || showDelete || (showCloseTab && onCloseTab)) && <DropdownMenuSeparator />}
         {showLeave && onLeave && (
           <DropdownMenuItem
             disabled={disableLeave}
@@ -350,6 +360,17 @@ export function ContextMenu({
             Delete
           </DropdownMenuItem>
         )}
+        {showCloseTab && onCloseTab && (
+          <DropdownMenuItem
+            onSelect={() => {
+              onCloseTab()
+              onClose()
+            }}
+          >
+            <X />
+            Close
+          </DropdownMenuItem>
+        )}
       </DropdownMenuContent>
     </DropdownMenu>
   )

apps/sim/lib/terminal/transport.ts

@@ -83,8 +83,8 @@ export function resizeTerminal(terminalId: string, cols: number, rows: number):
   bridge()?.resize(terminalId, cols, rows)
 }
 
-export async function openTerminal(): Promise<void> {
-  await bridge()?.openTerminal()
+export async function openTerminal(cwd?: string): Promise<void> {
+  await bridge()?.openTerminal(cwd)
 }
 
 export async function switchTerminal(terminalId: string): Promise<void> {

packages/browser-protocol/src/index.ts

@@ -135,12 +135,13 @@ export interface BrowserPanelAction {
     | 'back'
     | 'forward'
     | 'new-tab'
+    | 'duplicate-tab'
     | 'switch-tab'
     | 'close-tab'
     | 'takeover-done'
   /** Absolute URL for `navigate` (typed into the panel's URL bar). */
   url?: string
-  /** Stable tab id for `switch-tab` and `close-tab`. */
+  /** Stable tab id for `duplicate-tab`, `switch-tab`, and `close-tab`. */
   tabId?: string
 }