@pellux/goodvibes-tui 2.0.13 → 2.0.15

package/src/renderer/compaction-history-modal.ts

@@ -17,7 +17,7 @@ import type { CompactionQualityScore } from './compaction-quality.ts';
 // ─── formatCompactionEvent ────────────────────────────────────────────────────
 
 /**
- * Exported for direct unit testing () — the grade suffix is a pure
+ * Exported for direct unit testing (), the grade suffix is a pure
  * function of the event and an optional score, independent of whichever
  * runtime lookup (getCompactionQualityScore) supplies that score.
  */
@@ -35,7 +35,7 @@ export function formatCompactionEvent(
   const trigger = ev.trigger === 'auto' ? 'auto' : 'manual';
   // Honest omission: no score means none was computed for this event (e.g. it
   // pre-dates this feature, or ran through the small-window path, which has
-  // no CompactionEvent to key a score by) — never fabricate a grade.
+  // no CompactionEvent to key a score by), never fabricate a grade.
   const qualityStr = qualityScore ? `  quality=${qualityScore.grade} (${qualityScore.score.toFixed(2)})` : '';
   return (
     `#${n} ${timeStr} [${trigger}]  ` +
@@ -56,7 +56,7 @@ function fmtN(n: number): string {
 export function buildCompactionHistoryText(): string {
   const events = getCompactionEvents();
   if (events.length === 0) {
-    return '[Context] No compactions recorded this session. (Restore is not available — the SDK does not yet expose a snapshot restore API.)';
+    return '[Context] No compactions recorded this session. (Restore is not available: the SDK does not yet expose a snapshot restore API.)';
   }
   const lines: string[] = [
     `[Context] Compaction history (${events.length} total, most recent first):`,
@@ -66,6 +66,6 @@ export function buildCompactionHistoryText(): string {
     const ev = ordered[i]!;
     lines.push('  ' + formatCompactionEvent(ev, ordered.length - i, getCompactionQualityScore(ev.timestamp)));
   }
-  lines.push('  (Restore not available — the SDK does not yet expose a snapshot restore API.)');
+  lines.push('  (Restore not available: the SDK does not yet expose a snapshot restore API.)');
   return lines.join('\n');
 }

package/src/renderer/compaction-preview.ts

@@ -4,7 +4,7 @@
  * Pre-compact: shows an honest estimate of what compaction will do
  * (message count and token estimate before → after). The SDK has no dry-run
  * API; we derive the after-estimate from the DEFAULT_COMPACTION_CONFIG
- * totalCeiling (6500 tokens) — clearly labelled as an ESTIMATE.
+ * totalCeiling (6500 tokens), clearly labelled as an ESTIMATE.
  *
  * Post-compact: shows a before/after notice using the CompactionEvent
  * data returned by compactMessages(), which contains the real
@@ -49,7 +49,7 @@ export interface CompactionAfterOptions {
   /**
    * Out-of-band quality-score grade for this run (). Omitted or null
    * when no score was computed (e.g. the small-window compaction path, which
-   * has no CompactionEvent to key a score by) — no line is rendered then.
+   * has no CompactionEvent to key a score by), no line is rendered then.
    */
   readonly qualityScore?: CompactionQualityScore | null | undefined;
 }
@@ -78,7 +78,7 @@ export function buildCompactionPreview(opts: CompactionPreviewOptions): string {
 
   return (
     `[Context] ${triggerStr} conversation: ~${fmtN(tokensBefore)} tokens across ${msgCount} message${msgCount === 1 ? '' : 's'}${contextStr}.` +
-    ` Estimated result: ~${fmtN(tokensAfterEstimate)} tokens (estimate — actual depends on content).` +
+    ` Estimated result: ~${fmtN(tokensAfterEstimate)} tokens (estimate; actual depends on content).` +
     (pinStr ? ` ${pinStr.trim()}` : '')
   );
 }
@@ -129,7 +129,7 @@ function fmtN(n: number): string {
 /**
  * Build the /keep command usage text (shown when no args are provided).
  *
- * Exported for testability — the shell-core handler renders this string directly.
+ * Exported for testability, the shell-core handler renders this string directly.
  */
 export function buildPinUsageText(): string {
   return (
@@ -147,7 +147,7 @@ export function buildPinUsageText(): string {
  * @param text - The pinned text
  * @param count - Total pinned memory count after adding
  *
- * Exported for testability — the shell-core handler renders this string directly.
+ * Exported for testability, the shell-core handler renders this string directly.
  */
 export function buildPinSuccessText(id: string, text: string, count: number): string {
   return (

package/src/renderer/compaction-quality.ts

@@ -4,7 +4,7 @@
  * Reality check: the SDK's `computeQualityScore()`
  * (platform/runtime/compaction/quality-score.ts) is only ever called from a
  * SEPARATE pipeline, `CompactionManager` (platform/runtime/compaction/manager.ts).
- * The TUI's actual `/compact` path never instantiates `CompactionManager` — it
+ * The TUI's actual `/compact` path never instantiates `CompactionManager`, it
  * goes through `platform/core/context-compaction.ts` via
  * `ConversationManager.compact()` (see `compactConversation()` in
  * runtime-services.ts), which has no scoring of its own. Adopting the whole
@@ -20,7 +20,7 @@
  * Honesty note: the `strategy` value passed to `computeQualityScore()` is a
  * BORROWED rubric label describing what `compactConversation()`'s structured
  * multi-section extraction actually does (collapse the whole conversation
- * into one handoff message) — it is NOT a real `CompactionStrategy`
+ * into one handoff message), it is NOT a real `CompactionStrategy`
  * escalation result, and no strategy switch is triggered from it. Every
  * rendering of the score says so explicitly (see `formatQualityScoreLine`).
  *
@@ -29,7 +29,7 @@
  * api-extractor regen for a value the SDK's own compaction log never
  * populates). Keyed by `CompactionEvent.timestamp` rather than array index,
  * because `compactSmallWindow`'s null-event path (small-context-window
- * models — see runtime-services.ts) can create index gaps.
+ * models, see runtime-services.ts) can create index gaps.
  */
 
 import { computeQualityScore, describeScore } from '@/runtime/index.ts';
@@ -41,7 +41,7 @@ export type { CompactionQualityScore } from '@/runtime/index.ts';
 /**
  * Bound the TUI-local score store the same way the SDK bounds its own
  * compaction event log (context-compaction.ts caps at 50 and shifts the
- * oldest entry) — a long-running session should not accumulate this forever.
+ * oldest entry), a long-running session should not accumulate this forever.
  */
 const MAX_TRACKED_SCORES = 50;
 
@@ -80,7 +80,7 @@ export function scoreCompactionRun(input: ScoreCompactionRunInput): CompactionQu
     messages: input.messagesBefore,
     tokensBefore: input.tokensBefore,
     contextWindow: input.contextWindow,
-    // Borrowed rubric label, not a real strategy escalation — see module doc.
+    // Borrowed rubric label, not a real strategy escalation, see module doc.
     strategy: 'collapse',
   };
   const strategyOutput: StrategyOutput = {

package/src/renderer/composer-fragments.ts

@@ -1,5 +1,5 @@
 // ---------------------------------------------------------------------------
-// composer-fragments.ts — composer-adjacent transcript fragments extracted from
+// composer-fragments.ts, composer-adjacent transcript fragments extracted from
 // ui-factory.ts (file-size hygiene): the mid-turn queued-message editable list
 // and the optional "used N memories" provenance chip. Pure renderers; the
 // UIFactory static methods are thin wrappers over these.
@@ -15,14 +15,14 @@ import { activeTheme } from './theme.ts';
  * message shows a 1-based number so it can be named to `/queue edit <n> …` /
  * `/queue delete <n>` (the SDK editQueuedMessage / deleteQueuedMessage verbs). A
  * delivered message has already left the queue, so the list only ever shows what
- * is still editable — delivery is immutability, made visible.
+ * is still editable, delivery is immutability, made visible.
  */
 export function renderQueuedMessageList(width: number, items: readonly { readonly id: string; readonly text: string }[]): Line[] {
   if (items.length === 0) return [];
   const t = activeUiTones();
   const bodyBg = activeTheme().collapsedBodyBg;
   const lines: Line[] = [];
-  const header = `${items.length} queued — /queue edit·delete until delivered`;
+  const header = `${items.length} queued: /queue edit·delete until delivered`;
   lines.push(...renderConversationFragment(header, width, { prefix: ' ⧗ ', prefixFg: t.state.reasoning, text: t.fg.dim, bodyBg, dim: true }));
   items.forEach((item, index) => {
     lines.push(...renderConversationFragment(item.text, width, { prefix: `   ${index + 1}. `, prefixFg: t.state.reasoning, text: t.fg.dim, bodyBg, dim: true }));
@@ -32,7 +32,7 @@ export function renderQueuedMessageList(width: number, items: readonly { readonl
 
 /**
  * One drilled-in memory-provenance row. `record` is the RESOLVED record summary
- * (undefined = still resolving, null = no longer available) — the drill-in
+ * (undefined = still resolving, null = no longer available), the drill-in
  * shows a human summary, never a raw record id, matching the webui's detail
  * fetch (per-id, so one missing record never blanks the rest).
  */
@@ -44,7 +44,7 @@ export interface MemoryProvenanceEntry {
 /**
  * The optional "used N memories" turn chip. Collapsed: one small line naming the
  * count with the drill-in hint. Expanded (Alt+M): the same line followed by one
- * line per memory — its resolved SUMMARY (with the record class), a "resolving…"
+ * line per memory, its resolved SUMMARY (with the record class), a "resolving…"
  * placeholder while the fetch is in flight, or an honest "no longer available"
  * for a record that has since been forgotten. Empty when no memories were used.
  */
@@ -53,7 +53,7 @@ export function renderMemoryProvenanceChip(width: number, count: number, entries
   const t = activeUiTones();
   const bodyBg = activeTheme().collapsedBodyBg;
   const noun = count === 1 ? 'memory' : 'memories';
-  const header = expanded ? `used ${count} ${noun} — Alt+M to hide` : `used ${count} ${noun} — Alt+M to list`;
+  const header = expanded ? `used ${count} ${noun}: Alt+M to hide` : `used ${count} ${noun}: Alt+M to list`;
   const lines: Line[] = renderConversationFragment(header, width, { prefix: ' ◆ ', prefixFg: t.state.info, text: t.fg.dim, bodyBg, dim: true });
   if (expanded) {
     entries.forEach((entry, index) => {

package/src/renderer/compositor.ts

@@ -28,7 +28,7 @@ export interface SearchInfo {
 export interface PanelCompositeData {
   /**
    * The single consolidated workspace tab bar spanning all open panels across
-   * both panes. There are no per-pane tab bars — pane focus is shown by the
+   * both panes. There are no per-pane tab bars, pane focus is shown by the
    * accent border (see `topFocused`/`bottomFocused`).
    */
   workspaceBar: Line;
@@ -62,7 +62,7 @@ export interface CompositeRequest {
 
 /**
  * Compositor - Authoritative TUI layout engine with Selection Overlay.
- * Decoupled from global state — all needed data is passed as parameters.
+ * Decoupled from global state, all needed data is passed as parameters.
  */
 export class Compositor {
   /** Double-buffer reuse: back is written, front is the last-rendered reference. */
@@ -80,7 +80,7 @@ export class Compositor {
    * and anything that writes to the terminal outside composite() moves content
    * the compositor never wrote. After either, a row the model believes is
    * already blank is never re-emitted, so whatever the terminal is showing
-   * there — splash art, a stale rule line — survives every later frame. The
+   * there, splash art, a stale rule line, survives every later frame. The
    * erase is what clears cells the diff structurally skips (wide-glyph
    * continuation cells, see DiffEngine.diff), which a diff against a null
    * front buffer alone cannot do.
@@ -93,12 +93,12 @@ export class Compositor {
     this.diffEngine = new DiffEngine(this.caps);
   }
 
-  /** Exposed for unit tests — returns the detected color capability. */
+  /** Exposed for unit tests, returns the detected color capability. */
   public get termCapsForTest(): TermColorCaps {
     return this.caps;
   }
 
-  /** Exposed for unit tests — returns the last composited buffer. */
+  /** Exposed for unit tests, returns the last composited buffer. */
   public get lastBufferForTest(): TerminalBuffer | null {
     return this.frontBuffer;
   }
@@ -126,7 +126,7 @@ export class Compositor {
   public composite(params: CompositeRequest): void {
     const { width, height, header, viewport, footer, selection, search, panel, panelWidth } = params;
     // A size change reallocates the buffer, which drops every record of what
-    // the terminal is currently showing — repaint in full rather than diff
+    // the terminal is currently showing, repaint in full rather than diff
     // against a model that no longer describes the screen.
     const resized = this.frontBuffer !== null
       && (this.frontBuffer.width !== width || this.frontBuffer.height !== height);
@@ -148,7 +148,7 @@ export class Compositor {
     const leftWidth = hasPanel ? Math.max(1, width - panelWidth - 1) : width;
     const sepX = hasPanel ? leftWidth : -1;
 
-    // 1. Draw Header — always full width
+    // 1. Draw Header, always full width
     header.forEach((line, i) => newBuffer.blitLine(i, line));
 
     // 2. Draw Viewport directly after the supplied header.
@@ -320,7 +320,7 @@ export class Compositor {
     // (rows past the supplied viewport lines are covered by the loop above,
     // separator column included, so they can no longer keep a stale frame.)
 
-    // 3. Draw Footer (Pinned to Bottom) — always full width
+    // 3. Draw Footer (Pinned to Bottom), always full width
     const footerStart = height - footer.length;
     footer.forEach((line, i) => {
       const screenY = footerStart + i;
@@ -329,7 +329,7 @@ export class Compositor {
     });
 
     // 4. Diff and Render
-    // Diff against front-buffer (last-rendered), then swap front/back — no clone() needed.
+    // Diff against front-buffer (last-rendered), then swap front/back, no clone() needed.
     // On a full repaint the SGR run-state is reset (the erase below leaves the
     // terminal's attributes unknown) and the diff runs against no previous
     // frame, so every cell of the grid is emitted.

package/src/renderer/config-modal.ts

@@ -5,13 +5,13 @@ import { activeUiTones } from './theme.ts';
 import type { ConfigModal, ConfigModalRenderModel } from '../input/config-modal.ts';
 
 // Matches ModalFactory.createModal's own box-width computation (margin 4,
-// requested max width 76 — see renderConfigModalModel below) so the
+// requested max width 76, see renderConfigModalModel below) so the
 // wrap-clamp in ConfigModal.getRenderModel() measures against the SAME width
 // the list section will actually wrap at, at the current terminal size.
 const MODAL_MARGIN = 4;
 const MODAL_MAX_WIDTH = 76;
 /**
- * renderConfigModal — the single render path for every ConfigModalSurface.
+ * renderConfigModal, the single render path for every ConfigModalSurface.
  * Reads the host's frozen-structure-plus-live-values render model and maps it
  * onto ModalFactory.createModal (the same renderer settings-modal uses), so the
  * chrome, tab strip, list, and footer are one dialect across all migrated
@@ -32,7 +32,7 @@ export function renderConfigModal(modal: ConfigModal, width: number, height: num
   return renderConfigModalModel(model, width);
 }
 
-/** Pure model→Line[] mapping — the shared inner step of renderConfigModal. */
+/** Pure model→Line[] mapping, the shared inner step of renderConfigModal. */
 function renderConfigModalModel(model: ConfigModalRenderModel, width: number): Line[] {
   // Read the active-mode chrome tones per call (this render path is not on a
   // hot loop; the modal renders once per keystroke).

package/src/renderer/context-inspector.ts

@@ -8,7 +8,7 @@ import { UI_TONES } from './ui-primitives.ts';
 // ─── ContextInspectorModal ────────────────────────────────────────────────────
 
 /**
- * ContextInspectorModal — state for the context inspector overlay.
+ * ContextInspectorModal, state for the context inspector overlay.
  */
 export class ContextInspectorModal {
   public active = false;

package/src/renderer/context-status-hint.ts

@@ -1,5 +1,5 @@
 /**
- * Context status hint — TASK-056.
+ * Context status hint, TASK-056.
  *
  * Produces a short, dismissible status-line hint when the session maintenance
  * level indicates compaction is recommended or repair is needed.  The hint is
@@ -15,11 +15,11 @@
  *
  * Evidence policy (why the guards below exist)
  * ────────────────────────────────────────────
- * A fresh boot printed "Context high (0% used) — auto-compact will run before
+ * A fresh boot printed "Context high (0% used), auto-compact will run before
  * the next turn." before a single token had been counted. The evaluator was
  * not at fault: `evaluateSessionMaintenance` reaches 'suggest-compact' when
  * `remainingTokens <= 15_000`, and at boot the resolved window is still the
- * SDK's DEFAULT_CONTEXT_WINDOW fallback (8,192 — the "8.2k" the footer meter
+ * SDK's DEFAULT_CONTEXT_WINDOW fallback (8,192, the "8.2k" the footer meter
  * shows before the real window arrives), so 8,192 free tokens tripped that
  * rule instantly. Arithmetically consistent; factually false. No context had
  * been consumed and no auto-compact was imminent.
@@ -31,7 +31,7 @@
  *   2. The window is one the provider vouched for, not the fallback constant.
  *      A model whose real window happens to equal the fallback is
  *      indistinguishable from "not resolved yet", and in that ambiguity
- *      staying quiet is the honest choice — the Tokens panel and /context
+ *      staying quiet is the honest choice, the Tokens panel and /context
  *      still show the real numbers either way.
  * Neither guard touches 'compacting', which reports work that is provably
  * running rather than a prediction about headroom.
@@ -69,8 +69,8 @@ function hasRealContextNumbers(
  * Build the passive status-line hint text for context pressure.
  *
  * Returns null when no hint is warranted (stable / watch / unknown), and also
- * when a pressure level is not yet backed by real numbers — see the evidence
- * policy above. The caller renders this as a dim informational line — no
+ * when a pressure level is not yet backed by real numbers, see the evidence
+ * policy above. The caller renders this as a dim informational line, no
  * prompts, no blocking, no confirmation required.
  */
 function buildContextStatusHint(options: ContextStatusHintOptions): string | null {
@@ -79,17 +79,17 @@ function buildContextStatusHint(options: ContextStatusHintOptions): string | nul
   switch (level) {
     case 'needs-repair':
       if (!hasRealContextNumbers(options)) return null;
-      return `  Context pressure critical (${usagePct}% used) — compaction needs attention. Run /compact or /health review.`;
+      return `  Context pressure critical (${usagePct}% used); compaction needs attention. Run /compact or /health review.`;
 
     case 'suggest-compact':
       if (!hasRealContextNumbers(options)) return null;
       if (autoCompactEnabled) {
-        return `  Context high (${usagePct}% used) — auto-compact will run before the next turn.`;
+        return `  Context high (${usagePct}% used): auto-compact will run before the next turn.`;
       }
-      return `  Context high (${usagePct}% used) — run /compact to recover headroom.`;
+      return `  Context high (${usagePct}% used): run /compact to recover headroom.`;
 
     case 'compacting':
-      return `  Compacting context — freeing headroom...`;
+      return `  Compacting context: freeing headroom...`;
 
     default:
       return null;

package/src/renderer/conversation-layout.ts

@@ -42,11 +42,11 @@ export function buildConversationViewport(
  * Absolute history-line index of the bottom-most visible row of the
  * conversation viewport. The buffer bottom-aligns content shorter than the
  * viewport (InfiniteBuffer.getSnapshot pads with blank lines at the TOP, not
- * the bottom — see history.ts), so the naive `scrollTop + viewportHeight - 1`
+ * the bottom, see history.ts), so the naive `scrollTop + viewportHeight - 1`
  * overshoots past the real last line whenever the whole transcript already
  * fits on screen. Clamping to `lineCount - 1` handles both cases with one
  * formula: when scrollLocked at the true bottom (the common case), this is
- * exactly the last rendered content line — the block the user is actually
+ * exactly the last rendered content line, the block the user is actually
  * looking at, not whatever sits at the TOP of the viewport (scrollTop alone).
  */
 export function getViewportBottomLine(scrollTop: number, viewportHeight: number, lineCount: number): number {

package/src/renderer/conversation-overlays.ts

@@ -42,7 +42,7 @@ export function applyConversationOverlays(
   // Overlay posture rule: workspaces (onboarding, model, settings, mcp, help,
   // shortcuts) claim the full viewport and are mutually exclusive with docked
   // pickers/modals. Once a fullscreen workspace claims the screen, no docked
-  // overlay is drawn on top of it — enforcing a single visible overlay even if
+  // overlay is drawn on top of it, enforcing a single visible overlay even if
   // stray state survives the handler's clearModalStack. The input layer keeps
   // the modal stack to one entry; this is the renderer-side backstop.
   let fullscreenClaimed = false;
@@ -78,7 +78,7 @@ export function applyConversationOverlays(
   }
 
   // retirement: the process, agent-detail, and live-tail modal overlays
-  // were removed with those modals — F2 now opens the Fleet panel, which
+  // were removed with those modals, F2 now opens the Fleet panel, which
   // subsumes the live process tree.
 
   if (!fullscreenClaimed && input.contextInspectorModal.active) {

package/src/renderer/conversation-surface.ts

@@ -106,14 +106,14 @@ export function renderConversationFragment(
   indentCols = 0,
 ): Line[] {
   // A tree fragment is a continuation of the row above it, so it starts at that
-  // row's content column — which puts its ` ▸ ` prefix glyph in exactly the same
+  // row's content column, which puts its ` ▸ ` prefix glyph in exactly the same
   // column as the parent row's own ` ▸ N lines ` badge. A flush fragment (a user
   // message ghost box) keeps the box margin it has always used.
   const indent = Math.max(0, indentCols);
   const margin = indent > 0 ? treeContentCol(indent) : LAYOUT.USER_BOX_MARGIN;
   const prefixWidth = getDisplayWidth(palette.prefix);
   // The indent is charged to the content budget rather than allowed to push the
-  // fragment past the right edge — a narrow terminal shrinks the preview text
+  // fragment past the right edge, a narrow terminal shrinks the preview text
   // instead of silently truncating its tail.
   const maxContentWidth = Math.max(1, width - margin - LAYOUT.USER_BOX_MARGIN - prefixWidth - 2);
   const wrapped = wrapText(content, maxContentWidth);
@@ -177,15 +177,15 @@ export function renderConversationFragment(
  * A COLLAPSED block, as exactly one row: the event line, with its preview
  * riding on the same line right after the badges.
  *
- * Collapsed blocks used to render as a framed fragment — a `▄▄▄` cap, an
- * interior line of text, a `▀▀▀` cap — around what was, in every case, a single
+ * Collapsed blocks used to render as a framed fragment, a `▄▄▄` cap, an
+ * interior line of text, a `▀▀▀` cap, around what was, in every case, a single
  * line. Four rows of chrome per fold made a screen of collapsed output mostly
  * frame. The text those rows carried moves onto the event line and the frame
  * goes; the `▸ N lines` badge is the count, so nothing restates it.
  *
  * The preview is TRUNCATED to fit, never wrapped: wrapping would put the row
  * back to two lines, which is the thing being removed. Whitespace is flattened
- * by the caller's choice of preview text — a raw tab or newline would blow the
+ * by the caller's choice of preview text, a raw tab or newline would blow the
  * column out.
  */
 export function renderConversationFoldedRow(
@@ -202,7 +202,7 @@ export function renderConversationFoldedRow(
   // Whether a preview renders at all, and its flattening, are the policy's call
   // (foldPreviewText returns null for "no preview"). Only the TRUNCATION is
   // ours: display width is a product-local rule the policy deliberately leaves
-  // to the caller — and it must truncate, never wrap.
+  // to the caller, and it must truncate, never wrap.
   const trimmed = foldPreviewText(preview, availCols);
   if (trimmed === null) {
     return renderConversationEventLine(width, tone, details, indentCols);

package/src/renderer/diff.ts

@@ -74,7 +74,7 @@ export class DiffEngine {
           // the SGR state did not change, skip cursor re-addressing.
           // Emit a cursor move only on run breaks (new row, gap, or style change).
           if (sgrOutput === '' && this.lastEmitY === y && this.lastEmitX === x - 1) {
-            // Contiguous run on same row, same SGR — just append the char.
+            // Contiguous run on same row, same SGR, just append the char.
             output += newCell.char;
           } else {
             output += `\x1b[${y + 1};${x + 1}H`;
@@ -107,7 +107,7 @@ export class DiffEngine {
    * Convert a raw color string (hex or r;g;b) to the "r;g;b" form expected
    * by applyStyles.  For non-RGB palette indices the value is returned as-is.
    * This mirrors the original sanitizeColor contract but is now capability-aware
-   * only in the sense of normalizing hex → "r;g;b" — downsampling happens in
+   * only in the sense of normalizing hex → "r;g;b", downsampling happens in
    * applyStyles via downsampleColor.
    */
   private sanitizeColor(color: string): string {
@@ -148,7 +148,7 @@ export class DiffEngine {
         if (cell.strikethrough) style += '\x1b[9m';
       }
 
-      // Foreground color — capability-downsampled
+      // Foreground color, capability-downsampled
       const fgOut = downsampleColor(fg, this.caps, 'fg');
       if (fgOut !== null) {
         if (this.caps.capability === 'basic16') {
@@ -160,7 +160,7 @@ export class DiffEngine {
         }
       }
 
-      // Background color — capability-downsampled
+      // Background color, capability-downsampled
       const bgOut = downsampleColor(bg, this.caps, 'bg');
       if (bgOut !== null) {
         if (this.caps.capability === 'basic16') {

package/src/renderer/fleet-tab-strip.ts

@@ -5,7 +5,7 @@
 // renderTabStrip exactly as panel-workspace-bar.ts does for the workspace
 // tab bar. Deliberately a SEPARATE, visually distinct strip from the
 // workspace bar: workspace tabs switch PANELS, fleet session tabs switch
-// ATTACHED PROCESSES within one panel — conflating their styling would blur
+// ATTACHED PROCESSES within one panel, conflating their styling would blur
 // that distinction for the operator.
 // ---------------------------------------------------------------------------
 
@@ -22,7 +22,7 @@ const LABEL_FG = UI_TONES.fg.secondary;
 
 /**
  * Render the fleet session-tab strip, or `null` when there are no attached
- * tabs — the panel omits the strip entirely in that case (root-tab-only),
+ * tabs, the panel omits the strip entirely in that case (root-tab-only),
  * which is what keeps the pre-session-tab fleet-panel goldens byte-identical.
  */
 export function renderFleetTabStrip(

package/src/renderer/footer-tips.ts

@@ -1,5 +1,5 @@
 /**
- * footer-tips.ts — the persistent discoverability hint shown in the shell
+ * footer-tips.ts, the persistent discoverability hint shown in the shell
  * footer's tip slot.
  *
  * instead of a single frozen "/help for commands" line, the footer
@@ -8,7 +8,7 @@
  * is actively running, the process-monitor tip is promoted to the front so the
  * operator can jump to F2 while work is in flight. Selection is a pure function
  * of context so the footer renders deterministically (golden frames stay
- * stable) — no wall-clock input.
+ * stable), no wall-clock input.
  */
 
 export interface FooterTipContext {
@@ -25,7 +25,7 @@ export function isAgentActive(composerStatus: string | undefined): boolean {
 }
 
 // Ctrl+P is a TOGGLE (open+focus when nothing is open or unfocused,
-// hide when the workspace already has focus — see openPanelPicker in
+// hide when the workspace already has focus, see openPanelPicker in
 // shell/ui-openers.ts). The bare noun 'panels' undersold that; naming the verb
 // keeps the tip honest about what the chord actually does.
 const TIP_PANELS = 'Ctrl+P toggle panels';
@@ -33,7 +33,7 @@ const TIP_PANELS = 'Ctrl+P toggle panels';
 // tip names 'fleet', not 'processes'.
 const TIP_PROCESSES = 'F2 fleet';
 const TIP_HELP = '? help';
-// f: an empty-composer Ctrl+C does NOT quit on the first press — it arms a
+// f: an empty-composer Ctrl+C does NOT quit on the first press, it arms a
 // ~1s "press again to exit" confirm, and only a SECOND Ctrl+C within that window
 // exits (an intentional accidental-exit guard). The old 'Ctrl+C quit' tip
 // implied a single press quits, so two presses seconds apart appeared to do

package/src/renderer/fullscreen-primitives.ts

@@ -6,7 +6,7 @@ import { activeUiTones, registerThemeRefresh } from './theme.ts';
 
 // Built from the mode-resolved chrome tones (activeUiTones) and rebuilt IN PLACE
 // on a mode flip via the registered refresher (read by reference across the
-// fullscreen workspace surfaces — see theme.ts's active-mode runtime note).
+// fullscreen workspace surfaces, see theme.ts's active-mode runtime note).
 function buildFullscreenPalette(): Record<string, string> {
   const t = activeUiTones();
   return {

package/src/renderer/git-status.ts

@@ -13,11 +13,11 @@ export interface GitHeaderInfo {
 const FALLBACK: GitHeaderInfo = { branch: '?', dirty: false, ahead: 0, behind: 0 };
 
 /**
- * GitStatusProvider — Fetches git state for the header bar.
+ * GitStatusProvider, Fetches git state for the header bar.
  *
  * Results are cached for 2 seconds (TTL). The next call after expiry triggers
  * a fresh fetch and returns the cached value immediately (stale-while-revalidate).
- * Never throws — returns FALLBACK on any error.
+ * Never throws, returns FALLBACK on any error.
  */
 export class GitStatusProvider {
   private cache: GitHeaderInfo = { ...FALLBACK };
@@ -61,7 +61,7 @@ export class GitStatusProvider {
    * GitService.isGitRepo() check) but amortizes the cost for the header: the
    * cheap synchronous isGitRepo() spawn runs every tick; the heavier async
    * status()+branch() fetch (this.refresh()) only runs when that boolean
-   * actually flips — e.g. an external `git init`, or `.git` removed.
+   * actually flips, e.g. an external `git init`, or `.git` removed.
    */
   startPolling(intervalMs: number, onChange: (info: GitHeaderInfo) => void): void {
     if (this.pollTimer !== null) return;
@@ -110,7 +110,7 @@ export class GitStatusProvider {
         statusResult.conflicted.length > 0 ||
         statusResult.not_added.length > 0;
       this.cache = {
-        // Empty `current` means an unborn HEAD — a repo with no commits yet.
+        // Empty `current` means an unborn HEAD, a repo with no commits yet.
         // Show "new" instead of an unfriendly "?" for that first-run case. (5a)
         branch: branchResult.current || 'new',
         dirty,
@@ -119,7 +119,7 @@ export class GitStatusProvider {
       };
       this.lastFetch = Date.now();
     } catch {
-      // Never throw — return fallback
+      // Never throw, return fallback
       if (this.lastFetch === 0) {
         this.cache = { ...FALLBACK };
         this.lastFetch = Date.now();

package/src/renderer/header-line.ts

@@ -1,5 +1,5 @@
 /**
- * header-line.ts — the top-of-screen header row: brand, version, optional
+ * header-line.ts, the top-of-screen header row: brand, version, optional
  * conversation title, optional git segment, and the right-hand model/provider
  * pair.
  *
@@ -8,7 +8,7 @@
  * remains the call name everywhere and delegates here.
  *
  * The model/provider pair rendered here is the SERVING backend, resolved by
- * core/active-model-identity.ts — the same resolution the footer uses, so the
+ * core/active-model-identity.ts, the same resolution the footer uses, so the
  * two surfaces can no longer disagree about who is answering. `note` carries
  * the divergence marker naming the user's configured selection when serving
  * has moved off it.
@@ -43,7 +43,7 @@ function buildGitSegment(gitInfo: GitHeaderInfo): { text: string; width: number
  * Graded, never truncated mid-word: the full marker (which names the
  * configured selection) is preferred; a bare "divergent" keyword is the
  * fallback on a narrow terminal; on a very narrow terminal the marker is
- * dropped entirely. Dropping it is safe — what remains is still the SERVING
+ * dropped entirely. Dropping it is safe, what remains is still the SERVING
  * backend, so the header never claims the configured backend is answering.
  * The footer carries the full marker with far more room (see ui-factory's
  * prioritized context segments).
@@ -80,7 +80,7 @@ export function renderHeaderLine(
 ): Line[] {
   const lines: Line[] = [];
   // Header/footer/thinking paint on the transparent terminal background, so
-  // they read chrome.* (light-terminal-aware) — NOT fg.*/state.*, which stay
+  // they read chrome.* (light-terminal-aware), NOT fg.*/state.*, which stay
   // tuned for the opaque dark modal/panel boxes. Read live per render so a
   // mode flip re-resolves without any module reload (see theme.ts).
   const t = activeUiTones();
@@ -99,7 +99,7 @@ export function renderHeaderLine(
   let curX = 0;
   for (const char of brand) { line[curX++] = { char, fg: CYAN, bg: '', bold: true, dim: false, underline: false, italic: false, strikethrough: false }; }
   for (const char of ver) { line[curX++] = { char, fg: GREY, bg: '', bold: false, dim: true, underline: false, italic: false, strikethrough: false }; }
-  // Optional conversation title — shown after brand/ver, truncated to fit
+  // Optional conversation title, shown after brand/ver, truncated to fit
   if (title) {
     const titleStr = `│ ${title} `;
     // Reserve space for git info (if present) + model/provider + marker on the right