@pellux/goodvibes-tui 2.0.13 → 2.0.15

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

@@ -1,5 +1,5 @@
 /**
- * settings-modal-reset — pure reset helpers for SettingsModal.
+ * settings-modal-reset, pure reset helpers for SettingsModal.
  *
  * These functions encapsulate the three reset operations:
  *   - resetSelected: reset the currently selected setting to its schema default
@@ -9,7 +9,7 @@
  *
  * Each function takes its dependencies as explicit arguments rather than
  * accessing class-level state. resetCategoryConfirm and resetAllConfirm
- * remain public class fields on SettingsModal — the renderer reads them
+ * remain public class fields on SettingsModal, the renderer reads them
  * directly to decide the footer state.
  */
 
@@ -44,7 +44,7 @@ export function resetSelected({
     // Delete from the tier the write went to. A daemon-owned key was stored in
     // the daemon tier (defaultSecretBackedScope), so a delete narrowed to 'user'
     // would report a cleared setting while the daemon kept using the live
-    // credential — the reset that isn't one.
+    // credential, the reset that isn't one.
     void secretsManager.delete(buildGoodVibesSecretKey(key), { scope: defaultSecretBackedScope(key) }).catch((error) => {
       logger.error('SettingsModal: failed to clear secret while resetting setting', { key, error: summarizeError(error) });
     });

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

@@ -1,5 +1,5 @@
 /**
- * settings-modal-search — ranked search over the settings workspace.
+ * settings-modal-search, ranked search over the settings workspace.
  *
  * Split out of settings-modal-data.ts, which had reached its 800-line ceiling.
  * Scoring and search are a self-contained unit: they read a SettingEntry and a
@@ -11,7 +11,7 @@
 import type { SettingEntry, SettingsCategory } from './settings-modal-types.ts';
 
 // ---------------------------------------------------------------------------
-// fuzzyScoreSettingEntry — score an entry against a query for ranked search
+// fuzzyScoreSettingEntry, score an entry against a query for ranked search
 // ---------------------------------------------------------------------------
 
 /**
@@ -40,20 +40,20 @@ export function fuzzyScoreSettingEntry(
   const label = getLabel(entry).toLowerCase();
   const description = (entry.setting.description ?? '').toLowerCase();
 
-  // Tier 1: key substring — base 3000, position bonus up to 999
+  // Tier 1: key substring, base 3000, position bonus up to 999
   // A key match at position 0 scores 3999; at position 999 scores 3000.
   const keyIdx = key.indexOf(lq);
   if (keyIdx !== -1) return 3000 + Math.max(0, 999 - keyIdx);
 
-  // Tier 2: label substring — base 2000, position bonus up to 999
+  // Tier 2: label substring, base 2000, position bonus up to 999
   const labelIdx = label.indexOf(lq);
   if (labelIdx !== -1) return 2000 + Math.max(0, 999 - labelIdx);
 
-  // Tier 3: description substring — base 1000, position bonus up to 999
+  // Tier 3: description substring, base 1000, position bonus up to 999
   const descIdx = description.indexOf(lq);
   if (descIdx !== -1) return 1000 + Math.max(0, 999 - descIdx);
 
-  // Tier 4: subsequence across concatenated key + label + description — 1..99
+  // Tier 4: subsequence across concatenated key + label + description, 1..99
   const haystack = `${key} ${label} ${description}`;
   let qi = 0;
   let score = 0;

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

@@ -14,8 +14,8 @@ export type SettingsSecretsManager = Pick<SecretsManager, 'delete' | 'set'>;
 /**
  * The daemon's own credential write, when this app has a daemon to write to.
  *
- * A daemon-scoped credential is two writes that only work together — the secret
- * value and the config reference that points at it — and `credentials.set` does
+ * A daemon-scoped credential is two writes that only work together, the secret
+ * value and the config reference that points at it, and `credentials.set` does
  * both, verifying the value reads back before it touches the config. Splitting
  * them across a process boundary leaves a window where the config names a
  * reference resolving to nothing, which every reader treats as a
@@ -61,7 +61,7 @@ export function setSecretBackedSettingValue(args: {
   const update = buildSecretBackedConfigUpdate(key, value);
   // A daemon-owned key (surfaces.*, payments.*, controlPlane.*, ...) names a
   // credential the daemon itself executes with, so its secret material lands in
-  // the daemon tier — the one the daemon reads with every surface closed —
+  // the daemon tier, the one the daemon reads with every surface closed,
   // regardless of which client edited it. Everything else stays at user scope.
   // See secret-config.ts's defaultSecretBackedScope.
   const scope = defaultSecretBackedScope(key);

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

@@ -52,40 +52,40 @@ export type SettingsCategory =
   | 'update'
   | 'power'
   | 'memory'
-  // payments.* — the card on file for daemon-initiated purchases. Like
+  // payments.*, the card on file for daemon-initiated purchases. Like
   // 'profile' below, this entry and its SETTINGS_CATEGORY_GROUPS membership are
   // both mandatory or the whole prefix is dropped from the workspace.
   | 'payments'
-  // profile.* — the owner profile (docs/owner-profile.md §12.1). This entry and
+  // profile.*, the owner profile (docs/owner-profile.md §12.1). This entry and
   // its SETTINGS_CATEGORY_GROUPS membership below are both mandatory:
   // buildSettingGroups guards every push with `if (groups.has(cat))`, so a
   // config prefix with no category here is dropped from the workspace entirely
-  // and reachable only by hand-editing a settings file — see the push.* and
+  // and reachable only by hand-editing a settings file, see the push.* and
   // cluster.* comments in settings-modal-data.ts for the two times that happened.
   | 'profile'
-  // occasions.* — the proactive occasions/plans loop (docs/occasions.md §8):
+  // occasions.*, the proactive occasions/plans loop (docs/occasions.md §8):
   // lead time, active hours, nudge channel and cadence, away-date adjustment,
   // calendar mirroring, interview length, gift-history retention, and the
   // sweep interval. Same mandatory-dual-membership rule as profile above, or
   // all twelve keys are dropped from the workspace and reachable only by
-  // hand-editing a settings file — a third occurrence of the push.*/cluster.*
+  // hand-editing a settings file, a third occurrence of the push.*/cluster.*
   // class the profile comment already names two of.
   | 'occasions'
-  // hostedSessions.* — sessions whose conversation loop runs inside the daemon
+  // hostedSessions.*, sessions whose conversation loop runs inside the daemon
   // instead of this terminal: what detaching does to one (the owner-confirmed
   // default is `kill`, the behavior closing a client has always had), how many
   // may run at once, how much of a conversation is kept, and how long a
   // terminated one's record is retained. Same mandatory-dual-membership rule as
-  // profile and occasions above — without BOTH this entry and the
+  // profile and occasions above, without BOTH this entry and the
   // SETTINGS_CATEGORY_GROUPS membership below, buildSettingGroups drops every
   // hostedSessions.* key from the workspace, which is exactly what happened to
   // push.* and cluster.* before them.
   | 'hostedSessions'
-  // email.* / calendar.* / google.* — the connector keys the daemon really
+  // email.* / calendar.* / google.*, the connector keys the daemon really
   // reads for its Google-backed mail and calendar services (platform runtime
   // 2.0.8 registered them as described schema rows; before that they were
   // cast onto the live config invisibly and answered "Unknown setting").
-  // Same mandatory-dual-membership rule as profile and hostedSessions above —
+  // Same mandatory-dual-membership rule as profile and hostedSessions above,
   // without BOTH these entries and the SETTINGS_CATEGORY_GROUPS membership
   // below, every one of the 22 connector keys is dropped from the workspace.
   | 'email'
@@ -103,22 +103,22 @@ export const SETTINGS_CATEGORY_GROUPS: ReadonlyArray<{
   { label: 'AI Routing', categories: ['provider', 'pricing', 'subscriptions', 'helper', 'tools', 'tts', 'voice'] },
   { label: 'Service & Network', categories: ['service', 'daemon', 'network', 'controlPlane', 'httpListener', 'web', 'relay'] },
   // 'hostedSessions' sits beside 'surfaces' and 'conversationGate': all three
-  // answer "where does this conversation actually run, and who can reach it" —
+  // answer "where does this conversation actually run, and who can reach it",
   // the detach policy is a fact about the daemon holding a session open, not
   // about this terminal's presentation of it.
   // 'email', 'calendar' and 'google' sit beside 'surfaces': they configure
-  // the daemon's connections to the owner's mail and calendar — where those
+  // the daemon's connections to the owner's mail and calendar, where those
   // services reach the world, not how this terminal presents anything.
   { label: 'Surfaces & Cloud', categories: ['surfaces', 'hostedSessions', 'device', 'conversationGate', 'integrations', 'connections', 'email', 'calendar', 'google', 'mcp', 'cloudflare'] },
   // 'occasions' sits beside 'checkin': both are proactive background loops
   // (a cadence, a judgment pass, a conditional channel delivery) rather than
-  // facts held about the owner — the data occasions/plans declare lives in
+  // facts held about the owner, the data occasions/plans declare lives in
   // the owner profile file itself (docs/occasions.md §3), but these SETTINGS
   // rows tune the loop, exactly like checkin's.
   { label: 'Automation', categories: ['batch', 'automation', 'checkin', 'occasions', 'watchers', 'orchestration', 'planner', 'wrfc', 'payments'] },
   // 'profile' sits beside 'memory' and 'learning': all three are what the
   // platform retains about the person using it, and this is the group a reader
-  // looking for "what does it know about me" already scans — Interface is
+  // looking for "what does it know about me" already scans, Interface is
   // presentation and permission posture, and nothing there is a store of facts.
   { label: 'Runtime & Data', categories: ['storage', 'atRest', 'sandbox', 'fetch', 'agents', 'runtime', 'power', 'profile', 'memory', 'cache', 'telemetry', 'diagnostics', 'learning'] },
   { label: 'Advanced', categories: ['release', 'update', 'danger'] },
@@ -138,14 +138,14 @@ export interface SettingEntry {
   /**
    * Present when this row is a feature-unit header (a platform capability
    * rendered as one unit with the settings that tune it beneath). The row IS
-   * the real config row for the feature's enablement key — boolean headers
+   * the real config row for the feature's enablement key, boolean headers
    * toggle and enum headers cycle their mode choices through the ordinary
    * settings write path (a plain config write; the settings bridge keeps the
    * runtime gate in sync).
    */
   flag?: FlagEntry;
   /**
-   * Present when this row is a settings sub-option OWNED by a feature unit —
+   * Present when this row is a settings sub-option OWNED by a feature unit,
    * the id of the owning feature. Drives the indented rendering beneath its
    * header and marks the row as claimed so it does not double-list as an
    * orphan.
@@ -162,7 +162,7 @@ export interface FlagEntry {
   state: FlagState;
   /**
    * Last known config-layer (persisted) value. Equals `state` unless a
-   * startup-gated feature was changed this session — then it holds the value
+   * startup-gated feature was changed this session, then it holds the value
    * that will take effect on the next launch.
    */
   persistedState: FlagState;

package/src/input/settings-modal.ts

@@ -1,5 +1,5 @@
 /**
- * SettingsModal — state management for the /settings and /config fullscreen workspace.
+ * SettingsModal, state management for the /settings and /config fullscreen workspace.
  *
  * Loads CONFIG_SCHEMA, groups settings by category, and tracks UI state:
  *   - Active category (Tab to cycle)
@@ -88,7 +88,7 @@ export interface SettingsModalOpenOptions {
   readonly onSettingApplied?: SettingsModalChangeHandler;
   /**
    * The daemon's credential write. A daemon-scoped credential (a mailbox
-   * password, a bot token, a card) must land where the daemon reads it — this
+   * password, a bot token, a card) must land where the daemon reads it, this
    * surface storing a copy is a credential in a place nothing consults.
    */
   readonly daemonCredentials?: SettingsDaemonCredentialWriter | null;
@@ -141,7 +141,7 @@ export class SettingsModal {
   /**
    * Scroll offset (in wrapped lines) of the documentation pane for the
    * selected row. The renderer clamps it to the pane's real content height
-   * and shows honest more-above/below markers — long feature documentation
+   * and shows honest more-above/below markers, long feature documentation
    * scrolls, it never clips. Reset whenever the selection changes.
    */
   public contextScroll = 0;
@@ -721,7 +721,7 @@ export class SettingsModal {
     // No-op (result.changed === false, effectMessage === null): leave lastSettingEffectMessage untouched.
 
     // Selecting 'prompt' for payments.cvvHandling states the tradeoff at the
-    // moment of selection, not just in the documentation pane — see the
+    // moment of selection, not just in the documentation pane, see the
     // SDK's own CVV_PROMPT_TRADEOFF_WARNING (platform/payments).
     if (key === PAYMENTS_CVV_HANDLING_CONFIG_KEY && value === 'prompt') {
       this.lastSettingEffectMessage = CVV_PROMPT_TRADEOFF_WARNING;

package/src/input/voice-local-settings.ts

@@ -1,11 +1,11 @@
 // ---------------------------------------------------------------------------
-// voice-local-settings.ts — the managed local-voice setup affordance the voice
+// voice-local-settings.ts, the managed local-voice setup affordance the voice
 // settings surfaces render beside ElevenLabs.
 //
 // The local engine registers in the voice provider registry regardless of
 // whether it is provisioned yet (its status is honestly "unconfigured", never
 // an error). When it is NOT provisioned, the settings/provider-picker surface
-// offers the one-act install with its download size declared up front — the
+// offers the one-act install with its download size declared up front, the
 // size is computed synchronously from the pinned piper manifest
 // (piperProvisionBytes) so no daemon round-trip is needed just to label the
 // offer. Platforms with no pinned managed build say so honestly.
@@ -45,7 +45,7 @@ export function localVoiceSetupOffer(configGet: (key: string) => unknown): Local
   let detail: string;
   if (!supported) detail = 'local voice: no managed build for this platform (set voice.local.* by hand to use it)';
   else if (provisioned) detail = 'local voice: configured (managed engine set)';
-  else detail = `local voice: not set up — run /voice setup (${sizeLabel})`;
+  else detail = `local voice: not set up; run /voice setup (${sizeLabel})`;
 
   const actions = supported && !provisioned ? '[Enter] select · /voice setup to provision' : '[Enter] set provider';
   return { supported, provisioned, sizeLabel, detail, actions };

package/src/input/worktree-setup-config.ts

@@ -1,5 +1,5 @@
 /**
- * worktree-setup-config.ts — TUI-local synthetic settings entries for
+ * worktree-setup-config.ts, TUI-local synthetic settings entries for
  * worktree.setup.commands / worktree.setup.carryOverGlobs.
  *
  * Neither key is in the SDK's ConfigKey union: the SDK's cold-start worktree
@@ -12,8 +12,8 @@
  * settings-modal-data.ts (behavior.notifyAfterSeconds, storage.codeIndexEnabled, etc).
  *
  * Both keys hold a JSON array of strings on disk. The settings modal's
- * inline editor is a single-line text field, so — matching the SDK's own
- * convention for controlPlane.cors.allowedOrigins — they are displayed and
+ * inline editor is a single-line text field, so, matching the SDK's own
+ * convention for controlPlane.cors.allowedOrigins, they are displayed and
  * edited as a comma-separated list and parsed back into an array on commit
  * (see isWorktreeSetupListConfigKey's use in settings-modal.ts#commitEdit).
  */
@@ -34,14 +34,14 @@ export function isWorktreeSetupListConfigKey(key: ConfigKey): boolean {
 }
 
 /**
- * Read a config value as a string array — mirrors the SDK's own tolerant
+ * Read a config value as a string array, mirrors the SDK's own tolerant
  * readStringArray (non-array/malformed degrades to empty).
  *
  * Defensive try/catch (now belt-and-suspenders): the SDK registers the
  * `worktree` section in DEFAULT_CONFIG (setup.commands / setup.carryOverGlobs,
  * both empty lists) as of this repack, so ConfigManager.get no longer throws
- * for these keys on a fresh store. The guard is retained so an older SDK — or
- * a config store whose section was pruned — degrades to an empty list here
+ * for these keys on a fresh store. The guard is retained so an older SDK, or
+ * a config store whose section was pruned, degrades to an empty list here
  * (this read runs on every settings-modal build) rather than crashing /config
  * or /settings.
  */
@@ -65,14 +65,14 @@ export const WORKTREE_SETUP_COMMANDS_SYNTHETIC_SETTING: ConfigSetting = {
   key: WORKTREE_SETUP_COMMANDS_CONFIG_KEY,
   type: 'string',
   default: [],
-  description: 'Comma-separated shell command lines run in order in a freshly-created isolated worktree before it is handed to an agent (e.g. "bun install, bun run codegen"). A command that exits non-zero stops setup there and marks the worktree failed — visible on /worktree review and the worktree record, never silent. Empty = no setup commands run.',
+  description: 'Comma-separated shell command lines run in order in a freshly-created isolated worktree before it is handed to an agent (e.g. "bun install, bun run codegen"). A command that exits non-zero stops setup there and marks the worktree failed; visible on /worktree review and the worktree record, never silent. Empty = no setup commands run.',
 };
 
 export const WORKTREE_SETUP_CARRY_OVER_GLOBS_SYNTHETIC_SETTING: ConfigSetting = {
   key: WORKTREE_SETUP_CARRY_OVER_GLOBS_CONFIG_KEY,
   type: 'string',
   default: [],
-  description: 'Comma-separated glob patterns (e.g. ".env, .env.*, config/local.json") of UNTRACKED files copied from the source working tree into a freshly-created isolated worktree, after setup commands run. Only files git reports as untracked are eligible — committed files already carry over through the worktree checkout itself. Empty = nothing carried over.',
+  description: 'Comma-separated glob patterns (e.g. ".env, .env.*, config/local.json") of UNTRACKED files copied from the source working tree into a freshly-created isolated worktree, after setup commands run. Only files git reports as untracked are eligible; committed files already carry over through the worktree checkout itself. Empty = nothing carried over.',
 };
 
 function buildWorktreeSetupListEntry(configManager: Pick<ConfigManager, 'get'>, setting: ConfigSetting): SettingEntry {

package/src/main.ts

@@ -90,7 +90,7 @@ async function main() {
   const stdin = process.stdin;
   // Both roots come from the one resolver the daemon CLI also uses. This line
   // called homedir() unconditionally, so a harness that redirected the tree got
-  // a client that wrote the real one — secret store included.
+  // a client that wrote the real one, secret store included.
   const { cli, configManager, bootstrapWorkingDir, bootstrapHomeDirectory } = await prepareShellCliRuntime(process.argv.slice(2), {
     defaultWorkingDirectory: process.env['GOODVIBES_WORKING_DIR'] ?? process.cwd(),
     homeDirectory: resolveGoodVibesHome(),
@@ -140,7 +140,7 @@ async function main() {
   const { approvalBroker, agentManager, modeManager, processManager, providerRegistry, secretsManager, subscriptionManager } = ctx.services;
   conversation.setSessionMemoryStore(ctx.services.sessionMemoryStore);
   conversation.setSessionLineageTracker(ctx.services.sessionLineageTracker);
-  // Shared payload (single source of truth, includes the memoryRegistry —
+  // Shared payload (single source of truth, includes the memoryRegistry,
   // see orchestrator-core-services.ts) plus this site's favoritesStore.
   orchestrator.setCoreServices({
     ...buildSharedOrchestratorCoreServices({ services: ctx.services, configManager, providerRegistry }),
@@ -154,7 +154,7 @@ async function main() {
   applyTerminalRuntimeConfigDefaults(configManager);
 
   // Re-surface pre-TUI launch-update lines in-session (the alt screen wipes
-  // stdout). Launch-time update mechanics are routine, not urgent — low
+  // stdout). Launch-time update mechanics are routine, not urgent, low
   // priority, not high (a skipped/deferred check is not the kind of thing
   // that should compete with real session alerts for attention).
   for (const line of launchUpdateLines) systemMessageRouter.low(`[Update] ${line}`);
@@ -186,7 +186,7 @@ async function main() {
   let lastMaxScroll: number | null = null;
   // Stream and tool-timer state; mutated by wireStreamEventMetrics handlers, read during render.
   const streamMetrics: StreamMetrics = createStreamMetrics();
-  // Live failover record — written by the failover path, read every frame so the header and footer agree.
+  // Live failover record, written by the failover path, read every frame so the header and footer agree.
   const failoverState = createFailoverTurnState();
 
   const getPromptContentWidth = () => computePromptContentWidth(stdout.columns);
@@ -222,7 +222,7 @@ async function main() {
   const unsubs: Array<() => void> = [];
   let recoveryInterval: ReturnType<typeof setInterval> | null = null;
   let stopSpokenOutputForExit: (() => Promise<void>) | null = null;
-  // The optional "used N memories" provenance chip (default OFF) — see interaction-seams.ts.
+  // The optional "used N memories" provenance chip (default OFF), see interaction-seams.ts.
   const memoryProvenanceUi = createMemoryProvenanceUi({ render: () => render(), memorySpine: ctx.services.memorySpine });
   // Topology-aware keep-awake: the chip renders the DAEMON's state in adopted-external mode (power-chip-source.ts) and every toggle is forwarded to that daemon (power-keepawake-remote.ts) so keep-awake survives the TUI closing in BOTH topologies.
   const isExternalDaemon = () => uiServices.platform.externalServices?.inspect()?.daemonStatus?.mode === 'external';
@@ -426,10 +426,10 @@ async function main() {
   commandContext.openComposerEditor = makeComposerEditorOpener({ buffer: input, stdin, stdout, writeGuard: allowTerminalWrite, repaint: () => { compositor.resetDiff(); render(); }, cwd: workingDir, env: process.env, notify: (m) => systemMessageRouter.high(m) });
   input.setConversationManager(conversation);
   input.setContentWidth(getPromptContentWidth()); input.filePicker.setOnUpdate(() => render());
-  // retirement: agentDetailModal/processModal setOnRefresh wiring removed —
+  // retirement: agentDetailModal/processModal setOnRefresh wiring removed,
   // those modals were deleted (Fleet subsumes the live process tree via F2).
 
-  // Model picker callback is handled in bootstrap.ts — do not duplicate here.
+  // Model picker callback is handled in bootstrap.ts, do not duplicate here.
   input.setHistory(inputHistory);
   // ONE microphone path, shared by push-to-talk voice input (Alt+V) and wake-word detection; opens no device by itself (shell/voice-capture-shell.ts).
   voiceCaptureStatus = installVoiceCapture({ configManager, shellPaths: ctx.services.shellPaths, homeDirectory, sessionId: ctx.runtime.sessionId, commandContext, unsubs, buffer: input, submitInput, notify: (m) => { systemMessageRouter.high(m); render(); }, render: () => render() });
@@ -504,7 +504,7 @@ async function main() {
       // Cross-surface spine posture segment (adopted-daemon mode only).
       sessionSpineStatus: (() => { const s = uiServices.platform.externalServices?.inspect(); return s?.sessionSpineActive && s.sessionSpineStatus && s.sessionSpineStatus !== 'unknown' ? s.sessionSpineStatus : undefined; })(), runningAgentCount, runningProcessCount,
       webSurfaceUrl: configManager.get('web.enabled') ? resolveWebSurfaceUrl(configManager) : undefined,
-      // Always-visible "sleep disabled" chip — topology-aware: the DAEMON's state in adopted-external mode, the in-process manager otherwise (power-chip-source.ts).
+      // Always-visible "sleep disabled" chip, topology-aware: the DAEMON's state in adopted-external mode, the in-process manager otherwise (power-chip-source.ts).
       powerKeepAwake: powerChipSource.get().keepAwake,
       // Composer must not read as focused while the panel/process indicator owns keyboard focus.
       promptFocused: !input.panelFocused && !input.indicatorFocused,
@@ -579,7 +579,7 @@ async function main() {
       const partialToolPreview = showPreview ? sessionSnapshot.streamToolPreview : undefined;
       // Elapsed from turn start (stream or tool execution), used for the thinking indicator timer.
       const turnElapsedMs = streamMetrics.startTime > 0 ? Date.now() - streamMetrics.startTime : undefined;
-      // Suppressed while a tool executes — its ticking timer is the honest indicator then.
+      // Suppressed while a tool executes, its ticking timer is the honest indicator then.
       const stallInfo = UIFactory.computeRenderStallInfo(streamMetrics, Date.now());
       const thinking = UIFactory.createThinkingFragment(
         conversationWidth,
@@ -698,19 +698,19 @@ async function main() {
 
   unsubs.push(uiServices.events.turns.on('TURN_COMPLETED', (evt) => memoryProvenanceUi.onTurnCompleted(evt)));
 
-  // Stable turn context for failover retry — set in submitInput, read by retryTurn.
+  // Stable turn context for failover retry, set in submitInput, read by retryTurn.
   let retryCtx: { count: number; text: string; content?: ContentPart[]; opts?: Parameters<typeof orchestrator.handleUserInput>[2] } | null = null;
   // One-key retry affordance, active right after a user-visible TURN_ERROR: 'r' re-submits on the
   // current provider, 'm' opens the model picker, any other character clears it and routes normally.
   // Surfaced as a transient FOOTER hint (see retryAffordanceHint below), not a transcript message.
   // Time-bounded: onExpire repaints once the 60s disarm timer fires, so a stray keypress hours
-  // later can never trigger a real retry — see retry-affordance.ts.
+  // later can never trigger a real retry, see retry-affordance.ts.
   const retryAffordance = createRetryAffordanceState({ onExpire: render });
   const retryTurn = (notice?: string): boolean => {
     if (!retryCtx) return false; // nothing to roll back to; the caller narrates instead
     const { count, text, content: rContent, opts: rOpts } = retryCtx;
-    // Roll back to pre-submission count, then re-submit. SDK gap — no retry-in-place (see handoff).
-    // The rollback erases the failed turn's transcript — the failover notice included, which is how
+    // Roll back to pre-submission count, then re-submit. SDK gap, no retry-in-place (see handoff).
+    // The rollback erases the failed turn's transcript, the failover notice included, which is how
     // that notice used to vanish before anyone could read it. The caller hands it over instead, and
     // it is posted here: after the rollback, above the prompt it explains.
     conversation.removeMessagesAfter(count);
@@ -775,7 +775,7 @@ async function main() {
   });
   process.on('SIGINT', sigintHandler); process.on('unhandledRejection', unhandledRejectionHandler); stdout.on('resize', resizeHandler);
 
-  // State restores happen ONLY when the user explicitly asks — a CLI flag
+  // State restores happen ONLY when the user explicitly asks, a CLI flag
   // (--continue/--resume/--fork), a slash command (/session resume,
   // /checkpoints, /rewind), or the startup recovery offer's modal. There is
   // deliberately no unconditional auto-restore here: a bare launch never
@@ -793,7 +793,7 @@ async function main() {
   }));
 
   // Auto-save to recovery file every 60s + multi-instance liveness-marker
-  // refresh — see runtime/recovery-autosave.ts.
+  // refresh, see runtime/recovery-autosave.ts.
   recoveryInterval = startRecoveryAutosave({ conversation, runtime, surface: ctx.services.surface, buildSessionContinuityHints });
 }
 

package/src/panels/agent-inspector-shared.ts

@@ -11,7 +11,7 @@ export type AgentInspectorEntryKind = 'user' | 'assistant' | 'tool_call' | 'tool
 // AgentDetailModal, which was retired in).
 // ---------------------------------------------------------------------------
 
-/** Terminal statuses — cancel not offered; stall check skipped. */
+/** Terminal statuses, cancel not offered; stall check skipped. */
 export const AGENT_TERMINAL_STATUSES = new Set(['completed', 'failed', 'cancelled']);
 
 /** Agents in a non-terminal state for longer than this are considered STALLED. */
@@ -172,7 +172,7 @@ export function formatAgentCost(usd: number): string {
 export interface AgentUsageSummary {
   readonly tokens: number;
   readonly cost: number;
-  /** False when the model never resolved to a real price — `cost` is a zero placeholder, not a real reading. */
+  /** False when the model never resolved to a real price, `cost` is a zero placeholder, not a real reading. */
   readonly priced: boolean;
 }
 
@@ -190,12 +190,12 @@ interface AgentUsageLike {
  * True when a usage object carries actual reported token data rather than
  * being present-but-empty.: the SDK's AgentRecord.usage is initialised
  * to an all-zero object at agent spawn (platform/tools/agent/manager.ts) and
- * — in the currently pinned SDK version — is never updated past that, so a
+ *, in the currently pinned SDK version, is never updated past that, so a
  * plain truthiness check on `rec.usage` treats every agent, including ones
  * that did real work, as "has data" and renders a fabricated $0.00/0-token
  * reading. Guarding on real (nonzero) counts instead keeps today's "n/a"
  * honest and lights up automatically once a future SDK actually populates
- * usage — no further TUI change needed.
+ * usage, no further TUI change needed.
  */
 export function hasReportedUsage(
   usage: AgentUsageLike['usage'],
@@ -262,7 +262,7 @@ export function buildWrfcCostSegments(
 export interface AgentActivityHooks {
   /** Any AGENT_* activity event for this agent (running/progress/stream/awaiting/finalizing/spawning). */
   readonly onActivity: (agentId: string) => void;
-  /** AGENT_SPAWNING only — used for auto-select-newest-agent. */
+  /** AGENT_SPAWNING only, used for auto-select-newest-agent. */
   readonly onSpawn: (agentId: string) => void;
   /** AGENT_COMPLETED/FAILED/CANCELLED. */
   readonly onTerminal: (agentId: string) => void;

package/src/panels/base-panel.ts

@@ -12,7 +12,7 @@ const ERROR_FG = '#ef4444';
 /**
  * Frame requester wired by main.ts to the render scheduler. Without it,
  * markDirty() only sets a flag the compositor reads when a frame is ALREADY
- * being composed for another reason (input, streaming) — so a live panel
+ * being composed for another reason (input, streaming), so a live panel
  * (fleet ticks, registry subscriptions) sat visibly stale while the app was
  * idle until the next keypress (a replay finding). The scheduler
  * coalesces same-tick requests, so this stays cheap.
@@ -67,7 +67,7 @@ export abstract class BasePanel implements Panel {
    * Last error message to surface in the panel footer.
    * Auto-cleared on the next keystroke by `ScrollableListPanel.handleInput()` (and any
    * subclass that calls `super.handleInput()` or manually calls `this.clearError()` at
-   * the start of its handler). BasePanel itself does NOT auto-clear — only subclasses
+   * the start of its handler). BasePanel itself does NOT auto-clear, only subclasses
    * that opt into the contract do.
    */
   protected lastError: string | null = null;
@@ -227,7 +227,7 @@ export abstract class BasePanel implements Panel {
     return consumed;
   }
 
-  /** Mark this panel dirty — it will be re-rendered on the next compositor frame. */
+  /** Mark this panel dirty, it will be re-rendered on the next compositor frame. */
   public invalidate(): void { this.needsRender = true; }
 
   /** Called by the compositor after a successful render to clear the dirty flag. */

package/src/panels/builtin/agent.ts

@@ -3,7 +3,7 @@ import type { ResolvedBuiltinPanelDeps } from './shared.ts';
 
 // (the purge): thinking, tools, inspector, and wrfc were registered here
 // before the purge. thinking/tools were DELETE-disposition (no surviving
-// human surface — thinking already renders inline in the transcript, tool
+// human surface, thinking already renders inline in the transcript, tool
 // results render inline plus per-node in Fleet); inspector/wrfc were
 // RETIRE-INTO-FLEET (their live console is subsumed by the Fleet panel,
 // operations.ts:84). See registerOperationsPanels for the fleet
@@ -11,7 +11,7 @@ import type { ResolvedBuiltinPanelDeps } from './shared.ts';
 // the full disposition map.
 export function registerAgentPanels(manager: PanelManager, deps: ResolvedBuiltinPanelDeps): void {
   // Compat: '/panel open agent-logs' and 'inspector' (and any saved
-  // layout/muscle memory) still resolve — redirected straight to fleet.
+  // layout/muscle memory) still resolve, redirected straight to fleet.
   // Repointed from agent-logs->inspector now that inspector itself
   // is retired; alias resolution is a single hop (PanelManager._resolveId),
   // so this must point at the real surviving id, not at another alias.
@@ -19,7 +19,7 @@ export function registerAgentPanels(manager: PanelManager, deps: ResolvedBuiltin
   manager.registerAlias('inspector', 'fleet');
   manager.registerAlias('wrfc', 'fleet');
 
-  // (the purge) — group B: 'work-plan' and 'project-planning'
+  // (the purge), group B: 'work-plan' and 'project-planning'
   // migrated to the 'work-plan-modal' / 'planning-modal' config-modal surfaces.
   // Their surfaces AND panel→modal redirects are registered centrally in
   // registerBuiltinModals (builtin-modals.ts). Panel registrations retired here.

package/src/panels/builtin/development.ts

@@ -8,8 +8,8 @@ import { requireUiServices, withUnconfiguredFallback } from './shared.ts';
 
 // (the purge): plan, intelligence, explorer, preview, and symbols were
 // registered here before the purge. plan was RETIRE-INTO-FLEET (its
-// execution-plan view is subsumed by Fleet — operations.ts:84); the other
-// four were DELETE-disposition (no surviving human surface — intelligence's
+// execution-plan view is subsumed by Fleet, operations.ts:84); the other
+// four were DELETE-disposition (no surviving human surface, intelligence's
 // read-model still backs the surviving `/intelligence` CLI subcommands, see
 // intelligence-runtime.ts; explorer/preview/symbols had no dedicated
 // read-model to preserve). See .goodvibes/audit/2026-07-04-wave6-briefs.json
@@ -31,7 +31,7 @@ export function registerDevelopmentPanels(manager: PanelManager, deps: ResolvedB
   });
 
   // Compat: '/panel open plan' (and any saved layout/muscle memory) still
-  // resolves — redirected to fleet, which absorbs the execution-plan view.
+  // resolves, redirected to fleet, which absorbs the execution-plan view.
   manager.registerAlias('plan', 'fleet');
 
   manager.registerType({
@@ -57,7 +57,7 @@ export function registerDevelopmentPanels(manager: PanelManager, deps: ResolvedB
   });
 
   // 'inspector' registration moved to builtin/agent.ts (category
-  // 'agent') — it now absorbs the merged agent-logs capabilities.
+  // 'agent'), it now absorbs the merged agent-logs capabilities.
 
   // always registered (was gated behind `if (deps.getOrchestratorUsage)`,
   // so `/panel open cost` reported "Unknown panel" on builds without usage

package/src/panels/builtin/knowledge.ts

@@ -1,7 +1,7 @@
 import type { PanelManager } from '../panel-manager.ts';
 import type { ResolvedBuiltinPanelDeps } from './shared.ts';
 
-// (the purge) — group B: 'knowledge' and 'memory' migrated to the
+// (the purge), group B: 'knowledge' and 'memory' migrated to the
 // 'knowledge-modal' / 'memory-modal' config-modal surfaces. Both surfaces and
 // their panel→modal redirects are registered centrally in registerBuiltinModals
 // (src/panels/builtin-modals.ts). This registrar is intentionally empty