@pellux/goodvibes-tui 2.0.14 → 2.0.15

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

@@ -1,5 +1,5 @@
 // ---------------------------------------------------------------------------
-// checkpoint-runtime.ts — /checkpoints, /checkpoint
+// checkpoint-runtime.ts, /checkpoints, /checkpoint
 //
 // UX over WorkspaceCheckpointManager (@pellux/goodvibes-sdk/platform/workspace),
 // the whole-workspace, git-backed snapshot engine wired onto RuntimeServices as
@@ -8,7 +8,7 @@
 //
 // Restoring a checkpoint is now done through the unified, message-anchored
 // /rewind (rewind-runtime.ts), which rewinds files AND/OR conversation to a
-// completed turn — reusing this same checkpoint store for the files half.
+// completed turn, reusing this same checkpoint store for the files half.
 // ---------------------------------------------------------------------------
 
 import { summarizeError } from '@pellux/goodvibes-sdk/platform/utils';
@@ -77,7 +77,7 @@ export function registerCheckpointRuntimeCommands(registry: CommandRegistry): vo
       // them would cost one diff spawn per checkpoint (O(checkpoints)), which
       // stops being cheap once there are more than a handful. /rewind <id>
       // loads exactly one diff, so the exact file list is available there.
-      ctx.print('Use /rewind to preview and restore a completed turn (files and/or conversation) — it reuses these checkpoints for the files half. With no completed turns recorded this run (e.g. right after a restart), /rewind falls back to restoring one of these checkpoints directly, by number or id — files only, no conversation state.');
+      ctx.print('Use /rewind to preview and restore a completed turn (files and/or conversation); it reuses these checkpoints for the files half. With no completed turns recorded this run (e.g. right after a restart), /rewind falls back to restoring one of these checkpoints directly, by number or id; files only, no conversation state.');
     },
   });
 
