@pellux/goodvibes-tui 2.0.13 → 2.0.15

package/src/panels/builtin/operations.ts

@@ -14,16 +14,16 @@ import { GOODVIBES_TUI_SURFACE_ROOT } from '../../config/surface.ts';
 // split into five operator domains, applied per-registration below. Kept in
 // this file's original registration order (rather than physically regrouped
 // by category) because several registrations below close over shared local
-// state — `ui` — defined once near the top of this function.
+// state, `ui`, defined once near the top of this function.
 //
 // (the purge): cockpit, approval, automation, routes, control-plane,
 // worktrees, tasks, orchestration, ops, ops-control, and communication were
 // RETIRE-INTO-FLEET (their live views are subsumed by the Fleet panel below
-// — each id now redirects there via registerAlias); debug and eval were
+//, each id now redirects there via registerAlias); debug and eval were
 // DELETE-disposition (no surviving human surface). See
 // .goodvibes/audit/2026-07-04-wave6-briefs.json for the full
 // disposition map. The rosterReadModel/agent-lifecycle wiring that used to
-// feed CockpitPanel exclusively was removed along with it — Fleet reads the
+// feed CockpitPanel exclusively was removed along with it, Fleet reads the
 // process registry directly via fleetReadModel below, not via the roster
 // read-model.
 //
@@ -34,12 +34,12 @@ import { GOODVIBES_TUI_SURFACE_ROOT } from '../../config/surface.ts';
 // marketplace, policy, knowledge, memory, docs, qr-code, work-plan,
 // project-planning) migrated to config-modal surfaces. All surfaces AND
 // their panel-id redirects are registered centrally in registerBuiltinModals
