@pellux/goodvibes-tui 2.0.21 → 2.1.1

package/src/input/handler-command-route.ts

@@ -1,10 +1,10 @@
+import { activeTokens } from '../renderer/theme.ts';
 import { loadSkillByTrigger } from '@pellux/goodvibes-sdk/platform/tools';
 import { summarizeError } from '@pellux/goodvibes-sdk/platform/utils';
 import type { CommandContext, CommandRegistry } from './command-registry.ts';
 import type { AutocompleteEngine } from './autocomplete.ts';
 import type { InputToken } from '@pellux/goodvibes-sdk/platform/core';
 import type { ConversationManager } from '../core/conversation';
-import type { PanelManager } from '../panels/panel-manager.ts';
 import { handleClipboardPaste, type ClipboardPasteSource } from './handler-content-actions.ts';
 
 export type CommandModeRouteState = {
@@ -15,8 +15,6 @@ export type CommandModeRouteState = {
   modalStack: string[];
   commandRegistry: CommandRegistry | null;
   commandContext?: CommandContext;
-  panelFocused: boolean;
-  panelManager: PanelManager;
   conversationManager: ConversationManager | null;
   requestRender: () => void;
   handleEscape: () => void;
@@ -93,7 +91,7 @@ export function handleCommandModeToken(state: CommandModeRouteState, token: Inpu
       const parts = raw.slice(1).trim().split(/\s+/);
       const name = parts[0];
       const args = parts.slice(1);
-      const ctx = withPanelFocusSync(state.commandContext, state);
+      const ctx = withComposerBindings(state.commandContext, state);
       const commandPromise = state.commandRegistry.get(name)
         ? state.commandRegistry.execute(name, args, ctx)
         : (ctx.executeCommand?.(name, args) ?? Promise.resolve(false));
@@ -112,7 +110,7 @@ export function handleCommandModeToken(state: CommandModeRouteState, token: Inpu
             if (skillContent) {
               state.commandContext?.submitInput?.(skillContent);
             } else {
-              state.conversationManager?.log(`Unknown command: /${name}. Type /help for available commands.`, { fg: '#ef4444' });
+              state.conversationManager?.log(`Unknown command: /${name}. Type /help for available commands.`, { fg: activeTokens().error });
               state.requestRender();
             }
           }
@@ -122,7 +120,7 @@ export function handleCommandModeToken(state: CommandModeRouteState, token: Inpu
           // already guard their own internals: a handler that throws or
           // awaits a rejected promise must never become a silent unhandled
           // rejection, it renders the same way an unknown command does.
-          state.conversationManager?.log(`Command /${name} failed: ${summarizeError(err)}`, { fg: '#ef4444' });
+          state.conversationManager?.log(`Command /${name} failed: ${summarizeError(err)}`, { fg: activeTokens().error });
           state.requestRender();
         });
     } else {
@@ -134,43 +132,10 @@ export function handleCommandModeToken(state: CommandModeRouteState, token: Inpu
   return token.logicalName !== 'left' && token.logicalName !== 'right' && token.logicalName !== 'home' && token.logicalName !== 'end';
 }
 
-function withPanelFocusSync(context: CommandContext, state: CommandModeRouteState): CommandContext {
-  const panelIsFocusable = (): boolean =>
-    state.panelManager.isVisible() && state.panelManager.getAllOpen().length > 0;
-
+/** The command context with the composer-bound actions (clipboard paste, nested commands) bound to this route's state. */
+function withComposerBindings(context: CommandContext, state: CommandModeRouteState): CommandContext {
   return {
     ...context,
-    openPanelPicker: context.openPanelPicker
-      ? () => {
-          context.openPanelPicker?.();
-          state.panelFocused = panelIsFocusable();
-        }
-      : undefined,
-    showPanel: context.showPanel
-      ? (panelId, pane, target, opts) => {
-          context.showPanel?.(panelId, pane, target, opts);
-          // showPanel no longer transfers focus by default, the command
-          // path leaves the composer focused (ui-openers.ts's showPanel only
-          // calls focusPanels() when opts.focus is set). Only flip the local
-          // panelFocused mirror when the caller explicitly asked for focus;
-          // otherwise leave it exactly as it was (the composer, since you can
-          // only type a command from there in the first place). `target` is the
-          // deep-link jump target, forwarded verbatim.
-          if (opts?.focus) state.panelFocused = true;
-        }
-      : undefined,
-    focusPanels: context.focusPanels
-      ? () => {
-          context.focusPanels?.();
-          state.panelFocused = panelIsFocusable();
-        }
-      : undefined,
-    focusPrompt: context.focusPrompt
-      ? () => {
-          context.focusPrompt?.();
-          state.panelFocused = false;
-        }
-      : undefined,
     pasteFromClipboard: () => {
       const result = handleClipboardPaste({
         prompt: state.prompt,
@@ -190,7 +155,7 @@ function withPanelFocusSync(context: CommandContext, state: CommandModeRouteStat
       return result;
     },
     executeCommand: async (name, args) => {
-      const wrapped = withPanelFocusSync(context, state);
+      const wrapped = withComposerBindings(context, state);
       const handled = state.commandRegistry?.get(name)
         ? await state.commandRegistry.execute(name, args, wrapped)
         : false;

package/src/input/handler-content-actions.ts

@@ -1,3 +1,4 @@
+import { activeTokens } from '../renderer/theme.ts';
 import { readFileSync, writeFileSync, existsSync } from 'node:fs';
 import { copyToClipboard, pasteFromClipboard, pasteImageFromClipboard } from '../utils/clipboard.ts';
 import type { InfiniteBuffer } from '@pellux/goodvibes-terminal-shell';
@@ -299,7 +300,7 @@ export function handleBlockCopy(
   const lineIndex = getAnchorLine();
   const nearest = conversationManager.findNearestBlock(lineIndex);
   if (!nearest) {
-    conversationManager.log('[Ctrl+Y: No block found nearby]', { fg: '240' });
+    conversationManager.log('[Ctrl+Y: No block found nearby]', { fg: activeTokens().textFaint });
     requestRender();
     return;
   }
@@ -310,7 +311,7 @@ export function handleBlockCopy(
   // copying content the user just had search reveal is a deliberate choice
   // to keep it visible, not an accident of typing a query.
   conversationManager.searchExpansion.noteUserTouch(nearest.collapseKey);
-  conversationManager.log(`[Copied ${describeBlockForReceipt(nearest)}]`, { fg: '#22c55e' });
+  conversationManager.log(`[Copied ${describeBlockForReceipt(nearest)}]`, { fg: activeTokens().success });
   requestRender();
   setTimeout(() => requestRender(), 2005);
 }
@@ -325,7 +326,7 @@ export function handleBookmark(
   const lineIndex = getAnchorLine();
   const nearest = conversationManager.findNearestBlock(lineIndex);
   if (!nearest) {
-    conversationManager.log('[Ctrl+B: No block found nearby]', { fg: '240' });
+    conversationManager.log('[Ctrl+B: No block found nearby]', { fg: activeTokens().textFaint });
     requestRender();
     return;
   }
@@ -337,7 +338,7 @@ export function handleBookmark(
   const msg = added
     ? `[Bookmarked: ${target}]`
     : `[Bookmark removed: ${target}]`;
-  conversationManager.log(msg, { fg: added ? '#22c55e' : '244' });
+  conversationManager.log(msg, { fg: added ? activeTokens().success : activeTokens().textMuted });
   requestRender();
 }
 
@@ -351,7 +352,7 @@ export function handleBlockSave(
   const lineIndex = getAnchorLine();
   const nearest = conversationManager.findNearestBlock(lineIndex);
   if (!nearest) {
-    conversationManager.log('[Ctrl+S: No block found nearby]', { fg: '240' });
+    conversationManager.log('[Ctrl+S: No block found nearby]', { fg: activeTokens().textFaint });
     requestRender();
     return;
   }
@@ -362,10 +363,10 @@ export function handleBlockSave(
     const filePath = bookmarkManager.saveToFile(nearest.rawContent, nearest.type);
     const homePath = process.env.HOME || process.env.USERPROFILE || '';
     const displayPath = homePath ? filePath.replace(homePath, '~') : filePath;
-    conversationManager.log(`[Saved ${target} to: ${displayPath}]`, { fg: '#22c55e' });
+    conversationManager.log(`[Saved ${target} to: ${displayPath}]`, { fg: activeTokens().success });
   } catch (err) {
     const msg = summarizeError(err);
-    conversationManager.log(`[Save failed (${target}): ${msg}]`, { fg: '#ef4444' });
+    conversationManager.log(`[Save failed (${target}): ${msg}]`, { fg: activeTokens().error });
   }
   requestRender();
 }
@@ -382,7 +383,7 @@ export function handleBlockToggle(
   const wasCollapsed = conversationManager.isCollapsed(nearest.blockIndex);
   const blockIdx = conversationManager.toggleCollapseAtLine(lineIndex);
   if (blockIdx >= 0) {
-    conversationManager.log(`[${wasCollapsed ? 'Expanded' : 'Collapsed'} ${describeBlockForReceipt(nearest)}]`, { fg: '244' });
+    conversationManager.log(`[${wasCollapsed ? 'Expanded' : 'Collapsed'} ${describeBlockForReceipt(nearest)}]`, { fg: activeTokens().textMuted });
     requestRender();
   }
 }
@@ -415,14 +416,14 @@ export function handleDiffApply(
   }).then(({ approved }) => {
     if (!approved) return;
     if (!projectRoot) {
-      conversationManager.log('[Diff apply failed: missing working directory]', { fg: '#ef4444' });
+      conversationManager.log('[Diff apply failed: missing working directory]', { fg: activeTokens().error });
       return;
     }
     let resolvedPath: string;
     try {
       resolvedPath = resolveAndValidatePath(diff.filePath!, projectRoot);
     } catch (err) {
-      conversationManager.log(`[Diff apply failed: ${err instanceof Error ? err.message : err}]`, { fg: '#ef4444' });
+      conversationManager.log(`[Diff apply failed: ${err instanceof Error ? err.message : err}]`, { fg: activeTokens().error });
       return;
     }
     try {
@@ -430,22 +431,22 @@ export function handleDiffApply(
       if (diff.original && content.includes(diff.original)) {
         const occurrenceCount = content.split(diff.original).length - 1;
         if (occurrenceCount > 1) {
-          conversationManager.log(`[Diff apply failed: pattern found ${occurrenceCount} times in ${diff.filePath} - ambiguous]`, { fg: '#ef4444' });
+          conversationManager.log(`[Diff apply failed: pattern found ${occurrenceCount} times in ${diff.filePath} - ambiguous]`, { fg: activeTokens().error });
         } else {
           const newContent = content.replace(diff.original, diff.updated);
           writeFileSync(resolvedPath, newContent, 'utf-8');
-          conversationManager.log(`[Applied diff to ${diff.filePath}]`, { fg: '#22c55e' });
+          conversationManager.log(`[Applied diff to ${diff.filePath}]`, { fg: activeTokens().success });
         }
       } else {
-        conversationManager.log(`[Diff apply failed: original text not found in ${diff.filePath}]`, { fg: '#ef4444' });
+        conversationManager.log(`[Diff apply failed: original text not found in ${diff.filePath}]`, { fg: activeTokens().error });
       }
     } catch (err) {
       const msg = summarizeError(err);
-      conversationManager.log(`[Diff apply error: ${msg}]`, { fg: '#ef4444' });
+      conversationManager.log(`[Diff apply error: ${msg}]`, { fg: activeTokens().error });
     }
     requestRender();
   }).catch((err) => {
-    conversationManager.log(`[Diff apply error: ${summarizeError(err)}]`, { fg: '#ef4444' });
+    conversationManager.log(`[Diff apply error: ${summarizeError(err)}]`, { fg: activeTokens().error });
     requestRender();
   });
   return true;

package/src/input/handler-feed-routes.ts

@@ -1,4 +1,5 @@
 import type { InputToken } from '@pellux/goodvibes-sdk/platform/core';
+import type { FooterTarget } from '../renderer/footer-targets.ts';
 import type { BlockMeta, ConversationManager } from '../core/conversation';
 import type { InputHistory } from './input-history.ts';
 import type { ContentPart } from '@pellux/goodvibes-sdk/platform/providers';
@@ -12,185 +13,14 @@ import {
   moveCursorVertical as computeCursorVerticalMove,
 } from './handler-prompt-buffer.ts';
 import { cleanupMarkerRegistry, expandPrompt, findMarkerAtPos, registerPaste } from './handler-content-actions.ts';
-import type { PanelManager } from '../panels/panel-manager.ts';
-import { getPanelUnderMouse, workspaceTabAtMouse } from './panel-mouse-geometry.ts';
-import type { PanelMouseLayout } from './panel-mouse-geometry.ts';
-export type { PanelMouseLayout };
-import type { KeybindingsManager } from './keybindings.ts';
 import type { KillRing } from './kill-ring.ts';
 import { summarizeError } from '@pellux/goodvibes-sdk/platform/utils';
-import { trackPanelPasteFloodGuard, type PanelBurstGuardState } from './panel-paste-flood-guard.ts';
-
-export type PanelFocusRouteState = {
-  panelManager: PanelManager;
-  keybindingsManager: KeybindingsManager;
-  panelFocused: boolean;
-  commandMode: boolean;
-  searchActive: boolean;
-  autocompleteActive: boolean;
-  requestRender: () => void;
-  handlePathCompletion: () => boolean;
-  cyclePanelTab: (direction: 'next' | 'prev') => void;
-  onPanelInputConsumed?: (activePanel: import('../panels/types.ts').Panel | null, key: string) => void;
-  /**
-   * True when THIS token is a paste: a single 'text' token whose value holds
-   * more than one character. The SDK tokenizer emits a bracketed paste
-   * (\x1b[?2004h, enabled in main.ts init) as exactly one such token; discrete
-   * keystrokes, even several batched into one feed() by render-tick latency,
-   * arrive as separate 1-char tokens. See the text-token branch below.
-   */
-  isPasteToken: boolean;
-  /**
-   * One-shot hint emitted when a paste is dropped because the focused panel has
-   * no text capture (Invariant A: focus must not silently flip to the composer).
-   */
-  onPasteDropped?: (panelName: string) => void;
-  /** Wall-clock time (ms) for this token; `burstGuard` is item 5's flood-guard state, mutated in place across tokens (see panel-paste-flood-guard.ts). */
-  now: number;
-  burstGuard: PanelBurstGuardState;
-  /**
-   * True while a turn is actively streaming; gates Escape's cancel-first
-   * precedence (I6.1) ahead of the panel's own two-stage escape contract
-   * below.
-   */
-  isTurnActive: () => boolean;
-  /** Cancel the active turn. Wired to commandContext.cancelGeneration. */
-  cancelGeneration: () => void;
-};
-
-export function handlePanelFocusToken(state: PanelFocusRouteState, token: InputToken): {
-  handled: boolean;
-  panelFocused: boolean;
-} {
-  let panelFocused = state.panelFocused;
-
-  if (
-    token.type === 'key' &&
-    token.logicalName === 'tab' &&
-    !state.commandMode &&
-    !state.searchActive &&
-    !state.autocompleteActive
-  ) {
-    const pm = state.panelManager;
-    if (pm.isVisible() && pm.getAllOpen().length > 0) {
-      if (panelFocused) {
-        panelFocused = false;
-      } else if (!state.handlePathCompletion()) {
-        panelFocused = true;
-      }
-      state.requestRender();
-      return { handled: true, panelFocused };
-    }
-  }
-
-  if (!panelFocused) {
-    return { handled: false, panelFocused };
-  }
-
-  if (token.type === 'key') {
-    // I6: two-stage Escape, give the panel a chance to consume escape first
-    // (e.g. dismiss a confirm dialog or clear search). Only unfocus if the
-    // panel returns false (unconsumed) or there is no active panel.
-    if (token.logicalName === 'escape') {
-      // While streaming, a focused panel must not swallow the
-      // only way to cancel, Escape's first job is always cancel-turn; the
-      // panel's own two-stage consume-or-unfocus contract only runs once no
-      // turn is active, so a *second* Escape falls through to it normally
-      // (panelFocused stays unchanged here). Calls cancelGeneration()
-      // directly rather than the shared handleEscape() (handler-modal-stack.ts),
-      // whose fallthrough order doesn't guarantee reaching it first.
-      if (state.isTurnActive()) {
-        state.cancelGeneration();
-        state.requestRender();
-        return { handled: true, panelFocused };
-      }
-      const activePanel = state.panelManager.getActive();
-      const panelConsumedEscape = activePanel?.handleInput?.('escape') ?? false;
-      if (panelConsumedEscape) {
-        state.onPanelInputConsumed?.(activePanel!, 'escape');
-        state.requestRender();
-        return { handled: true, panelFocused };
-      }
-      panelFocused = false;
-      state.requestRender();
-      return { handled: true, panelFocused };
-    }
-    const kb = state.keybindingsManager;
-    // NOTE: panel-tab-next/prev, panel-close, and panel-close-all are handled
-    // globally in handleGlobalShortcutToken (runs earlier in the feed loop),
-    // their old, drifted copies here were unreachable and are gone. Only
-    // panel-focus-toggle stays, since it's meaningful only once the panel
-    // workspace already owns focus (it swaps between the two panes).
-    if (kb.matches('panel-focus-toggle', token)) {
-      // Switch keyboard focus between the top and bottom panes (no-op when
-      // there is no visible, non-empty bottom pane).
-      state.panelManager.togglePaneFocus();
-      state.requestRender();
-      return { handled: true, panelFocused };
-    }
-    // Alt+1..9 (panel-tab-1..9) is a global KeyAction consumed earlier by
-    // handleGlobalShortcutToken, so those meta+digit tokens never reach here.
-    // Any other ctrl/meta combo this route does not own falls through so the
-    // global handler (which ran first) or the prompt routes can act on it.
-    if (token.ctrl || token.meta) {
-      return { handled: false, panelFocused };
-    }
-    const activePanel = state.panelManager.getActive();
-    if (activePanel?.handleInput) {
-      const consumed = activePanel.handleInput(token.logicalName);
-      if (consumed) {
-        state.onPanelInputConsumed?.(activePanel, token.logicalName);
-        state.requestRender();
-        return { handled: true, panelFocused };
-      }
-    }
-  }
-
-  if (token.type === 'text' && token.value) {
-    const activePanel = state.panelManager.getActive();
-    // '/' is an explicit transfer verb back to the composer (a capturing panel, isCapturingTextBurst, keeps '/' for itself).
-    if (token.value === '/' && !activePanel?.isCapturingTextBurst?.()) {
-      panelFocused = false;
-      return { handled: false, panelFocused };
-    }
-    // Invariant A/B: a paste (isPasteToken, one multi-char token) into
-    // a capturing panel forwards verbatim below; elsewhere it is DROPPED with
-    // a one-shot hint (never exploded into hotkeys, never a silent focus
-    // flip, focus moves only on an explicit verb). Discrete keystrokes fall
-    // through to the per-char dispatch below.
-    if (state.isPasteToken && !activePanel?.isCapturingTextBurst?.()) {
-      state.onPasteDropped?.(activePanel?.name ?? 'the panel');
-      state.requestRender();
-      return { handled: true, panelFocused };
-    }
-
-    // item 5, paste flood guard (rate-based; see panel-paste-flood-guard.ts).
-    if (!state.isPasteToken && !activePanel?.isCapturingTextBurst?.()) {
-      const guard = trackPanelPasteFloodGuard(state.burstGuard, state.now);
-      if (!guard.dispatch) {
-        if (guard.showHintNow) state.onPasteDropped?.(activePanel?.name ?? 'the panel');
-        state.requestRender();
-        return { handled: true, panelFocused };
-      }
-    }
-
-    if (activePanel?.handleInput) {
-      for (const ch of token.value) {
-        activePanel.handleInput(ch);
-      }
-      state.requestRender();
-    }
-    return { handled: true, panelFocused };
-  }
-
-  return { handled: true, panelFocused };
-}
 
 export type IndicatorFocusRouteState = {
   indicatorFocused: boolean;
   modalOpened: (name: string) => void;
-  /** The footer process indicator's [Enter] opens the Fleet panel (F2 also opens it; the process modal was retired). */
-  openFleetPanel: () => void;
+  /** The footer process indicator's [Enter] opens the Agents modal (F2 also opens it). */
+  openAgentsView: () => void;
   requestRender: () => void;
 };
 
@@ -211,8 +41,7 @@ export function handleIndicatorFocusToken(state: IndicatorFocusRouteState, token
     }
     if (token.logicalName === 'enter') {
       indicatorFocused = false;
-      // opens the Fleet panel (see openFleetPanel's own doc, line ~352).
-      state.openFleetPanel();
+      state.openAgentsView();
       state.requestRender();
       return { handled: true, indicatorFocused };
     }
@@ -349,8 +178,8 @@ export type KeyRouteState = {
    *  Anchors Enter-on-empty-composer's block-actions menu to that block
    *  instead of an arbitrary fixed line. */
   getBlockAnchorLine: () => number;
-  /** F2 opens+focuses the Fleet panel (which subsumes the retired process modal). */
-  openFleetPanel: () => void;
+  /** F2 opens the Agents modal. */
+  openAgentsView: () => void;
   modalOpened: (name: string) => void;
   saveUndoState: () => void;
   /** Break the undo coalescing group (call on cursor moves). */
@@ -641,12 +470,11 @@ export function handlePromptKeyToken(state: KeyRouteState, token: InputToken): {
     return { handled: true, prompt, cursorPos, inputScrollTop, commandMode, indicatorFocused };
   }
 
-  // F2 is now handled globally with toggle semantics (handler-shortcuts.ts)
-  // BEFORE it ever reaches this route; this branch is dead in the real
-  // pipeline but stays for panel-entry-points-reachable.test.ts's direct test.
+  // F2 is handled globally (handler-shortcuts.ts) before it reaches this
+  // route; this branch keeps the route correct when it is driven directly.
   if (token.logicalName === 'f2') {
     indicatorFocused = false;
-    state.openFleetPanel();
+    state.openAgentsView();
     return { handled: true, prompt, cursorPos, inputScrollTop, commandMode, indicatorFocused };
   }
 
@@ -656,8 +484,6 @@ export function handlePromptKeyToken(state: KeyRouteState, token: InputToken): {
 export type MouseRouteState = {
   conversationManager: ConversationManager | null;
   selection: SelectionManager;
-  panelManager: PanelManager;
-  panelMouseLayout: PanelMouseLayout | null;
   mouseDownRow: number;
   mouseDownCol: number;
   scrollTop: number;
@@ -667,20 +493,13 @@ export type MouseRouteState = {
   requestRender: () => void;
   handlePaste: () => void;
   handleCopy: () => void;
+  /** The view a footer screen row opens on a click (the usage rows open Usage). */
+  footerTargetAt?: (row: number) => FooterTarget | undefined;
+  openFooterTarget?: (target: FooterTarget) => void;
+  /** Screen rows above the transcript (the header, the chips, the gap under them); 2 when not given. */
+  bodyTopRow?: () => number;
 };
 
-function scrollPanelUnderMouse(
-  state: MouseRouteState,
-  token: Extract<InputToken, { type: 'mouse' }>,
-  deltaRows: number,
-): boolean {
-  const panel = getPanelUnderMouse(state.panelManager, state.panelMouseLayout, token.row, token.col);
-  if (!panel?.handleScroll) return false;
-  const consumed = panel.handleScroll(deltaRows);
-  if (consumed) state.requestRender();
-  return true;
-}
-
 export function handleMouseToken(state: MouseRouteState, token: InputToken): {
   handled: boolean;
   mouseDownRow: number;
@@ -692,20 +511,14 @@ export function handleMouseToken(state: MouseRouteState, token: InputToken): {
     return { handled: false, mouseDownRow, mouseDownCol };
   }
 
-  const headerH = 2;
-  const viewportRow = token.row - headerH;
+  // Mouse rows are 1-based; the transcript's first row is the one after the header block.
+  const viewportRow = token.row - 1 - (state.bodyTopRow?.() ?? 2);
 
   if (token.button === 64) {
-    if (scrollPanelUnderMouse(state, token, -3)) {
-      return { handled: true, mouseDownRow, mouseDownCol };
-    }
     state.scroll(-3);
     return { handled: true, mouseDownRow, mouseDownCol };
   }
   if (token.button === 65) {
-    if (scrollPanelUnderMouse(state, token, 3)) {
-      return { handled: true, mouseDownRow, mouseDownCol };
-    }
     state.scroll(3);
     return { handled: true, mouseDownRow, mouseDownCol };
   }
@@ -713,15 +526,17 @@ export function handleMouseToken(state: MouseRouteState, token: InputToken): {
     state.handlePaste();
     return { handled: true, mouseDownRow, mouseDownCol };
   }
+  // A click on a clickable footer row (press and release on the same row)
+  // opens its view; it never starts a text selection.
+  const footerTarget = state.footerTargetAt?.(token.row);
+  if (footerTarget && token.button === 0 && token.action === 'press') {
+    return { handled: true, mouseDownRow: token.row, mouseDownCol: token.col };
+  }
+  if (footerTarget && token.action === 'release' && mouseDownRow === token.row) {
+    state.openFooterTarget?.(footerTarget);
+    return { handled: true, mouseDownRow: -1, mouseDownCol: -1 };
+  }
   if (token.button === 0 && token.action === 'press') {
-    // Click on a workspace tab activates it (checked before starting a text
-    // selection so a tab click doesn't begin a drag-select).
-    const tabIndex = workspaceTabAtMouse(state.panelManager, state.panelMouseLayout, token.row, token.col);
-    if (tabIndex !== null) {
-      state.panelManager.activateWorkspaceIndex(tabIndex);
-      state.requestRender();
-      return { handled: true, mouseDownRow, mouseDownCol };
-    }
     mouseDownRow = token.row;
     mouseDownCol = token.col;
     state.selection.startSelection(token.col, viewportRow, state.scrollTop, state.viewportHeight, state.lineCount);