@pellux/goodvibes-tui 2.0.14 → 2.0.15

package/src/renderer/help-overlay.ts

@@ -1,8 +1,8 @@
 /**
- * renderHelpOverlay — renders the help overlay with keyboard shortcuts and slash commands.
+ * renderHelpOverlay, renders the help overlay with keyboard shortcuts and slash commands.
  *
  * Opened via `/commands`. `?` and `/help` open the searchable command
- * browser (a selection modal that runs the picked command) instead — they
+ * browser (a selection modal that runs the picked command) instead, they
  * are a different, complementary surface, not this one.
  */
 
@@ -80,7 +80,7 @@ export function renderHelpOverlay(
     `  ${'Shift+Enter'.padEnd(20)}  Insert newline`,
     `  ${kb('paste').padEnd(20)}  Paste (image priority)`,
     `  ${(kb('undo') + ' / ' + kb('redo')).padEnd(20)}  Undo / redo`,
-    shortcutLine('(paste >8 lines)', 'Folds to [TEXT: pN, M lines] — /pastes previews it before you submit'),
+    shortcutLine('(paste >8 lines)', 'Folds to [TEXT: pN, M lines]; /pastes previews it before you submit'),
     '',
     '  Overlays And Panels',
     '  ' + '\u2500'.repeat(40),
@@ -213,7 +213,7 @@ export function renderHelpOverlay(
 
   const allRows = [...shortcutRows, ...commandRows];
 
-  // Apply scroll offset — show a window of rows
+  // Apply scroll offset, show a window of rows
   const metrics = getOverlaySurfaceMetrics(width, viewportHeight, {
     chromeRows: 4,
     minContentRows: 8,
@@ -249,7 +249,7 @@ export function renderHelpOverlay(
 }
 
 /**
- * renderShortcutsOverlay — renders keyboard shortcuts as Line[].
+ * renderShortcutsOverlay, renders keyboard shortcuts as Line[].
  * Accessed via /shortcuts command. Reflects live keybindings (user overrides included).
  */
 export function renderShortcutsOverlay(

package/src/renderer/hint-grammar.ts

@@ -1,5 +1,5 @@
 /**
- * hint-grammar.ts — the single hint-bar grammar for every overlay footer.
+ * hint-grammar.ts, the single hint-bar grammar for every overlay footer.
  *
  * one grammar across the model picker, settings modal, help/shortcuts
  * overlays, and ModalFactory variants so footers stop drifting into three
@@ -44,7 +44,7 @@ export function formatHints(specs: readonly HintSpec[]): string {
 
 /**
  * Join already-rendered segments (hint bars, state tags) with the shared
- * separator, dropping empties. Order is preserved — use this to append state
+ * separator, dropping empties. Order is preserved, use this to append state
  * indicators after a {@link formatHints} bar.
  */
 export function joinHints(...segments: Array<string | null | undefined>): string {

package/src/renderer/layout.ts

@@ -1,7 +1,7 @@
 import { GLYPHS, SPINNER_FRAMES, UI_TONES } from './ui-primitives.ts';
 
 /**
- * Layout constants — single source of truth for margins and content width.
+ * Layout constants, single source of truth for margins and content width.
  * All renderers import these instead of hardcoding indent values. The object
  * itself lives in @pellux/goodvibes-terminal-shell alongside the transcript
  * tree geometry that reads LEFT_MARGIN/RIGHT_MARGIN out of it, so the margins

package/src/renderer/markdown-inline.ts

@@ -1,5 +1,5 @@
 /**
- * markdown-inline.ts — inline markdown tokenizer (bold, italic, inline code,
+ * markdown-inline.ts, inline markdown tokenizer (bold, italic, inline code,
  * links, bare URLs, absolute file paths).
  *
  * Extracted from markdown.ts so both the block renderer (markdown.ts) and the
@@ -63,7 +63,7 @@ export function renderInlineMarkdown(text: string): InlineToken[] {
         i = end + 3;
         continue;
       }
-      // No closing *** found — emit the leading * as plain text so the ** bold
+      // No closing *** found, emit the leading * as plain text so the ** bold
       // check can handle the remaining ** on the next iteration.
       tokens.push({ type: 'text', text: '*', style: {} });
       i += 1;
@@ -108,7 +108,7 @@ export function renderInlineMarkdown(text: string): InlineToken[] {
       }
     }
 
-    // Plain text — accumulate until next special char, detect bare URLs and file paths
+    // Plain text, accumulate until next special char, detect bare URLs and file paths
     let end = i + 1;
     while (end < text.length && !INLINE_SPECIAL_CHARS.has(text[end])) {
       const code = text.charCodeAt(end);

package/src/renderer/markdown-table.ts

@@ -1,11 +1,11 @@
 /**
- * markdown-table.ts — box-drawn markdown tables whose cells WRAP.
+ * markdown-table.ts, box-drawn markdown tables whose cells WRAP.
  *
  * The previous implementation (inside markdown.ts) allocated each column a
  * proportional slice of the terminal width and then ellipsized any cell that
  * did not fit: a comparison table rendered "Files, code, and…" in its label
  * column and "citation coverage, checkpoin…" in its content columns, and the
- * cut text was unrecoverable — it was never written to the buffer at all, so
+ * cut text was unrecoverable, it was never written to the buffer at all, so
  * no scroll, copy, or expand could bring it back.
  *
  * This renderer never ellipsizes. A cell that exceeds its column width wraps
@@ -17,7 +17,7 @@
  * gets its full natural width, so a narrow label column is never starved down
  * to one word plus an ellipsis while a wide prose column takes the rest:
  *
- *   1. Each column asks for `desiredMin` — enough to hold its longest single
+ *   1. Each column asks for `desiredMin`, enough to hold its longest single
  *      word (capped at WORD_MIN_CAP so one long URL cannot starve the table),
  *      but never more than the column's natural width and never less than
  *      MIN_COL when the budget allows it.
@@ -31,7 +31,7 @@
  * When even HARD_MIN_COL per column plus the borders exceeds the terminal
  * width, no allocation helps: the box drawing itself does not fit. That case
  * used to write the overflowing columns past the end of the line buffer, where
- * they were discarded — an eight-column table at width 40 lost its last
+ * they were discarded, an eight-column table at width 40 lost its last
  * columns, headers included, with nothing on screen to say so. The renderer now
  * abandons the box at that point and stacks each row as `Header: value` records
  * wrapped to the available width, so every column header and every cell is
@@ -131,7 +131,7 @@ function flattenCellChars(raw: string, isHdr: boolean): StyledChar[] {
 /**
  * Word-aware wrap of a styled character run into lines no wider than `maxW`
  * display columns. A word wider than `maxW` is broken across lines character
- * by character — the text is always emitted in full, never cut.
+ * by character, the text is always emitted in full, never cut.
  */
 function wrapStyledChars(chars: readonly StyledChar[], maxW: number): StyledChar[][] {
   if (maxW <= 0) return [[]];
@@ -306,7 +306,7 @@ function emitStyledLines(
  * Each data row becomes a block of `Header: value` lines wrapped to the
  * available width, separated by a rule. Header text is styled as a header and
  * force-broken rather than cut if it is somehow wider than the terminal, so the
- * guarantee the box layout makes — every column present, nothing truncated —
+ * guarantee the box layout makes, every column present, nothing truncated,
  * holds at any width.
  */
 function renderStackedTable(
@@ -370,7 +370,7 @@ export function renderTable(rows: string[], width: number, indent: number): Line
   const availW = width - indent;
 
   // Measure using stripped text (no markdown markers), both the natural width
-  // and the widest single word — the latter is what a column needs before it
+  // and the widest single word, the latter is what a column needs before it
   // has to start breaking words apart.
   const naturalWidths: number[] = new Array(colCount).fill(0);
   const minWordWidths: number[] = new Array(colCount).fill(0);

package/src/renderer/modal-factory.ts

@@ -95,7 +95,7 @@ export interface ModalConfig {
 // DEFAULT_STYLE is built from the mode-resolved chrome tones (activeUiTones) and
 // rebuilt IN PLACE on a mode flip via the registered refresher. createModal reads
 // it per call via `{ ...DEFAULT_STYLE, ...config.style }`, so the in-place rebuild
-// reaches every modal — see theme.ts's active-mode runtime note.
+// reaches every modal, see theme.ts's active-mode runtime note.
 function buildDefaultModalStyle(): Required<ModalStyle> {
   const t = activeUiTones();
   return {
@@ -120,7 +120,7 @@ registerThemeRefresh(() => Object.assign(DEFAULT_STYLE, buildDefaultModalStyle()
 // ── ModalFactory ─────────────────────────────────────────────────────────────
 
 /**
- * ModalFactory — unified modal rendering for goodvibes-tui.
+ * ModalFactory, unified modal rendering for goodvibes-tui.
  *
  * Renders modals as Line[] (Cell arrays) consistent with the compositor
  * overlay pipeline. All existing modals (file-picker, model-picker,
@@ -271,7 +271,7 @@ export class ModalFactory {
 
   /**
    * Render a single selectable list item row.
-   * ┤ text ├  — bordered left/right, with optional selection highlight.
+   * ┤ text ├ , bordered left/right, with optional selection highlight.
    */
   static renderListItem(
     boxW: number,

package/src/renderer/model-picker-overlay.ts

@@ -4,8 +4,8 @@
  *
  * this file used to also hold `renderModelPickerOverlay`, the
  * overlay-style model-picker renderer. It was superseded by
- * `renderModelWorkspace` (model-workspace.ts) — conversation-overlays.ts
- * routes `input.modelPicker.active` there, not here — and had no remaining
+ * `renderModelWorkspace` (model-workspace.ts), conversation-overlays.ts
+ * routes `input.modelPicker.active` there, not here, and had no remaining
  * non-test import site, so it (and its dedicated unit/golden tests) were
  * removed. This constant is still live: handler-picker-routes.ts uses it to
  * size the current workspace's visible-row window.

package/src/renderer/model-workspace.ts

@@ -124,7 +124,7 @@ function modelKey(model: ModelDefinition): string {
 
 function targetSummary(info: ModelPickerTargetInfo): string {
   if (!info.enabled) return 'disabled';
-  // 'embeddings' has no model concept — configuredNote carries the honest
+  // 'embeddings' has no model concept, configuredNote carries the honest
   // provider id + dimensions + configured-state summary instead of a
   // computed provider:model route (which would print a phantom "model:").
   if (info.configuredNote) return info.configuredNote;
@@ -189,7 +189,7 @@ function detailLines(picker: ModelPickerModal, width: number): string[] {
   } else {
     lines.push(`Context cap overrides the detected local-model context window for this selection.`);
   }
-  // A visible cursor glyph on the query itself when search is focused —
+  // A visible cursor glyph on the query itself when search is focused,
   // the only prior affordance was a footer-hint text swap ('/ search' vs.
   // 'Typing filters search...'), buried at the end of a long hint line and
   // easy to miss (confirmed via live tmux repro). This puts the "you are
@@ -283,8 +283,8 @@ function renderModelRows(picker: ModelPickerModal, lines: Line[], rows: number,
 /**
  * The effort step of the model picker.
  *
- * The list is exactly the levels the chosen model resolved to — not a fixed
- * four — and it is introduced by a headline naming what this model does with
+ * The list is exactly the levels the chosen model resolved to, not a fixed
+ * four, and it is introduced by a headline naming what this model does with
  * the setting (named levels, a thinking-token budget, an on/off toggle, or
  * nothing at all). A best-guess spec prints its caveat under the list rather
  * than presenting unverified levels as if the provider had confirmed them.
@@ -433,8 +433,8 @@ export function renderModelWorkspace(picker: ModelPickerModal, width: number, vi
   const details = detailLines(picker, contentW - 2);
   // Never drop detail lines (the modal rule: UI-authored descriptive text is
   // always shown in full). Size the detail band to the ACTUAL content,
-  // growing past the old fixed proportional default up to maxDetailRows —
-  // the ceiling that keeps at least a header row and one list row available —
+  // growing past the old fixed proportional default up to maxDetailRows,
+  // the ceiling that keeps at least a header row and one list row available,
   // rather than silently slicing off whatever didn't fit a 32% guess.
   const detailRows = clamp(Math.max(details.length, Math.round(bodyRows * 0.32)), minDetailRows, maxDetailRows);
   const listRows = Math.max(1, bodyRows - detailRows - 1);
@@ -475,7 +475,7 @@ export function renderModelWorkspace(picker: ModelPickerModal, width: number, vi
   const footer = contentLine(safeWidth, PALETTE.footerBg);
   const targetHint = picker.focusPane === 'targets' ? 'Focus targets' : 'Focus list';
   // W3 Finding 2: while search is focused, every printable key (including
-  // c/a/b/g) types into the query — handler-picker-routes.ts guards those
+  // c/a/b/g) types into the query, handler-picker-routes.ts guards those
   // single-key shortcuts with `!searchFocused` (Up/Down blur search first).
   // Advertising them here regardless of focus state was a footer that lied
   // about what the keys currently do. Tab still cycles price in both states

package/src/renderer/onboarding/onboarding-wizard.ts

@@ -306,7 +306,7 @@ function renderWideLayout(
   const selectedText = selectedFieldText(wizard);
   // The selected field's own hint is the direct analog of the reported
   // defect (descriptive text explaining what the currently-focused option
-  // does) — some hints run past 400 characters, e.g. security-caveat text.
+  // does), some hints run past 400 characters, e.g. security-caveat text.
   // It is never clipped to a single row: it wraps onto as many lines as it
   // needs, and the field list below starts after it, growing/shrinking the
   // field-list area rather than losing hint text. Title/description/controls

package/src/renderer/overlay-box.ts

@@ -18,7 +18,7 @@ export interface OverlayBoxPalette {
 
 // Built from the mode-resolved chrome tones (activeUiTones) and rebuilt IN PLACE
 // on a mode flip via the registered refresher (60+ call sites read it by
-// reference — see theme.ts's active-mode runtime note).
+// reference, see theme.ts's active-mode runtime note).
 function buildOverlayPalette(): OverlayBoxPalette {
   const t = activeUiTones();
   return {

package/src/renderer/panel-composite.ts

@@ -9,7 +9,7 @@ import { renderPanelWorkspaceBar } from './panel-workspace-bar.ts';
 /**
  * Per-panel render cache for dirty-flag skipping.
  *
- * Maintainability hazard — mid-render invalidation race:
+ * Maintainability hazard, mid-render invalidation race:
  *
  * If an event listener (e.g. a runtime-bus subscriber) fires DURING a panel's
  * `render()` call and mutates state such that `needsRender = true`, the
@@ -25,7 +25,7 @@ import { renderPanelWorkspaceBar } from './panel-workspace-bar.ts';
  *
  * The fix (now applied): snapshot `needsRender` before calling `render()` and
  * only call `markRendered()` when no concurrent invalidation occurred during
- * the render pass — preserving any mid-render invalidation.
+ * the render pass, preserving any mid-render invalidation.
  */
 interface PanelRenderCache {
   lines: Line[];
@@ -40,7 +40,7 @@ const panelRenderCache = new WeakMap<Panel, PanelRenderCache>();
  * its invalidate() to bump the counter stored in this map.
  *
  * Race-guard: snapshot the generation before render(), compare after. If it
- * changed, a mid-render invalidation occurred — leave needsRender=true.
+ * changed, a mid-render invalidation occurred, leave needsRender=true.
  */
 const panelRenderGen = new WeakMap<Panel, { gen: number }>();
 
@@ -72,13 +72,13 @@ export function renderPanel(panel: Panel, width: number, height: number): Line[]
   const genBefore = genState.gen;
   const lines = panel.render(width, height);
   // Only call markRendered() when no mid-render invalidation occurred.
-  // If the generation changed during render(), a concurrent invalidate() fired —
+  // If the generation changed during render(), a concurrent invalidate() fired,
   // leave needsRender=true so the next frame re-renders with the new state.
   if (genState.gen === genBefore) {
     panel.markRendered();
   }
   // If gen changed, needsRender is already true (invalidate() set it); do not
-  // call markRendered() — the next frame will pick it up.
+  // call markRendered(), the next frame will pick it up.
   panelRenderCache.set(panel, { lines, width, height });
   return lines;
 }

package/src/renderer/panel-workspace-bar.ts

@@ -3,7 +3,7 @@ import type { WorkspaceTab } from '../panels/panel-manager.ts';
 import { renderTabStrip, type TabHitRegion } from './tab-strip.ts';
 import { UI_TONES } from './ui-primitives.ts';
 
-// Theme tokens (no raw hex) — keeps the bar in sync with the shared palette.
+// Theme tokens (no raw hex), keeps the bar in sync with the shared palette.
 const ACTIVE_FG = UI_TONES.fg.primary;       // selected tab text
 const ACTIVE_BG = UI_TONES.bg.selected;      // selected tab fill (stronger contrast)
 const INACTIVE_FG = UI_TONES.fg.muted;       // unselected tabs

package/src/renderer/process-indicator.ts

@@ -7,7 +7,7 @@ import { formatHints } from './hint-grammar.ts';
 import { voiceCaptureRowVisible, type VoiceCaptureIndicatorState } from '../core/voice-capture-status.ts';
 
 /**
- * renderProcessIndicator — shows a one-line summary of active background
+ * renderProcessIndicator, shows a one-line summary of active background
  * processes below the input area.
  *
  * Dimmed when no processes are active, highlighted (cyan) when agents or
@@ -17,7 +17,7 @@ import { voiceCaptureRowVisible, type VoiceCaptureIndicatorState } from '../core
 /**
  * Agent-count label. The footer counts only ACTIVE agents, while the fleet
  * lists every node (running, terminal, chains, watchers). Label the count
- * "active" so it is never misread as a grand total — the [Enter] View hint
+ * "active" so it is never misread as a grand total, the [Enter] View hint
  * opens the fleet for the full picture. (item 5d.)
  */
 function agentCountLabel(agentCount: number): string {
@@ -100,21 +100,21 @@ export function renderProcessIndicator(
 /**
  * Sentence each capture state renders as. Written out per state rather than
  * assembled from fragments because these are the words that tell a user whether
- * their microphone is open — "listening" and "recording" mean different things
+ * their microphone is open, "listening" and "recording" mean different things
  * and a row that blurred them would be worse than no row.
  */
 const VOICE_CAPTURE_LABELS: Record<VoiceCaptureIndicatorState['kind'], string> = {
   requesting: 'opening the microphone',
-  recording: 'recording — press the voice-input key again to stop',
+  recording: 'recording: press the voice-input key again to stop',
   transcribing: 'transcribing what you said',
   'wake-listening': 'listening for the wake phrase',
-  'wake-capturing': 'wake heard — recording what follows',
-  'wake-restarting': 'capture stream ended — restarting',
+  'wake-capturing': 'wake heard: recording what follows',
+  'wake-restarting': 'capture stream ended: restarting',
   'wake-latched': 'wake detection stopped',
 };
 
 /**
- * renderVoiceCaptureIndicator — the persistent row shown while a microphone is
+ * renderVoiceCaptureIndicator, the persistent row shown while a microphone is
  * open, below the input area beside the process indicator.
  *
  * It exists because a held-open capture device is otherwise invisible: wake
@@ -125,7 +125,7 @@ const VOICE_CAPTURE_LABELS: Record<VoiceCaptureIndicatorState['kind'], string> =
  * moment ago and is waiting on it.
  *
  * Returns no lines when nothing is captured, or when the wake rows are turned
- * off — the caller splices whatever comes back, so an empty array is "no row".
+ * off, the caller splices whatever comes back, so an empty array is "no row".
  */
 export function renderVoiceCaptureIndicator(
   width: number,

package/src/renderer/profile-picker-modal.ts

@@ -1,5 +1,5 @@
 /**
- * renderProfilePickerModal — renders the /profiles picker modal as Line[]
+ * renderProfilePickerModal, renders the /profiles picker modal as Line[]
  * using ModalFactory.
  *
  * Shows a list of saved profiles with:
@@ -57,9 +57,9 @@ export function renderProfilePickerModal(
   } else {
     // Proportional column widths that adapt to the modal's content width:
     // timestamp ~22% (clamped 10..16); the name column absorbs the remainder
-    // (ported from session-picker-modal.ts). Reserves 4 cols — 2 for the
+    // (ported from session-picker-modal.ts). Reserves 4 cols, 2 for the
     // name/timestamp separator and 2 for the list row's selection indicator
-    // ("▸ ") that ModalFactory prepends outside the wrapped label — so the
+    // ("▸ ") that ModalFactory prepends outside the wrapped label, so the
     // row never spills the timestamp's embedded space onto a wrapped line.
     const tsW = Math.min(16, Math.max(10, Math.floor(contentW * 0.22)));
     const nameW = Math.max(8, contentW - tsW - 4);

package/src/renderer/prompt-content-width.ts

@@ -1,5 +1,5 @@
 /**
- * computePromptContentWidth — content width available for prompt text inside
+ * computePromptContentWidth, content width available for prompt text inside
  * the footer input box, after the box margin, inner padding, and the ' > '
  * prefix are subtracted.
  *

package/src/renderer/search-overlay.ts

@@ -14,7 +14,7 @@ export function renderSearchOverlay(
   manager: SearchManager,
   width: number
 ): Line[] {
-  // Match count / status text — displayed in dim grey, right of query
+  // Match count / status text, displayed in dim grey, right of query
   const hasMatches = manager.matches.length > 0;
   const matchCount = hasMatches
     ? `${manager.currentMatch + 1}/${manager.matches.length}${manager.wrapAround ? ' (wrap)' : ''}`
@@ -31,11 +31,11 @@ export function renderSearchOverlay(
     : '  [Enter/Tab] lock  [Esc] close';
   const shortHints = locked ? '  [n/N]  [Esc]' : '  [Esc]';
 
-  // Build left portion: label + query (no match count — that gets separate styling)
+  // Build left portion: label + query (no match count, that gets separate styling)
   const leftPart = label + queryDisplay;
 
   // Tiered degradation: as the terminal narrows, first shorten the hints, then
-  // drop them, then drop the match-count segment — so the query area (leftWidth)
+  // drop them, then drop the match-count segment, so the query area (leftWidth)
   // never collapses to zero or negative and the overlay never renders garbage.
   const minLeft = getDisplayWidth(label) + 4; // label + a few query cells
   let hints = fullHints;

package/src/renderer/selection-modal-overlay.ts

@@ -88,7 +88,7 @@ function frameRowsFor(modal: SelectionModal): number {
 }
 
 /**
- * Physical rows every row of the list needs, category headers included — the
+ * Physical rows every row of the list needs, category headers included, the
  * height the box would have to give the list for nothing to be scrolled away.
  */
 function totalContentRowsFor(modal: SelectionModal, remaining: number): number {
@@ -124,7 +124,7 @@ export function renderSelectionModalOverlay(
   // Size to content before falling back to the viewport's height ratio. When
   // every row (and its wrapped detail) fits in the space the terminal
   // actually has, ask for exactly that many rows, so a short list is never
-  // split behind a "(n below)" hint with screen still empty below the box —
+  // split behind a "(n below)" hint with screen still empty below the box,
   // a two-item question in particular must always show both its answers. A
   // list genuinely taller than the terminal keeps the ratio-derived budget
   // and scrolls as before.
@@ -142,7 +142,7 @@ export function renderSelectionModalOverlay(
     // Modals size to content: items whose detail wraps onto extra lines can
     // need more physical rows than a plain one-row-per-item modal, so this
     // ceiling grows with the viewport rather than staying pinned at a small
-    // constant (still bounded — never larger than the terminal can show).
+    // constant (still bounded, never larger than the terminal can show).
     maxContentRows: Math.max(10, viewportHeight - chromeRows - 4),
     // A floor only. The shared budget keeps its own target and ceiling, so a
     // modal whose content is shorter than that target is unaffected: this can
@@ -208,7 +208,7 @@ export function renderSelectionModalOverlay(
     // visible window never silently shows fewer full items than the budget
     // allows just because a fixed item-count cap ran out. Always shows the
     // selected item (even alone, if its wrapped detail is taller than the
-    // whole budget — descriptive text is never clipped to make it fit),
+    // whole budget, descriptive text is never clipped to make it fit),
     // then grows outward, roughly centered, until the budget is spent.
     const selected = Math.max(0, Math.min(modal.selectedIndex, items.length - 1));
     let startIdx = selected;
@@ -278,7 +278,7 @@ export function renderSelectionModalOverlay(
       });
       if (item.detail) {
         if (detailLayout.wrappedDetail === null) {
-          // Fits beside the label at a readable width — no truncation needed.
+          // Fits beside the label at a readable width, no truncation needed.
           putText(labelLine, layout.margin + 2 + INDICATOR_WIDTH + labelWidth, 2, '  ', {
             fg: BODY_FG,
             bg: isSelected ? SELECTED_BG : DEFAULT_OVERLAY_PALETTE.bodyBg,
@@ -289,7 +289,7 @@ export function renderSelectionModalOverlay(
           });
           lines.push(labelLine);
         } else {
-          // Doesn't fit beside the label — wrap the FULL detail text onto as
+          // Doesn't fit beside the label, wrap the FULL detail text onto as
           // many lines as it needs below the label, rather than clipping it.
           lines.push(labelLine);
           for (const detailLineText of detailLayout.wrappedDetail) {

package/src/renderer/semantic-diff.ts

@@ -1,5 +1,5 @@
 /**
- * semantic-diff.ts — Functional change summary after file edits.
+ * semantic-diff.ts, Functional change summary after file edits.
  *
  * Compares AST before/after an edit and extracts:
  *   - Added / removed / modified functions, methods, classes, etc.
@@ -8,7 +8,7 @@
  *
  * Returns a compact SemanticDiff that can be rendered alongside a regular diff.
  * Gracefully returns null when tree-sitter is unavailable or the language is
- * unsupported — callers should treat null as "no semantic info available".
+ * unsupported, callers should treat null as "no semantic info available".
  */
 
 import { TreeSitterService } from '@pellux/goodvibes-sdk/platform/intelligence';
@@ -28,9 +28,9 @@ export interface SymbolChange {
   kind: ChangeKind;
   symbolKind: SymbolInfo['kind'];
   name: string;
-  /** Present for 'modified' — the old signature. */
+  /** Present for 'modified', the old signature. */
   oldSignature?: string;
-  /** Present for 'added' or 'modified' — the new signature. */
+  /** Present for 'added' or 'modified', the new signature. */
   newSignature?: string;
 }
 
@@ -49,7 +49,7 @@ export interface SemanticDiff {
 }
 
 // ---------------------------------------------------------------------------
-// Import extraction (regex-based — fast, no grammar needed)
+// Import extraction (regex-based, fast, no grammar needed)
 // ---------------------------------------------------------------------------
 
 interface ParsedImport {
@@ -179,7 +179,6 @@ function diffImports(
     if (!aImp) {
       changes.push({ kind: 'removed', specifier: spec, names: bImp.names });
     } else {
-      // Check if named imports changed
       const bSet = new Set(bImp.names);
       const aSet = new Set(aImp.names);
       const added = aImp.names.filter(n => !bSet.has(n));
@@ -231,7 +230,7 @@ export async function computeSemanticDiff(
 
   const langId = detectLanguage(filePath);
   if (!langId) {
-    // No language support — return import changes only if any
+    // No language support, return import changes only if any
     if (imports.length === 0) return null;
     return { symbols: [], imports, totalChanges: imports.length };
   }