@pellux/goodvibes-tui 2.0.14 → 2.0.15

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

@@ -1,11 +1,11 @@
 // ---------------------------------------------------------------------------
-// rewind-runtime.ts — /rewind (message-anchored, unified files + conversation),
+// rewind-runtime.ts, /rewind (message-anchored, unified files + conversation),
 // plus the /undo rewind and /redo rewind reversals over the last receipt.
 //
 // The rewind engine is the SDK's UnifiedRewindService (platform/rewind:
 // plan() previews + issues a single-use confirm token; apply() is token-gated
 // and returns a receipt carrying an undo block). It JOINS the platform's three
-// existing history stores — it never adds a fourth. This module wires that
+// existing history stores, it never adds a fourth. This module wires that
 // service to the TUI's in-process stores through the two ports the service
 // defines:
 //   • RewindWorkspacePort  ← the in-process WorkspaceCheckpointManager (the
@@ -51,7 +51,7 @@ export type RewindScope = SdkRewindScope;
 const REWIND_SCOPES: readonly RewindScope[] = ['files', 'conversation', 'both'];
 
 // ---------------------------------------------------------------------------
-// Ports — bridge the SDK rewind service onto the TUI's in-process stores.
+// Ports, bridge the SDK rewind service onto the TUI's in-process stores.
 // ---------------------------------------------------------------------------
 
 /**
@@ -75,7 +75,7 @@ function makeWorkspacePort(mgr: WorkspaceCheckpointManager): RewindWorkspacePort
 }
 
 // ---------------------------------------------------------------------------
-// Per-session rewind state — the SDK service (with its wired ports) plus the
+// Per-session rewind state, the SDK service (with its wired ports) plus the
 // TUI-owned undo/redo stacks of applied receipts.
 // ---------------------------------------------------------------------------
 
@@ -120,7 +120,7 @@ function undoAvailable(receipt: RewindReceipt): boolean {
 }
 
 // ---------------------------------------------------------------------------
-// /undo rewind & /redo rewind — reversal over the last receipt's undo points.
+// /undo rewind & /redo rewind, reversal over the last receipt's undo points.
 // Called from session-content.ts so the /undo and /redo commands stay single.
 // ---------------------------------------------------------------------------
 
@@ -201,7 +201,7 @@ function renderTurnList(sessionId: string): string {
   if (turns.length === 0) {
     return 'No completed turns recorded this run yet. /rewind targets turns completed in the current session run.';
   }
-  const lines = ['Recent turns (newest first) — /rewind <n> [files|conversation|both]:'];
+  const lines = ['Recent turns (newest first): /rewind <n> [files|conversation|both]:'];
   turns.slice(0, 20).forEach((t, i) => {
     lines.push(`  ${String(i + 1).padStart(2)}. ${t.label}  (${formatAge(t.at)}, ${t.messageCount} msgs)`);
   });
@@ -233,12 +233,12 @@ function parseScope(token: string | undefined): RewindScope | null {
 }
 
 // ---------------------------------------------------------------------------
-// Checkpoint-only fallback — no completed turns recorded THIS run (e.g. right
+// Checkpoint-only fallback, no completed turns recorded THIS run (e.g. right
 // after a restart) means the message-anchored picker above has nothing to
 // list, even though real workspace checkpoints exist on disk. Rather than
 // dead-ending on "no completed turns" while /checkpoints and the boot notice
 // both point back here, /rewind falls back to listing and restoring
-// checkpoints directly — a FILES-ONLY restore (there is no turn anchor to
+// checkpoints directly, a FILES-ONLY restore (there is no turn anchor to
 // join a conversation boundary against), reusing the same diff-panel
 // preview/confirm idiom as the turn-anchored path above.
 // ---------------------------------------------------------------------------
@@ -247,9 +247,9 @@ const CHECKPOINT_FALLBACK_NO_ANCHORS_MESSAGE = 'No completed turns recorded this
 
 function renderCheckpointFallbackList(checkpoints: readonly WorkspaceCheckpoint[]): string {
   const lines = [
-    'No completed turns recorded this run yet — falling back to workspace checkpoints (FILES ONLY, no conversation state):',
+    'No completed turns recorded this run yet; falling back to workspace checkpoints (FILES ONLY, no conversation state):',
     ...checkpoints.slice(0, 20).map((cp, i) => `  ${String(i + 1).padStart(2)}. ${shortId(cp.id).padEnd(12)} ${cp.kind.padEnd(9)} ${cp.label}  (${formatAge(cp.createdAt)})`),
-    'Restore one: /rewind <n|id> — files only; conversation history is untouched.',
+    'Restore one: /rewind <n|id>; files only; conversation history is untouched.',
   ];
   return lines.join('\n');
 }
@@ -272,7 +272,7 @@ function resolveCheckpointRef(checkpoints: readonly WorkspaceCheckpoint[], ref:
 /**
  * Handles bare `/rewind` and `/rewind <ref>` when this run has recorded no
  * turn anchors at all. Returns true if it handled the invocation (always,
- * once called), false is never returned — kept boolean-shaped only to mirror
+ * once called), false is never returned, kept boolean-shaped only to mirror
  * the calling convention at the call site.
  */
 async function handleCheckpointOnlyRewind(args: string[], ctx: CommandContext): Promise<void> {
@@ -325,13 +325,13 @@ async function handleCheckpointOnlyRewind(args: string[], ctx: CommandContext):
 
   diffPanel.confirmOverlay.arm({
     id: checkpoint.id,
-    label: `${checkpoint.label} — restore FILES ONLY (no conversation state this run)`,
+    label: `${checkpoint.label}: restore FILES ONLY (no conversation state this run)`,
     verb: 'Restore',
     onConfirm: async () => {
       try {
         await mgr.restore(checkpoint.id, { safetyCheckpoint: true });
         ctx.session.conversationManager.addTypedSystemMessage(
-          `[Rewind] Restored files from checkpoint ${shortId(checkpoint.id)} ("${checkpoint.label}"). Files only — no turn anchors exist this run, so conversation state is unchanged.`,
+          `[Rewind] Restored files from checkpoint ${shortId(checkpoint.id)} ("${checkpoint.label}"). Files only: no turn anchors exist this run, so conversation state is unchanged.`,
           'operational',
         );
         pm.close('diff');
@@ -342,14 +342,14 @@ async function handleCheckpointOnlyRewind(args: string[], ctx: CommandContext):
       }
     },
     onCancel: () => {
-      ctx.print('Restore cancelled — nothing changed.');
+      ctx.print('Restore cancelled: nothing changed.');
       pm.close('diff');
       ctx.focusPrompt?.();
       ctx.renderRequest();
     },
   });
 
-  ctx.print(`Previewing checkpoint restore: "${checkpoint.label}" (FILES ONLY — no conversation state this run). Confirm in the diff panel: Enter/y to restore, n/Esc to cancel.`);
+  ctx.print(`Previewing checkpoint restore: "${checkpoint.label}" (FILES ONLY; no conversation state this run). Confirm in the diff panel: Enter/y to restore, n/Esc to cancel.`);
   ctx.renderRequest();
 }
 