@@ -96,7 +96,7 @@ export function registerCheckpointRuntimeCommands(registry: CommandRegistry): vo
       try {
         const checkpoint = await mgr.create({ kind: 'manual', label, retentionClass: 'forensic' });
         if (!checkpoint) {
-          ctx.print('No changes since the last checkpoint — nothing to save.');
+          ctx.print('No changes since the last checkpoint; nothing to save.');
           return;
         }
         ctx.print(`Checkpoint created: ${shortId(checkpoint.id)} "${checkpoint.label}" (forensic retention).`);

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

@@ -1,7 +1,7 @@
 /**
  * ci-runtime.ts
  *
- * `/ci` — the CI-watch surface: a one-shot per-job status check (`ci.status`)
+ * `/ci`, the CI-watch surface: a one-shot per-job status check (`ci.status`)
  * and standing watches (`ci.watches.*`) over the operator wire (see
  * operator-rpc.ts for why this goes over HTTP rather than the in-process
  * OperatorClient facade). The doctrine, verbatim from the operator contract's
@@ -10,7 +10,7 @@
  * violations and force the verdict off 'passed'." `platform/ci-watch` (the
  * module implementing that doctrine server-side) is not in this SDK's public
  * export map yet, so the per-job rendering below is a thin, doctrine-faithful
- * mirror of the contract's own `ci.status`/`ci.watches.run` output — every job
+ * mirror of the contract's own `ci.status`/`ci.watches.run` output, every job
  * name and conclusion listed individually, never summarized.
  */
 import type { CommandRegistry } from '../command-registry.ts';
@@ -83,7 +83,7 @@ export function renderWatch(watch: CiWatchSubscription): string {
 export function registerCiRuntimeCommands(registry: CommandRegistry): void {
   registry.register({
     name: 'ci',
-    description: 'CI-watch: one-shot per-job status and standing watches over the operator surface',
+    description: 'CI-watch: one-shot per-job status and standing watches over the operator panel',
     usage: 'status <repo-or-pr> | watch <repo-or-pr> <deliveryChannel> [--fix-session] | watches | unwatch <id>',
     argsHint: '[status|watch|watches|unwatch]',
     async handler(args, ctx) {
@@ -158,7 +158,7 @@ export function registerCiRuntimeCommands(registry: CommandRegistry): void {
             lines.push(`fix-session triggered: ${result.fixSessionTriggered ? 'yes' : 'no'}`);
             ctx.print(lines.join('\n'));
             // Route the spawned session through the shared one-key jump affordance
-            // instead of printing the id for the user to retype — same attach flow
+            // instead of printing the id for the user to retype, same attach flow
             // as the accepted fix-this card.
             if (result.fixSessionId) ctx.armFixSessionAttach?.(result.fixSessionId);
           } catch (error) {

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

@@ -1,5 +1,5 @@
 /**
- * cluster-runtime.ts — `/cluster` in the TUI.
+ * cluster-runtime.ts, `/cluster` in the TUI.
  *
  * The same operations as the `goodvibes-daemon cluster` subcommands, reaching
  * the same daemon verbs through the same caller. There is no cluster logic in
@@ -8,7 +8,7 @@
  *
  * The one honest difference is `join` without arguments. The CLI can stop and
  * ask which group and for the join key; a slash command has nowhere to ask, so
- * it requires `--group` and `--key` and says so — which is exactly what the
+ * it requires `--group` and `--key` and says so, which is exactly what the
  * shared caller already reports for any non-interactive invocation.
  */
 import { resolveDaemonHomeDir } from '@pellux/goodvibes-sdk/platform/workspace';
@@ -33,7 +33,7 @@ export function registerClusterRuntimeCommands(registry: CommandRegistry): void
   registry.register({
     name: 'cluster',
     description: 'Share inbound channel work with your other goodvibes machines on this network',
-    usage: `[${CLUSTER_SUBCOMMANDS.join('|')}] — status is the default; `
+    usage: `[${CLUSTER_SUBCOMMANDS.join('|')}]: status is the default; `
       + 'add --host/--port/--token to reach a daemon on another machine',
     argsHint: `[${CLUSTER_SUBCOMMANDS.join('|')}]`,
     async handler(args, ctx) {

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

@@ -1,5 +1,5 @@
 // ---------------------------------------------------------------------------
-// codebase-runtime.ts — /codebase
+// codebase-runtime.ts, /codebase
 //
 // Explicit-query + transparency surface over the repo source-tree code index
 // (CodeIndexStore, @pellux/goodvibes-sdk/platform/state, landed on SDK main
@@ -10,10 +10,10 @@
 //
 // Mirrors /recall's transparency idiom (recall-query.ts's handleRecallVector:
 // status/rebuild) and /workstream's registerXRuntimeCommands function-export
-// shape: build (schedules the index build — visible as a fleet 'code-index'
+// shape: build (schedules the index build, visible as a fleet 'code-index'
 // node while it runs), status (counts, skips, degradation state, last
 // build), search <query> (explicit retrieval, results labeled
-// 'lexical'|'semantic' honestly — never implied as more precise than they
+// 'lexical'|'semantic' honestly, never implied as more precise than they
 // are). Passive auto-injection into coding turns and tool-site incremental
 // reindex followed later; `status` surfaces both honestly (the
 // auto-injection flag+setting gate, and the last reindex activity).
@@ -37,15 +37,15 @@ function formatBytes(bytes: number): string {
 /**
  * Honest one-line auto-injection state: both the (default-off)
  * `agent-passive-code-injection` feature flag AND storage.codeIndexEnabled must
- * be on for passive code injection to run — state which is off, and why.
+ * be on for passive code injection to run, state which is off, and why.
  */
 function renderAutoInjectionState(flagEnabled: boolean, settingEnabled: boolean): string {
   if (flagEnabled && settingEnabled) {
     return '  auto-injection: on (code hits may be injected into coding turns within the shared knowledge budget)';
   }
   const reasons: string[] = [];
-  if (!flagEnabled) reasons.push('agent-passive-code-injection flag off (default off) — enable in the settings modal flags section');
-  if (!settingEnabled) reasons.push('storage.codeIndexEnabled off — /config to change');
+  if (!flagEnabled) reasons.push('agent-passive-code-injection flag off (default off); enable in the settings modal flags section');
+  if (!settingEnabled) reasons.push('storage.codeIndexEnabled off: /config to change');
   return `  auto-injection: off (${reasons.join('; ')})`;
 }
 
@@ -58,7 +58,7 @@ function renderReindexActivity(activity: CodeIndexReindexActivity | null, now: n
     : activity.status === 'skipped'
       ? `skipped (${activity.mode ?? 'empty'})`
       : `indexed (${activity.mode ?? 'symbols'})`;
-  return `  last reindex: ${activity.path} — ${detail}, ${agoSec}s ago`;
+  return `  last reindex: ${activity.path}; ${detail}, ${agoSec}s ago`;
 }
 
 interface CodeIndexStatusExtras {
@@ -69,18 +69,18 @@ interface CodeIndexStatusExtras {
 
 function renderCodeIndexStatus(stats: CodeIndexStats, configManager: Pick<ConfigManager, 'get'>, extras: CodeIndexStatusExtras): string {
   const lines: string[] = [];
-  lines.push(`Code index — backend: ${stats.backend}, available: ${stats.available ? 'yes' : 'no'}`);
+  lines.push(`Code index: backend: ${stats.backend}, available: ${stats.available ? 'yes' : 'no'}`);
   lines.push(`  path: ${stats.path}`);
   lines.push(`  indexed: ${stats.indexedFiles} file(s), ${stats.indexedChunks} chunk(s), ${stats.dimensions} dims`);
   lines.push(`  embedding provider: ${stats.embeddingProviderLabel} (${stats.embeddingProviderId})`);
   lines.push(
     stats.semanticRetrievalAvailable
       ? '  search results labeled: semantic'
-      : '  search results labeled: lexical (hashed fallback — low precision)',
+      : '  search results labeled: lexical (hashed fallback; low precision)',
   );
   const autoStart = isCodeIndexAutoStartEnabled(configManager);
   lines.push(
-    `  auto-build on startup: ${autoStart ? 'on' : 'off'} (storage.codeIndexEnabled, default off — /config to change)`,
+    `  auto-build on startup: ${autoStart ? 'on' : 'off'} (storage.codeIndexEnabled, default off; /config to change)`,
   );
   // Passive code auto-injection state (flag + setting) and the last tool-site reindex.
   lines.push(renderAutoInjectionState(extras.flagEnabled, autoStart));
@@ -91,9 +91,9 @@ function renderCodeIndexStatus(stats: CodeIndexStats, configManager: Pick<Config
   );
   // Provider-space honesty (SDK finding): vectors embedded under a different
   // provider than the current default disable the vector search path until a
-  // rebuild re-embeds — say so, in the store's own words.
+  // rebuild re-embeds, say so, in the store's own words.
   if (stats.embeddingProviderMismatch) {
-    lines.push(`  provider mismatch: ${stats.embeddingProviderMismatch} (vector search disabled — lexical fallback only)`);
+    lines.push(`  provider mismatch: ${stats.embeddingProviderMismatch} (vector search disabled; lexical fallback only)`);
   }
 
   if (stats.building) {
@@ -115,7 +115,7 @@ function renderCodeIndexStatus(stats: CodeIndexStats, configManager: Pick<Config
     if (skip.chunkedByWindow) skipParts.push(`${skip.chunkedByWindow} windowed (no tree-sitter symbols)`);
     lines.push(skipParts.length > 0 ? `  skipped: ${skipParts.join(', ')}` : '  skipped: none');
   } else {
-    lines.push('  last build: never — run /codebase build');
+    lines.push('  last build: never; run /codebase build');
   }
 
   if (stats.error) lines.push(`  error: ${stats.error}`);
@@ -170,7 +170,7 @@ const USAGE = 'Usage:\n'
 export function registerCodebaseRuntimeCommands(registry: CommandRegistry): void {
   registry.register({
     name: 'codebase',
-    description: 'Repo source-tree code index — build, inspect, and search',
+    description: 'Repo source-tree code index: build, inspect, and search',
     usage: 'build | status | search <query...> [--limit n]',
     argsHint: 'build | status | search <query>',
     handler(args: string[], ctx: CommandContext) {
@@ -201,7 +201,7 @@ export function registerCodebaseRuntimeCommands(registry: CommandRegistry): void
           return;
         }
         store.scheduleBuild();
-        ctx.print('Build scheduled — track progress with /codebase status or the fleet panel (code-index node).');
+        ctx.print('Build scheduled: track progress with /codebase status or the fleet panel (code-index node).');
         return;
       }
 
@@ -217,7 +217,7 @@ export function registerCodebaseRuntimeCommands(registry: CommandRegistry): void
           const stats = store.stats();
           ctx.print(
             stats.indexedChunks === 0
-              ? 'No results — the code index is empty. Run /codebase build first.'
+              ? 'No results: the code index is empty. Run /codebase build first.'
               : 'No matching chunks found.',
           );
           return;

package/src/input/commands/config.ts

@@ -18,13 +18,13 @@ export function registerConfigCommand(registry: CommandRegistry): void {
     argsHint: '[category|key] | set <key> <value>',
     async handler(args, ctx) {
       // An earlier replay fix: `/config set <key> <value>` used to fall through to
-      // the workspace with the assignment silently ignored — the dishonest-
+      // the workspace with the assignment silently ignored, the dishonest-
       // fallthrough class. `set` is now a real verb.
       if (args[0] === 'set') {
         const key = args[1];
         const value = args.slice(2).join(' ');
         if (!key || value === '') {
-          ctx.print('Usage: /config set <key> <value> — e.g. /config set display.themeMode light');
+          ctx.print('Usage: /config set <key> <value>; e.g. /config set display.themeMode light');
           return;
         }
         // A credential key must never take the generic path below: setDynamic
@@ -72,9 +72,9 @@ export function registerConfigCommand(registry: CommandRegistry): void {
  * writing it into a config file: the value goes to the secret store and the
  * config key keeps only a `goodvibes://secrets/...` reference to it.
  *
- * The entered value is never echoed. The transcript is a diagnostic surface —
+ * The entered value is never echoed. The transcript is a diagnostic surface,
  * it is scrolled back through, copied into support threads and captured in
- * screen recordings — so it gets the reference and the store key, never the
+ * screen recordings, so it gets the reference and the store key, never the
  * secret.
  *
  * With no secret manager in this runtime there is nowhere safe to put it, and
@@ -89,7 +89,7 @@ async function handleSecretConfigSet(
   const secretsManager = ctx.platform?.secretsManager;
   if (!secretsManager) {
     ctx.print(
-      `${key} holds a credential, and this runtime has no secret store to put it in — `
+      `${key} holds a credential, and this runtime has no secret store to put it in; `
       + `refusing rather than writing it in cleartext into a config file.\n`
       + `  Store it with: /secrets set ${secretKey} <value>`,
     );
@@ -105,7 +105,7 @@ async function handleSecretConfigSet(
     const scope = defaultSecretBackedScope(key);
     ctx.print(
       `Set ${key}: stored in the ${scope} secret tier as ${secretKey} (value hidden).\n`
-      + `  ${key} now holds the reference ${configValue} — the value itself is never written to a config file.`
+      + `  ${key} now holds the reference ${configValue}; the value itself is never written to a config file.`
       + (scope === 'daemon'
         ? '\n  The daemon reads that tier, so this keeps working with this client closed.'
         : ''),

package/src/input/commands/connection-status.ts

@@ -1,5 +1,5 @@
 /**
- * connection-status.ts — honest connection state for the mail and calendar
+ * connection-status.ts, honest connection state for the mail and calendar
  * capabilities, derived from the daemon's own answer rather than re-decided
  * here.
  *
@@ -7,7 +7,7 @@
  * settings resolution, the credential read, the connectors, the confirmation
  * posture (see src/daemon/handlers/email and src/daemon/handlers/calendar).
  * The TUI holds no second copy of any of it. So "is mail connected?" is not a
- * question this file answers from config — it is a question it ASKS, by
+ * question this file answers from config, it is a question it ASKS, by
  * invoking the cheapest read verb and reading the failure code that comes back:
  *
  *   EMAIL_NOT_CONFIGURED       host/user are missing
@@ -23,7 +23,7 @@
  * Every `email.*` / `calendar.*` descriptor the SDK catalogs carries
  * `invokable: false`. On the generic HTTP/WS dispatch path that flag is checked
  * by `validateGatewayInvocation` BEFORE any handler lookup and answered with a
- * flat 400 `NOT_INVOKABLE` — so `sdk.operator.invoke('email.inbox.list', …)`
+ * flat 400 `NOT_INVOKABLE`, so `sdk.operator.invoke('email.inbox.list', …)`
  * cannot reach these verbs no matter what is configured, and a surface built on
  * it would be a dead button.
  *
@@ -32,7 +32,7 @@
  * own method works. This product registers those handlers into its own catalog
  * at composition time (`createDaemonHandlerComposition`, called by the shared
  * `createRuntimeServices` root that both the TUI and the daemon boot from), so
- * the in-process catalog is a live, correct route to the same code — the same
+ * the in-process catalog is a live, correct route to the same code, the same
  * seam `/review` already uses for `checkpoints.*`. That is what these surfaces
  * call.
  *
@@ -48,7 +48,7 @@ export type ConnectionSurface = 'mail' | 'calendar';
  * What a surface can be.
  *
  * `checking` exists because the probe is real I/O and the settings workspace
- * renders synchronously — a row has to say something true while the answer is
+ * renders synchronously, a row has to say something true while the answer is
  * in flight, and "checking" is true where "not configured" would be invented.
  */
 export type ConnectionState = 'ready' | 'needs-setup' | 'unreachable' | 'checking';
@@ -67,7 +67,7 @@ export function connectionSurfaceLabel(surface: ConnectionSurface): string {
   return surface === 'mail' ? 'Mail' : 'Calendar';
 }
 
-/** The read verb each surface is probed with — the cheapest read it has. */
+/** The read verb each surface is probed with, the cheapest read it has. */
 export const CONNECTION_PROBE_METHOD = {
   mail: 'email.inbox.list',
   calendar: 'calendar.events.list',
@@ -78,12 +78,12 @@ export const CONNECTION_PROBE_METHOD = {
  *
  * The `surfaces.*` settings are daemon-owned by prefix (see the SDK's
  * `config-ownership`), so `/config set` on them lands in the daemon's own
- * store and keeps working with this client closed — which is the requirement.
+ * store and keeps working with this client closed, which is the requirement.
  *
  * The PASSWORDS now have it too, and these steps changed the round the platform
  * gave it to them. A credential is filed in the daemon's tier only when a
  * daemon-owned config path declares it, and the two keys this product's daemon
- * reads — `surfaces.email.password` and `surfaces.calendar.caldavPassword` —
+ * reads, `surfaces.email.password` and `surfaces.calendar.caldavPassword`,
  * were declared in neither `CONFIG_SCHEMA` nor the platform's non-schema
  * daemon-owned path list. `isDaemonOwnedSecretKey()` answered false for both,
  * so a `/secrets set` filed them in THIS client's store where the daemon never
@@ -144,7 +144,7 @@ export function daemonErrorCode(error: unknown): string | null {
 }
 
 /**
- * Turn a probe outcome into a status. Pure — the caller does the I/O, so every
+ * Turn a probe outcome into a status. Pure, the caller does the I/O, so every
  * daemon answer is exercised in tests without a daemon.
  *
  * `error === null` means the read verb returned. That is the only evidence
@@ -160,7 +160,7 @@ export function describeConnectionProbe(
     return {
       surface,
       state: 'ready',
-      detail: `${label} is connected — the daemon reached the server and returned a result.`,
+      detail: `${label} is connected: the daemon reached the server and returned a result.`,
       nextActions: [],
     };
   }
@@ -215,7 +215,7 @@ export const READ_INVOCATION_CONTEXT = { context: { clientKind: 'tui' as const }
  * The daemon's register wrapper demands BOTH `body.confirm === true` and
  * `context.metadata.explicitUserRequest === true`. The second is set only on
  * paths where a person typed the command that performs the write, which is the
- * whole point of the flag — it is never set on a read or on a background
+ * whole point of the flag, it is never set on a read or on a background
  * refresh.
  */
 export const EXPLICIT_WRITE_INVOCATION_CONTEXT = {
@@ -225,7 +225,7 @@ export const EXPLICIT_WRITE_INVOCATION_CONTEXT = {
 /**
  * Probe one surface against the in-process gateway catalog.
  *
- * Asks for a single item — enough to prove the whole path (settings →
+ * Asks for a single item, enough to prove the whole path (settings →
  * credential → connector → server) without pulling a mailbox into memory.
  */
 export async function probeConnection(

package/src/input/commands/context-window.ts

@@ -1,5 +1,5 @@
 /**
- * /context window — view, set, or clear a custom context window for the
+ * /context window, view, set, or clear a custom context window for the
  * current model.
  *
  * The override is stored by the SDK's ProviderRegistry (persisted under the
@@ -84,7 +84,7 @@ export function handleContextWindowSubcommand(args: readonly string[], ctx: Comm
   } else {
     const size = parseContextWindowSize(arg);
     if (size === null) {
-      output = `Invalid size '${args[0]}'. Use a token count between 1 and ${MAX_CONTEXT_WINDOW_OVERRIDE.toLocaleString()} — e.g. 120000, 200k, 1m — or 'clear'.`;
+      output = `Invalid size '${args[0]}'. Use a token count between 1 and ${MAX_CONTEXT_WINDOW_OVERRIDE.toLocaleString()}, e.g. 120000, 200k, 1m, or 'clear'.`;
     } else {
       registry.setModelContextCap(model.registryKey, size);
       output = `Context window for ${model.displayName} set to ${size.toLocaleString()} tokens (was ${registry.getContextWindowForModel(model).toLocaleString()}). Clear with /context window clear.`;

package/src/input/commands/cost-attribution-format.ts

@@ -1,11 +1,11 @@
 /**
- * cost-attribution-format.ts — pure rendering helpers for /cost attribution.
+ * cost-attribution-format.ts, pure rendering helpers for /cost attribution.
  *
  * cost.attribution.get (SDK 1.6.1) is honest-unpriced by design: an unknown
  * model contributes to unpricedRecordCount with a null cost rather than a
  * fabricated amount, and costState is 'priced' | 'estimated' | 'unpriced'
  * (a mix across contributors). These helpers preserve that distinction all
- * the way to the rendered line — "unpriced" always reads as unpriced,
+ * the way to the rendered line, "unpriced" always reads as unpriced,
  * "estimated" is always labeled, and a dollar figure only ever comes from a
  * real priced/estimated costUsd.
  */
@@ -21,7 +21,7 @@ export const COST_ATTRIBUTION_PRIMARY_DIMENSIONS: readonly CostAttributionDimens
 /**
  * Dimensions that exist on the wire but return honest-empty until their
  * emit sites tag records with this dimension. Queried the same as the
- * primary set, but rendered ONLY when non-empty — never a fabricated
+ * primary set, but rendered ONLY when non-empty, never a fabricated
  * "0 tool calls" section implying tool-level attribution works today.
  */
 export const COST_ATTRIBUTION_OPTIONAL_DIMENSIONS: readonly CostAttributionDimension[] = ['tool', 'hook', 'mcp'];
@@ -47,6 +47,6 @@ export function formatCostAttributionSection(result: CostAttributionResult, isOp
   for (const row of result.rows) {
     lines.push(`  ${row.key.padEnd(24)} ${formatCostAmount(row.costUsd, row.costState).padEnd(20)} ${formatTokens(row.tokens)}`);
   }
-  lines.push(`  total: ${formatCostAmount(result.totalCostUsd, result.costState)} — ${result.pricedRecordCount} priced, ${result.unpricedRecordCount} unpriced record(s)`);
+  lines.push(`  total: ${formatCostAmount(result.totalCostUsd, result.costState)}; ${result.pricedRecordCount} priced, ${result.unpricedRecordCount} unpriced record(s)`);
   return lines;
 }

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

@@ -11,7 +11,7 @@ import {
 } from './cost-attribution-format.ts';
 
 /**
- * /cost budget <usd> — makes the CostTrackerPanel's budget alert real.
+ * /cost budget <usd>, makes the CostTrackerPanel's budget alert real.
  * Opens the cost panel (creating it via the registered factory if not already
  * open) and sets its budget threshold directly, so the meter+alert already
  * built into CostTrackerPanel.render() (:266-290) actually fires. 0 disables
@@ -73,7 +73,7 @@ export function registerCostRuntimeCommands(registry: CommandRegistry): void {
           ctx.print(JSON.stringify(results, null, 2));
           return;
         }
-        const lines: string[] = [`Cost Attribution — ${window}`];
+        const lines: string[] = [`Cost Attribution: ${window}`];
         for (const result of results) {
           const isOptional = (COST_ATTRIBUTION_OPTIONAL_DIMENSIONS as readonly string[]).includes(result.dimension);
           const section = formatCostAttributionSection(result, isOptional);

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

@@ -2,7 +2,7 @@ import type { CommandContext, CommandRegistry } from '../command-registry.ts';
 import { formatDeviceLine, resolveTokenByIdPrefix } from '@pellux/goodvibes-sdk/platform/pairing';
 
 /**
- * Register the /devices command — manage the per-device pairing tokens minted by
+ * Register the /devices command, manage the per-device pairing tokens minted by
  * the pairing surfaces. Mirrors the pairing.tokens.* gateway verbs the daemon
  * serves (list/rename/revoke/migrate/revoke-shared) so the terminal has the same
  * capability as the web app's device manager.
@@ -42,7 +42,7 @@ export function registerDevicesRuntimeCommands(registry: CommandRegistry): void
         }
         const resolved = resolveTokenByIdPrefix(pairingTokens.list(), idOrPrefix);
         if (!resolved.ok) {
-          ctx.print(resolved.reason === 'ambiguous' ? `Ambiguous device id "${idOrPrefix}" — use more characters.` : `No device matches "${idOrPrefix}".`);
+          ctx.print(resolved.reason === 'ambiguous' ? `Ambiguous device id "${idOrPrefix}": use more characters.` : `No device matches "${idOrPrefix}".`);
           return;
         }
         pairingTokens.rename(resolved.token.id, name);
@@ -58,11 +58,11 @@ export function registerDevicesRuntimeCommands(registry: CommandRegistry): void
         }
         const resolved = resolveTokenByIdPrefix(pairingTokens.list(), idOrPrefix);
         if (!resolved.ok) {
-          ctx.print(resolved.reason === 'ambiguous' ? `Ambiguous device id "${idOrPrefix}" — use more characters.` : `No device matches "${idOrPrefix}".`);
+          ctx.print(resolved.reason === 'ambiguous' ? `Ambiguous device id "${idOrPrefix}": use more characters.` : `No device matches "${idOrPrefix}".`);
           return;
         }
         const revoked = pairingTokens.revoke(resolved.token.id);
-        ctx.print(revoked ? `Revoked "${resolved.token.name}" — its token stops working immediately.` : `Device "${resolved.token.name}" was already absent.`);
+        ctx.print(revoked ? `Revoked "${resolved.token.name}": its token stops working immediately.` : `Device "${resolved.token.name}" was already absent.`);
         return;
       }
 
@@ -82,7 +82,7 @@ export function registerDevicesRuntimeCommands(registry: CommandRegistry): void
           return;
         }
         pairingTokens.revokeLegacyShared();
-        ctx.print('Revoked the legacy shared token — devices still on it must re-pair with their own token.');
+        ctx.print('Revoked the legacy shared token; devices still on it must re-pair with their own token.');
         return;
       }
 
@@ -95,7 +95,7 @@ function renderList(ctx: CommandContext, pairingTokens: NonNullable<CommandConte
   const list = pairingTokens.list();
   const lines: string[] = ['Paired devices:'];
   if (list.length === 0) {
-    lines.push('  (none yet — pair one with /pair)');
+    lines.push('  (none yet: pair one with /pair)');
   } else {
     for (const token of list) lines.push(`  ${formatDeviceLine(token)}`);
   }
@@ -103,7 +103,7 @@ function renderList(ctx: CommandContext, pairingTokens: NonNullable<CommandConte
     '',
     pairingTokens.isLegacyRevoked()
       ? 'Legacy shared token: revoked.'
-      : 'Legacy shared token: active — run /devices migrate-shared then /devices revoke-shared to retire it.',
+      : 'Legacy shared token: active; run /devices migrate-shared then /devices revoke-shared to retire it.',
   );
   ctx.print(lines.join('\n'));
 }

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

@@ -7,7 +7,7 @@ import { requirePanelManager, requireSessionChangeTracker, requireShellPaths } f
 /**
  * Run a git command that lists changed file names, capturing stderr instead
  * of letting it fall through to the process's real stderr (which, for a
- * spawned child with no stdio option, is inherited from the parent — i.e.
+ * spawned child with no stdio option, is inherited from the parent, i.e.
  * written straight to the TUI's controlling terminal, corrupting the screen
  * outside the renderer's front/back-buffer diffing). Mirrors the 'staged'
  * subcommand's existing pattern below.
@@ -89,7 +89,7 @@ export function registerDiffRuntimeCommands(registry: CommandRegistry): void {
       const shellPaths = requireShellPaths(ctx);
       const workingDirectory = shellPaths.workingDirectory;
       // Gate before any git spawn happens (same defensive check /git already
-      // has) — every subcommand below shells out to git, and without this
+      // has), every subcommand below shells out to git, and without this
       // they each produced a differently-shaped error for the same
       // not-a-git-repo condition.
       if (!GitService.isGitRepo(workingDirectory)) {

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

@@ -1,7 +1,7 @@
 import type { CommandRegistry } from '../command-registry.ts';
 
 /**
- * `/editor` — open the current composer draft in the user's $EDITOR / $VISUAL,
+ * `/editor`, open the current composer draft in the user's $EDITOR / $VISUAL,
  * then load the edited text back into the composer. The actual terminal
  * suspend/resume and buffer round-trip are wired in main.ts as the
  * `openComposerEditor` command action; this command is the front door for it.

package/src/input/commands/eval.ts

@@ -3,10 +3,10 @@
  *
  * Implements the Evaluation Harness commands:
  *
- *   /eval list                    — List all available eval suites
- *   /eval run <suite>             — Run a named suite (or 'all')
- *   /eval compare <baseline-file> — Compare last run against a baseline file
- *   /eval gate <suite>            — Run suite and apply CI gate (exits 1 on regression)
+ *   /eval list                   , List all available eval suites
+ *   /eval run <suite>            , Run a named suite (or 'all')
+ *   /eval compare <baseline-file>, Compare last run against a baseline file
+ *   /eval gate <suite>           , Run suite and apply CI gate (exits 1 on regression)
  */
 
 import type { SlashCommand, CommandContext } from '../command-registry.ts';

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

@@ -237,7 +237,7 @@ export function registerExperienceRuntimeCommands(registry: CommandRegistry): vo
         // download), not through a daemon verb, so this path needs no gateway.
         // Installation already provisions them and a daemon retries at boot;
         // `setup` here is the recovery act, and nothing downloads unless the user
-        // typed it — see wake-provision-runner.ts.
+        // typed it, see wake-provision-runner.ts.
         const wakeSub = (args[1] ?? 'status').toLowerCase();
         const managedRoot = shellPaths.resolveUserPath('voice');
         const runner = {

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

@@ -7,7 +7,7 @@ export function registerGitRuntimeCommands(registry: CommandRegistry): void {
   registry.register({
     name: 'git',
     aliases: ['g'],
-    description: 'Git repository commands — status, log, diff',
+    description: 'Git repository commands: status, log, diff',
     usage: '[status|log|diff]',
     argsHint: '[status|log|diff]',
     async handler(args, ctx) {
@@ -29,7 +29,7 @@ export function registerGitRuntimeCommands(registry: CommandRegistry): void {
             const st = await git.status();
             const lines: string[] = ['Git status:'];
             if (st.isClean()) {
-              lines.push('  Working tree clean — nothing to commit.');
+              lines.push('  Working tree clean: nothing to commit.');
             } else {
               if (st.staged.length > 0) {
                 lines.push(`  Staged (${st.staged.length}):`);
@@ -65,7 +65,7 @@ export function registerGitRuntimeCommands(registry: CommandRegistry): void {
         }
         case 'diff': {
           // Route the full, uncapped working-tree diff (GitService.diffStructured
-          // — parsed per-file/per-hunk with no size limit) into the real diff
+          //, parsed per-file/per-hunk with no size limit) into the real diff
           // panel. The old path sliced the raw text at 4,000 chars and printed a
           // stub; a large diff now renders complete.
           try {

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

@@ -3,11 +3,11 @@ import { getOperatorRpc, describeOperatorRpcError } from './operator-rpc.ts';
 import { renderWorkstreamGraphLines } from '../../panels/workstream-graph-render.ts';
 
 // ---------------------------------------------------------------------------
-// /graph — a workstream's task graph (fleet.graph.get) rendered legibly.
+// /graph, a workstream's task graph (fleet.graph.get) rendered legibly.
 //
 // The observability-layer surface: nodes, edges, per-node states (ready /
 // running / blocked-with-"waiting on: X" / stalled / orphaned) and the elastic
-// pool posture ("N ready, M running, at cap (fleet.maxSize=N)") — the shape of
+// pool posture ("N ready, M running, at cap (fleet.maxSize=N)"), the shape of
 // the dependency graph WITHOUT opening any transcript. The fix-phase chain's
 // task graph is one such workstream.
 // ---------------------------------------------------------------------------
@@ -20,7 +20,7 @@ function renderWidth(): number {
 export function registerGraphRuntimeCommands(registry: CommandRegistry): void {
   registry.register({
     name: 'graph',
-    description: "Show a workstream's task graph — nodes, edges, states, and pool posture",
+    description: "Show a workstream's task graph: nodes, edges, states, and pool posture",
     usage: '<workstreamId>',
     argsHint: '<workstreamId>',
     async handler(args, ctx) {

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

@@ -8,7 +8,7 @@ export function registerGuidanceRuntimeCommands(registry: CommandRegistry): void
   registry.register({
     name: 'welcome',
     aliases: ['guide'],
-    description: 'Open the product entry surface for the onboarding wizard, security, marketplace, remote, and operator workflows',
+    description: 'Open the product entry point for the onboarding wizard, security, marketplace, remote, and operator workflows',
     usage: '[open|print]',
     handler(args, ctx) {
       const sub = args[0] ?? 'open';

package/src/input/commands/health-metrics-format.ts

@@ -28,12 +28,12 @@ export function renderMetricMap(label: string, map: Record<string, unknown> | un
 
 /**
  * When a telemetry verb fails because the companion token lacks the required
- * scope, say exactly that — never render zeros in place of missing authority.
+ * scope, say exactly that, never render zeros in place of missing authority.
  * Returns the honest line, or null when the error is not a scope refusal.
  */
 export function telemetryScopeRefusalLine(error: unknown, what: string): string | null {
   if (error instanceof GoodVibesSdkError && (error.status === 401 || error.status === 403)) {
-    return `  ${what}: unavailable — the companion token lacks the read:telemetry scope this data requires (daemon returned ${error.status}).`;
+    return `  ${what}: unavailable; the companion token lacks the read:telemetry scope this data requires (daemon returned ${error.status}).`;
   }
   return null;
 }