-// (builtin-modals.ts) now — the interim ecosystem redirect bridge was deleted
-// with the group-B port. local-auth is a DELIBERATE EXCEPTION — see below.
+// (builtin-modals.ts) now, the interim ecosystem redirect bridge was deleted
+// with the group-B port. local-auth is a DELIBERATE EXCEPTION, see below.
 export function registerOperationsPanels(manager: PanelManager, deps: ResolvedBuiltinPanelDeps): void {
   const ui = requireUiServices(deps);
 
-  // Fleet — everything running, not just what this terminal started. The read
+  // Fleet, everything running, not just what this terminal started. The read
   // model is composed once in runtime/services.ts (so its refresh timer has an
   // owner that stops it) and unions this surface's own live process registry
   // with the adopted daemon's rows; see runtime/client/fleet-union.ts for who
@@ -108,7 +108,7 @@ export function registerOperationsPanels(manager: PanelManager, deps: ResolvedBu
       resume: (id: string) => fleetReadModel.resume(id),
       kill: (id: string, opts: { cascade: boolean }) => fleetReadModel.kill(id, opts),
       // Full-fidelity transcript source for an attached agent
-      // tab — live while the agent runs, frozen briefly after it completes
+      // tab, live while the agent runs, frozen briefly after it completes
       // (AgentManager's bounded retention ring), empty once evicted (the
       // panel degrades to the on-disk ledger fallback in that case).
       getConversationSnapshot: (agentId: string) => ui.agents.agentManager.getConversationSnapshot(agentId),
@@ -119,7 +119,7 @@ export function registerOperationsPanels(manager: PanelManager, deps: ResolvedBu
   });
 
   // Compat: '/panel open <id>' (and any saved layout/muscle memory) for
-  // every retired runtime-ops console still resolves — redirected to fleet,
+  // every retired runtime-ops console still resolves, redirected to fleet,
   // which absorbs orchestration, permissions, communication, task, and
   // process-tree views (see FleetPanel's own description above).
   manager.registerAlias('cockpit', 'fleet');
@@ -135,16 +135,16 @@ export function registerOperationsPanels(manager: PanelManager, deps: ResolvedBu
   manager.registerAlias('communication', 'fleet');
   manager.registerAlias('incident', 'fleet');
   // Re-pointed: forensics used to resolve to the (now-retired) incident
-  // workspace; both ids now land on fleet directly — alias
+  // workspace; both ids now land on fleet directly, alias
   // resolution is a single hop, so this cannot chain through 'incident'.
   manager.registerAlias('forensics', 'fleet');
 
-  // Notifications — the visible home for the panel_only notification
+  // Notifications, the visible home for the panel_only notification
   // target: anything a NotificationRouter (SDK) routes there (including
   // burst/batch-collapsed groups, shown with their real running count)
   // renders here instead of having nowhere to go. See notifications-feed.ts
   // for why this exists. Not preloaded (only 'tokens' preloads, per the
-  // panel-consolidation cleanup below) — the shared feed singleton
+  // panel-consolidation cleanup below), the shared feed singleton
   // accumulates independently of whether this view has been opened yet, so
   // nothing is lost by instantiating it lazily on first open like every
   // other panel here.
@@ -160,7 +160,7 @@ export function registerOperationsPanels(manager: PanelManager, deps: ResolvedBu
   // local-auth is a DELIBERATE EXCEPTION to the purge: it stays a
   // registered panel because it is the host for the masked password-entry
   // sub-mode (LocalAuthPanel.openMaskedEntry, driven by
-  // ctx.openLocalAuthMaskedEntry — the only path that keeps a plaintext
+  // ctx.openLocalAuthMaskedEntry, the only path that keeps a plaintext
   // password out of argv/history/scrollback). Its browse view is mirrored by
   // local-auth-modal (reached via the /local-auth front-door), but the panel
   // itself cannot be retired without regressing that secure input, and no

package/src/panels/builtin/session.ts

@@ -5,17 +5,17 @@ import type { ResolvedBuiltinPanelDeps } from './shared.ts';
 import { requireUiServices } from './shared.ts';
 
 // (the purge): panel-list and system-messages were registered here
-// before the purge — both DELETE-disposition. panel-list was a picker over a
-// handful of panels (dead weight now that the registry is much smaller —
+// before the purge, both DELETE-disposition. panel-list was a picker over a
+// handful of panels (dead weight now that the registry is much smaller,
 // the picker itself is replaced by a live-registry selection modal on
 // Ctrl+P, see shell/ui-openers.ts). system-messages' buffered notices are
 // rerouted to the transcript's system channel instead of vanishing: with no
 // panel attached, SystemMessageRouter's delivery resolution
 // (resolveSystemMessageDelivery, SDK) already falls back to
-// conversation.addTypedSystemMessage for every kind/target combination — see
+// conversation.addTypedSystemMessage for every kind/target combination, see
 // bootstrap-shell.ts and core/system-message-router.ts.
 //
-// (the purge) — group B: 'qr-code', 'sessions', and 'docs' also
+// (the purge), group B: 'qr-code', 'sessions', and 'docs' also
 // migrated. 'qr-code' → the 'pairing-modal' surface; 'docs' → the
 // 'keybindings-modal' surface (merged with the shortcuts-overlay content);
 // 'sessions' folds into the existing session-picker modal ('sessions' redirects
@@ -27,7 +27,7 @@ export function registerSessionPanels(manager: PanelManager, deps: ResolvedBuilt
     id: 'tokens',
     name: 'Tokens',
     // registry previously said 'K' while the live panel's own
-    // super() call used 'T' — a pre-existing registry/instance mismatch as
+    // super() call used 'T', a pre-existing registry/instance mismatch as
     // well as a collision ('K' with knowledge/skills, 'T' with thinking).
     // Unified to a single unique glyph in both places.
     icon: '▢',
@@ -36,7 +36,7 @@ export function registerSessionPanels(manager: PanelManager, deps: ResolvedBuilt
     // Preloaded (absorbed from the retired ContextVisualizerPanel) so turn
     // history and pressure accumulate in the background even before the user
     // opens the tab. The only builtin panel that still preloads post-purge
-    // — see registerBuiltinPanels callers for the others' preload
+    //, see registerBuiltinPanels callers for the others' preload
     // removal.
     preload: true,
     retainOnClose: true,
@@ -59,7 +59,7 @@ export function registerSessionPanels(manager: PanelManager, deps: ResolvedBuilt
     },
   });
 
-  // Hosted Session — a conversation whose loop runs INSIDE the daemon rather
+  // Hosted Session, a conversation whose loop runs INSIDE the daemon rather
   // than in this process, rendered from the same `turn`/`tools` event domains a
   // local turn emits (see panels/hosted-session-feed.ts). Not preloaded: a
   // terminal that never opts into hosting should not carry an empty tab for it,
@@ -75,7 +75,7 @@ export function registerSessionPanels(manager: PanelManager, deps: ResolvedBuilt
     factory: () => new HostedSessionPanel(),
   });
 
-  // compat: the retired 'context' panel id still resolves — redirected
+  // compat: the retired 'context' panel id still resolves, redirected
   // to the merged tokens console ('/panel open context', saved layouts).
   manager.registerAlias('context', 'tokens');
 }

package/src/panels/builtin/shared.ts

@@ -73,13 +73,13 @@ export interface BuiltinPanelDeps {
   forensicsRegistry?: import('@/runtime/index.ts').ForensicsRegistry;
   /**
    * EvalRegistry for the `/eval` command surface.: 'eval' the panel was
-   * deleted (DELETE-disposition), but this field is left in place — no
+   * deleted (DELETE-disposition), but this field is left in place, no
    * builtin panel factory reads it anymore, and it was never wired at
    * bootstrap in production either way (the eval CLI command reads its own
    * copy via CommandContext.extensions.evalRegistry).
    */
   evalRegistry?: import('../eval-registry.ts').EvalRegistry;
-  /** Host-vs-client memory access for the Memory modal — the spine client, never the raw registry (routes over the wire when a daemon is adopted). */
+  /** Host-vs-client memory access for the Memory modal, the spine client, never the raw registry (routes over the wire when a daemon is adopted). */
   memoryRegistry?: MemoryAccess;
   /** Shared policy runtime state for governance/policy diagnostics. */
   policyRuntimeState?: import('@/runtime/index.ts').PolicyRuntimeState;
@@ -105,7 +105,7 @@ export interface BuiltinPanelDeps {
   /** Shared sandbox session registry for sandbox surfaces and tools. */
   sandboxSessionRegistry: SandboxSessionRegistry;
   /**
-   * Resolved daemon home directory (e.g. `~/.goodvibes/daemon`) — owned by the composition root
+   * Resolved daemon home directory (e.g. `~/.goodvibes/daemon`), owned by the composition root
    * and passed explicitly so panel factories do not discover cwd/home implicitly.
    */
   daemonHomeDir?: string;
@@ -123,7 +123,7 @@ export interface BuiltinPanelDeps {
   workPlanStore?: import('@pellux/goodvibes-sdk/platform/workflow').WorkPlanStore;
   /** Explicit UI-facing runtime services for agent/process/WRFC/remote panels and modals. */
   uiServices?: UiRuntimeServices;
-  /** Shared plugin manager for plugin and security panels (widened past the read-only observer surface — — so PluginsPanel can drive enable/disable/verify/lift-quarantine). */
+  /** Shared plugin manager for plugin and security panels (widened past the read-only observer surface, — so PluginsPanel can drive enable/disable/verify/lift-quarantine). */
   pluginManager?: PluginManagerControls;
   /** Shared hook dispatcher for the hooks control-room panel. */
   hookDispatcher?: Pick<HookDispatcher, 'listHooks' | 'getChains'>;
@@ -153,7 +153,7 @@ export interface BuiltinPanelDeps {
   /**
    * Surface a Fleet act result/receipt/error to the operator (late-bound to the
    * command context's print, which the Fleet pick/conflict/discard acts use for
-   * their receipts — the same conversation sink the command flow prints to).
+   * their receipts, the same conversation sink the command flow prints to).
    */
   fleetActsNotify?: (message: string) => void;
   /**
@@ -309,7 +309,7 @@ export function requireKnowledgeApi(deps: BuiltinPanelDeps): KnowledgeApi {
  * was not wired at bootstrap for this build/session. Renders a single
  * "dependency not configured" empty state via `buildEmptyState` so opening
  * the panel id (`/panel open <id>`, a saved layout, a cross-panel jump)
- * always resolves to a real panel instead of "Unknown panel" — the panel
+ * always resolves to a real panel instead of "Unknown panel", the panel
  * type is always registered; only its data source is sometimes absent.
  */
 class UnconfiguredDependencyPanel extends BasePanel {

package/src/panels/builtin-modals.ts

@@ -36,12 +36,12 @@ import { createPlanningModalSurface } from './modals/planning-modal.ts';
  * panel-consolidation cleanup. Called once at startup from registerBuiltinPanels, AFTER the panels'
  * deps are resolved (the surfaces close over the same read-models the retired
  * panels used). For each surface migrated from a standalone panel this does two things:
- *   1. registerModalSurface — the data + actions the config-modal host renders.
- *   2. registerModalRedirect — so `/panel open <old-id>`, saved layouts, and any
+ *   1. registerModalSurface, the data + actions the config-modal host renders.
+ *   2. registerModalRedirect, so `/panel open <old-id>`, saved layouts, and any
  *      alias still resolve to the modal.
  *
  * The Providers & Connectivity + Security subset is registered first, followed by the
- * Ecosystem & Governance set — the 12 ported config-modal
+ * Ecosystem & Governance set, the 12 ported config-modal
  * surfaces plus the `sessions` fold into the existing session-picker modal.
  */
 export function registerBuiltinModals(manager: PanelManager, deps: ResolvedBuiltinPanelDeps): void {
@@ -93,7 +93,7 @@ export function registerBuiltinModals(manager: PanelManager, deps: ResolvedBuilt
 
   // plugins/hooks/knowledge deps are wired at bootstrap in production
   // (bootstrap-shell.ts) but may be absent in a partially-wired context (e.g. a
-  // release-gate harness) — register a degraded placeholder rather than throw,
+  // release-gate harness), register a degraded placeholder rather than throw,
   // so the surface + redirect always resolve (the "always register, degrade
   // honestly" charter pattern).
   manager.registerModalSurface(deps.pluginManager
@@ -133,8 +133,8 @@ export function registerBuiltinModals(manager: PanelManager, deps: ResolvedBuilt
 
   manager.registerModalSurface(createMemoryModalSurface({
     memoryRegistry: deps.memoryRegistry as MemoryModalDeps['memoryRegistry'],
-    // Lazily resolved — a fresh factory call per Proposals-tab fetch, exactly
-    // like the Fleet gateway (fleet-gateway.ts) — so a daemon that comes up
+    // Lazily resolved, a fresh factory call per Proposals-tab fetch, exactly
+    // like the Fleet gateway (fleet-gateway.ts), so a daemon that comes up
     // AFTER this session started is still reachable on the next refresh.
     resolveConsolidationGateway: () => createMemoryConsolidationGateway({
       configManager: deps.configManager,
@@ -202,7 +202,7 @@ function buildPairingConnectionInfo(deps: ResolvedBuiltinPanelDeps): PairingModa
       // presentable; the web app declines it if the device has no authenticator.
       stepUpAvailable: true,
     });
-    // Mint a fresh per-device token and encode the canonical `#pair=` deep link —
+    // Mint a fresh per-device token and encode the canonical `#pair=` deep link,
     // the same mint the pairing.handoff.create verb performs, no raw JSON blob.
     const handoff = mintPairingHandoff({
       pairingTokens: ui.platform.pairingTokens,
@@ -224,7 +224,7 @@ function buildPairingConnectionInfo(deps: ResolvedBuiltinPanelDeps): PairingModa
 }
 
 /** A minimal config-modal surface that renders an honest "not wired" degraded
- *  state — used when a group-B surface's bootstrap dependency is absent. */
+ *  state, used when a group-B surface's bootstrap dependency is absent. */
 function unwiredSurface(name: string, title: string, reason: string): ConfigModalSurface {
   return {
     name,

package/src/panels/confirm-state.ts

@@ -1,5 +1,5 @@
 // ---------------------------------------------------------------------------
-// useConfirmState<T> — reusable inline confirm/cancel helper
+// useConfirmState<T>, reusable inline confirm/cancel helper
 //
 // ── Project-standard confirm contract (all panels must match) ──────────────
 //
@@ -45,11 +45,11 @@ export interface ConfirmState<T = string> {
  *   - ABSORBED: any other key while confirm is active (keeps confirm pending)
  *
  * Returns:
- *   - `'confirmed'` — user pressed Enter, Return, or y; caller must execute
+ *   - `'confirmed'`, user pressed Enter, Return, or y; caller must execute
  *     the action and clear state (set confirm to null)
- *   - `'cancelled'` — user pressed n or Esc; caller must clear state
- *   - `'absorbed'`  — any other key while confirm is active; caller returns true
- *   - `'inactive'`  — no confirm pending; caller continues normal dispatch
+ *   - `'cancelled'`, user pressed n or Esc; caller must clear state
+ *   - `'absorbed'` , any other key while confirm is active; caller returns true
+ *   - `'inactive'` , no confirm pending; caller continues normal dispatch
  */
 export function handleConfirmInput<T = string>(
   confirm: ConfirmState<T> | null,

package/src/panels/cost-tracker-panel.ts

@@ -1,5 +1,5 @@
 // ---------------------------------------------------------------------------
-// CostTrackerPanel — per-session / per-agent / per-plan cost estimates
+// CostTrackerPanel, per-session / per-agent / per-plan cost estimates
 // ---------------------------------------------------------------------------
 
 import type { Line } from '@pellux/goodvibes-sdk/platform/types';
@@ -107,9 +107,9 @@ export class CostTrackerPanel extends BasePanel {
   private agents = new Map<string, AgentEntry>();
 
   // Budget alert threshold in USD (0 = disabled). Mutable at runtime via the
-  // in-panel 'b' numeric entry or the /cost budget <usd> command — both call
+  // in-panel 'b' numeric entry or the /cost budget <usd> command, both call
   // setBudgetThreshold() below. When configAccess is wired (production path),
-  // this field is a cache only — currentBudgetThreshold() re-reads
+  // this field is a cache only, currentBudgetThreshold() re-reads
   // behavior.budgetAlertUsd from config on every use so the panel and the
   // background budget-breach notifier (core/budget-breach-notifier.ts) never
   // disagree about the threshold. Tests that omit configAccess fall back to
@@ -147,7 +147,7 @@ export class CostTrackerPanel extends BasePanel {
   // Getter for live orchestrator usage
   private readonly getOrchestratorUsage: () => UsageSnapshot & { model?: string };
 
-  // Optional resolver for agent usage on completion — enables real cost attribution.
+  // Optional resolver for agent usage on completion, enables real cost attribution.
   // When omitted, completed agents show $0 (honest: data unavailable).
   private readonly getAgentStatus: ((agentId: string) => AgentRecord | null) | undefined;
 
@@ -188,7 +188,7 @@ export class CostTrackerPanel extends BasePanel {
 
     // Refresh mid-turn too: a single turn can span many LLM calls (tool
     // loops), so waiting for TURN_COMPLETED leaves the meter frozen for the
-    // whole turn. LLM_RESPONSE_RECEIVED fires per call — same refresh path.
+    // whole turn. LLM_RESPONSE_RECEIVED fires per call, same refresh path.
     this.unsubs.push(
       turnEvents.on('LLM_RESPONSE_RECEIVED', () => this.refreshSessionCost()),
     );
@@ -209,7 +209,7 @@ export class CostTrackerPanel extends BasePanel {
       }),
     );
 
-    // Agent completed — capture real token usage via AgentRecord when available
+    // Agent completed, capture real token usage via AgentRecord when available
     this.unsubs.push(
       agentEvents.on('AGENT_COMPLETED', (payload) => {
         const entry = this.agents.get(payload.agentId);
@@ -281,7 +281,7 @@ export class CostTrackerPanel extends BasePanel {
   }
 
   override onDeactivate(): void {
-    // Stop polling while hidden — nothing to catch up on that a fresh
+    // Stop polling while hidden, nothing to catch up on that a fresh
     // onActivate() poll won't cover.
     this.stopStatusPollTimer();
   }
@@ -306,7 +306,7 @@ export class CostTrackerPanel extends BasePanel {
   /**
    * Poll getAgentStatus() for every agent still marked 'running'. Completion
    * already captures real usage via AGENT_COMPLETED, but a long-running agent
-   * would otherwise show 'unknown' tokens / $0 cost for its entire lifetime —
+   * would otherwise show 'unknown' tokens / $0 cost for its entire lifetime,
    * this fills the row in as usage becomes available mid-flight.
    */
   private pollRunningAgents(): void {
@@ -349,7 +349,7 @@ export class CostTrackerPanel extends BasePanel {
   /**
    * The budget-threshold entry field wants every character of a burst
    * (paste, or fast typing landing in one input.feed() call) delivered one
-   * at a time, same as it always has — see the interface doc on
+   * at a time, same as it always has, see the interface doc on
    * `Panel.isCapturingTextBurst`.
    */
   isCapturingTextBurst(): boolean {
@@ -369,7 +369,7 @@ export class CostTrackerPanel extends BasePanel {
 
     if (key === 'p') {
       // One action from any price display: edit the manual price for the
-      // session's provider:model. Needs a wired config (production path) —
+      // session's provider:model. Needs a wired config (production path),
       // without one the key falls through rather than opening a dead editor.
       if (!this.configAccess) return false;
       this.priceEntry = '';
@@ -410,7 +410,7 @@ export class CostTrackerPanel extends BasePanel {
       }
       return true;
     }
-    // Digits and a single decimal point only — this is a USD amount, not free text.
+    // Digits and a single decimal point only, this is a USD amount, not free text.
     if (key.length === 1 && /[0-9.]/.test(key) && !(key === '.' && this.budgetEntry?.includes('.'))) {
       this.budgetEntry = (this.budgetEntry ?? '') + key;
       this.markDirty();
@@ -456,7 +456,7 @@ export class CostTrackerPanel extends BasePanel {
       }
       return true;
     }
-    // Two USD amounts separated by a comma (or space) — digits, dots, comma, space.
+    // Two USD amounts separated by a comma (or space), digits, dots, comma, space.
     if (key.length === 1 && /[0-9., ]/.test(key)) {
       this.priceEntry = (this.priceEntry ?? '') + key;
       this.markDirty();
@@ -467,7 +467,7 @@ export class CostTrackerPanel extends BasePanel {
 
   /**
    * Scroll the agent list. Over-consumption fix: only absorbs the key (and
-   * only advances) when the agent list is actually long enough to scroll —
+   * only advances) when the agent list is actually long enough to scroll,
    * mirrors the same `> 5` threshold render() uses to decide whether to show
    * scroll hints (below). Previously this always returned true and grew
    * scrollOffset unboundedly even with a handful of agents, swallowing
@@ -506,7 +506,7 @@ export class CostTrackerPanel extends BasePanel {
     const sourceDescription = sessionPriced ? describePricingSource(this.sessionModel) : null;
     const sourceStr = sessionPriced
       ? (sourceDescription ? ` (${sourceDescription})` : '')
-      : ' — press p to set a price';
+      : ': press p to set a price';
     const budgetStr = budgetThreshold > 0
       ? ` / ${formatCost(budgetThreshold)}`
       : '';
@@ -514,7 +514,7 @@ export class CostTrackerPanel extends BasePanel {
     const sessionLines: Line[] = [
       this.renderKeyValue(width, ' Total', `${costStr}${budgetStr}${alertStr}${sourceStr}`, costFg),
     ];
-    // Budget meter — the single most important glance for this panel: how much
+    // Budget meter, the single most important glance for this panel: how much
     // of the configured budget the session has consumed. Only shown when a
     // budget is set (otherwise the bar would be meaningless).
     if (budgetThreshold > 0) {
@@ -570,7 +570,7 @@ export class CostTrackerPanel extends BasePanel {
       const planCost = agentList.reduce((sum, a) => sum + a.cost, 0);
       const running = agentList.filter((a) => a.status === 'running').length;
       const failed = agentList.filter((a) => a.status === 'failed').length;
-      // Plan total mixes the session model with every agent's model — flag it
+      // Plan total mixes the session model with every agent's model, flag it
       // as unpriced (rather than showing a sum that silently omits unpriced
       // agents' contribution) if any one of them has no real pricing.
       const planHasUnpriced = !isModelPriced(this.sessionModel) || agentList.some((a) => !isModelPriced(a.model));
@@ -582,7 +582,7 @@ export class CostTrackerPanel extends BasePanel {
           ...(running > 0 ? [{ text: `  ${running} running`, fg: C.running }] : []),
           ...(failed > 0 ? [{ text: `  ${failed} failed`, fg: C.bad }] : []),
         ]),
-        // Per-agent cost ledger as an aligned table — agent, model, tokens, cost
+        // Per-agent cost ledger as an aligned table, agent, model, tokens, cost
         // line up in columns instead of wrapping across two ragged rows.
         ...buildTable(
           width,
@@ -684,7 +684,7 @@ export class CostTrackerPanel extends BasePanel {
     ]);
   }
 
-  // Non-masked numeric entry for the 'b' budget-alert key — the value is a
+  // Non-masked numeric entry for the 'b' budget-alert key, the value is a
   // USD amount, not a secret, so (unlike LocalAuthPanel's masked entry) it is
   // echoed directly rather than dotted out.
   private renderBudgetEntryPrompt(width: number, height: number): Line[] {
@@ -698,7 +698,7 @@ export class CostTrackerPanel extends BasePanel {
       buildPanelLine(width, [[' [Enter] Confirm   [Esc] Cancel   [Backspace] Delete char', C.dim]]),
     ];
     const workspace = buildPanelWorkspace(width, height, {
-      title: ' Cost Tracker — Budget',
+      title: ' Cost Tracker: Budget',
       intro: 'Type a USD amount and press Enter to set the budget alert threshold.',
       sections: [{ lines: promptLines }],
       palette: DEFAULT_PANEL_PALETTE,
@@ -710,7 +710,7 @@ export class CostTrackerPanel extends BasePanel {
   // Non-masked entry for the 'p' manual-price key. Two USD-per-1M-token
   // amounts (input,output); persists to pricing.modelPrices for the session's
   // provider:model live. A model without a provider prefix cannot carry a
-  // manual price (the settings key is keyed provider:model) — the prompt says
+  // manual price (the settings key is keyed provider:model), the prompt says
   // so instead of accepting a value it could not store.
   private renderPriceEntryPrompt(width: number, height: number): Line[] {
     const editable = this.canEditSessionModelPrice();
@@ -719,7 +719,7 @@ export class CostTrackerPanel extends BasePanel {
     const promptLines: Line[] = editable
       ? [
           buildPanelLine(width, [[` Set your manual price for ${this.sessionModel} (USD per 1M tokens).`, C.label]]),
-          buildPanelLine(width, [[' Format: input,output — for example 3.00,15.00', C.dim]]),
+          buildPanelLine(width, [[' Format: input,output, for example 3.00,15.00', C.dim]]),
           buildPanelLine(width, [['', C.label]]),
           buildPanelLine(width, [[' Price  ', C.label], [display, C.cost]]),
           buildPanelLine(width, [['', C.label]]),
@@ -732,7 +732,7 @@ export class CostTrackerPanel extends BasePanel {
           buildPanelLine(width, [[' [Esc] Close', C.dim]]),
         ];
     const workspace = buildPanelWorkspace(width, height, {
-      title: ' Cost Tracker — Model Price',
+      title: ' Cost Tracker: Model Price',
       intro: editable
         ? 'Your price wins over provider and catalog prices, immediately and everywhere.'
         : 'Manual prices are stored per provider:model.',

package/src/panels/diff-panel.ts

@@ -1,5 +1,5 @@
 // ---------------------------------------------------------------------------
-// DiffPanel — unified diff view of agent file changes
+// DiffPanel, unified diff view of agent file changes
 // ---------------------------------------------------------------------------
 
 import type { Line } from '@pellux/goodvibes-sdk/platform/types';
@@ -22,15 +22,15 @@ import {
 } from './polish.ts';
 
 // ---------------------------------------------------------------------------
-// Colour palette — dedicated diff-viewer scheme (like a mini syntax
+// Colour palette, dedicated diff-viewer scheme (like a mini syntax
 // highlighter). Each hex literal is named once and reused for every role
 // that shares it, including the workspace-chrome aliases (info/dim/value/
 // empty) so the raw-hex count never grows. The title band itself is NOT
-// overridden — buildPanelWorkspace always falls back to the canonical
+// overridden, buildPanelWorkspace always falls back to the canonical
 // DEFAULT_PANEL_PALETTE.headerBg (one title band everywhere).
 // ---------------------------------------------------------------------------
 
-// Hunk blue is the shared DIFF_TONES token — diff-view.ts (conversation)
+// Hunk blue is the shared DIFF_TONES token, diff-view.ts (conversation)
 // and git-panel.ts's inline diff converge onto this file's pre-existing value.
 const HUNK_BLUE: string = DIFF_TONES.hunk;
 // Context rows and line-number gutter use the shared theme's muted/dim
@@ -38,7 +38,7 @@ const HUNK_BLUE: string = DIFF_TONES.hunk;
 const CONTEXT_GRAY = UI_TONES.fg.muted;
 const FILENAME_WHITE = '#ffffff';
 // Add/del text colors are the shared DIFF_TONES tokens, whose values ARE this
-// panel's shipped colors — this panel is the reference diff look (diff-view
+// panel's shipped colors, this panel is the reference diff look (diff-view
 // was unwired dead code until, so "majority of surfaces" was a mirage);
 // the conversation surface converges onto these, not the reverse.
 const ADD_GREEN: string = DIFF_TONES.add;
@@ -56,7 +56,7 @@ const TAB_BG = '#222222';
 const STATUS_BAR_BG = '#444444';
 
 const COLOR = extendPalette(DEFAULT_PANEL_PALETTE, {
-  // Workspace-chrome aliases (title band excluded — no headerBg override)
+  // Workspace-chrome aliases (title band excluded, no headerBg override)
   info:  HUNK_BLUE,
   dim:   CONTEXT_GRAY,
   value: FILENAME_WHITE,
@@ -148,7 +148,7 @@ function parseDiff(raw: string): ParsedLine[] {
       result.push({ kind: 'deletion', text: line.slice(1), beforeNum: before, afterNum: null });
       before++;
     } else if (line.startsWith('\\')) {
-      // "No newline at end of file" note — treat as header
+      // "No newline at end of file" note, treat as header
       result.push({ kind: 'header', text: line, beforeNum: null, afterNum: null });
     } else {
       // context line (starts with space, or empty for blank context)
@@ -168,7 +168,7 @@ function parseDiff(raw: string): ParsedLine[] {
 
 /**
  * Synthetic placeholder entries ('(error)', '(no changes)', '(no staged
- * changes)', '(not a git repo)') are not real diffs — parenthesized so they
+ * changes)', '(not a git repo)') are not real diffs, parenthesized so they
  * can be styled distinctly in the tab/status bar instead of reading as a fake
  * diff of a real file.
  */
@@ -181,7 +181,7 @@ function isPlaceholderPath(filePath: string): boolean {
  * order: the standard two-sided header (`diff --git a/x b/y`, bare or
  * quoted), the combined/merge-conflict header (`diff --cc <path>` /
  * `diff --combined <path>`, which carries no a/b prefixes), then falls back
- * to the `+++`/`---` file lines (present across every variant, uniformly) —
+ * to the `+++`/`---` file lines (present across every variant, uniformly),
  * skipping a `/dev/null` side (deleted/new file). 'unknown' is reserved for
  * genuinely unparseable input.
  */
@@ -274,7 +274,7 @@ export class DiffPanel extends BasePanel {
   private selectedFile = 0;
   private scrollOffset = 0;
 
-  /** One-line confirmation for the 'w'/'h'/'s' self-load hotkeys — otherwise a
+  /** One-line confirmation for the 'w'/'h'/'s' self-load hotkeys, otherwise a
    * reload that produces identical content is visually indistinguishable
    * from the keypress doing nothing at all. */
   private hotkeyStatus: string | null = null;
@@ -320,7 +320,7 @@ export class DiffPanel extends BasePanel {
   }
 
   /**
-   * Ingest a StructuredDiff (GitService.diffStructured — the FULL, uncapped
+   * Ingest a StructuredDiff (GitService.diffStructured, the FULL, uncapped
    * working-tree diff). Each file becomes one DiffEntry (unified text
    * reconstructed) so the existing tabs/renderer work unchanged; the old
    * 4,000-char slice is gone and a diff of any length renders complete.
@@ -340,7 +340,7 @@ export class DiffPanel extends BasePanel {
   }
 
   /**
-   * Defensive gate mirroring GitPanel's own notGitRepo messaging — short-
+   * Defensive gate mirroring GitPanel's own notGitRepo messaging, short-
    * circuits before any of this panel's self-load git spawns run, instead of
    * letting git fail per-subcommand with an inconsistent error shape.
    */
@@ -414,7 +414,7 @@ export class DiffPanel extends BasePanel {
 
   /**
    * Run `git diff --cached` and populate the staged changes. Self-load entry
-   * point for the 's' key — its own diff plumbing, independent of the
+   * point for the 's' key, its own diff plumbing, independent of the
    * `/diff staged` command handler.
    */
   async showStagedDiff(): Promise<void> {
@@ -444,7 +444,7 @@ export class DiffPanel extends BasePanel {
   /**
    * Best-effort semantic-diff enrichment for a set of files, run against the
    * before-content at `ref` and the current on-disk after-content. Self-contained
-   * diff plumbing — does not depend on the `/diff` command handler.
+   * diff plumbing, does not depend on the `/diff` command handler.
    */
   private async enrichSemanticDiffs(files: string[], ref: string): Promise<void> {
     if (files.length === 0) return;
@@ -517,7 +517,7 @@ export class DiffPanel extends BasePanel {
 
   /**
    * Wraps a self-load hotkey ('w'/'h'/'s') with visible before/after status
-   * text in the status bar — otherwise a reload whose content is unchanged
+   * text in the status bar, otherwise a reload whose content is unchanged
    * from what's already shown is indistinguishable from the keypress having
    * done nothing at all (contrast with `/diff working`'s ctx.print() calls).
    */
@@ -540,7 +540,7 @@ export class DiffPanel extends BasePanel {
       // Shift+Tab: most terminals send the bare CSI-Z ("backtab") escape
       // sequence rather than a Tab keypress with a shift modifier, and the
       // input tokenizer passes that sequence through unchanged as the
-      // logical key name — so it (and the friendlier aliases some terminals/
+      // logical key name, so it (and the friendlier aliases some terminals/
       // future tokenizer versions may emit) are matched explicitly here.
       case '\x1b[Z':
       case 'shift-tab':
@@ -559,7 +559,7 @@ export class DiffPanel extends BasePanel {
   // handlePanelIntegrationAction cross-panel hook (the same bridge
   // FileExplorerPanel used). 'preview' is DELETE-disposition with no
   // successor surface (no file-picker-overlay preview to repoint to
-  // either — verified), so the key and the hook were removed rather than
+  // either, verified), so the key and the hook were removed rather than
   // repointed; diff no longer has an 'o' action.
 
   private scrollUp(): void {
@@ -630,7 +630,7 @@ export class DiffPanel extends BasePanel {
             width,
             ' No diff to display.',
             'Load a git diff or select a changed file to populate the workspace. w=working h=HEAD s=staged self-load right here.',
-            // corrected — bare /diff loads files changed *this
+            // corrected, bare /diff loads files changed *this
             // session* (falling back to the full HEAD diff when none are
             // tracked yet), not the working-tree diff the w key already
             // loads in-panel; the old summary text described the wrong
@@ -711,7 +711,7 @@ export class DiffPanel extends BasePanel {
       const placeholder = isPlaceholderPath(entry.filePath);
       const stat = diffStat(entry);
       // Placeholder entries ('(error)', '(no changes)', ...) get a distinct
-      // warn color regardless of active/inactive — they are not a real diff
+      // warn color regardless of active/inactive, they are not a real diff
       // of a real file and must not read as one.
       const fg = placeholder ? COLOR.warn : (active ? COLOR.tabActive : COLOR.tabInactive);
       const bg = active ? COLOR.tabActiveBg : COLOR.tabBg;
@@ -745,7 +745,7 @@ export class DiffPanel extends BasePanel {
     const fileInfo = truncateDisplay(entry.filePath, pathBudget);
     const segments: Array<{ text: string; fg: string; bg?: string; bold?: boolean }> = [
       // Placeholder states ('(error)', '(no changes)', ...) are not a real
-      // diff of a real file — a distinct warn color keeps them from being
+      // diff of a real file, a distinct warn color keeps them from being
       // mistaken for one.
       { text: ` ${fileInfo} `, fg: placeholder ? COLOR.warn : COLOR.filename, bg: COLOR.statusBar, bold: true },
       { text: `[${this.selectedFile + 1}/${this.entries.length}]`, fg: COLOR.tabInactive, bg: COLOR.statusBar },