@pellux/goodvibes-tui 2.0.13 → 2.0.15

package/src/core/conversation-user-receipts.ts

@@ -1,16 +1,16 @@
 /**
- * conversation-user-receipts.ts — tracks which system-role message indices
+ * conversation-user-receipts.ts, tracks which system-role message indices
  * are a direct receipt for an explicit user action (e.g. answering the
  * startup recovery modal's Resume/Keep/Remove choice), as opposed to ambient
  * boot chatter (provider registration, the resume notice, etc.). Kept in its
- * own file so ConversationManager (core/conversation.ts) — already near its
- * line-count gate — doesn't have to carry the bookkeeping inline.
+ * own file so ConversationManager (core/conversation.ts), already near its
+ * line-count gate, doesn't have to carry the bookkeeping inline.
  *
  * ConversationManager.rebuildHistory()'s splash-vs-transcript check consults
  * this via ConversationManager.addUserActionReceipt(): a receipt the user is
  * actively waiting on must displace the splash the way a user message does,
  * while quiet startup plumbing keeps rendering underneath it. Indices mirror
- * messageKindRegistry's own lifecycle — cleared on resetAll(), purged for
+ * messageKindRegistry's own lifecycle, cleared on resetAll(), purged for
  * indices an undo() frees for reuse, deleted when a later add overwrites the
  * same recycled index with a non-receipt message.
  */
@@ -33,7 +33,7 @@ export class UserReceiptIndices {
     this.indices.clear();
   }
 
