@pellux/goodvibes-tui 2.0.13 → 2.0.15

package/src/core/turn-event-wiring.ts

@@ -20,7 +20,7 @@ import { readBudgetAlertUsd } from '@pellux/goodvibes-sdk/platform/providers';
 /** Minimal orchestrator surface required by turn-event wiring. */
 interface TurnOrchestrator {
   readonly lastInputTokens: number;
-  /** Cumulative session usage — same object CostTrackerPanel reads, used here for budget-breach checks. */
+  /** Cumulative session usage, same object CostTrackerPanel reads, used here for budget-breach checks. */
   readonly usage: { readonly input: number; readonly output: number; readonly cacheRead: number; readonly cacheWrite: number };
 }
 
@@ -50,7 +50,7 @@ export interface WireTurnEventHandlersOptions {
   readonly events: UiRuntimeEvents;
   /**
    * Raw runtime event bus. Needed for domains NOT surfaced by the grouped
-   * UiRuntimeEvents feed — specifically the 'compaction' domain, whose
+   * UiRuntimeEvents feed, specifically the 'compaction' domain, whose
    * mandatory COMPACTION_RECEIPT the transcript renders as a distinct block.
    * Optional so headless/test callers can omit it.
    */
@@ -65,7 +65,7 @@ export interface WireTurnEventHandlersOptions {
   /**
    * The app's declare-once session-storage handle. The turn snapshot, the
    * last-session pointer it writes, the transcript journal, and the rewind
-   * anchor sidecar all resolve off this one handle — so the boot notice and
+   * anchor sidecar all resolve off this one handle, so the boot notice and
    * `--continue`, which read through the same handle, land on the files this
    * writer actually produced.
    */
@@ -77,13 +77,13 @@ export interface WireTurnEventHandlersOptions {
   /**
    * Outbound webhook notifier. When provided and URLs are configured,
    * long-task push notifications are delivered to configured ntfy/webhook
-   * endpoints after the configured threshold. Optional — silently skipped
+   * endpoints after the configured threshold. Optional, silently skipped
    * when absent.
    */
   readonly webhookNotifier?: WebhookNotifier | null;
   /**
    * Terminal focus tracker. Gates the long-task, budget-breach, and
-   * agent/chain-failure desktop alerts wired in this module — see
+   * agent/chain-failure desktop alerts wired in this module, see
    * alert-gating.ts. Optional; when absent, none of the new alert
    * behavior is gated by focus (long-task keeps its always-fire
    * behavior from before focus gating existed, and budget-breach/failure alerts are
@@ -102,7 +102,7 @@ export interface WireTurnEventHandlersOptions {
   /**
    * Minimal test seam: injectable clock for controlling Date.now() in tests.
    * Defaults to the real Date.now when absent.
-   * @internal — tests only
+   * @internal, tests only
    */
   readonly _clock?: () => number;
 }
@@ -179,7 +179,7 @@ export function wireTurnEventHandlers(
   // /session fork reassign `sessionId` on in place (session-workflow.ts,
   // bootstrap-hook-bridge.ts), so a session switch is visible here just by
   // re-reading `runtime.sessionId`. Wrapped so every write re-checks it and
-  // rebinds to the new session's file first — otherwise the journal opened
+  // rebinds to the new session's file first, otherwise the journal opened
   // for the OLD session keeps appending the NEW session's snapshots into the
   // OLD session's file, and a later resume of the old session replays the
   // wrong conversation.
@@ -192,7 +192,7 @@ export function wireTurnEventHandlers(
   // Track turn start time for long-task notification threshold.
   let turnStartTime: number | null = null;
 
-  // Budget-breach edge-trigger checker — one instance per session,
+  // Budget-breach edge-trigger checker, one instance per session,
   // piggybacking on the same TURN_COMPLETED handler as the long-task
   // notification below rather than adding a second TURN_COMPLETED subscription.
   const budgetBreachNotifier: BudgetBreachNotifier | null = focusTracker
@@ -220,7 +220,7 @@ export function wireTurnEventHandlers(
     const notifyThreshold = readNotifyAfterSeconds((k) => configManager.get(k as Parameters<typeof configManager.get>[0]));
     // stopReason 'empty_response' signals a non-successful completion.
     const taskStatus: LongTaskStatus = evt.stopReason === 'completed' ? 'ok' : 'fail';
-    // In-terminal (OSC 9) turn-end notification — fires on its own per-signal
+    // In-terminal (OSC 9) turn-end notification, fires on its own per-signal
     // config + focus gate, independent of the long-task duration threshold below.
     terminalNotifier?.notify('turn-end', taskStatus === 'ok' ? 'Turn finished' : 'Turn finished with errors');
     maybeNotifyLongTask({
@@ -256,10 +256,10 @@ export function wireTurnEventHandlers(
         'auto',
       );
       hookDispatcher.fire({ path: 'Lifecycle:session:save' as HookEventPath, phase: 'Lifecycle' as HookPhase, category: 'session' as HookCategory, specific: 'save', sessionId: runtime.sessionId, timestamp: Date.now(), payload: { sessionId: runtime.sessionId } }).catch((err: unknown) => logger.debug('hook fire error', { error: summarizeError(err) }));
-      // Snapshot succeeded — rotate the journal (gap-filler no longer needed).
+      // Snapshot succeeded, rotate the journal (gap-filler no longer needed).
       transcriptJournal.rotate();
     } catch (e) {
-      // Snapshot failed — append the turn to the journal so recovery can
+      // Snapshot failed, append the turn to the journal so recovery can
       // reconstruct it. Best-effort; never crash the TUI.
       try {
         const snap = conversation.toJSON() as { messages: Array<import('./conversation.ts').ConversationMessageSnapshot> };
@@ -270,7 +270,7 @@ export function wireTurnEventHandlers(
     // Record this turn's rewind anchor: pair the turnId (shared with the
     // workspace checkpoint the turn engine snapshots for this same turn) with
     // the live conversation message count, so a later message-anchored /rewind
-    // can truncate the conversation to exactly this boundary — the join key
+    // can truncate the conversation to exactly this boundary, the join key
     // between conversation and files rewind (see core/rewind-turn-anchors.ts).
     try {
       recordTurnAnchor(runtime.sessionId, {
@@ -294,7 +294,7 @@ export function wireTurnEventHandlers(
   // In-terminal (OSC 9) agent-blocked notification: a delegated agent parked
   // waiting for a human message (AGENT_AWAITING_MESSAGE) is "blocked on you".
   // Gated by the notifier's own per-signal config + focus rule. PRIVACY: the
-  // short agent id only — never task text.
+  // short agent id only, never task text.
   if (terminalNotifier) {
     unsubs.push(events.agents.on('AGENT_AWAITING_MESSAGE', (payload) => {
       terminalNotifier.notify('agent-blocked', `agent ${payload.agentId.slice(0, 8)} is waiting for your input`);
@@ -311,16 +311,16 @@ export function wireTurnEventHandlers(
   // Agent/chain-failure desktop alerts. The SDK's WebhookNotifier and
   // Notifier already fire webhook/Slack/Discord notifications for these two
   // events unconditionally (attachToRuntimeBus in the SDK's
-  // platform/integrations — pre-existing, not focus-gated: an out-of-band
+  // platform/integrations, pre-existing, not focus-gated: an out-of-band
   // push to another device is useful regardless of terminal focus). What was
   // missing was a desktop notification, gated by focus like the other three
-  // alert classes — that's the only thing added here, to avoid double-firing
+  // alert classes, that's the only thing added here, to avoid double-firing
   // a webhook that's already covered.
   if (focusTracker) {
     unsubs.push(events.agents.on('AGENT_FAILED', (payload) => {
       if (!shouldFireAlert(focusTracker, configGet, 'behavior.notifyOnAgentFailure')) return;
       try {
-        notifyCompletion('GoodVibes — agent failed', `agent ${payload.agentId.slice(0, 8)} failed: ${payload.error}`, FORCE_NOTIFY_DURATION_MS);
+        notifyCompletion('GoodVibes: agent failed', `agent ${payload.agentId.slice(0, 8)} failed: ${payload.error}`, FORCE_NOTIFY_DURATION_MS);
       } catch (err) {
         logger.debug('turn-event-wiring: agent-failure notify error', { error: String(err) });
       }

package/src/core/voice-capture-status.ts

@@ -1,5 +1,5 @@
 /**
- * voice-capture-status.ts — what the shell shows while a microphone is open.
+ * voice-capture-status.ts, what the shell shows while a microphone is open.
  *
  * A capture indicator is not decoration. Two of the three states here exist
  * because the alternative is a user who cannot tell whether their microphone is
@@ -41,7 +41,7 @@ export interface VoiceCaptureIndicatorState {
    * one keypress ago and is owed the confirmation.
    */
   readonly indicator: 'off' | 'statusline' | 'banner';
-  /** Extra words for the row — a restart delay, a latch reason, an elapsed time. */
+  /** Extra words for the row, a restart delay, a latch reason, an elapsed time. */
   readonly detail?: string | undefined;
 }
 

package/src/core/voice-provision-gateway.ts

@@ -2,15 +2,15 @@
 // voice-provision-gateway.ts
 //
 // The daemon-backed verbs the /voice setup + /voice status surfaces read:
-// voice.local.status (GET /api/voice/local/status) — the no-network installed
-// snapshot — and voice.local.install (POST /api/voice/local/install) — the
+// voice.local.status (GET /api/voice/local/status), the no-network installed
+// snapshot, and voice.local.install (POST /api/voice/local/install), the
 // one-act managed install. Neither has a named facade on the in-process
-// OperatorClient, so — exactly like the memory-consolidation gateway — they go
+// OperatorClient, so, exactly like the memory-consolidation gateway, they go
 // over the generic operator invoke path (operator-rpc.ts's resolveOperatorRpc
 // -> sdk.operator.invoke), reaching the SAME daemon the command layer does.
 //
 // A daemon that predates managed voice provisioning answers an honest 501 (verb
-// cataloged but no handler) or a 404 (older daemon without the route) — both
+// cataloged but no handler) or a 404 (older daemon without the route), both
 // are "verb unavailable", NOT a fabricated empty/failed install (see
 // classifyVoiceProvisionError). The interface is injectable so the command and
 // its tests round-trip against a mocked daemon.
@@ -33,7 +33,7 @@ export interface VoiceProvisionGateway {
 /**
  * Why the gateway could not be built (daemon disabled / no control-plane URL),
  * surfaced verbatim so the surface prints an honest "unavailable" line rather
- * than guessing — mirrors MemoryConsolidationGatewayResolution.
+ * than guessing, mirrors MemoryConsolidationGatewayResolution.
  */
 export type VoiceProvisionGatewayResolution =
   | { readonly available: true; readonly gateway: VoiceProvisionGateway }
@@ -46,7 +46,7 @@ export interface VoiceProvisionGatewayDeps {
 
 /**
  * Build the live voice-provisioning gateway over the generic operator invoke
- * path — the same daemon resolution the command layer uses. Returns an honest
+ * path, the same daemon resolution the command layer uses. Returns an honest
  * unavailable reason when no daemon is reachable, so the surface refuses
  * cleanly instead of throwing mid-render.
  */
@@ -70,7 +70,7 @@ export type VoiceProvisionFetchFailure =
 
 /**
  * Classify a voice verb rejection: a 501 (a daemon that predates managed voice
- * provisioning — cataloged but unhandled) or a 404 (an older daemon without the
+ * provisioning, cataloged but unhandled) or a 404 (an older daemon without the
  * route) are both "verb unavailable", distinct from a generic request failure
  * (network error, 401/403, 500). Reuses describeOperatorRpcError's wording so
  * the call sites never diverge on how they describe the same daemon response.
@@ -86,12 +86,12 @@ export function classifyVoiceProvisionError(error: unknown): VoiceProvisionFetch
 export const VOICE_SETUP_ANNOUNCEMENT = [
   'Local Voice Setup',
   '  provisioning the managed local voice runtime (piper TTS + a default voice; whisper STT where a bundle is published)…',
-  '  downloads are checksum-verified and resumable — re-run /voice setup to retry any failed component.',
+  '  downloads are checksum-verified and resumable; re-run /voice setup to retry any failed component.',
 ].join('\n');
 
 /**
  * Render the /voice status or /voice setup RESULT block from an already-resolved
- * gateway resolution — the testable core the command wraps. Kept separate from
+ * gateway resolution, the testable core the command wraps. Kept separate from
  * the live gateway construction so a wire test injects a fake resolution
  * (available / unavailable / failing) with no HTTP, mirroring the Memory modal's
  * injectable resolveConsolidationGateway seam. The caller prints
@@ -120,7 +120,7 @@ export async function renderVoiceProvision(
   // setup
   try {
     const receipt = await resolution.gateway.runInstall();
-    return ['Local Voice Setup — receipt', ...voiceInstallReceiptLines(receipt)].join('\n');
+    return ['Local Voice Setup: receipt', ...voiceInstallReceiptLines(receipt)].join('\n');
   } catch (error) {
     const failure = classifyVoiceProvisionError(error);
     return failure.kind === 'unavailable'
@@ -146,7 +146,7 @@ const realSleep = (ms: number): Promise<void> => new Promise((resolve) => setTim
  * `installInProgress` snapshot, printing a per-component line each time a
  * component's phase advances (download → verify → extract → done). When the
  * install settles, print the final receipt (or an honest failure). The install
- * verb streams nothing itself — this polling is exactly how the SDK intends
+ * verb streams nothing itself, this polling is exactly how the SDK intends
  * surfaces to render real progress instead of busy→receipt.
  *
  * The caller prints VOICE_SETUP_ANNOUNCEMENT before calling this. gateway
@@ -192,7 +192,7 @@ export async function runVoiceSetupWithProgress(
 
   const result = await installPromise;
   if (result.ok) {
-    opts.print(['Local Voice Setup — receipt', ...voiceInstallReceiptLines(result.receipt)].join('\n'));
+    opts.print(['Local Voice Setup: receipt', ...voiceInstallReceiptLines(result.receipt)].join('\n'));
   } else {
     const failure = classifyVoiceProvisionError(result.error);
     opts.print(failure.kind === 'unavailable'

package/src/core/voice-provision-status.ts

@@ -1,10 +1,10 @@
 // ---------------------------------------------------------------------------
-// voice-provision-status.ts — the surface-facing projection of the managed
+// voice-provision-status.ts, the surface-facing projection of the managed
 // local-voice runtime (voice.local.status / voice.local.install).
 //
 // The SDK owns the provisioning policy: a one-act install that fetches the
 // piper TTS engine + a default voice (fully hosted) and, where a pinned bundle
-// is hosted for the platform, the whisper STT engine + model — atomic,
+// is hosted for the platform, the whisper STT engine + model, atomic,
 // checksum-verified, resumable, and honest about every terminal state. This
 // module is the read-only projection the /voice surfaces render from: sizes
 // declared up front, a receipt of what was installed / skipped / failed, and
@@ -18,9 +18,9 @@
 
 import type { OperatorMethodOutput } from '@pellux/goodvibes-sdk';
 
-/** voice.local.status output — the no-network installed/available snapshot. */
+/** voice.local.status output, the no-network installed/available snapshot. */
 export type VoiceRuntimeStatusResult = OperatorMethodOutput<'voice.local.status'>;
-/** voice.local.install receipt — the one-act install outcome. */
+/** voice.local.install receipt, the one-act install outcome. */
 export type VoiceLocalInstallResult = OperatorMethodOutput<'voice.local.install'>;
 /** The live per-component progress carried on voice.local.status WHILE an install runs. */
 export type VoiceInstallProgress = NonNullable<VoiceRuntimeStatusResult['installInProgress']>;
@@ -43,7 +43,7 @@ export function formatVoiceBytes(bytes: number | null | undefined): string {
 
 /**
  * True when a whisper STT bundle is PINNED for this platform (bytes + checksum
- * fixed) but not yet hosted/sideloaded — the "supported but not yet published"
+ * fixed) but not yet hosted/sideloaded, the "supported but not yet published"
  * state. The SDK signals it as supported === true with an unprovisioned state
  * and an honest reason string; contrast an unsupported platform (supported ===
  * false), which has no pin at all.
@@ -60,7 +60,7 @@ export function voiceRuntimeNeedsSetup(status: VoiceRuntimeStatusResult): boolea
 }
 
 /**
- * The /voice status lines — an honest, plain-language account of the managed
+ * The /voice status lines, an honest, plain-language account of the managed
  * local-voice runtime with sizes declared up front. Order: platform + overall
  * state, then the TTS (piper) engine, then the STT (whisper) engine with its
  * honest unsupported / not-yet-published wording, then the one-act offer size.
@@ -70,7 +70,7 @@ export function voiceStatusLines(status: VoiceRuntimeStatusResult): string[] {
   lines.push(`  platform: ${status.platform ?? 'unrecognized (no managed voice build)'}`);
   lines.push(`  runtime: ${status.state}`);
 
-  // TTS — piper, fully hosted where a platform build exists.
+  // TTS, piper, fully hosted where a platform build exists.
   const tts = status.tts;
   if (tts.binaryPresent && tts.voicePresent) {
     lines.push(`  tts (${tts.engine}): installed`);
@@ -81,9 +81,9 @@ export function voiceStatusLines(status: VoiceRuntimeStatusResult): string[] {
     lines.push(`  tts (${tts.engine}): not installed${have.length ? ` (have ${have.join(' + ')})` : ''}`);
   }
 
-  // STT — whisper, hosted only where a pinned bundle is published. Render the
+  // STT, whisper, hosted only where a pinned bundle is published. Render the
   // REAL state honestly: a platform with no pinned build (supported === false,
-  // or state 'unsupported-platform' — every platform except linux-x64 today)
+  // or state 'unsupported-platform', every platform except linux-x64 today)
   // reads "unsupported on this platform"; a platform whose bundle is pinned but
   // not yet hosted (supported === true, unprovisioned, with a reason) reads
   // "not yet published for this platform".
@@ -102,7 +102,7 @@ export function voiceStatusLines(status: VoiceRuntimeStatusResult): string[] {
   }
 
   if (status.offerBytes !== null && status.offerBytes !== undefined && voiceRuntimeNeedsSetup(status)) {
-    lines.push(`  setup download: ${formatVoiceBytes(status.offerBytes)} — run /voice setup to install`);
+    lines.push(`  setup download: ${formatVoiceBytes(status.offerBytes)}; run /voice setup to install`);
   }
   return lines;
 }
@@ -127,7 +127,7 @@ export function voiceInstallComponentLine(component: VoiceInstallComponentProgre
     const done = component.bytesDone !== undefined ? `${formatVoiceBytes(component.bytesDone)}/` : '';
     size = ` (${done}${formatVoiceBytes(component.bytesTotal)})`;
   }
-  const msg = component.message ? ` — ${component.message}` : '';
+  const msg = component.message ? `: ${component.message}` : '';
   return `    ${component.component}: ${voiceProgressPhaseLabel(component.phase)}${size}${msg}`;
 }
 
@@ -142,22 +142,22 @@ export function voiceEngineOutcomeLine(engine: string, state: string, reason: st
     case 'provisioned':
       return `  ${engine}: installed`;
     case 'download-failed':
-      return `  ${engine}: download failed${reason ? ` — ${reason}` : ''} (re-run /voice setup to retry; completed parts are kept)`;
+      return `  ${engine}: download failed${reason ? `; ${reason}` : ''} (re-run /voice setup to retry; completed parts are kept)`;
     case 'checksum-mismatch':
-      return `  ${engine}: checksum mismatch — the downloaded file did not match its pinned checksum and was discarded${reason ? ` (${reason})` : ''}`;
+      return `  ${engine}: checksum mismatch; the downloaded file did not match its pinned checksum and was discarded${reason ? ` (${reason})` : ''}`;
     case 'bundle-unavailable':
-      return `  ${engine}: not yet published for this platform${reason ? ` — ${reason}` : ''}`;
+      return `  ${engine}: not yet published for this platform${reason ? `; ${reason}` : ''}`;
     case 'sideload-mismatch':
-      return `  ${engine}: a sideloaded bundle was present but did not match the pinned checksum${reason ? ` — ${reason}` : ''}`;
+      return `  ${engine}: a sideloaded bundle was present but did not match the pinned checksum${reason ? `; ${reason}` : ''}`;
     case 'unsupported-platform':
-      return `  ${engine}: unsupported on this platform${reason ? ` — ${reason}` : ''}`;
+      return `  ${engine}: unsupported on this platform${reason ? `; ${reason}` : ''}`;
     default:
-      return `  ${engine}: ${state}${reason ? ` — ${reason}` : ''}`;
+      return `  ${engine}: ${state}${reason ? `; ${reason}` : ''}`;
   }
 }
 
 /**
- * The /voice setup receipt lines — the final one-act outcome. Renders the
+ * The /voice setup receipt lines, the final one-act outcome. Renders the
  * per-component breakdown (installed / skipped-as-present / failed with size),
  * the TTS and STT terminal states with honest wording, and the config write
  * receipt (which voice.local.* keys were configured vs skipped as user-set).
@@ -174,7 +174,7 @@ export function voiceInstallReceiptLines(result: VoiceLocalInstallResult): strin
     lines.push('  components:');
     for (const component of result.components) {
       const size = component.bytes !== undefined ? ` (${formatVoiceBytes(component.bytes)})` : '';
-      const err = component.error ? ` — ${component.error}` : '';
+      const err = component.error ? `: ${component.error}` : '';
       const state = component.state === 'skipped' ? 'already present' : component.state;
       lines.push(`    ${component.id}: ${state}${size}${err}`);
     }

package/src/core/voice-stt-gateway.ts

@@ -1,16 +1,16 @@
 /**
- * voice-stt-gateway.ts — speech-to-text for captured audio.
+ * voice-stt-gateway.ts, speech-to-text for captured audio.
  *
  * Both voice consumers end here: push-to-talk hands over what the user just
  * said, and a confirmed wake hands over the utterance that followed it. The
- * transcription itself is the daemon's, not this process's — `voice.stt`
+ * transcription itself is the daemon's, not this process's, `voice.stt`
  * (POST /api/voice/stt) transcribes an audio artifact through whichever voice
  * provider is registered, including the managed local whisper that
  * `/voice setup` provisions.
  *
  * The verb has no named facade on the in-process OperatorClient, so it goes over
  * the generic operator invoke path exactly like the voice PROVISIONING verbs do
- * (see core/voice-provision-gateway.ts) — same daemon, same resolution, same
+ * (see core/voice-provision-gateway.ts), same daemon, same resolution, same
  * honest refusal reasons up front rather than a throw mid-capture.
  */
 
@@ -20,7 +20,7 @@ import type { OperatorMethodOutput } from '@pellux/goodvibes-sdk';
 import type { UtteranceAudioArtifact } from '@pellux/goodvibes-sdk/platform/voice';
 import { resolveOperatorRpc, describeOperatorRpcError } from '../input/commands/operator-rpc.ts';
 
-/** voice.stt output — the recognised text plus the provider that produced it. */
+/** voice.stt output, the recognised text plus the provider that produced it. */
 export type VoiceTranscriptionResult = OperatorMethodOutput<'voice.stt'>;
 
 /** The narrow verb surface a capture consumer needs. */
@@ -64,12 +64,12 @@ export function createVoiceSttGateway(deps: VoiceSttGatewayDeps): VoiceSttGatewa
 /**
  * Render a `voice.stt` rejection honestly. A 501 (the verb is cataloged but this
  * daemon has no voice provider wired behind it) and a 404 (an older daemon
- * without the route) are both "no transcription here" — distinct from a request
+ * without the route) are both "no transcription here", distinct from a request
  * that reached a provider and failed, which the user can act on differently.
  */
 export function describeTranscriptionFailure(error: unknown): string {
   if (error instanceof GoodVibesSdkError && (error.status === 501 || error.status === 404)) {
-    return 'this daemon has no speech-to-text provider wired up — run /voice setup to provision the managed local runtime.';
+    return 'this daemon has no speech-to-text provider wired up; run /voice setup to provision the managed local runtime.';
   }
   return describeOperatorRpcError(error);
 }

package/src/core/wake-provision-runner.ts

@@ -1,5 +1,5 @@
 // ---------------------------------------------------------------------------
-// wake-provision-runner.ts — what `/voice wake status` and `/voice wake setup` do.
+// wake-provision-runner.ts, what `/voice wake status` and `/voice wake setup` do.
 //
 // `/voice wake setup` IS NO LONGER HOW THE MODEL ARRIVES. It used to be the only
 // way, and the result was that installing goodvibes produced a wake-word feature
@@ -11,17 +11,17 @@
 // So what this command is FOR, on a normal machine, is recovery and
 // re-provisioning: an install that was offline, an artifact that stopped
 // verifying, or a re-fetch after the pinned model changes. It still works
-// exactly as it did, which is the point — the degraded path an offline install
+// exactly as it did, which is the point, the degraded path an offline install
 // falls back to is this one.
 //
 // What has NOT changed is the runtime rule. Nothing in this path downloads on its
 // own: `/voice wake status` only reads what is on disk (verifying by content,
 // never by existence), and a download happens only because the user typed
-// `setup`. Turning `voice.wake.enabled` on never fetches anything either — see
+// `setup`. Turning `voice.wake.enabled` on never fetches anything either, see
 // audio/wake-runtime.ts, which reports what is missing and names this command.
 //
-// Both SDK calls are injectable so a wire test drives the whole flow — a fresh
-// host, a corrupt artifact, a failed component — with no network and no files.
+// Both SDK calls are injectable so a wire test drives the whole flow, a fresh
+// host, a corrupt artifact, a failed component, with no network and no files.
 // ---------------------------------------------------------------------------
 
 import {
@@ -74,12 +74,12 @@ export async function runWakeProvision(deps: WakeProvisionRunnerDeps): Promise<v
   const onProgress = (progress: WakeProvisionProgress): void => {
     if (lastPhase.get(progress.component) === progress.phase) return;
     lastPhase.set(progress.component, progress.phase);
-    const extra = progress.message !== undefined ? ` — ${progress.message}` : '';
+    const extra = progress.message !== undefined ? `: ${progress.message}` : '';
     deps.print(`  ${progress.component}: ${progress.phase}${extra}`);
   };
   try {
     const result = await provision(deps.managedRoot, onProgress);
-    deps.print(['Wake-Word Setup — receipt', ...wakeProvisionReceiptLines(result)].join('\n'));
+    deps.print(['Wake-Word Setup: receipt', ...wakeProvisionReceiptLines(result)].join('\n'));
   } catch (error) {
     deps.print(`Wake-Word Setup\n  provisioning failed: ${error instanceof Error ? error.message : String(error)}`);
   }

package/src/core/wake-provision-status.ts

@@ -1,16 +1,16 @@
 // ---------------------------------------------------------------------------
-// wake-provision-status.ts — the surface-facing projection of wake-word
+// wake-provision-status.ts, the surface-facing projection of wake-word
 // provisioning and of the rows that are or are not in force.
 //
 // The wake models are checksum-pinned and arrive WITH THE INSTALLATION: the
 // installer and the npm postinstall provision them, and a daemon retries at boot
 // whatever the install could not fetch. `/voice wake setup` is the recovery act
-// for the case where that did not land, not the normal way in — so the lines
+// for the case where that did not land, not the normal way in, so the lines
 // below name it where it helps and do not imply it is a required step.
 //
 // This module is the read-only projection the /voice wake surfaces render from:
 // per-artifact present / verified / corrupt / bytes taken straight from the SDK's
-// wakeProvisionStatus, which verifies by CONTENT rather than by existence — a
+// wakeProvisionStatus, which verifies by CONTENT rather than by existence, a
 // truncated or wrong-asset file reports corrupt instead of present, because a
 // detector loading it would fail in a way the user could not diagnose.
 //
@@ -31,15 +31,15 @@ import { formatVoiceBytes } from './voice-provision-status.ts';
 
 /**
  * What a terminal can actually do, so a `voice.wake.*` row is refused or reported
- * rather than faked. Lives here — beside the status projection and away from the
- * inference runtime — so the command layer can resolve settings without pulling
+ * rather than faked. Lives here, beside the status projection and away from the
+ * inference runtime, so the command layer can resolve settings without pulling
  * onnxruntime into a `/voice wake status` call.
  */
 export function terminalWakeCapabilities(status?: Pick<WakeProvisionStatus, 'vadReady'>): WakeSurfaceCapabilities {
   return {
     // Asked of the SDK rather than declared here. The filter is a WebAssembly
     // module carried in the package, so the only question is whether this runtime
-    // has WebAssembly at all — which the SDK answers, with a reason a settings
+    // has WebAssembly at all, which the SDK answers, with a reason a settings
     // surface can show. A host constant would go stale the moment the stage
     // shipped, which is exactly what happened to the previous one.
     speexAvailable: noiseSuppressionSupport().supported,
@@ -58,14 +58,14 @@ export function terminalWakeCapabilities(status?: Pick<WakeProvisionStatus, 'vad
 export function wakeArtifactLine(label: string, artifact: WakeArtifactStatus): string {
   if (artifact.verified) return `  ${label}: verified (${formatVoiceBytes(artifact.bytes)})`;
   if (artifact.corrupt) {
-    return `  ${label}: PRESENT BUT FAILS VERIFICATION (${formatVoiceBytes(artifact.bytes)}) — torn, truncated, or the wrong asset; /voice wake setup replaces it`;
+    return `  ${label}: PRESENT BUT FAILS VERIFICATION (${formatVoiceBytes(artifact.bytes)}); torn, truncated, or the wrong asset; /voice wake setup replaces it`;
   }
   return `  ${label}: missing`;
 }
 
 /**
  * Blockers, in the SDK's own words. A blocker means the detector must NOT start,
- * so the row's key and the written reason are both shown — a swallowed blocker is
+ * so the row's key and the written reason are both shown, a swallowed blocker is
  * a user staring at a feature that is on and doing nothing.
  */
 export function describeWakeBlockers(settings: Pick<WakeRuntimeSettings, 'blockers'>): string[] {
@@ -102,14 +102,14 @@ export function wakeStatusLines(
     // someone turns it on, so a missing gate is not a broken detector. It is
     // printed rather than left out because vadThreshold above 0 refuses to start
     // when the gate is absent, and a reader needs to see why.
-    `  speech gate provisioned: ${status.vadReady ? 'yes' : 'no — voice.wake.vadThreshold above 0 will refuse to start'}`,
+    `  speech gate provisioned: ${status.vadReady ? 'yes' : 'no; voice.wake.vadThreshold above 0 will refuse to start'}`,
     wakeArtifactLine('speech gate', status.vad),
     wakeArtifactLine('attribution NOTICE (speech gate)', status.vadNotice),
   ];
   if (!status.ready) {
     lines.push(
       `  a fresh provision would download ${formatVoiceBytes(status.downloadBytes)}. Installing goodvibes normally does this,`,
-      '  and a running daemon retries at boot — run /voice wake setup to fetch it now (nothing downloads on its own).',
+      '  and a running daemon retries at boot; run /voice wake setup to fetch it now (nothing downloads on its own).',
     );
   }
   lines.push(
@@ -120,7 +120,7 @@ export function wakeStatusLines(
     `  retained audio: voice.wake.retainAudio=${settings.retainAudio}`,
   );
   if (status.recallIsSyntheticOnly) {
-    lines.push('  the published recall figures for this model are measured on synthesised speech only — no human recording of the phrase exists behind them.');
+    lines.push('  the published recall figures for this model are measured on synthesised speech only; no human recording of the phrase exists behind them.');
   }
   const blockers = describeWakeBlockers(settings);
   if (blockers.length > 0) lines.push('  rows blocking startup:', ...blockers);
@@ -136,15 +136,15 @@ export function wakeProvisionReceiptLines(result: WakeProvisionResult): string[]
     `  model version: ${result.modelVersion ?? 'unpinned'}`,
     // Separate from `ready` on purpose: this terminal loads the onnx build, so a
     // missing tflite twin is not a detector that cannot run.
-    `  tflite form (for runtimes that cannot load onnx): ${result.mobileFormatReady ? 'installed' : 'not installed — nothing on this terminal needs it'}`,
+    `  tflite form (for runtimes that cannot load onnx): ${result.mobileFormatReady ? 'installed' : 'not installed; nothing on this terminal needs it'}`,
     // Separate from `ready` for its own reason: voice.wake.vadThreshold ships at
-    // 0, so the detector runs without the gate — but a value above 0 refuses to
+    // 0, so the detector runs without the gate, but a value above 0 refuses to
     // start without it, which is what this line lets a reader act on.
-    `  speech gate (voice.wake.vadThreshold): ${result.vadReady ? 'installed' : 'not installed — voice.wake.vadThreshold above 0 will refuse to start'}`,
+    `  speech gate (voice.wake.vadThreshold): ${result.vadReady ? 'installed' : 'not installed; voice.wake.vadThreshold above 0 will refuse to start'}`,
   ];
   for (const outcome of result.outcomes) {
     const detail = outcome.state === 'failed'
-      ? ` — ${outcome.error ?? 'no reason reported'}`
+      ? `: ${outcome.error ?? 'no reason reported'}`
       : outcome.bytes !== undefined ? ` (${formatVoiceBytes(outcome.bytes)})` : '';
     lines.push(`  ${outcome.component}: ${outcome.state}${detail}`);
     lines.push(`    ${outcome.path}`);
@@ -156,7 +156,7 @@ export function wakeProvisionReceiptLines(result: WakeProvisionResult): string[]
     lines.push(`  attribution NOTICE (travels with the front end): ${result.embeddingNoticePath}`);
   }
   if (result.recallIsSyntheticOnly) {
-    lines.push('  the published recall figures for this model are measured on synthesised speech only — no human recording of the phrase exists behind them.');
+    lines.push('  the published recall figures for this model are measured on synthesised speech only; no human recording of the phrase exists behind them.');
   }
   return lines;
 }
@@ -165,6 +165,6 @@ export function wakeProvisionReceiptLines(result: WakeProvisionResult): string[]
 export const WAKE_SETUP_ANNOUNCEMENT = [
   'Wake-Word Setup',
   '  downloading the pinned "hey goodvibes" classifier (both runtime formats), the shared speech-embedding front end, and the speech gate voice.wake.vadThreshold runs…',
-  '  every artifact is checksum-verified and the download is resumable — one that already matches is skipped, so re-running this only fetches what is missing.',
+  '  every artifact is checksum-verified and the download is resumable; one that already matches is skipped, so re-running this only fetches what is missing.',
   '  installing goodvibes normally does this for you; running it by hand is how you recover an install that could not reach the network.',
 ].join('\n');

package/src/core/workstream-notification.ts

@@ -1,16 +1,16 @@
 /**
- * workstream-notification.ts — the desktop notification a finished workstream
+ * workstream-notification.ts, the desktop notification a finished workstream
  * pushes, in words rather than identifiers.
  *
  * These three notifications used to read:
  *
- *     GoodVibes — WRFC chain failed
+ *     GoodVibes, WRFC chain failed
  *     chain 7f3a91c02b4e failed: review rejected
  *
  * A desktop notification is a message TO a person, so the standing rule applies
  * to it the same way it applies to a chat message: no internal name for the
  * machinery, and no register id. `WRFC` is the first; `7f3a91c02b4e` is the
- * second, and it is not something the reader can do anything with — it is not
+ * second, and it is not something the reader can do anything with, it is not
  * the commit, not the branch, not the session.
  *
  * What the reader can act on is why the work stopped, so that is what the body
@@ -19,7 +19,7 @@
  * plain words, never by an opaque identifier.
  *
  * Split out of turn-event-wiring.ts as a pure function so the text is testable
- * on its own — the wiring itself cannot be asserted against without mocking a
+ * on its own, the wiring itself cannot be asserted against without mocking a
  * process-global notifier.
  */
 import { formatTurnBudgetOutcome } from './turn-budget-outcome.ts';
@@ -40,7 +40,7 @@ export interface WorkstreamNotification {
 /**
  * Title and body for a workstream that reached a terminal state.
  *
- * An operator cancellation is an intended stop, not a failure — it is narrated
+ * An operator cancellation is an intended stop, not a failure, it is narrated
  * as cancelled (the reason already carries the landed-work count from the
  * workstream's edit ledger) so the notification never contradicts the cancelled
  * workstream/owner/cohort surfaces. A turn-budget exhaustion is a spent ceiling
@@ -52,18 +52,18 @@ export function workstreamFailureNotification(
 ): WorkstreamNotification {
   if (payload.failureKind === 'cancelled') {
     return {
-      title: 'GoodVibes — workstream cancelled',
+      title: 'GoodVibes: workstream cancelled',
       body: `Cancelled: ${payload.reason}`,
     };
   }
   if (payload.failureKind === 'max_turns') {
     return {
-      title: 'GoodVibes — workstream hit its turn budget',
+      title: 'GoodVibes: workstream hit its turn budget',
       body: `The workstream ${formatTurnBudgetOutcome({ limit: payload.turnLimit, source: payload.turnLimitSource })}`,
     };
   }
   return {
-    title: 'GoodVibes — workstream failed',
+    title: 'GoodVibes: workstream failed',
     body: `Failed: ${payload.failureKind === 'transport' ? 'transient transport error' : payload.reason}`,
   };
 }