@pellux/goodvibes-tui 2.0.14 → 2.0.16

package/src/input/search.ts

@@ -6,7 +6,7 @@ export interface SearchMatch {
   col: number;
   length: number;
   /**
-   * Present when this match lives inside content that is still collapsed —
+   * Present when this match lives inside content that is still collapsed,
    * `line`/`col` are placeholders (the containing block's own header line,
    * col 0) rather than a real navigable position. `collapseKeys` names every
    * collapse key that must be expanded to make the hit a real rendered
@@ -18,7 +18,7 @@ export interface SearchMatch {
   primaryKey?: string;
 }
 
-/** Count non-overlapping occurrences of `needle` in `haystack` — used for
+/** Count non-overlapping occurrences of `needle` in `haystack`, used for
  *  the collapsed-content scan below, mirroring the per-line indexOf loop
  *  search() already runs over the rendered buffer, so a hidden hit counts
  *  the same way a visible one would once revealed. */
@@ -63,19 +63,19 @@ export class SearchManager {
     this.wrapAround = false;
   }
 
-  /** Lock the query — switches from typing mode to navigation mode. */
+  /** Lock the query, switches from typing mode to navigation mode. */
   lock(): void {
     this.locked = true;
   }
 
-  /** Unlock — return to typing mode. */
+  /** Unlock, return to typing mode. */
   unlock(): void {
     this.locked = false;
   }
 
   /**
    * Close search mode. Re-collapses every block/group search auto-expanded
-   * while it was open — except ones the user explicitly acted on (toggled,
+   * while it was open, except ones the user explicitly acted on (toggled,
    * copied, bookmarked, saved) while they were expanded, which stay exactly
    * as the user left them (see SearchExpansionTracker.restoreOnto).
    * `conversationManager` is optional so existing call sites that never had
@@ -88,19 +88,19 @@ export class SearchManager {
   }
 
   /**
-   * Update query and find matches — honestly counting hits inside collapsed
+   * Update query and find matches, honestly counting hits inside collapsed
    * blocks and folded tool-result groups WITHOUT expanding them. Expansion
    * only happens on navigation (see revealCurrentMatch): typing a query must
    * never collapse-destroy the transcript the user folded on purpose, but a
    * "no matches" while text the user watched stream by sits collapsed
-   * somewhere would be a lie too — so the count includes it, tagged as
+   * somewhere would be a lie too, so the count includes it, tagged as
    * hidden until the user actually navigates there.
    *
-   * That collapsed corpus is the block's own RAW content, plus — for a
-   * folded tool-result group — the raw content of every one of its member
+   * That collapsed corpus is the block's own RAW content, plus, for a
+   * folded tool-result group, the raw content of every one of its member
    * messages (a group's own rawContent is only its summary header line, and
-   * its members push no BlockMeta of their own while folded — see
-   * conversation-tool-groups.ts — so without the member corpus a needle
+   * its members push no BlockMeta of their own while folded, see
+   * conversation-tool-groups.ts, so without the member corpus a needle
    * living inside a member would count as zero hits for text the user
    * watched stream by).
    *
@@ -124,7 +124,7 @@ export class SearchManager {
   }
 
   /**
-   * Shared match builder for search() and revealCurrentMatch() — scans the
+   * Shared match builder for search() and revealCurrentMatch(), scans the
    * currently-rendered buffer for visible matches and every collapsed
    * block/folded group for hidden ones, without mutating any collapse
    * state. Kept separate from search() so revealCurrentMatch can rebuild the
@@ -142,7 +142,7 @@ export class SearchManager {
 
         if (block.type === 'assistant_turn' && block.groupMemberIndexes) {
           // The header's own synthetic summary rarely matches, but count it
-          // honestly too — expanding the whole group is the only way to even
+          // honestly too, expanding the whole group is the only way to even
           // attempt to reveal a header-corpus hit, since there is no single
           // member to isolate it to.
           const headerHits = countOccurrences(block.rawContent.toLowerCase(), lowerQuery);
@@ -169,7 +169,7 @@ export class SearchManager {
                   line: block.startLine, col: 0, length: query.length,
                   // Revealing a member-specific hit only needs the group
                   // header (to unfold the group at all) and that member's own
-                  // key — not its siblings, which stay exactly as they were.
+                  // key, not its siblings, which stay exactly as they were.
                   collapseKeys: [block.collapseKey, memberKey], primaryKey: memberKey,
                 });
               }
@@ -203,7 +203,7 @@ export class SearchManager {
 
     // Stable sort (V8/JSC have guaranteed stable Array#sort) puts each
     // block's hidden hits in the same relative order they were generated
-    // in — self-corpus first, then members in order — right where that
+    // in, self-corpus first, then members in order, right where that
     // block's header line falls among the visible matches.
     return [...visibleMatches, ...hiddenMatches].sort((a, b) => a.line - b.line || a.col - b.col);
   }
@@ -216,7 +216,7 @@ export class SearchManager {
    * the match list against the now-expanded buffer.
    *
    * `currentMatch` is repointed at the first now-visible match inside the
-   * revealed block's new line range, rather than reset to 0 — expansion
+   * revealed block's new line range, rather than reset to 0, expansion
    * only changes how many lines that ONE block renders as, so every match
    * belonging to earlier blocks keeps its relative position.
    *
@@ -273,7 +273,7 @@ export class SearchManager {
   }
 
   /** Get the line number of the current match (for scroll). Returns -1 for a
-   *  match that's still hidden inside collapsed content — revealCurrentMatch
+   *  match that's still hidden inside collapsed content, revealCurrentMatch
    *  must run first to give it a real line. */
   getCurrentMatchLine(): number {
     if (this.matches.length === 0) return -1;
@@ -284,7 +284,7 @@ export class SearchManager {
   }
 
   /** Get matches on a given line. Hidden (still-collapsed) matches are
-   *  excluded — they have no real line to render a highlight on yet. */
+   *  excluded, they have no real line to render a highlight on yet. */
   getMatchesOnLine(lineIdx: number): SearchMatch[] {
     return this.matches.filter(m => m.line === lineIdx && !m.collapseKeys);
   }

package/src/input/selection-modal-queue.ts

@@ -1,9 +1,9 @@
 /**
- * selection-modal-queue.ts — Serializes overlapping InputHandler.openSelection() calls.
+ * selection-modal-queue.ts, Serializes overlapping InputHandler.openSelection() calls.
  *
  * openSelection (handler.ts) writes into a singleton SelectionModal plus a
  * single callback slot. A second openSelection call while one is already
- * showing used to silently overwrite both — the pre-empted caller's callback
+ * showing used to silently overwrite both, the pre-empted caller's callback
  * was never invoked, not even with null, so any caller that awaits it (see
  * the `ask()` helper in runtime/recovery-prompt.ts, which wraps the callback
  * in a Promise) hung forever. This is reachable in production: the
@@ -13,12 +13,12 @@
  *
  * Fix: queue overlapping requests FIFO and open each automatically once the
  * previous one resolves, through the SAME callback the caller already
- * supplied — openSelection's public signature never changes.
+ * supplied, openSelection's public signature never changes.
  *
  * Why the drain is deferred to a microtask instead of running inline inside
- * the resolving callback: both resolution paths — the select/enter path in
+ * the resolving callback: both resolution paths, the select/enter path in
  * handler-modal-routes.ts's dispatchSelectionAction, and the Escape path in
- * handler-modal-stack.ts's handleEscape/closeSelection — take a snapshot of
+ * handler-modal-stack.ts's handleEscape/closeSelection, take a snapshot of
  * the current selectionCallback before dispatch and write some form of that
  * snapshot back into handler/context state AFTER invoking the callback (the
  * select path re-reads via getSelectionCallback; the Escape path threads a
@@ -28,7 +28,7 @@
  * write-backs would immediately clobber the freshly-set callback for the
  * newly-opened modal. Deferring past the end of the synchronous resolution
  * (queueMicrotask) lets all of that settle first, then opens the next
- * request cleanly — the same shape as the production race this fixes, which
+ * request cleanly, the same shape as the production race this fixes, which
  * itself resolves via a deferred (macrotask) call.
  */
 import type { SelectionItem, SelectionAction, SelectionResult, SelectionModal } from './selection-modal.ts';
@@ -55,7 +55,7 @@ export interface PendingSelectionRequest {
  * modal vs. the recovery offer's 0ms macrotask), so 8 is generous headroom
  * for any future chained-picker sequence without letting a misbehaving
  * caller queue without bound. Once full, the overflowing call is resolved
- * with `null` immediately — a bounded queue must still give every caller a
+ * with `null` immediately, a bounded queue must still give every caller a
  * definite answer, never silently drop one.
  */
 export const MAX_QUEUED_SELECTIONS = 8;
@@ -115,7 +115,7 @@ export class SelectionModalQueue {
     const wrapped: SelectionModalCallback = (result) => {
       request.callback(result);
       // Only a terminal resolution (select/cancel/escape) actually closes
-      // the modal — dispatchSelectionAction's toggle/increment/decrement
+      // the modal, dispatchSelectionAction's toggle/increment/decrement
       // branch invokes the callback but leaves the modal open for further
       // interaction, so only drain the queue once the modal has gone
       // inactive, never on every callback invocation.
@@ -133,7 +133,7 @@ export class SelectionModalQueue {
   private drain(): void {
     // Something else already opened a modal in the meantime (e.g. a
     // synchronously chained openSelection call from inside the resolving
-    // callback) — leave the queue alone; that modal's own resolution will
+    // callback), leave the queue alone; that modal's own resolution will
     // drain it in turn.
     if (this.host.selectionModal.active) return;
     const next = this.pending.shift();
@@ -142,7 +142,7 @@ export class SelectionModalQueue {
   }
 }
 
-/** InputHandler's actual field/method shape — satisfies this structurally, no import needed. */
+/** InputHandler's actual field/method shape, satisfies this structurally, no import needed. */
 export interface SelectionQueueOwner {
   readonly selectionModal: Pick<SelectionModal, 'active' | 'open'>;
   selectionCallback: SelectionModalCallback | null;

package/src/input/selection-modal.ts

@@ -88,7 +88,7 @@ export class SelectionModal {
   }
 
   moveUp(): void {
-    // Skip category headers (items with no id-based content — filtered items include only selectable ones)
+    // Skip category headers (items with no id-based content, filtered items include only selectable ones)
     if (this.filteredItems.length === 0) return;
     this.selectedIndex = this.selectedIndex > 0
       ? this.selectedIndex - 1
@@ -127,7 +127,7 @@ export class SelectionModal {
     return this.filteredItems[this.selectedIndex] ?? null;
   }
 
-  /** Fuzzy match items against query — resets filteredItems */
+  /** Fuzzy match items against query, resets filteredItems */
   private filterItems(): void {
     if (this.query.length === 0) {
       this.filteredItems = this.items.slice();

package/src/input/session-picker-modal.ts

@@ -1,13 +1,13 @@
 /**
- * SessionPickerModal — state management for the /sessions picker modal.
+ * SessionPickerModal, state management for the /sessions picker modal.
  *
  * Lists LOCAL saved transcript files from SessionManager.list() (load/delete
- * operate on these — a saved JSONL session is a different thing from a live
+ * operate on these, a saved JSONL session is a different thing from a live
  * control-plane session, see below), tracks selected index, and handles
  * load/delete actions.
  *
  * Union-sessions surface: additionally surfaces the cross-surface
- * session union from `sessionBroker` (a SessionReadFacade — normally
+ * session union from `sessionBroker` (a SessionReadFacade, normally
  * `uiServices.sessions.sessionBroker`, the SessionUnionCache) so a user can
  * SEE what sessions are live/closed across every surface sharing this
  * daemon (TUI, webui, companion, automation), not just this process's own
@@ -21,7 +21,7 @@
  * Hosted sessions: a third, also read-only section lists the conversations
  * running INSIDE the daemon (`sessions.hosted.list`, held by the shared roster
  * in runtime/client/hosted-roster.ts). They belong in the place a person
- * already goes to ask "what sessions are there" — a hosted session is exactly
+ * already goes to ask "what sessions are there", a hosted session is exactly
  * that, one whose loop happens to live elsewhere. Joining one is `/hosted
  * attach <id>` rather than an Enter here, for the same reason cross-surface
  * rows are not loadable: there is no on-disk transcript this process can
@@ -38,7 +38,7 @@ import type { ConversationManager } from '../core/conversation';
 import { summarizeError } from '@pellux/goodvibes-sdk/platform/utils';
 import type { HostedRosterSnapshot } from '../runtime/client/hosted-roster.ts';
 // Re-exported so the composition site (input/handler.ts) reaches the picker and
-// its hosted roster through ONE import — that file sits on this repo's 800-line
+// its hosted roster through ONE import, that file sits on this repo's 800-line
 // per-file gate and a second import line there is a real cost.
 export { getSharedHostedSessionRoster } from '../runtime/client/hosted-roster.ts';
 
@@ -67,7 +67,7 @@ const DORMANT_CROSS_SURFACE_VIEW: CrossSurfaceView = {
 export class SessionPickerModal {
   public active = false;
   public sessions: SessionInfo[] = [];
-  /** Cross-surface session union (view-only) — see class doc. Empty until open(). */
+  /** Cross-surface session union (view-only), see class doc. Empty until open(). */
   public crossSurfaceSessions: readonly SharedSessionRecord[] = [];
   /** Honest posture for crossSurfaceSessions: mode/online/stale/offlineNote. */
   public crossSurfaceView: CrossSurfaceView = DORMANT_CROSS_SURFACE_VIEW;
@@ -76,7 +76,7 @@ export class SessionPickerModal {
   public visibleRows = 8;
   public deleteConfirmationTarget: string | null = null;
 
-  /** Daemon-hosted sessions (view-only) — see class doc. Never a fabricated empty list. */
+  /** Daemon-hosted sessions (view-only), see class doc. Never a fabricated empty list. */
   public hostedRoster: HostedRosterSnapshot = UNREAD_HOSTED_ROSTER;
 
   /** Last status message to show in the modal (e.g. error or success). */

package/src/input/settings-modal-activation.ts

@@ -1,5 +1,5 @@
 /**
- * settings-modal-activation — pure action helpers for SettingsModal.
+ * settings-modal-activation, pure action helpers for SettingsModal.
  *
  * These functions encapsulate the two activation/interaction operations:
  *   - activateSelected: toggle boolean, cycle enum, enter edit mode, launch pickers

package/src/input/settings-modal-adjustment.ts

@@ -1,5 +1,5 @@
 /**
- * settings-modal-adjustment — pure adjustment helpers for SettingsModal.
+ * settings-modal-adjustment, pure adjustment helpers for SettingsModal.
  *
  * These functions encapsulate the directional-adjustment operation:
  *   - adjustSelected: cycle enum/boolean/number values via left/right arrow keys
@@ -43,7 +43,7 @@ export function adjustSelected(
   if (ctx.editingMode) return;
 
   // Feature-unit headers are the real config rows for their enablement keys
-  // (boolean or enum), so they fall through to the ordinary branches below —
+  // (boolean or enum), so they fall through to the ordinary branches below,
   // a plain config write the settings bridge forwards to the gate manager.
 
   if (ctx.currentCategory === 'mcp') {

package/src/input/settings-modal-commit.ts

@@ -1,5 +1,5 @@
 /**
- * settings-modal-commit.ts — commitEdit(), extracted out of the SettingsModal
+ * settings-modal-commit.ts, commitEdit(), extracted out of the SettingsModal
  * class body so the class file can stay under the repo's architecture
  * line-count gate without trimming arbitrary lines to clear the number.
  *
@@ -31,7 +31,7 @@ export interface CommitEditContext {
   /**
    * The daemon's credential write, when a daemon is adopted. A daemon-scoped
    * credential must land where the daemon reads it, not in this surface's own
-   * tree — see settings-modal-secrets.ts.
+   * tree, see settings-modal-secrets.ts.
    */
   readonly daemonCredentials?: SettingsDaemonCredentialWriter | null;
   /** Renders a refused credential write; without it a failure would be silent. */
@@ -50,7 +50,7 @@ export interface CommitEditContext {
 /**
  * Commit the current editBuffer to the config. Returns true on success,
  * false if validation failed (the caller always clears editingMode/editBuffer
- * regardless — a failed commit never leaves stale edit state behind).
+ * regardless, a failed commit never leaves stale edit state behind).
  */
 export function commitEditValue(ctx: CommitEditContext): boolean {
   if (!ctx.editingMode) return false;
@@ -127,16 +127,16 @@ export function commitEditValue(ctx: CommitEditContext): boolean {
 
   if (isWorktreeSetupListConfigKey(setting.key)) {
     // Comma-separated display/edit convention for the array-backed
-    // worktree.setup.* keys — see worktree-setup-config.ts.
+    // worktree.setup.* keys, see worktree-setup-config.ts.
     ctx.setValue(setting.key, parseWorktreeSetupListInput(ctx.editBuffer));
   } else if (isSandboxExecListConfigKey(setting.key)) {
     // Same comma-separated convention for the array-backed
-    // sandbox.egressAllowlist / sandbox.workspaceWritable keys — see
+    // sandbox.egressAllowlist / sandbox.workspaceWritable keys, see
     // sandbox-exec-config.ts.
     ctx.setValue(setting.key, parseSandboxExecListInput(ctx.editBuffer));
   } else if (isExecEnvScrubAllowlistConfigKey(setting.key)) {
     // Same comma-separated convention for permissions.execEnvScrubAllowlist
-    // — see exec-env-scrub-config.ts.
+    //, see exec-env-scrub-config.ts.
     ctx.setValue(setting.key, parseExecEnvScrubAllowlistInput(ctx.editBuffer));
   } else if (setting.type === 'string' && isSecretConfigKey(setting.key)) {
     setSecretBackedSettingValue({

package/src/input/settings-modal-connections.ts

@@ -1,5 +1,5 @@
 /**
- * settings-modal-connections.ts — the Connections category of the settings
+ * settings-modal-connections.ts, the Connections category of the settings
  * workspace: whether mail and calendar are actually usable, and if not, the
  * exact next step.
  *
@@ -14,7 +14,7 @@
  * Every other settings category reads a value out of config and renders it.
  * This one cannot: "is mail connected?" is only answerable by asking the
  * daemon's handler, which is real I/O. The workspace renders synchronously, so
- * the rows start in `checking` — which is true — and the refresh replaces them
+ * the rows start in `checking`, which is true, and the refresh replaces them
  * and asks for a re-render, the same contract the Services surface uses for its
  * inspections. Nothing is ever shown as "not configured" merely because an
  * answer has not arrived yet; that would be a guess rendered as a fact.

package/src/input/settings-modal-data.ts

@@ -1,5 +1,5 @@
 /**
- * settings-modal-data — pure data-assembly helpers for SettingsModal.
+ * settings-modal-data, pure data-assembly helpers for SettingsModal.
  *
  * All functions are stateless: they take dependencies as arguments and return
  * derived data without mutating state. The class in settings-modal.ts delegates
@@ -55,7 +55,7 @@ import {
 } from './payments-config.ts';
 
 // ---------------------------------------------------------------------------
-// deepEqual — structural equality for isDefault comparisons
+// deepEqual, structural equality for isDefault comparisons
 // ---------------------------------------------------------------------------
 
 /**
@@ -89,7 +89,7 @@ export function deepEqual(a: unknown, b: unknown): boolean {
 }
 
 // ---------------------------------------------------------------------------
-// buildSettingGroups — loads CONFIG_SCHEMA into per-category SettingEntry maps
+// buildSettingGroups, loads CONFIG_SCHEMA into per-category SettingEntry maps
 // ---------------------------------------------------------------------------
 
 export function buildSettingGroups(
@@ -127,13 +127,13 @@ export function buildSettingGroups(
     // pushCompletion), so the contact address and the subscription
     // housekeeping bounds belong in that same category rather than in one of
     // their own. Without this they match no category at all and would be
-    // dropped from the workspace entirely — reachable only by editing the
+    // dropped from the workspace entirely, reachable only by editing the
     // config file by hand.
     if (rawCat === 'push' && groups.has('notifications')) groups.get('notifications')!.push(entry);
     // cluster.* decides which node on this network reads the inbox, and it is
     // configured in the same terms the network category already speaks: a
     // group address, a port, a shared phrase, a peer list. Given no category of
-    // its own it would match nothing and drop out of the workspace entirely —
+    // its own it would match nothing and drop out of the workspace entirely,
     // reachable only by hand-editing a settings file.
     if (rawCat === 'cluster' && groups.has('network')) groups.get('network')!.push(entry);
   }
@@ -183,7 +183,7 @@ export function buildSettingGroups(
         behaviorEntries.push(entry);
       }
     }
-    // In-terminal (OSC 9) notification toggles + audible bell — same category,
+    // In-terminal (OSC 9) notification toggles + audible bell, same category,
     // per-key defaults (see terminal-notifier.ts).
     for (const entry of buildTerminalNotifySyntheticEntries(configManager)) {
       if (!behaviorEntries.some((e) => e.setting.key === entry.setting.key)) {
@@ -194,7 +194,7 @@ export function buildSettingGroups(
 
   // Inject the storage.codeIndexEnabled toggle into the
   // storage category. TUI-local synthetic setting (not in the SDK ConfigKey
-  // union — see code-index-services.ts), same rationale as
+  // union, see code-index-services.ts), same rationale as
   // notifyAfterSeconds above: opt-in, default off, states its own bounds.
   const storageEntries = groups.get('storage');
   if (storageEntries && !storageEntries.some((e) => e.setting.key === (CODE_INDEX_ENABLED_CONFIG_KEY as ConfigKey))) {
@@ -202,13 +202,13 @@ export function buildSettingGroups(
   }
 
   // Inject the memory.projection.dir synthetic entry into the storage
-  // category (alongside codeIndexEnabled — both are local-storage posture
+  // category (alongside codeIndexEnabled, both are local-storage posture
   // settings). Not in the SDK ConfigKey union, same rationale as above.
   if (storageEntries && !storageEntries.some((e) => e.setting.key === MEMORY_PROJECTION_DIR_CONFIG_KEY)) {
     storageEntries.push(buildMemoryProjectionDirSyntheticEntry(configManager));
   }
 
-  // Synthetic memory.showProvenance (TUI-local, default OFF — see memory-provenance.ts).
+  // Synthetic memory.showProvenance (TUI-local, default OFF, see memory-provenance.ts).
   const memoryEntries = groups.get('memory');
   if (memoryEntries && !memoryEntries.some((e) => e.setting.key === (MEMORY_SHOW_PROVENANCE_CONFIG_KEY as ConfigKey))) {
     memoryEntries.push(buildMemoryProvenanceSyntheticEntry(configManager) as unknown as SettingEntry);
@@ -216,7 +216,7 @@ export function buildSettingGroups(
 
   // Inject the worktree.setup.commands / worktree.setup.carryOverGlobs
   // synthetic entries into the orchestration category (worktree isolation is
-  // an orchestration-engine concern — see WorkstreamIsolation 'worktree').
+  // an orchestration-engine concern, see WorkstreamIsolation 'worktree').
   // Neither key is in the SDK ConfigKey union (see worktree-setup-config.ts),
   // same rationale as the other synthetic settings above.
   const orchestrationEntries = groups.get('orchestration');
@@ -230,7 +230,7 @@ export function buildSettingGroups(
   }
 
   // Inject the sandbox.egressAllowlist / sandbox.workspaceWritable and
-  // permissions.execEnvScrubAllowlist synthetic entries — see
+  // permissions.execEnvScrubAllowlist synthetic entries, see
   // sandbox-exec-config.ts / exec-env-scrub-config.ts for why they are not in
   // CONFIG_SCHEMA, same rationale as the other synthetic settings above.
   injectSandboxExecSyntheticEntries(groups, configManager);
@@ -242,7 +242,7 @@ export function buildSettingGroups(
   // windows, notifyChannels). What's injected here are the four secret-tier
   // card-material fields and the two ordinary address fields, which are
   // TUI-local synthetic sub-keys under that real section (not yet SDK
-  // CONFIG_SCHEMA entries — see payments-config.ts's header comment), backed
+  // CONFIG_SCHEMA entries, see payments-config.ts's header comment), backed
   // by the same real configManager as everything else in this group.
   const paymentsEntries = groups.get('payments');
   if (paymentsEntries && !paymentsEntries.some((e) => isPaymentsSyntheticConfigKey(e.setting.key))) {
@@ -254,7 +254,7 @@ export function buildSettingGroups(
   enrichRelaySettingDescriptions(groups);
 
   // Daemon-owned keys (e.g. surfaces.telegram.*) live only in the daemon's
-  // own config store — say so in the detail text (daemon-owned-settings-descriptions.ts).
+  // own config store, say so in the detail text (daemon-owned-settings-descriptions.ts).
   enrichDaemonOwnedSettingDescriptions(groups, configManager);
 
   // Feature-unit presentation: each capability becomes ONE unit (its real
@@ -322,7 +322,7 @@ export function buildNotifyAfterSecondsSyntheticEntry(configManager: Pick<Config
 /**
  * The synthetic ConfigSetting descriptor for behavior.budgetAlertUsd.
  *
- * This key is TUI-local and is not yet in the SDK ConfigKey union — it never
+ * This key is TUI-local and is not yet in the SDK ConfigKey union, it never
  * appeared in CONFIG_SCHEMA, so it was invisible to every schema-driven
  * inspection surface (/config, /settings-sync show <key>) even though the
  * value round-trips correctly through configManager.get/set. The descriptor
@@ -357,7 +357,7 @@ export function buildBudgetAlertUsdSyntheticEntry(configManager: Pick<ConfigMana
 }
 
 // ---------------------------------------------------------------------------
-// alert-class synthetic settings — behavior.notifyOn* + notifyOnlyWhenUnfocused
+// alert-class synthetic settings, behavior.notifyOn* + notifyOnlyWhenUnfocused
 // ---------------------------------------------------------------------------
 
 /**
@@ -365,7 +365,7 @@ export function buildBudgetAlertUsdSyntheticEntry(configManager: Pick<ConfigMana
  * ConfigKey union), all defaulting to on. Read/written generically by
  * core/alert-gating.ts (readBooleanConfig) and the per-alert-class modules
  * (budget-breach-notifier.ts, approval-alert.ts, turn-event-wiring.ts,
- * long-task-notifier.ts) — this is only the settings-modal-visible surface.
+ * long-task-notifier.ts), this is only the settings-modal-visible surface.
  */
 const NOTIFY_ALERT_SYNTHETIC_SETTINGS: ReadonlyArray<{ readonly key: string; readonly description: string }> = [
   {
@@ -411,7 +411,7 @@ export function buildNotifyAlertSyntheticEntries(configManager: Pick<ConfigManag
 }
 
 // ---------------------------------------------------------------------------
-// In-terminal (OSC 9) notification settings — see core/terminal-notifier.ts
+// In-terminal (OSC 9) notification settings, see core/terminal-notifier.ts
 // ---------------------------------------------------------------------------
 
 /**
@@ -421,7 +421,7 @@ export function buildNotifyAlertSyntheticEntries(configManager: Pick<ConfigManag
  * on, turn-end/agent-blocked off, bell off. Read generically by
  * core/terminal-notifier.ts (readBooleanConfig); this is only the
  * settings-modal-visible surface. These are independent of the master
- * `notifyOnlyWhenUnfocused` gate — an in-terminal notification is always
+ * `notifyOnlyWhenUnfocused` gate, an in-terminal notification is always
  * focus-gated on its own (only fires for an unfocused/unknown terminal).
  */
 const TERMINAL_NOTIFY_SYNTHETIC_SETTINGS: ReadonlyArray<{ readonly key: string; readonly description: string; readonly defaultOn: boolean }> = [
@@ -460,7 +460,7 @@ export function buildTerminalNotifySyntheticEntries(configManager: Pick<ConfigMa
  * The repo source-tree code index's auto-build-on-startup toggle
  * (code-index-services.ts). TUI-local, default OFF: /codebase build is the
  * explicit trigger unless a user opts in here. Honest bounds stated inline
- * so enabling this isn't a surprise — see code-index-services.ts's
+ * so enabling this isn't a surprise, see code-index-services.ts's
  * CODE_INDEX_MAX_FILES/CODE_INDEX_MAX_FILE_BYTES for the numbers this
  * description would otherwise duplicate as magic numbers.
  */
@@ -468,13 +468,13 @@ export function buildCodeIndexEnabledSyntheticEntry(configManager: Pick<ConfigMa
   return buildBooleanSyntheticEntry(
     configManager,
     CODE_INDEX_ENABLED_CONFIG_KEY,
-    'Auto-build the repo source-tree code index on startup (bounded file/size scan; see /codebase status for bounds). Off by default — /codebase build indexes on demand.',
+    'Auto-build the repo source-tree code index on startup (bounded file/size scan; see /codebase status for bounds). Off by default: /codebase build indexes on demand.',
     false,
   );
 }
 
 // ---------------------------------------------------------------------------
-// buildFlagEntries — snapshot of current feature flag states
+// buildFlagEntries, snapshot of current feature flag states
 // ---------------------------------------------------------------------------
 
 export function buildFlagEntries(featureFlagManager: FeatureFlagManager | null): FlagEntry[] {
@@ -489,7 +489,7 @@ export function buildFlagEntries(featureFlagManager: FeatureFlagManager | null):
 }
 
 // ---------------------------------------------------------------------------
-// buildMcpEntries — snapshot of current MCP server security entries
+// buildMcpEntries, snapshot of current MCP server security entries
 // ---------------------------------------------------------------------------
 
 export function buildMcpEntries(mcpRegistry: McpRegistry | null): McpEntry[] {
@@ -505,13 +505,13 @@ export function buildMcpEntries(mcpRegistry: McpRegistry | null): McpEntry[] {
 }
 
 // ---------------------------------------------------------------------------
-// buildSubscriptionEntries — re-export for use by SettingsModal
+// buildSubscriptionEntries, re-export for use by SettingsModal
 // ---------------------------------------------------------------------------
 
 export { buildSubscriptionEntries };
 
 // ---------------------------------------------------------------------------
-// buildNetworkFilteredItems — applies host-mode visibility rules for 'network' tab
+// buildNetworkFilteredItems, applies host-mode visibility rules for 'network' tab
 // ---------------------------------------------------------------------------
 
 export function buildNetworkFilteredItems(
@@ -533,7 +533,7 @@ export function buildNetworkFilteredItems(
 }
 
 // ---------------------------------------------------------------------------
-// refreshEntryValues — re-reads currentValue/isDefault for all loaded entries
+// refreshEntryValues, re-reads currentValue/isDefault for all loaded entries
 // ---------------------------------------------------------------------------
 
 export function refreshEntryValues(
@@ -560,19 +560,19 @@ export function refreshEntryValues(
         entry.isDefault = (entry.currentValue as string[]).length === 0;
         continue;
       }
-      // 'memory' has no DEFAULT_CONFIG entry — its synthetic leaves read defensively below.
+      // 'memory' has no DEFAULT_CONFIG entry, its synthetic leaves read defensively below.
       if (entry.setting.key === MEMORY_PROJECTION_DIR_CONFIG_KEY) {
         const refreshed = buildMemoryProjectionDirSyntheticEntry(configManager);
         entry.currentValue = refreshed.currentValue; entry.isDefault = refreshed.isDefault;
         continue;
       }
-      // memory.showProvenance: same 'memory'-has-no-DEFAULT_CONFIG trap — read defensively.
+      // memory.showProvenance: same 'memory'-has-no-DEFAULT_CONFIG trap, read defensively.
       if (entry.setting.key === (MEMORY_SHOW_PROVENANCE_CONFIG_KEY as ConfigKey)) {
         const refreshed = buildMemoryProvenanceSyntheticEntry(configManager);
         entry.currentValue = refreshed.currentValue; entry.isDefault = refreshed.isDefault;
         continue;
       }
-      // payments.* synthetic sub-keys (card material + addresses — see
+      // payments.* synthetic sub-keys (card material + addresses, see
       // payments-config.ts): real configManager-backed now, but still
       // defensively normalized (undefined -> '') the same way the initial
       // build does, so isDefault stays consistent between build and refresh.
@@ -591,7 +591,7 @@ export function refreshEntryValues(
 }
 
 // ---------------------------------------------------------------------------
-// updateEntryForKey — updates a single setting entry after a value change
+// updateEntryForKey, updates a single setting entry after a value change
 // ---------------------------------------------------------------------------
 
 export function updateEntryForKey(

package/src/input/settings-modal-mutations.ts

@@ -1,8 +1,8 @@
 /**
- * settings-modal-mutations — pure mutation helpers for SettingsModal.
+ * settings-modal-mutations, pure mutation helpers for SettingsModal.
  *
  * These functions encapsulate the side-effectful write operations. Every
- * write — feature-unit headers included — is a plain domain-settings config
+ * write, feature-unit headers included, is a plain domain-settings config
  * write; the SDK's settings bridge keeps the runtime gate manager in sync
  * (flipping runtime-toggleable gates live and recording honest
  * pending-restart markers for startup-gated ones). Each function takes its
@@ -17,7 +17,7 @@ import type { FlagEntry, SettingEntry } from './settings-modal-types.ts';
 import { deepEqual } from './settings-modal-data.ts';
 
 // ---------------------------------------------------------------------------
-// ApplyValueResult — returned by applySettingValue so the caller can react
+// ApplyValueResult, returned by applySettingValue so the caller can react
 // ---------------------------------------------------------------------------
 
 export interface ApplyValueResult {
@@ -45,7 +45,7 @@ export type SettingAppliedCallback = (change: {
  * `surfaces.*`, `controlPlane.*`, `watchers.*`, `device.*`, `automation.*` and
  * the rest are executed by the daemon, unattended, with every surface closed.
  * Writing one into this surface's own settings file reports success and
- * configures nothing — the daemon reads a different file and never sees it.
+ * configures nothing, the daemon reads a different file and never sees it.
  * So a daemon-owned write goes over `config.set`, and a write with no reachable
  * daemon FAILS rather than landing somewhere harmless-looking.
  */
@@ -80,8 +80,8 @@ export function applySettingValue({
   // live under a config section CONFIG_SCHEMA/DEFAULT_CONFIG has never populated
   // (e.g. 'worktree' as of the SDK 1.6.1 repack), so configManager.get can throw
   // "Invalid config path" here even though the write attempt below is already
-  // guarded. Without this, that read — which runs unconditionally, before the
-  // write's own try/catch — would crash the whole settings modal on save instead
+  // guarded. Without this, that read, which runs unconditionally, before the
+  // write's own try/catch, would crash the whole settings modal on save instead
   // of surfacing the honest "Save failed" message the write path below produces.
   let previousValue: unknown;
   try {
@@ -101,7 +101,7 @@ export function applySettingValue({
       // manager is still written so this session's own reads and the rows
       // below reflect the edit immediately; the daemon's copy is what the
       // daemon acts on, and a refusal from it is surfaced rather than
-      // swallowed — the local value would then be the lie.
+      // swallowed, the local value would then be the lie.
       void daemonConfig.set(key, value).catch((error) => {
         logger.error('SettingsModal: the daemon refused a daemon-owned config write', { key, error: summarizeError(error) });
         onAsyncError?.(`That setting is applied by the daemon and saving it there failed: ${summarizeError(error)}`);