-  /** Drop every tracked index >= minIndex — mirrors undo()'s tail-splice. */
+  /** Drop every tracked index >= minIndex, mirrors undo()'s tail-splice. */
   purgeFrom(minIndex: number): void {
     for (const key of this.indices) {
       if (key >= minIndex) this.indices.delete(key);

package/src/core/conversation.ts

@@ -65,11 +65,11 @@ export type { OrchestratorUsageTotals };
 /**
  * sumConversationUsage - Fold every assistant message's per-turn usage into
  * running totals, plus the *last* assistant message's own input-token figure
- * (context-window occupancy — not a running sum).
+ * (context-window occupancy, not a running sum).
  *
  * TUI-side counterpart to SDK Orchestrator.usage: after a session resume
  * replays historical messages into a freshly-constructed Orchestrator (whose
- * `usage` starts at all zeros — bootstrap.ts always constructs a fresh
+ * `usage` starts at all zeros, bootstrap.ts always constructs a fresh
  * instance, see), this lets the caller hydrate the footer's token
  * counters from messages that already carry real usage data instead of
  * waiting for the first new turn to populate them.
@@ -118,9 +118,9 @@ export class ConversationManager extends SdkConversationManager {
   protected blockRegistry: BlockMeta[] = [];
   /**
    * Absolute message index -> first rendered line index in the history buffer.
-   * Keyed by the message's ABSOLUTE position in the full (unsliced) snapshot —
+   * Keyed by the message's ABSOLUTE position in the full (unsliced) snapshot,
    * appendConversationMessages writes `messageLineRegistry[absoluteIdx]`, not the
-   * slice-relative loop counter — so transcript-event navigation
+   * slice-relative loop counter, so transcript-event navigation
    * (next/prevTranscriptEventLine), whose event.messageIndex is absolute, resolves
    * correctly even after clearDisplay() renders only a tail slice. Holes (indices
    * not currently rendered) read back as undefined and are filtered out by consumers.
@@ -131,10 +131,10 @@ export class ConversationManager extends SdkConversationManager {
    * navigable (error-navigation worthy).
    *
    * Kind → navigable mapping:
-   *   - 'system'      YES — generic/catch-all messages (failures, provider errors,
+   *   - 'system'      YES, generic/catch-all messages (failures, provider errors,
    *                         session events); the default for any un-prefixed message
-   *   - 'wrfc'        YES — WRFC chain events (failures matter for navigation)
-   *   - 'operational' NO  — noisy tool/scan/plugin/MCP status; not worth jumping to
+   *   - 'wrfc'        YES, WRFC chain events (failures matter for navigation)
+   *   - 'operational' NO , noisy tool/scan/plugin/MCP status; not worth jumping to
    *
    * This replaces the old /error/i substring test, which missed failure phrases
    * like "request failed" / "rate limited" and false-positived on benign info
@@ -237,7 +237,7 @@ export class ConversationManager extends SdkConversationManager {
   /**
    * addTypedSystemMessage - System message with an explicit kind tag, stored
    * in messageKindRegistry. `isUserReceipt` (conversation-user-receipts.ts)
-   * additionally marks it for rebuildHistory()'s splash check — only for a
+   * additionally marks it for rebuildHistory()'s splash check, only for a
    * direct receipt to an explicit user action, never ambient boot chatter.
    */
   public addTypedSystemMessage(content: string, kind: SystemMessageKind, opts?: { isUserReceipt?: boolean }): void {
@@ -303,7 +303,7 @@ export class ConversationManager extends SdkConversationManager {
 
   /**
    * updateStreamingBlock - Update the in-progress streaming block with accumulated content.
-   * Called per-delta during streaming. Does NOT trigger a full rebuild — instead it
+   * Called per-delta during streaming. Does NOT trigger a full rebuild, instead it
    * directly updates the history buffer from streamingStartLine onward.
    */
   public override updateStreamingBlock(content: string): void {
@@ -445,7 +445,7 @@ export class ConversationManager extends SdkConversationManager {
   /**
    * getLineCacheSize - Number of per-message Line[] entries currently retained.
    * The cache mark-and-sweeps to the visible message set on every rebuild, so
-   * this is bounded by what is on screen — exposed for memory-hygiene assertions.
+   * this is bounded by what is on screen, exposed for memory-hygiene assertions.
    */
   public getLineCacheSize(): number {
     return this.lineCache.size;
@@ -471,7 +471,7 @@ export class ConversationManager extends SdkConversationManager {
     // During streaming, the in-progress placeholder (always the last message) is
     // rendered here as EMPTY; the incremental streaming path (updateStreamingBlock)
     // owns its content. This keeps streamingStartLine valid across width-change
-    // rebuilds — otherwise the placeholder would be re-rendered with its content at
+    // rebuilds, otherwise the placeholder would be re-rendered with its content at
     // the new width while streamingStartLine still pointed into the old buffer.
     const lastMsg = snapshot[snapshot.length - 1];
     const isStreaming = this.streamingStartLine >= 0 && lastMsg?.role === 'assistant';
@@ -480,7 +480,7 @@ export class ConversationManager extends SdkConversationManager {
       : snapshot;
 
     // When _displayFromMessageIndex > 0, clearDisplay() was called. Only render
-    // messages added after the clear — the pre-clear history stays off-screen.
+    // messages added after the clear, the pre-clear history stays off-screen.
     const displayStart = this._displayFromMessageIndex;
     const visibleSnapshot = displayStart > 0 ? renderSnapshot.slice(displayStart) : renderSnapshot;
 
@@ -657,14 +657,14 @@ export class ConversationManager extends SdkConversationManager {
 
   /** First rendered line for message `absoluteIdx` (undefined if never
    *  rendered). For a folded tool-group member this is the group's own
-   *  header line, not the following message's position — see
+   *  header line, not the following message's position, see
    *  messageLineRegistry's doc. Flushes history if dirty. */
   public getMessageLine(absoluteIdx: number): number | undefined {
     this.flushHistory();
     return this.messageLineRegistry[absoluteIdx];
   }
 
-  /** Set a collapseKey's state directly, bypassing block lookup — needed for
+  /** Set a collapseKey's state directly, bypassing block lookup, needed for
    *  a key with no BlockMeta yet (a folded tool-group member's own
    *  `msg_<idx>` key only becomes a real block once its group expands). */
   public setCollapsed(collapseKey: string, collapsed: boolean): void {
@@ -726,7 +726,7 @@ export class ConversationManager extends SdkConversationManager {
     this.markDirty();
   }
 
-  /** Retire the splash for the run — any submission does it (SplashGateState). */
+  /** Retire the splash for the run, any submission does it (SplashGateState). */
   public dismissSplash(): void {
     if (this.splashGate.dismiss()) this.markDirty();
   }
@@ -772,7 +772,7 @@ export class ConversationManager extends SdkConversationManager {
    *
    * Contract:
    * - getDisplayBlocks() returns an empty array immediately after this call.
-   * - getMessageSnapshot() is unaffected — full LLM history is preserved.
+   * - getMessageSnapshot() is unaffected, full LLM history is preserved.
    * - resetAll() (which clears both display and messages) continues to work.
    * - rebuildHistory() can be called by callers that need a full display rebuild.
    */
@@ -792,7 +792,7 @@ export class ConversationManager extends SdkConversationManager {
     this._displayFromMessageIndex = this.getMessageSnapshot().length;
     this.appendedUpTo = this._displayFromMessageIndex;
     this.dirty = false;
-    // Do NOT re-render here — display stays blank until the next message is added.
+    // Do NOT re-render here, display stays blank until the next message is added.
     // The lastRenderedWidth is kept so subsequent appends use the correct width.
   }
 }

package/src/core/format-user-error.ts

@@ -1,5 +1,5 @@
 /**
- * formatUserFacingError — classifies provider/network errors into plain-language
+ * formatUserFacingError, classifies provider/network errors into plain-language
  * one-liners with a suggested action.
  *
  * Classification order (most-specific first):
@@ -146,28 +146,28 @@ export function formatUserFacingError(err: unknown): UserFacingError {
     case 'auth':
       return {
         kind,
-        message: 'Authentication failed — the provider rejected your API key.',
+        message: 'Authentication failed: the provider rejected your API key.',
         action: 'Run /login to re-authenticate or check your API key.',
       };
 
     case 'rate-limit':
       return {
         kind,
-        message: 'Rate limit reached — the provider is throttling requests.',
+        message: 'Rate limit reached: the provider is throttling requests.',
         action: 'Wait a moment and retry, or switch models with /model.',
       };
 
     case 'context-overflow':
       return {
         kind,
-        message: 'Context window exceeded — the conversation is too long for this model.',
+        message: 'Context window exceeded: the conversation is too long for this model.',
         action: 'Run /compact to summarise the conversation and free context.',
       };
 
     case 'network':
       return {
         kind,
-        message: 'Network error — could not reach the provider.',
+        message: 'Network error: could not reach the provider.',
         action: 'Check your connection and retry, or switch models with /model.',
       };
 

package/src/core/long-task-notifier.ts

@@ -1,26 +1,26 @@
 /**
- * long-task-notifier — fires push notifications when a turn or agent task
+ * long-task-notifier, fires push notifications when a turn or agent task
  * completes after running longer than the configured threshold.
  *
  * PRIVACY GUARANTEE: Notification text must never include conversation content
  * (user messages, assistant replies, tool outputs). Only metadata is included:
  * task kind, elapsed time, ok/fail status, and session id. This module enforces
- * that constraint by construction — it receives no conversation object and
+ * that constraint by construction, it receives no conversation object and
  * builds all message text from structural metadata only.
  *
  * Delivery targets (in preference order):
  *   1. Desktop notification (linux notify-send / mac osascript) via SDK
- *      notifyCompletion — detected and dispatched by the SDK; silently
+ *      notifyCompletion, detected and dispatched by the SDK; silently
  *      no-ops when the platform does not support it.
  *   2. Configured outbound webhook channel (ntfy topic / webhook URL) via
- *      WebhookNotifier.send() — only fires when the user has URLs configured.
+ *      WebhookNotifier.send(), only fires when the user has URLs configured.
  *
  * When neither target is available the function is an honest no-op (debug log
  * only; no user-facing error spam).
  *
  * Focus tracking: when `focusTracker` is supplied, notifications are
  * gated the same way as the other unfocused-user alert classes (see
- * alert-gating.ts) — they fire when the terminal is unfocused or focus state
+ * alert-gating.ts), they fire when the terminal is unfocused or focus state
  * was never observed, and are suppressed when it's known to be focused,
  * unless `behavior.notifyOnlyWhenUnfocused` is turned off. `focusTracker` is
  * optional and defaults to "always fire" (the behavior before focus gating existed) when
@@ -76,7 +76,7 @@ export interface MaybeNotifyLongTaskOptions {
   /**
    * Outbound webhook notifier. When provided and the user has URLs
    * configured, the notification is also sent to all configured endpoints
-   * (e.g. ntfy.sh topics). Optional — absent means outbound delivery is
+   * (e.g. ntfy.sh topics). Optional, absent means outbound delivery is
    * skipped silently.
    */
   readonly webhookNotifier?: WebhookNotifier | null;
@@ -99,7 +99,7 @@ export interface MaybeNotifyLongTaskOptions {
  * exceeds the configured threshold.
  *
  * Returns true when at least one delivery was attempted, false when the
- * call was a no-op (threshold not reached, off-state, or focus-gated off —
+ * call was a no-op (threshold not reached, off-state, or focus-gated off,
  * see `focusTracker`/`configGet` above).
  *
  * PRIVACY: builds message text from structural metadata only (kind, elapsed,
@@ -125,13 +125,13 @@ export function maybeNotifyLongTask(opts: MaybeNotifyLongTaskOptions): boolean {
   // are supplied. Absent either one, behavior is unchanged from before focus
   // gating existed (always fire once the threshold is met).
   if (focusTracker && configGet && readNotifyOnlyWhenUnfocused(configGet) && !focusTracker.shouldAlertWhenUnfocused()) {
-    logger.debug('long-task-notifier: suppressed — terminal focused');
+    logger.debug('long-task-notifier: suppressed; terminal focused');
     return false;
   }
 
   // Build concise, metadata-only message. No conversation text.
   const statusLabel = status === 'ok' ? 'completed' : 'failed';
-  const title = `GoodVibes — ${kind} ${statusLabel}`;
+  const title = `GoodVibes: ${kind} ${statusLabel}`;
   // PRIVACY: message contains only structural metadata, never conversation content.
   const message = `${kind} ${statusLabel} in ${elapsedSeconds}s  ·  session ${sessionId.slice(0, 8)}`;
 

package/src/core/memory-diagnostics-gateway.ts

@@ -2,15 +2,15 @@
 // memory-diagnostics-gateway.ts
 //
 // The daemon-backed verb the /health memory (doctor) surface reads:
-// ops.memory.get (GET /api/ops/memory) — the read-only MemoryGovernor snapshot.
-// No named facade exists on the in-process OperatorClient, so — exactly like
-// the memory-consolidation and voice-provisioning gateways — it goes over the
+// ops.memory.get (GET /api/ops/memory), the read-only MemoryGovernor snapshot.
+// No named facade exists on the in-process OperatorClient, so, exactly like
+// the memory-consolidation and voice-provisioning gateways, it goes 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 the memory governor answers an honest 501 (the verb is
 // cataloged but has no handler when no live governor is composed) or a 404 (an
-// older daemon without the route) — both mean "this daemon does not serve
+// older daemon without the route), both mean "this daemon does not serve
 // memory diagnostics", NOT a fabricated all-zero snapshot (see
 // classifyMemoryDiagnosticsError). The interface is injectable so the command
 // and its tests round-trip against a mocked daemon.
@@ -30,7 +30,7 @@ export interface MemoryDiagnosticsGateway {
 /**
  * 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 the memory-consolidation / voice gateways.
+ * than guessing, mirrors the memory-consolidation / voice gateways.
  */
 export type MemoryDiagnosticsGatewayResolution =
   | { readonly available: true; readonly gateway: MemoryDiagnosticsGateway }
@@ -43,7 +43,7 @@ export interface MemoryDiagnosticsGatewayDeps {
 
 /**
  * Build the live memory-diagnostics 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.
  */
 export function createMemoryDiagnosticsGateway(deps: MemoryDiagnosticsGatewayDeps): MemoryDiagnosticsGatewayResolution {
@@ -66,7 +66,7 @@ export const MEMORY_DIAGNOSTICS_UNAVAILABLE = 'this daemon does not serve memory
 
 /**
  * Classify an ops.memory.get rejection: a 501 (a daemon composed without a live
- * governor — the verb is cataloged but unhandled) or a 404 (an older daemon
+ * governor, the verb is cataloged but unhandled) or a 404 (an older daemon
  * without the route) are both "verb unavailable"; anything else (network,
  * 401/403, 500) is a generic error.
  */
@@ -79,7 +79,7 @@ export function classifyMemoryDiagnosticsError(error: unknown): MemoryDiagnostic
 
 /**
  * Render the /health memory (doctor) block from an already-resolved gateway
- * resolution — the testable core the command wraps. Kept separate from the live
+ * 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.
  */

package/src/core/memory-provenance.ts

@@ -1,9 +1,9 @@
 // ---------------------------------------------------------------------------
-// memory-provenance.ts — the optional "used N memories" turn chip (owner-ruled
+// memory-provenance.ts, the optional "used N memories" turn chip (owner-ruled
 // 2026-07-13, default OFF).
 //
 // The SDK tags each turn payload with the memory-sourced injection ids it drew
-// on: `metadata.memory.recordIds: string[]` (absent when none) — the SAME
+// on: `metadata.memory.recordIds: string[]` (absent when none), the SAME
 // convention the webui's chip reads, so both surfaces name the same records.
 // When the memory-provenance setting is ON, a turn that used memories shows a
 // small chip naming how many, with a drill-in listing them. When OFF (default),
@@ -22,7 +22,7 @@ export const MEMORY_SHOW_PROVENANCE_DESCRIPTION =
 /**
  * Read the memory-provenance toggle defensively. `memory.*` is a TUI-local
  * section with no SDK DEFAULT_CONFIG entry, so ConfigManager.get() throws for
- * it until something has been written — an unset key means the default (OFF),
+ * it until something has been written, an unset key means the default (OFF),
  * not an error (same pattern as memory.projection.dir).
  */
 export function readMemoryShowProvenance(configManager: Pick<ConfigManager, 'get'>): boolean {
@@ -37,7 +37,7 @@ export function readMemoryShowProvenance(configManager: Pick<ConfigManager, 'get
  * Extract the memory-sourced record ids a turn drew on, from the SDK's
  * `metadata.memory.recordIds` convention. Structural (not a named import) so it
  * reads the field whether or not the pinned SDK's TurnEvent type surfaces it
- * yet, and yields an empty list for any turn without the field — never a guess,
+ * yet, and yields an empty list for any turn without the field, never a guess,
  * never a throw.
  */
 export function memoryRecordIdsFromTurn(turn: unknown): readonly string[] {
@@ -50,7 +50,7 @@ export function memoryRecordIdsFromTurn(turn: unknown): readonly string[] {
 
 /**
  * Build the settings-modal synthetic entry for memory.showProvenance (TUI-local,
- * default OFF). Read defensively — memory.* has no SDK DEFAULT_CONFIG entry, so
+ * default OFF). Read defensively, memory.* has no SDK DEFAULT_CONFIG entry, so
  * an unset key means the OFF default, not an error.
  */
 export function buildMemoryProvenanceSyntheticEntry(configManager: Pick<ConfigManager, 'get'>): {

package/src/core/model-routing-chip.ts

@@ -1,9 +1,9 @@
 /**
- * model-routing-chip — turns a mid-session model change into a visible one-line
+ * model-routing-chip, turns a mid-session model change into a visible one-line
  * conversation notice so a route/fallback/downgrade is NEVER silent.
  *
  * Driven by the provider registry's `MODEL_CHANGED` runtime event (fired on
- * every setCurrentModel — user command, automatic capability route, or
+ * every setCurrentModel, user command, automatic capability route, or
  * SDK-internal downgrade). The chip's job is the OTHERWISE-SILENT changes: the
  * SDK-internal auto-route/downgrade paths call setCurrentModel with no TUI-side
  * narration of their own.
@@ -83,7 +83,7 @@ export function findRecentFallbackReason(
 
 /**
  * The routing-chip line to emit for a MODEL_CHANGED event, or null to stay
- * silent. Null only when there is no real change (no/same previous model) —
+ * silent. Null only when there is no real change (no/same previous model),
  * changes the failover path narrates itself never reach this function.
  *
  * A change that correlates to a logged fallback transition is reported WITH

package/src/core/pairing-tailscale-gateway.ts

@@ -1,8 +1,8 @@
 // ---------------------------------------------------------------------------
 // pairing-tailscale-gateway.ts
 //
-// The two tailscale verbs the pairing surfaces drive — a read-only detection
-// (tailscale.get) and the one-action serve (tailscale.serve.run) — over the
+// The two tailscale verbs the pairing surfaces drive, a read-only detection
+// (tailscale.get) and the one-action serve (tailscale.serve.run), over the
 // same generic operator invoke path the fleet acts use (resolveOperatorRpc ->
 // sdk.operator.invoke), reaching the SAME daemon the command layer does.
 //
@@ -25,7 +25,7 @@ export interface PairingTailscaleGatewayDeps {
 
 /**
  * Probe tailscale (tailscale.get). Returns the honest status, or null when the
- * daemon is unreachable / the call fails — the pairing surface then stays quiet.
+ * daemon is unreachable / the call fails, the pairing surface then stays quiet.
  */
 export async function probePairingTailscale(
   deps: PairingTailscaleGatewayDeps,

package/src/core/permission-mode.ts

@@ -1,20 +1,20 @@
 /**
- * Session permission mode — the small vocabulary and cycle logic shared by the
+ * Session permission mode, the small vocabulary and cycle logic shared by the
  * Shift+Tab mode-cycle key, the `/plan` toggle, and the footer mode pill.
  *
  * The SDK owns the permission mode: it lives at config key `permissions.mode`
  * and is read by the SDK's PermissionManager.getMode() (which the orchestrator
  * consults for the standing plan-mode instruction). Setting the config value is
- * the SDK-sanctioned way to change the mode — the SDK's mode-change emitter then
+ * the SDK-sanctioned way to change the mode, the SDK's mode-change emitter then
  * broadcasts PERMISSION_MODE_CHANGED on the runtime bus so every attached
  * surface can reflect it. This module only maps and cycles the values; it never
  * owns the state.
  *
  * Two vocabularies exist and must not be confused:
  *  - the CONFIG values ('prompt' | 'allow-all' | 'custom' | 'plan' |
- *    'accept-edits') — what is stored and what this module operates on.
+ *    'accept-edits'), what is stored and what this module operates on.
  *  - the user-facing LABELS ('normal' | 'auto' | 'custom' | 'plan' |
- *    'accept-edits') — what the pill shows.
+ *    'accept-edits'), what the pill shows.
  */
 
 /** The config `permissions.mode` values, exactly as the SDK schema defines them. */
@@ -23,7 +23,7 @@ export type PermissionModeValue = 'prompt' | 'allow-all' | 'custom' | 'plan' | '
 /**
  * The Shift+Tab cycle order over the four named SESSION modes. `custom` is a
  * per-rule policy, not a session posture, so it is deliberately excluded from
- * the cycle — cycling FROM custom starts at the first entry (normal).
+ * the cycle, cycling FROM custom starts at the first entry (normal).
  *
  * Order (matches the settled convention: escalating autonomy, then wrap):
  *   normal → accept-edits → plan → auto → normal

package/src/core/power-chip-source.ts

@@ -1,5 +1,5 @@
 // ---------------------------------------------------------------------------
-// power-chip-source.ts — the topology-aware source of truth behind the
+// power-chip-source.ts, the topology-aware source of truth behind the
 // always-visible "sleep disabled" chip.
 //
 // Embedded topology: the in-process PowerManager IS the daemon's, so reading
@@ -11,7 +11,7 @@
 // External/adopted topology: the local manager is NOT the daemon's, and the
 // adopted-daemon wire is plain HTTP (no realtime event bridge), so the chip
 // POLLS power.status.get on an unref'd interval and renders the DAEMON's
-// state — a webui-originated toggle on the daemon lights the TUI chip within
+// state, a webui-originated toggle on the daemon lights the TUI chip within
 // one poll, and a TUI toggle (forwarded over power.keepAwake.set by
 // power-keepawake-remote.ts) lights the webui's the same way. While the
 // daemon's state is unknown (fetch pending/unreachable) the local projection
@@ -38,7 +38,7 @@ export interface PowerChipSourceDeps {
 }
 
 export interface PowerChipSource {
-  /** The current chip projection — daemon-synced in external topology, local otherwise. */
+  /** The current chip projection, daemon-synced in external topology, local otherwise. */
   get(): PowerSurfaceState;
   /** Stop the poll + event subscription. */
   stop(): void;

package/src/core/power-status.ts

@@ -1,15 +1,15 @@
 // ---------------------------------------------------------------------------
-// power-status.ts — the surface-facing view of host sleep ownership (power.*).
+// power-status.ts, the surface-facing view of host sleep ownership (power.*).
 //
 // The SDK's PowerManager owns the policy: a keep-awake toggle (daemon-held,
 // persisted), automatic work-signal inhibition ("held because X"), and the
 // honest lid-split note when idle-sleep can be blocked but lid-close suspend
-// can't. This module is the read-only projection every surface renders from —
-// the always-visible "sleep disabled" chip, and the ops/status detail lines —
+// can't. This module is the read-only projection every surface renders from,
+// the always-visible "sleep disabled" chip, and the ops/status detail lines,
 // seeded from powerManager.getState() and kept live by OPS_POWER_STATE_CHANGED.
 //
 // Owner ruling: the chip IS the safety mechanism. No timers, no AC-only
-// options — the keep-awake toggle is exactly one boolean.
+// options, the keep-awake toggle is exactly one boolean.
 // ---------------------------------------------------------------------------
 
 import { LID_SWITCH_HONEST_SPLIT, type PowerState } from '@pellux/goodvibes-sdk/platform/power';
@@ -19,7 +19,7 @@ export const SLEEP_DISABLED_CHIP = 'sleep disabled';
 
 /** The flattened power state a surface renders from. */
 export interface PowerSurfaceState {
-  /** The owner keep-awake toggle — the "sleep disabled" chip source. */
+  /** The owner keep-awake toggle, the "sleep disabled" chip source. */
   readonly keepAwake: boolean;
   /** True while any sleep inhibitor (work or keep-awake) is held. */
   readonly inhibited: boolean;
@@ -29,7 +29,7 @@ export interface PowerSurfaceState {
   readonly note: string | null;
 }
 
-/** The neutral state (no inhibition, nothing held) — the pre-event seed default. */
+/** The neutral state (no inhibition, nothing held), the pre-event seed default. */
 export const IDLE_POWER_STATE: PowerSurfaceState = {
   keepAwake: false,
   inhibited: false,
@@ -66,7 +66,7 @@ export function powerSurfaceFromEvent(ev: PowerStateChangedPayload): PowerSurfac
 }
 
 /**
- * The ops/status detail lines — the honest, plain-language account of sleep
+ * The ops/status detail lines, the honest, plain-language account of sleep
  * ownership. Order: the chip's meaning, then each "held because X" work reason,
  * then the lid-split note VERBATIM when the SDK served it. Empty when nothing
  * is held and no note applies.
@@ -74,7 +74,7 @@ export function powerSurfaceFromEvent(ev: PowerStateChangedPayload): PowerSurfac
 export function powerStatusLines(state: PowerSurfaceState): string[] {
   const lines: string[] = [];
   if (state.keepAwake) {
-    lines.push(`${SLEEP_DISABLED_CHIP} — keep-awake is on (this host will not idle-sleep)`);
+    lines.push(`${SLEEP_DISABLED_CHIP}: keep-awake is on (this host will not idle-sleep)`);
   }
   for (const reason of state.workReasons) {
     lines.push(`held because ${reason}`);

package/src/core/rewind-receipt.ts

@@ -1,11 +1,11 @@
 /**
- * rewind-receipt.ts — transcript rendering for an applied unified rewind.
+ * rewind-receipt.ts, transcript rendering for an applied unified rewind.
  *
  * The SDK's UnifiedRewindService returns a RewindReceipt from rewind.apply (and
  * emits a matching REWIND_APPLIED workspace event). This module turns that
  * receipt into a distinct, multi-line `[Rewind]` block, mirroring the
  * `[Compaction]` receipt (compaction-receipt.ts) so a rewind is never a silent
- * mutation of files or conversation history — the transcript records exactly
+ * mutation of files or conversation history, the transcript records exactly
  * what was restored and whether it can be reversed.
  *
  * The `[Rewind]` prefix is one of the FORCE_CONVERSATION_PREFIXES the
@@ -42,13 +42,13 @@ function describeAnchor(turnId: string | null): string {
 }
 
 /**
- * Build the distinct multi-line `[Rewind]` receipt block. Pure — no state, no
+ * Build the distinct multi-line `[Rewind]` receipt block. Pure, no state, no
  * I/O. Lines after the header are indented two spaces so the transcript renders
  * them as one grouped block under the `[Rewind] Receipt` heading.
  */
 export function buildRewindReceiptBlock(receipt: RewindReceiptInput): string {
   const lines: string[] = [];
-  lines.push(`[Rewind] Receipt — rewound ${receipt.scope} to ${describeAnchor(receipt.turnId)}.`);
+  lines.push(`[Rewind] Receipt: rewound ${receipt.scope} to ${describeAnchor(receipt.turnId)}.`);
 
   if (receipt.files) {
     if (receipt.files.restored) {
@@ -57,7 +57,7 @@ export function buildRewindReceiptBlock(receipt: RewindReceiptInput): string {
         `removed ${plural(receipt.files.removedFileCount, 'file')}.`,
       );
     } else {
-      lines.push('  Files: not restored — no workspace checkpoint matched this anchor.');
+      lines.push('  Files: not restored; no workspace checkpoint matched this anchor.');
     }
   }
 
@@ -65,7 +65,7 @@ export function buildRewindReceiptBlock(receipt: RewindReceiptInput): string {
     if (receipt.conversation.rewound) {
       lines.push(`  Conversation: dropped ${plural(receipt.conversation.droppedMessages, 'message')} after this turn.`);
     } else {
-      lines.push('  Conversation: not rewound — no conversation boundary was recorded for this anchor.');
+      lines.push('  Conversation: not rewound; no conversation boundary was recorded for this anchor.');
     }
   }
 

package/src/core/session-resume-core.ts

@@ -1,5 +1,5 @@
 /**
- * session-resume-core.ts — the ONE resume routine both resume seams call.
+ * session-resume-core.ts, the ONE resume routine both resume seams call.
  *
  * Two independent call sites used to duplicate (and diverge from) this
  * sequence: `/session resume` (session-workflow.ts) and the panel/session-
@@ -23,7 +23,7 @@
  * logging the outcome in their own idiom (`ctx.print` vs `conversation.log`),
  * and any extra plumbing that only one seam performs (hookDispatcher.fire,
  * sessionSpine.reopen, sharedSessionBroker.reopenSession,
- * writeLastSessionPointer — all bootstrap-hook-bridge-only, pre-existing
+ * writeLastSessionPointer, all bootstrap-hook-bridge-only, pre-existing
  * asymmetries out of this module's scope).
  */
 import type { SessionManager, SessionMeta } from '@pellux/goodvibes-sdk/platform/sessions';
@@ -52,7 +52,7 @@ export interface SessionResumeDeps {
    * The app's declare-once session-storage handle. Both resume seams pass the
    * SAME one the runtime writes through, so the anchor sidecar and the
    * transcript journal this routine reads are guaranteed to be the files the
-   * live session actually wrote — the scope can no longer be re-guessed per
+   * live session actually wrote, the scope can no longer be re-guessed per
    * seam.
    */
   readonly surface: SessionSurface;
@@ -65,7 +65,7 @@ export interface SessionResumeDeps {
   readonly selectModel?: (model: string) => Promise<{ readonly registryKey: string; readonly providerId: string }>;
   readonly hydrateSessionUsage?: () => void;
   /**
-   * Deliberate cap on how many saved panels are reopened at once — resuming
+   * Deliberate cap on how many saved panels are reopened at once, resuming
    * into a workspace crowded with every panel that happened to be open is
    * its own kind of surprise. Overflow beyond the cap is reported in the
    * outcome (`panels.notReopened`), never silently dropped. Defaults to 4.
@@ -76,7 +76,7 @@ export interface SessionResumeDeps {
 export interface PanelReopenOutcome {
   readonly reopened: readonly string[];
   readonly movedToModal: readonly string[];
-  /** Saved panel ids beyond `panelReopenLimit` — not attempted, honestly reported (see /panels to open the rest). */
+  /** Saved panel ids beyond `panelReopenLimit`, not attempted, honestly reported (see /panels to open the rest). */
   readonly notReopened: readonly string[];
 }
 
@@ -90,7 +90,7 @@ export interface SessionResumeOutcome {
 
 /**
  * Exported so both session-workflow.ts's standalone `reopenPanelsFromReturnContext`
- * (kept for its existing direct unit-test coverage — see
+ * (kept for its existing direct unit-test coverage, see
  * session-workflow-panel-restore.test.ts) and `resumeSessionCore` below share
  * the exact same default cap.
  */
@@ -107,7 +107,7 @@ export function reopenPanelsWithModalSkip(
   const reopened: string[] = [];
   const movedToModal: string[] = [];
   for (const panelId of within) {
-    // A MIGRATE-TO-MODAL id has no panel to restore — a modal is not part of
+    // A MIGRATE-TO-MODAL id has no panel to restore, a modal is not part of
     // the saved panel layout. Skip it (don't pop a modal mid-resume) and note
     // it once, rather than firing open() and revealing an empty workspace.
     if (panelManager.getModalRedirect(panelId) !== undefined) {
@@ -144,7 +144,7 @@ export async function resumeSessionCore(sessionId: string, deps: SessionResumeDe
   deps.runtime.sessionId = sessionId;
 
   // Restore this session's message-anchored rewind anchors from its sidecar
-  // so /rewind works identically before and after the resume — the in-memory
+  // so /rewind works identically before and after the resume, the in-memory
   // registry is process-local, so a fresh process starts with none for the
   // loaded session.
   const restoredAnchorCount = restoreTurnAnchors(sessionId, deps.surface);