@pellux/goodvibes-tui 2.0.14 → 2.0.16

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

@@ -36,7 +36,7 @@ export type PanelFocusRouteState = {
    * 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 —
+   * 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;
@@ -88,12 +88,12 @@ export function handlePanelFocusToken(state: PanelFocusRouteState, token: InputT
   }
 
   if (token.type === 'key') {
-    // I6: two-stage Escape — give the panel a chance to consume escape first
+    // 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
+      // 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()
@@ -117,7 +117,7 @@ export function handlePanelFocusToken(state: PanelFocusRouteState, token: InputT
     }
     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) —
+    // 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).
@@ -156,7 +156,7 @@ export function handlePanelFocusToken(state: PanelFocusRouteState, token: InputT
     // 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
+    // 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');
@@ -164,7 +164,7 @@ export function handlePanelFocusToken(state: PanelFocusRouteState, token: InputT
       return { handled: true, panelFocused };
     }
 
-    // item 5 — paste flood guard (rate-based; see panel-paste-flood-guard.ts).
+    // 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) {
@@ -293,7 +293,7 @@ export function handlePromptTextToken(state: TextRouteState, token: InputToken):
 
   if (prompt === '/') {
     // Arm commandMode as soon as the prompt becomes a bare '/', regardless of
-    // whether commandRegistry has been (re)attached yet — this is a one-shot
+    // whether commandRegistry has been (re)attached yet, this is a one-shot
     // transition (the only place commandMode ever becomes true), and gating
     // it on commandRegistry meant a transient null during a modal/overlay
     // handoff would permanently miss the window: every following keystroke
@@ -329,7 +329,7 @@ export type KeyRouteState = {
   commandContext: CommandContext | undefined;
   /**
    * Deliver a concealed submission. When it returns true, concealed mode was
-   * active and consumed the value — the plaintext went straight to the
+   * active and consumed the value, the plaintext went straight to the
    * requester, bypassing input history and the transcript. Optional so bare
    * test callers omit it.
    */
@@ -337,14 +337,14 @@ export type KeyRouteState = {
   /**
    * Optional: only used by the enter-key desync safety net below (a stray
    * slash-prefixed submission with commandMode somehow still false). When
-   * absent, that fallback simply doesn't trigger — the primary fix (arming
+   * absent, that fallback simply doesn't trigger, the primary fix (arming
    * commandMode on the '/' keystroke regardless of registry attachment, in
    * handlePromptTextToken above) is what actually prevents the desync.
    */
   commandRegistry?: CommandRegistry | null;
   autocomplete: AutocompleteEngine | null;
   blockActionsMenu: { open: (block: BlockMeta) => void };
-  /** The absolute history line of the bottom-most visible block — the block
+  /** The absolute history line of the bottom-most visible block, the block
    *  the user is actually looking at (see getViewportBottomLine's doc).
    *  Anchors Enter-on-empty-composer's block-actions menu to that block
    *  instead of an arbitrary fixed line. */
@@ -455,7 +455,7 @@ export function handlePromptKeyToken(state: KeyRouteState, token: InputToken): {
     }
     if (text) {
       // Safety net for a desynced commandMode: text is command-shaped (starts
-      // with '/') but commandMode never armed — e.g. the '/' keystroke landed
+      // with '/') but commandMode never armed, e.g. the '/' keystroke landed
       // during a modal/overlay handoff window. handleCommandModeToken (the
       // normal dispatch path for '/name ...') never runs in this state since
       // it early-returns when commandMode is false, so without this the text
@@ -611,12 +611,12 @@ export function handlePromptKeyToken(state: KeyRouteState, token: InputToken): {
   }
 
   if (token.logicalName === 'down') {
-    // Announce the focus move honestly — this handoff is otherwise silent
+    // Announce the focus move honestly, this handoff is otherwise silent
     // and easy to miss (the next keystrokes look like they're doing nothing,
     // because they're now landing on the indicator, not the composer).
     const announceIndicatorFocus = () => {
       indicatorFocused = true;
-      state.commandContext?.print('[Down] Focus moved to the process indicator — Up or Esc returns to the composer.');
+      state.commandContext?.print('[Down] Focus moved to the process indicator; Up or Esc returns to the composer.');
     };
     const move = computeCursorVerticalMove(prompt, cursorPos, inputScrollTop, state.contentWidth, state.maxInputRows, 1);
     if (move.moved) {

package/src/input/handler-feed.ts

@@ -48,39 +48,39 @@ import type { KillRing } from './kill-ring.ts';
 import type { FocusTracker } from '@pellux/goodvibes-sdk/platform/runtime/operations';
 
 /**
- * InputFeedContext — The single long-lived context object passed to feedInputTokens
+ * InputFeedContext, The single long-lived context object passed to feedInputTokens
  * on every keystroke. Allocated once at InputHandler construction; mutated in place
  * per-feed to avoid per-keystroke GC pressure from ~80-field object allocation.
  *
  * **Mutable per-feed** (synced from handler at the top of every feed() call, and
  * updated inside action closures via syncFeedContextMutableFields):
- *   - `prompt`, `cursorPos` — current text buffer state
- *   - `commandMode`, `panelFocused`, `indicatorFocused` — focus-mode flags
- *   - `helpOverlayActive`, `helpScrollOffset` — help overlay visibility and scroll
- *   - `shortcutsOverlayActive`, `shortcutsScrollOffset` — shortcuts overlay state
- *   - `nextPasteId`, `nextImageId` — monotonically increasing ID counters
- *   - `mouseDownRow`, `mouseDownCol` — drag-tracking coordinates
- *   - `contentWidth` — reflow width (semi-stable; synced at feed() entry only)
- *   - `selectionCallback` — current in-flight selection modal callback (nullable)
- *   - `requestRender` — swapped per-feed to a buffered version, restored after
+ *   - `prompt`, `cursorPos`, current text buffer state
+ *   - `commandMode`, `panelFocused`, `indicatorFocused`, focus-mode flags
+ *   - `helpOverlayActive`, `helpScrollOffset`, help overlay visibility and scroll
+ *   - `shortcutsOverlayActive`, `shortcutsScrollOffset`, shortcuts overlay state
+ *   - `nextPasteId`, `nextImageId`, monotonically increasing ID counters
+ *   - `mouseDownRow`, `mouseDownCol`, drag-tracking coordinates
+ *   - `contentWidth`, reflow width (semi-stable; synced at feed() entry only)
+ *   - `selectionCallback`, current in-flight selection modal callback (nullable)
+ *   - `requestRender`, swapped per-feed to a buffered version, restored after
  *
  * **Stable service handles** (set once at construction, never reallocated):
- *   - `commandRegistry`, `commandContext` — wired via setCommandRegistry() after
+ *   - `commandRegistry`, `commandContext`, wired via setCommandRegistry() after
  *     construction; synced at feed() entry (not per-action) since no action changes them
- *   - `autocomplete` — wired after construction; synced at feed() entry
- *   - `inputHistory`, `conversationManager` — late-wired service handles; synced at
+ *   - `autocomplete`, wired after construction; synced at feed() entry
+ *   - `inputHistory`, `conversationManager`, late-wired service handles; synced at
  *     feed() entry only since no in-feed action rewires them
- *   - `pasteRegistry`, `imageRegistry` — owned Maps, never replaced
+ *   - `pasteRegistry`, `imageRegistry`, owned Maps, never replaced
  *   - `selectionModal`, `bookmarkModal`, `settingsModal`, `sessionPickerModal`,
- *     `profilePickerModal` — modal objects constructed once in InputHandler constructor
+ *     `profilePickerModal`, modal objects constructed once in InputHandler constructor
  *   - `filePicker`, `modelPicker`, `onboardingWizard`,
- *     `contextInspectorModal`, `blockActionsMenu`, `searchManager`, `historySearch` —
+ *     `contextInspectorModal`, `blockActionsMenu`, `searchManager`, `historySearch`,
  *     service objects constructed once
- *   - `panelManager`, `keybindingsManager` — from uiServices, stable for app lifetime
- *   - `modalStack` — reference to the handler's shared array (mutated in place)
- *   - `getHistory`, `getViewportHeight`, `getScrollTop`, `scroll`, `exitApp` — stable
+ *   - `panelManager`, `keybindingsManager`, from uiServices, stable for app lifetime
+ *   - `modalStack`, reference to the handler's shared array (mutated in place)
+ *   - `getHistory`, `getViewportHeight`, `getScrollTop`, `scroll`, `exitApp`, stable
  *     callbacks bound in the InputHandler constructor
- *   - All method closures (`modalOpened`, `handleEscape`, etc.) — bound once at init
+ *   - All method closures (`modalOpened`, `handleEscape`, etc.), bound once at init
  *
  * **Rationale:** per-feed mutation avoids per-keystroke allocation cost; stable
  * references are service handles whose identity never changes after construction.
@@ -130,9 +130,9 @@ export interface InputFeedContext {
   inputHistory: InputHistory | null;
   conversationManager: ConversationManager | null;
   readonly killRing: KillRing;
-  /** Terminal focus tracker — updated here from 'focus' tokens, read by the unfocused-alert notifiers in core/. */
+  /** Terminal focus tracker, updated here from 'focus' tokens, read by the unfocused-alert notifiers in core/. */
   readonly focusTracker: FocusTracker;
-  /** item 5 — the paste-flood guard's persistent state, mutated in place across tokens by handlePanelFocusToken (see panel-paste-flood-guard.ts). Never reallocated. */
+  /** item 5, the paste-flood guard's persistent state, mutated in place across tokens by handlePanelFocusToken (see panel-paste-flood-guard.ts). Never reallocated. */
   readonly panelBurstGuard: PanelBurstGuardState;
   readonly getHistory: () => InfiniteBuffer;
   readonly getViewportHeight: () => number;
@@ -186,7 +186,7 @@ export function feedInputTokens(context: InputFeedContext, tokens: readonly Inpu
   const keybindings = context.keybindingsManager;
 
   // Shared opener for the Fleet panel: makes it visible AND transfers keyboard
-  // focus to it. panelManager.open() only makes the panel active — focus is a
+  // focus to it. panelManager.open() only makes the panel active, focus is a
   // separate axis (see PanelManager.focusPanels()/getFocusTarget()); without the
   // focusPanels() call, j/k/i/K land silently in the composer until Tab. Used by
   // the footer indicator's [Enter] and by F2 (F2 and the footer
@@ -199,20 +199,20 @@ export function feedInputTokens(context: InputFeedContext, tokens: readonly Inpu
   };
 
   // Paste-ness is a per-TOKEN property, computed at the handlePanelFocusToken
-  // call below — never a per-feed character sum. The SDK tokenizer emits a
+  // call below, never a per-feed character sum. The SDK tokenizer emits a
   // bracketed paste (\x1b[?2004h, enabled in main.ts terminal init) as ONE
-  // 'text' token holding the whole payload, while discrete keystrokes — even
-  // several batched into one feed() by render-tick latency — arrive as
+  // 'text' token holding the whole payload, while discrete keystrokes, even
+  // several batched into one feed() by render-tick latency, arrive as
   // separate 1-char 'text' tokens. The old per-feed sum misread two quick nav
   // keystrokes (e.g. j then k in one drain) as a "burst" and yanked focus.
   // item 5's flood guard reuses that same per-token model but adds
-  // real timing: one `now` per feed() call (not per token) is intentional —
+  // real timing: one `now` per feed() call (not per token) is intentional,
   // a genuine flood delivers many tokens in one drain, and they should all
   // measure as arriving "at once", not spread across meaningless sub-ms noise.
   const now = Date.now();
   for (const token of tokens) {
     // Focus-reporting tokens (CSI ?1004h) never reach the composer or any
-    // modal route — consumed here, first, unconditionally. No render needed.
+    // modal route, consumed here, first, unconditionally. No render needed.
     if (token.type === 'focus') {
       context.focusTracker.setFocused(token.action === 'in');
       continue;
@@ -289,7 +289,7 @@ export function feedInputTokens(context: InputFeedContext, tokens: readonly Inpu
     if (token.type === 'key') {
       // Focus can never disagree with workspace visibility: PanelManager owns
       // focusTarget and self-heals it, and context.panelFocused was seeded from
-      // it at feed entry — so no manual "unfocus if panels vanished" patch is
+      // it at feed entry, so no manual "unfocus if panels vanished" patch is
       // needed here anymore.
       // Snapshot these four BEFORE dispatch so the write-back below can tell
       // a pipeline-driven change (see comment there) from a stale copy.
@@ -332,7 +332,7 @@ export function feedInputTokens(context: InputFeedContext, tokens: readonly Inpu
       if (handleGlobalShortcutToken(shortcutState, token, viewportHeight)) {
         // Some branches (handleEscape, handleCtrlC, handleUndo, handleRedo,
         // handlePaste) mutate handler state directly and immediately sync it
-        // into this same `context` via syncFeedContextMutableFields — e.g.
+        // into this same `context` via syncFeedContextMutableFields, e.g.
         // Escape closing the slash-command palette clears context.prompt and
         // context.commandMode on the live handler mid-call. `shortcutState`
         // above is a snapshot taken BEFORE that call, so applying it
@@ -383,7 +383,7 @@ export function feedInputTokens(context: InputFeedContext, tokens: readonly Inpu
       // focused panel (Invariant A: focus never silently flips to the composer).
       onPasteDropped: (panelName: string) =>
         context.commandContext?.print(
-          `paste ignored — focus is on ${panelName}; Tab returns to composer`,
+          `paste ignored: focus is on ${panelName}; Tab returns to composer`,
         ),
       isTurnActive: () => context.commandContext?.isGenerating?.() ?? false,
       cancelGeneration: () => context.commandContext?.cancelGeneration?.(),

package/src/input/handler-interactions.ts

@@ -68,7 +68,7 @@ export async function hydrateOnboardingWizardFromRuntimeForHandler(handler: Inpu
  * The prompt is edited on the HANDLER here, not on the shortcut route state
  * that dispatched the key. feedInputTokens snapshots the prompt before
  * dispatching and would write that snapshot back over this edit, erasing the
- * paste — it does not, because it only restores a field the action left
+ * paste, it does not, because it only restores a field the action left
  * untouched (see the promptBefore guard in handler-feed.ts). That guard is the
  * only reason Ctrl+V works, so it is pinned by test.
  */
@@ -150,7 +150,7 @@ export function handleCopyForHandler(handler: InputHandler): void {
 
   /**
    * handleBlockCopy - Ctrl+Y: Copy the content of the block the user is
-   * looking at (the viewport's bottom-most visible block — see
+   * looking at (the viewport's bottom-most visible block, see
    * InputHandler.getBlockAnchorLine).
    */
 export function handleBlockCopyForHandler(handler: InputHandler): void {

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

@@ -99,7 +99,7 @@ export function handleSelectionModalToken(state: SelectionRouteState, token: Inp
         // instant filter: an unclaimed keystroke arms search AND starts
         // the query immediately, instead of silently doing nothing until the
         // user discovers '/' first (the "help search needs '/' arming while
-        // the palette filters instantly" evaluator finding — /help has no
+        // the palette filters instantly" evaluator finding, /help has no
         // customActions, so every letter used to be swallowed here). '/'
         // above still works too; this is additive, not a replacement, and
         // never fires for a claimed hotkey letter (checked first, above).
@@ -110,10 +110,10 @@ export function handleSelectionModalToken(state: SelectionRouteState, token: Inp
   } else if (token.type === 'key') {
     if (token.logicalName === 'escape') {
       // ONE Escape always closes the modal, regardless of search state.
-      // Clearing an in-progress query is Backspace's job, not Esc's — the old
+      // Clearing an in-progress query is Backspace's job, not Esc's, the old
       // two-stage contract (1st Esc clears query, 2nd blurs search, 3rd
       // finally closes) was the "help modal took 3 Escapes after searching"
-      // evaluator finding (searchFocused is not modal-specific — every
+      // evaluator finding (searchFocused is not modal-specific, every
       // SelectionModal-based picker had the same multi-Esc pattern).
       state.handleEscape();
       return true;
@@ -372,7 +372,7 @@ export function handleSettingsModalToken(state: SettingsRouteState, token: Input
   if (token.type === 'key') {
     const focusPane = state.settingsModal.focusPane ?? 'settings';
     if (token.logicalName === 'escape') {
-      // Cancel inline edit first — mirrors the global contract in handler-modal-stack.ts.
+      // Cancel inline edit first, mirrors the global contract in handler-modal-stack.ts.
       // Must check editingMode before searchFocused: the reachable path
       // search→Enter(string/number)→Esc must cancel the edit, NOT just clear search.
       if (state.settingsModal.editingMode) {
@@ -400,7 +400,7 @@ export function handleSettingsModalToken(state: SettingsRouteState, token: Input
       else {
         // Feature-unit toggle headers are boolean settings rows now, so
         // activateSelected toggles them (via the flag-toggle chokepoint) the same
-        // way it edits/toggles any other setting — no flags-category special case.
+        // way it edits/toggles any other setting, no flags-category special case.
         state.settingsModal.activateSelected();
         consumeSettingsPickerRequest(state);
       }
@@ -456,7 +456,7 @@ export function handleSettingsModalToken(state: SettingsRouteState, token: Input
     }
   } else if (token.type === 'text') {
     if (state.settingsModal.editingMode) {
-      // editingMode takes priority over search — Enter on a string/number search
+      // editingMode takes priority over search, Enter on a string/number search
       // result enters inline edit; subsequent chars must go to editChar, not the query.
       state.settingsModal.editChar(token.value);
     } else if (state.settingsModal.searchFocused) {
@@ -577,16 +577,16 @@ type ConfigModalRouteState = {
 /**
  * Route a key to the generic config-modal host (MIGRATE-TO-MODAL
  * surfaces). The host owns the reserved navigation keys (Esc, up/down/j/k tab
- * left/right, and now '/' — item 1's type-to-filter, armed the same
+ * left/right, and now '/', item 1's type-to-filter, armed the same
  * way as scrollable-list-panel.ts's opt-in filter and SettingsModal's own
  * '/'-armed search); every other key is offered to the active surface's
  * declarative action table (fireAction handles the two-press confirm for
  * destructive actions). Any unrecognised key is absorbed so the modal stays
- * modal — this is the same "active modal captures all keys" shape as the
+ * modal, this is the same "active modal captures all keys" shape as the
  * other modal routers.
  *
  * While the filter is armed (`configModal.isFilterActive()`), printable text
- * tokens go to the query instead of nav/action dispatch — this is WHY 'j'/'k'
+ * tokens go to the query instead of nav/action dispatch, this is WHY 'j'/'k'
  * lose their vim-nav meaning mid-filter (you're typing, not navigating) but
  * Escape, Backspace, and the arrow/tab nav keys keep their ordinary meaning
  * (arrows navigate the FILTERED list; Enter still acts on the selected row).
@@ -594,7 +594,7 @@ type ConfigModalRouteState = {
 export function handleConfigModalToken(state: ConfigModalRouteState, token: InputToken): boolean {
   if (!state.configModal.active) return false;
 
-  // Every token that reaches the modal is a user interaction — after the
+  // Every token that reaches the modal is a user interaction, after the
   // first one, structure freezes to interaction boundaries (liveness rule).
   // Before it, renders may sync structure so async onOpen loads appear
   // without a keypress (batch refutation finding 3).
@@ -646,7 +646,7 @@ export function handleConfigModalToken(state: ConfigModalRouteState, token: Inpu
         return true;
     }
     // Any other 'key' token (e.g. 'enter') falls through to the action table
-    // below even while filtering — only printable TEXT tokens are captured
+    // below even while filtering, only printable TEXT tokens are captured
     // by the filter (below), so Enter still fires on the current selection.
   } else if (token.type === 'text') {
     if (!filtering && token.value === '/') {
@@ -655,8 +655,8 @@ export function handleConfigModalToken(state: ConfigModalRouteState, token: Inpu
       return true;
     }
     if (filtering) {
-      // The WHOLE token value lands in the filter in one call — including a
-      // multi-char paste token — never split into per-char nav/action
+      // The WHOLE token value lands in the filter in one call, including a
+      // multi-char paste token, never split into per-char nav/action
       // dispatch (the text-capture invariant this item's brief calls out).
       state.configModal.appendFilterText(token.value);
       state.requestRender();

package/src/input/handler-modal-stack.ts

@@ -150,7 +150,7 @@ export function handleEscape(state: EscapeState): {
         state.autocompleteReset();
         // Esc's palette-dismiss convention: a bare '/' with nothing typed
         // after it IS the palette, so Esc clears it along with the slash
-        // (first Esc, one step, done — no lingering '/' to backspace out).
+        // (first Esc, one step, done, no lingering '/' to backspace out).
         // Once the user has typed a command name (or anything else) past
         // the slash, Esc's job is only to dismiss the ghost-suggestion
         // overlay; the typed text is real composer content and stays put,

package/src/input/handler-modal-token-routes.ts

@@ -78,7 +78,7 @@ export type ModalTokenRouteState = {
   searchManager: SearchManager;
   scroll: (delta: number) => void;
   getScrollTop: () => number;
-  /** Callback to open the model picker with a specific target (helper or tool). Optional — only wired when available. */
+  /** Callback to open the model picker with a specific target (helper or tool). Optional, only wired when available. */
   openModelPickerWithTarget?: (
     target: import('./model-picker.ts').ModelPickerTarget,
     source?: 'settings' | 'onboarding',

package/src/input/handler-onboarding-daemon-adopt.ts

@@ -27,11 +27,11 @@ import { GOODVIBES_TUI_SURFACE_ROOT } from '../config/surface.ts';
 /**
  * Immediately (not deferred to the final wizard Apply):
  *   1. Installs the pasted token into <homeDirectory>/.goodvibes/daemon/operator-tokens.json
- *      via `resolveDaemonCompanionToken` — the same durable-persistence logic
+ *      via `resolveDaemonCompanionToken`, the same durable-persistence logic
  *      the `GOODVIBES_DAEMON_TOKEN` non-interactive override uses.
  *   2. Points `controlPlane.host`/`controlPlane.port` at the pasted host/port.
  *   3. Restarts the external-services controller and reports back, honestly,
- *      whether the daemon at that host/port was actually adopted — reusing
+ *      whether the daemon at that host/port was actually adopted, reusing
  *      the same `onboarding-runtime-status.ts` helpers the normal
  *      start-a-new-daemon path uses to report success/failure after Apply.
  *
@@ -64,7 +64,7 @@ export async function handleConnectExistingDaemonForHandler(handler: InputHandle
     resolveDaemonCompanionToken(daemonHomeDir, GOODVIBES_TUI_SURFACE_ROOT, token);
     // hostMode has to move with the host. The control-plane URL is DERIVED from
     // hostMode/host/port, and 'local' pins the dial target to 127.0.0.1 no
-    // matter what host is stored — so adopting a daemon on another machine
+    // matter what host is stored, so adopting a daemon on another machine
     // while leaving hostMode alone would derive a loopback URL for a remote
     // daemon. Adopting a loopback address stays 'local'.
     handler.uiServices.platform.configManager.setDynamic(
@@ -103,7 +103,7 @@ export async function handleConnectExistingDaemonForHandler(handler: InputHandle
  * `goodvibes-daemon.service` unit (an earlier release shipped detect+disclose only; this
  * closes that inheritance). This is a thin wrapper over
  * `detectLegacyUnit` + `runLegacyDaemonMigration`
- * (`../runtime/legacy-daemon-migration.ts`) — the actual migration mechanics
+ * (`../runtime/legacy-daemon-migration.ts`), the actual migration mechanics
  * (never auto-migrate, new-up-then-old-down with a health check before the
  * legacy unit is touched, adopt-or-warn for an unrecognized process on the
  * port) live there and are exercised against fakes in
@@ -117,7 +117,7 @@ export async function handleConnectExistingDaemonForHandler(handler: InputHandle
  * This lives in `src/input/` rather than importing
  * `../daemon/service-commands.ts` directly because the architecture gate's
  * `input-no-entrypoints` rule forbids `src/input/**` from depending on
- * `src/daemon/**` (input must stay a pure event-handling layer) — the shared
+ * `src/daemon/**` (input must stay a pure event-handling layer), the shared
  * engine lives in the entrypoint-agnostic `src/runtime/` layer instead.
  *
  * `confirm` is caller-supplied rather than read from a fixed field name here
@@ -125,7 +125,7 @@ export async function handleConnectExistingDaemonForHandler(handler: InputHandle
  * step (`onboarding-wizard-network-adopt.ts`'s `pushLegacyDaemonMigrationFields`,
  * dispatched from `handler-onboarding.ts`'s `migrate-legacy-daemon-service`
  * action) reads its own `network.migrate-legacy-daemon-confirm` checklist
- * toggle and passes it through as this parameter — unchecked previews the
+ * toggle and passes it through as this parameter, unchecked previews the
  * dry-run plan (`confirm: false`), checked executes the real migration
  * (`confirm: true`).
  */

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

@@ -21,7 +21,7 @@ type ModelPickerRouteState = {
 
 /**
  * The user's REQUESTED reasoning level for this session, or undefined when the
- * command context is not attached. Read from config through the shared helper —
+ * command context is not attached. Read from config through the shared helper,
  * never from `session.runtime.reasoningEffort`, which holds the EFFECTIVE level
  * for whichever model is serving and would re-seed a resolution with an already
  * snapped-down value.
@@ -36,7 +36,7 @@ function readRequestedEffort(state: ModelPickerRouteState): string | undefined {
  * The level the effort step should open on for a model about to be selected:
  * the requested level SNAPPED to that model. Opening on the raw requested level
  * would miss the list entirely whenever the target model caps lower, and
- * showEffortPicker falls back to index 0 — landing the cursor on the LOWEST
+ * showEffortPicker falls back to index 0, landing the cursor on the LOWEST
  * level rather than on what pressing Enter would actually give you.
  */
 function effortStepPreselect(state: ModelPickerRouteState, model: { id: string; provider?: string; displayName?: string }): string {
@@ -52,7 +52,7 @@ export function handleModelPickerToken(state: ModelPickerRouteState, token: Inpu
       // 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
+      // 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
@@ -92,9 +92,9 @@ export function handleModelPickerToken(state: ModelPickerRouteState, token: Inpu
           // even while a model that caps lower is serving.
           const currentEffort = readRequestedEffort(state) ?? 'medium';
           // Whether an effort step appears at all is now this model's own
-          // resolved spec: a model with no configurable reasoning level — or
+          // resolved spec: a model with no configurable reasoning level, or
           // one whose levels are only a labelled best guess the adapters will
-          // discard — skips straight to commit instead of showing a list it
+          // discard, skips straight to commit instead of showing a list it
           // does not honour.
           if (state.modelPicker.target === 'main' && offersConfigurableEffort(toEffortModel(selected))) {
             state.modelPicker.showEffortPicker(selected, effortStepPreselect(state, selected));
@@ -152,7 +152,7 @@ export function handleModelPickerToken(state: ModelPickerRouteState, token: Inpu
           const rawInput = state.modelPicker.contextCapQuery.trim();
           const parsedCap = rawInput.length > 0 ? parseInt(rawInput, 10) : null;
           if (parsedCap !== null && (parsedCap <= 0 || parsedCap > 10_000_000)) {
-            // Non-empty but out of valid range — surface notice; keep picker open for correction.
+            // Non-empty but out of valid range, surface notice; keep picker open for correction.
             state.modelPicker.contextCapError = 'Context cap must be 1–10,000,000';
           } else {
             state.modelPicker.contextCapError = null;
@@ -231,7 +231,7 @@ export function handleModelPickerToken(state: ModelPickerRouteState, token: Inpu
     if (state.modelPicker.mode === 'contextCap') {
       if (token.value.length === 1) state.modelPicker.appendContextCapChar(token.value);
     } else if ((state.modelPicker.mode === 'model' || state.modelPicker.mode === 'provider') && state.modelPicker.searchFocused) {
-      // When search is focused every printable char — including space — goes to the query.
+      // When search is focused every printable char, including space, goes to the query.
       // The space=context-cap shortcut remains active only in the non-search branch below.
       const ch = token.value;
       if (ch.length === 1 && ch >= ' ') state.modelPicker.appendChar(ch);

package/src/input/handler-prompt-buffer.ts

@@ -23,7 +23,7 @@ export const UNDO_COALESCE_MS = 500;
 export type EditKind = 'text' | 'kill' | 'yank' | 'other';
 
 /**
- * shouldCoalesceUndo — returns true when the new edit should be merged into
+ * shouldCoalesceUndo, returns true when the new edit should be merged into
  * the most recent undo group rather than creating a new snapshot.
  *
  * Coalesces only when:

package/src/input/handler-shortcuts.ts

@@ -77,13 +77,13 @@ export function handleGlobalShortcutToken(
   }
 
   // Bare F2 is also not in the keybinding table (hardcoded, like pageup/
-  // pagedown/escape above) — it must be handled here, GLOBALLY, rather than
+  // 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 —
+  // once; every subsequent press while already focused vanished silently,
   // "F2 pressed 4x never closed the panel" (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
@@ -94,8 +94,8 @@ export function handleGlobalShortcutToken(
   }
 
   // Shift+Tab cycles the session permission mode (normal → accept-edits → plan
-  // → auto → normal), the settled convention. Handled here — hardcoded, like
-  // escape/f2 — because it is not in the keybinding table and bare Tab is
+  // → auto → normal), the settled convention. Handled here, hardcoded, like
+  // escape/f2, because it is not in the keybinding table and bare Tab is
   // already heavily overloaded. Arrives as the legacy xterm backtab literal
   // ('\x1b[Z') OR, under a kitty/CSI-u terminal, as tab-with-shift. Only when
   // the composer owns focus (a focused panel keeps its own Shift+Tab, e.g. the
@@ -121,7 +121,7 @@ export function handleGlobalShortcutToken(
 
     case 'cancel-tool-call':
       // Cancel JUST the currently-running tool call (the live transcript row).
-      // No-op (still consumed) when nothing is running — the composer keeps the
+      // No-op (still consumed) when nothing is running, the composer keeps the
       // keystroke rather than letting it fall through as text.
       state.commandContext?.cancelToolCall?.();
       return true;
@@ -134,7 +134,7 @@ export function handleGlobalShortcutToken(
 
     case 'voice-input':
       // Start recording, or stop the recording already running. Routed globally so
-      // the key works whether focus is on the composer or the panel workspace —
+      // the key works whether focus is on the composer or the panel workspace,
       // dictation is not a composer-only act. Still consumed when voice input is
       // not wired, so a stray Alt+V never lands in the prompt as text.
       state.commandContext?.toggleVoiceInput?.();
@@ -168,11 +168,11 @@ export function handleGlobalShortcutToken(
       const active = pm.getActivePanel();
       // 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.
+      // panel outright, see Panel.interceptPanelClose's doc comment.
       if (active?.interceptPanelClose?.()) {
         // 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
+        //, 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.
@@ -203,7 +203,7 @@ export function handleGlobalShortcutToken(
       // Global entry point for the focus-toggle key (Ctrl+G): from the prompt
       // it grabs focus for the panel workspace. Once the workspace already has
       // focus we let it fall through (return false) so handlePanelFocusToken
-      // can do the top/bottom pane swap — keeping that behavior in one place.
+      // can do the top/bottom pane swap, keeping that behavior in one place.
       if (state.panelFocused) return false;
       const pm = state.panelManager;
       if (pm.isVisible() && pm.getAllOpen().length > 0) {
@@ -234,7 +234,7 @@ 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.
-      // a chord jump is "I'm going panel-driving" (focus rule 1a) — the
+      // 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;
@@ -434,7 +434,7 @@ export function handleGlobalShortcutToken(
 }
 
 /**
- * toggleFleetPanel — the shared F2 / Ctrl+O TOGGLE (item 2): if the
+ * toggleFleetPanel, the shared F2 / Ctrl+O TOGGLE (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
@@ -446,7 +446,7 @@ export function handleGlobalShortcutToken(
  */
 /**
  * Cycle the session permission mode (Shift+Tab). The change goes through the
- * SDK config surface — configManager.set('permissions.mode', ...) — which is
+ * SDK config surface, configManager.set('permissions.mode', ...), which is
  * what the SDK PermissionManager reads and what the orchestrator consults for
  * its standing plan-mode instruction, so every attached surface converges on
  * the same mode. The footer pill re-renders off the config-key subscription

package/src/input/handler-types.ts

@@ -1,5 +1,5 @@
 /**
- * handler-types.ts — Leaf interface for InputHandler.
+ * handler-types.ts, Leaf interface for InputHandler.
  *
  * Extracted from handler.ts to break circular import chains between handler.ts
  * and handler-interactions.ts / handler-onboarding.ts / handler-onboarding-cloudflare.ts.
@@ -137,8 +137,8 @@ export interface InputHandlerLike {
   handleBookmark(): void;
   handleBlockToggle(): void;
   handleDiffApply(): boolean;
-  /** The absolute history line of the bottom-most visible block — the block
-   *  the user is actually looking at — used to anchor every one of the
+  /** The absolute history line of the bottom-most visible block, the block
+   *  the user is actually looking at, used to anchor every one of the
    *  actions above instead of the (possibly off-screen-above) raw scrollTop. */
   getBlockAnchorLine(): number;
 

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

@@ -203,7 +203,7 @@ type SearchRouteState = {
   scroll: (delta: number) => void;
   getScrollTop: () => number;
   getViewportHeight: () => number;
-  /** Optional — when present, search() also expands collapsed blocks whose
+  /** Optional, when present, search() also expands collapsed blocks whose
    *  raw content matches (see SearchManager.search's doc). */
   conversationManager?: ConversationManager | null;
 };