From 8d98c8f24c1817bead8a18d3c089e53911fdf152 Mon Sep 17 00:00:00 2001
From: ciregenz <ciregenz@berkeley.edu>
Date: Tue, 28 Jul 2026 15:11:41 -0700
Subject: [PATCH] [eric] dock: slot height off percentages (nested-wrapper
 resolution pushed it out of the card), sub-agent browser tether also
 suppressed while docked

---
 frontend/src/app/pages/AgentChat/AgentChat.tsx           | 9 +++++++--
 .../src/app/pages/Dashboard/geometry/dashboardTethers.ts | 2 ++
 2 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/frontend/src/app/pages/AgentChat/AgentChat.tsx b/frontend/src/app/pages/AgentChat/AgentChat.tsx
index a600a792..87d9768a 100644
--- a/frontend/src/app/pages/AgentChat/AgentChat.tsx
+++ b/frontend/src/app/pages/AgentChat/AgentChat.tsx
@@ -2406,8 +2406,13 @@ const AgentChat: React.FC<AgentChatProps> = ({ sessionId: sessionIdProp, onClose
                 <Box
                   data-browser-slot={id}
                   sx={{
-                    flexShrink: 0,
-                    height: 'clamp(240px, 42%, 560px)',
+                    // Percentage heights resolve against a NESTED wrapper here, not the card, which
+                    // pushed the slot (and the docked browser riding it) clean out of the chat.
+                    // Viewport units are the only stable yardstick in this column; shrink allowed so
+                    // a short card squeezes the slot instead of overflowing.
+                    flex: '0 1 auto',
+                    height: 'min(360px, 38vh)',
+                    minHeight: 180,
                     mx: 1.5,
                     mb: 1,
                     borderRadius: '10px',
diff --git a/frontend/src/app/pages/Dashboard/geometry/dashboardTethers.ts b/frontend/src/app/pages/Dashboard/geometry/dashboardTethers.ts
index b9a9bc31..0c618e6b 100644
--- a/frontend/src/app/pages/Dashboard/geometry/dashboardTethers.ts
+++ b/frontend/src/app/pages/Dashboard/geometry/dashboardTethers.ts
@@ -279,6 +279,8 @@ export function useTethers({
       if (s.status !== 'running' && s.status !== 'waiting_approval') continue;
       if (!s.browser_id || !s.parent_session_id) continue;
       if (glowTethers.has(s.browser_id)) continue;
+      // Docked browsers live INSIDE the chat; an arrow to them points at nothing.
+      if (browserCards[s.browser_id]?.docked_to) continue;
       // A browser docked below the hub keeps a "Browser" pointer so the link reads at a glance; the right-docked agent/run cases stay label-free (their glow already said it on spawn).
       const parent = sessionById.get(s.parent_session_id);
       const t = cardTether(