@pellux/goodvibes-tui 2.0.14 → 2.0.16

package/src/panels/fleet-spawn.ts

@@ -2,14 +2,14 @@
 // fleet-spawn.ts
 //
 // The Fleet panel's ACP spawn affordance: 'n' on the fleet surface lists the
-// third-party coding agents the daemon discovered (acp.agents.list — read-only,
+// third-party coding agents the daemon discovered (acp.agents.list, read-only,
 // quiet when none), you pick one, then pick a working directory from the known
-// candidates (the registered workspaces + the current dir — no free-text path
+// candidates (the registered workspaces + the current dir, no free-text path
 // retyping where a known dir exists), and acp.sessions.create hosts it as a
 // long-lived daemon session. The new row shows up on the next fleet snapshot as
 // kind 'acp-agent' and steers/stops/flags-for-attention like any other row.
 //
-// A structured spawn failure ({binary, stage, message}) is surfaced verbatim —
+// A structured spawn failure ({binary, stage, message}) is surfaced verbatim,
 // never a hung row: the daemon bounds the handshake and returns a 'failed'
 // record, and this controller reports its three honest fields.
 //
@@ -34,7 +34,7 @@ import {
 
 /** One discovered third-party ACP agent (acp.agents.list). */
 export type AcpDiscoveredAgent = OperatorMethodOutput<'acp.agents.list'>['agents'][number];
-/** acp.sessions.create output — the hosted record + whether it started. */
+/** acp.sessions.create output, the hosted record + whether it started. */
 export type AcpSpawnResult = OperatorMethodOutput<'acp.sessions.create'>;
 /** One registered workspace root (workspaces.registrations.list). */
 export type AcpWorkspaceRegistration = OperatorMethodOutput<'workspaces.registrations.list'>['workspaces'][number];
@@ -47,13 +47,13 @@ export interface AcpDirCandidate {
 }
 
 /**
- * The async verb surface the spawn flow drives — real daemon round-trips in
+ * The async verb surface the spawn flow drives, real daemon round-trips in
  * production (createAcpSpawnGateway), a mocked shape in tests.
  */
 export interface AcpSpawnGateway {
   /** Discovered third-party agents (read-only; empty is a quiet, honest absence). */
   listAgents(): Promise<readonly AcpDiscoveredAgent[]>;
-  /** The registered workspace roots — the known-dir candidates for the picker. */
+  /** The registered workspace roots, the known-dir candidates for the picker. */
   listWorkspaces(): Promise<readonly AcpWorkspaceRegistration[]>;
   /** Host a discovered agent in a directory as a long-lived daemon session. */
   createSession(input: { readonly agentId: string; readonly cwd: string }): Promise<AcpSpawnResult>;
@@ -69,7 +69,7 @@ export interface AcpSpawnGatewayDeps {
 }
 
 /**
- * Build the live ACP spawn gateway over the generic operator invoke path — the
+ * Build the live ACP spawn gateway over the generic operator invoke path, the
  * same daemon resolution the fleet acts use. Honest unavailable reason when no
  * daemon is reachable, so the spawn key refuses cleanly instead of throwing.
  */
@@ -88,7 +88,7 @@ export function createAcpSpawnGateway(deps: AcpSpawnGatewayDeps): AcpSpawnGatewa
 export interface FleetSpawnDeps {
   /** Resolve a live gateway per action (a daemon that comes up mid-session is seen). */
   readonly resolveGateway: () => AcpSpawnGatewayResolution;
-  /** The current working directory — always a candidate dir (labeled "current directory"). */
+  /** The current working directory, always a candidate dir (labeled "current directory"). */
   readonly currentDirectory: () => string;
   /** Surface a result/receipt/error line to the operator (system message). */
   readonly notify: (message: string) => void;
@@ -204,10 +204,10 @@ export class FleetSpawn {
       const result = await gateway.createSession({ agentId, cwd });
       const hosted = result.hosted;
       if (hosted.error) {
-        // Structured failure rendered verbatim — never a hung row.
-        this.deps.notify(`[Fleet] Could not host ${hosted.title || agentId}: ${hosted.error.stage} stage failed for ${hosted.error.binary} — ${hosted.error.message}`);
+        // Structured failure rendered verbatim, never a hung row.
+        this.deps.notify(`[Fleet] Could not host ${hosted.title || agentId}: ${hosted.error.stage} stage failed for ${hosted.error.binary}; ${hosted.error.message}`);
       } else {
-        this.deps.notify(`[Fleet] Hosting ${hosted.title || agentId} in ${cwd} — it appears as an acp-agent row; steer and stop it like any agent.`);
+        this.deps.notify(`[Fleet] Hosting ${hosted.title || agentId} in ${cwd}; it appears as an acp-agent row; steer and stop it like any agent.`);
       }
     } catch (err) {
       this.deps.notify(`ACP session create failed: ${summarizeError(err)}`);
@@ -225,10 +225,10 @@ export class FleetSpawn {
     const lines: Line[] = [];
     if (this.creating) {
       lines.push(buildPanelLine(width, [[' Hosting the agent…', P.dim]]));
-      return buildPanelWorkspace(width, height, { title: 'Fleet — Host an agent', sections: [{ lines }], footerLines: [], palette: P });
+      return buildPanelWorkspace(width, height, { title: 'Fleet: Host an agent', sections: [{ lines }], footerLines: [], palette: P });
     }
     if (this.mode.step === 'agent') {
-      lines.push(buildPanelLine(width, [[' Host a third-party coding agent — pick one, then a directory.', P.dim]]));
+      lines.push(buildPanelLine(width, [[' Host a third-party coding agent; pick one, then a directory.', P.dim]]));
       lines.push(buildPanelLine(width, [['', P.dim]]));
       this.mode.agents.forEach((agent, i) => {
         const selected = i === (this.mode as AgentStep).index;
@@ -239,7 +239,7 @@ export class FleetSpawn {
         ]));
       });
     } else {
-      lines.push(buildPanelLine(width, [[` Directory for ${this.mode.agent.title} — a known dir, no path to type.`, P.dim]]));
+      lines.push(buildPanelLine(width, [[` Directory for ${this.mode.agent.title}: a known dir, no path to type.`, P.dim]]));
       lines.push(buildPanelLine(width, [['', P.dim]]));
       this.mode.candidates.forEach((cand, i) => {
         const selected = i === (this.mode as DirStep).index;
@@ -255,7 +255,7 @@ export class FleetSpawn {
       { keys: 'Enter', label: this.mode.step === 'agent' ? 'pick agent' : 'host here' },
       { keys: 'Esc', label: 'cancel' },
     ], P)];
-    return buildPanelWorkspace(width, height, { title: 'Fleet — Host an agent', sections: [{ lines }], footerLines, palette: P });
+    return buildPanelWorkspace(width, height, { title: 'Fleet: Host an agent', sections: [{ lines }], footerLines, palette: P });
   }
 
   private requireGateway(): AcpSpawnGateway | null {

package/src/panels/fleet-steer.ts

@@ -34,7 +34,7 @@ export function steerBadgeTone(status: SteerBadgeStatus, palette: PanelPalette):
   }
 }
 
-/** Labels of the currently live, steerable agent nodes other than `excludeNodeId` — offered when a steer target has gone inactive (WO item 4). */
+/** Labels of the currently live, steerable agent nodes other than `excludeNodeId`, offered when a steer target has gone inactive (WO item 4). */
 export function liveSteerableLabels(nodes: readonly ProcessNode[], excludeNodeId: string): string[] {
   return nodes
     .filter((node) => node.id !== excludeNodeId && node.kind === 'agent' && node.capabilities.steerable && !isTerminalProcessState(node.state))
@@ -44,8 +44,8 @@ export function liveSteerableLabels(nodes: readonly ProcessNode[], excludeNodeId
 /** Refusal message for a steer that could not be queued: states why + preserves the draft + suggests live targets. */
 export function steerRefusalMessage(reason: string, siblingLabels: readonly string[]): string {
   const suggestion = siblingLabels.length > 0
-    ? ` Draft kept — steerable now: ${siblingLabels.slice(0, 3).join(', ')}${siblingLabels.length > 3 ? '…' : ''}.`
-    : ' Draft kept — no other agents are currently steerable.';
+    ? ` Draft kept: steerable now: ${siblingLabels.slice(0, 3).join(', ')}${siblingLabels.length > 3 ? '…' : ''}.`
+    : ' Draft kept: no other agents are currently steerable.';
   return `${reason}.${suggestion}`;
 }
 
@@ -55,10 +55,10 @@ export function renderSteerBadgeLine(badge: SteerBadge, width: number, palette:
   const tone = steerBadgeTone(badge.status, palette);
   const forTarget = targetLabel ? ` for ${targetLabel}` : '';
   const label = badge.status === 'queued'
-    ? `steer queued${forTarget} — delivers on its next turn (watch the ${glyph} badge)`
+    ? `steer queued${forTarget}: delivers on its next turn (watch the ${glyph} badge)`
     : badge.status === 'consumed'
       ? 'steer consumed'
-      : `steer dropped — ${badge.note ?? 'the target ended before delivery'}`;
+      : `steer dropped: ${badge.note ?? 'the target ended before delivery'}`;
   return buildPanelLine(width, [
     [' ', palette.dim],
     [glyph, tone],
@@ -75,7 +75,7 @@ export function renderSteerBadgeLine(badge: SteerBadge, width: number, palette:
  * its short linger so a tab doesn't accumulate stale indicators.
  *
  * Mutates `tab.steerBadge` in place (same mutable-slot convention as
- * `FleetTab.ledgerEntries` — see fleet-panel.ts's ensureLedgerLoaded).
+ * `FleetTab.ledgerEntries`, see fleet-panel.ts's ensureLedgerLoaded).
  * Returns true when anything changed, so the caller knows whether to mark
  * itself dirty.
  */
@@ -102,7 +102,7 @@ export function reconcileSteerBadges(
         changed = true;
       } else if (badge.queuedAt !== undefined && now - badge.queuedAt > STEER_TTL_MS) {
         // Long-tool-call case: the target is still healthy and non-terminal,
-        // but the underlying steer message's own TTL (the SDK's MessageBus —
+        // but the underlying steer message's own TTL (the SDK's MessageBus,
         // see registry.js's steer(), which stamps every steer with
         // STEER_TTL_MS) has lapsed without a COMMUNICATION_CONSUMED ever
         // arriving. The SDK gives no explicit expiry signal, so without this

package/src/panels/fleet-stop.ts

@@ -1,7 +1,7 @@
 // ---------------------------------------------------------------------------
 // fleet-stop.ts
 //
-// d1 — the "stopping…" write-window overlay, split out of fleet-panel.ts
+// d1, the "stopping…" write-window overlay, split out of fleet-panel.ts
 // to keep that file under the architecture 800-line gate (same rationale as
 // fleet-steer.ts). FleetPanel owns nothing here beyond a FleetStopTracker
 // instance; all the state->display logic lives in these pure helpers.
@@ -26,7 +26,7 @@ export const BLOCKED_GLYPH = '⚑';
 /**
  * Tracks node ids the operator just asked to stop (kill/interrupt/pause) mapped
  * to the keypress time. While an id is present AND within STOP_SETTLE_MS its row
- * renders a display-only 'stopping…' instead of the raw state — so the tree
+ * renders a display-only 'stopping…' instead of the raw state, so the tree
  * never claims the past-tense 'killed'/'interrupted' during the brief write
  * window between the keypress and the state actually flipping. After the settle
  * window the TRUE state is ALWAYS shown, so a genuinely stuck kill is never
@@ -90,7 +90,7 @@ export interface FleetPauseDeps {
  * resumable node is resumed (interrupt's inverse); a live pausable node is
  * paused via the registry's interrupt() (its disable path). Honest refusal for
  * non-resumable/non-pausable kinds. Returns true when the key is consumed,
- * false to fall through (terminal, non-paused node — nothing to toggle).
+ * false to fall through (terminal, non-paused node, nothing to toggle).
  */
 export function toggleFleetPause(node: ProcessNode, deps: FleetPauseDeps): boolean {
   if (node.state === 'paused') {
@@ -141,7 +141,7 @@ export function buildFleetTreeHints(
   ];
   // Jump-to-blocked is offered whenever something is waiting on the operator,
   // in either view (the blocked nodes live in the active fleet, and the jump
-  // switches back to it) — so it is placed before the view-specific branch.
+  // switches back to it), so it is placed before the view-specific branch.
   if (blockedCount > 0) hints.push({ keys: 'b', label: `blocked (${blockedCount})` });
   if (viewMode === 'archived') {
     // Archive view: restore + return to the live fleet; nothing here is live,
@@ -194,7 +194,7 @@ export function countDescendantStats(rows: readonly FleetTreeRow[], nodeId: stri
 }
 
 /**
- * Build the confirm-overlay arm options for a cascade Kill of one node — the
+ * Build the confirm-overlay arm options for a cascade Kill of one node, the
  * label carries the descendant count so the operator sees what a confirm takes
  * down. Extracted from FleetPanel to keep that file under the 800-line cap.
  */

package/src/panels/fleet-tabs.ts

@@ -14,7 +14,7 @@
 //
 // Attachable kinds: only 'agent' and 'wrfc-chain' carry anything worth
 // attaching to (a transcript for an agent; a live member summary for a
-// chain — see fleet-transcript.ts). workflow/trigger/schedule/watcher/
+// chain, see fleet-transcript.ts). workflow/trigger/schedule/watcher/
 // background-process/wrfc-subtask nodes have no transcript and are never
 // attachable (FleetPanel shows a status message instead, matching the
 // existing i/K "not supported" convention).
@@ -34,14 +34,14 @@ export function isAttachableFleetKind(kind: ProcessKind): kind is FleetAttachabl
 /**
  * Lifecycle of a queued steer message, tracked
  * per-tab. `queued` means `ProcessRegistry.steer()` accepted the message
- * onto the target's inbox, NOT that the agent has seen it — that is the
+ * onto the target's inbox, NOT that the agent has seen it, that is the
  * later, honest `consumed` transition (a `COMMUNICATION_CONSUMED` runtime-bus
  * event matching this badge's `messageId`). `dropped` is a TUI-side
  * inference: the SDK emits no "expired"/"cancelled" signal for a queued
  * steer, so if the target node goes terminal (done/failed/killed/
  * interrupted) while the badge is still `queued`, FleetPanel resolves it to
  * `dropped` itself rather than leaving the badge hanging forever (see
- * fleet-panel.ts reconcileSteerBadges — cross-WO note: the SDK engineer
+ * fleet-panel.ts reconcileSteerBadges, cross-WO note: the SDK engineer
  * confirmed no dropped signal exists).
  */
 export type SteerBadgeStatus = 'queued' | 'consumed' | 'dropped';
@@ -50,19 +50,19 @@ export type SteerBadgeStatus = 'queued' | 'consumed' | 'dropped';
 export interface SteerBadge {
   readonly messageId: string;
   readonly status: SteerBadgeStatus;
-  /** Present for 'dropped' — a one-line honest explanation shown in the tab. */
+  /** Present for 'dropped', a one-line honest explanation shown in the tab. */
   readonly note?: string;
-  /** epoch ms when status left 'queued' (consumed or dropped) — drives FleetPanel's linger-then-clear tick. */
+  /** epoch ms when status left 'queued' (consumed or dropped), drives FleetPanel's linger-then-clear tick. */
   readonly resolvedAt?: number;
   /**
    * epoch ms when status entered 'queued' (set once, at submit time). Drives
    * fleet-steer.ts's reconcileSteerBadges TTL-expiry fallback: the SDK's
    * MessageBus attaches its own STEER_TTL_MS to the underlying steer message
    * (see registry.js's steer()) but never tells the TUI when that TTL lapses
-   * without delivery — the "agent stays healthy/non-terminal through one
+   * without delivery, the "agent stays healthy/non-terminal through one
    * very long tool call, and the steer simply expires unseen in the bus"
    * case. Without this, that badge would show 'queued' forever. Optional
-   * because older/hand-built badges (tests, pre-fix data) may not carry it —
+   * because older/hand-built badges (tests, pre-fix data) may not carry it,
    * absence just means "no TTL-expiry inference possible for this badge",
    * not an error.
    */
@@ -73,7 +73,7 @@ export interface SteerBadge {
  * Append one character of steer-composer input to a draft, normalizing a
  * pasted multi-line block's line breaks (terminals transmit bracketed-paste
  * newlines as literal `\r`/`\n` characters delivered one at a time through
- * the same per-char burst pipeline as ordinary typing — see
+ * the same per-char burst pipeline as ordinary typing, see
  * handler-feed-routes.ts's isCapturingTextBurst contract) to a single
  * collapsed space instead of either corrupting the one-line field with a
  * raw control character or silently dropping the content. Mirrors how a
@@ -89,10 +89,10 @@ export function appendSteerText(draft: string, ch: string): string {
 
 /**
  * One attached session tab. `agentId` is the SDK attach handle for
- * `AgentManager.getConversationSnapshot(agentId)` — populated only for
+ * `AgentManager.getConversationSnapshot(agentId)`, populated only for
  * 'agent' tabs (agent.ts sets ProcessNode.id = record.id, so node.id IS the
  * agentId). 'wrfc-chain' tabs render a live member-summary instead of a
- * transcript (a chain has no single conversation of its own — see
+ * transcript (a chain has no single conversation of its own, see
  * fleet-transcript.ts renderFleetChainSummary) and carry an empty agentId,
  * unused.
  *
@@ -112,7 +112,7 @@ export interface FleetTab {
    * Cached parsed ledger fallback for a terminal agent whose conversation
    * snapshot has been evicted from the SDK's retention ring (the degraded
    * fallback path). `null` = not yet loaded/attempted; `[]` = loaded and
-   * empty (or the load failed) — both render the same honest "no transcript"
+   * empty (or the load failed), both render the same honest "no transcript"
    * state once loaded is true.
    */
   ledgerEntries: Record<string, unknown>[] | null;
@@ -122,7 +122,7 @@ export interface FleetTab {
    * when not composing. Mirrors git-panel.ts's `commitMessage` mutable-slot
    * convention (FleetPanel.isCapturingTextBurst() gates on this being
    * non-null so a burst/paste lands here char-by-char, never as tree/tab
-   * hotkeys — see FleetPanel.handleSteerInput).
+   * hotkeys, see FleetPanel.handleSteerInput).
    */
   steerDraft: string | null;
   /** This tab's most recent steer message's lifecycle, or null. */
@@ -172,7 +172,7 @@ export function activeFleetTab(state: FleetTabsState): FleetTab | null {
 /**
  * Attach a node as a tab (Enter on an attachable tree row). Re-focuses an
  * already-open tab for the same node instead of creating a duplicate.
- * Returns `state` unchanged if `node.kind` is not attachable — callers
+ * Returns `state` unchanged if `node.kind` is not attachable, callers
  * (FleetPanel) are expected to guard with `isAttachableFleetKind` first and
  * report a status message rather than silently doing nothing; this
  * function's own no-op fallback is defense in depth.
@@ -222,7 +222,7 @@ export function switchFleetTab(state: FleetTabsState, activeTabIndex: number): F
   return { tabs: state.tabs, activeTabIndex };
 }
 
-/** Move focus to the next/previous tab in the strip (wrapping is deliberately NOT applied — clamps at the ends). */
+/** Move focus to the next/previous tab in the strip (wrapping is deliberately NOT applied, clamps at the ends). */
 export function stepFleetTab(state: FleetTabsState, direction: 1 | -1): FleetTabsState {
   const next = state.activeTabIndex + direction;
   return switchFleetTab(state, Math.max(0, Math.min(state.tabs.length, next)));

package/src/panels/fleet-transcript.ts

@@ -3,12 +3,12 @@
 //
 // Renders the content of an attached FleetPanel
 // session tab. Three sources, chosen by what the SDK can actually provide
-// (see the design brief's "central reality check" — a full-fidelity live
+// (see the design brief's "central reality check", a full-fidelity live
 // transcript needs a ConversationMessageSnapshot[] history source, which
 // only exists for a RUNNING agent or a just-completed one still inside the
 // SDK's bounded retention ring):
 //
-//   - 'live'/'frozen'  — a non-empty ConversationMessageSnapshot[] from
+//   - 'live'/'frozen' , a non-empty ConversationMessageSnapshot[] from
 //     AgentManager.getConversationSnapshot(agentId), rendered through the
 //     EXACT SAME conversation-rendering machinery the main session surface
 //     uses (MessageLineCache -> conversation-rendering.ts render functions),
@@ -16,9 +16,9 @@
 //     user would see in their own session. 'live' while the agent is still
 //     running, 'frozen' once it has completed but the snapshot has not yet
 //     been evicted from the retention ring.
-//   - 'unavailable'    — a terminal agent whose snapshot came back empty
-//     (evicted past the retention bound, or an agent kind — e.g. the WRFC
-//     owner — that never registered a live conversation source at all).
+//   - 'unavailable'   , a terminal agent whose snapshot came back empty
+//     (evicted past the retention bound, or an agent kind, e.g. the WRFC
+//     owner, that never registered a live conversation source at all).
 //     FleetPanel degrades to the on-disk event ledger (renderFleetLedgerFallback)
 //     for this case; this module never fabricates transcript content.
 //
@@ -45,7 +45,7 @@ import {
 } from './fleet-read-model.ts';
 
 /** Duplicated from fleet-panel.ts's private helper (same tiny mapping) rather than
- *  cross-importing — fleet-panel.ts imports THIS module to render tab content, so
+ *  cross-importing, fleet-panel.ts imports THIS module to render tab content, so
  *  importing back would form a cycle. See fleet-panel.ts's own toneColor for the
  *  canonical copy; keep both in sync if the tone set ever changes. */
 function toneColor(tone: FleetStateTone, palette: PanelPalette): string {
@@ -70,17 +70,17 @@ export interface FleetTranscriptRender {
 }
 
 /**
- * (design point 4) — a 'frozen' transcript is a static capture of a
+ * (design point 4), a 'frozen' transcript is a static capture of a
  * completed process, not something that will change on the next render. The
  * distinction matters because this wave also introduced kill/interrupt
  * display bugs (elapsed/usage briefly climbing after a terminal state
- * shows) — a done-section tab that looked indistinguishable from a live one
+ * shows), a done-section tab that looked indistinguishable from a live one
  * would misleadingly suggest the underlying agent is still doing something.
  * Shown once at the top of a 'frozen' tab's content; 'live' tabs never show
  * it, and 'unavailable' tabs get the ledger fallback's own notice instead
  * (renderFleetLedgerFallback below) rather than this one.
  */
-const FROZEN_TRANSCRIPT_NOTICE = 'Read-only — this agent finished; not a live view.';
+const FROZEN_TRANSCRIPT_NOTICE = 'Read-only: this agent finished; not a live view.';
 
 function renderFrozenTranscriptNotice(width: number): Line[] {
   const palette = DEFAULT_PANEL_PALETTE;
@@ -93,15 +93,15 @@ function renderFrozenTranscriptNotice(width: number): Line[] {
 
 /**
  * Render an agent tab's transcript from a `ConversationMessageSnapshot[]`
- * already fetched by the caller (this module does no I/O of its own — it is
+ * already fetched by the caller (this module does no I/O of its own, it is
  * a pure renderer, same convention as fleet-read-model.ts).
  *
- * `isTerminal` distinguishes 'live' (the agent is still running — the
+ * `isTerminal` distinguishes 'live' (the agent is still running, the
  * snapshot came from the SDK's live source, so it will change on the next
- * render) from 'frozen' (the agent finished — the snapshot is a static final
+ * render) from 'frozen' (the agent finished, the snapshot is a static final
  * capture; caller is showing a completed conversation, not a live one).
  * An empty snapshot on a terminal agent means the source was never
- * registered or has been evicted — the caller falls back to
+ * registered or has been evicted, the caller falls back to
  * `renderFleetLedgerFallback`; an empty snapshot on a non-terminal agent
  * (e.g. the SDK hasn't sent a turn yet) legitimately means "no messages yet".
  */
@@ -124,7 +124,7 @@ export function renderFleetAgentTranscript(
     addLines: (lines: Line[]): void => { historyLines.push(...lines); },
     getLineCount: (): number => historyLines.length,
   };
-  // Scratch block/collapse/error registries — collapse-toggle and block-copy
+  // Scratch block/collapse/error registries, collapse-toggle and block-copy
   // interactions are out of scope for this tab (a later item, per the brief's
   // "transcript browse" follow-on); this render is a read-only tail view.
   const context: ConversationRenderContext = {
@@ -147,7 +147,7 @@ export function renderFleetAgentTranscript(
   // No `&& budgetHeight > 0` guard: when the notice alone already consumes
   // the whole height budget, `historyLines.slice(historyLines.length - 0)`
   // correctly yields an empty tail (nothing fits), rather than falling
-  // through to the full, un-sliced history — which the caller would then
+  // through to the full, un-sliced history, which the caller would then
   // head-clip, silently swapping in the OLDEST lines instead of the
   // most-recent tail this view promises.
   const visible = historyLines.length > budgetHeight
@@ -157,7 +157,7 @@ export function renderFleetAgentTranscript(
 }
 
 // ---------------------------------------------------------------------------
-// Chain summary — 'wrfc-chain' tabs have no single conversation
+// Chain summary, 'wrfc-chain' tabs have no single conversation
 // ---------------------------------------------------------------------------
 
 /**
@@ -177,7 +177,7 @@ export function renderFleetChainSummary(
   const C = DEFAULT_PANEL_PALETTE;
   if (memberRows.length === 0) {
     const message = chainDoneOrAbsent
-      ? ' chain completed — members no longer tracked'
+      ? ' chain completed: members no longer tracked'
       : ' (no member agents yet)';
     return [buildPanelLine(width, [[message, C.dim]])];
   }
@@ -196,13 +196,13 @@ export function renderFleetChainSummary(
 }
 
 // ---------------------------------------------------------------------------
-// Ledger fallback — degraded activity view for a terminal agent whose
+// Ledger fallback, degraded activity view for a terminal agent whose
 // full-fidelity snapshot is unavailable (an honest fallback)
 // ---------------------------------------------------------------------------
 
 /**
  * Parse an agent's `<agentId>.jsonl` event ledger. Tolerant of malformed
- * lines (skipped, not thrown) — matches agent-detail-modal.ts's existing
+ * lines (skipped, not thrown), matches agent-detail-modal.ts's existing
  * JSONL-reading convention for the same file shape.
  */
 export function parseAgentLedger(raw: string): Record<string, unknown>[] {
@@ -231,7 +231,7 @@ function renderLedgerEntry(width: number, entry: Record<string, unknown>, palett
       const model = typeof entry['model'] === 'string' ? entry['model'] : 'unknown model';
       const provider = typeof entry['provider'] === 'string' ? entry['provider'] : 'unknown';
       return renderConversationEventLine(width, { marker: '●', markerFg: palette.dim, label: 'session', labelFg: palette.dim }, [
-        { text: `started — ${model} (${provider})`, fg: palette.value },
+        { text: `started: ${model} (${provider})`, fg: palette.value },
       ]);
     }
     case 'session_config': {
@@ -258,13 +258,13 @@ function renderLedgerEntry(width: number, entry: Record<string, unknown>, palett
       const success = entry['success'] !== false;
       const bad = palette.bad ?? DEFAULT_PANEL_PALETTE.bad;
       // the writer already truncates this to 500 chars (session.ts's
-      // resultPreview field) — passed through verbatim (collapsed to one
+      // resultPreview field), passed through verbatim (collapsed to one
       // line), never re-summarized or fabricated, then tail-truncated again
       // to fit the row.
       const rawPreview = typeof entry['resultPreview'] === 'string' ? entry['resultPreview'] : '';
       const preview = rawPreview.replace(/\s+/g, ' ').trim();
       const label = `${toolName}${success ? '' : ' (failed)'}`;
-      const text = preview ? `${label} — ${preview}` : label;
+      const text = preview ? `${label}: ${preview}` : label;
       return renderConversationEventLine(width, {
         marker: success ? '●' : '✗',
         markerFg: success ? palette.info : bad,
@@ -288,7 +288,7 @@ function renderLedgerEntry(width: number, entry: Record<string, unknown>, palett
     // records, appended to this same JSONL ledger as
     // `{type:'knowledge_injection', turn, ...record}` (orchestrator-runner.ts).
     // Without this case these fell through to the generic 'event' default
-    // below (just the bare type name) — this renders the honest outcome
+    // below (just the bare type name), this renders the honest outcome
     // (what was injected, or why nothing was) instead.
     case 'knowledge_injection': {
       const turn = typeof entry['turn'] === 'number' ? entry['turn'] : '?';
@@ -300,7 +300,7 @@ function renderLedgerEntry(width: number, entry: Record<string, unknown>, palett
           ? entry['reason']
           : 'nothing cleared the relevance floor';
         return renderConversationEventLine(width, { marker: '◇', markerFg: palette.dim, label: 'knowledge', labelFg: palette.dim }, [
-          { text: `turn ${turn} · nothing injected — ${reason}${backendTag}`, fg: palette.dim },
+          { text: `turn ${turn} · nothing injected; ${reason}${backendTag}`, fg: palette.dim },
         ]);
       }
       return renderConversationEventLine(width, { marker: '◇', markerFg: palette.info, label: 'knowledge', labelFg: palette.info }, [
@@ -318,7 +318,7 @@ function renderLedgerEntry(width: number, entry: Record<string, unknown>, palett
  * Degraded activity view for a terminal agent whose full-fidelity
  * conversation snapshot is unavailable (evicted, or never registered).
  * Explicitly framed as an activity log, never presented as a transcript
- * replay (the ledger is a truncated event record — see session.ts /
+ * replay (the ledger is a truncated event record, see session.ts /
  * orchestrator-runner.ts: tool args/results are sliced to 500 chars and
  * assistant response TEXT is never written at all, only its length).
  *
@@ -336,7 +336,7 @@ export function renderFleetLedgerFallback(
 ): Line[] {
   const palette = DEFAULT_PANEL_PALETTE;
   const notice = renderConversationNotice(
-    'Read-only. Full transcript unavailable for this agent — showing its activity log instead.',
+    'Read-only. Full transcript unavailable for this agent; showing its activity log instead.',
     width,
     { accent: palette.warn, text: palette.dim, dim: true },
   );

package/src/panels/git-panel.ts

@@ -57,7 +57,7 @@ type ViewItem =
   | { kind: 'commit'; entry: CommitEntry }
   | { kind: 'empty'; label: string };
 
-/** Subject carried by the panel's single ConfirmState — either a repo init or a commit. */
+/** Subject carried by the panel's single ConfirmState, either a repo init or a commit. */
 type GitConfirmSubject =
   | { kind: 'init' }
   | { kind: 'commit'; message: string };
@@ -119,7 +119,7 @@ export class GitPanel extends BasePanel {
   /** True when the last refresh failed because `workingDirectory` isn't a git repo. */
   private notGitRepo = false;
 
-  /** Single pending confirm — either "init this repo" or "commit with this message". */
+  /** Single pending confirm, either "init this repo" or "commit with this message". */
   private confirm: ConfirmState<GitConfirmSubject> | null = null;
 
   /** Non-null while composing a commit message; null means the compose UI is closed. */
@@ -211,12 +211,12 @@ export class GitPanel extends BasePanel {
       this.error = null;
       this.notGitRepo = false;
       this.rebuildItems();
-      // Do not clear expandedDiff during auto-refresh — only clear on explicit user action
+      // Do not clear expandedDiff during auto-refresh, only clear on explicit user action
       this._markDirtyAndRender();
     } catch (err) {
       const msg = summarizeError(err);
       if (/not a git\b/i.test(msg)) {
-        // I4: no more auto `git init` side effect — surface the state and let
+        // I4: no more auto `git init` side effect, surface the state and let
         // the user explicitly confirm initialisation with 'i'.
         this.notGitRepo = true;
         this.error = 'Not a git repository here. Press i to initialize one (explicit confirm).';
@@ -276,7 +276,7 @@ export class GitPanel extends BasePanel {
 
   // Input handling
 
-  /** Commit-message entry wants every char of a burst delivered one at a time — see Panel.isCapturingTextBurst. */
+  /** Commit-message entry wants every char of a burst delivered one at a time, see Panel.isCapturingTextBurst. */
   isCapturingTextBurst(): boolean {
     return this.commitMessage !== null;
   }
@@ -346,7 +346,7 @@ export class GitPanel extends BasePanel {
         return true;
       }
       case 'i': {
-        // Only meaningful when no repo is loaded — otherwise leave the key
+        // Only meaningful when no repo is loaded, otherwise leave the key
         // unconsumed instead of absorbing it as a silent no-op.
         if (!this.notGitRepo) return false;
         this.confirm = { subject: { kind: 'init' }, label: this.workingDirectory, verb: 'Init' };
@@ -437,7 +437,7 @@ export class GitPanel extends BasePanel {
     }
 
     if (item.kind === 'commit') {
-      // Root commits have no parent (`<hash>^` is invalid) — the catch below
+      // Root commits have no parent (`<hash>^` is invalid), the catch below
       // surfaces that instead of pretending a diff exists.
       const before = `${item.entry.hash}^`;
       const after = item.entry.hash;
@@ -543,7 +543,7 @@ export class GitPanel extends BasePanel {
       buildKeyboardHints(width, [{ keys: 'Enter', label: 'review & confirm' }, { keys: 'Esc', label: 'cancel' }], DEFAULT_PANEL_PALETTE),
     ];
     return buildPanelWorkspace(width, height, {
-      title: ' Git — Commit',
+      title: ' Git: Commit',
       sections: [{ title: 'Message', lines: [inputLine] }],
       footerLines,
       palette: DEFAULT_PANEL_PALETTE,