@@ -367,7 +367,7 @@ export function registerRewindRuntimeCommands(registry: CommandRegistry): void {
       const sessionId = ctx.session.runtime.sessionId;
 
       // No turn anchors recorded this run (most commonly: right after a
-      // restart, before /session resume has restored any) — fall back to a
+      // restart, before /session resume has restored any), fall back to a
       // files-only restore over the real workspace checkpoints instead of a
       // dead end (see handleCheckpointOnlyRewind's header doc).
       if (recentTurnsNewestFirst(sessionId).length === 0) {
@@ -393,7 +393,7 @@ export function registerRewindRuntimeCommands(registry: CommandRegistry): void {
       const anchor: RewindAnchor = { sessionId, turnId: resolved.turnId };
 
       if (wants(scope, 'files') && !ctx.workspace.workspaceCheckpointManager) {
-        ctx.print('Checkpoints are not available in this session — files rewind cannot run. Try /rewind <n> conversation.');
+        ctx.print('Checkpoints are not available in this session; files rewind cannot run. Try /rewind <n> conversation.');
         return;
       }
 
@@ -413,7 +413,7 @@ export function registerRewindRuntimeCommands(registry: CommandRegistry): void {
       }
 
       // Preview in the DiffPanel + arm its confirm overlay (same idiom the
-      // former checkpoint-id /rewind used — see checkpoint-runtime.ts).
+      // former checkpoint-id /rewind used, see checkpoint-runtime.ts).
       const { DiffPanel } = await import('../../panels/diff-panel.ts');
       const pm = requirePanelManager(ctx);
       let panel = pm.getAllOpen().find((p) => p.id === 'diff');
@@ -438,7 +438,7 @@ export function registerRewindRuntimeCommands(registry: CommandRegistry): void {
         const drop = plan.conversation?.messagesToDrop ?? 0;
         const remaining = plan.conversation?.messagesRemaining ?? 0;
         diffPanel.showDiff(
-          `Conversation rewind — drop ${drop} message(s)`,
+          `Conversation rewind: drop ${drop} message(s)`,
           `@@ rewind @@\n Keep ${remaining} message(s), drop ${drop} after this turn. No files change.`,
         );
       }
@@ -449,7 +449,7 @@ export function registerRewindRuntimeCommands(registry: CommandRegistry): void {
 
       diffPanel.confirmOverlay.arm({
         id: anchor.turnId ?? sessionId,
-        label: `${resolved.label} — rewind ${scope} (${summaryParts.join(', ')})`,
+        label: `${resolved.label}: rewind ${scope} (${summaryParts.join(', ')})`,
         verb: 'Rewind',
         onConfirm: async () => {
           try {
@@ -478,7 +478,7 @@ export function registerRewindRuntimeCommands(registry: CommandRegistry): void {
           }
         },
         onCancel: () => {
-          ctx.print('Rewind cancelled — nothing changed.');
+          ctx.print('Rewind cancelled: nothing changed.');
           pm.close('diff');
           ctx.focusPrompt?.();
           ctx.renderRequest();

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

@@ -117,7 +117,7 @@ export function openCommandPanel(
  * (group-B migration): open a config-modal surface by name via the
  * ctx.openModal seam (ui-openers wires it; the host replaces the interim
  * "not available yet" implementation with real dispatch). Front-doors for
- * panels that migrated to modals call this instead of openCommandPanel — the
+ * panels that migrated to modals call this instead of openCommandPanel, the
  * modal is the surface's new home. Stubbed in tests by setting ctx.openModal.
  */
 export function openModalCommand(
@@ -336,7 +336,6 @@ export async function compactConversation(context: CommandContext): Promise<Comp
     compactionCtx,
   );
   const eventAfter = getLastCompactionEvent();
-  // Return the new event only if it differs from the one recorded before the call.
   if (eventAfter !== null && eventAfter !== eventBefore) {
     //: score this run out-of-band (see compaction-quality.ts for why
     // this can't just subscribe to the SDK's own CompactionManager pipeline)

package/src/input/commands/schedule-nl.ts

@@ -2,8 +2,8 @@
  * Local natural-language schedule parsing for `/schedule add when "<phrase>"`.
  *
  * Parses common English scheduling phrases entirely offline into one of the
- * three concrete schedule shapes the automation manager already understands —
- * `cron`, `every` (fixed interval), or `at` (one-shot timestamp) — and returns
+ * three concrete schedule shapes the automation manager already understands,
+ * `cron`, `every` (fixed interval), or `at` (one-shot timestamp), and returns
  * a plain-English description of exactly what it decided, so the caller can
  * ALWAYS echo the concrete interpretation back to the user before anything is
  * saved.

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

@@ -58,7 +58,7 @@ function parseAutomationTarget(raw: string | undefined): AutomationSessionTarget
  * the SDK's severity ladder, narrowed to the current model's own resolved
  * levels whenever the runtime has published them. A scheduled job may well run
  * on a different model than the one in use now, so a level this model does not
- * offer is still accepted when it is a real level — it is re-resolved against
+ * offer is still accepted when it is a real level, it is re-resolved against
  * whichever model actually serves the run.
  */
 function parseReasoningEffort(raw: string | undefined): AutomationExecutionPolicy['reasoningEffort'] | undefined {
@@ -123,7 +123,7 @@ export function registerScheduleRuntimeCommands(registry: CommandRegistry): void
           // The SDK owns the presence + title of the how-to-create-your-first-
           // routine empty state (shown while automation is on and idle). Its
           // body points at a command name this registry doesn't have, so we
-          // render its title with THIS surface's real /schedule commands — a
+          // render its title with THIS surface's real /schedule commands, a
           // command pointer that actually exists in the registry.
           const emptyState = buildAutomationEmptyState({ enabled: true, routineCount: jobs.length });
           ctx.print([

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

@@ -1,5 +1,5 @@
 /**
- * /secret — capture a password-like value through the composer's concealed
+ * /secret, capture a password-like value through the composer's concealed
  * input mode, without the plaintext ever appearing in the transcript or input
  * history.
  *
@@ -27,7 +27,7 @@ export function registerSecretRuntimeCommands(registry: CommandRegistry): void {
     handler(args, ctx) {
       const name = toEnvVarName(args[0] ?? '');
       if (!name) {
-        ctx.print('[Secret] Usage: /secret <NAME> — the value is typed on the next line with the keystrokes masked.');
+        ctx.print('[Secret] Usage: /secret <NAME>; the value is typed on the next line with the keystrokes masked.');
         return;
       }
       if (!ctx.beginConcealedInput) {
@@ -43,7 +43,7 @@ export function registerSecretRuntimeCommands(registry: CommandRegistry): void {
             return;
           }
           process.env[name] = value;
-          // Never echo the value — only a redacted confirmation of length.
+          // Never echo the value, only a redacted confirmation of length.
           ctx.print(`[Secret] Stored ${name} (${value.length} character${value.length === 1 ? '' : 's'}, value hidden). Available to shell/tool runs as $${name}.`);
         },
         onCancel: () => {

package/src/input/commands/session-content.ts

@@ -169,7 +169,7 @@ export function registerSessionContentCommands(registry: CommandRegistry): void
   registry.register({
     name: 'undo',
     aliases: [],
-    description: 'Undo last action. /undo rewind — reverse the last /rewind. /undo file — revert last file write/edit. /undo — remove last conversation turn',
+    description: 'Undo last action. /undo rewind reverses the last /rewind. /undo file reverts the last file write/edit. /undo alone removes the last conversation turn',
     usage: '[rewind|file]',
     argsHint: '[rewind|file]',
     async handler(args, ctx) {
@@ -203,7 +203,7 @@ export function registerSessionContentCommands(registry: CommandRegistry): void
 
   registry.register({
     name: 'redo',
-    description: 'Redo last undone action. /redo rewind — re-apply the last undone /rewind. /redo file — re-apply last reverted file. /redo — restore conversation turn',
+    description: 'Redo last undone action. /redo rewind re-applies the last undone /rewind. /redo file re-applies the last reverted file. /redo alone restores a conversation turn',
     usage: '[rewind|file]',
     argsHint: '[rewind|file]',
     async handler(args, ctx) {
@@ -259,7 +259,7 @@ export function registerSessionContentCommands(registry: CommandRegistry): void
     argsHint: '[resume <id|name>]',
     async handler(_args, ctx) {
       // Splash's "resume last session" hint and users' muscle memory both say
-      // `/sessions resume <id>` (plural, matching this command's own name) —
+      // `/sessions resume <id>` (plural, matching this command's own name),
       // but subcommands like `resume` are only implemented on the singular
       // `/session` command. Forward instead of silently listing and dropping
       // the subcommand+id on the floor.
@@ -268,7 +268,7 @@ export function registerSessionContentCommands(registry: CommandRegistry): void
         return;
       }
       const sessionManager = requireSessionManager(ctx);
-      // Excludes subagent-shaped transcripts (see session-picker-filter.ts) —
+      // Excludes subagent-shaped transcripts (see session-picker-filter.ts),
       // an unfiltered list here drowns real user sessions in spawned-agent
       // noise. A direct `/sessions resume agent-xxxxxxxx` still reaches them.
       const sessions = filterUserFacingSessions(sessionManager.list());
@@ -403,12 +403,12 @@ export function registerSessionContentCommands(registry: CommandRegistry): void
     },
   });
 
-  // From the core-verb naming pass (MEMORY fragmentation — worst-class
+  // From the core-verb naming pass (MEMORY fragmentation, worst-class
   // collision #2): this command used to be named /memory, colliding with
   // the agent's /memory (an alias that forwards to /recall, the durable
-  // cross-session MemoryStore) — same command name, two unrelated features.
+  // cross-session MemoryStore), same command name, two unrelated features.
   // Renamed to /note (this feature manages EPHEMERAL, session-scoped sticky
-  // notes pinned across context compaction — an entirely different resource
+  // notes pinned across context compaction, an entirely different resource
   // from /recall's durable records). /memory is now registered as an alias
   // of /recall below (memory.ts), matching the agent, so the word means the
   // same thing on both surfaces. See

package/src/input/commands/session-picker-filter.ts

@@ -1,18 +1,18 @@
 /**
- * session-picker-filter.ts — keep subagent transcripts out of user-facing
+ * session-picker-filter.ts, keep subagent transcripts out of user-facing
  * session pickers.
  *
  * Subagents spawned during a session get their own SessionManager-persisted
  * transcript, named `agent-<id>` (see the SDK's agent manager, which mints
  * ids as `agent-${uuid.slice(0, 8)}`). Those files are real, loadable
- * sessions — useful for forensic/debugging access — but they are not
+ * sessions, useful for forensic/debugging access, but they are not
  * sessions a user ever started, and in a workspace with many spawned agents
  * they drown the handful of sessions the user actually wants to pick from
  * in `/resume` and `/sessions`.
  *
  * Both pickers filter agent-shaped entries out of the LIST/BROWSE view only.
  * Direct resume by exact id/name (`/resume agent-xxxxxxxx`, `/session resume
- * agent-xxxxxxxx`) is untouched and still reaches them — this only affects
+ * agent-xxxxxxxx`) is untouched and still reaches them, this only affects
  * what shows up when the id isn't already known.
  */
 import type { SessionInfo } from '@pellux/goodvibes-sdk/platform/sessions';

package/src/input/commands/session-workflow.ts

@@ -75,7 +75,7 @@ function buildTranscriptReviewLines(
   return [
     `Transcript Events${kind === 'all' ? '' : `: ${kind}`}`,
     `  events: ${events.length}`,
-    ...events.slice(0, 16).map((event) => `  #${String(event.messageIndex).padStart(3)}  ${event.kind.padEnd(20)} ${event.title} — ${event.detail}`),
+    ...events.slice(0, 16).map((event) => `  #${String(event.messageIndex).padStart(3)}  ${event.kind.padEnd(20)} ${event.title}: ${event.detail}`),
     ...(events.length > 16 ? [`  … ${events.length - 16} more event(s)`] : []),
   ];
 }
@@ -84,14 +84,14 @@ function buildTranscriptReviewLines(
 // (W6 review, finding 3: saved-layout restore with 'sessions' must not lie)
 // can be unit-tested directly instead of through a full /resume harness.
 // Delegates to the shared reopenPanelsWithModalSkip (core/session-resume-core.ts)
-// — the same routine resumeSessionCore uses — so this standalone entry point
+//, the same routine resumeSessionCore uses, so this standalone entry point
 // and the canonical resume sequence can never diverge on panel-reopen behavior.
 export function reopenPanelsFromReturnContext(ctx: CommandContext, summary: SessionReturnContextSummary | undefined): string[] {
   if (!summary?.openPanels || summary.openPanels.length === 0) return [];
   const panelManager = requirePanelManager(ctx);
   const { reopened, movedToModal, notReopened } = reopenPanelsWithModalSkip(panelManager, summary.openPanels, DEFAULT_PANEL_REOPEN_LIMIT);
   if (movedToModal.length > 0) {
-    ctx.print(`Note: ${movedToModal.join(', ')} moved to a modal — reopen via its command instead of as a panel.`);
+    ctx.print(`Note: ${movedToModal.join(', ')} moved to a modal; reopen via its command instead of as a panel.`);
   }
   if (notReopened.length > 0) {
     ctx.print(`  …and ${notReopened.length} more not reopened (/panels to open)`);
@@ -213,7 +213,7 @@ export async function handleSessionWorkflowCommand(args: string[], ctx: CommandC
           provider: ctx.session.runtime.provider,
           timestamp: Date.now(),
           // Naming a session is an act of curation, so the file this
-          // materializes for rename to act on is a user save, not machinery —
+          // materializes for rename to act on is a user save, not machinery,
           // the retention sweep must not expire something the user named.
           saveSource: 'user',
         });
@@ -229,7 +229,7 @@ export async function handleSessionWorkflowCommand(args: string[], ctx: CommandC
   }
 
   if (sub === 'resume') {
-    // '--force' bypasses the multi-instance liveness confirm below — never a
+    // '--force' bypasses the multi-instance liveness confirm below, never a
     // session-id/name token itself (session ids never start with '--').
     const rawArgs = args.slice(1);
     const force = rawArgs.includes('--force');
@@ -250,14 +250,14 @@ export async function handleSessionWorkflowCommand(args: string[], ctx: CommandC
     }
 
     // Multi-instance safety: warn (and require --force) when this session
-    // looks open in another still-running terminal right now — resuming it
+    // looks open in another still-running terminal right now, resuming it
     // here would fork its live state out from under that other instance.
     // Best-effort: a missing/stale marker (or this process's OWN marker,
     // e.g. re-resuming the session already active here) never blocks anything.
     const liveness = checkSessionLiveness(requireSurface(ctx), found.name);
     if (liveness.live && liveness.pid !== process.pid && !force) {
       ctx.print(
-        `Session appears open in another terminal (pid ${liveness.pid}) — resuming will fork its live state.\n` +
+        `Session appears open in another terminal (pid ${liveness.pid}): resuming will fork its live state.\n` +
         `Re-run \`/session resume ${found.name} --force\` to proceed anyway.`,
       );
       return true;
@@ -279,10 +279,10 @@ export async function handleSessionWorkflowCommand(args: string[], ctx: CommandC
       ctx.renderRequest();
       ctx.print(`Resumed session: ${found.name}\n  Name: ${meta.title || '(untitled)'}\n  Messages: ${resumedMessageCount}\n  Model: ${meta.model || ctx.session.runtime.model}`);
       if (restoredAnchorCount > 0) {
-        ctx.print(`  Rewind: restored ${restoredAnchorCount} turn anchor(s) — /rewind is available for this resumed session.`);
+        ctx.print(`  Rewind: restored ${restoredAnchorCount} turn anchor(s); /rewind is available for this resumed session.`);
       }
       if (journalReplay.replayed > 0) {
-        ctx.print(`  [Recovery] Replayed ${journalReplay.replayed} journal record(s) — restored turns since last snapshot.`);
+        ctx.print(`  [Recovery] Replayed ${journalReplay.replayed} journal record(s): restored turns since last snapshot.`);
       }
       if (journalReplay.hadCorruptTail && journalReplay.replayed === 0) {
         ctx.print('  [Recovery] Journal tail was corrupt or unrecognised (quarantined). Proceeding with snapshot only.');
@@ -290,7 +290,7 @@ export async function handleSessionWorkflowCommand(args: string[], ctx: CommandC
         ctx.print('  [Recovery] Journal tail was partially corrupt (quarantined). Replay stopped at last good record.');
       }
       if (panels.movedToModal.length > 0) {
-        ctx.print(`Note: ${panels.movedToModal.join(', ')} moved to a modal — reopen via its command instead of as a panel.`);
+        ctx.print(`Note: ${panels.movedToModal.join(', ')} moved to a modal; reopen via its command instead of as a panel.`);
       }
       if (panels.notReopened.length > 0) {
         ctx.print(`  …and ${panels.notReopened.length} more not reopened (/panels to open)`);

package/src/input/commands/session.ts

@@ -1,20 +1,20 @@
 /**
- * /session command handler — Multi-session Orchestration.
+ * /session command handler, Multi-session Orchestration.
  *
  * Implements session and workflow commands:
  *
  *   /session link-task <taskId> [--session <sessionId>] [--depends-on <ref>] [--label <label>]
- *     — Register a task as a global cross-session ref, optionally linking it to a
+ *    , Register a task as a global cross-session ref, optionally linking it to a
  *       dependency.
  *
  *   /session handoff <taskId> --to <sessionId> [--session <sessionId>] [--reason <reason>]
- *     — Initiate a task handoff from the current session to another.
+ *    , Initiate a task handoff from the current session to another.
  *
  *   /session graph [--session <sessionId>] [--format text|json]
- *     — Display the cross-session task dependency graph.
+ *    , Display the cross-session task dependency graph.
  *
  *   /session cancel <taskId|--scope session> [--session <sessionId>] [--scope task|subtree|session]
- *     — Cancel tasks with configurable scope semantics.
+ *    , Cancel tasks with configurable scope semantics.
  */
 
 import type { SlashCommand, CommandContext } from '../command-registry.ts';
@@ -92,7 +92,7 @@ function handleLinkTask(args: string[], context: CommandContext): void {
   }
 
   if (taskId.includes(':')) {
-    context.print('Error: taskId cannot contain ":" — it conflicts with the composite key format.');
+    context.print('Error: taskId cannot contain ":"; it conflicts with the composite key format.');
     return;
   }
 
@@ -318,11 +318,11 @@ function handleCancel(args: string[], context: CommandContext): void {
 // ── Top-level command definition ───────────────────────────────────────────────
 
 /**
- * SESSION_SUBCOMMAND_ARG_HINTS — the single source of truth for each
+ * SESSION_SUBCOMMAND_ARG_HINTS, the single source of truth for each
  * `/session <subcommand>` argument hint, co-located with the switch below so
  * the two can't silently drift apart (command-args-hint.ts imports this
- * directly instead of hand-maintaining its own copy; a drift test —
- * src/test/input/command-args-hint.test.ts — cross-checks every key here
+ * directly instead of hand-maintaining its own copy; a drift test,
+ * src/test/input/command-args-hint.test.ts, cross-checks every key here
  * against the default usage text's own subcommand list). Aliases (`link`,
  * `ho`, `g`) share their canonical subcommand's hint text.
  */
@@ -349,14 +349,14 @@ export const SESSION_SUBCOMMAND_ARG_HINTS: Record<string, string> = {
 };
 
 /**
- * sessionCommand — The `/session` slash command.
+ * sessionCommand, The `/session` slash command.
  *
  * The ONE front-door for all session operations. Owns two domains:
  *
  * Lifecycle (continuity, export, resume, pruning):
  *   list | rename | resume | fork | save | info | events | groups | hotspots | export | search | delete
  *
- * Orchestration (cross-session task DAG — 40 tests, cycle detection):
+ * Orchestration (cross-session task DAG, 40 tests, cycle detection):
  *   link-task | handoff | graph | cancel
  *
  * Orchestration-command decision (TASK-032):
@@ -380,7 +380,7 @@ export const sessionCommand: SlashCommand = {
     switch (sub) {
       // ── Lifecycle subcommands ────────────────────────────────────────────────
       // Each delegates explicitly to handleSessionWorkflowCommand so every
-      // subcommand has a deterministic, named path — no silent fallthrough.
+      // subcommand has a deterministic, named path, no silent fallthrough.
       case 'list':
       case 'rename':
       case 'resume':
@@ -457,7 +457,7 @@ export const sessionCommand: SlashCommand = {
 };
 
 /**
- * /resume — the discoverable front door to session resume.
+ * /resume, the discoverable front door to session resume.
  *
  * `/session resume <id>` has always existed but is buried behind a
  * subcommand nobody remembers mid-context-switch. `/resume` with no
@@ -469,7 +469,7 @@ export const sessionCommand: SlashCommand = {
 export const resumeCommand: SlashCommand = {
   name: 'resume',
   aliases: [],
-  description: 'Resume a previous session — pick from a list, or pass an id/name',
+  description: 'Resume a previous session, pick from a list or pass an id/name',
   usage: '[session-id-or-name]',
   argsHint: '[id|name]',
   handler: async (args: string[], ctx: CommandContext): Promise<void> => {
@@ -479,7 +479,7 @@ export const resumeCommand: SlashCommand = {
     }
     const sm = requireSessionManager(ctx);
     // Excludes the current session and subagent-shaped transcripts (see
-    // session-picker-filter.ts) — a direct `/resume agent-xxxxxxxx` still
+    // session-picker-filter.ts), a direct `/resume agent-xxxxxxxx` still
     // reaches them, this only keeps the picker itself readable.
     const sessions = filterUserFacingSessions(sm.list()).filter((s) => s.name !== ctx.session.runtime.sessionId);
     if (sessions.length === 0) {

package/src/input/commands/settings-sync-runtime.ts

@@ -27,14 +27,14 @@ export function registerSettingsSyncRuntimeCommands(registry: CommandRegistry):
     name: 'settings-sync',
     aliases: ['settingssync'],
     description: 'Open the settings sync modal (bare); review posture, export/import bundles, or resolve conflicts by subcommand',
-    usage: '[panel|report|review|show <key>|staged|conflicts|resolve <key> <local|synced>|failures|rollback-history|export <path>|inspect <path>|pull <path>|push <path>|lock <key> <source> <reason...>|unlock <key>] — bare opens the modal',
+    usage: '[panel|report|review|show <key>|staged|conflicts|resolve <key> <local|synced>|failures|rollback-history|export <path>|inspect <path>|pull <path>|push <path>|lock <key> <source> <reason...>|unlock <key>]: bare opens the modal',
     handler(args, ctx) {
       const shellPaths = requireShellPaths(ctx);
       const controlPlaneConfigDir = ctx.platform.configManager.getControlPlaneConfigDir();
       const sub = (args[0] ?? '').toLowerCase();
       // item 3 (report-vs-modal front doors): settings-sync has a full
       // modal surface (settings-sync-modal), so the bare command now opens it
-      // — the old bare/`review` transcript report moved to an explicit
+      //, the old bare/`review` transcript report moved to an explicit
       // `report` subcommand (scriptability preserved: /settings-sync report).
       if (sub === '') {
         ctx.openModal?.('settings-sync-modal');

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

@@ -80,7 +80,7 @@ export function registerShareRuntimeCommands(registry: CommandRegistry): void {
       };
 
       if (!convData.messages || convData.messages.length === 0) {
-        ctx.print('Nothing to export — conversation is empty.');
+        ctx.print('Nothing to export: conversation is empty.');
         return;
       }
 
@@ -113,12 +113,12 @@ export function registerShareRuntimeCommands(registry: CommandRegistry): void {
       const sessionCostUsd = calcSessionCost(
         totalInput, totalOutput, totalCacheRead, totalCacheWrite, sessionModel,
       );
-      // The exported document has no field for "this cost is a placeholder" —
+      // The exported document has no field for "this cost is a placeholder",
       // omit `cost` entirely rather than embed a misleading 0 when the model
       // never resolved to a real price.
       const costIsUnpriced = !isModelPriced(sessionModel);
       if (costIsUnpriced) {
-        ctx.print(`Note: cost omitted from export — no pricing data for model "${sessionModel}".`);
+        ctx.print(`Note: cost omitted from export; no pricing data for model "${sessionModel}".`);
       }
 
       const metadata = {
@@ -167,7 +167,7 @@ export function registerShareRuntimeCommands(registry: CommandRegistry): void {
             authHeaders = await svcRegistry.resolveAuth('github').catch(() => null);
           }
         } catch {
-          // serviceRegistry absent or resolveAuth threw — fall through to env var
+          // serviceRegistry absent or resolveAuth threw, fall through to env var
         }
 
         const token = resolveGithubToken(authHeaders ?? undefined);