@pellux/goodvibes-tui 2.0.13 → 2.0.15

package/src/core/conversation-fold.ts

@@ -1,11 +1,11 @@
 /**
- * conversation-fold.ts — which transcript rows render FOLDED, and what spacing
+ * conversation-fold.ts, which transcript rows render FOLDED, and what spacing
  * that earns them.
  *
  * A folded tool result is exactly ONE row: its header, with the preview riding
- * on the same line after the `▸ N lines` badge. It used to be four rows — a
+ * on the same line after the `▸ N lines` badge. It used to be four rows, a
  * `▄▄▄` cap, an interior preview line carrying a second `[▸ N hidden]` count, a
- * `▀▀▀` cap, and a blank after all of it — for one line of text the header had
+ * `▀▀▀` cap, and a blank after all of it, for one line of text the header had
  * already sized.
  *
  * These predicates live in their own leaf module because TWO callers must agree
@@ -13,7 +13,7 @@
  * (conversation-rendering.ts) decides how to draw the row, and the line cache
  * (conversation-line-cache.ts) decides the blank separator that follows it. If
  * those two ever disagreed about which rows are folded, a warm cache and a cold
- * rebuild would space the transcript differently — the exact class of bug the
+ * rebuild would space the transcript differently, the exact class of bug the
  * cache's byte-identical contract exists to rule out.
  *
  * Everything here is PURE: collapse state is read, never written, so a row can
@@ -30,8 +30,8 @@ import type { ConversationMessageSnapshot } from '@pellux/goodvibes-sdk/platform
 type FoldContext = Pick<ConversationRenderContext, 'assistantTurns' | 'collapseState'>;
 
 /**
- * Does a tool-result row render FOLDED — one compact header row with its preview
- * on it — rather than header-plus-expanded-body?
+ * Does a tool-result row render FOLDED, one compact header row with its preview
+ * on it, rather than header-plus-expanded-body?
  *
  * An unset collapse key answers the same as the default the row itself will
  * store (collapsed), so asking before and asking after the row renders give the
@@ -42,7 +42,7 @@ export function isToolResultFolded(
   collapseState: ReadonlyMap<string, boolean>,
   collapseKey: string,
 ): boolean {
-  // Threading only — the decision, the short-content threshold and the
+  // Threading only, the decision, the short-content threshold and the
   // unset-key default all belong to foldedToolResult(). summarizeToolResult is
   // this product's own renderer concern, so resolving it stays here: a
   // summarizable result stays folded even when short, because the one-line
@@ -77,7 +77,7 @@ function rendersFoldedToolRow(node: RenderNode, context: FoldContext): boolean {
  * The blank separator that follows a planned row.
  *
  * Branch rows sit tight under their parent, so the blank lands only after the
- * last row of a top-level unit — that is what keeps a turn's whole subtree
+ * last row of a top-level unit, that is what keeps a turn's whole subtree
  * reading as one block. On top of that, a folded row followed by more tool
  * machinery gets NO blank at all: N consecutive folded results stack as N
  * adjacent single rows. A folded row followed by anything that is not tool
@@ -89,7 +89,7 @@ export function trailingBlankAfter(
   next: RenderNode | undefined,
   context: FoldContext,
 ): boolean {
-  // Adaptation only — RenderNode is this product's shape, so reading depth and
+  // Adaptation only, RenderNode is this product's shape, so reading depth and
   // node kind off it stays here; the rule those facts feed belongs to
   // trailingBlankAfterRow(). No `next` at all means end of transcript: neither
   // a branch row nor tool machinery follows, which is the plain-blank case.

package/src/core/conversation-line-cache.ts

@@ -1,11 +1,11 @@
 /**
- * conversation-line-cache.ts — per-message Line[] production cache.
+ * conversation-line-cache.ts, per-message Line[] production cache.
  *
  * The measured defect (perf baseline 2026-07-03, transcript.build_1k): appending
  * ONE message to an N-message conversation re-rendered all N messages
  * (45.5 ms p50 / 71.4 MB / 695 k objects per rebuild), because
  * ConversationManager.rebuildHistory() clears the buffer and calls
- * appendConversationMessages() over the entire snapshot on every dirty flag —
+ * appendConversationMessages() over the entire snapshot on every dirty flag,
  * and markDirty() fires on every mutation. The marginal work for one appended
  * message is ~1/1000th of that.
  *
@@ -13,33 +13,33 @@
  * a PURE function of its complete inputs:
  *   - message identity + content (via a content signature; snapshots are fresh
  *     structuredClone copies each call, so string content compares by value and
- *     array-valued fields — user ContentPart[] and assistant toolCalls — are
+ *     array-valued fields, user ContentPart[] and assistant toolCalls, are
  *     serialised; a streaming assistant message mutates its content string in
  *     place, which changes the signature and invalidates the entry)
  *   - render width
  *   - the four display-config values the render reads (line-number mode,
  *     collapse threshold, showThinking, showReasoningSummary)
  *   - the block-registry base at message start (the code-block collapseKey embeds
- *     the GLOBAL block index — `code_${msgIdx}_${blockIdx}` — so a shift in an
+ *     the GLOBAL block index, `code_${msgIdx}_${blockIdx}`, so a shift in an
  *     earlier message's block count changes this message's keys)
  *   - the absolute message index (embedded in every collapseKey and used for the
  *     system-message kind lookup)
  *   - the system-message kind (drives the error-navigation registry side effect)
  *   - this message's tool-group membership (see conversation-tool-groups.ts):
  *     whether it's folded under a group header, which member owns that header,
- *     and the group's honest tool/line counts — a group growing as new tool
+ *     and the group's honest tool/line counts, a group growing as new tool
  *     results stream in changes these for every existing member
  *   - the live values of every collapseState key the render READS (recorded via a
  *     proxy during a miss; a collapse toggle flips a recorded value and
  *     invalidates exactly the owning message)
  *
- * Invalidation is by comparison of those complete inputs — the file-preview
+ * Invalidation is by comparison of those complete inputs, the file-preview
  * contentVersion precedent generalised: instead of a single version counter we
  * compare the full input tuple, which is provably complete (a from-scratch
  * rebuild reads nothing else).
  *
  * Correctness contract: a cache-served rebuild is BYTE-IDENTICAL to a cold
- * appendConversationMessages() rebuild. This is guaranteed by construction — a
+ * appendConversationMessages() rebuild. This is guaranteed by construction, a
  * miss renders through the exact same per-message render functions into an
  * isolated scratch context, and the captured lines / block metas / error lines
  * are replayed at the same buffer offsets a cold render would have produced.
@@ -111,7 +111,7 @@ interface KeyMeta {
   readonly kind: SystemMessageKind | undefined;
   /**
    * Per-call completion signature for an assistant message's tool calls (see
-   * pendingToolKeyOf) — undefined when the message has none. Each call's id
+   * pendingToolKeyOf), undefined when the message has none. Each call's id
    * is paired with whether it has a matching result yet, in call order, so a
    * SINGLE call finishing invalidates and re-renders that call's glyph from
    * ◌ to ✓ immediately. A prior aggregate boolean only flipped once EVERY
@@ -134,7 +134,7 @@ interface KeyMeta {
   readonly turnToolCount: number;
   readonly turnSharedLabel: string | undefined;
   readonly turnHasReasoning: boolean;
-  /** Row depth and connector — both change when structure around a row changes. */
+  /** Row depth and connector, both change when structure around a row changes. */
   readonly depth: number;
   readonly connector: string | undefined;
   /** Ancestor gutter pattern, flattened; a change repaints the row's gutters. */
