@pellux/goodvibes-tui 2.0.14 → 2.0.15

package/src/input/config-modal.ts

@@ -33,11 +33,11 @@ export interface ConfigModalRenderRow {
   readonly selected: boolean;
   readonly selectable: boolean;
   /** True when this frozen row has no live counterpart this tick (value went
-   *  stale — kept in place, dimmed, until the next interaction boundary). */
+   *  stale, kept in place, dimmed, until the next interaction boundary). */
   readonly stale: boolean;
 }
 
-/** Everything renderConfigModal needs — computed by overlaying live values onto
+/** Everything renderConfigModal needs, computed by overlaying live values onto
  *  the frozen structure so layout stays stable between key presses. */
 export interface ConfigModalRenderModel {
   readonly title: string;
@@ -60,7 +60,7 @@ const DEFAULT_VISIBLE_ROWS = 10;
  * minus the 4-column text margin and 2-column selection indicator (see
  * ModalFactory._renderListSection's `contentW - 2`). renderConfigModal()
  * itself always computes and passes the ACTUAL width for the current
- * terminal size — this constant only matters when getRenderModel() is
+ * terminal size, this constant only matters when getRenderModel() is
  * called directly without going through the renderer.
  */
 const DEFAULT_LABEL_WRAP_WIDTH = 70;
@@ -68,16 +68,16 @@ const DEFAULT_LABEL_WRAP_WIDTH = 70;
 /**
  * Stable id for the synthesized "no matches" row item 1 injects when a
  * filter query excludes every selectable row in a tab. Non-selectable (info
- * rows are never filtered — see ConfigModal's filter doc) so this id can
+ * rows are never filtered, see ConfigModal's filter doc) so this id can
  * never collide with a real surface row id.
  */
 const FILTER_NO_MATCH_ROW_ID = '__config_modal_filter_no_match__';
 
 /**
- * ConfigModal — the single, generic, named config-modal host. One instance
+ * ConfigModal, the single, generic, named config-modal host. One instance
  * lives on the InputHandler (like `settingsModal`); it renders whatever
  * `ConfigModalSurface` is currently open. Key routing is a single
- * `handleConfigModalToken` path (handler-modal-routes.ts) — no parallel input
+ * `handleConfigModalToken` path (handler-modal-routes.ts), no parallel input
  * system. See config-modal-types.ts for the surface contract and the liveness
  * doctrine this class enforces.
  */
@@ -97,28 +97,28 @@ export class ConfigModal {
   private pendingConfirmKey: string | null = null;
 
   /**
-   * item 1 — the host's own type-to-filter, armed with '/' (matching
+   * item 1, the host's own type-to-filter, armed with '/' (matching
    * the pre-existing "'/' to filter" convention: scrollable-list-panel.ts's
    * opt-in filter, and SettingsModal's own '/'-armed search). `filterActive`
    * gates whether printable keys go to the query instead of nav/actions;
    * `filterQuery` is the TEXT-CAPTURE buffer itself (a multi-char paste token
-   * appends in one shot — see handleConfigModalToken). Filtering only ever
+   * appends in one shot, see handleConfigModalToken). Filtering only ever
    * narrows the ACTIVE tab's rows and is reset on tab switch (a query typed
    * against one tab's rows has no defined meaning against another's).
    *
-   * FILTER-CONVENTION RULING (batch integration — where '/'-armed vs instant
+   * FILTER-CONVENTION RULING (batch integration, where '/'-armed vs instant
    * filtering is decided across the TUI's list UIs):
    *   - '/'-ARMED here (config-modal host surfaces): these surfaces bind PLAIN
    *     single keys to ACTION HOTKEYS (e.g. 'r' refresh, 'd' delete). A key
    *     can't be both an action and a filter character, so filtering must be
    *     explicitly armed with '/' first. This is a deliberate design choice.
-   *   - INSTANT filtering (pure pickers — help overlay, command palette,
+   *   - INSTANT filtering (pure pickers, help overlay, command palette,
    *     selection-modal): these have no single-key actions, so every printable
    *     key is unambiguously a filter character and narrows the list on the
    *     first keystroke. This is a deliberate design choice (selection-modal-overlay.ts).
    *   - Both keep single-Esc-close semantics. The only extra step is here: a
    *     two-stage Esc (first Esc clears a NON-EMPTY query, second Esc closes)
-   *     — see the Esc branch in handleConfigModalToken. With an empty query,
+   *    , see the Esc branch in handleConfigModalToken. With an empty query,
    *     one Esc closes, exactly like the instant pickers.
    */
   private filterActive = false;
@@ -201,8 +201,8 @@ export class ConfigModal {
 
   /**
    * Re-capture the structure from the current live view. Called on every key
-   * press (never during a pure render tick — with one exception: before the
-   * FIRST interaction, getRenderModel syncs so async loads paint) — this is
+   * press (never during a pure render tick, with one exception: before the
+   * FIRST interaction, getRenderModel syncs so async loads paint), this is
    * what makes structural changes appear only at an interaction boundary.
    * Selection + active tab are preserved by id and clamped if their target
    * vanished.
@@ -221,7 +221,7 @@ export class ConfigModal {
     this._clampScroll();
   }
 
-  // ── Filter (item 1 — each mutation is an interaction boundary) ──────
+  // ── Filter (item 1, each mutation is an interaction boundary) ──────
 
   isFilterActive(): boolean {
     return this.filterActive;
@@ -240,7 +240,7 @@ export class ConfigModal {
   }
 
   /**
-   * Append text to the query — the WHOLE token value in one call, so a
+   * Append text to the query, the WHOLE token value in one call, so a
    * multi-char paste token lands in the filter atomically rather than being
    * split into per-char nav/action dispatch (the text-capture invariant this
    * item's brief calls out). A no-op when the filter isn't armed.
@@ -261,7 +261,7 @@ export class ConfigModal {
    * Esc while filtering: a non-empty query is CLEARED (stays armed, ready to
    * retype without pressing '/' again) and this returns true so the caller
    * (handleConfigModalToken) stops here instead of closing the modal. An
-   * empty query returns false — the caller falls through to the ordinary
+   * empty query returns false, the caller falls through to the ordinary
    * close path, preserving single-Esc-close for the no-filter case (the one
    * documented exception: two-stage Esc only when there is something to
    * clear).
@@ -305,7 +305,7 @@ export class ConfigModal {
 
   private _switchTab(dir: 1 | -1): void {
     this._clearConfirm();
-    // A filter query is scoped to the tab it was typed against — switching
+    // A filter query is scoped to the tab it was typed against, switching
     // tabs resets it (item 1), same as statusMessage below.
     this.filterActive = false;
     this.filterQuery = '';
@@ -367,7 +367,7 @@ export class ConfigModal {
 
   /**
    * In-surface "jump": switch the active tab (by id) and select a specific
-   * row (by id) within it — the mechanism a surface's own action handler
+   * row (by id) within it, the mechanism a surface's own action handler
    * uses to move the user from one tab to a specific row in another (e.g.
    * the Memory modal's Proposals tab jumping to an affected record in the
    * Review Queue tab). An interaction boundary (re-syncs structure first,
@@ -414,20 +414,20 @@ export class ConfigModal {
   }
 
   /**
-   * Compute the render model: frozen structure + live value overlay. Pure —
+   * Compute the render model: frozen structure + live value overlay. Pure,
    * does NOT sync structure, so calling it repeatedly with only value mutations
    * yields byte-identical layout (the liveness contract).
    *
    * `labelWrapWidth` is the wrap column ModalFactory's list section will use
    * for row labels (renderConfigModal computes and passes the real one for
    * the current terminal size). It exists so the wrap-clamp below (* item 2) can pre-empt a live label growing past ModalFactory's wrap width
-   * mid-tick — a structural change (an extra visible line) without an
-   * interaction — by measuring wrapped line counts here, before the label
+   * mid-tick, a structural change (an extra visible line) without an
+   * interaction, by measuring wrapped line counts here, before the label
    * ever reaches the renderer.
    */
   getRenderModel(labelWrapWidth: number = DEFAULT_LABEL_WRAP_WIDTH): ConfigModalRenderModel {
     // Pre-first-interaction: async onOpen loads may restructure freely (the
-    // user hasn't engaged a cursor yet) — sync so "Loading…" is replaced by
+    // user hasn't engaged a cursor yet), sync so "Loading…" is replaced by
     // real content on the load's own requestRender, not the next keypress.
     if (this.active && !this.interactedSinceOpen) this.syncStructure();
     const live = this.surface?.buildView() ?? null;
@@ -471,7 +471,7 @@ export class ConfigModal {
 
     // item 1: while filtering, the surface's own action/tab hints are
     // unreliable (their printable-letter keys are captured by the filter
-    // instead of firing — see handleConfigModalToken), so the footer swaps
+    // instead of firing, see handleConfigModalToken), so the footer swaps
     // to just the filter status (query + a truthful match count) and the
     // Esc contract for this mode. See getFilterQuery/isFilterActive callers.
     const filtering = this.filterActive;
@@ -512,7 +512,7 @@ export class ConfigModal {
    * `frozenLabel` is what the row wrapped to when the structure was captured
    * (open/nav/filter keystroke); `liveLabel` is this tick's value. If the
    * live label would wrap into MORE lines than the frozen one, that is a
-   * structural change (an extra visible row) happening without a keypress —
+   * structural change (an extra visible row) happening without a keypress,
    * exactly the hazard this closes. Clamp to the frozen line count, ellipsis
    * on the last line to signal truncation; a live label that wraps to the
    * SAME or FEWER lines passes through untouched (not the documented hazard).
@@ -532,20 +532,20 @@ export class ConfigModal {
 
   /**
    * Modal sizing rule (owner, zero tolerance: a modal/list must never clip
-   * its full descriptive text — size to content or scroll, never clip).
+   * its full descriptive text, size to content or scroll, never clip).
    * `scrollOffset`/`visible` are ROW-count based, which is exactly right when
-   * every row wraps to one line — but ModalFactory renders each row's WRAPPED
+   * every row wraps to one line, but ModalFactory renders each row's WRAPPED
    * lines and then bounds the whole section to a fixed content-row budget
    * computed from `visible` (see renderConfigModal's targetContentRows). A
    * row whose label wraps to MULTIPLE lines can therefore push the total past
    * that budget, and the LAST row handed to the renderer gets cut off
-   * mid-line instead of being deferred to the next scroll page — the
+   * mid-line instead of being deferred to the next scroll page, the
    * clipping hazard this closes.
    *
    * Windows `rows` starting at `scrollOffset`, greedily including whole rows
    * while their CUMULATIVE wrapped-line count (at `wrapWidth`) stays within
    * the `visible` line budget. Always includes at least the first candidate
-   * row so the view is never empty — a single row that alone exceeds the
+   * row so the view is never empty, a single row that alone exceeds the
    * budget is shown in full rather than replaced with nothing; only a
    * SUBSEQUENT row that would overflow is deferred to scrolling instead of
    * being rendered partially.
@@ -572,7 +572,7 @@ export class ConfigModal {
   private _filterStatusHint(matched: number, total: number): string {
     const q = this.filterQuery;
     if (q.length === 0) return `/ type to filter (${total} row${total === 1 ? '' : 's'})`;
-    return `/${q} — ${matched} of ${total} match${matched === 1 ? '' : 'es'}`;
+    return `/${q}: ${matched} of ${total} match${matched === 1 ? '' : 'es'}`;
   }
 
   /** Apply the active filter query to every tab's rows. A no-op passthrough when not filtering. */
@@ -585,7 +585,7 @@ export class ConfigModal {
   /**
    * Narrow one tab's rows to those matching `query` (case-insensitive
    * substring on the label). Non-selectable rows (section titles, honest
-   * empty-state copy, warning banners) always pass through unfiltered —
+   * empty-state copy, warning banners) always pass through unfiltered,
    * they're context, not data. When the query excludes every selectable row
    * but the tab genuinely had some, append the honest "no rows match" line
    * (item 1's empty-result case) instead of silently showing nothing

package/src/input/daemon-owned-settings-descriptions.ts

@@ -1,11 +1,11 @@
 /**
- * daemon-owned-settings-descriptions.ts — appends an honest storage-location
+ * daemon-owned-settings-descriptions.ts, appends an honest storage-location
  * note to any settings-modal entry whose config key is daemon-owned (see
  * `@pellux/goodvibes-sdk/platform/config`'s `isDaemonOwnedConfigKey`).
  *
  * Before the daemon-owned-config migration, every product wrote every key
  * into its own per-surface silo, and a value like `surfaces.telegram.token`
- * looked — from the settings modal — exactly like any other TUI-local
+ * looked, from the settings modal, exactly like any other TUI-local
  * setting. It never was: the daemon is the only process that reads it, so a
  * value the TUI wrote to its own surface file could silently do nothing.
  * This enrichment tells the truth in the one place a user checks: the
@@ -25,8 +25,8 @@ import type { SettingEntry, SettingsCategory } from './settings-modal-types.ts';
 /** Build the daemon-owned disclosure note for a given daemon store path. */
 export function daemonOwnedSettingNote(daemonTierPath: string | null): string {
   return daemonTierPath
-    ? `This value is stored in the daemon's own configuration (${daemonTierPath}) and applies to every client — not just this one.`
-    : "This value is stored in the daemon's own configuration and applies to every client — not just this one.";
+    ? `This value is stored in the daemon's own configuration (${daemonTierPath}) and applies to every client; not just this one.`
+    : "This value is stored in the daemon's own configuration and applies to every client; not just this one.";
 }
 
 /**

package/src/input/delete-key-policy.ts

@@ -1,5 +1,5 @@
 // ---------------------------------------------------------------------------
-// Delete-key policy — single source of truth for text-editing key semantics
+// Delete-key policy, single source of truth for text-editing key semantics
 //
 // In text-editing contexts:
 //   'backspace'  → delete one character backward (the character before cursor /

package/src/input/exec-env-scrub-config.ts

@@ -1,5 +1,5 @@
 /**
- * exec-env-scrub-config.ts — TUI-local synthetic settings entry for
+ * exec-env-scrub-config.ts, TUI-local synthetic settings entry for
  * permissions.execEnvScrubAllowlist.
  *
  * The SDK's credential-bearing env-var scrub (platform/tools/exec/credential-env.ts)
@@ -8,24 +8,24 @@
  * exec command, and is threaded into `registerAllTools`'s `credentialEnvScrub`
  * option (bootstrap-core.ts) so a consumer can wire an operator-configured
  * allowlist instead of the scrub always resolving to its built-in default. The
- * scrub's master switch stays on: this entry configures ONLY the allowlist —
+ * scrub's master switch stays on: this entry configures ONLY the allowlist,
  * variable NAMES always kept even though their name looks credential-bearing
  * (case-insensitive), for a command that genuinely needs one (e.g. a CI runner
  * re-reading its own signing key). NEVER a value; the scrub only ever inspects
  * and reports NAMES.
  *
- * `permissions.execEnvScrubAllowlist` is not in CONFIG_SCHEMA — there is no SDK
+ * `permissions.execEnvScrubAllowlist` is not in CONFIG_SCHEMA, there is no SDK
  * config domain for the exec env scrub at all (it is consumer-wired, not
  * schema-registered), so this mirrors the other TUI-local synthetic settings
  * (worktree.setup.*, sandbox.egressAllowlist, …). No defensive try/catch is
  * needed: 'permissions' already exists as a DEFAULT_CONFIG section (it backs
  * permissions.mode/tools.*), so a plain get()/set() resolves the path without
- * throwing — only the LEAF field is new, and ConfigManager.resolvePath never
+ * throwing, only the LEAF field is new, and ConfigManager.resolvePath never
  * validates leaf existence.
  *
  * Stored as a JSON array of strings on disk; the settings modal's inline
- * editor is a single-line text field, so — matching worktree.setup.* and
- * sandbox.egressAllowlist/workspaceWritable — it is displayed and edited as a
+ * editor is a single-line text field, so, matching worktree.setup.* and
+ * sandbox.egressAllowlist/workspaceWritable, it is displayed and edited as a
  * comma-separated list and parsed back into an array on commit (see
  * isExecEnvScrubAllowlistConfigKey's use in settings-modal.ts#commitEdit).
  */
@@ -55,7 +55,7 @@ export const EXEC_ENV_SCRUB_ALLOWLIST_SYNTHETIC_SETTING: ConfigSetting = {
   key: EXEC_ENV_SCRUB_ALLOWLIST_CONFIG_KEY,
   type: 'string',
   default: [],
-  description: 'Comma-separated environment variable NAMES always kept in a spawned exec command\'s environment even when the name looks credential-bearing (case-insensitive). Names only — this never inspects or reports values. The scrub itself stays on; this only widens what it lets through. Empty = the built-in credential-name scrub applies with no extra exceptions.',
+  description: 'Comma-separated environment variable NAMES always kept in a spawned exec command\'s environment even when the name looks credential-bearing (case-insensitive). Names only: this never inspects or reports values. The scrub itself stays on; this only widens what it lets through. Empty = the built-in credential-name scrub applies with no extra exceptions.',
 };
 
 /** Build the synthetic SettingEntry for permissions.execEnvScrubAllowlist. */

package/src/input/feature-unit-layout.ts

@@ -1,5 +1,5 @@
 /**
- * feature-unit-layout — every platform capability renders as a feature UNIT in
+ * feature-unit-layout, every platform capability renders as a feature UNIT in
  * its settings DOMAIN: the feature's real enablement row (its domain settings
  * key, with the feature's name and full description attached) immediately
  * followed by the settings keys that tune it.
@@ -12,7 +12,7 @@
  * configures the feature.
  *
  * A feature-unit header is the REAL config row for the feature's enablement
- * key — boolean headers toggle, enum headers cycle their mode choices through
+ * key, boolean headers toggle, enum headers cycle their mode choices through
  * the ordinary settings interactions. Two features can share one enablement
  * key (e.g. behavior.compactionStrategy drives both compaction features);
  * each still gets its own header row so every capability keeps its own name,
@@ -25,7 +25,7 @@ import type { ConfigSetting } from '@pellux/goodvibes-sdk/platform/config';
 import { getConfigSchemaSetting } from '@pellux/goodvibes-terminal-shell';
 import type { FlagEntry, SettingEntry, SettingsCategory } from './settings-modal-types.ts';
 
-/** Every enablement key — these rows render as feature headers, never as plain sub-rows. */
+/** Every enablement key, these rows render as feature headers, never as plain sub-rows. */
 const ENABLEMENT_KEYS: ReadonlySet<string> = new Set(
   FEATURE_SETTINGS.map((feature) => feature.enablement.key),
 );
@@ -41,7 +41,7 @@ export function getFeatureUnitHostCategory(featureId: string): SettingsCategory
 
 /**
  * Map every non-enablement settings key claimed by any feature to the feature
- * that OWNS it — the first feature (in FEATURE_SETTINGS declaration order)
+ * that OWNS it, the first feature (in FEATURE_SETTINGS declaration order)
  * whose settings list contains it. A key claimed by two features is listed
  * once, under its owner, so no settings key double-lists across feature units.
  */
@@ -60,7 +60,7 @@ export function buildConfigKeyOwnership(): Map<string, string> {
  * Build a feature unit's header row: the REAL config row for the feature's
  * enablement key (so booleans toggle and enums cycle their mode choices
  * through the ordinary settings paths), cloned with the feature attached.
- * When the category has no row for the key (it should always have one — every
+ * When the category has no row for the key (it should always have one, every
  * binding key is CONFIG_SCHEMA-registered), the schema descriptor is used
  * directly so the header still renders honestly.
  */

package/src/input/feed-context-factory.ts

@@ -1,13 +1,13 @@
 /**
- * feed-context-factory.ts — Construction and mutable-field sync for InputFeedContext.
+ * feed-context-factory.ts, Construction and mutable-field sync for InputFeedContext.
  *
  * Extracted from handler.ts (review follow-up, 0.18.23) to keep handler.ts under the
  * 800-line architecture cap.
  *
  * Two exported functions:
- *   - `buildInitialFeedContext` — assembles the long-lived InputFeedContext from
+ *   - `buildInitialFeedContext`, assembles the long-lived InputFeedContext from
  *     pre-built field groups. Called once at InputHandler construction time.
- *   - `syncFeedContextMutableFields` — copies mutable scalar fields into the
+ *   - `syncFeedContextMutableFields`, copies mutable scalar fields into the
  *     already-allocated context before each feed() dispatch.
  */
 import type { InputFeedContext } from './handler-feed.ts';
@@ -46,14 +46,14 @@ import type { PanelBurstGuardState } from './panel-paste-flood-guard.ts';
  *
  * **Mutable fields** (synced per-feed via syncFeedContextMutableFields or inside
  * action closures that call syncFeedContextMutableFields):
- *   - `prompt`, `cursorPos` — current text buffer state
- *   - `commandMode`, `panelFocused`, `indicatorFocused` — focus-mode flags
- *   - `helpOverlayActive`, `helpScrollOffset` — help overlay state
- *   - `shortcutsOverlayActive`, `shortcutsScrollOffset` — shortcuts overlay state
- *   - `nextPasteId`, `nextImageId` — monotonically increasing ID counters
- *   - `mouseDownRow`, `mouseDownCol` — drag-tracking coordinates
- *   - `contentWidth` — reflow width, updated by setContentWidth()
- *   - `selectionCallback` — current selection modal callback (nullable)
+ *   - `prompt`, `cursorPos`, current text buffer state
+ *   - `commandMode`, `panelFocused`, `indicatorFocused`, focus-mode flags
+ *   - `helpOverlayActive`, `helpScrollOffset`, help overlay state
+ *   - `shortcutsOverlayActive`, `shortcutsScrollOffset`, shortcuts overlay state
+ *   - `nextPasteId`, `nextImageId`, monotonically increasing ID counters
+ *   - `mouseDownRow`, `mouseDownCol`, drag-tracking coordinates
+ *   - `contentWidth`, reflow width, updated by setContentWidth()
+ *   - `selectionCallback`, current selection modal callback (nullable)
  */
 export interface FeedContextMutableInit {
   prompt: string;
@@ -79,18 +79,18 @@ export interface FeedContextMutableInit {
  * Stable (readonly) service references for InputFeedContext.
  *
  * **Stable references** (set once at construction, never reallocated):
- *   - `pasteRegistry`, `imageRegistry` — owned Maps, never replaced
+ *   - `pasteRegistry`, `imageRegistry`, owned Maps, never replaced
  *   - `selectionModal`, `bookmarkModal`, `settingsModal`, `sessionPickerModal`,
- *     `profilePickerModal` — modal objects constructed once
+ *     `profilePickerModal`, modal objects constructed once
  *   - `filePicker`, `modelPicker`, `contextInspectorModal`, `blockActionsMenu`,
- *     `searchManager`, `historySearch`, `onboardingWizard` — service objects constructed once
- *   - `panelManager`, `keybindingsManager` — from uiServices, stable
- *   - `modalStack` — reference to the handler's shared array
- *   - `getHistory`, `getViewportHeight`, `getScrollTop`, `scroll`, `exitApp` — callbacks
+ *     `searchManager`, `historySearch`, `onboardingWizard`, service objects constructed once
+ *   - `panelManager`, `keybindingsManager`, from uiServices, stable
+ *   - `modalStack`, reference to the handler's shared array
+ *   - `getHistory`, `getViewportHeight`, `getScrollTop`, `scroll`, `exitApp`, callbacks
  *   - `commandRegistry`, `commandContext`, `autocomplete`, `inputHistory`,
- *     `conversationManager` — wired after construction; synced at feed() entry only
+ *     `conversationManager`, wired after construction; synced at feed() entry only
  *     (not per-action) since no in-feed action changes them
- *   - `focusTracker` — shared instance from uiServices.platform, mutated
+ *   - `focusTracker`, shared instance from uiServices.platform, mutated
  *     directly by feedInputTokens() on 'focus' tokens; never reallocated
  *
  * **Rationale:** per-feed mutation on a single object avoids per-keystroke GC pressure
@@ -126,7 +126,7 @@ export interface FeedContextStableRefs {
   keybindingsManager: KeybindingsManager;
   killRing: KillRing;
   focusTracker: FocusTracker;
-  /** item 5 — paste-flood guard state, mutated in place (never reallocated). */
+  /** item 5, paste-flood guard state, mutated in place (never reallocated). */
   panelBurstGuard: PanelBurstGuardState;
   getHistory: () => InfiniteBuffer;
   getViewportHeight: () => number;
@@ -176,12 +176,12 @@ export interface FeedContextClosures {
 }
 
 /**
- * buildInitialFeedContext — Allocate the single InputFeedContext that lives for the
+ * buildInitialFeedContext, Allocate the single InputFeedContext that lives for the
  * lifetime of the InputHandler.
  *
  * Accepts pre-built field groups from handler.ts where private access is available.
  * The resulting context object is mutated in place on every feed() call via
- * syncFeedContextMutableFields — no re-allocation occurs per keystroke.
+ * syncFeedContextMutableFields, no re-allocation occurs per keystroke.
  *
  * @param mutable  Initial mutable scalar values (synced per-feed).
  * @param stable   Stable service references (set once, never replaced).
@@ -206,31 +206,31 @@ export function buildInitialFeedContext(
 }
 
 /**
- * syncFeedContextMutableFields — Copy mutable scalar fields into the reused
+ * syncFeedContextMutableFields, Copy mutable scalar fields into the reused
  * InputFeedContext object.
  *
  * **Fields synced (updated on every call):**
- *   - `prompt` — current prompt text buffer
- *   - `cursorPos` — caret position within prompt
- *   - `commandMode` — whether command-mode prefix is active
- *   - `panelFocused` — whether the active panel owns keyboard focus
- *   - `indicatorFocused` — whether the status indicator owns focus
- *   - `helpOverlayActive` / `helpScrollOffset` — help overlay visibility and scroll
- *   - `shortcutsOverlayActive` / `shortcutsScrollOffset` — shortcuts overlay state
- *   - `selectionCallback` — the current in-flight selection modal callback
- *   - `nextPasteId` / `nextImageId` — monotonically increasing allocation counters
- *   - `mouseDownRow` / `mouseDownCol` — drag-start coordinates
+ *   - `prompt`, current prompt text buffer
+ *   - `cursorPos`, caret position within prompt
+ *   - `commandMode`, whether command-mode prefix is active
+ *   - `panelFocused`, whether the active panel owns keyboard focus
+ *   - `indicatorFocused`, whether the status indicator owns focus
+ *   - `helpOverlayActive` / `helpScrollOffset`, help overlay visibility and scroll
+ *   - `shortcutsOverlayActive` / `shortcutsScrollOffset`, shortcuts overlay state
+ *   - `selectionCallback`, the current in-flight selection modal callback
+ *   - `nextPasteId` / `nextImageId`, monotonically increasing allocation counters
+ *   - `mouseDownRow` / `mouseDownCol`, drag-start coordinates
  *
  * **Fields intentionally excluded (synced only at feed() entry, not here):**
- *   - `contentWidth` — semi-stable; only changes on terminal resize via setContentWidth().
+ *   - `contentWidth`, semi-stable; only changes on terminal resize via setContentWidth().
  *     Synced at feed() entry because it is only relevant for layout, not action-reaction
  *     sequences within a single feed.
- *   - `commandRegistry`, `commandContext` — wired after construction via
+ *   - `commandRegistry`, `commandContext`, wired after construction via
  *     setCommandRegistry(); synced at feed() entry since no in-feed action changes them.
- *   - `autocomplete` — wired after construction; synced at feed() entry.
- *   - `inputHistory`, `conversationManager` — late-wired service handles; stable within
+ *   - `autocomplete`, wired after construction; synced at feed() entry.
+ *   - `inputHistory`, `conversationManager`, late-wired service handles; stable within
  *     a feed. Synced at feed() entry only.
- *   - `requestRender` — swapped to a buffered version at feed() entry and restored in
+ *   - `requestRender`, swapped to a buffered version at feed() entry and restored in
  *     the finally block; not managed by this function.
  *
  * Called from within action closures (`handleEscape`, `handleCtrlC`, `handleUndo`,

package/src/input/file-picker.ts

@@ -15,7 +15,7 @@ export class FilePickerModal {
   public searchFocused = true;
   public results: string[] = [];
   public selectedIndex = 0;
-  /** Position in the prompt where @ was typed — used to replace @query with the selected path */
+  /** Position in the prompt where @ was typed, used to replace @query with the selected path */
   public insertPos = 0;
   /** When true, selected file inserts as !@path (inject mode) instead of @path */
   public injectMode = false;

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

@@ -121,7 +121,7 @@ export function handleCommandModeToken(state: CommandModeRouteState, token: Inpu
           // Defense in depth for EVERY command, not just the ones that
           // already guard their own internals: a handler that throws or
           // awaits a rejected promise must never become a silent unhandled
-          // rejection — it renders the same way an unknown command does.
+          // rejection, it renders the same way an unknown command does.
           state.conversationManager?.log(`Command /${name} failed: ${summarizeError(err)}`, { fg: '#ef4444' });
           state.requestRender();
         });
@@ -149,7 +149,7 @@ function withPanelFocusSync(context: CommandContext, state: CommandModeRouteStat
     showPanel: context.showPanel
       ? (panelId, pane, target, opts) => {
           context.showPanel?.(panelId, pane, target, opts);
-          // showPanel no longer transfers focus by default — the command
+          // showPanel no longer transfers focus by default, the command
           // path leaves the composer focused (ui-openers.ts's showPanel only
           // calls focusPanels() when opts.focus is set). Only flip the local
           // panelFocused mirror when the caller explicitly asked for focus;

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

@@ -123,7 +123,7 @@ export function registerPaste(
 
   // Terminals transmit the line breaks inside a bracketed paste as \r (the
   // byte Enter sends), and external clipboards can carry \r\n. Normalize to
-  // \n only here in the text branch — the image sniffing above must see the
+  // \n only here in the text branch, the image sniffing above must see the
   // raw bytes (PNG's magic sequence itself contains \r\n).
   const text = content.replace(/\r\n?/g, '\n');
   const lines = text.split('\n');
@@ -246,7 +246,7 @@ export function findMarkerAtPos(prompt: string, pos: number): { start: number; e
 /**
  * describeBlockForReceipt - Honest, human-readable name for a block, used by
  * every anchor-based action's receipt (Ctrl+Y/B/S/A, Tab, block-actions menu)
- * so the transcript always says exactly what was acted on — never just an
+ * so the transcript always says exactly what was acted on, never just an
  * internal collapseKey or a bare "done".
  */
 export function describeBlockForReceipt(block: BlockMeta): string {
@@ -286,7 +286,7 @@ export function handleCopy(
  * handleBlockCopy - Ctrl+Y: copy the content of the block the user is
  * actually looking at. `getAnchorLine` resolves to the viewport's
  * bottom-most visible line (see getViewportBottomLine in
- * conversation-layout.ts), not the raw scroll position — the top of the
+ * conversation-layout.ts), not the raw scroll position, the top of the
  * viewport is off-screen-above once the transcript is longer than one page.
  */
 export function handleBlockCopy(
@@ -306,7 +306,7 @@ export function handleBlockCopy(
   copyToClipboard(nearest.rawContent);
   onCopied();
   // An explicit action on this block permanently exempts it from the
-  // search-close auto-re-collapse (see ConversationManager.noteUserTouch) —
+  // search-close auto-re-collapse (see ConversationManager.noteUserTouch),
   // copying content the user just had search reveal is a deliberate choice
   // to keep it visible, not an accident of typing a query.
   conversationManager.searchExpansion.noteUserTouch(nearest.collapseKey);
@@ -331,7 +331,7 @@ export function handleBookmark(
   }
   const label = `${nearest.type}: ${nearest.rawContent.slice(0, 40).replace(/\n/g, ' ')}`;
   const added = bookmarkManager.toggle(nearest.collapseKey, label);
-  // See handleBlockCopy's note — bookmarking is an explicit block action too.
+  // See handleBlockCopy's note, bookmarking is an explicit block action too.
   conversationManager.searchExpansion.noteUserTouch(nearest.collapseKey);
   const target = describeBlockForReceipt(nearest);
   const msg = added
@@ -355,7 +355,7 @@ export function handleBlockSave(
     requestRender();
     return;
   }
-  // See handleBlockCopy's note — saving to a file is an explicit block action too.
+  // See handleBlockCopy's note, saving to a file is an explicit block action too.
   conversationManager.searchExpansion.noteUserTouch(nearest.collapseKey);
   const target = describeBlockForReceipt(nearest);
   try {
@@ -483,7 +483,7 @@ export function handleCtrlC(
     // Non-fatal to the exit chord; the quit-window bookkeeping still runs.
   }
   // A press that silenced LIVE speech is consumed by that job (an earlier replay
-  // fix) — symmetric with the prompt-clearing press above. The quit chord
+  // fix), symmetric with the prompt-clearing press above. The quit chord
   // ("Ctrl+C x2") starts from a quiet state; turn-aborts still count toward
   // the double-press.
   if (stoppedSpeech) return;
@@ -492,7 +492,7 @@ export function handleCtrlC(
   // one's outcome, whichever branch below runs. Without this, a hide-timer
   // scheduled by an earlier press could still fire after a newer notice
   // window opened, or after exitApp() was just called (if exitApp isn't
-  // synchronous) — flipping showExitNotice/requestRender during a shutdown
+  // synchronous), flipping showExitNotice/requestRender during a shutdown
   // the user already believes is in progress.
   if (lastCtrlCTimeoutId !== null) {
     clearTimeout(lastCtrlCTimeoutId);