@pellux/goodvibes-tui 1.1.0 → 1.9.0

package/src/input/handler-onboarding.ts

@@ -1,8 +1,10 @@
-import { createOAuthLocalListener } from '@pellux/goodvibes-sdk/platform/config';
+import { createOAuthLocalListener, resolveDaemonEnabled } from '@pellux/goodvibes-sdk/platform/config';
 import { beginOpenAICodexLogin, exchangeOpenAICodexCode } from '@pellux/goodvibes-sdk/platform/config';
 import { summarizeError, openExternalUrl } from '@pellux/goodvibes-sdk/platform/utils';
 import { getProviderIdFromModel } from '../config/provider-model.ts';
 import { buildProviderAccountSnapshot } from '@/runtime/index.ts';
+import { handleConnectExistingDaemonForHandler, handleMigrateLegacyDaemonServiceForHandler } from './handler-onboarding-daemon-adopt.ts';
+import { detectLegacyUnit, resolveConfiguredServiceName } from '../runtime/legacy-daemon-migration.ts';
 import { OnboardingWizardController, type OnboardingWizardAction, type OnboardingWizardApplyFeedback } from './onboarding/onboarding-wizard.ts';
 import { handleCloudflareOnboardingActionForHandler, maybeProvisionCloudflareOnFinalApplyForHandler } from './handler-onboarding-cloudflare.ts';
 import { applyOnboardingRequest, collectOnboardingSnapshot, deleteWizardProgress, verifyOnboardingRequest, writeOnboardingCheckMarker, writeWizardProgress } from '../runtime/onboarding/index.ts';
@@ -74,6 +76,41 @@ function formatRuntimeStoppedFailureMessage(
   return `${binding.label} was disabled for ${disabledSurface}, but ${binding.host}:${binding.port} is still occupied. Settings were saved; another GoodVibes process or external service may still be running on that port.${statusDetail}`;
 }
 
+/**
+ * Verification item(s) for one runtime endpoint given whether it is expected
+ * to be running. Shared by both the "nothing server-backed selected" early
+ * path and the full server-backed path so daemon-by-default (the loopback
+ * daemon can be `expected` even when nothing else was selected) is handled
+ * identically in both places.
+ */
+function buildEndpointVerificationItems(
+  handler: InputHandler,
+  request: OnboardingApplyRequest,
+  endpoint: OnboardingRuntimeEndpoint,
+  expected: boolean,
+  state: OnboardingExternalServiceState | undefined,
+): OnboardingVerificationItem[] {
+    const idPrefix = endpoint === 'daemon' ? 'daemon' : 'http-listener';
+    if (expected) {
+      const active = isRuntimeEndpointActive(state, endpoint);
+      return [{
+        id: `runtime:${idPrefix}-active`,
+        status: active ? 'pass' : 'fail',
+        message: active
+          ? formatRuntimeActiveSuccessMessage(endpoint, state)
+          : formatRuntimeActiveFailureMessage(handler, request, endpoint, state),
+        target: 'service',
+      }];
+    }
+    if (!isRuntimeEndpointOccupyingConfiguredPort(state, endpoint)) return [];
+    return [{
+      id: `runtime:${idPrefix}-stopped`,
+      status: 'fail',
+      message: formatRuntimeStoppedFailureMessage(handler, request, endpoint, state),
+      target: 'service',
+    }];
+  }
+
 function showOnboardingApplyFeedbackForHandler(handler: InputHandler, feedback: OnboardingWizardApplyFeedback): void {
     handler.onboardingWizard.setApplyFeedback(feedback);
     const reviewIndex = handler.onboardingWizard.steps.findIndex((step) => step.id === 'review');
@@ -185,6 +222,15 @@ export function handleModelPickerCommitForHandler(handler: InputHandler): boolea
   }
 
 export async function handleOnboardingActionForHandler(handler: InputHandler, action: OnboardingWizardAction): Promise<void> {
+    if (action === 'connect-existing-daemon') {
+      await handleConnectExistingDaemonForHandler(handler);
+      return;
+    }
+    if (action === 'migrate-legacy-daemon-service') {
+      const confirm = handler.onboardingWizard.getBooleanFieldValue('network.migrate-legacy-daemon-confirm', false);
+      await handleMigrateLegacyDaemonServiceForHandler(handler, confirm);
+      return;
+    }
     if (action === 'start-openai-subscription') {
       await handler.handleOpenAiSubscriptionStart();
       return;
@@ -309,6 +355,7 @@ export async function handleOnboardingActionForHandler(handler: InputHandler, ac
     handler.requestRender();
   }
 
+
 export async function refreshOnboardingHydrationForHandler(handler: InputHandler, options: {
     readonly preserveValues?: boolean;
     readonly targetStepId?: string;
@@ -336,6 +383,9 @@ export async function refreshOnboardingHydrationForHandler(handler: InputHandler
             secretsManager: handler.uiServices.platform.secretsManager,
           }),
         },
+        legacyDaemon: {
+          detect: () => ({ ...detectLegacyUnit({ homeDir: handler.uiServices.environment.homeDirectory }), trackedServiceName: resolveConfiguredServiceName(handler.uiServices.platform.configManager) }),
+        },
       });
       if (!handler.onboardingWizard.active || hydrationSerial !== handler.onboardingHydrationSerial) return;
       handler.onboardingWizard.hydrateRuntimeState({ snapshot }, { resetValues: !(options.preserveValues ?? false) });
