@pellux/goodvibes-tui 2.0.13 → 2.0.15

package/src/panels/hosted-session-feed.ts

@@ -1,25 +1,25 @@
 /**
- * hosted-session-feed.ts — what this terminal knows about the daemon-hosted
+ * hosted-session-feed.ts, what this terminal knows about the daemon-hosted
  * session it is attached to.
  *
  * One place holds it because two surfaces read it: the `/hosted` command family
  * (which prints status and drives the verbs) and the Hosted Session panel
  * (which renders the conversation). The same shared-feed shape
- * notifications-feed.ts already uses in this repo — a module-level instance a
- * command writes and a panel subscribes to — rather than threading a service
+ * notifications-feed.ts already uses in this repo, a module-level instance a
+ * command writes and a panel subscribes to, rather than threading a service
  * through the command context and the panel deps for one feature.
  *
  * ── What is a fact here, and what is a rendering ──────────────────────────
  *
  * The RECORD is the daemon's, verbatim: status, attached clients, turn count,
- * and `effectiveDetachPolicy` — what leaving would do right now. This feed
+ * and `effectiveDetachPolicy`, what leaving would do right now. This feed
  * never recomputes those; it stores the last record a verb or a lifecycle event
  * handed it.
  *
  * The ROWS are this terminal's rendering of the transcript: the history
  * `attach` backfilled, plus what the live `turn` and `tools` frames have said
  * since. A row that is still being streamed is marked `streaming` so the panel
- * can show it growing rather than waiting for the turn to end — the same thing
+ * can show it growing rather than waiting for the turn to end, the same thing
  * a local turn does.
  *
  * ── Bounded, because it is a live buffer ──────────────────────────────────
@@ -142,7 +142,7 @@ export class HostedSessionFeed {
     this.emit();
   }
 
-  /** Replace the record — every verb answer and every lifecycle notice lands here. */
+  /** Replace the record, every verb answer and every lifecycle notice lands here. */
   setRecord(record: HostedSessionRecord): void {
     if (this.state.record && this.state.record.id !== record.id) return;
     this.state = { ...this.state, record };
@@ -167,7 +167,7 @@ export class HostedSessionFeed {
     close?.();
   }
 
-  /** Append a note of this terminal's own — a detach, a refusal, a stream drop. */
+  /** Append a note of this terminal's own, a detach, a refusal, a stream drop. */
   note(text: string, kind: HostedRowKind = 'system'): void {
     this.appendRow({ kind, text, at: Date.now(), streaming: false });
   }
@@ -231,7 +231,7 @@ export class HostedSessionFeed {
         const reason = readString(event.payload, 'reason');
         this.appendRow({
           kind: 'system',
-          text: reason ? `turn cancelled — ${reason}` : 'turn cancelled',
+          text: reason ? `turn cancelled: ${reason}` : 'turn cancelled',
           at: event.at,
           streaming: false,
         });
@@ -256,7 +256,7 @@ export class HostedSessionFeed {
             { callId, tool, startedAt: event.at },
           ],
         };
-        this.appendRow({ kind: 'tool', text: `${tool} — running`, at: event.at, streaming: true, callId });
+        this.appendRow({ kind: 'tool', text: `${tool}: running`, at: event.at, streaming: true, callId });
         return;
       }
       case 'TOOL_SUCCEEDED':