@@ -145,7 +145,7 @@ interface KeyMeta {
 
 /**
  * Per-call completion signature for an assistant message's tool calls, in
- * call order — `id:0` or `id:1` per call, joined. Undefined for a message
+ * call order, `id:0` or `id:1` per call, joined. Undefined for a message
  * with no tool calls at all (nothing to key). Comparing this instead of an
  * aggregate boolean lets exactly the calls whose completion status changed
  * invalidate the entry, rather than waiting for every call in the message to
@@ -172,7 +172,7 @@ interface CacheEntry {
   /** [collapseKey, value] pairs the render read; a change invalidates the entry. */
   readonly collapseDeps: Array<[string, boolean | undefined]>;
   /** Memoised rebase of blocks/errors at appliedBase (avoids realloc when the
-   *  message's buffer offset is unchanged across rebuilds — the common case). */
+   *  message's buffer offset is unchanged across rebuilds, the common case). */
   appliedBase: number;
   appliedBlocks: BlockMeta[] | null;
   appliedErrors: number[] | null;
@@ -237,7 +237,7 @@ function contentUnchanged(sig: ContentSig, m: Message): boolean {
 
 /**
  * Wrap the real collapseState so reads (.get/.has) are recorded while writes
- * (.set — the auto-collapse default) pass straight through to the real map, so
+ * (.set, the auto-collapse default) pass straight through to the real map, so
  * the persistent collapse defaults are established exactly as a cold render
  * would establish them.
  */
@@ -290,7 +290,7 @@ function renderOne(
 }
 
 /**
- * MessageLineCache — per-message Line[] memoisation for ConversationManager.
+ * MessageLineCache, per-message Line[] memoisation for ConversationManager.
  *
  * Keyed by absolute message index; each entry validates its COMPLETE input tuple
  * before serving. A rebuild that reuses entries is byte-identical to a cold
@@ -538,7 +538,7 @@ export class MessageLineCache {
 
   /**
    * Replay an entry into the live context at buffer offset `base`. Line objects
-   * are shared (never mutated post-production — the compositor reads them into a
+   * are shared (never mutated post-production, the compositor reads them into a
    * separate back-buffer). Block/error rebasing is memoised per base so an
    * unchanged message pays zero allocation across rebuilds.
    */

package/src/core/conversation-render-context.ts

@@ -1,5 +1,5 @@
 /**
- * conversation-render-context.ts — the inputs a transcript row render reads,
+ * conversation-render-context.ts, the inputs a transcript row render reads,
  * and the small pure derivations over them.
  *
  * Kept apart from conversation-rendering.ts (which draws) so that both the
@@ -25,8 +25,8 @@ type Message = ConversationMessageSnapshot;
  *
  * The transcript stores a failure as content leading with `Error: ` (see the
  * SDK's ConversationManager.addToolResults), and a per-call user cancellation
- * as the more specific `Error: cancelled by user`. Reading the outcome —
- * rather than only "did a result arrive" — is what lets the CALL row show
+ * as the more specific `Error: cancelled by user`. Reading the outcome,
+ * rather than only "did a result arrive", is what lets the CALL row show
  * ✓ / ✕ / ⊘ honestly instead of a ✓ that means nothing more than "it finished".
  */
 export type ToolCallOutcome = 'ok' | 'error' | 'cancelled';
@@ -38,7 +38,7 @@ function outcomeOfToolContent(content: string): ToolCallOutcome {
 }
 
 /**
- * Collect, per tool-call id, how that call settled — for the calls that have a
+ * Collect, per tool-call id, how that call settled, for the calls that have a
  * matching tool-result message in the given slice. A call absent from the map
  * has not run yet (e.g. it is still awaiting an approval decision) and renders
  * with the pending glyph. (item 2c.)
@@ -54,7 +54,7 @@ export function collectToolCallOutcomes(messages: readonly Message[]): Map<strin
 }
 
 /**
- * Ids of tool calls that have a matching tool-result message — i.e. the tools
+ * Ids of tool calls that have a matching tool-result message, i.e. the tools
  * that actually ran, regardless of how they settled. Derived from
  * collectToolCallOutcomes so the two can never disagree about what "ran" means.
  */
@@ -80,7 +80,7 @@ export interface ConversationRenderContext {
    * actually ran). An assistant tool call whose id is NOT in this set is still
    * awaiting a decision (e.g. an approval prompt) and renders with a pending
    * glyph instead of the completed ✓. When undefined (single-message callers
-   * without sibling context), every tool call renders as done — the prior
+   * without sibling context), every tool call renders as done, the prior
    * behaviour. (item 2c.)
    */
   readonly completedToolCallIds?: ReadonlySet<string>;

package/src/core/conversation-rendering.ts

@@ -21,7 +21,7 @@ import {
   type ConversationRenderContext,
 } from './conversation-render-context.ts';
 import { renderCompactionContinuationMessage } from './conversation-compaction-render.ts';
-// Which rows fold, and the blank separator that earns — shared verbatim with
+// Which rows fold, and the blank separator that earns, shared verbatim with
 // conversation-line-cache.ts so a warm cache and a cold rebuild space alike.
 import { isToolResultFolded, trailingBlankAfter } from './conversation-fold.ts';
 import { drawTreeRails, treeIndentCols, treeTextCol } from '@pellux/goodvibes-terminal-shell';
@@ -48,10 +48,10 @@ import type { SystemMessageKind } from '@/runtime/index.ts';
  * Navigable system message kinds for error-navigation (nextErrorLine/prevErrorLine).
  *
  * Kind → navigable mapping:
- *   - 'system'      YES — generic/catch-all messages (provider failures, session
+ *   - 'system'      YES, generic/catch-all messages (provider failures, session
  *                         events, user-visible errors). Default for un-prefixed messages.
- *   - 'wrfc'        YES — WRFC chain events are important and worth navigating to.
- *   - 'operational' NO  — tool/scan/plugin/MCP status noise; not useful to jump to.
+ *   - 'wrfc'        YES, WRFC chain events are important and worth navigating to.
+ *   - 'operational' NO , tool/scan/plugin/MCP status noise; not useful to jump to.
  *
  * When a message has no recorded kind (added via bare addSystemMessage), it
  * defaults to 'system' and is therefore navigable.
@@ -90,7 +90,7 @@ export function renderConversationUserMessage(
   }
   // Compaction-continuation handoff: a user-ROLE message the compactor
   // authored, not something the user typed. Rendered in full it repeats the
-  // entire re-injected instruction block after every automatic compaction —
+  // entire re-injected instruction block after every automatic compaction,
   // a multi-kilobyte wall in the transcript. Fold it like a tool result; the
   // full payload stays reachable through the normal expand toggle.
   if (msgIdx !== undefined && displayText.startsWith(COMPACTION_HANDOFF_HEADER)) {
@@ -120,19 +120,19 @@ export function renderConversationAssistantMessage(
     if (message.model) {
       assistantHeaderDetails.push({ text: ` ${message.model}${message.provider ? ` (${message.provider})` : ''} `, fg: T.modelNameDim, dim: true });
     }
-    // The count spans the whole run, not just this message — that is the point
+    // The count spans the whole run, not just this message, that is the point
     // of merging. `tools:1` repeated over five headers becomes one `5 tools`.
     const toolCount = turn?.toolCallCount ?? message.toolCalls?.length ?? 0;
     if (toolCount > 0) {
       assistantHeaderDetails.push({ text: ` ${GLYPHS.status.pending} ${toolCount} tool${toolCount === 1 ? '' : 's'} `, fg: T.toolAccent });
     }
     // A label every call in the run shares belongs here, once, instead of on
-    // every branch — the branches then lead with what distinguishes them.
+    // every branch, the branches then lead with what distinguishes them.
     //
     // Hoisted ONLY when it fits whole. A label chopped mid-word ("Calling the
     // assistant serv") is worse than no label: it costs a third of the header
     // and tells you less than the rows below it already do, since each row
-    // leads with its own distinguishing token. So it either fits or it goes —
+    // leads with its own distinguishing token. So it either fits or it goes,
     // never a truncated stub. Dropping it loses nothing reachable: the rows
     // still name every call, and omitToolName is unaffected either way.
     if (turn?.sharedToolLabel) {
@@ -153,7 +153,7 @@ export function renderConversationAssistantMessage(
     if (turnCollapsed && toolCount > 0) {
       assistantHeaderDetails.push({ text: ` ${GLYPHS.navigation.collapsed} hidden `, fg: '244', dim: true });
     }
-    // An empty run — no model, no tools, no reasoning — emits no header rather
+    // An empty run, no model, no tools, no reasoning, emits no header rather
     // than a bare `● assistant` with nothing under it.
     if (assistantHeaderDetails.length > 0) {
       const headerStartLine = context.history.getLineCount();
@@ -166,7 +166,7 @@ export function renderConversationAssistantMessage(
         detailFg: '244',
       }, assistantHeaderDetails));
 
-      // Turns are collapsible as a unit and default to EXPANDED — a turn
+      // Turns are collapsible as a unit and default to EXPANDED, a turn
       // collapsed by default would hide the activity the transcript exists to
       // show. Registered only when there is machinery to hide.
       if (turn && toolCount > 0) {
@@ -179,7 +179,7 @@ export function renderConversationAssistantMessage(
           type: 'assistant_turn',
           startLine: headerStartLine,
           lineCount: 1,
-          rawContent: `assistant turn — ${toolCount} tool call${toolCount === 1 ? '' : 's'}${turn.sharedToolLabel ? ` (${turn.sharedToolLabel})` : ''}`,
+          rawContent: `assistant turn: ${toolCount} tool call${toolCount === 1 ? '' : 's'}${turn.sharedToolLabel ? ` (${turn.sharedToolLabel})` : ''}`,
           groupMemberIndexes: turn.resultIndexes,
           toolName: turn.sharedToolLabel,
         });
@@ -201,7 +201,7 @@ export function renderConversationAssistantMessage(
     const thinkingStartLine = context.history.getLineCount();
     const thinkingBlockIdx = context.blockRegistry.length;
     const thinkingCollapseKey = `msg_${msgIdx}_thinking`;
-    // Collapsed by default, like every other collapsible block — Tab (or the
+    // Collapsed by default, like every other collapsible block, Tab (or the
     // block-actions menu) expands it via the same collapseKey the BlockMeta
     // below registers. Previously this key was registered but never
     // consulted: the toggle existed but did nothing.
@@ -215,7 +215,7 @@ export function renderConversationAssistantMessage(
       // One row: label plus size badge, no frame and no interior line.
       //
       // Deliberately NO preview, unlike a folded tool result. Reasoning text
-      // stays behind the toggle until it is asked for — the fold is the quiet
+      // stays behind the toggle until it is asked for, the fold is the quiet
       // state of an already-opt-in display, and previewing the first reasoning
       // line here would disclose content the collapsed form has never shown.
       const thinkingLineCount = message.reasoningContent.split('\n').length;
@@ -255,7 +255,7 @@ export function renderConversationAssistantMessage(
 }
 
 /**
- * Assistant prose — the model's actual answer — at full prominence and full
+ * Assistant prose, the model's actual answer, at full prominence and full
  * width.
  *
  * Prose is never drawn as a tree branch. It is what closes a group (see
@@ -274,9 +274,6 @@ function renderAssistantProse(
   {
     const showAllLineNumbers = lineNumberMode === 'all';
     const showCodeBlockLineNumbers = lineNumberMode === 'all' ? false : lineNumberMode === 'code';
-    // First pass: measure totalLines for gutter sizing (only when line-numbers='all').
-    // When line numbers are off, skip the measurement pass entirely.
-    //
     // NOTE: The 'all' mode intentionally calls renderMarkdownTracked twice:
     //   1. Measure pass: render at full `width` to get the total line count, which
     //      determines `numWidth` (digit count) and thus `gutterW` (gutter column width).
@@ -365,7 +362,7 @@ export function renderConversationSystemMessage(
  *
  * The call row is its own plan node rather than part of the assistant
  * message's render, which is what lets a result be interleaved BETWEEN two
- * calls of the same message — the ordering guarantee that makes a
+ * calls of the same message, the ordering guarantee that makes a
  * late-finishing call's result land inside its own subtree instead of after a
  * call issued later.
  */
@@ -409,7 +406,7 @@ export function renderConversationToolCallNode(
     { indentCols: indent, omitToolName: turn?.sharedToolLabel !== undefined },
   );
 
-  // Recursion stopped here — say so rather than silently showing a subtree as
+  // Recursion stopped here, say so rather than silently showing a subtree as
   // if it were a leaf.
   if (node.truncated) {
     const noteIndent = treeIndentCols(node.depth + 1, width);
@@ -477,15 +474,15 @@ export function renderConversationToolMessage(
   const inTree = depth > 0 && indent > 0;
   // The EXPANDED render, computed unconditionally (even while collapsed) so
   // the header's "N lines" badge always names what Tab would actually reveal
-  // — a raw JSON blob that pretty-prints to 50 lines must say 50, not 1, even
+  //, a raw JSON blob that pretty-prints to 50 lines must say 50, not 1, even
   // while it's still folded. Reused below as the actual body when expanded,
   // so this is never rendered twice.
   //
   // In the tree it is rendered at the width it will actually OCCUPY (the row's
   // width less its indent) and then shifted into place, so the badge still
   // counts exactly the lines expansion produces. Rendering it full-width and
-  // displaying it indented would desynchronise the two — the
-  // markdown-table-drops-columns bug class — and rendering it full-width and
+  // displaying it indented would desynchronise the two, the
+  // markdown-table-drops-columns bug class, and rendering it full-width and
   // displaying it flush punches the row's rails out for the whole body.
   // The body is a continuation of its row, so it starts at the row's own text
   // column; the shift is that column expressed relative to the flush left
@@ -516,15 +513,15 @@ export function renderConversationToolMessage(
   // A per-call user cancellation settles as a tool result whose content leads
   // with "Error: cancelled by user" (the SDK's structured cancelled shape; any
   // partial output the tool produced before it stopped follows on later lines).
-  // Render it structurally as a distinct "cancelled" block in the warn tone —
-  // not a generic tool result, and not a hard error — so the transcript reads
+  // Render it structurally as a distinct "cancelled" block in the warn tone,
+  // not a generic tool result, and not a hard error, so the transcript reads
   // the user's decision honestly while the partial output stays visible below.
   const isCancelled = blockType === 'tool' && /^Error: cancelled by user\b/.test(contentLines[0] ?? '');
   const warnTone = activeUiTones().chrome.warn;
 
   // In the tree a result hangs under the call that produced it, so repeating
-  // the generic label ("tool result") and the tool's own name — which the
-  // parent call row already shows — is exactly the boilerplate this layout
+  // the generic label ("tool result") and the tool's own name, which the
+  // parent call row already shows, is exactly the boilerplate this layout
   // removes. The row leads with its size badge instead. `diff` and `cancelled`
   // keep their labels: those carry information the parent row does not.
   const label = isCancelled ? 'cancelled' : (blockType === 'diff' ? 'diff' : (inTree ? '' : 'tool result'));
@@ -536,7 +533,7 @@ export function renderConversationToolMessage(
 
   // A FOLDED result is exactly one row: this header, with its preview riding on
   // the same line right after the `▸ N lines` badge (see
-  // renderConversationFoldedRow). The badge is the count — the old fold also
+  // renderConversationFoldedRow). The badge is the count, the old fold also
   // carried a separate `[▸ N hidden]` marker, which restated it.
   const headerTone = {
     marker: isCancelled ? GLYPHS.status.blocked : (blockType === 'diff' ? GLYPHS.status.dualPane : GLYPHS.status.active),
@@ -560,14 +557,14 @@ export function renderConversationToolMessage(
   const rows: Line[] = [headerLine];
 
   if (!isCollapsed) {
-    // Diff or plain result — either way this is exactly `expandedLines`,
+    // Diff or plain result, either way this is exactly `expandedLines`,
     // already computed above (and already what the header's line count is
     // honest about), so there is nothing left to render here.
     //
     // Deliberately NOT indented. renderExpandedToolResultLines is the single
     // source of truth for the "N lines" badge above; re-wrapping it at a
     // reduced width would desynchronise that count from what expansion
-    // actually reveals — the markdown-table-drops-columns bug class. The body
+    // actually reveals, the markdown-table-drops-columns bug class. The body
     // is already visually bound to its row by the header directly above it.
     // In the tree it was rendered at `bodyWidth` above and is shifted into the
     // row's indent here, which is what leaves the rail columns to its left free
@@ -671,7 +668,7 @@ export function appendConversationMessages(
     // Branch rows sit tight under their parent; the blank separator lands only
     // after the last row of a top-level unit, which is what keeps a turn's
     // whole subtree reading as one block instead of a run of spaced-out rows.
-    // A folded result followed by more tool machinery gets no blank at all —
+    // A folded result followed by more tool machinery gets no blank at all,
     // consecutive folded results stack as adjacent single rows.
     if (trailingBlankAfter(node, plan[i + 1], turnContext)) {
       turnContext.history.addLine(createEmptyLine(width));

package/src/core/conversation-search-expansion.ts

@@ -1,5 +1,5 @@
 /**
- * conversation-search-expansion.ts — tracks which collapse keys transcript
+ * conversation-search-expansion.ts, tracks which collapse keys transcript
  * search auto-expanded so they can be restored on search close, unless the
  * user explicitly acted on the block while it was open.
  *
@@ -11,14 +11,14 @@
  *
  * The contract (see SearchManager.revealCurrentMatch and .close in
  * search.ts): a single keystroke in the search field must never expand
- * collapsed content — only navigating TO a match hidden inside it does, and
+ * collapsed content, only navigating TO a match hidden inside it does, and
  * that expansion is undone when search closes UNLESS the user separately
  * touched the block (toggled, copied, bookmarked, saved, or bulk-expanded
  * via /expand) while it sat auto-expanded. An explicit user action always
  * wins over search's own bookkeeping, regardless of which happened first.
  */
 
-/** Structural shape restoreOnto() needs — matches ConversationManager's own
+/** Structural shape restoreOnto() needs, matches ConversationManager's own
  *  public setCollapsed() without importing the class (avoids a cycle). */
 export interface CollapseSettable {
   setCollapsed(collapseKey: string, collapsed: boolean): void;
@@ -28,13 +28,13 @@ export class SearchExpansionTracker {
   /**
    * Collapse keys currently expanded because the user navigated to a search
    * match hidden inside them. Never contains a key the user has touched
-   * explicitly — noteUserTouch() removes it on the way in.
+   * explicitly, noteUserTouch() removes it on the way in.
    */
   private searchExpandedKeys = new Set<string>();
 
   /**
    * Collapse keys the user has explicitly acted on at least once. Growth is
-   * unbounded for the life of the conversation — membership only ever gates
+   * unbounded for the life of the conversation, membership only ever gates
    * one decision (whether restoreSearchExpansions() may re-collapse a key),
    * so a stale positive is harmless and pruning would add complexity for no
    * behavioral gain.
@@ -57,7 +57,7 @@ export class SearchExpansionTracker {
   }
 
   /** Re-collapse every key search auto-expanded and not since user-touched,
-   *  through `conversationManager`'s public setCollapsed() — called from
+   *  through `conversationManager`'s public setCollapsed(), called from
    *  SearchManager.close() to restore pre-search collapse state. */
   restoreOnto(conversationManager: CollapseSettable): void {
     const keys = [...this.searchExpandedKeys];

package/src/core/conversation-splash-state.ts

@@ -1,18 +1,18 @@
 /**
- * conversation-splash-state.ts — when the splash owns the conversation area,
+ * conversation-splash-state.ts, when the splash owns the conversation area,
  * and what has to happen the moment it stops.
  *
  * Two pieces of state, kept out of ConversationManager (core/conversation.ts,
  * already at its line-count gate) and out of `suppressSplash`, which is a
  * per-frame posture the panel workspace sets both ways on every render:
  *
- *   - `dismissed` — sticky for the run. Owner rule: the splash yields to ANY
+ *   - `dismissed`, sticky for the run. Owner rule: the splash yields to ANY
  *     submission, a chat message or a slash command alike. A slash command
  *     whose whole output is a modal (or nothing) still means the session has
  *     started, so the splash must not reappear underneath it, and must not
  *     come back in the gap before the first reply arrives.
  *
- *   - the splash→transcript EDGE — latched when a rebuild that would have
+ *   - the splash→transcript EDGE, latched when a rebuild that would have
  *     drawn the splash draws transcript content instead. The shell consumes
  *     it to repaint the whole viewport for that one frame. Without that, the
  *     differential renderer only rewrites rows it knows changed, so every row

package/src/core/conversation-turn-structure.ts

@@ -1,5 +1,5 @@
 /**
- * conversation-turn-structure.ts — turns the flat message array into the
+ * conversation-turn-structure.ts, turns the flat message array into the
  * structural render plan the transcript tree draws from.
  *
  * Three jobs, deliberately not conflated:
@@ -8,7 +8,7 @@
  *     header (see computeAssistantTurns).
  *
  *  2. **Structural placement.** Where each row goes. A tool result is a child
- *     of the CALL that produced it, identified by callId — not a row appended
+ *     of the CALL that produced it, identified by callId, not a row appended
  *     wherever it happens to land in the message array. Two calls issued
  *     together run concurrently and settle in whatever order they finish; the
  *     later-finishing call's result must still render inside its own call's
@@ -20,7 +20,7 @@
  *
  * ## Node identity
  *
- * Every planned row carries a stable string id — `m:<absIdx>` for a message,
+ * Every planned row carries a stable string id, `m:<absIdx>` for a message,
  * `c:<absIdx>:<callIndex>` for one tool call, each prefixed by its scope (see
  * nesting below). Ids derive from append-only indices that are never
  * renumbered, so a row inserted in the middle does not change any other row's
@@ -34,7 +34,7 @@
  * `AgentManager.getConversationSnapshot(agentId)` already exposes it live (the
  * same read `fleet-panel.ts` uses to draw an attached agent's transcript). So
  * when a tool call spawned an agent, the plan SPLICES that agent's own plan in
- * beneath the call, recursively — the identical operation to lifting a result
+ * beneath the call, recursively, the identical operation to lifting a result
  * into its call's subtree, applied to a nested snapshot. The parent
  * transcript, the persisted format, and the message schema are untouched; only
  * the render plan nests.
@@ -52,15 +52,15 @@
  *
  * Proper box drawing: `├` for a branch with siblings below it, `└` for the
  * last sibling, `│` carried down the gutter for every ancestor whose subtree
- * is still open. Connectors are a PURE FUNCTION of current structure —
- * recomputed from scratch on every plan build, never cached — so a row that
+ * is still open. Connectors are a PURE FUNCTION of current structure,
+ * recomputed from scratch on every plan build, never cached, so a row that
  * stops being last flips `└`→`├` on the next rebuild.
  *
  * That flip is one cell. Indentation width, content column, and row text are
  * all independent of sibling status by construction: `indentCols` depends only
  * on depth and width, and the connector occupies a column that exists whether
  * it holds `├` or `└`. So a sibling arriving repaints a connector cell and
- * moves no text. (The rejected alternative — a uniform glyph — cannot express
+ * moves no text. (The rejected alternative, a uniform glyph, cannot express
  * "this subtree continues past here", which is what makes a deep tree
  * scannable.)
  */
@@ -80,12 +80,12 @@ export const MAX_NEST_DEPTH = 8;
 export type TreeConnector = '├' | '└';
 
 export interface RenderNode {
-  /** Stable identity — survives insertions above it. Scope-prefixed. */
+  /** Stable identity, survives insertions above it. Scope-prefixed. */
   readonly id: string;
   readonly kind: 'message' | 'toolcall';
   /** Index of the message within ITS OWN snapshot array. */
   readonly absIdx: number;
-  /** Position within the owning assistant message's toolCalls — 'toolcall' only. */
+  /** Position within the owning assistant message's toolCalls, 'toolcall' only. */
   readonly callIndex?: number;
   /** 0 = flush; >=1 = branch depth. */
   readonly depth: number;
@@ -106,7 +106,7 @@ export interface RenderNode {
    * renderer draws `│` in those gutters. Excludes this row's own depth.
    */
   readonly openAncestorDepths: readonly number[];
-  /** Set when recursion stopped here (depth ceiling or cycle) — surfaced honestly. */
+  /** Set when recursion stopped here (depth ceiling or cycle), surfaced honestly. */
   readonly truncated?: 'depth' | 'cycle';
 }
 
@@ -233,7 +233,7 @@ export interface RenderPlanOptions {
  * The agent id a tool call spawned, when it spawned one.
  *
  * Read from the call's own result payload first (the spawn tool reports the id
- * it allocated — see AgentInput.agentId in the SDK's agent schema), falling
+ * it allocated, see AgentInput.agentId in the SDK's agent schema), falling
  * back to an explicit `agentId` argument on the call. Returns undefined for
  * every ordinary tool call, which is the overwhelmingly common case, so this
  * stays a cheap check.
@@ -253,7 +253,7 @@ function spawnedAgentIdOf(
         if (typeof id === 'string' && id.length > 0) return id;
       }
     } catch {
-      // Non-JSON result — fall through to the argument check.
+      // Non-JSON result, fall through to the argument check.
     }
   }
   const argId = call.arguments.agentId;
@@ -301,7 +301,7 @@ function buildScopeNodes(
   const placed = new Set<number>();
   const messageAt = (absIdx: number): Message => messages[absIdx - indexOffset]!;
   // Calls issued by later messages of the SAME run attach to the run's head
-  // node, because the head owns the only visible header — that is what makes
+  // node, because the head owns the only visible header, that is what makes
   // them siblings of each other rather than only children of their own
   // (headerless) message, and therefore what makes ├/└ resolve correctly
   // across a merged turn.

package/src/core/conversation-types.ts

@@ -1,5 +1,5 @@
 /**
- * conversation-types.ts — shared TUI extension types for ConversationManager.
+ * conversation-types.ts, shared TUI extension types for ConversationManager.
  *
  * Extracted from conversation.ts so that conversation-rendering.ts can import
  * BlockMeta without creating a circular dependency:
@@ -17,7 +17,7 @@ import type { BlockMeta as SdkBlockMeta } from '@pellux/goodvibes-sdk/platform/c
  * a run of >=2 consecutive tool-result messages under one collapsible header
  * (see conversation-tool-groups.ts). Defined as an intersection rather than
  * `interface X extends SdkBlockMeta` because TypeScript requires an extending
- * interface's members to be subtypes of the base interface's — widening the
+ * interface's members to be subtypes of the base interface's, widening the
  * `type` union that way is a compile error. Omit + intersection adds the new
  * variant without touching the SDK's published type.
  */
@@ -42,7 +42,7 @@ export type BlockMeta = Omit<SdkBlockMeta, 'type'> & {
   /**
    * The tool call's name, when this block renders a 'tool' result (or is an
    * 'assistant_turn' header whose calls all share one label). Used to name the block honestly in action
-   * receipts (e.g. "tool result: exec") — see describeBlockForReceipt in
+   * receipts (e.g. "tool result: exec"), see describeBlockForReceipt in
    * handler-content-actions.ts. Undefined for non-tool block types and for
    * standalone tool results with no recorded name.
    */