@@ -572,7 +622,8 @@ export function getOnboardingRuntimePostureForHandler(handler: InputHandler, req
     const serviceEnabled = getConfigValue('service.enabled') === true;
     const serviceAutostart = getConfigValue('service.autostart') === true;
     const restartOnFailure = getConfigValue('service.restartOnFailure') === true;
-    const daemonEnabled = getConfigValue('danger.daemon') === true || getConfigValue('controlPlane.enabled') === true;
+    const daemonReader = { get: (key: string): boolean | string | number | undefined => getConfigValue(key) as boolean | string | number | undefined };
+    const daemonEnabled = resolveDaemonEnabled(daemonReader) || getConfigValue('controlPlane.enabled') === true;
     const listenerEnabled = getConfigValue('danger.httpListener') === true;
     const webEnabled = getConfigValue('web.enabled') === true;
     const controlPlaneRemote = getConfigValue('controlPlane.hostMode') === 'network'
@@ -593,7 +644,10 @@ export function getOnboardingRuntimePostureForHandler(handler: InputHandler, req
       restartOnFailure,
       expectedDaemon: daemonEnabled || webEnabled,
       expectedHttpListener: listenerEnabled,
-      serverBacked: serviceEnabled || daemonEnabled || listenerEnabled || webEnabled,
+      // Daemon-by-default: the loopback session daemon can be expected to run
+      // even when nothing else was explicitly enabled here, so it does not
+      // belong in "did onboarding opt into a network-facing bundle."
+      serverBacked: serviceEnabled || listenerEnabled || webEnabled,
       remoteExposure,
     };
   }
@@ -683,28 +737,20 @@ export function verifyOnboardingRuntimePostureForHandler(handler: InputHandler,
         }];
       }
 
-      const stoppedItems: OnboardingVerificationItem[] = [];
-      if (isRuntimeEndpointOccupyingConfiguredPort(externalState, 'daemon')) {
-        stoppedItems.push({
-          id: 'runtime:daemon-stopped',
-          status: 'fail',
-          message: formatRuntimeStoppedFailureMessage(handler, request, 'daemon', externalState),
-          target: 'service',
-        });
-      }
-      if (isRuntimeEndpointOccupyingConfiguredPort(externalState, 'httpListener')) {
-        stoppedItems.push({
-          id: 'runtime:http-listener-stopped',
-          status: 'fail',
-          message: formatRuntimeStoppedFailureMessage(handler, request, 'httpListener', externalState),
-          target: 'service',
-        });
-      }
+      // Daemon-by-default: expectedDaemon can be true here (loopback daemon
+      // running for cross-surface visibility) even though nothing else was
+      // selected, so this is not simply "everything must be stopped."
+      const stoppedItems: OnboardingVerificationItem[] = [
+        ...buildEndpointVerificationItems(handler, request, 'daemon', posture.expectedDaemon, externalState),
+        ...buildEndpointVerificationItems(handler, request, 'httpListener', posture.expectedHttpListener, externalState),
+      ];
       if (externalState && stoppedItems.length === 0) {
         stoppedItems.push({
           id: 'runtime:external-services-stopped',
           status: 'pass',
-          message: 'Background daemon and HTTP listener are stopped for Local TUI Only.',
+          message: posture.expectedDaemon
+            ? 'HTTP listener is stopped for Local TUI Only; the background daemon is running as expected for cross-surface visibility on this machine.'
+            : 'Background daemon and HTTP listener are stopped for Local TUI Only.',
           target: 'service',
         });
       }
