@pellux/goodvibes-tui 2.0.13 → 2.0.15

package/src/input/commands/shell-core.ts

@@ -103,13 +103,13 @@ export function registerShellCoreCommands(registry: CommandRegistry): void {
   registry.register({
     name: 'find',
     aliases: ['findtext'],
-    description: 'Search THIS conversation transcript (same as Ctrl+F) — searching the web instead? use /search',
+    description: 'Search THIS conversation transcript (same as Ctrl+F). Searching the web instead? Use /search',
     handler(_args, ctx) {
       if (ctx.openTranscriptSearch) {
         ctx.openTranscriptSearch();
         return;
       }
-      ctx.print('Transcript search is not available in this context — try Ctrl+F.');
+      ctx.print('Transcript search is not available in this context; try Ctrl+F.');
     },
   });
 
@@ -121,7 +121,7 @@ export function registerShellCoreCommands(registry: CommandRegistry): void {
       const km = requireKeybindingsManager(ctx);
       // The full binding table (~45 rows) is presented via the same
       // generated, scrollable overlay /shortcuts opens (see
-      // renderShortcutsOverlay) rather than a one-shot transcript dump —
+      // renderShortcutsOverlay) rather than a one-shot transcript dump,
       // the file path (dynamic, per-user) still gets a short transcript
       // line since the overlay itself doesn't show it.
       ctx.print(`Keybindings config: ${km.getConfigPath()}  (create it to override any binding: { "action": { "key": "x", "ctrl": true } })`);
@@ -196,8 +196,8 @@ export function registerShellCoreCommands(registry: CommandRegistry): void {
     aliases: ['h', '?'],
     description: 'Browse every command with its description; picking one runs it',
     handler(_args, ctx) {
-      // Generated live from the command registry — never a hand-maintained
-      // list — so /help can never drift from the real command set. Category
+      // Generated live from the command registry, never a hand-maintained
+      // list, so /help can never drift from the real command set. Category
       // labels come from the same single source of truth as the generated
       // reference and the palette (ctx.getCommandCategories; see its doc for
       // why it is threaded through the context rather than imported here).
@@ -217,7 +217,7 @@ export function registerShellCoreCommands(registry: CommandRegistry): void {
             category: categoryByName.get(cmd.name) ?? 'Other',
           };
         });
-        // every item here is a command, and picking one RUNS it — label
+        // every item here is a command, and picking one RUNS it, label
         // the verb "Run" (matching the slash-command palette) instead of the
         // generic "Select" default.
         ctx.openSelection('Help  —  Commands', items, { allowSearch: true, primaryVerbLabel: 'Run' }, (result) => {
@@ -269,7 +269,7 @@ export function registerShellCoreCommands(registry: CommandRegistry): void {
     description: 'Summarize conversation to free context window',
     async handler(_args, ctx) {
       const messages = ctx.session.conversationManager.getMessagesForLLM();
-      // contextWindow IS reachable from CommandContext — the same
+      // contextWindow IS reachable from CommandContext, the same
       // call compactConversation() (runtime-services.ts) already makes to scale
       // its own behaviour by window size. Previously hardcoded to 0 here, which
       // silently suppressed buildCompactionPreview()'s capacity-% clause even
@@ -320,7 +320,7 @@ export function registerShellCoreCommands(registry: CommandRegistry): void {
       const memStore = requireSessionMemoryStore(ctx);
       const id = memStore.add(text);
       if (!id) {
-        ctx.print('[Pin] Nothing pinned — text was blank.');
+        ctx.print('[Pin] Nothing pinned: text was blank.');
       } else {
         const count = memStore.list().length;
         ctx.print(buildPinSuccessText(id, text, count));
@@ -370,7 +370,7 @@ export function registerShellCoreCommands(registry: CommandRegistry): void {
     usage: '[level]',
     // No fixed list here. The accepted levels belong to the serving model, so
     // the old '[instant|low|medium|high]' hint was wrong for every model that
-    // offers 'none', 'minimal', 'xhigh' or 'max' — and for every model that
+    // offers 'none', 'minimal', 'xhigh' or 'max', and for every model that
     // rejects 'instant'. Running `/effort` with no argument prints the real
     // ones for the model currently in use.
     argsHint: '[level]',
@@ -394,7 +394,7 @@ export function registerShellCoreCommands(registry: CommandRegistry): void {
       // command is where the preference is read back and re-chosen, so it must
       // show what was asked for even while a capped model is serving.
       const current = requestedEffortLevel(ctx.platform.configManager);
-      // What the list should open on, though, is the level in EFFECT — the
+      // What the list should open on, though, is the level in EFFECT, the
       // requested level snapped to this model. The requested level may not be
       // in the list at all when the model caps lower, and preselecting a
       // missing id lands the cursor on the lowest level instead of on what is
@@ -402,7 +402,7 @@ export function registerShellCoreCommands(registry: CommandRegistry): void {
       const inEffect = servingEffortForLevel(current, model).effective ?? current;
 
       const applyLevel = (level: string): void => {
-        // An explicit user choice — the one kind of write that is allowed to
+        // An explicit user choice, the one kind of write that is allowed to
         // change the stored preference.
         ctx.platform.configManager.set('provider.reasoningEffort', level);
         const serving = servingEffortForLevel(level, model);

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

@@ -1,17 +1,17 @@
 /**
- * tasks-runtime.ts — `/tasks`.
+ * tasks-runtime.ts, `/tasks`.
  *
  * The reads are a UNION now: this surface's own runtime tasks (the exec, agent
  * and ACP work the loop here spawned) plus the daemon's (scheduled work,
  * channel-driven runs, tasks other surfaces submitted). Before the daemon
  * became its own product one registry held both, and `tasks.list(500)` off the
  * in-process transport answered synchronously. It no longer holds both, so this
- * command crosses a wire and is async — the same shape `/ci` and the fleet act
+ * command crosses a wire and is async, the same shape `/ci` and the fleet act
  * verbs already use.
  *
  * The list output is unchanged for a local-only fleet: same header, same
  * columns, same 20-row cap, same filter semantics. What is added is the daemon
- * half, and one honest line when the daemon could not be reached — a partial
+ * half, and one honest line when the daemon could not be reached, a partial
  * list of real tasks beats an error page, but it must not read as complete.
  *
  * Writes do NOT degrade that way. A lifecycle act against a task this terminal
@@ -226,7 +226,7 @@ export function registerTasksRuntimeCommands(registry: CommandRegistry): void {
 
       // Which registry owns this id decides what may be done to it. An act
       // aimed at a daemon task must never be applied to a local record that
-      // does not exist — that reports success and changes nothing.
+      // does not exist, that reports success and changes nothing.
       const found = await tasksClient.get(taskId);
       if (!found) {
         ctx.print(`Unknown task: ${taskId}`);

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

@@ -5,7 +5,7 @@ import { requireShellPaths } from './runtime-services.ts';
 
 /**
  * Timeout for a /test run, in milliseconds. Deliberately much longer than
- * WRFC's WRFC_GATE_TIMEOUT_MS (120s, tuned for CI gates) — this repo's own
+ * WRFC's WRFC_GATE_TIMEOUT_MS (120s, tuned for CI gates), this repo's own
  * suite is 600+ test files (scripts/run-tests.ts), and /test is an
  * interactive command with no other caller waiting on it.
  *
@@ -48,7 +48,7 @@ interface ParsedTestResults {
  * Parse this repo's own scripts/run-tests.ts output shape:
  *   per-file:  `==> path/to/file.test.ts  [FAIL]` (only present on failure)
  *   summary:   `Test files: N, passed: P, failed: F`
- * Returns null when the summary line isn't present at all — the fallback
+ * Returns null when the summary line isn't present at all, the fallback
  * path (raw tail, no fabricated counts) covers a different project's
  * jest/vitest/pytest output, or any run that bypassed run-tests.ts.
  */
@@ -96,7 +96,7 @@ export function parseTestOutput(output: string): ParsedTestResults | null {
  * quote into `'\''` (close quote, escaped literal quote, reopen quote).
  * Mirrors the injection-risk class already present in the SDK's
  * wrfc-gates.ts (executeGateCommand) and this repo's git-runtime.ts /
- * diff-runtime.ts, which build shell command strings the same way — a
+ * diff-runtime.ts, which build shell command strings the same way, a
  * pattern arg is never string-concatenated into the command raw.
  */
 export function shQuote(value: string): string {
@@ -215,13 +215,13 @@ export async function runTestCommand(
     // Durable end state: logToolResultBlock (above) renders straight into the
     // display-only history buffer and never touches the real message list, so
     // it does not survive the next full rebuildHistory() (which rebuilds
-    // strictly from conversationManager.getMessageSnapshot()) — the very next
+    // strictly from conversationManager.getMessageSnapshot()), the very next
     // dirty render silently wipes it back to how the transcript looked before
     // /test ran. addSystemMessage persists the same content as a real message
     // (same durable pattern as /rewind's confirm notice in
     // checkpoint-runtime.ts) so the timeout outcome survives like any other
     // command's output. The streamed output during the run is allowed to stay
-    // transient — only this final state needs to persist.
+    // transient, only this final state needs to persist.
     ctx.session.conversationManager.addSystemMessage(
       `[Test] Timed out after ${timeoutSeconds}s and was killed.`,
     );
@@ -241,7 +241,7 @@ export async function runTestCommand(
     const failedNoun = parsed.unit === 'files' ? 'file' : 'test';
     const errorMsg = ok ? undefined : `${parsed.failed} ${failedNoun}${parsed.failed === 1 ? '' : 's'} failed`;
     ctx.session.conversationManager.logToolResultBlock(toolCall, ok ? 'done' : 'error', summary, durationMs, errorMsg);
-    const durableLines = [`[Test] ${summary}${errorMsg ? ` — ${errorMsg}` : ''}`];
+    const durableLines = [`[Test] ${summary}${errorMsg ? `: ${errorMsg}` : ''}`];
     if (parsed.failingFiles.length > 0) {
       const shown = parsed.failingFiles.slice(0, MAX_FAILING_NAMES_SHOWN);
       const failListHeader = parsed.unit === 'files' ? 'Failing test files:' : 'Failing tests:';

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

@@ -8,14 +8,14 @@ export function registerTtsRuntimeCommands(registry: CommandRegistry): void {
     handler(args, ctx) {
       const first = (args[0] ?? '').toLowerCase();
 
-      // /tts stop — cancel active playback
+      // /tts stop, cancel active playback
       if (first === 'stop' || first === 'cancel') {
         ctx.stopSpokenOutput?.();
         ctx.print('Live TTS playback stopped.');
         return;
       }
 
-      // /tts on — enable always-speak mode (every turn spoken automatically)
+      // /tts on, enable always-speak mode (every turn spoken automatically)
       if (first === 'on') {
         if (!ctx.platform.voiceService) {
           ctx.print('Live TTS is not available in this runtime.');
@@ -28,7 +28,7 @@ export function registerTtsRuntimeCommands(registry: CommandRegistry): void {
         return;
       }
 
-      // /tts off — disable always-speak mode
+      // /tts off, disable always-speak mode
       if (first === 'off') {
         ctx.platform.configManager.setDynamic('ui.voiceEnabled', false);
         ctx.platform.configManager.save();
@@ -37,7 +37,7 @@ export function registerTtsRuntimeCommands(registry: CommandRegistry): void {
         return;
       }
 
-      // /tts <prompt> — mark this prompt for spoken output
+      // /tts <prompt>, mark this prompt for spoken output
       const prompt = args.join(' ').trim();
       if (!prompt) {
         const enabled = ctx.platform.configManager.get('ui.voiceEnabled');

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

@@ -1,16 +1,16 @@
 /**
- * `/update` — a real self-update path for binary installs. The
+ * `/update`, a real self-update path for binary installs. The
  * download-verify-swap mechanics are the SDK's canonical update policy
- * module (platform/runtime/self-update — hoisted from this file's
+ * module (platform/runtime/self-update, hoisted from this file's
  * semantics), the same mechanism the daemon's hourly loop and
  * the suite installer follow: one update mechanism everywhere. This file owns
  * only the /update UX: install-kind gating, target selection, and the
  * printed report.
  *
  * Subcommands:
- *   /update [check]           — resolve the latest release tag and report
+ *   /update [check]          , resolve the latest release tag and report
  *                                whether this build is already current.
- *   /update apply              — for a binary install (the curl installer),
+ *   /update apply             , for a binary install (the curl installer),
  *                                download + verify + atomically swap THIS
  *                                app's binary, and refresh the sqlite-vec
  *                                native addon in lockstep so the vector index
@@ -18,9 +18,9 @@
  *                                swap parks the outgoing file at
  *                                `<path>.previous`, so the replaced version is
  *                                always kept. For any other install kind,
- *                                prints the exact command to run instead — it
+ *                                prints the exact command to run instead, it
  *                                never attempts a swap it can't do safely.
- *   /update rollback           — exchange each installed file with its kept
+ *   /update rollback          , exchange each installed file with its kept
  *                                `.previous` counterpart: one command back to
  *                                the version that ran before the last update
  *                                (and, being an exchange, one more command
@@ -30,15 +30,15 @@
  * binary as part of the same release, and this command swapped both. The daemon
  * is its own product now, released from its own repository on its own version
  * line, and it updates itself on an hourly loop. A swap from here would fetch
- * an asset this repository's releases do not publish — and if one ever appeared
+ * an asset this repository's releases do not publish, and if one ever appeared
  * under that name, would overwrite a working daemon with a build from the wrong
  * line. The daemon is a neighbour in the install directory, not cargo.
- *   /update review              — install/subscription/sandbox posture,
+ *   /update review             , install/subscription/sandbox posture,
  *                                unrelated to the update mechanics above.
- *   /update bundle export|inspect <path> — portable posture bundle, as before.
+ *   /update bundle export|inspect <path>, portable posture bundle, as before.
  *
  * The former `/update channel <stable|preview>` subcommand wrote a
- * release.channel config value that nothing downstream ever read — it only
+ * release.channel config value that nothing downstream ever read, it only
  * ever changed what a later `/update review` printed back to you. It has
  * been removed rather than kept decorative; this repo's release process
  * (scripts/release.ts) does not publish separate stable/preview channels,
@@ -94,13 +94,13 @@ async function downloadText(fetchImpl: UpdateFetchLike, url: string): Promise<st
  * to and including the moment BEFORE the first file is written: every fetch on
  * the way there carries the caller's signal, and every await boundary re-checks
  * it. From the first write onward the swap owns the installed files and always
- * runs to completion — a half-applied swap is the one outcome worse than a slow
+ * runs to completion, a half-applied swap is the one outcome worse than a slow
  * one.
  *
  * This record makes that boundary readable from outside the call. The launch
  * updater gives `applyUpdate` a budget and abandons the promise when it runs
  * out, so it cannot learn from the return value which side of the line the work
- * was on — it reads these flags instead, and prints the receipt that is
+ * was on, it reads these flags instead, and prints the receipt that is
  * actually true (see src/cli/launch-auto-update.ts).
  */
 export interface UpdateSwapProgress {
@@ -116,7 +116,7 @@ export function createUpdateSwapProgress(): UpdateSwapProgress {
   return { begun: false, committed: false, targetTag: null };
 }
 
-/** The failure an aborted update ends with — raised only while nothing has been written yet. */
+/** The failure an aborted update ends with, raised only while nothing has been written yet. */
 export const UPDATE_ABORTED_MESSAGE = 'update cancelled before any file was replaced';
 
 function throwIfAborted(signal: AbortSignal | undefined): void {
@@ -127,8 +127,8 @@ function throwIfAborted(signal: AbortSignal | undefined): void {
 
 /**
  * The shared UpdateFetchLike init shape predates cancellation, so the signal
- * rides on a widened version of it: the real `fetch` reads it — which is what
- * makes the DOWNLOAD itself cancellable rather than merely abandoned — and a
+ * rides on a widened version of it: the real `fetch` reads it, which is what
+ * makes the DOWNLOAD itself cancellable rather than merely abandoned, and a
  * test stub that ignores the extra field behaves exactly as it did before.
  */
 type AbortableFetchInit = NonNullable<Parameters<UpdateFetchLike>[1]> & { signal?: AbortSignal };
@@ -175,7 +175,7 @@ function trackSwapProgress(io: UpdateFileIo, progress: UpdateSwapProgress): Upda
 }
 
 /**
- * Suffix under which every swap keeps the file it replaced — re-exported
+ * Suffix under which every swap keeps the file it replaced, re-exported
  * from the SDK's canonical update policy module so rollback and swap share
  * one definition everywhere.
  */
@@ -202,7 +202,7 @@ const defaultRunCommand: RunCommand = (command, args) => {
 
 /**
  * Detects, honestly, whether the daemon is currently running as a systemd
- * user service — so the post-update message can tell the user the real
+ * user service, so the post-update message can tell the user the real
  * restart command instead of assuming one. Only Linux/systemd is checked
  * (this repo's PlatformServiceManager also supports launchd on macOS and
  * Scheduled Tasks on Windows, but those aren't restarted with `systemctl`,
@@ -246,7 +246,7 @@ export interface ApplyUpdateOptions {
   /** Injectable filesystem seam (the SDK's UpdateFileIo) so tests observe swaps in memory. */
   readonly io?: UpdateFileIo;
   /**
-   * Cancels the update — for real: it is passed to every fetch in the path and
+   * Cancels the update, for real: it is passed to every fetch in the path and
    * re-checked at every await boundary, so an abort stops the download instead
    * of leaving it running unwatched. Honoured only up to the moment before the
    * swap begins; from the first file write onward it is deliberately ignored
@@ -259,13 +259,13 @@ export interface ApplyUpdateOptions {
 
 /**
  * The real self-update path, delegating the download-verify-swap mechanics
- * to the SDK's canonical update policy module (applyVerifiedUpdate — the
+ * to the SDK's canonical update policy module (applyVerifiedUpdate, the
  * same mechanism the daemon's hourly loop uses). For a binary install:
  * resolve the latest tag, compare to the running version, and if newer,
  * download + checksum-verify EVERY artifact before swapping any one (so a
  * checksum failure never leaves a mismatched pair installed), then
  * atomically swap each in place with the outgoing file kept at
- * `<path>.previous`. For any other install kind, never attempts a swap — it
+ * `<path>.previous`. For any other install kind, never attempts a swap, it
  * prints the exact command for that install method instead.
  */
 export async function applyUpdate(options: ApplyUpdateOptions): Promise<void> {
@@ -313,7 +313,7 @@ export async function applyUpdate(options: ApplyUpdateOptions): Promise<void> {
   // same download-verify-swap pass so /update never leaves a new binary beside a
   // stale addon. It lands at <execDir>/lib/sqlite-vec-<os>-<arch>/vec0.<suffix>,
   // exactly where the SDK's loader resolves it. The manifest entry decides
-  // whether the target release ships it — an entry that IS present makes the
+  // whether the target release ships it, an entry that IS present makes the
   // download and checksum mandatory (a mismatch is fatal, verified before any
   // swap), while an absent entry means the target predates the addon and is
   // skipped rather than blocking an otherwise-valid binary update. On macOS the
@@ -343,7 +343,7 @@ export async function applyUpdate(options: ApplyUpdateOptions): Promise<void> {
   // This is the last point at which the update can be called off. The
   // downloads inside applyVerifiedUpdate are still cancellable (the signal
   // rides on every request), but its swap loop is synchronous and runs to
-  // completion once its first write lands — which is precisely what
+  // completion once its first write lands, which is precisely what
   // `progress.begun` records, and why nothing below this call re-checks the
   // signal.
   throwIfAborted(signal);
@@ -368,8 +368,8 @@ export async function applyUpdate(options: ApplyUpdateOptions): Promise<void> {
       // Named, not silently omitted: someone who used to see two binaries swap
       // here should read why only one did.
       serviceInfo.managed
-        ? `The daemon updates itself on its own release line — this changed nothing about it (it runs under ${serviceInfo.unitName}.service).`
-        : 'The daemon updates itself on its own release line — this changed nothing about it.',
+        ? `The daemon updates itself on its own release line; this changed nothing about it (it runs under ${serviceInfo.unitName}.service).`
+        : 'The daemon updates itself on its own release line; this changed nothing about it.',
     ].join('\n'),
   );
 }
@@ -390,7 +390,7 @@ export interface RollbackUpdateOptions {
  * delegating the exchange mechanics to the SDK's rollbackKeptPrevious (the
  * same module the swap uses): every installed file this command owns (the app
  * binary and the vector addon) that has a kept `.previous` counterpart is
- * EXCHANGED with it — the previous version becomes live, and the version being rolled
+ * EXCHANGED with it, the previous version becomes live, and the version being rolled
  * back is itself kept at `.previous`, so a second `/update rollback` rolls
  * forward again. Files without a kept counterpart are reported and left
  * untouched; nothing is downloaded.
@@ -434,8 +434,8 @@ export function rollbackUpdate(options: RollbackUpdateOptions): void {
       '',
       'Restart goodvibes to run the restored version.',
       serviceInfo.managed
-        ? `The daemon was not rolled back — it updates itself on its own release line (it runs under ${serviceInfo.unitName}.service).`
-        : 'The daemon was not rolled back — it updates itself on its own release line.',
+        ? `The daemon was not rolled back; it updates itself on its own release line (it runs under ${serviceInfo.unitName}.service).`
+        : 'The daemon was not rolled back; it updates itself on its own release line.',
     ].join('\n'),
   );
 }

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

@@ -25,7 +25,7 @@ function parseSearchArgs(args: string[]): ParsedSearchArgs {
  * registerWebSearchRuntimeCommands - `/search <query> [--limit <n>]`.
  *
  * Calls webSearchService.search() directly (skipping the agent-tool wrapper's
- * JSON-stringify contract — same shape as /schedule's ctx.ops.automationManager
+ * JSON-stringify contract, same shape as /schedule's ctx.ops.automationManager
  * pattern: read the service off the context, guard on undefined, print) and
  * renders ranked results + the instant answer into the transcript with
  * source labels. webSearchService is already constructed in services.ts and
@@ -35,7 +35,7 @@ function parseSearchArgs(args: string[]): ParsedSearchArgs {
 export function registerWebSearchRuntimeCommands(registry: CommandRegistry): void {
   registry.register({
     name: 'search',
-    description: 'Web search — ranked results with source labels (searching THIS transcript? use /find)',
+    description: 'Web search, ranked results with source labels (searching THIS transcript? use /find)',
     usage: '<query> [--limit <n>]',
     argsHint: '<query> [--limit <n>]',
     async handler(args, ctx) {

package/src/input/commands/work-plan-runtime.ts

@@ -25,7 +25,7 @@ function getStore(ctx: import('../command-registry.ts').CommandContext): WorkPla
 }
 
 function openPanel(ctx: import('../command-registry.ts').CommandContext): void {
-  // work-plan migrated to the 'work-plan' modal — open it via the seam.
+  // work-plan migrated to the 'work-plan' modal, open it via the seam.
   openModalCommand(ctx, 'work-plan-modal');
 }
 
@@ -99,7 +99,7 @@ export function registerWorkPlanRuntimeCommands(registry: CommandRegistry): void
           return;
         }
         if (subcommand === 'export') {
-          // Thin wrapper over WorkPlanStore.exportMarkdown() — writes the plan's
+          // Thin wrapper over WorkPlanStore.exportMarkdown(), writes the plan's
           // markdown alongside the JSON and reports the path. (W6 command-path
           // parity: the work-plan modal's export action routes here.)
           const { path } = store.exportMarkdown();

package/src/input/commands/workstream-attempt-dependents.ts

@@ -1,5 +1,5 @@
 // ---------------------------------------------------------------------------
-// workstream-attempt-dependents.ts — held-dependent rendering for non-leaf
+// workstream-attempt-dependents.ts, held-dependent rendering for non-leaf
 // best-of-N groups.
 //
 // A best-of-N item may be NON-LEAF: other items can depend on it. The scheduler
@@ -7,7 +7,7 @@
 // resolves the dependency to that winner (SDK scheduler.ts attemptGroup
 // DependencyStatus). This module mirrors that status computation as a pure
 // function so the /workstream attempts surface can show, honestly, which
-// dependents are held/waiting/released for a group — and reflect the losing
+// dependents are held/waiting/released for a group, and reflect the losing
 // attempts' cleanup that the winner pick performs.
 // ---------------------------------------------------------------------------
 
@@ -65,9 +65,9 @@ export interface DependentHold {
 function holdDescriptor(status: AttemptGroupDependencyStatus): string {
   switch (status) {
     case 'satisfied':
-      return 'released — the winner was picked and merged';
+      return 'released: the winner was picked and merged';
     case 'failed':
-      return 'blocked — every attempt in the group failed';
+      return 'blocked: every attempt in the group failed';
     default:
       return 'held until the winner is picked and merged';
   }

package/src/input/commands/workstream-attempts.ts

@@ -1,16 +1,16 @@
 // ---------------------------------------------------------------------------
-// workstream-attempts.ts — the /workstream attempts surface for best-of-N.
+// workstream-attempts.ts, the /workstream attempts surface for best-of-N.
 //
 // The engine runs a best-of-N work item as N sibling attempts in isolated
 // worktrees and HOLDS the merge, exposing the candidates for a winner pick
 // (orchestration engine.listHeldMergeGroups / proposeAttemptWinner /
-// pickAttemptWinner — the same seam the fleet.attempts.* gateway verbs use).
+// pickAttemptWinner, the same seam the fleet.attempts.* gateway verbs use).
 // This module drives that seam from the /workstream command:
-//   • list             — the held-merge groups awaiting a pick, with candidates.
-//   • diff <c>         — one candidate's worktree diff in the existing DiffPanel.
-//   • judge            — the OPTIONAL model proposal, clearly labelled a model
+//   • list            , the held-merge groups awaiting a pick, with candidates.
+//   • diff <c>        , one candidate's worktree diff in the existing DiffPanel.
+//   • judge           , the OPTIONAL model proposal, clearly labelled a model
 //                        judgment (scoredBy 'model'), with its reasons; never a pick.
-//   • pick <c>         — behind a DiffPanel confirm; merges the winner and lets
+//   • pick <c>        , behind a DiffPanel confirm; merges the winner and lets
 //                        the engine clean the losers, then renders the receipt.
 // ---------------------------------------------------------------------------
 
@@ -64,7 +64,7 @@ function candidateLine(c: AttemptCandidate): string {
     : isTurnBudgetReason(c.failureReason)
       ? describeFailureReason(c.failureReason) // a spent budget, not an infrastructure failure
       : `failed${c.failureReason ? `: ${c.failureReason}` : ''}`;
-  return `    ${c.attemptIndex + 1}. [${state}] ${c.title} — ${files}, item ${shortId(c.itemId)}`;
+  return `    ${c.attemptIndex + 1}. [${state}] ${c.title}: ${files}, item ${shortId(c.itemId)}`;
 }
 
 function renderGroupList(groups: readonly HeldMergeGroup[], resolveItems?: ResolveWorkstreamItems): string {
@@ -73,7 +73,7 @@ function renderGroupList(groups: readonly HeldMergeGroup[], resolveItems?: Resol
   }
   const lines: string[] = ['Best-of-N groups (held for a winner pick):'];
   for (const g of groups) {
-    lines.push(`  ${shortId(g.groupId)} — "${g.sourceTitle}" (workstream ${shortId(g.workstreamId)})  ${g.ready ? 'READY' : 'still running'}${g.autoAccept ? ', auto-accept' : ''}`);
+    lines.push(`  ${shortId(g.groupId)}: "${g.sourceTitle}" (workstream ${shortId(g.workstreamId)})  ${g.ready ? 'READY' : 'still running'}${g.autoAccept ? ', auto-accept' : ''}`);
     for (const c of g.candidates) lines.push(candidateLine(c));
     if (g.judgment?.proposedWinnerItemId) {
       lines.push(`    judge proposal: item ${shortId(g.judgment.proposedWinnerItemId)} (model, advisory)`);
@@ -91,13 +91,13 @@ function renderGroupList(groups: readonly HeldMergeGroup[], resolveItems?: Resol
 
 /** The model proposal, CLEARLY labelled a model judgment (never ground truth), with reasons. */
 function renderJudgment(group: HeldMergeGroup, judgment: AttemptJudgment): string {
-  const lines: string[] = [`Judge proposal for group ${shortId(group.groupId)} — "${group.sourceTitle}"`];
-  lines.push('  MODEL PROPOSAL (scoredBy: model) — advisory only; a human still confirms the pick.');
+  const lines: string[] = [`Judge proposal for group ${shortId(group.groupId)}: "${group.sourceTitle}"`];
+  lines.push('  MODEL PROPOSAL (scoredBy: model); advisory only; a human still confirms the pick.');
   if (judgment.proposedWinnerItemId) {
     const cand = group.candidates.find((c) => c.itemId === judgment.proposedWinnerItemId);
-    lines.push(`  Proposed winner: ${cand ? `attempt ${cand.attemptIndex + 1} — ${cand.title}` : `item ${shortId(judgment.proposedWinnerItemId)}`}`);
+    lines.push(`  Proposed winner: ${cand ? `attempt ${cand.attemptIndex + 1}: ${cand.title}` : `item ${shortId(judgment.proposedWinnerItemId)}`}`);
   } else {
-    lines.push('  Proposed winner: none — the judge declined to choose.');
+    lines.push('  Proposed winner: none; the judge declined to choose.');
   }
   if (judgment.model) lines.push(`  Model: ${judgment.model}`);
   if (judgment.reasons.length > 0) {
@@ -199,10 +199,10 @@ export async function handleAttemptsSubcommand(ctx: CommandContext, service: Wor
   if (action === 'pick') {
     const candRef = args[3];
     if (!candRef) { ctx.print(`Usage: /workstream attempts pick ${shortId(group.groupId)} <#>`); return true; }
-    if (!group.ready) { ctx.print(`Group ${shortId(group.groupId)} is not ready — its siblings are still running. Wait until every attempt is terminal.`); return true; }
+    if (!group.ready) { ctx.print(`Group ${shortId(group.groupId)} is not ready: its siblings are still running. Wait until every attempt is terminal.`); return true; }
     const cand = resolveCandidate(group, candRef);
     if ('error' in cand) { ctx.print(cand.error); return true; }
-    if (cand.state !== 'held-merge') { ctx.print(`Attempt ${cand.attemptIndex + 1} failed — only a held (pick-ready) candidate can win.`); return true; }
+    if (cand.state !== 'held-merge') { ctx.print(`Attempt ${cand.attemptIndex + 1} failed: only a held (pick-ready) candidate can win.`); return true; }
 
     const { DiffPanel } = await import('../../panels/diff-panel.ts');
     const pm = requirePanelManager(ctx);
@@ -217,7 +217,7 @@ export async function handleAttemptsSubcommand(ctx: CommandContext, service: Wor
 
     diffPanel.confirmOverlay.arm({
       id: `${group.groupId}:${cand.itemId}`,
-      label: `Pick attempt ${cand.attemptIndex + 1} ("${cand.title}") — merge it, clean the ${group.candidates.length - 1} other worktree(s)`,
+      label: `Pick attempt ${cand.attemptIndex + 1} ("${cand.title}"): merge it, clean the ${group.candidates.length - 1} other worktree(s)`,
       verb: 'Pick',
       onConfirm: async () => {
         try {
@@ -235,7 +235,7 @@ export async function handleAttemptsSubcommand(ctx: CommandContext, service: Wor
       onCancel: () => {
         pm.close('diff');
         ctx.focusPrompt?.();
-        ctx.print('Pick cancelled — nothing merged, no worktree cleaned.');
+        ctx.print('Pick cancelled: nothing merged, no worktree cleaned.');
         ctx.renderRequest();
       },
     });