@pellux/goodvibes-tui 2.0.14 → 2.0.16

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

@@ -42,7 +42,7 @@ export function registerHealthRuntimeCommands(registry: CommandRegistry): void {
     name: 'health',
     aliases: ['doctor'],
     description: 'Health workspace for startup posture, service readiness, sandbox posture, and provider health',
-    usage: '[report|review|open|setup|services|sandbox|provider|accounts|auth|settings|intelligence|remote|mcp|memory|metrics|continuity|worktrees|maintenance|term|repair [domain]] — bare and report stay a cross-domain transcript report (see also /health provider for the providers modal)',
+    usage: '[report|review|open|setup|services|sandbox|provider|accounts|auth|settings|intelligence|remote|mcp|memory|metrics|continuity|worktrees|maintenance|term|repair [domain]]: bare and report stay a cross-domain transcript report (see also /health provider for the providers modal)',
     async handler(args, ctx) {
       const sub = (args[0] ?? 'review').toLowerCase();
       const readModels = requireReadModels(ctx);
@@ -54,7 +54,7 @@ export function registerHealthRuntimeCommands(registry: CommandRegistry): void {
 
       if (sub === 'memory') {
         // The MemoryGovernor snapshot (ops.memory.get): tier, budget vs RSS,
-        // per-cache footprints, paused jobs, tripwire — read-only, honest 501
+        // per-cache footprints, paused jobs, tripwire, read-only, honest 501
         // when a daemon predates the governor verb.
         const resolution = createMemoryDiagnosticsGateway({
           configManager: ctx.platform.configManager,
@@ -354,9 +354,9 @@ export function registerHealthRuntimeCommands(registry: CommandRegistry): void {
       if (sub === 'term') {
         const caps = probeTermCaps(process.stdout as NodeJS.WriteStream);
         const issues: string[] = [];
-        if (caps.capability === 'none') issues.push('terminal reports no color support — UI rendering will be degraded (no ANSI colors)');
-        if (caps.capability === 'basic16') issues.push('terminal limited to 16 ANSI colors — gradient and true-color UI elements will be approximated');
-        if (!caps.syncedOutput) issues.push('DEC Synchronized Output (mode 2026) is disabled — screen-tearing may be visible on slow connections');
+        if (caps.capability === 'none') issues.push('terminal reports no color support; UI rendering will be degraded (no ANSI colors)');
+        if (caps.capability === 'basic16') issues.push('terminal limited to 16 ANSI colors; gradient and true-color UI elements will be approximated');
+        if (!caps.syncedOutput) issues.push('DEC Synchronized Output (mode 2026) is disabled; screen-tearing may be visible on slow connections');
         ctx.print([
           'Health Review: Terminal Capabilities',
           `  color capability: ${caps.capability}`,
@@ -490,13 +490,13 @@ export function registerHealthRuntimeCommands(registry: CommandRegistry): void {
       ctx.print([
         'Health Review',
         // item 3: /health stays a cross-domain transcript report (it
-        // genuinely spans settings/accounts/auth/sandbox/etc — no single
+        // genuinely spans settings/accounts/auth/sandbox/etc, no single
         // modal owns this data), but the providers domain DOES have one, so
         // point at it honestly. "/provider" (no domain) is a DIFFERENT
-        // command (switch/manage custom providers) — the real front door to
+        // command (switch/manage custom providers), the real front door to
         // the providers modal is /health provider (see sub === 'provider'
         // above), so that is what this hint names.
-        '  see also: the providers modal — /health provider',
+        '  see also: the providers modal; /health provider',
         `  session: ${snapshot.sessionId}`,
         `  setup issues: ${snapshot.issues.length}`,
         `  service issues: ${snapshot.serviceIssues.length}`,

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

@@ -1,10 +1,10 @@
 /**
- * hosted-runtime.ts — `/hosted`, the front door to a conversation the DAEMON
+ * hosted-runtime.ts, `/hosted`, the front door to a conversation the DAEMON
  * runs.
  *
  * Local sessions are unchanged and remain the default experience: a turn typed
  * into the composer runs in this process, exactly as it always has. `/hosted`
- * is the opt-in — it asks the daemon to compose the same loop on its side, so
+ * is the opt-in, it asks the daemon to compose the same loop on its side, so
  * the conversation does not depend on this window staying open.
  *
  * ── The subcommands, and why they are these ───────────────────────────────
@@ -18,7 +18,7 @@
  *   /hosted say <text…>        steer the attached session (ordinary verb)
  *   /hosted later <text…>      queue a follow-up (ordinary verb)
  *   /hosted cancel [callId]    cancel one in-flight tool call (ordinary verb)
- *   /hosted detach             leave — the policy the record names then applies
+ *   /hosted detach             leave, the policy the record names then applies
  *   /hosted kill [id]          end it regardless of policy
  *
  * There is no `/hosted steer` beside `say` and no hosted-only cancel: those
@@ -54,7 +54,7 @@ let lastListedIds: readonly string[] = [];
 export function renderHostedRecordLine(record: HostedSessionRecord, index: number): string {
   const attached = record.attachedClients.length > 0 ? record.attachedClients.join(', ') : 'nobody';
   const ended = record.status === 'terminated'
-    ? ` — ended: ${record.terminatedReason ?? 'no reason recorded'}`
+    ? `: ended: ${record.terminatedReason ?? 'no reason recorded'}`
     : '';
   return [
     `  ${index + 1}. ${hostedSessionLabel(record)} [${record.status}]${ended}`,
@@ -84,7 +84,7 @@ export function renderHostedStatus(feed: HostedSessionFeed): string {
     `  ${record.turnCount} turn(s), ${record.messageCount} message(s)`,
     state.streaming
       ? '  live event stream open'
-      : `  no live stream — ${state.streamNote ?? 'not subscribed'}`,
+      : `  no live stream: ${state.streamNote ?? 'not subscribed'}`,
     state.runningToolCalls.length > 0
       ? `  running: ${state.runningToolCalls.map((call) => `${call.tool} (${call.callId})`).join(', ')}`
       : '  no tool call is running',
@@ -248,7 +248,7 @@ export function registerHostedRuntimeCommands(registry: CommandRegistry): void {
             const records = await seams.client.list({ includeTerminated });
             lastListedIds = records.map((record) => record.id);
             // The session picker reads the same roster, so a list typed here is
-            // the list it shows — one answer, not two that drift.
+            // the list it shows, one answer, not two that drift.
             if (!includeTerminated) getSharedHostedSessionRoster().accept(records);
             if (records.length === 0) {
               ctx.print(includeTerminated
@@ -272,14 +272,14 @@ export function registerHostedRuntimeCommands(registry: CommandRegistry): void {
           }
           case 'say':
           case 'later': {
-            if (!attachedId) { ctx.print('[hosted] no hosted session is attached — /hosted attach <id> first.'); return; }
+            if (!attachedId) { ctx.print('[hosted] no hosted session is attached; /hosted attach <id> first.'); return; }
             const body = args.slice(1).join(' ').trim();
             if (!body) { ctx.print(`Usage: /hosted ${sub} <text>`); return; }
             if (sub === 'say') await seams.client.steer(attachedId, body);
             else await seams.client.followUp(attachedId, body);
             ctx.print(sub === 'say'
-              ? '[hosted] sent — its output arrives on the Hosted Session panel.'
-              : '[hosted] queued — it runs after the current turn.');
+              ? '[hosted] sent: its output arrives on the Hosted Session panel.'
+              : '[hosted] queued: it runs after the current turn.');
             return;
           }
           case 'cancel': {
@@ -290,7 +290,7 @@ export function registerHostedRuntimeCommands(registry: CommandRegistry): void {
             const cancelled = await seams.client.cancelToolCall(attachedId, callId);
             ctx.print(cancelled
               ? `[hosted] cancelled tool call ${callId}.`
-              : `[hosted] the daemon did not cancel ${callId} — it may have already settled.`);
+              : `[hosted] the daemon did not cancel ${callId}; it may have already settled.`);
             return;
           }
           case 'detach': {
@@ -298,8 +298,8 @@ export function registerHostedRuntimeCommands(registry: CommandRegistry): void {
             const record = await seams.client.detach(attachedId);
             feed.clear();
             ctx.print(record.status === 'terminated'
-              ? `[hosted] detached — the session ended (${record.terminatedReason ?? 'no reason recorded'}), which is what its detach policy said would happen.`
-              : '[hosted] detached — the session is still running in the daemon and can be reattached.');
+              ? `[hosted] detached: the session ended (${record.terminatedReason ?? 'no reason recorded'}), which is what its detach policy said would happen.`
+              : '[hosted] detached: the session is still running in the daemon and can be reattached.');
             return;
           }
           case 'kill': {
@@ -307,7 +307,7 @@ export function registerHostedRuntimeCommands(registry: CommandRegistry): void {
             if (!target) { ctx.print('Usage: /hosted kill <id> (or attach one first)'); return; }
             const record = await seams.client.kill(target);
             if (target === attachedId) feed.clear();
-            ctx.print(`[hosted] ended ${hostedSessionLabel(record)} — ${record.terminatedReason ?? 'no reason recorded'}.`);
+            ctx.print(`[hosted] ended ${hostedSessionLabel(record)}: ${record.terminatedReason ?? 'no reason recorded'}.`);
             return;
           }
           default:

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

@@ -4,17 +4,17 @@ import { summarizeError } from '@pellux/goodvibes-sdk/platform/utils';
 /**
  * registerImageRuntimeCommands - `/imagine <prompt>`.
  *
- * Named 'imagine', not 'image' — `/image <path> [prompt]` already exists
+ * Named 'imagine', not 'image', `/image <path> [prompt]` already exists
  * (local-runtime.ts) and does something entirely different (attach a local
  * image file to the next message). Re-verified against the current registry
  * before wiring this in: the brief that scoped this command was written
  * before checking for the collision.
  *
- * First production caller of MediaProviderRegistry.generate() — the SDK
+ * First production caller of MediaProviderRegistry.generate(), the SDK
  * plumbing existed with zero non-test call sites before this command.
  * findProvider('generate') returns whichever generation-capable provider is
  * configured; when none are, the command prints the registry's own
- * per-provider status (id/state/detail) verbatim — those detail strings
+ * per-provider status (id/state/detail) verbatim, those detail strings
  * already name the exact env var per builtin provider
  * (builtin-generation-providers.ts's buildStatus()), so this command never
  * has to (and never should) guess or hardcode an env-var name itself.
@@ -48,7 +48,7 @@ export function registerImageRuntimeCommands(registry: CommandRegistry): void {
           lines.push('No media-generation providers are registered in this build.');
         } else {
           for (const status of generationStatuses) {
-            lines.push(`  ${status.id} (${status.label}): ${status.state}${status.detail ? ` — ${status.detail}` : ''}`);
+            lines.push(`  ${status.id} (${status.label}): ${status.state}${status.detail ? `; ${status.detail}` : ''}`);
           }
         }
         ctx.print(lines.join('\n'));
@@ -70,7 +70,7 @@ export function registerImageRuntimeCommands(registry: CommandRegistry): void {
         const lines = [`Generated ${result.artifacts.length} artifact(s) via ${provider.label} (${result.providerId}).`];
         for (const artifact of result.artifacts) {
           if (!artifact.mimeType.startsWith('image/')) {
-            lines.push(`  Note: ${provider.label} produced ${artifact.mimeType}, not an image — no image-capable provider is configured in this build.`);
+            lines.push(`  Note: ${provider.label} produced ${artifact.mimeType}, not an image; no image-capable provider is configured in this build.`);
           }
           if (artifact.dataBase64) {
             const descriptor = await artifactStore.create({
@@ -97,7 +97,7 @@ export function registerImageRuntimeCommands(registry: CommandRegistry): void {
               sourceUri: artifact.uri,
               metadata: { ...artifact.metadata, generatedBy: result.providerId, remote: true },
             });
-            lines.push(`  artifact: ${descriptor.id} (remote reference — not downloaded)`);
+            lines.push(`  artifact: ${descriptor.id} (remote reference; not downloaded)`);
             lines.push(`  url: ${artifact.uri}`);
           } else {
             lines.push('  artifact: (no retrievable content returned)');

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

@@ -56,11 +56,11 @@ export function registerIntelligenceRuntimeCommands(registry: CommandRegistry):
       const sub = (args[0] ?? 'review').toLowerCase();
       if (sub === 'panel' || sub === 'open') {
         // (the purge): 'intelligence'/IntelligencePanel was
-        // DELETE-disposition (no surviving human surface — its read model
+        // DELETE-disposition (no surviving human surface, its read model
         // stays and still backs every subcommand below). There is no alias
         // to resolve through, so this prints an honest notice and opens
         // Fleet instead of throwing "Unknown panel".
-        ctx.print('Intelligence panel retired — use /intelligence review|diagnostics|symbols|outline|definition|references|hover|repair for the live data. Opening Fleet.');
+        ctx.print('Intelligence panel retired: use /intelligence review|diagnostics|symbols|outline|definition|references|hover|repair for the live data. Opening Fleet.');
         try {
           openCommandPanel(ctx, 'fleet');
         } catch {

package/src/input/commands/local-auth-runtime.ts

@@ -12,7 +12,7 @@ export function handleLocalAuthCommand(args: string[], ctx: CommandContext): voi
   if (sub === 'panel' || sub === 'open') {
     // browse view moved to local-auth-modal. The LocalAuthPanel itself is
     // kept (masked password-entry host) but is no longer the /local-auth panel
-    // destination — masked entry is reached via add-user/rotate-password below.
+    // destination, masked entry is reached via add-user/rotate-password below.
     ctx.openModal?.('local-auth-modal');
     return;
   }
@@ -25,7 +25,7 @@ export function handleLocalAuthCommand(args: string[], ctx: CommandContext): voi
       return;
     }
     if (!password) {
-      // No password supplied — open masked-entry mode on the LocalAuthPanel.
+      // No password supplied, open masked-entry mode on the LocalAuthPanel.
       if (ctx.openLocalAuthMaskedEntry) {
         ctx.openLocalAuthMaskedEntry('add-user', username);
       } else {
@@ -68,7 +68,7 @@ export function handleLocalAuthCommand(args: string[], ctx: CommandContext): voi
       return;
     }
     if (!password) {
-      // No password supplied — open masked-entry mode on the LocalAuthPanel.
+      // No password supplied, open masked-entry mode on the LocalAuthPanel.
       if (ctx.openLocalAuthMaskedEntry) {
         ctx.openLocalAuthMaskedEntry('rotate-password', username);
       } else {

package/src/input/commands/local-provider-runtime.ts

@@ -69,7 +69,7 @@ export function registerLocalProviderRuntimeCommands(registry: CommandRegistry):
             }
           }
         } catch {
-          ctx.print(`Could not reach ${baseURL}/models — creating provider with a minimal starter config.`);
+          ctx.print(`Could not reach ${baseURL}/models: creating provider with a minimal starter config.`);
         }
 
         let contextWindows: Record<string, number> = {};
@@ -117,7 +117,7 @@ export function registerLocalProviderRuntimeCommands(registry: CommandRegistry):
         }
         ctx.print(`Provider '${name}' added with ${models.length} model(s):\n${discoveredModelIds.length > 0 ? discoveredModelIds.map((id) => `  • ${id} (${(contextWindows[id] ?? 8192).toLocaleString()} ctx)`).join('\n') : `  • ${defaultModel} (starter entry)`}\nThe file watcher will auto-register it shortly.`);
         // Capture the key with the keystrokes masked, straight into the secrets
-        // manager — never a cleartext argument, never written into the JSON.
+        // manager, never a cleartext argument, never written into the JSON.
         if (ctx.beginConcealedInput) {
           const secretsManager = requireSecretsManager(ctx);
           ctx.print(`Enter the API key for provider '${name}' (input is masked; press Enter to store, Esc to skip if the endpoint needs no key).`);
@@ -139,7 +139,7 @@ export function registerLocalProviderRuntimeCommands(registry: CommandRegistry):
                   // has to reach the daemon tier or a locally-hosted endpoint
                   // is usable only while this TUI is open.
                   await secretsManager.set(keyName, value, { scope: 'daemon', medium: 'secure' });
-                  ctx.print(`Stored the API key for '${name}' (value hidden). It resolves live for this provider — no restart needed.`);
+                  ctx.print(`Stored the API key for '${name}' (value hidden). It resolves live for this provider; no restart needed.`);
                 } catch (error) {
                   ctx.print(`Could not store the key for '${name}': ${summarizeError(error)}`);
                 }

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

@@ -37,7 +37,7 @@ function toggleBlocks(typeFilter: string, collapsed: boolean, ctx: CommandContex
     // A merged assistant turn (type 'assistant_turn', see
     // conversation-turn-structure.ts) owns the whole tool subtree beneath its
     // header, so '/expand tool'/'collapse tool' treats it the same as a plain
-    // 'tool' block — toggling the header's collapse key shows or hides every
+    // 'tool' block, toggling the header's collapse key shows or hides every
     // tool row underneath it.
     const matchesType = typeFilter === 'all'
       || (typeFilter === 'tool' && (block.type === 'tool' || block.type === 'assistant_turn'))
@@ -50,7 +50,7 @@ function toggleBlocks(typeFilter: string, collapsed: boolean, ctx: CommandContex
       // Expanding a turn also expands each result's own collapse key in the
       // same pass. A result hidden by a collapsed turn pushes no BlockMeta of
       // its own, so it never surfaces from this loop to be toggled
-      // individually — without this, '/expand tool' would only open the turn
+      // individually, without this, '/expand tool' would only open the turn
       // header and each result would still render at its own default collapse
       // state, needing a second pass. '/collapse tool' needs no matching step:
       // collapsing the turn hides every result regardless of its own key.
@@ -59,7 +59,7 @@ function toggleBlocks(typeFilter: string, collapsed: boolean, ctx: CommandContex
           const memberKey = `msg_${memberIdx}`;
           ctx.session.conversationManager.setCollapsed(memberKey, false);
           // An explicit /expand is a deliberate user action on this key, same
-          // as Tab/Ctrl+Y/Ctrl+B — exempts it from search's close-time
+          // as Tab/Ctrl+Y/Ctrl+B, exempts it from search's close-time
           // auto-re-collapse (see ConversationManager.noteUserTouch).
           ctx.session.conversationManager.searchExpansion.noteUserTouch(memberKey);
         }
@@ -88,17 +88,17 @@ export function registerLocalRuntimeCommands(registry: CommandRegistry): void {
   registry.register({
     name: 'tools',
     aliases: ['t'],
-    description: 'List available tools and review compact native tool capability surfaces',
+    description: 'List available tools and review compact native tool capabilities',
     usage: '[review|panel]',
     handler(args, ctx) {
       const sub = (args[0] ?? '').toLowerCase();
       if (sub === 'panel' || sub === 'review') {
         // (the purge): 'tools'/ToolInspectorPanel was DELETE-disposition
-        // (no surviving human surface — tool results render inline in the
+        // (no surviving human surface, tool results render inline in the
         // transcript, plus a per-node tool list in Fleet). There is no alias
         // to resolve through, so this prints an honest notice and opens
         // Fleet instead of the retired inspector.
-        ctx.print('Tools panel retired — tool activity now renders inline in the transcript and per-agent in the Fleet panel. Opening Fleet.');
+        ctx.print('Tools panel retired: tool activity now renders inline in the transcript and per-agent in the Fleet panel. Opening Fleet.');
         try {
           openCommandPanel(ctx, 'fleet');
         } catch {
@@ -407,7 +407,7 @@ export function registerLocalRuntimeCommands(registry: CommandRegistry): void {
       } catch (e) {
         ctx.print(`Live model discovery failed: ${summarizeError(e)}`);
       }
-      if (!catalogOk || !benchmarksOk || !limitsOk || !discoveryOk) ctx.print('Some refreshes failed — see messages above.');
+      if (!catalogOk || !benchmarksOk || !limitsOk || !discoveryOk) ctx.print('Some refreshes failed: see messages above.');
     },
   });
 

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

@@ -1,12 +1,12 @@
 /**
- * mail-runtime.ts — `/mail`, the terminal surface over the daemon's mail
+ * mail-runtime.ts, `/mail`, the terminal surface over the daemon's mail
  * capability.
  *
  * This command implements no mail. Every verb here is one call into the
  * gateway method catalog, where `src/daemon/handlers/email/` owns IMAP, SMTP,
  * settings resolution, the credential read, and the confirmation gate. What
  * lives in this file is argument parsing, rendering, and an honest refusal when
- * the capability is not set up — nothing else.
+ * the capability is not set up, nothing else.
  *
  * ## What is exposed, and what is not
  *
@@ -14,8 +14,8 @@
  * without leaving the terminal is squarely what this product is for, and
  * because a draft changes nothing outside the account's own Drafts folder.
  *
- * Sending is exposed too — a shipped route with no way to reach it is a route
- * that is not wired up — but never in one step. `/mail send` first renders
+ * Sending is exposed too, a shipped route with no way to reach it is a route
+ * that is not wired up, but never in one step. `/mail send` first renders
  * exactly what would leave the machine and stops; only a re-run carrying
  * `--confirm` sets both flags the daemon's gate demands. That mirrors the
  * preview-then-confirm shape `/review` uses for hunk reverts, and it means the
@@ -65,12 +65,12 @@ function truncate(value: string, width: number): string {
 
 export function renderInboxList(messages: readonly InboxMessage[]): string {
   if (messages.length === 0) return 'Inbox is empty (nothing matched).';
-  const lines = [`Inbox — ${messages.length} message${messages.length === 1 ? '' : 's'}:`];
+  const lines = [`Inbox: ${messages.length} message${messages.length === 1 ? '' : 's'}:`];
   for (const message of messages) {
     const mark = message.unread ? '●' : ' ';
     lines.push(`  ${mark} ${String(message.uid).padStart(6)}  ${truncate(message.from, 28).padEnd(28)}  ${truncate(message.subject, 48)}`);
   }
-  lines.push('', '  /mail read <uid> — open one message');
+  lines.push('', '  /mail read <uid>: open one message');
   return lines.join('\n');
 }
 
@@ -191,7 +191,7 @@ async function listInbox(ctx: CommandContext, gateway: Gateway, limit: number, w
       ctx.print(renderConnectionStatus({
         surface: 'mail',
         state: 'ready',
-        detail: 'Mail is connected — the daemon reached the server and returned a result.',
+        detail: 'Mail is connected: the daemon reached the server and returned a result.',
         nextActions: [],
       }));
     }
@@ -202,7 +202,7 @@ async function listInbox(ctx: CommandContext, gateway: Gateway, limit: number, w
 async function readMessage(ctx: CommandContext, gateway: Gateway, rawUid: string | undefined): Promise<void> {
   const uid = Number(rawUid);
   if (!Number.isFinite(uid) || uid <= 0) {
-    ctx.print('Usage: /mail read <uid> — the uid column from /mail list.');
+    ctx.print('Usage: /mail read <uid>; the uid column from /mail list.');
     return;
   }
   await withStatusOnFailure(ctx, gateway, 'read', async () => {
@@ -221,7 +221,7 @@ async function compose(ctx: CommandContext, gateway: Gateway, verb: 'draft' | 's
   if (parsed === null) {
     ctx.print([
       `Usage: /mail ${verb} <to | subject | body>`,
-      '  Three fields separated by | — for example:',
+      '  Three fields separated by |, for example:',
       `  /mail ${verb} me@example.com | Build is green | All gates passed.`,
     ].join('\n'));
     return;

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

@@ -184,7 +184,7 @@ export function registerMcpRuntimeCommands(registry: CommandRegistry): void {
             lines.push(`\n  [${tool.serverName}]`);
             lastServer = tool.serverName;
           }
-          lines.push(`    ${tool.toolName}${tool.description ? `  — ${tool.description}` : ''}`);
+          lines.push(`    ${tool.toolName}${tool.description ? `: ${tool.description}` : ''}`);
         }
         ctx.print(lines.join('\n'));
         return;

package/src/input/commands/memory.ts

@@ -3,14 +3,14 @@
  *
  * Implements the Project Memory Substrate commands:
  *
- *   /recall add <class> <summary>           — Add a new memory record
+ *   /recall add <class> <summary>          , Add a new memory record
  *   /recall add <class> <summary> --detail <text> --tags <tag,tag>
- *   /recall search [query]                  — Search memory records
- *   /recall search --cls <class>            — Filter by class
- *   /recall link <fromId> <toId> <relation> — Link two records
- *   /recall get <id>                        — Show a single record with provenance
- *   /recall list [class]                    — List all records (optionally by class)
- *   /recall remove <id>                     — Delete a record
+ *   /recall search [query]                 , Search memory records
+ *   /recall search --cls <class>           , Filter by class
+ *   /recall link <fromId> <toId> <relation>, Link two records
+ *   /recall get <id>                       , Show a single record with provenance
+ *   /recall list [class]                   , List all records (optionally by class)
+ *   /recall remove <id>                    , Delete a record
  */
 
 import type { SlashCommand, CommandContext } from '../command-registry.ts';
@@ -27,17 +27,17 @@ import { handleRecallFilesApply, handleRecallFilesReview, handleRecallFilesSync
  * item 3 divergence note (historical): the work order that shipped
  * this command named its front door "/memory", but at the time `/memory` was
  * already a distinct, unrelated command (session-pinned sticky notes,
- * src/input/commands/session-content.ts) with no modal surface — so that
+ * src/input/commands/session-content.ts) with no modal surface, so that
  * work order deliberately did NOT touch /memory and used /recall instead.
  *
- * Update from the core-verb naming pass (MEMORY fragmentation — worst-class
+ * Update from the core-verb naming pass (MEMORY fragmentation, worst-class
  * collision #2): the agent's own `/memory` command was a plain alias for its
  * `/recall`-equivalent the whole time, meaning "/memory" meant two unrelated
  * things depending which surface you were on. The session-notes command was
  * renamed to `/note` (session-content.ts) to free the word, and `/memory` is
- * now registered here as a real alias of `/recall` — the word means the same
+ * now registered here as a real alias of `/recall`, the word means the same
  * durable Project Memory Substrate on both surfaces. The modal that exists
- * for this data — the Project Memory Substrate — is still `memory-modal.ts`,
+ * for this data, the Project Memory Substrate, is still `memory-modal.ts`,
  * owned by THIS command, confirmed by the panel-id redirect
  * `registerModalRedirect('memory', 'memory-modal')` in builtin-modals.ts.
  */
@@ -79,12 +79,12 @@ export const recallCommand: SlashCommand = {
   name: 'recall',
   aliases: ['rc', 'memory', 'mem'],
   description: 'Bare opens the Memory modal; project memory subcommands add decisions, constraints, incidents, and patterns with provenance',
-  usage: '[<subcommand> [args]] — bare opens the modal; report prints the subcommand usage text',
+  usage: '[<subcommand> [args]]: bare opens the modal; report prints the subcommand usage text',
   argsHint: 'add|search|link|get|list|remove|report',
   handler: async (args: string[], context: CommandContext): Promise<void> => {
     const [sub, ...rest] = args;
 
-    // item 3: bare `/recall` opens the memory-modal surface — the old
+    // item 3: bare `/recall` opens the memory-modal surface, the old
     // bare/unknown-subcommand usage block moved to an explicit `report`
     // subcommand (scriptability preserved: /recall report).
     if (sub === undefined) {

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

@@ -73,7 +73,7 @@ export function registerNotifyRuntimeCommands(registry: CommandRegistry): void {
         ctx.print(`Testing ${urls.length} webhook${urls.length !== 1 ? 's' : ''}...`);
         notifier.setUrls(urls);
         const results = await notifier.test();
-        ctx.print(results.map((r) => r.ok ? `  [ok] ${r.url}` : `  [fail] ${r.url} — ${r.error ?? 'unknown error'}`).join('\n'));
+        ctx.print(results.map((r) => r.ok ? `  [ok] ${r.url}` : `  [fail] ${r.url}: ${r.error ?? 'unknown error'}`).join('\n'));
         return;
       }
 

package/src/input/commands/operator-panel-runtime.ts

@@ -8,7 +8,7 @@ import { summarizeError } from '@pellux/goodvibes-sdk/platform/utils';
  * [top|bottom]` (item 4). Splits the args array in place (removing the
  * flag + its value) so positional `pane` parsing downstream is unaffected by
  * where the flag appears. Only `open fleet` currently has a deep-link
- * consumer (FleetPanel.receiveDeepLink) — other panel ids just ignore an
+ * consumer (FleetPanel.receiveDeepLink), other panel ids just ignore an
  * unused target via PanelManager's optional-chaining delivery.
  */
 function extractTargetFlag(rest: string[]): PanelDeepLinkTarget | undefined {
@@ -33,7 +33,7 @@ export function registerOperatorPanelCommand(registry: CommandRegistry): void {
       const sub = args[0]?.toLowerCase() ?? '';
       if (!sub || sub === 'toggle') {
         // (the purge): 'panel-list' (the browse-all-panels picker) was
-        // DELETE-disposition — a picker over a handful of panels is dead
+        // DELETE-disposition, a picker over a handful of panels is dead
         // weight now (see the Ctrl+P selectionModal repoint in
         // shell/ui-openers.ts for its interactive replacement). Bare
         // `/panel` and `/panel toggle` now do what "toggle" actually means:
@@ -43,7 +43,7 @@ export function registerOperatorPanelCommand(registry: CommandRegistry): void {
         pm.toggle();
         ctx.renderRequest();
       } else if (sub === 'list') {
-        // Print a text listing instead of opening a picker panel — grouped
+        // Print a text listing instead of opening a picker panel, grouped
         // by category from the live registry, so this can never drift from
         // what's actually registered.
         const byCategory = pm.getTypesByCategory();
@@ -51,7 +51,7 @@ export function registerOperatorPanelCommand(registry: CommandRegistry): void {
         for (const [category, entries] of byCategory) {
           lines.push(`${category}:`);
           for (const entry of entries) {
-            lines.push(`  ${entry.icon} ${entry.id} — ${entry.name}`);
+            lines.push(`  ${entry.icon} ${entry.id}: ${entry.name}`);
           }
         }
         ctx.print(lines.length > 0 ? lines.join('\n') : 'No panels registered.');
@@ -72,7 +72,7 @@ export function registerOperatorPanelCommand(registry: CommandRegistry): void {
           // claiming "Panel opened: <id>".
           const redirectTarget = pm.getModalRedirect(id);
           // focus rule 1a: the command path leaves focus in the composer
-          // ("mid-command-flow") — showPanel does not grab panel focus here.
+          // ("mid-command-flow"), showPanel does not grab panel focus here.
           // forward the deep-link target so the panel lands on the row.
           if (ctx.showPanel) ctx.showPanel(id, pane as 'top' | 'bottom' | undefined, target);
           else {
@@ -81,13 +81,13 @@ export function registerOperatorPanelCommand(registry: CommandRegistry): void {
             ctx.renderRequest();
           }
           if (redirectTarget) {
-            ctx.print(`"${id}" moved to the ${redirectTarget} modal — opening it.`);
+            ctx.print(`"${id}" moved to the ${redirectTarget} modal; opening it.`);
           } else {
             ctx.print(`Panel opened: ${id}${pane ? ` (${pane} pane)` : ''}`);
           }
         } catch (e) {
           // A deleted/unknown panel id throws "No panel type registered with
-          // id: <id>" from PanelManager.open — surface the same friendly line
+          // id: <id>" from PanelManager.open, surface the same friendly line
           // the bare-/panel path uses instead of leaking the raw error. Any
           // genuinely unexpected failure still shows its real message.
           const message = summarizeError(e);
@@ -170,7 +170,7 @@ export function registerOperatorPanelCommand(registry: CommandRegistry): void {
         const id = args[0]!;
         try {
           // bare `/panel <id>` is the same command path as `/panel open
-          // <id>` — composer stays focused.
+          // <id>`, composer stays focused.
           if (ctx.showPanel) ctx.showPanel(id);
           else {
             pm.open(id);

package/src/input/commands/operator-rpc.ts

@@ -1,10 +1,10 @@
 /**
- * operator-rpc.ts — the command layer's handle on the daemon.
+ * operator-rpc.ts, the command layer's handle on the daemon.
  *
  * The resolution itself (is the daemon enabled, what base URL, what bearer)
  * moved to runtime/client/operator-endpoint.ts when the terminal app became a
- * pure client: the runtime seams — approvals, config, credentials, sessions,
- * fleet, tasks, devices, checkpoints — all need the same answer, and two copies
+ * pure client: the runtime seams, approvals, config, credentials, sessions,
+ * fleet, tasks, devices, checkpoints, all need the same answer, and two copies
  * of it is how one call ended up on a DirectTransport while another made a real
  * request. This module keeps the command-context-shaped entry point and
  * re-exports the rest so no command import site had to change.

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

@@ -66,9 +66,9 @@ export function registerOperatorRuntimeCommands(registry: CommandRegistry): void
   registry.register({
     name: 'profiles',
     // The singular `profile` alias was given up to /profile, the owner profile
-    // (docs/owner-profile.md). These are two unrelated things — this command
+    // (docs/owner-profile.md). These are two unrelated things, this command
     // loads saved display/provider settings presets, and the other one is what
-    // the platform knows about the person who owns it — and the singular word
+    // the platform knows about the person who owns it, and the singular word
     // now belongs to the second. `/profiles` is unchanged; only the abbreviation
     // moved. See the naming note in docs/owner-profile.md §3.
     description: 'Browse and load config profiles',
@@ -254,7 +254,7 @@ export function registerOperatorRuntimeCommands(registry: CommandRegistry): void
 
   registry.register({
     name: 'tool',
-    description: 'Tool contract verification — verify registered tool contracts',
+    description: 'Verify registered tool contracts',
     usage: 'verify <name> | verify-all | contract show <name>',
     argsHint: 'verify <name> | verify-all | contract show <name>',
     handler(args, ctx) {

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

@@ -1,7 +1,7 @@
 import type { CommandRegistry } from '../command-registry.ts';
 
 /**
- * /palette — open the fuzzy command palette, a searchable picker over every
+ * /palette, open the fuzzy command palette, a searchable picker over every
  * registered slash command. The palette itself is generated live from the
  * registry (see openCommandPalette in shell/ui-openers.ts); this command is
  * just the discoverable front door, mirroring the Ctrl+K chord.