@@ -742,44 +788,10 @@ export function verifyOnboardingRuntimePostureForHandler(handler: InputHandler,
       });
     }
 
-    if (posture.expectedDaemon) {
-      const daemonActive = isRuntimeEndpointActive(externalState, 'daemon');
-      items.push({
-        id: 'runtime:daemon-active',
-        status: daemonActive ? 'pass' : 'fail',
-        message: daemonActive
-          ? formatRuntimeActiveSuccessMessage('daemon', externalState)
-          : formatRuntimeActiveFailureMessage(handler, request, 'daemon', externalState),
-        target: 'service',
-      });
-    }
-    if (!posture.expectedDaemon && isRuntimeEndpointOccupyingConfiguredPort(externalState, 'daemon')) {
-      items.push({
-        id: 'runtime:daemon-stopped',
-        status: 'fail',
-        message: formatRuntimeStoppedFailureMessage(handler, request, 'daemon', externalState),
-        target: 'service',
-      });
-    }
-    if (posture.expectedHttpListener) {
-      const httpListenerActive = isRuntimeEndpointActive(externalState, 'httpListener');
-      items.push({
-        id: 'runtime:http-listener-active',
-        status: httpListenerActive ? 'pass' : 'fail',
-        message: httpListenerActive
-          ? formatRuntimeActiveSuccessMessage('httpListener', externalState)
-          : formatRuntimeActiveFailureMessage(handler, request, 'httpListener', externalState),
-        target: 'service',
-      });
-    }
-    if (!posture.expectedHttpListener && isRuntimeEndpointOccupyingConfiguredPort(externalState, 'httpListener')) {
-      items.push({
-        id: 'runtime:http-listener-stopped',
-        status: 'fail',
-        message: formatRuntimeStoppedFailureMessage(handler, request, 'httpListener', externalState),
-        target: 'service',
-      });
-    }
+    items.push(
+      ...buildEndpointVerificationItems(handler, request, 'daemon', posture.expectedDaemon, externalState),
+      ...buildEndpointVerificationItems(handler, request, 'httpListener', posture.expectedHttpListener, externalState),
+    );
 
     return items;
   }

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

@@ -5,7 +5,6 @@ import type { CapabilityFilter, CategoryFilter, ModelPickerModal } from './model
 import { MODEL_PICKER_CHROME_LINES } from '../renderer/model-picker-overlay.ts';
 import { resolveAndValidatePath } from '@pellux/goodvibes-sdk/platform/utils';
 import { logger } from '@pellux/goodvibes-sdk/platform/utils';