@@ -265,14 +265,14 @@ export class HostedSessionFeed {
         const outcome = event.type === 'TOOL_SUCCEEDED'
           ? 'done'
           : event.type === 'TOOL_FAILED'
-            ? `failed — ${readString(event.payload, 'error') || 'no reason reported'}`
-            : `cancelled${readString(event.payload, 'reason') ? ` — ${readString(event.payload, 'reason')}` : ''}`;
+            ? `failed: ${readString(event.payload, 'error') || 'no reason reported'}`
+            : `cancelled${readString(event.payload, 'reason') ? `: ${readString(event.payload, 'reason')}` : ''}`;
         this.state = {
           ...this.state,
           runningToolCalls: this.state.runningToolCalls.filter((call) => call.callId !== callId),
           rows: this.state.rows.map((row) => (
             row.callId === callId && row.streaming
-              ? { ...row, text: `${tool} — ${outcome}`, streaming: false }
+              ? { ...row, text: `${tool}: ${outcome}`, streaming: false }
               : row
           )),
         };
@@ -290,7 +290,7 @@ export class HostedSessionFeed {
     if (update.event === 'hosted-session-terminated') {
       this.settleStreamingRow();
       const reason = update.session.terminatedReason ?? 'no reason recorded';
-      this.note(`this hosted session ended — ${reason}${update.detail ? ` (${update.detail})` : ''}`);
+      this.note(`this hosted session ended: ${reason}${update.detail ? ` (${update.detail})` : ''}`);
     }
   }
 

package/src/panels/hosted-session-panel.ts

@@ -1,11 +1,11 @@
 /**
- * HostedSessionPanel — the conversation the daemon is running, rendered here.
+ * HostedSessionPanel, the conversation the daemon is running, rendered here.
  *
  * A hosted session's turn arrives on the same `turn` and `tools` event domains
  * a local turn does, so what this panel shows is the same thing the transcript
  * shows for a local conversation: what was said, what tool is running, and
  * whether text is still streaming. The difference is where the loop lives, and
- * the header says so plainly — including `effectiveDetachPolicy`, so a person
+ * the header says so plainly, including `effectiveDetachPolicy`, so a person
  * about to quit knows whether that ends the work.
  *
  * The panel renders; it decides nothing. Every fact in the header is a field of
@@ -33,7 +33,7 @@ import {
 
 const C = DEFAULT_PANEL_PALETTE;
 
-const INTRO = 'A conversation whose loop runs inside the daemon. Steer it with /hosted say, leave it with /hosted detach — the header says what leaving would do.';
+const INTRO = 'A conversation whose loop runs inside the daemon. Steer it with /hosted say, leave it with /hosted detach; the header says what leaving would do.';
 
 function rowColor(kind: HostedSessionRow['kind']): string {
   switch (kind) {
@@ -80,7 +80,7 @@ export function buildHostedHeaderLines(state: HostedSessionFeedState, width: num
     : 'detaching ends it (kill)';
   const policySource = record.detachPolicy === null ? 'from the setting' : 'per-session override';
   const rows: { text: string; fg: string }[] = [
-    { text: `${record.title.trim() || record.id} — ${record.status}`, fg: C.header },
+    { text: `${record.title.trim() || record.id}: ${record.status}`, fg: C.header },
     { text: `id ${record.id}`, fg: C.dim },
     { text: `workspace ${record.workspaceRoot}`, fg: C.dim },
     {
@@ -93,14 +93,14 @@ export function buildHostedHeaderLines(state: HostedSessionFeedState, width: num
     },
   ];
   if (record.status === 'terminated') {
-    rows.push({ text: `ended — ${record.terminatedReason ?? 'no reason recorded'}`, fg: C.bad });
+    rows.push({ text: `ended: ${record.terminatedReason ?? 'no reason recorded'}`, fg: C.bad });
   }
   if (record.restoredFromDisk) {
     rows.push({ text: 'restored from disk after a daemon restart', fg: C.warn });
   }
   rows.push(state.streaming
     ? { text: 'live event stream open', fg: C.good }
-    : { text: `no live stream — ${state.streamNote ?? 'not subscribed'}`, fg: C.warn });
+    : { text: `no live stream: ${state.streamNote ?? 'not subscribed'}`, fg: C.warn });
   if (state.runningToolCalls.length > 0) {
     rows.push({
       text: `running: ${state.runningToolCalls.map((call) => `${call.tool} (${call.callId})`).join(', ')}`,
@@ -108,7 +108,7 @@ export function buildHostedHeaderLines(state: HostedSessionFeedState, width: num
     });
   }
   if (state.droppedRows > 0) {
-    rows.push({ text: `${state.droppedRows} earlier row(s) dropped by this panel's buffer — reattach to backfill`, fg: C.dim });
+    rows.push({ text: `${state.droppedRows} earlier row(s) dropped by this panel's buffer; reattach to backfill`, fg: C.dim });
   }
   const lines: Line[] = [];
   for (const row of rows) {

package/src/panels/index.ts

@@ -7,7 +7,7 @@ export { CostTrackerPanel } from './cost-tracker-panel.ts';
 export { GitPanel } from './git-panel.ts';
 export { registerBuiltinPanels } from './builtin-panels.ts';
 export type { BuiltinPanelDeps } from './builtin-panels.ts';
-// (the purge) — group B: plugins/skills panels are register-retired but
+// (the purge), group B: plugins/skills panels are register-retired but
 // their modules are retained for shared non-class exports (PluginManagerControls;
 // discoverSkills + SkillRecord/SkillOrigin, consumed by command runtimes and the
 // skills modal). The other group-A and group-B panel classes are deleted outright

package/src/panels/local-auth-panel.ts

@@ -20,7 +20,7 @@ import { isTextBackspace } from '../input/delete-key-policy.ts';
 import { summarizeError } from '@pellux/goodvibes-sdk/platform/utils';
 import { type ConfirmState, handleConfirmInput } from './confirm-state.ts';
 
-// Base chrome only — state colors and text tokens come straight from
+// Base chrome only, state colors and text tokens come straight from
 // DEFAULT_PANEL_PALETTE.
 const C = DEFAULT_PANEL_PALETTE;
 
@@ -40,7 +40,7 @@ interface MaskedEntryState {
   buffer: string;
 }
 
-/** Mutation surface the panel needs for p/a/d/b — a subset of UserAuthManager. */
+/** Mutation surface the panel needs for p/a/d/b, a subset of UserAuthManager. */
 type LocalAuthMutations = Pick<UserAuthManager, 'addUser' | 'deleteUser' | 'rotatePassword' | 'clearBootstrapCredentialFile'>;
 
 function hasLocalAuthMutations(value: LocalAuthInspectionQuery): value is LocalAuthInspectionQuery & LocalAuthMutations {
@@ -54,12 +54,12 @@ function hasLocalAuthMutations(value: LocalAuthInspectionQuery): value is LocalA
 export class LocalAuthPanel extends ScrollableListPanel<LocalAuthUser> {
   private readonly authManager: LocalAuthInspectionQuery;
   private maskedState: MaskedEntryState | null = null;
-  /** Pending delete-user confirmation — project-standard ConfirmState contract. */
+  /** Pending delete-user confirmation, project-standard ConfirmState contract. */
   private deleteConfirm: ConfirmState<string> | null = null;
   /** Draft username buffer for the 'a' add-user flow's first step (username,
-   * then openMaskedEntry for the password). Not masked — usernames aren't secret. */
+   * then openMaskedEntry for the password). Not masked, usernames aren't secret. */
   private usernameEntry: string | null = null;
-  /** Cached inspect() snapshot for the current render pass — see render(). */
+  /** Cached inspect() snapshot for the current render pass, see render(). */
   private cachedSnapshot: LocalAuthSnapshot | null = null;
 
   public constructor(authManager: LocalAuthInspectionQuery) {
@@ -99,7 +99,7 @@ export class LocalAuthPanel extends ScrollableListPanel<LocalAuthUser> {
   /**
    * Masked password/username entry 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
+   * time, same as it always has, see the interface doc on
    * `Panel.isCapturingTextBurst`.
    */
   public override isCapturingTextBurst(): boolean {
@@ -107,7 +107,7 @@ export class LocalAuthPanel extends ScrollableListPanel<LocalAuthUser> {
   }
 
   public override handleInput(key: KeyName): boolean {
-    // Masked entry takes priority when active — it must capture every
+    // Masked entry takes priority when active, it must capture every
     // keystroke (including letters that would otherwise be p/a/d/b actions)
     // as part of the password being typed.
     if (this.maskedState !== null) {
@@ -200,7 +200,7 @@ export class LocalAuthPanel extends ScrollableListPanel<LocalAuthUser> {
       this.invalidate();
       return true;
     }
-    // 'absorbed' (or defensively, 'inactive' — deleteConfirm is non-null here so this never fires)
+    // 'absorbed' (or defensively, 'inactive', deleteConfirm is non-null here so this never fires)
     return true;
   }
 
@@ -242,7 +242,7 @@ export class LocalAuthPanel extends ScrollableListPanel<LocalAuthUser> {
   }
 
   private handleMaskedEntryInput(key: KeyName): boolean {
-    // Masked entry is active — capture all keystrokes.
+    // Masked entry is active, capture all keystrokes.
     const state = this.maskedState!;
 
     if (key === 'escape') {
@@ -330,7 +330,7 @@ export class LocalAuthPanel extends ScrollableListPanel<LocalAuthUser> {
     }
 
     const intro = 'Manage local daemon and HTTP-listener auth users, bootstrap state, and active sessions.';
-    // Single inspect() call for this whole render pass — getItems() (called
+    // Single inspect() call for this whole render pass, getItems() (called
     // both directly below and again inside renderList()'s internal
     // getVisibleItems()) reads from this cache instead of re-inspecting.
     this.cachedSnapshot = this.authManager.inspect();
@@ -338,7 +338,7 @@ export class LocalAuthPanel extends ScrollableListPanel<LocalAuthUser> {
     const users = this.getItems();
 
     const issueMessages: string[] = [];
-    if (snapshot.bootstrapCredentialPresent) issueMessages.push('Bootstrap credential file still exists — press b to clear it after password rotation.');
+    if (snapshot.bootstrapCredentialPresent) issueMessages.push('Bootstrap credential file still exists; press b to clear it after password rotation.');
     if (snapshot.sessionCount === 0) issueMessages.push('No active local auth sessions are currently tracked.');
 
     const headerLines: Line[] = [
@@ -407,7 +407,7 @@ export class LocalAuthPanel extends ScrollableListPanel<LocalAuthUser> {
     });
   }
 
-  /** Footer keyboard hints, adapted to the current state — confirm-pending
+  /** Footer keyboard hints, adapted to the current state, confirm-pending
    * vs. normal browsing, and whether there's a bootstrap credential to clear. */
   private buildHints(
     selected: LocalAuthUser | null,
@@ -435,14 +435,14 @@ export class LocalAuthPanel extends ScrollableListPanel<LocalAuthUser> {
   private renderUsernameEntryPrompt(width: number, height: number): Line[] {
     const draft = this.usernameEntry ?? '';
     const promptLines: Line[] = [
-      buildPanelLine(width, [[' Add local auth user — enter a username, then set a masked password.', C.value]]),
+      buildPanelLine(width, [[' Add local auth user: enter a username, then set a masked password.', C.value]]),
       buildPanelLine(width, [['', C.label]]),
       buildPanelLine(width, [[' Username  ', C.label], [`${draft}█`, C.info]]),
       buildPanelLine(width, [['', C.label]]),
       buildPanelLine(width, [[' [Enter] Continue to password entry   [Esc] Cancel   [Backspace] Delete char', C.dim]]),
     ];
     const workspace = buildPanelWorkspace(width, height, {
-      title: 'Local Auth — Add User',
+      title: 'Local Auth: Add User',
       intro: 'Type the new username and press Enter to continue to masked password entry.',
       sections: [{ lines: promptLines }],
       palette: C,
@@ -467,7 +467,7 @@ export class LocalAuthPanel extends ScrollableListPanel<LocalAuthUser> {
     ];
 
     const workspace = buildPanelWorkspace(width, height, {
-      title: 'Local Auth — Password Entry',
+      title: 'Local Auth: Password Entry',
       intro: `Type a password for ${state.username}. The value is never echoed in plaintext or stored in history.`,
       sections: [{ lines: promptLines }],
       palette: C,

package/src/panels/memory-consolidation-gateway.ts

@@ -2,7 +2,7 @@
 // memory-consolidation-gateway.ts
 //
 // The async, daemon-backed verb the Memory modal's Proposals tab reads:
-// memory.consolidation.receipts (GET /api/memory/consolidation/receipts) — the
+// memory.consolidation.receipts (GET /api/memory/consolidation/receipts), the
 // retained memory-consolidation run receipts plus the flattened
 // pendingProposals (contradiction / cross-scope-duplicate / stale-delete)
 // awaiting human review. The records a proposal references are already
@@ -11,12 +11,12 @@
 // how the Memory modal learns WHAT was proposed and WHY, so it can show that
 // reason next to the affected records and offer a jump to them.
 //
-// No named facade exists on the in-process OperatorClient for this verb, so —
-// exactly like the Fleet gateway (fleet-gateway.ts) — it goes over the
+// No named facade exists on the in-process OperatorClient for this verb, so,
+// exactly like the Fleet gateway (fleet-gateway.ts), it goes over the
 // generic operator invoke path (operator-rpc.ts's resolveOperatorRpc ->
 // sdk.operator.invoke), reaching the SAME daemon the command layer does. A
 // runtime without the consolidation scheduler answers an honest 501; an older
-// daemon that has not wired this route yet answers 404 — both are "verb
+// daemon that has not wired this route yet answers 404, both are "verb
 // unavailable", not "no proposals" (see classifyConsolidationFetchError).
 //
 // The interface is injectable so the modal's proposals fetch round-trips
@@ -30,7 +30,7 @@ import type { OperatorMethodOutput } from '@pellux/goodvibes-sdk';
 import { GoodVibesSdkError } from '@pellux/goodvibes-sdk';
 import { resolveOperatorRpc, describeOperatorRpcError } from '../input/commands/operator-rpc.ts';
 
-/** memory.consolidation.receipts output — retained receipts + the flattened pending proposals. */
+/** memory.consolidation.receipts output, retained receipts + the flattened pending proposals. */
 export type MemoryConsolidationReceiptsResult = OperatorMethodOutput<'memory.consolidation.receipts'>;
 /** One pending judgment proposal (contradiction / cross-scope-duplicate / stale-delete) awaiting human review. */
 export type MemoryConsolidationProposal = MemoryConsolidationReceiptsResult['pendingProposals'][number];
@@ -44,7 +44,7 @@ export interface MemoryConsolidationGateway {
 /**
  * Why the gateway could not be built (daemon disabled / no control-plane URL),
  * surfaced verbatim so the modal can print an honest "unavailable" line
- * rather than guessing — mirrors FleetGatewayResolution.
+ * rather than guessing, mirrors FleetGatewayResolution.
  */
 export type MemoryConsolidationGatewayResolution =
   | { readonly available: true; readonly gateway: MemoryConsolidationGateway }
@@ -57,7 +57,7 @@ export interface MemoryConsolidationGatewayDeps {
 
 /**
  * Build the live consolidation-receipts gateway over the generic operator
- * invoke path — the same daemon resolution the command layer and the Fleet
+ * invoke path, the same daemon resolution the command layer and the Fleet
  * gateway use. Returns an honest unavailable reason when no daemon is
  * reachable, so the modal can refuse cleanly instead of throwing mid-render.
  */
@@ -79,7 +79,7 @@ export type MemoryConsolidationFetchFailure =
 /**
  * Classify a `fetchReceipts()` rejection: a 501 (the documented "no
  * consolidation scheduler on this runtime" response) or a 404 (an older
- * daemon that has not wired this route yet) are both "verb unavailable" —
+ * daemon that has not wired this route yet) are both "verb unavailable",
  * distinct from a generic request failure (network error, 401/403, 500),
  * which renders as an honest fetch error instead. Reuses
  * describeOperatorRpcError's wording so the two call sites (command layer,

package/src/panels/modals/devices-modal.ts

@@ -10,7 +10,7 @@ import type { MintedPairingToken, PublicPairingToken } from '@pellux/goodvibes-s
 import { formatDeviceLine, shortTokenId } from '@pellux/goodvibes-sdk/platform/pairing';
 
 // ---------------------------------------------------------------------------
-// Paired Devices — the settings security-domain device/token management surface.
+// Paired Devices, the settings security-domain device/token management surface.
 // Lists every per-device pairing token (name · created · last-seen · id) and
 // mirrors the pairing.tokens.* gateway verbs: revoke a device, migrate off the
 // legacy shared token, revoke the shared token. Rename needs a name, which the
@@ -19,7 +19,7 @@ import { formatDeviceLine, shortTokenId } from '@pellux/goodvibes-sdk/platform/p
 // fresh every render, so a revoke/migrate reflects immediately.
 // ---------------------------------------------------------------------------
 
-/** The token-store capability this surface needs — satisfied by PairingTokenManager. */
+/** The token-store capability this surface needs, satisfied by PairingTokenManager. */
 export interface DevicesModalPairingTokens {
   list(): PublicPairingToken[];
   rename(id: string, name: string): boolean;
@@ -62,10 +62,10 @@ class DevicesModalSurface implements ConfigModalSurface {
     const now = this.deps.now?.() ?? Date.now();
     const tokens = this.deps.pairingTokens.list();
     const rows: ConfigModalRow[] = [];
-    rows.push(infoRow('intro', 'Devices paired to this daemon. Each has its own token — revoke one without affecting the others.', { dim: true }));
+    rows.push(infoRow('intro', 'Devices paired to this daemon. Each has its own token; revoke one without affecting the others.', { dim: true }));
 
     if (tokens.length === 0) {
-      rows.push(infoRow('empty', 'No devices paired yet — run /pair (or open the pairing QR) to add one.'));
+      rows.push(infoRow('empty', 'No devices paired yet: run /pair (or open the pairing QR) to add one.'));
     } else {
       for (const token of tokens) {
         rows.push({ id: `${DEVICE_ROW_PREFIX}${token.id}`, label: formatDeviceLine(token, now) });
@@ -77,7 +77,7 @@ class DevicesModalSurface implements ConfigModalSurface {
       'legacy',
       legacyRevoked
         ? 'Legacy shared token: revoked.'
-        : 'Legacy shared token: active — migrate a device off it (m), then revoke it (s).',
+        : 'Legacy shared token: active; migrate a device off it (m), then revoke it (s).',
       legacyRevoked ? { dim: true } : { fg: MODAL_TONES.reasoning },
     ));
 
@@ -93,7 +93,7 @@ class DevicesModalSurface implements ConfigModalSurface {
     if (id === 'rename') {
       const deviceId = deviceIdOf(ctx.row);
       if (!deviceId) { ctx.setStatus('Select a device row first.'); return; }
-      // The stable-layout modal cannot capture a name inline — route to the
+      // The stable-layout modal cannot capture a name inline, route to the
       // command with the id pre-filled so the user only types the new name.
       ctx.print(`To rename this device, run:  /devices rename ${shortTokenId(deviceId)} <new name>`);
       return;
@@ -104,7 +104,7 @@ class DevicesModalSurface implements ConfigModalSurface {
       if (!deviceId) { ctx.setStatus('Select a device row first.'); return; }
       const record = tokens.list().find((t) => t.id === deviceId);
       const revoked = tokens.revoke(deviceId);
-      ctx.setStatus(revoked ? `Revoked "${record?.name ?? shortTokenId(deviceId)}" — its token stops working now.` : 'That device was already absent.');
+      ctx.setStatus(revoked ? `Revoked "${record?.name ?? shortTokenId(deviceId)}": its token stops working now.` : 'That device was already absent.');
       this.requestRender();
       return;
     }
@@ -124,7 +124,7 @@ class DevicesModalSurface implements ConfigModalSurface {
     if (id === 'revokeShared') {
       if (tokens.isLegacyRevoked()) { ctx.setStatus('The legacy shared token is already revoked.'); return; }
       tokens.revokeLegacyShared();
-      ctx.setStatus('Revoked the legacy shared token — devices on it must re-pair with their own token.');
+      ctx.setStatus('Revoked the legacy shared token; devices on it must re-pair with their own token.');
       this.requestRender();
       return;
     }

package/src/panels/modals/hooks-modal.ts

@@ -18,7 +18,7 @@ import type {
 // ---------------------------------------------------------------------------
 // Hooks → config-modal surface (group-B port). Two tabs: 'Hooks' (active
 // registrations + contract posture) and 'Activity' (recent runtime activity +
-// managed authoring + last simulation — the panel's expand-activity view, now a
+// managed authoring + last simulation, the panel's expand-activity view, now a
 // dedicated tab). All sources are in-memory (dispatcher/workbench/tracker hold
 // live state), so buildView reads live and refresh() is a no-op. Toggle/remove/
 // simulate route to their `/hooks` command path. Selection-blind port: the

package/src/panels/modals/keybindings-modal.ts

@@ -12,8 +12,8 @@ import { KeybindingsManager } from '../../input/keybindings.ts';
 // Docs + Shortcuts → 'keybindings' config-modal surface (group-B port).
 // A MERGE of the retired DocsPanel (tools/models/shortcuts) and the
 // shortcuts-overlay reference. Three tabs: 'Tools', 'Models', and 'Shortcuts'
-// (the categorized keyboard reference — driven live off keybindingsManager so
-// user overrides show up — followed by an exhaustive 'All Bindings (live)'
+// (the categorized keyboard reference, driven live off keybindingsManager so
+// user overrides show up, followed by an exhaustive 'All Bindings (live)'
 // table). Enter opens the tool-inspector successor ('fleet') or switches the
 // active model, both via the command path (fleet is a panel; /model is a
 // settings mutation). Selection-blind port: the panel's selected tool/model
@@ -114,11 +114,11 @@ class KeybindingsModalSurface implements ConfigModalSurface {
     if (ctx.tabId === 'tools') {
       // item 4: pass the tool name through as a deep-link target.
       // Honest caveat (documented, not a bug to silently paper over): fleet's
-      // ProcessKind set has no 'tool' node — a docs Tools row names a static
+      // ProcessKind set has no 'tool' node, a docs Tools row names a static
       // tool DEFINITION, not a live process, so FleetPanel.receiveDeepLink
       // will not currently find a match and shows its honest "node no longer
       // present" line. The plumbing is still worth wiring now (matches the
-      // work-plan agent/wrfc jumps' shape) — it starts resolving for free the
+      // work-plan agent/wrfc jumps' shape), it starts resolving for free the
       // day fleet grows a per-tool-call node (the retired DocsPanel's own
       // comment already anticipated this: "no filter-by-tool equivalent
       // there yet").
@@ -145,7 +145,7 @@ export function createKeybindingsModalSurface(deps: KeybindingsModalDeps): Confi
 /**
  * Deterministic golden fixture. KeybindingsManager is pointed at a nonexistent
  * config path so it always resolves to DEFAULT_KEYBINDINGS. Tool and model
- * fixtures are frozen literals — no disk, no live registry.
+ * fixtures are frozen literals, no disk, no live registry.
  */
 export function keybindingsModalGoldenSurface(): ConfigModalSurface {
   const keybindingsManager = new KeybindingsManager({ configPath: '/nonexistent/golden-keybindings.json' });

package/src/panels/modals/knowledge-modal.ts

@@ -11,7 +11,7 @@ import { summarizeError } from '@pellux/goodvibes-sdk/platform/utils';
 // ---------------------------------------------------------------------------
 // Knowledge (graph) → config-modal surface (group-B port). Two tabs:
 // 'Browse' (sources/nodes/issues + schedules) and 'Review' (the open-issue
-// queue) — the panel's browse/review toggle, now host tabs. Issue review
+// queue), the panel's browse/review toggle, now host tabs. Issue review
 // mutations (accept/reject/resolve/reopen) route to the `/knowledge
 // review-issue` command. 'm' cross-opens the memory surface. Selection-blind
 // port: the panel's selected-row id/detail is folded into each row label.
@@ -140,7 +140,7 @@ export function createKnowledgeModalSurface(deps: KnowledgeModalDeps): ConfigMod
 }
 
 /**
- * Deterministic golden fixture: fixed node/source/issue/schedule records —
+ * Deterministic golden fixture: fixed node/source/issue/schedule records,
  * no live SDK calls, no random ids, no wall-clock reads.
  */
 export function knowledgeModalGoldenSurface(): ConfigModalSurface {

package/src/panels/modals/local-auth-modal.ts

@@ -4,7 +4,7 @@ import type { UserAuthManager } from '@pellux/goodvibes-sdk/platform/security';
 import { summarizeError } from '@pellux/goodvibes-sdk/platform/utils';
 import { postureLine, kv } from './modal-surface-helpers.ts';
 
-/** Mutation surface the surface needs to gate p/d/b on — a subset of UserAuthManager,
+/** Mutation surface the surface needs to gate p/d/b on, a subset of UserAuthManager,
  *  mirroring LocalAuthPanel's own `hasLocalAuthMutations` guard. */
 type LocalAuthMutations = Pick<UserAuthManager, 'addUser' | 'deleteUser' | 'rotatePassword' | 'clearBootstrapCredentialFile'>;
 
@@ -29,7 +29,7 @@ function usernameOf(row: ConfigModalRow | null): string | null {
  * `inspect()` is synchronous, so buildView reads it live every tick. All
  * mutations (add-user, rotate-password, delete-user, clear-bootstrap-file)
  * dispatch through the existing `/local-auth` command rather than touching
- * the auth manager directly — the command opens masked password entry for
+ * the auth manager directly, the command opens masked password entry for
  * add-user/rotate-password when no password argument is supplied, which is
  * exactly how the retired panel kept plaintext out of history and the
  * transcript. This surface never renders, accepts, or holds a password.
@@ -37,7 +37,7 @@ function usernameOf(row: ConfigModalRow | null): string | null {
 class LocalAuthModalSurface implements ConfigModalSurface {
   readonly name = 'local-auth-modal';
   readonly title = 'Local Auth';
-  /** Cached from the last buildView() — read by the 'b' action's enabledFor,
+  /** Cached from the last buildView(), read by the 'b' action's enabledFor,
    *  set fresh every tick (buildView always runs before enabledFor is
    *  consulted; see settings-sync-modal's `hasStaged` for the same idiom). */
   private bootstrapPresent = false;
@@ -97,7 +97,7 @@ class LocalAuthModalSurface implements ConfigModalSurface {
       // which renders on the LocalAuthPanel and cannot draw or capture input
       // underneath this fullscreen modal. Point the operator at the command
       // (which opens masked entry) rather than dispatching it into a hidden
-      // surface — the secure flow stays a keystroke away, just not from here.
+      // surface, the secure flow stays a keystroke away, just not from here.
       case 'add-user':
         ctx.print('To add a user securely, run  /local-auth add-user <username>  (opens masked password entry).');
         ctx.setStatus('Run /local-auth add-user <username> for masked entry.');

package/src/panels/modals/marketplace-modal.ts

@@ -46,7 +46,7 @@ const KINDS: readonly EcosystemEntryKind[] = ['plugin', 'skill', 'hook-pack', 'p
  * destructive confirm folded into a modal).
  *
  * Honest empty-state: the panel's original "No curated marketplace
- * entries found yet." copy implied a curated remote catalog. It isn't one —
+ * entries found yet." copy implied a curated remote catalog. It isn't one,
  * loadEcosystemCatalog only ever reads .goodvibes/ecosystem/<kind>s.json under
  * the project/home, populated solely by `/marketplace publish` and
  * `/marketplace bundle import`. The empty-state copy below (locked byte-for-byte
@@ -126,7 +126,7 @@ class MarketplaceModalSurface implements ConfigModalSurface {
       if (!this.deps.ecosystemPaths) {
         rows.push(infoRow('empty:unwired', 'Marketplace catalog roots aren’t wired into this session, so there’s nothing to read yet.'));
       } else {
-        rows.push(infoRow('empty:0', 'This is your local plugin, skill, hook-pack, and policy-pack catalog — not a remote store.'));
+        rows.push(infoRow('empty:0', 'This is your local plugin, skill, hook-pack, and policy-pack catalog; not a remote store.'));
         rows.push(infoRow('empty:1', 'It’s empty because nothing has been published or imported into this workspace yet. Entries appear here once you publish a local component or import a bundle.'));
       }
       rows.push(infoRow('empty:title', 'Populate it', { bold: true }));
@@ -152,7 +152,7 @@ class MarketplaceModalSurface implements ConfigModalSurface {
       rows.push(infoRow(`issue:${i}`, `⚠ ${issue}`, { fg: MODAL_TONES.warn }));
     }
 
-    // Catalog list — selection-blind: fold provenance/compat/risk/state into the label.
+    // Catalog list, selection-blind: fold provenance/compat/risk/state into the label.
     for (const row of this.rows) {
       const review = row.review;
       const provenance = row.entry.provenance ?? 'local';