@pellux/goodvibes-tui 2.0.14 → 2.0.16

package/src/input/handler.ts

@@ -181,10 +181,10 @@ export class InputHandler implements InputHandlerLike {
   public nextPasteId = 1;
   public lastCtrlCTime = 0;
   /** Pending "hide the exit notice" timer from the last empty-prompt Ctrl+C
-   * press, if any — cleared before every subsequent press decides its own
+   * press, if any, cleared before every subsequent press decides its own
    * outcome (see handleCtrlC in handler-content-actions.ts). */
   public lastCtrlCTimeoutId: ReturnType<typeof setTimeout> | null = null;
-  /** Long-lived feed context — reused across every feed() call to avoid per-keystroke allocation. */
+  /** Long-lived feed context, reused across every feed() call to avoid per-keystroke allocation. */
   public feedContext!: import('./handler-feed.ts').InputFeedContext;
   public commandRegistry: CommandRegistry | null = null;
   public commandContext: CommandContext | undefined = undefined;
@@ -197,7 +197,7 @@ export class InputHandler implements InputHandlerLike {
   public blockActionsMenu = new BlockActionsMenu();
   public settingsModal = new SettingsModal();
   public configModal = new ConfigModal();
-  /** item 5 — paste-flood guard state, mutated in place across tokens (never reallocated). */
+  /** item 5, paste-flood guard state, mutated in place across tokens (never reallocated). */
   private panelBurstGuard: PanelBurstGuardState = { timestamps: [], suspended: false, hintShown: false };
   public mcpWorkspace = new McpWorkspace();
   public onboardingWizard = new OnboardingWizardController();
@@ -225,7 +225,7 @@ export class InputHandler implements InputHandlerLike {
   public conversationManager: ConversationManager | null = null;
   public selectionCallback: SelectionModalCallback | null = null;
   public syncFeedSelectionCallback: ((callback: SelectionModalCallback | null) => void) | null = null;
-  /** FIFO queue for overlapping openSelection() calls — see selection-modal-queue.ts. */
+  /** FIFO queue for overlapping openSelection() calls, see selection-modal-queue.ts. */
   private selectionQueue!: SelectionModalQueue;
   /** Time of last [COPIED] block feedback, for brief display. */
   public lastBlockCopyTime = 0;
@@ -283,7 +283,7 @@ export class InputHandler implements InputHandlerLike {
       uiServices.providers.providerRegistry,
     );
     // retirement: ProcessModal/LiveTailModal/AgentDetailModal were removed
-    // — F2 now opens the Fleet panel, which subsumes the live process tree.
+    //, F2 now opens the Fleet panel, which subsumes the live process tree.
     this.bookmarkModal = new BookmarkModal(uiServices.shell.bookmarkManager);
     this.sessionPickerModal = new SessionPickerModal(uiServices.sessions.sessionManager, uiServices.sessions.sessionBroker, getSharedHostedSessionRoster(), () => this.requestRender()); // third source: the daemon-hosted roster
     this.profilePickerModal = new ProfilePickerModal(uiServices.shell.profileManager);
@@ -292,7 +292,7 @@ export class InputHandler implements InputHandlerLike {
   }
 
   /**
-   * initFeedContext — Build the long-lived InputFeedContext once via factory.
+   * initFeedContext, Build the long-lived InputFeedContext once via factory.
    * See feed-context-factory.ts for full field documentation.
    */
   public initFeedContext(): void {
@@ -411,7 +411,7 @@ export class InputHandler implements InputHandlerLike {
    * openSelection - Open the generic selection modal with a callback.
    * The callback receives SelectionResult on selection, or null on cancel/escape.
    * A second call while one is already showing queues FIFO instead of
-   * overwriting it — see selection-modal-queue.ts.
+   * overwriting it, see selection-modal-queue.ts.
    */
   public openSelection(
     title: string,
@@ -443,7 +443,7 @@ export class InputHandler implements InputHandlerLike {
   public handleBlockSave(): void { handleBlockSaveForHandler(this); }
   public executeBlockAction(actionId: string): void { executeBlockActionForHandler(this, actionId); }
   /** The block the user is actually looking at: the viewport's bottom-most
-   *  visible line (see getViewportBottomLine's doc) — not the raw scrollTop,
+   *  visible line (see getViewportBottomLine's doc), not the raw scrollTop,
    *  which is off-screen-above once the transcript exceeds one page. Every
    *  anchor-based block action (Ctrl+Y/B/S/A, Tab, the block-actions menu)
    *  resolves its target through this, so they all agree on what "nearest
@@ -585,7 +585,7 @@ export class InputHandler implements InputHandlerLike {
    */
   public clipboardSource: ClipboardPasteSource = { pasteImageFromClipboard, pasteFromClipboard };
 
-  /** Content width for wrapping — set by main.ts via setContentWidth(). */
+  /** Content width for wrapping, set by main.ts via setContentWidth(). */
   public contentWidth = 76;
   public panelMouseLayout: PanelMouseLayout | null = null;
 

package/src/input/input-history.ts

@@ -5,7 +5,7 @@ import { logger } from '@pellux/goodvibes-sdk/platform/utils';
 import { summarizeError } from '@pellux/goodvibes-sdk/platform/utils';
 
 /**
- * InputHistory — Persisted command history with arrow-key navigation.
+ * InputHistory, Persisted command history with arrow-key navigation.
  *
  * Navigation model:
  *   position = -1   : not browsing (at the live draft)

package/src/input/keybindings.ts

@@ -1,5 +1,5 @@
 /**
- * KeybindingsManager — loads and merges keyboard shortcut configuration.
+ * KeybindingsManager, loads and merges keyboard shortcut configuration.
  *
  * Default bindings are hardcoded here. Users can override any binding by
  * creating ~/.goodvibes/tui/keybindings.json.
@@ -119,7 +119,7 @@ export const ACTION_DESCRIPTIONS: Record<KeyAction, string> = {
   'toggle-keep-awake':     'Toggle keep-awake (the "sleep disabled" chip)',
   'toggle-memory-provenance': 'List/hide the memories a turn used (provenance chip drill-in)',
   // Named for what it actually does. A terminal delivers key EVENTS, not held-key
-  // state — there is no release event to watch — so this is two discrete presses,
+  // state, there is no release event to watch, so this is two discrete presses,
   // and the description says so instead of implying a hold that cannot exist.
   'voice-input':           'Voice input: press to start recording, press again to stop and transcribe into the composer',
 };
@@ -138,7 +138,7 @@ export const DEFAULT_KEYBINDINGS: Record<KeyAction, KeyCombo[]> = {
   'panel-close-all':         [{ key: 'x', ctrl: true, shift: true }],
   // Ctrl+] stays the primary next-tab chord; Ctrl+PageDown is added as a second
   // binding. Ctrl+[ was REMOVED as prev-tab: it is byte 0x1B (ESC), so the
-  // legacy tokenizer path emits it as 'escape' — the binding never matched on
+  // legacy tokenizer path emits it as 'escape', the binding never matched on
   // most terminals AND the key fired Escape (a split-brain chord). Ctrl+PageUp
   // replaces it: Ctrl+PageUp/PageDown tokenize consistently (\x1b[5;5~ /
   // \x1b[6;5~ -> pageup/pagedown ctrl:true) in both the legacy and CSI-u paths.
@@ -158,7 +158,7 @@ export const DEFAULT_KEYBINDINGS: Record<KeyAction, KeyCombo[]> = {
   'panel-tab-8':           [{ key: '8', alt: true }],
   'panel-tab-9':           [{ key: '9', alt: true }],
   // Ctrl+O: TOGGLE the Fleet panel (open+focus / bring-to-front+focus / close
-  // — see toggleFleetPanel in handler-shortcuts.ts). The former Ops
+  //, see toggleFleetPanel in handler-shortcuts.ts). The former Ops
   // Control panel was retired to an 'ops-control' -> 'fleet' alias; the
   // binding is KEPT (repointed, not removed) so the Ctrl+O muscle memory still
   // lands somewhere useful. Routed globally in handleGlobalShortcutToken.
@@ -185,7 +185,7 @@ export const DEFAULT_KEYBINDINGS: Record<KeyAction, KeyCombo[]> = {
   'undo':                  [{ key: 'z', ctrl: true }],
   'redo':                  [{ key: 'z', ctrl: true, shift: true }],
   'paste':                 [{ key: 'v', ctrl: true }],
-  // Word navigation (Alt+B / Alt+F — emacs readline standard)
+  // Word navigation (Alt+B / Alt+F, emacs readline standard)
   'word-back':             [{ key: 'b', alt: true }],
   'word-forward':          [{ key: 'f', alt: true }],
   // Kill-ring operations.
@@ -214,7 +214,7 @@ export const DEFAULT_KEYBINDINGS: Record<KeyAction, KeyCombo[]> = {
   'toggle-memory-provenance': [{ key: 'm', alt: true }],
   // Alt+V: start/stop voice input. Alt+V is unused by any other default (Ctrl+V
   // is paste and stays paste). Two presses rather than a hold: see the action's
-  // description — a terminal has no key-release event to hold against.
+  // description, a terminal has no key-release event to hold against.
   'voice-input':           [{ key: 'v', alt: true }],
 };
 
@@ -243,7 +243,7 @@ function resolveKeybindingsPath(options?: KeybindingsManagerOptions): string {
 }
 
 /**
- * KeybindingsManager — owns the resolved keybinding table.
+ * KeybindingsManager, owns the resolved keybinding table.
  *
  * Call loadFromDisk() once at startup (in main.ts) to merge user config.
  * Then use matches() anywhere a key token is being evaluated.
@@ -304,7 +304,7 @@ export class KeybindingsManager {
   }
 
   /**
-   * buildLookupMap — Rebuild the inverted lookup map from the current bindings table.
+   * buildLookupMap, Rebuild the inverted lookup map from the current bindings table.
    * Called after constructor init and after loadFromDisk().
    * Map key format: "logicalName:ctrl:shift:alt" (booleans as 0/1).
    * Last writer wins for duplicate combos (deterministic: iterate actions in order).
@@ -320,7 +320,7 @@ export class KeybindingsManager {
   }
 
   /**
-   * lookup — O(1) keybinding lookup by token.
+   * lookup, O(1) keybinding lookup by token.
    * Returns the matching KeyAction, or null if no binding matches.
    */
   lookup(token: { logicalName?: string; ctrl?: boolean; shift?: boolean; alt?: boolean; meta?: boolean }): KeyAction | null {
@@ -342,7 +342,7 @@ export class KeybindingsManager {
   }
 
   /**
-   * matches — Check whether a keyboard token matches the given action.
+   * matches, Check whether a keyboard token matches the given action.
    *
    * @param action  The action to test.
    * @param token   The parsed keyboard token from InputTokenizer.
@@ -370,7 +370,7 @@ export class KeybindingsManager {
   }
 
   /**
-   * getAll — Return the full resolved binding table for display purposes.
+   * getAll, Return the full resolved binding table for display purposes.
    */
   getAll(): Array<{ action: KeyAction; combos: KeyCombo[]; description: string }> {
     return (Object.keys(this.bindings) as KeyAction[]).map((action) => ({
@@ -381,7 +381,7 @@ export class KeybindingsManager {
   }
 
   /**
-   * getComboLabel — Return a human-readable label for the first combo of an action.
+   * getComboLabel, Return a human-readable label for the first combo of an action.
    * Example: { key: 'f', ctrl: true } → "Ctrl+F"
    */
   getComboLabel(action: KeyAction): string {
@@ -391,7 +391,7 @@ export class KeybindingsManager {
   }
 
   /**
-   * formatCombo — Format a KeyCombo as a human-readable string.
+   * formatCombo, Format a KeyCombo as a human-readable string.
    */
   formatCombo(combo: KeyCombo): string {
     const parts: string[] = [];

package/src/input/kill-ring.ts

@@ -1,5 +1,5 @@
 // ---------------------------------------------------------------------------
-// KillRing — emacs/readline-compatible kill ring implementation.
+// KillRing, emacs/readline-compatible kill ring implementation.
 //
 // The ring holds up to MAX_ENTRIES text strings. Kill commands push to the
 // head. Yank pastes from the current yank pointer (default: head). Yank-pop
@@ -13,7 +13,7 @@
 export const KILL_RING_MAX = 32;
 
 /**
- * KillRing — bounded circular ring of killed text segments.
+ * KillRing, bounded circular ring of killed text segments.
  *
  * All public mutation methods are pure-functional helpers at module level;
  * this class owns the mutable state so the handler can hold a single ref.
@@ -41,7 +41,7 @@ export class KillRing {
   }
 
   /**
-   * Yank — return the entry at the current yank pointer.
+   * Yank, return the entry at the current yank pointer.
    * Returns '' if the ring is empty.
    * Sets lastActionWasYank so a subsequent yank-pop is valid.
    */
@@ -53,7 +53,7 @@ export class KillRing {
   }
 
   /**
-   * YankPop — advance the yank pointer by one step (wrapping) and return the
+   * YankPop, advance the yank pointer by one step (wrapping) and return the
    * new entry. Only valid after a yank; if the ring has <=1 entry, returns the
    * same string. Returns '' if the ring is empty.
    */
@@ -85,7 +85,7 @@ export class KillRing {
 // ---------------------------------------------------------------------------
 
 /**
- * isWordChar — true when the character is a "word" character.
+ * isWordChar, true when the character is a "word" character.
  * Word = letter (Unicode), digit, or underscore. This is the emacs/readline
  * word boundary definition used for Alt+B, Alt+F, Ctrl+W, Alt+D.
  */
@@ -94,7 +94,7 @@ function isWordChar(ch: string): boolean {
 }
 
 /**
- * wordBoundaryBack — find the start of the word that the cursor is in, or the
+ * wordBoundaryBack, find the start of the word that the cursor is in, or the
  * start of the previous word if the cursor is at a non-word character.
  *
  * Emacs Alt+B semantics:
@@ -106,15 +106,13 @@ function isWordChar(ch: string): boolean {
  */
 export function wordBoundaryBack(text: string, pos: number): number {
   let p = pos;
-  // Skip non-word chars first (move past punctuation/spaces)
   while (p > 0 && !isWordChar(text[p - 1]!)) p--;
-  // Then skip word chars (the word body)
   while (p > 0 && isWordChar(text[p - 1]!)) p--;
   return p;
 }
 
 /**
- * wordBoundaryForward — find the position just past the end of the next word.
+ * wordBoundaryForward, find the position just past the end of the next word.
  *
  * Emacs Alt+F semantics:
  *   - Skip non-word chars forward
@@ -126,9 +124,7 @@ export function wordBoundaryBack(text: string, pos: number): number {
 export function wordBoundaryForward(text: string, pos: number): number {
   let p = pos;
   const len = text.length;
-  // Skip non-word chars first
   while (p < len && !isWordChar(text[p]!)) p++;
-  // Then skip word chars
   while (p < len && isWordChar(text[p]!)) p++;
   return p;
 }

package/src/input/model-picker-filter.ts

@@ -1,5 +1,5 @@
 /**
- * model-picker-filter — pure filtering, sorting, and cache-key helpers for ModelPickerModal.
+ * model-picker-filter, pure filtering, sorting, and cache-key helpers for ModelPickerModal.
  *
  * All functions are stateless: they receive inputs and return results without
  * side-effects. The class in model-picker.ts uses these as delegates.
@@ -139,7 +139,7 @@ export function buildFilteredModels(
     result = result.filter(m => m.capabilities?.multimodal === true);
   }
 
-  // Query filter — fuzzy: every space-separated word must appear somewhere
+  // Query filter, fuzzy: every space-separated word must appear somewhere
   if (query.trim().length > 0) {
     const words = query.toLowerCase().split(/\s+/).filter(Boolean);
     result = result.filter(m => {
@@ -168,7 +168,7 @@ export function buildFilteredModels(
           scoreB = bB ? compositeScore(bB.benchmarks) : null;
         }
       } else {
-        // swe/gpqa sort — individual scores not available for synthetic models
+        // swe/gpqa sort, individual scores not available for synthetic models
         const bA = a.provider === 'synthetic' ? null : (benchmarkStore.getBenchmarks(a.id) ?? benchmarkStore.getBenchmarks(a.displayName));
         const bB = b.provider === 'synthetic' ? null : (benchmarkStore.getBenchmarks(b.id) ?? benchmarkStore.getBenchmarks(b.displayName));
         if (benchmarkSort === 'swe') {

package/src/input/model-picker-items.ts

@@ -1,5 +1,5 @@
 /**
- * model-picker-items — pure item-building helpers for ModelPickerModal.
+ * model-picker-items, pure item-building helpers for ModelPickerModal.
  *
  * Converts filtered ModelDefinition/provider arrays into PickerItem lists
  * with group headers, quality-tier badges, pin markers, and configured-via flags.
@@ -55,7 +55,7 @@ export function getModelGroupKey(
 }
 
 // ---------------------------------------------------------------------------
-// toModelItem — single model -> PickerItem
+// toModelItem, single model -> PickerItem
 // ---------------------------------------------------------------------------
 
 export function toModelItem(
@@ -94,7 +94,7 @@ export function toModelItem(
 }
 
 // ---------------------------------------------------------------------------
-// buildModelItems — filtered models -> PickerItem[] with group headers
+// buildModelItems, filtered models -> PickerItem[] with group headers
 // ---------------------------------------------------------------------------
 
 export function buildModelItems(
@@ -151,7 +151,7 @@ export function buildModelItems(
 }
 
 // ---------------------------------------------------------------------------
-// buildProviderItems — filtered providers -> PickerItem[] with group headers
+// buildProviderItems, filtered providers -> PickerItem[] with group headers
 // ---------------------------------------------------------------------------
 
 export function buildProviderItems(
@@ -200,7 +200,7 @@ export function buildProviderItems(
 }
 
 // ---------------------------------------------------------------------------
-// buildEffortItems — effort levels -> PickerItem[]
+// buildEffortItems, effort levels -> PickerItem[]
 // ---------------------------------------------------------------------------
 
 /**

package/src/input/model-picker-types.ts

@@ -9,7 +9,7 @@ export type PickerMode = 'model' | 'provider' | 'effort' | 'contextCap' | 'embed
  * 'tool'       -> tools.llmProvider + tools.llmModel (+ tools.llmEnabled: true)
  * 'tts'        -> tts.llmProvider + tts.llmModel
  * 'embeddings' -> provider.embeddingProvider, via MemoryEmbeddingProviderRegistry.setDefaultProvider()
- *                 (not an LLM route — no model concept, see ModelPickerTargetInfo.configuredNote)
+ *                 (not an LLM route, no model concept, see ModelPickerTargetInfo.configuredNote)
  */
 export type ModelPickerTarget = 'main' | 'helper' | 'tool' | 'tts' | 'embeddings';
 
@@ -25,7 +25,7 @@ export interface ModelPickerTargetInfo {
   readonly inherited: boolean;
   /**
    * Honest override for the "Current:" summary line. Used by the 'embeddings'
-   * target (which has no model concept — only a provider id + dimensions +
+   * target (which has no model concept, only a provider id + dimensions +
    * configured state) so the renderer never prints a phantom "model:" value.
    * When set, this replaces the computed provider:model route text.
    */

package/src/input/model-picker.ts

@@ -100,7 +100,7 @@ export class ModelPickerModal {
    * spec carries a caveat. Null outside effort mode.
    */
   public effortPresentation: EffortPresentation | null = null;
-  /** Entries for PickerMode 'embeddingProvider' — the 'embeddings' target's own tiny item list. */
+  /** Entries for PickerMode 'embeddingProvider', the 'embeddings' target's own tiny item list. */
   public embeddingProviders: EmbeddingProviderPickerEntry[] = [];
   /** Mode to restore when navigating the targets rail away from 'embeddings'. */
   private nonEmbeddingMode: PickerMode = 'model';
@@ -124,11 +124,11 @@ export class ModelPickerModal {
   public availableOnly = true;
   /** Set of provider names that have a configured key (used for availableOnly filter). */
   public configuredProviders: Set<string> = new Set();
-  /** How each provider is configured — drives badge display in provider mode. */
+  /** How each provider is configured, drives badge display in provider mode. */
   public configuredViaMap: Map<string, 'env' | 'secrets' | 'subscription' | 'anonymous'> = new Map();
-  /** IDs of pinned/favorite models — shown at top of list. */
+  /** IDs of pinned/favorite models, shown at top of list. */
   public pinnedIds: Set<string> = new Set();
-  /** IDs of recently used models — shown after pinned, before the rest. */
+  /** IDs of recently used models, shown after pinned, before the rest. */
   public recentIds: string[] = [];
   /** Benchmark score sort order. */
   public benchmarkSort: BenchmarkSort = 'none';
@@ -274,13 +274,13 @@ export class ModelPickerModal {
   }
 
   /**
-   * Open showing all models — entry point for /model.
+   * Open showing all models, entry point for /model.
    *
    * Search starts FOCUSED. A live tmux repro showed the actual
-   * friction wasn't the filter itself (it already works — fuzzy multi-word
+   * friction wasn't the filter itself (it already works, fuzzy multi-word
    * substring over id/displayName/provider) but that with search unfocused by
    * default, typing a search term went character-by-character into single-key
-   * shortcuts instead (g=group, c=capability, a=available-only, b=benchmark) —
+   * shortcuts instead (g=group, c=capability, a=available-only, b=benchmark),
    * e.g. typing "claude" silently cycled the capability filter (on 'c') and
    * toggled available-only off (on 'a') while the visible list never
    * filtered, with no error and no indication anything happened. Reaching
@@ -307,7 +307,7 @@ export class ModelPickerModal {
     this.scrollOffset = 0;
   }
 
-  /** Open showing providers first — entry point for /provider. Search starts focused — see openAllModels(). */
+  /** Open showing providers first, entry point for /provider. Search starts focused, see openAllModels(). */
   openProviders(providers: string[], currentProvider: string): void {
     this.previousMode = null;
     this.providers = providers;
@@ -327,9 +327,9 @@ export class ModelPickerModal {
   }
 
   /**
-   * Open the embedding-provider list — entry point for the 'embeddings' target.
+   * Open the embedding-provider list, entry point for the 'embeddings' target.
    * Deliberately NOT routed through openAllModels (ModelDefinition-shaped) or
-   * openProviders (LLM-provider label resolution) — embedding providers are a
+   * openProviders (LLM-provider label resolution), embedding providers are a
    * flat {id, label, dimensions, configured} list with no model concept, and
    * reusing either existing opener would mislabel or drop unrecognized ids.
    */
@@ -356,7 +356,7 @@ export class ModelPickerModal {
     this.query = '';
     this.categoryFilter = 'all';
     this.capabilityFilter = 'none';
-    // User explicitly chose this provider — disable availability filter so synthetic
+    // User explicitly chose this provider, disable availability filter so synthetic
     // models (which have no real API key) are not filtered out.
     this.availableOnly = false;
     this.selectedIndex = 0;
@@ -521,8 +521,8 @@ export class ModelPickerModal {
    * Used for inserting group headers in getItems().
    *
    * For synthetic provider models with groupBy 'provider', returns sub-group keys:
-   * - 'Top Models'   — benchmark composite score ≥ 0.65 (A-tier or S-tier)
-   * - 'All Synthetic' — remaining synthetic models
+   * - 'Top Models'  , benchmark composite score ≥ 0.65 (A-tier or S-tier)
+   * - 'All Synthetic', remaining synthetic models
    */
   getModelGroupKey(model: ModelDefinition): string {
     return getModelGroupKey(model, this.groupBy, this.providerRegistry, this.benchmarkStore);
@@ -555,7 +555,7 @@ export class ModelPickerModal {
       return result;
     }
     if (this.mode === 'embeddingProvider') {
-      // Tiny list (typically 2-4 entries) — no caching infrastructure needed.
+      // Tiny list (typically 2-4 entries), no caching infrastructure needed.
       // Unconfigured providers are shown honestly (isConfigured: false), never hidden.
       return this.embeddingProviders.map((provider) => ({
         id: provider.id,
@@ -577,7 +577,7 @@ export class ModelPickerModal {
   }
 
   /**
-   * Move selection up (stops at 0 — no wrap to avoid going off-screen).
+   * Move selection up (stops at 0, no wrap to avoid going off-screen).
    * Updates scrollOffset to keep selection visible.
    */
   moveUp(maxVisible = 20): void {
@@ -587,7 +587,7 @@ export class ModelPickerModal {
       this.selectedIndex--;
       this._scrollToSelection(maxVisible);
     }
-    // At index 0 — stop. Do NOT wrap to count-1 (that puts selection off-screen).
+    // At index 0, stop. Do NOT wrap to count-1 (that puts selection off-screen).
   }
 
   /**
@@ -632,10 +632,10 @@ export class ModelPickerModal {
    */
   _scrollToSelection(maxVisible: number): void {
     if (this.selectedIndex < this.scrollOffset) {
-      // Selection moved above viewport — scroll up
+      // Selection moved above viewport, scroll up
       this.scrollOffset = this.selectedIndex;
     } else if (this.selectedIndex >= this.scrollOffset + maxVisible) {
-      // Selection moved below viewport — scroll down
+      // Selection moved below viewport, scroll down
       this.scrollOffset = this.selectedIndex - maxVisible + 1;
     }
   }

package/src/input/onboarding/onboarding-feature-units.ts

@@ -3,7 +3,7 @@
  *
  * Every feature flag that is NOT already reached through a surface/server
  * capability selection (or the HITL experience step) is presented here as ONE
- * unit — its enable toggle plus a meaningful subset of its config sub-options —
+ * unit, its enable toggle plus a meaningful subset of its config sub-options,
  * grouped into thematic first-run steps (safety & sandboxing, memory & context,
  * telemetry & observability, automation & initiative, provider & runtime, and a
  * compact advanced-toggles step). Full sub-option depth lives in /settings; each
@@ -154,7 +154,7 @@ export const FEATURE_ONBOARDING_SECTIONS: readonly FeatureSection[] = [
             key: 'source',
             configKey: 'policy.bundleSource' as ConfigKey,
             label: 'Startup bundle source',
-            hint: 'none (bundles supplied via commands) or file (load policy.bundlePath — set it in /settings).',
+            hint: 'none (bundles supplied via commands) or file (load policy.bundlePath: set it in /settings).',
             valueType: 'enum',
             options: ['none', 'file'],
             defaultValue: 'none',
@@ -231,7 +231,7 @@ export const FEATURE_ONBOARDING_SECTIONS: readonly FeatureSection[] = [
       {
         flagId: 'otel-remote-export',
         label: 'OTLP remote export',
-        hint: 'Export spans over OTLP/HTTP JSON to a collector. Enabling this also turns on the OpenTelemetry foundation. Span export reads the OTEL_EXPORTER_OTLP_* environment variables; decision export reads telemetry.decisionOtlpEndpoint — this unit sets the latter.',
+        hint: 'Export spans over OTLP/HTTP JSON to a collector. Enabling this also turns on the OpenTelemetry foundation. Span export reads the OTEL_EXPORTER_OTLP_* environment variables; decision export reads telemetry.decisionOtlpEndpoint: this unit sets the latter.',
         requiresFlags: ['otel-foundation'],
         impliedConfig: [{ key: 'telemetry.decisionOtlpEnabled' as ConfigKey, value: true }],
         subOptions: [
@@ -284,7 +284,7 @@ export const FEATURE_ONBOARDING_SECTIONS: readonly FeatureSection[] = [
       {
         flagId: 'watcher-triggers',
         label: 'Trigger family',
-        hint: 'Three unattended watcher kinds over one supervision spine: stream watchers that regex-filter and batch a long-lived command\'s output, model-free condition checks running a probe/extract/rule pipeline with no LLM in the loop, and one-shot on-exit triggers that fire exactly one payload when a launched command terminates. A firing trigger runs an agent turn or a pre-registered digest-pinned action grant — never a command composed at fire time. Off by default because a trigger supervises real processes with nobody watching; with it on and no triggers defined the supervisor idles and consumes nothing.',
+        hint: 'Three unattended watcher kinds over one supervision spine: stream watchers that regex-filter and batch a long-lived command\'s output, model-free condition checks running a probe/extract/rule pipeline with no LLM in the loop, and one-shot on-exit triggers that fire exactly one payload when a launched command terminates. A firing trigger runs an agent turn or a pre-registered digest-pinned action grant; never a command composed at fire time. Off by default because a trigger supervises real processes with nobody watching; with it on and no triggers defined the supervisor idles and consumes nothing.',
         note: 'Backoff ladder, strike breaker, retention bounds, batching and process caps are tuned through watchers.triggers.* in /settings.',
         impliedConfig: [{ key: 'watchers.triggers.enabled' as ConfigKey, value: true }],
       },
@@ -331,14 +331,14 @@ export const FEATURE_ONBOARDING_SECTIONS: readonly FeatureSection[] = [
       {
         flagId: 'paired-device-capabilities',
         label: 'Paired phone capabilities',
-        hint: 'Turning this on grants access to nothing by itself — it lets the agent ASK to use a paired phone as a tool: either camera, its screen, its location, its clipboard, and a small set of device effects (notification, link, buzz). It rides the existing peer transport as a native contract, never an MCP server. Every capture and every effect asks the person holding the phone first, and answering one request grants nothing beyond it; choosing "always allow" is a separate, explicit choice you make later on that prompt, and it writes one durable, revocable grant for that one capability on that one phone, with an age limit and a count cap so nothing is granted forever.',
+        hint: 'Turning this on grants access to nothing by itself; it lets the agent ASK to use a paired phone as a tool: either camera, its screen, its location, its clipboard, and a small set of device effects (notification, link, buzz). It rides the existing peer transport as a native contract, never an MCP server. Every capture and every effect asks the person holding the phone first, and answering one request grants nothing beyond it; choosing "always allow" is a separate, explicit choice you make later on that prompt, and it writes one durable, revocable grant for that one capability on that one phone, with an age limit and a count cap so nothing is granted forever.',
         note: 'Captures are kept 24 hours by default and then deleted, and every housekeeping sweep records exactly what it removed and why. Retention, grant limits, clipboard and location posture are tuned through device.* in /settings.',
         subOptions: [
           {
             key: 'mode',
             configKey: 'device.capabilities.mode' as ConfigKey,
             label: 'Consent posture',
-            hint: 'honor-grants (stock) asks the first time and every time after unless you chose "always allow" for that capability on that phone; ask-every-time prompts on every single request and never consults a durable grant — use it when someone else is holding the phone; off stops any capability request reaching any paired device.',
+            hint: 'honor-grants (stock) asks the first time and every time after unless you chose "always allow" for that capability on that phone; ask-every-time prompts on every single request and never consults a durable grant; use it when someone else is holding the phone; off stops any capability request reaching any paired device.',
             valueType: 'enum',
             options: ['off', 'ask-every-time', 'honor-grants'],
             defaultValue: 'honor-grants',
@@ -349,7 +349,7 @@ export const FEATURE_ONBOARDING_SECTIONS: readonly FeatureSection[] = [
         flagId: 'wake-word-detection',
         label: 'Wake-word detection',
         hint: 'Listen continuously on a capture device for the pinned "hey goodvibes" phrase and hand the utterance that follows to speech-to-text. Off by default because holding a microphone open must be an explicit act.',
-        note: 'On this terminal, turning this on starts a recorder (pw-record, parecord, arecord, ffmpeg or sox — whichever is installed), scores every frame with the pinned classifier, plays a short chime the moment a wake confirms, and shows a persistent "listening" row in the footer for as long as it runs. What follows a wake goes to speech-to-text and lands in the composer, or is sent straight away if you turn voice.wake.autoSubmit on. The model it scores with is already here — installing GoodVibes downloads and checksum-verifies it, and a running daemon retries at boot if the install could not reach the network — so turning this on normally needs no setup step. What it will NOT do on its own is download anything at the moment you turn it on: a host whose artifacts are missing or fail verification says which and names /voice wake setup, instead of pretending to listen. That same provisioning fetches the speech gate, so voice.wake.vadThreshold above 0 screens frames once it has run and refuses to start before it has, rather than scoring frames it claims to be screening. voice.wake.noiseSuppression: speex runs here too — the filter travels with the platform, so there is nothing to install for it.',
+        note: 'On this terminal, turning this on starts a recorder (pw-record, parecord, arecord, ffmpeg or sox; whichever is installed), scores every frame with the pinned classifier, plays a short chime the moment a wake confirms, and shows a persistent "listening" row in the footer for as long as it runs. What follows a wake goes to speech-to-text and lands in the composer, or is sent straight away if you turn voice.wake.autoSubmit on. The model it scores with is already here: installing GoodVibes downloads and checksum-verifies it, and a running daemon retries at boot if the install could not reach the network, so turning this on normally needs no setup step. What it will NOT do on its own is download anything at the moment you turn it on: a host whose artifacts are missing or fail verification says which and names /voice wake setup, instead of pretending to listen. That same provisioning fetches the speech gate, so voice.wake.vadThreshold above 0 screens frames once it has run and refuses to start before it has, rather than scoring frames it claims to be screening. voice.wake.noiseSuppression: speex runs here too, the filter travels with the platform, so there is nothing to install for it.',
       },
     ],
   },
@@ -447,10 +447,6 @@ function radioOptionsForSubOption(sub: FeatureSubOption): readonly OnboardingWiz
   return (sub.options ?? []).map((id) => ({ id, label: id, hint: `Use ${id}.` }));
 }
 
-// ---------------------------------------------------------------------------
-// Step builder
-// ---------------------------------------------------------------------------
-
 export function buildFeatureUnitStep(
   controller: OnboardingWizardControllerLike,
   section: FeatureSection,
@@ -512,7 +508,7 @@ export function buildFeatureUnitStep(
     summaryTitle: `${section.title} selections`,
     summaryLines: [
       `${enabledCount}/${section.units.length} feature(s) enabled in this step`,
-      'Skippable — the honest defaults above apply if you make no change.',
+      'Skippable: the honest defaults above apply if you make no change.',
       'Full sub-option depth for every feature lives in /settings.',
     ],
     fields,
@@ -571,7 +567,7 @@ export function applyFeatureUnitOperations(
         }
         // A sub-option that writes the unit's OWN enablement key (e.g. the
         // provider-optimizer routing mode) is the authoritative enablement
-        // value — drop the pending override so the flush cannot clobber it.
+        // value, drop the pending override so the flush cannot clobber it.
         if (wrote && enablementKey !== undefined && sub.configKey === enablementKey) {
           overrides.delete(unit.flagId);
         }

package/src/input/onboarding/onboarding-provider-key-fields.ts

@@ -15,15 +15,15 @@ import type {
 } from './onboarding-wizard-types.ts';
 
 /**
- * Providers with a wired in-wizard subscription sign-in flow. This is DATA — the
- * set of providers whose OAuth login the wizard can start/finish itself — not an
+ * Providers with a wired in-wizard subscription sign-in flow. This is DATA, the
+ * set of providers whose OAuth login the wizard can start/finish itself, not an
  * assumption that any one vendor is the only subscription option.
  */
 const SUBSCRIPTION_SIGN_IN_PROVIDER_IDS = new Set<string>(['openai']);
 
 /**
  * One subscription-status row per subscription-capable provider. The row's copy
- * names the provider from data — never a vendor hardcoded in the field set — so
+ * names the provider from data, never a vendor hardcoded in the field set, so
  * the status surface tracks whatever subscription-capable providers the runtime
  * reports, not a single assumed one.
  */

package/src/input/onboarding/onboarding-runtime-status.ts

@@ -41,7 +41,7 @@ export function runtimePortDiagnostic(
     // The only mode left after every named branch above is 'embedded', and this
     // function reads either endpoint's status (see getRuntimeEndpointStatus):
     // the daemon status can never carry it in this product (adoptOnly is
-    // hardcoded — see bootstrap.ts), but the HTTP listener's status legitimately
+    // hardcoded, see bootstrap.ts), but the HTTP listener's status legitimately
     // does, since the SDK always reports its in-process listener as 'embedded'.
     return `An embedded GoodVibes service is running at ${status.baseUrl}.`;
   }
@@ -66,7 +66,7 @@ export function isRuntimeEndpointActive(
   // This helper serves both endpoints (see handler-onboarding.ts's daemon and
   // httpListener calls), so the 'embedded' arm stays live for httpListener even
   // though a daemon status here is never 'embedded' in this product (adoptOnly
-  // is hardcoded — see bootstrap.ts). Dropping the arm would mark a running
+  // is hardcoded, see bootstrap.ts). Dropping the arm would mark a running
   // HTTP listener as inactive.
   if (status) return status.mode === 'embedded' || status.mode === 'external';
   return endpoint === 'daemon'
@@ -80,7 +80,7 @@ export function isRuntimeEndpointOccupyingConfiguredPort(
 ): boolean {
   const status = getRuntimeEndpointStatus(state, endpoint);
   // 'incompatible' means a GoodVibes daemon holds the configured port (we just
-  // did not adopt it) — the port IS occupied, so report it as such. As with
+  // did not adopt it), the port IS occupied, so report it as such. As with
   // isRuntimeEndpointActive above, 'embedded' stays live here for httpListener
   // (daemon status never carries it in this product).
   if (status) return status.mode === 'embedded' || status.mode === 'external' || status.mode === 'blocked' || status.mode === 'incompatible';