-import type { ProcessEntry } from '../renderer/process-modal.ts';
 import type { BlockActionId } from '../renderer/block-actions.ts';
 
 type ModelPickerRouteState = {
@@ -23,13 +22,15 @@ export function handleModelPickerToken(state: ModelPickerRouteState, token: Inpu
 
   if (token.type === 'key') {
     if (token.logicalName === 'escape') {
-      if (state.modelPicker.searchFocused && state.modelPicker.mode !== 'contextCap') {
-        if (state.modelPicker.query.length > 0) {
-          state.modelPicker.clearQuery();
-        } else {
-          state.modelPicker.blurSearch();
-        }
-      } else if (state.modelPicker.query.length > 0) {
+      // W3-T2: search now starts focused by default (see
+      // ModelPickerModal.openAllModels() doc comment). Escape with an empty
+      // query used to only blurSearch() here, leaving the picker open and
+      // requiring a SECOND Escape to actually close it — confusing on a
+      // freshly-opened, untouched picker where there is nothing to "clear".
+      // Clearing a non-empty query is still a distinct first Escape (whether
+      // or not search is focused); an empty query falls through to the same
+      // effort/contextCap/previousMode/close cascade as before.
+      if (state.modelPicker.mode !== 'contextCap' && state.modelPicker.query.length > 0) {
         state.modelPicker.clearQuery();
       } else if (state.modelPicker.mode === 'effort') {
         state.modelPicker.mode = 'model';
@@ -93,6 +94,13 @@ export function handleModelPickerToken(state: ModelPickerRouteState, token: Inpu
         }
         state.modelPicker.close();
         if (state.modalStack[state.modalStack.length - 1] === 'modelPicker') state.modalStack.pop();
+      } else if (mode === 'embeddingProvider') {
+        const selectedProvider = state.modelPicker.embeddingProviders[idx];
+        if (selectedProvider) {
+          state.commandContext?.completeEmbeddingProviderSelection?.(selectedProvider.id);
+        }
+        state.modelPicker.close();
+        if (state.modalStack[state.modalStack.length - 1] === 'modelPicker') state.modalStack.pop();
       } else if (mode === 'contextCap') {
         const capModel = state.modelPicker.contextCapPendingModel;
         if (capModel) {
@@ -123,7 +131,12 @@ export function handleModelPickerToken(state: ModelPickerRouteState, token: Inpu
       }
       if (state.modelPicker.canFocusSearch() && !state.modelPicker.searchFocused && state.modelPicker.selectedIndex === 0) {
         state.modelPicker.focusSearch();
-      } else if (!state.modelPicker.searchFocused) {
+      } else if (state.modelPicker.searchFocused) {
+        // Symmetric with 'down' (below): search now starts focused by default
+        // (W3-T2), so 'up' must have a way out of it too, or it would be a
+        // silent no-op the very first time a user reaches for the list.
+        state.modelPicker.blurSearch();
+      } else {
         const maxVis = Math.max(5, state.getViewportHeight() - MODEL_PICKER_CHROME_LINES - 4);
         state.modelPicker.moveUp(maxVis);
       }
@@ -201,99 +214,10 @@ function cycleCapabilityFilter(modelPicker: ModelPickerModal): void {
   modelPicker.setCapabilityFilter(cycle[(cur + 1) % cycle.length]!);
 }
 
-type ProcessRouteState = {
-  processModal: {
-    active: boolean;
-    moveUp: () => void;
-    moveDown: () => void;
-    getSelected: () => ProcessEntry | undefined;
-    close: () => void;
-    open: () => void;
-    killSelected: () => boolean;
-    refresh: () => void;
-  };
-  liveTailModal: {
-    open: (entry: ProcessEntry) => void;
-  };
-  agentDetailModal: {
-    open: (id: string) => void;
-  };
-  modalOpened: (name: string) => void;
-  requestRender: () => void;
-  handleEscape: () => void;
-};
-
-export function handleProcessModalToken(state: ProcessRouteState, token: InputToken): boolean {
-  if (!state.processModal.active) return false;
-
-  if (token.type === 'key') {
-    if (token.logicalName === 'escape') {
-      state.handleEscape();
-      return true;
-    }
-    if (token.logicalName === 'up') state.processModal.moveUp();
-    else if (token.logicalName === 'down') state.processModal.moveDown();
-    else if (token.logicalName === 'enter') {
-      const entry = state.processModal.getSelected();
-      if (entry) {
-        if (entry.type === 'agent') {
-          state.modalOpened('agentDetail');
-          state.processModal.close();
-          state.agentDetailModal.open(entry.id);
-        } else {
-          state.modalOpened('liveTail');
-          state.processModal.close();
-          state.liveTailModal.open(entry);
-        }
-      }
-    }
-  } else if (token.type === 'text' && token.value === 'k') {
-    const killed = state.processModal.killSelected();
-    if (killed) state.processModal.refresh();
-  }
-
-  state.requestRender();
-  return true;
-}
-
-type LiveTailRouteState = {
-  liveTailModal: {
-    active: boolean;
-    scrollUp: () => void;
-    scrollDown: () => void;
-    killProcess: () => void;
-    close: () => void;
-  };
-  processModal: {
-    open: () => void;
-  };
-  requestRender: () => void;
-  handleEscape: () => void;
-};
-
-export function handleLiveTailToken(state: LiveTailRouteState, token: InputToken): boolean {
-  if (!state.liveTailModal.active) return false;
-
-  const killAndReturn = (): void => {
-    state.liveTailModal.killProcess();
-    state.handleEscape();
-  };
-
-  if (token.type === 'key') {
-    if (token.logicalName === 'escape') {
-      state.handleEscape();
-      return true;
-    }
-    if (token.logicalName === 'up') state.liveTailModal.scrollUp();
-    else if (token.logicalName === 'down') state.liveTailModal.scrollDown();
-    else if (token.logicalName === 'k') killAndReturn();
-  } else if (token.type === 'text' && token.value === 'k') {
-    killAndReturn();
-  }
-
-  state.requestRender();
-  return true;
-}
+// W6.2 e / W6.1 retirement: the process-modal and live-tail-modal token routes
+// were removed with those modals (F2 now opens Fleet, which subsumes the live
+// process tree). ProcessModal/AgentDetailModal/LiveTailModal were structurally
+// unreachable after the F2 repoint and were deleted.
 
 type EscapeOnlyModalRouteState = {
   active: boolean;

package/src/input/handler-shortcuts.ts

@@ -55,13 +55,16 @@ export function handleGlobalShortcutToken(
 ): boolean {
   if (token.type !== 'key') return false;
 
-  // Fast-path: bare pageup/pagedown have no keybinding entry.
-  if (token.logicalName === 'pageup') {
+  // Fast-path: BARE pageup/pagedown scroll the transcript. The `!token.ctrl`
+  // guard is load-bearing: Ctrl+PageUp/PageDown are the panel-tab-prev/next
+  // chords, so they must fall through to the keybinding lookup below instead of
+  // being swallowed here as a scroll.
+  if (token.logicalName === 'pageup' && !token.ctrl) {
     if (state.panelFocused) return false;
     state.scroll(-Math.max(1, viewportHeight - 2));
     return true;
   }
-  if (token.logicalName === 'pagedown') {
+  if (token.logicalName === 'pagedown' && !token.ctrl) {
     if (state.panelFocused) return false;
     state.scroll(Math.max(1, viewportHeight - 2));
     return true;
@@ -72,6 +75,23 @@ export function handleGlobalShortcutToken(
     return true;
   }
 
+  // Bare F2 is also not in the keybinding table (hardcoded, like pageup/
+  // pagedown/escape above) — it must be handled here, GLOBALLY, rather than
+  // in handlePromptKeyToken (handler-feed-routes.ts) where it used to live.
+  // That location only ever runs when the panel workspace does NOT already
+  // own focus (handlePanelFocusToken, which runs before it, swallows every
+  // unclaimed key once panelFocused is true and reports it handled). The
+  // practical effect was that F2 could OPEN+focus the Fleet panel exactly
+  // once; every subsequent press while already focused vanished silently —
+  // "F2 pressed 4x never closed the panel" (UX-C evaluator finding). Routing
+  // it here, before handlePanelFocusToken ever sees the token, gives F2 the
+  // same toggle semantics as Ctrl+O below, matching how Ctrl+P/panel-picker
+  // was already reachable regardless of panelFocused.
+  if (token.logicalName === 'f2' && !token.ctrl && !token.meta) {
+    toggleFleetPanel(state);
+    return true;
+  }
+
   // O(1) lookup via inverted map.
   const kb = state.keybindingsManager;
   const action = kb.lookup(token);
@@ -101,6 +121,20 @@ export function handleGlobalShortcutToken(
     case 'panel-close': {
       const pm = state.panelManager;
       const active = pm.getActivePanel();
+      // Wave-3: give the active panel a chance to consume Ctrl+X for an
+      // in-panel action (FleetPanel session-tab detach) before it closes the
+      // panel outright — see Panel.interceptPanelClose's doc comment.
+      if (active?.interceptPanelClose?.()) {
+        // UX-C fix: a consumed Ctrl+X (the Fleet panel's session-tab detach)
+        // used to leave panelFocused untouched, so focus stayed on the panel
+        // — the evaluator's "Ctrl+X detach landed focus in the panel and a
+        // typed question became nav keys". Detach is a leave-taking action:
+        // like the ordinary close below, it hands control back to the
+        // composer rather than leaving the user stranded on the fleet tree.
+        state.panelFocused = false;
+        state.requestRender();
+        return true;
+      }
       if (active) {
         pm.close(active.id);
         state.requestRender();
@@ -150,27 +184,27 @@ export function handleGlobalShortcutToken(
       // Alt+1..9: jump directly to the Nth workspace tab. Routed globally (like
       // panel-tab-next/prev) so the jump works whether focus is on the prompt or
       // the workspace; gated on visibility, matching cyclePanelTab semantics.
+      // UX-C: a chord jump is "I'm going panel-driving" (focus rule 1a) — the
+      // jump now also grabs keyboard focus, matching F2/Ctrl+O/Ctrl+P, so j/k
+      // land in the newly-active tab immediately instead of the composer.
       const pm = state.panelManager;
       if (pm.isVisible()) {
         const index = Number(action.slice('panel-tab-'.length)) - 1;
         pm.activateWorkspaceIndex(index);
+        pm.focusPanels();
+        state.panelFocused = true;
         state.requestRender();
       }
       return true;
     }
 
     case 'panel-ops': {
-      // Ctrl+O: open the Ops Control panel. Prefer the command-context callback
-      // (wired only when the operator-control-plane feature flag is on) so it
-      // reuses whatever show/open semantics bootstrap.ts set up; otherwise fall
-      // back to opening the always-registered 'ops-control' panel type directly
-      // — it renders an honest not-configured state when its deps are absent.
-      if (state.commandContext?.openOpsPanel) {
-        state.commandContext.openOpsPanel();
-      } else {
-        state.panelManager.open('ops-control');
-        state.requestRender();
-      }
+      // Ctrl+O: TOGGLE the Fleet panel (UX-C — same semantics as F2 above; see
+      // toggleFleetPanel's doc comment). The former Ops Control panel was
+      // retired to an 'ops-control' -> 'fleet' alias (W6.1); rather than route
+      // through the now-aliased openOpsPanel callback (which opens without
+      // transferring focus), this operates on 'fleet' directly.
+      toggleFleetPanel(state);
       return true;
     }
 
@@ -348,3 +382,29 @@ export function handleGlobalShortcutToken(
       return false;
   }
 }
+
+/**
+ * toggleFleetPanel — the shared F2 / Ctrl+O TOGGLE (UX-C item 2): if the
+ * Fleet panel is open AND the panel workspace currently owns keyboard focus
+ * with Fleet as the active tab, the chord CLOSES it and returns focus to the
+ * composer; if Fleet is open but not the focused/active tab, the chord brings
+ * it to front and focuses it; if Fleet isn't open at all, the chord opens and
+ * focuses it. Uses `state.panelFocused` (not a panelManager query) for the
+ * focus check, consistent with every other case in this file and with what
+ * the mocked PanelManager test doubles in global-shortcuts.test.ts actually
+ * implement.
+ */
+function toggleFleetPanel(state: GlobalShortcutRouteState): void {
+  const pm = state.panelManager;
+  const fleetOpen = pm.getAllOpen().some((p) => p.id === 'fleet');
+  const fleetIsFocusedActive = fleetOpen && state.panelFocused && pm.getActivePanel()?.id === 'fleet';
+  if (fleetIsFocusedActive) {
+    pm.close('fleet');
+    state.panelFocused = false;
+  } else {
+    pm.open('fleet');
+    pm.focusPanels();
+    state.panelFocused = true;
+  }
+  state.requestRender();
+}

package/src/input/handler-types.ts

@@ -23,14 +23,12 @@ import type { SelectionManager } from './selection.ts';
 import type { InfiniteBuffer } from '../core/history.ts';
 import type { AutocompleteEngine } from './autocomplete.ts';
 import type { BookmarkModal } from './bookmark-modal.ts';
-import type { AgentDetailModal } from '../renderer/agent-detail-modal.ts';
-import type { LiveTailModal } from '../renderer/live-tail-modal.ts';
 import type { SettingsModal } from './settings-modal.ts';
 import type { McpWorkspace } from './mcp-workspace.ts';
 import type { SessionPickerModal } from './session-picker-modal.ts';
+import type { ConfigModal } from './config-modal.ts';
 import type { ProfilePickerModal } from './profile-picker-modal.ts';
 import type { ContextInspectorModal } from '../renderer/context-inspector.ts';
-import type { ProcessModal } from '../renderer/process-modal.ts';
 import type { FilePickerModal } from './file-picker.ts';
 import type { BlockActionsMenu } from '../renderer/block-actions.ts';
 import type { SelectionModal } from './selection-modal.ts';
@@ -105,14 +103,12 @@ export interface InputHandlerLike {
 
   // ── Modal objects ─────────────────────────────────────────────────────────
   bookmarkModal: BookmarkModal;
-  agentDetailModal: AgentDetailModal;
-  liveTailModal: LiveTailModal;
   settingsModal: SettingsModal;
   mcpWorkspace: McpWorkspace;
   sessionPickerModal: SessionPickerModal;
   profilePickerModal: ProfilePickerModal;
+  configModal: ConfigModal;
   contextInspectorModal: ContextInspectorModal;
-  processModal: ProcessModal;
   modelPicker: ModelPickerModal;
   filePicker: FilePickerModal;
   blockActionsMenu: BlockActionsMenu;

package/src/input/handler-ui-state.ts

@@ -55,14 +55,12 @@ export type ActiveModalState = {
   helpOverlayActive: boolean;
   shortcutsOverlayActive: boolean;
   bookmarkModal: { active: boolean; close: () => void };
-  agentDetailModal: { active: boolean; close: () => void };
-  liveTailModal: { active: boolean; close: () => void };
   settingsModal: { active: boolean; close: () => void };
   mcpWorkspace?: { active: boolean; close: () => void; reopen: () => void };
   sessionPickerModal: { active: boolean; close: () => void };
   profilePickerModal: { active: boolean; close: () => void };
+  configModal: { active: boolean; close: () => void };
   contextInspectorModal: { active: boolean; close: () => void };
-  processModal: { active: boolean; close: () => void };
   modelPicker: { active: boolean; close: () => void };
   filePicker: { active: boolean; close: () => void };
   blockActionsMenu: { active: boolean; close: () => void };
@@ -75,14 +73,12 @@ export function getActiveModalName(state: ActiveModalState): string | null {
   if (state.helpOverlayActive) return 'help';
   if (state.shortcutsOverlayActive) return 'shortcuts';
   if (state.bookmarkModal.active) return 'bookmark';
-  if (state.agentDetailModal.active) return 'agentDetail';
-  if (state.liveTailModal.active) return 'liveTail';
   if (state.settingsModal.active) return 'settings';
   if (state.mcpWorkspace?.active) return 'mcpWorkspace';
   if (state.sessionPickerModal.active) return 'sessionPicker';
   if (state.profilePickerModal.active) return 'profilePicker';
+  if (state.configModal.active) return 'config';
   if (state.contextInspectorModal.active) return 'contextInspector';
-  if (state.processModal.active) return 'process';
   if (state.modelPicker.active) return 'modelPicker';
   if (state.filePicker.active) return 'filePicker';
   if (state.blockActionsMenu.active) return 'blockActions';
@@ -96,14 +92,12 @@ export type ModalCloseOps = {
   resetHelp: () => void;
   resetShortcuts: () => void;
   closeBookmark: () => void;
-  closeAgentDetail: () => void;
-  closeLiveTail: () => void;
   closeSettings: () => void;
   closeMcpWorkspace: () => void;
   closeSessionPicker: () => void;
   closeProfilePicker: () => void;
+  closeConfigModal: () => void;
   closeContextInspector: () => void;
-  closeProcess: () => void;
   closeModelPicker: () => void;
   closeFilePicker: () => void;
   closeBlockActions: () => void;
@@ -123,12 +117,6 @@ export function closeModalByName(name: string, ops: ModalCloseOps): void {
     case 'bookmark':
       ops.closeBookmark();
       break;
-    case 'agentDetail':
-      ops.closeAgentDetail();
-      break;
-    case 'liveTail':
-      ops.closeLiveTail();
-      break;
     case 'settings':
       ops.closeSettings();
       break;
@@ -141,12 +129,12 @@ export function closeModalByName(name: string, ops: ModalCloseOps): void {
     case 'profilePicker':
       ops.closeProfilePicker();
       break;
+    case 'config':
+      ops.closeConfigModal();
+      break;
     case 'contextInspector':
       ops.closeContextInspector();
       break;
-    case 'process':
-      ops.closeProcess();
-      break;
     case 'modelPicker':
       ops.closeModelPicker();
       break;
@@ -172,10 +160,10 @@ export type ModalOpenOps = {
   openHelp: () => void;
   openShortcuts: () => void;
   openBookmark: () => void;
-  openProcess: () => void;
   openContextInspector: () => void;
   openMcpWorkspace?: () => void;
   openOnboarding?: () => void;
+  openConfigModal?: () => void;
   openCommandMode: () => void;
 };
 
@@ -190,15 +178,15 @@ export function reopenModalByName(name: string, ops: ModalOpenOps): void {
     case 'bookmark':
       ops.openBookmark();
       break;
-    case 'process':
-      ops.openProcess();
-      break;
     case 'contextInspector':
       ops.openContextInspector();
       break;
     case 'mcpWorkspace':
       ops.openMcpWorkspace?.();
       break;
+    case 'config':
+      ops.openConfigModal?.();
+      break;
     case 'onboarding':
       ops.openOnboarding?.();
       break;