@pellux/goodvibes-tui 2.0.14 → 2.0.16

package/src/export/gist-uploader.ts

@@ -1,5 +1,5 @@
 // ---------------------------------------------------------------------------
-// gist-uploader — upload export content to a GitHub Gist
+// gist-uploader, upload export content to a GitHub Gist
 // ---------------------------------------------------------------------------
 //
 // Architecture: UploadTarget interface with a single GistUploadTarget
@@ -7,12 +7,12 @@
 // UploadTarget without changing the caller in share-runtime.
 //
 // Token resolution:
-//   1. serviceRegistry.resolveAuth('github') — standard service registry path
+//   1. serviceRegistry.resolveAuth('github'), standard service registry path
 //      (configured via .goodvibes/tui/services.json with tokenKey: GITHUB_TOKEN)
 //   2. process.env.GITHUB_TOKEN fallback
 //   3. No token → honest guidance; no upload.
 //
-// Privacy: Gist is created as secret=true (unlisted, not private — anyone with
+// Privacy: Gist is created as secret=true (unlisted, not private, anyone with
 // the URL can view it).
 // ---------------------------------------------------------------------------
 
@@ -23,7 +23,7 @@ export type UploadResult =
   | { ok: false; error: string };
 
 /**
- * UploadTarget — interface for pluggable export upload backends.
+ * UploadTarget, interface for pluggable export upload backends.
  * Future HTTP PUT / other targets implement this.
  */
 export interface UploadTarget {
@@ -41,7 +41,7 @@ export interface GistUploaderOptions {
 }
 
 /**
- * resolveGithubToken — try auth header map then env var.
+ * resolveGithubToken, try auth header map then env var.
  * Returns undefined when no token is available.
  */
 export function resolveGithubToken(
@@ -65,7 +65,7 @@ export function resolveGithubToken(
 }
 
 /**
- * GistUploadTarget — uploads content to a secret (unlisted) GitHub Gist.
+ * GistUploadTarget, uploads content to a secret (unlisted) GitHub Gist.
  */
 export class GistUploadTarget implements UploadTarget {
   private readonly token: string;
@@ -127,7 +127,7 @@ export class GistUploadTarget implements UploadTarget {
 }
 
 /**
- * noTokenGuidance — message printed when no GitHub PAT is found.
+ * noTokenGuidance, message printed when no GitHub PAT is found.
  */
 export const NO_TOKEN_GUIDANCE = [
   'No GitHub token found for --upload.',

package/src/input/autocomplete.ts

@@ -20,7 +20,7 @@ export interface AutocompleteState {
   selectedIndex: number;
   /**
    * (item 4): how many leading entries of `results` belong to the
-   * curated "common" tier — only meaningful when `query === ''` (0
+   * curated "common" tier, only meaningful when `query === ''` (0
    * otherwise, since a typed filter searches everything and the common/rest
    * split no longer applies). The renderer uses this to draw a separator
    * before the alphabetical rest.
@@ -57,11 +57,11 @@ export class AutocompleteEngine {
     this.state.results = this.registry.fuzzyMatch(query);
     this.state.active = true;
     // fuzzyMatch('') sorts the curated common tier first (score 2),
-    // then the alphabetical rest (score 1) — both tiers alphabetical within
+    // then the alphabetical rest (score 1), both tiers alphabetical within
     // themselves. Count the leading common run to tell the renderer where to
     // draw the separator. A non-empty query means every command was scored on
     // its actual match quality, not tier, so the common/rest split does not
-    // apply — the resulting 0 is not a real answer to "how many are common"
+    // apply, the resulting 0 is not a real answer to "how many are common"
     // and is unused because renderAutocompleteOverlay itself gates on the
     // query string too, but zeroing here keeps the field internally honest.
     let commonCount = 0;

package/src/input/bookmark-modal.ts

@@ -1,5 +1,5 @@
 /**
- * BookmarkModal — state management for the /bookmarks command modal.
+ * BookmarkModal, state management for the /bookmarks command modal.
  *
  * Lists bookmarks from BookmarkManager and tracks UI state:
  * selected index, scroll offset, and pending action.

package/src/input/command-args-hint.ts

@@ -4,8 +4,8 @@ import { CLUSTER_SUBCOMMAND_ARG_HINTS } from './commands/cluster-runtime.ts';
 
 /**
  * Per-command subcommand → argument-hint maps. `session`'s entries come
- * straight from session.ts (SESSION_SUBCOMMAND_ARG_HINTS) — the same table
- * the `/session` switch itself is built from — so the two can never drift
+ * straight from session.ts (SESSION_SUBCOMMAND_ARG_HINTS), the same table
+ * the `/session` switch itself is built from, so the two can never drift
  * apart; the others are hand-maintained here since they're small and their
  * owning modules don't export an equivalent table. A drift test
  * (src/test/input/command-args-hint.test.ts) checks every top-level key

package/src/input/command-reference.ts

@@ -1,12 +1,12 @@
 /**
- * command-reference.ts — pure renderer for the generated slash-command
+ * command-reference.ts, pure renderer for the generated slash-command
  * reference (docs/commands-reference.md).
  *
  * The reference is generated from the live command registry (via
  * {@link categorizeBuiltinCommands} in ./commands.ts), never hand-maintained.
  * This module is deliberately side-effect free so both the generator
  * (scripts/project-surfaces.ts) and the drift-check test import the same
- * rendering logic — the committed file and a fresh render must be byte-identical
+ * rendering logic, the committed file and a fresh render must be byte-identical
  * or the gate fails.
  */
 import type { SlashCommand } from './command-registry.ts';
@@ -31,10 +31,10 @@ function slug(text: string): string {
 }
 
 /**
- * renderCommandReferenceMarkdown — deterministic markdown for the full command
+ * renderCommandReferenceMarkdown, deterministic markdown for the full command
  * set. Categories are sorted alphabetically and commands alphabetically by
  * primary name within each category, so the output depends only on the command
- * set and their metadata — not on registration order — which keeps the drift
+ * set and their metadata, not on registration order, which keeps the drift
  * check meaningful (a genuinely new/renamed command moves the bytes; a pure
  * reordering does not).
  */
@@ -50,7 +50,7 @@ export function renderCommandReferenceMarkdown(entries: readonly CategorizedComm
   const total = entries.length;
 
   const lines: string[] = [];
-  lines.push('<!-- GENERATED FILE — DO NOT EDIT BY HAND.');
+  lines.push('<!-- GENERATED FILE: DO NOT EDIT BY HAND.');
   lines.push('     Regenerate with `bun run docs:commands`.');
   lines.push('     Source of truth: the slash-command registry (src/input/commands.ts).');
   lines.push('     A drift check (src/test/release-gates/command-reference-gate.test.ts)');
@@ -69,7 +69,7 @@ export function renderCommandReferenceMarkdown(entries: readonly CategorizedComm
   lines.push('');
   for (const category of categories) {
     const count = byCategory.get(category)?.length ?? 0;
-    lines.push(`- [${escapeCell(category)}](#${slug(category)}) — ${count}`);
+    lines.push(`- [${escapeCell(category)}](#${slug(category)}): ${count}`);
   }
   lines.push('');
 
@@ -83,10 +83,10 @@ export function renderCommandReferenceMarkdown(entries: readonly CategorizedComm
       const name = `\`/${escapeCell(cmd.name)}\``;
       const aliases = (cmd.aliases ?? []).length > 0
         ? (cmd.aliases ?? []).map((a) => `\`/${escapeCell(a)}\``).join(', ')
-        : '—';
+        : 'None';
       const usageText = cmd.usage ?? cmd.argsHint ?? '';
-      const usage = usageText ? `\`${escapeCell(usageText)}\`` : '—';
-      const description = cmd.description ? escapeCell(cmd.description) : '—';
+      const usage = usageText ? `\`${escapeCell(usageText)}\`` : 'None';
+      const description = cmd.description ? escapeCell(cmd.description) : 'None';
       lines.push(`| ${name} | ${aliases} | ${usage} | ${description} |`);
     }
     lines.push('');

package/src/input/command-registry.ts

@@ -55,7 +55,7 @@ export interface CommandRuntimeState {
   sessionId: string;
   /**
    * Cumulative count of direct terminal writes the output guard intercepted
-   * this session — surfaced by /debug. Optional so the SDK's MutableRuntimeState
+   * this session, surfaced by /debug. Optional so the SDK's MutableRuntimeState
    * (which has no such field) stays assignable; the guard sets it lazily on the
    * shared runtime object. (item 1a.)
    */
@@ -83,7 +83,7 @@ export interface CommandUiActions {
   };
   /**
    * Read-only view of pending `[TEXT: pN, M lines]` fold markers currently
-   * sitting in the composer (registerPaste in handler-content-actions.ts) —
+   * sitting in the composer (registerPaste in handler-content-actions.ts),
    * backs /pastes, which previews a folded paste's actual content before the
    * user submits it. A paste over the fold threshold is otherwise invisible
    * until the LLM sees it.
@@ -95,7 +95,7 @@ export interface CommandUiActions {
   cancelGeneration?: () => void;
   /**
    * Cancel JUST the currently-running tool call (the live transcript row),
-   * leaving the turn to continue — the local-session equivalent of the
+   * leaving the turn to continue, the local-session equivalent of the
    * sessions.toolCalls.cancel wire verb over the in-process orchestrator.
    * Returns true when a running call was found and cancelled.
    */
@@ -114,7 +114,7 @@ export interface CommandUiActions {
    * terminal delivers key events and never key-release state.
    */
   toggleVoiceInput?: () => void;
-  /** The current host sleep-ownership projection (power.status.get, flattened) — chip, held-because, lid-split. */
+  /** The current host sleep-ownership projection (power.status.get, flattened), chip, held-because, lid-split. */
   getPowerState?: () => import('../core/power-status.ts').PowerSurfaceState;
   /** Set the daemon-held keep-awake toggle (power.keepAwake.set), acquiring/releasing the inhibitor live; returns the new state. */
   setKeepAwake?: (enabled: boolean) => Promise<import('../core/power-status.ts').PowerSurfaceState>;
@@ -128,7 +128,7 @@ export interface CommandUiActions {
     /** Which config target to write the selected model to. Defaults to 'main'. */
     target?: import('./model-picker.ts').ModelPickerTarget;
     /**
-     * True only when `effort` came from the picker's effort STEP — an explicit
+     * True only when `effort` came from the picker's effort STEP, an explicit
      * user choice, which is the only thing allowed to update the stored
      * preference `provider.reasoningEffort`. The model-only and context-cap
      * commit routes pass a level merely carried over from the previous model;
@@ -139,7 +139,7 @@ export interface CommandUiActions {
   /**
    * Commit an embedding-provider selection from the model picker's
    * 'embeddingProvider' mode (the 'embeddings' target). Deliberately separate
-   * from completeModelSelection — embedding providers are not
+   * from completeModelSelection, embedding providers are not
    * ModelDefinition-shaped, so routing them through the same commit path
    * would require fabricating a fake model object.
    */
@@ -157,7 +157,7 @@ export interface CommandUiActions {
    * Force a full-screen repaint on the next frame (reuses Compositor.resetDiff(),
    * the same call resize/bootstrap already use). Defense-in-depth for command
    * handlers whose spawned subprocess may have written to the real tty (e.g. a
-   * stderr-capture regression) — nulls the diff buffers so the next composite()
+   * stderr-capture regression), nulls the diff buffers so the next composite()
    * repaints over any stray output instead of leaving it until an unrelated resize.
    */
   requestFullRepaint?: () => void;
@@ -175,7 +175,7 @@ export interface CommandShellUiOpeners {
   jumpToBookmark?: (key: string) => void;
   scrollToLine?: (line: number) => void;
   openHelpOverlay?: () => void;
-  /** Toggle the transcript search overlay (same action as Ctrl+F) — backs /find. */
+  /** Toggle the transcript search overlay (same action as Ctrl+F), backs /find. */
   openTranscriptSearch?: () => void;
   openSelection?: (
     title: string,
@@ -185,7 +185,7 @@ export interface CommandShellUiOpeners {
   ) => void;
   openSettingsModal?: (target?: string) => void;
   /**
-   * Open a MIGRATE-TO-MODAL surface by name (purge skeleton — B
+   * Open a MIGRATE-TO-MODAL surface by name (purge skeleton, B
    * command runtimes call this instead of openCommandPanel once their panel
    * is converted to a ModalFactory config). Threaded from ui-openers.ts the
    * same way openSettingsModal is.
@@ -197,7 +197,7 @@ export interface CommandShellUiOpeners {
   getScrollTop?: () => number;
   openPanelPicker?: () => void;
   /**
-   * Open the fuzzy command palette — a searchable picker over every registered
+   * Open the fuzzy command palette, a searchable picker over every registered
    * slash command (generated live from the registry). Bound to Ctrl+K and the
    * /palette command. Selecting a command pre-fills it into the composer.
    */
@@ -207,16 +207,16 @@ export interface CommandShellUiOpeners {
    * truth as the generated command reference (categorizeBuiltinCommands),
    * memoized by the shell. Lets registry-driven surfaces built inside command
    * handlers (/help) group by category without importing the aggregation
-   * module — a static import there would cycle (commands.ts registers those
+   * module, a static import there would cycle (commands.ts registers those
    * very handlers).
    */
   getCommandCategories?: () => ReadonlyMap<string, string>;
   /**
    * Open (and optionally focus) a panel. focus rule: the command path is
-   * "the user is mid-command-flow" — opening a panel this way leaves keyboard
+   * "the user is mid-command-flow", opening a panel this way leaves keyboard
    * focus in the composer by default. Pass `{ focus: true }` for a caller that
    * genuinely wants to grab focus (chords use panelManager.focusPanels()
-   * directly instead of this method, so no current call site needs it — but
+   * directly instead of this method, so no current call site needs it, but
    * the intent is explicit rather than implicit here). `target` is a
    * fleet deep-link jump target forwarded to PanelManager.open.
    */
@@ -265,17 +265,17 @@ export interface CommandSessionServices {
    * fresh Orchestrator's zeroed default.
    */
   readonly hydrateSessionUsage?: () => void;
-  /** The orchestration engine's command-facing facade — see runtime/workstream-services.ts. */
+  /** The orchestration engine's command-facing facade, see runtime/workstream-services.ts. */
   readonly workstreamEngine?: import('@pellux/goodvibes-sdk/platform/orchestration').WorkstreamCommandService;
-  /** The repo source-tree code index — see runtime/code-index-services.ts. */
+  /** The repo source-tree code index, see runtime/code-index-services.ts. */
   readonly codeIndexStore?: import('@pellux/goodvibes-sdk/platform/state').CodeIndexStore;
-  /** Tool-site reindex scheduler — `/codebase status` reports its last activity. */
+  /** Tool-site reindex scheduler, `/codebase status` reports its last activity. */
   readonly codeIndexReindexScheduler?: import('@pellux/goodvibes-sdk/platform/state').CodeIndexReindexScheduler;
-  /** Whether the (default-off) `agent-passive-code-injection` flag is on — for `/codebase status`. */
+  /** Whether the (default-off) `agent-passive-code-injection` flag is on, for `/codebase status`. */
   readonly isPassiveCodeInjectionFlagEnabled?: () => boolean;
   /**
    * The MAIN interactive session's per-turn passive-injection
-   * honesty ring — `Orchestrator.getTurnInjections()`, the main-session
+   * honesty ring, `Orchestrator.getTurnInjections()`, the main-session
    * counterpart to `AgentRecord.turnInjections`. `/recall injections`
    * with no agent id reads this. Optional so command contexts built without an
    * orchestrator (headless, tests) fall back to the honest empty state.
@@ -297,8 +297,8 @@ export interface CommandProviderServices {
 export interface CommandWorkspaceUiServices {
   /**
    * The app's declare-once session-storage handle (runtime/services.ts). Every
-   * command that reads or writes session state — resume, save, the liveness
-   * markers, the rewind anchor sidecar — takes its paths from here instead of
+   * command that reads or writes session state, resume, save, the liveness
+   * markers, the rewind anchor sidecar, takes its paths from here instead of
    * re-deriving a scope from shellPaths, so a command can never land on a
    * different directory than the runtime that wrote the file.
    */
@@ -306,7 +306,7 @@ export interface CommandWorkspaceUiServices {
   keybindingsManager?: KeybindingsManager;
   fileUndoManager?: FileUndoManager;
   workspaceCheckpointManager?: WorkspaceCheckpointManager;
-  /** In-process gateway catalog — the /review hunk-revert flow invokes checkpoints.revertHunk(Preview) over it. */
+  /** In-process gateway catalog, the /review hunk-revert flow invokes checkpoints.revertHunk(Preview) over it. */
   gatewayMethods?: GatewayMethodCatalog;
   workspaceTrustManager?: import('@pellux/goodvibes-sdk/platform/runtime/operations').WorkspaceTrustManager;
   workspaceRegistrationManager?: import('@pellux/goodvibes-sdk/platform/runtime/operations').WorkspaceRegistrationManager;
@@ -325,15 +325,15 @@ export interface CommandWorkspaceServices
 export interface CommandPlatformConfigServices {
   readonly config: DeepReadonly<GoodVibesConfig>;
   readonly configManager: ConfigManager;
-  /** Per-device pairing tokens — backs the /devices command family. */
+  /** Per-device pairing tokens, backs the /devices command family. */
   readonly pairingTokens?: import('@pellux/goodvibes-sdk/platform/pairing').PairingTokenManager;
-  /** Capability gate runtime — feature states derive from domain settings keys; surfaces read live/pending state here. */
+  /** Capability gate runtime, feature states derive from domain settings keys; surfaces read live/pending state here. */
   readonly featureFlagManager?: FeatureFlagManager;
   readonly voiceProviderRegistry?: VoiceProviderRegistry;
   readonly voiceService?: VoiceService;
   /** Direct-command consumer (`/search`) alongside the existing agent-tool consumer. */
   readonly webSearchService?: import('@pellux/goodvibes-sdk/platform/web-search').WebSearchService;
-  /** Direct-command consumer (`/image`) — first production caller of `.generate()`. */
+  /** Direct-command consumer (`/image`), first production caller of `.generate()`. */
   readonly mediaProviders?: import('@pellux/goodvibes-sdk/platform/media').MediaProviderRegistry;
   readonly artifactStore?: import('@pellux/goodvibes-sdk/platform/artifacts').ArtifactStore;
   /** Background daemon/HTTP-listener/relay controller. Mutable: main.ts patches it in post-bootstrap (see bootstrap-command-parts.ts ordering note). */
@@ -378,8 +378,8 @@ export interface CommandContext
     readonly knowledgeApi?: KnowledgeApi;
     /**
      * The cross-surface memory spine client (SDK 1.2.0 full-detach). `/recall`'s
-     * browse/link/queue/export/import subcommands read and write through this —
-     * not `knowledgeApi.memory` — so they fully detach from the local store file
+     * browse/link/queue/export/import subcommands read and write through this,
+     * not `knowledgeApi.memory`, so they fully detach from the local store file
      * when a daemon has been adopted, per
      * docs/decisions/2026-07-06-memory-wire-full-detach.md in the SDK repo.
      * `explain` and the `vector` diagnostic subcommand stay on `knowledgeApi.memory`
@@ -403,9 +403,9 @@ export interface CommandContext
 /**
  * palette curation (item 4): the "common" first tier the slash-command
  * autocomplete dropdown shows before the alphabetical rest when it opens with
- * no filter typed yet (bare '/') — the "132-command palette unranked"
+ * no filter typed yet (bare '/'), the "132-command palette unranked"
  * evaluator finding. Typed filtering (any non-empty query) is completely
- * unaffected — fuzzyMatch still searches every registered command exactly as
+ * unaffected, fuzzyMatch still searches every registered command exactly as
  * before; this only reorders the empty-query case. Curated for breadth across
  * the product's main workflows rather than raw usage frequency: help/config
  * (orientation), panel/model (workspace + provider), recall/codebase/search
@@ -446,7 +446,7 @@ export class CommandRegistry {
   /**
    * Register a command. Also indexes all aliases for O(1) lookup.
    * Throws if the primary name or any alias collides with an existing
-   * registration — silent last-write-wins previously shadowed whole
+   * registration, silent last-write-wins previously shadowed whole
    * commands, so collisions must fail fast at startup (and are caught
    * statically by the alias lint test).
    */
@@ -454,9 +454,9 @@ export class CommandRegistry {
     // Every registered command is rendered with its description in /help, the
     // command palette, and the generated reference (docs/commands-reference.md).
     // A blank description would ship an unexplained row in all three, so it
-    // fails fast here — the release gate red-tests this invariant.
+    // fails fast here, the release gate red-tests this invariant.
     if (!command.description || command.description.trim().length === 0) {
-      throw new Error(`Command registration for "${command.name}" has no description — every command must describe itself.`);
+      throw new Error(`Command registration for "${command.name}" has no description: every command must describe itself.`);
     }
     const existingByName = this.commands.get(command.name) ?? this.aliasIndex.get(command.name);
     if (existingByName) {
@@ -527,7 +527,7 @@ export class CommandRegistry {
 
       if (bestScore > 0 || q === '') {
         // with no query yet, rank the curated common tier (score 2)
-        // ahead of everything else (score 1) — the tie-break below then sorts
+        // ahead of everything else (score 1), the tie-break below then sorts
         // each tier alphabetically, so the result is "common tier, then the
         // alphabetical rest" rather than one flat alphabetical list.
         const score = q === '' ? (COMMON_COMMAND_NAMES.has(cmd.name) ? 2 : 1) : bestScore;

package/src/input/commands/acp-runtime.ts

@@ -1,11 +1,11 @@
 // ---------------------------------------------------------------------------
-// acp-runtime.ts — the scripted path for hosting third-party coding agents,
+// acp-runtime.ts, the scripted path for hosting third-party coding agents,
 // over the SAME verbs the Fleet panel's spawn affordance drives (acp.agents.list
 // / acp.sessions.create). `/agents list` shows what the daemon discovered
 // (read-only, quiet when none); `/agents host <agentId> [dir]` spawns one as a
 // long-lived daemon session that shows up as an acp-agent fleet row. A working
 // directory defaults to the current one; a known workspace root can be named
-// instead — but no path is ever required beyond what the operator already has.
+// instead, but no path is ever required beyond what the operator already has.
 //
 // A structured spawn failure ({binary, stage, message}) is printed verbatim,
 // never left as a hung row: the daemon bounds the handshake and returns a
@@ -37,7 +37,7 @@ export function registerAcpRuntimeCommands(registry: CommandRegistry): void {
           }
           ctx.print([
             'Discovered ACP agents (host with /agents host <id> [directory]):',
-            ...agents.map((a) => `  ${a.id} — ${a.title}  (${a.binaryPath})`),
+            ...agents.map((a) => `  ${a.id}: ${a.title}  (${a.binaryPath})`),
           ].join('\n'));
         } catch (error) {
           ctx.print(`[agents list] ${describeOperatorRpcError(error)}`);
@@ -55,11 +55,11 @@ export function registerAcpRuntimeCommands(registry: CommandRegistry): void {
         try {
           const { hosted, started } = await rpc.sdk.operator.invoke('acp.sessions.create', { agentId, cwd });
           if (hosted.error) {
-            // Structured failure, verbatim — never a hung row.
-            ctx.print(`[agents host] Could not host ${hosted.title || agentId}: ${hosted.error.stage} stage failed for ${hosted.error.binary} — ${hosted.error.message}`);
+            // Structured failure, verbatim, never a hung row.
+            ctx.print(`[agents host] Could not host ${hosted.title || agentId}: ${hosted.error.stage} stage failed for ${hosted.error.binary}; ${hosted.error.message}`);
             return;
           }
-          ctx.print(`[agents host] Hosting ${hosted.title || agentId} in ${cwd}${started ? '' : ' (queued)'} — it appears as an acp-agent row in the fleet; steer and stop it like any agent.`);
+          ctx.print(`[agents host] Hosting ${hosted.title || agentId} in ${cwd}${started ? '' : ' (queued)'}; it appears as an acp-agent row in the fleet; steer and stop it like any agent.`);
         } catch (error) {
           ctx.print(`[agents host] ${describeOperatorRpcError(error)}`);
         }

package/src/input/commands/calendar-runtime.ts

@@ -1,5 +1,5 @@
 /**
- * calendar-runtime.ts — `/calendar`, the terminal surface over the daemon's
+ * calendar-runtime.ts, `/calendar`, the terminal surface over the daemon's
  * calendar capability.
  *
  * As with `/mail`, no calendar logic lives here. CalDAV, ICS parsing, settings
@@ -17,7 +17,7 @@
  * nuisance to undo.
  *
  * `calendar.ics.import` and `calendar.ics.export` are deliberately NOT given a
- * command. They are bulk file-level transfers of a whole calendar — an
+ * command. They are bulk file-level transfers of a whole calendar, an
  * operation whose natural home is a file path and a scheduled job, not a line
  * typed into a coding session. They stay available to any surface through the
  * same catalog; this product simply does not claim to be the right place for
@@ -52,7 +52,7 @@ const MAX_LIST_LIMIT = 200;
  * Render a timestamp for a terminal agenda.
  *
  * An unparseable value is printed verbatim rather than replaced with a
- * placeholder — showing what the server actually returned is more useful than
+ * placeholder, showing what the server actually returned is more useful than
  * hiding it behind "invalid date".
  */
 export function formatWhen(raw: string): string {
@@ -70,12 +70,12 @@ function truncate(value: string, width: number): string {
 
 export function renderAgenda(events: readonly CalendarEvent[]): string {
   if (events.length === 0) return 'No events in this window.';
-  const lines = [`Agenda — ${events.length} event${events.length === 1 ? '' : 's'}:`];
+  const lines = [`Agenda: ${events.length} event${events.length === 1 ? '' : 's'}:`];
   for (const event of events) {
     const where = event.location ? `  @ ${truncate(event.location, 24)}` : '';
     lines.push(`  ${formatWhen(event.start)}  ${truncate(event.title, 44).padEnd(44)}${where}`);
   }
-  lines.push('', '  /calendar get <id> — open one event');
+  lines.push('', '  /calendar get <id>: open one event');
   return lines.join('\n');
 }
 
@@ -95,7 +95,7 @@ export function renderEvent(event: CalendarEvent): string {
 }
 
 /**
- * Split `title | start | end` on unescaped pipes — same separator rule as
+ * Split `title | start | end` on unescaped pipes, same separator rule as
  * `/mail`, for the same reason: titles contain spaces and per-command quoting
  * conventions are their own bug source.
  */
@@ -188,7 +188,7 @@ async function listEvents(ctx: CommandContext, gateway: Gateway, limit: number,
       ctx.print(renderConnectionStatus({
         surface: 'calendar',
         state: 'ready',
-        detail: 'Calendar is connected — the daemon reached the server and returned a result.',
+        detail: 'Calendar is connected: the daemon reached the server and returned a result.',
         nextActions: [],
       }));
     }
@@ -198,7 +198,7 @@ async function listEvents(ctx: CommandContext, gateway: Gateway, limit: number,
 
 async function getEvent(ctx: CommandContext, gateway: Gateway, eventId: string | undefined): Promise<void> {
   if (!eventId) {
-    ctx.print('Usage: /calendar get <id> — the id from /calendar list.');
+    ctx.print('Usage: /calendar get <id>; the id from /calendar list.');
     return;
   }
   await withStatusOnFailure(ctx, gateway, 'get', async () => {
@@ -217,7 +217,7 @@ async function createEvent(ctx: CommandContext, gateway: Gateway, rest: readonly
   if (parsed === null) {
     ctx.print([
       'Usage: /calendar create <title | start | end>',
-      '  Three fields separated by | — for example:',
+      '  Three fields separated by |, for example:',
       '  /calendar create Design review | 2026-08-01T14:00:00Z | 2026-08-01T15:00:00Z',
     ].join('\n'));
     return;

package/src/input/commands/channel-pairing.ts

@@ -1,8 +1,8 @@
 /**
- * channel-pairing.ts — guided `/channel pair` flow.
+ * channel-pairing.ts, guided `/channel pair` flow.
  *
  * Adapters and their credential requirements are driven from the SDK's
- * getBuiltinSetupSchema — the same setup-schema source every GoodVibes
+ * getBuiltinSetupSchema, the same setup-schema source every GoodVibes
  * consumer (TUI, webui, agent) reads, not a hand-maintained list here.
  * Secret (masked) credentials are stored the same secure way onboarding
  * stores them, via persistSecretBackedConfigValue (secret manager + a
@@ -25,7 +25,7 @@ import { persistSecretBackedConfigValue } from '../../config/secret-config.ts';
 import { describeOperatorRpcError, getOperatorRpc } from './operator-rpc.ts';
 
 /**
- * Surfaces pairable through /channel pair — every ChannelSurface except the
+ * Surfaces pairable through /channel pair, every ChannelSurface except the
  * two internal render surfaces ('tui', 'web'), which are not externally
  * paired channels. The `satisfies Record<...>` shape makes this exhaustive:
  * a new ChannelSurface value that isn't listed here fails to compile.
@@ -71,7 +71,7 @@ function fieldConfigured(field: ChannelSetupFieldDescriptor, configManager: Conf
   return fieldValue(field, configManager).length > 0;
 }
 
-/** Credential fields the user must supply — secret (masked) and plain text; booleans/selects are defaulted. */
+/** Credential fields the user must supply, secret (masked) and plain text; booleans/selects are defaulted. */
 function credentialFields(schema: ChannelSetupSchema): ChannelSetupFieldDescriptor[] {
   return schema.fields.filter((field) => field.kind === 'secret' || field.kind === 'string' || field.kind === 'url' || field.kind === 'number');
 }
@@ -87,7 +87,7 @@ function fieldHint(field: ChannelSetupFieldDescriptor, schema: ChannelSetupSchem
 
 function listAdapters(ctx: CommandContext): void {
   const configManager = ctx.platform.configManager;
-  const lines: string[] = ['Channel adapters — pair with /channel pair <surface>:', ''];
+  const lines: string[] = ['Channel adapters: pair with /channel pair <surface>:', ''];
   for (const surface of PAIRABLE_SURFACES) {
     const schema = getBuiltinSetupSchema(surface);
     const creds = credentialFields(schema);
@@ -108,7 +108,7 @@ function renderSurface(schema: ChannelSetupSchema, configManager: ConfigManager)
     const configured = fieldConfigured(field, configManager);
     const kindTag = field.kind === 'secret' ? 'secret' : 'value';
     const status = configured ? 'set' : 'not set';
-    lines.push(`  - ${field.label} (${kindTag}): ${status} — ${fieldHint(field, schema)}`);
+    lines.push(`  - ${field.label} (${kindTag}): ${status}; ${fieldHint(field, schema)}`);
   }
   if (schema.externalSteps.length > 0) {
     lines.push('', 'External setup steps:');
@@ -143,7 +143,7 @@ function promptMissingSecrets(
     ctx.print('[pair] Concealed input is unavailable on this surface; set credentials via /config and re-run verify.');
     return;
   }
-  ctx.print(`[pair] Enter ${field.label} (${field.placeholder || 'value'}) — masked; Enter to store, Esc to stop.`);
+  ctx.print(`[pair] Enter ${field.label} (${field.placeholder || 'value'}): masked; Enter to store, Esc to stop.`);
   ctx.beginConcealedInput({
     label: field.label,
     onSubmit: (value) => {
@@ -171,7 +171,7 @@ function promptMissingSecrets(
 /**
  * Verify a surface: report which declared credentials resolve locally, then
  * perform a REAL round-trip through the daemon (channels.test.send) and
- * report the actual delivered/error outcome — never a fabricated success.
+ * report the actual delivered/error outcome, never a fabricated success.
  */
 async function verifySurface(ctx: CommandContext, schema: ChannelSetupSchema): Promise<void> {
   const configManager = ctx.platform.configManager;
@@ -181,7 +181,7 @@ async function verifySurface(ctx: CommandContext, schema: ChannelSetupSchema): P
   for (const field of creds) {
     const ok = fieldConfigured(field, configManager);
     if (!ok) missing.push(field);
-    lines.push(`  ${ok ? '[ok]  ' : '[fail]'} ${field.label}${ok ? '' : ' — missing'}`);
+    lines.push(`  ${ok ? '[ok]  ' : '[fail]'} ${field.label}${ok ? '' : '; missing'}`);
   }
   lines.push('');
   if (missing.length === 0) {
@@ -203,7 +203,7 @@ async function verifySurface(ctx: CommandContext, schema: ChannelSetupSchema): P
     if (result.delivered) {
       ctx.print(`[verify] delivered:true${result.responseId ? ` (responseId: ${result.responseId})` : ''}${result.address ? ` → ${result.address}` : ''}`);
     } else {
-      ctx.print(`[verify] delivered:false — ${result.error ?? 'the daemon returned no error detail'}`);
+      ctx.print(`[verify] delivered:false; ${result.error ?? 'the daemon returned no error detail'}`);
     }
   } catch (error) {
     ctx.print(`[verify] round-trip request failed: ${describeOperatorRpcError(error)}`);
@@ -240,6 +240,6 @@ export async function runChannelPairing(args: readonly string[], ctx: CommandCon
     ctx.print(`\nNo missing secrets. Verify with: /channel pair ${schema.surface} verify`);
     return;
   }
-  ctx.print(`\nEntering ${missingSecrets.length} secret credential(s) — Esc to stop.`);
+  ctx.print(`\nEntering ${missingSecrets.length} secret credential(s): Esc to stop.`);
   promptMissingSecrets(ctx, schema, missingSecrets, 0);
 }

package/src/input/commands/channel-runtime.ts

@@ -231,7 +231,7 @@ export function registerChannelRuntimeCommands(registry: CommandRegistry): void
       if (sub === 'policy') {
         const configManager = ctx.platform.configManager;
         // Channel policy is persisted by ChannelPolicyManager in
-        // .goodvibes/tui/channels/policies.json — surface via configManager
+        // .goodvibes/tui/channels/policies.json, surface via configManager
         // category (runtime-accessible without a daemon round-trip).
         const surfaces = [
           'slack', 'discord', 'ntfy', 'webhook', 'homeassistant',

package/src/input/commands/checkin-runtime.ts

@@ -1,11 +1,11 @@
 /**
  * checkin-runtime.ts
  *
- * `/checkin` — the proactive check-in surface: config (enabled/cadence/
+ * `/checkin`, the proactive check-in surface: config (enabled/cadence/
  * delivery channel/quiet hours) and its receipt trail, over the operator
  * wire (see operator-rpc.ts for why this goes over HTTP rather than the
- * in-process OperatorClient facade). Off by default; every run — scheduled
- * or manual — leaves a receipt (`checkin.receipts.list`) recording whether it
+ * in-process OperatorClient facade). Off by default; every run, scheduled
+ * or manual, leaves a receipt (`checkin.receipts.list`) recording whether it
  * stayed quiet, delivered a message, or was skipped, so the config surface
  * in /config's Automation group is never the only place this feature's
  * behavior is visible.
@@ -79,7 +79,7 @@ export function registerCheckinRuntimeCommands(registry: CommandRegistry): void
           lines.push('Recent receipts:');
           lines.push(...receipts.map(renderReceipt));
         }
-        lines.push('', '  /checkin run — trigger one now');
+        lines.push('', '  /checkin run: trigger one now');
         ctx.print(lines.join('\n'));
       } catch (error) {
         ctx.print(`[checkin] ${describeOperatorRpcError(error)}`);