@pellux/goodvibes-tui 2.0.14 → 2.0.15

package/src/shell/ui-openers.ts

@@ -56,7 +56,7 @@ type WireShellUiOpenersOptions = {
   getConfiguredProviderIds: () => string[];
   getPinned: () => Promise<string[]>;
   render: () => void;
-  /** Trust-at-consequence-time bridge — patched here with the real modal-driving implementation. */
+  /** Trust-at-consequence-time bridge, patched here with the real modal-driving implementation. */
   trustPromptRef: { requestTrustDecision: () => Promise<WorkspaceTrustLevel> };
 };
 
@@ -90,7 +90,7 @@ function deriveConfiguredVia(
 ): 'env' | 'secrets' | 'subscription' | 'anonymous' | undefined {
   if (!configuredIds.has(providerId)) return undefined;
 
-  // Tier 1: subscription check (most specific — subscription overrides env for this provider)
+  // Tier 1: subscription check (most specific, subscription overrides env for this provider)
   const subs = subscriptionManager.list();
   if (subs.some((s) => s.provider === providerId)) return 'subscription';
 
@@ -148,7 +148,7 @@ export function wireShellUiOpeners(options: WireShellUiOpenersOptions): void {
 
   // Trust-at-consequence-time: raised by trustGatedAsk on the first non-read
   // request in an undecided workspace; the answer persists via setLevel().
-  // Registration self-records here too when trusted (never for restricted —
+  // Registration self-records here too when trusted (never for restricted,
   // see selfRecordWorkspaceRegistration's doc).
   trustPromptRef.requestTrustDecision = () =>
     new Promise((resolve) => {
@@ -205,7 +205,7 @@ export function wireShellUiOpeners(options: WireShellUiOpenersOptions): void {
    * Fetch the embedding-provider status list once per picker-open and shape
    * it into the picker's own tiny item type. Uses the registry's async
    * `.status()` (not the sync `.list()`) specifically because `.list()` does
-   * not carry `configured` — and unconfigured providers must be shown
+   * not carry `configured`, and unconfigured providers must be shown
    * honestly, not hidden.
    */
   async function resolveEmbeddingProviderEntries(): Promise<EmbeddingProviderPickerEntry[]> {
@@ -274,7 +274,7 @@ export function wireShellUiOpeners(options: WireShellUiOpenersOptions): void {
       {
         target: 'embeddings',
         label: 'Embeddings',
-        description: 'Embedding provider used for memory search and the code index. Not an LLM route — no model concept.',
+        description: 'Embedding provider used for memory search and the code index. Not an LLM route: no model concept.',
         provider: embeddingProviderId,
         model: '',
         enabled: true,
@@ -386,7 +386,7 @@ export function wireShellUiOpeners(options: WireShellUiOpenersOptions): void {
   };
 
   // Mirrors the Ctrl+F chord's own toggle exactly (handler-shortcuts.ts's
-  // 'search' case) — the transcript search overlay isn't tracked on the
+  // 'search' case), the transcript search overlay isn't tracked on the
   // modal stack, so this doesn't call modalOpened either.
   commandContext.openTranscriptSearch = () => {
     if (input.searchManager.active) input.searchManager.close(input.conversationManager);
@@ -411,7 +411,7 @@ export function wireShellUiOpeners(options: WireShellUiOpenersOptions): void {
     input.modalOpened('settings');
     input.settingsModal.open(configManager, featureFlags, subscriptionManager, serviceRegistry, mcpRegistry, secretsManager, {
       // The Connections category probes through this app's catalog, which is
-      // empty by construction — so those rows read 'unreachable' rather than
+      // empty by construction, so those rows read 'unreachable' rather than
       // inventing a settled state nothing established. Retargeting that probe
       // at the daemon is the remaining half of this seam.
       ...(commandContext.workspace?.gatewayMethods
@@ -451,11 +451,11 @@ export function wireShellUiOpeners(options: WireShellUiOpenersOptions): void {
   // open()-time modal-redirect callback (a legacy panel id resolving to a modal
   // name). Surfaces are registered in builtin-modals.ts; a name with no
   // registered surface degrades honestly to a print rather than a blank modal.
-  // The stack name is the stable 'config' slot (one config modal at a time —
+  // The stack name is the stable 'config' slot (one config modal at a time,
   // opening another swaps the surface), so Esc close/return and the modal-stack
   // machinery need only the single 'config' case (handler-ui-state.ts).
   // Some panel-id redirects (and migrated front-doors) resolve to a NATIVE
-  // modal that is not a ConfigModalSurface — e.g. `sessions` -> `sessionPicker`,
+  // modal that is not a ConfigModalSurface, e.g. `sessions` -> `sessionPicker`,
   // where 'sessionPicker' is the real session-picker modal opened by
   // commandContext.openSessionPicker below, NOT a registered config surface.
   // getModalSurface can never find these, so consult this small name->opener
@@ -513,7 +513,7 @@ export function wireShellUiOpeners(options: WireShellUiOpenersOptions): void {
     }
     if (panelManager.getAllOpen().length === 0) {
       // (the purge): 'panel-list' (a browse-all-panels picker PANEL)
-      // was DELETE-disposition — a picker over a handful of panels is dead
+      // was DELETE-disposition, a picker over a handful of panels is dead
       // weight now. Its replacement is this selection MODAL, built from the
       // live registry (PanelManager.getRegisteredTypes()) rather than a
       // hardcoded list, so it can never list a retired/deleted id.
@@ -547,8 +547,8 @@ export function wireShellUiOpeners(options: WireShellUiOpenersOptions): void {
   commandContext.getCommandCategories = () => getCommandCategoryMap();
 
   commandContext.openCommandPalette = () => {
-    // The palette is built live from the command registry — never a hardcoded
-    // list — so it can never drift from the real command set. Category labels
+    // The palette is built live from the command registry, never a hardcoded
+    // list, so it can never drift from the real command set. Category labels
     // come from the same source the generated docs use (getCommandCategoryMap).
     const registry = input.commandRegistry;
     if (!registry) return;
@@ -582,7 +582,7 @@ export function wireShellUiOpeners(options: WireShellUiOpenersOptions): void {
         // Pre-fill the composer with the command (and a trailing space so the
         // inline args hint shows what the command wants). Keyboard-first: focus
         // returns to the composer with command mode armed so the user fills any
-        // args and presses Enter — matching tab-completion's fill behavior.
+        // args and presses Enter, matching tab-completion's fill behavior.
         input.prompt = `/${cmd.name} `;
         input.cursorPos = input.prompt.length;
         input.commandMode = true;
@@ -610,7 +610,7 @@ export function wireShellUiOpeners(options: WireShellUiOpenersOptions): void {
     panelManager.open(panelId, pane, target);
     panelManager.show();
     // focus rule 1a: every registered caller of showPanel is a slash
-    // command (/panel open, /routes, /approval, /tasks, /ops-control, ...) —
+    // command (/panel open, /routes, /approval, /tasks, /ops-control, ...),
     // the command path leaves focus in the composer ("the user is
     // mid-command-flow") unless the caller explicitly asks to grab it.
     if (opts?.focus) panelManager.focusPanels();

package/src/shell/voice-capture-shell.ts

@@ -1,11 +1,11 @@
 /**
- * voice-capture-shell.ts — the shell's side of microphone capture.
+ * voice-capture-shell.ts, the shell's side of microphone capture.
  *
  * Extracted from main() for the same reason the process-lifecycle wiring was:
  * the entrypoint is a wiring file held under a source-line gate, and new shell
  * composition gets its own module and a single call there.
  *
- * What this owns is only the shell-facing half — where a transcript lands (the
+ * What this owns is only the shell-facing half, where a transcript lands (the
  * composer draft, through the same public `prompt`/`cursorPos` fields the
  * external-editor path writes), how a turn is submitted, which keybinding seam
  * the Alt+V action hangs off, and the teardown registration that guarantees a
@@ -30,7 +30,7 @@ export interface VoiceCaptureShellDeps {
   readonly commandContext: CommandContext;
   /** The shell's teardown registry; the device release is appended to it. */
   readonly unsubs: Array<() => void>;
-  /** The live composer buffer — InputHandler exposes public `prompt`/`cursorPos`. */
+  /** The live composer buffer, InputHandler exposes public `prompt`/`cursorPos`. */
   readonly buffer: { prompt: string; cursorPos: number };
   readonly submitInput: (text: string) => void;
   readonly notify: (message: string) => void;

package/src/tools/wrfc-agent-guard.ts

@@ -46,8 +46,8 @@ type AgentTaskArgs = {
 
 /**
  * Emitted whenever the guard changes the effective routing decision:
- *   - 'spawn-forced-wrfc'      — implementation-like spawn promoted to WRFC
- *   - 'spawn-suppressed-wrfc'  — spawn judged read-only; WRFC suppressed
+ *   - 'spawn-forced-wrfc'     , implementation-like spawn promoted to WRFC
+ *   - 'spawn-suppressed-wrfc' , spawn judged read-only; WRFC suppressed
  *   - 'batch-collapsed-to-wrfc'— batch-spawn collapsed into a single WRFC owner chain
  */
 export type WrfcGuardTraceKind =
@@ -248,7 +248,7 @@ function isImplementationLikeTask(task: AgentTaskArgs): boolean {
 function isReadOnlyTask(text: string): boolean {
   // Branch A: explicit do-not-write guards
   if (/\bdo\s+not\s+(?:write|edit|modify|change|create)\b|\bread[-\s]*only\b|\bwithout\s+(?:writing|editing|modifying|changing|creating)\b/i.test(text)) return true;
-  // Branch B: task leads with an analysis/reporting verb — treat it as read-only regardless
+  // Branch B: task leads with an analysis/reporting verb, treat it as read-only regardless
   // of any action verbs that appear later in the sentence. A task that LEADS with
   // "report", "investigate", "describe", "audit", etc. is describing or evaluating an
   // action, not performing it. Examples that must NOT reach WRFC:
@@ -259,7 +259,7 @@ function isReadOnlyTask(text: string): boolean {
   //   "audit which modules to delete"
   //   "evaluate whether to migrate"
   // The negative-lookahead is intentionally absent: the leading verb is authoritative.
-  // Note: 'review' is intentionally excluded — tasks leading with 'review' are caught
+  // Note: 'review' is intentionally excluded, tasks leading with 'review' are caught
   // by isRootReviewRoleTask() and routed to a WRFC chain as a reviewer role.
   return /^\s*(?:inspect|research|read|find|list|summarize|analy[sz]e|explain|report|investigate|document|describe|audit|evaluate|assess|check|compare|tell|show)\b/i.test(text);
 }

package/src/utils/browser.ts

@@ -1,5 +1,5 @@
 /**
- * browser.ts — cross-platform browser launcher utility.
+ * browser.ts, cross-platform browser launcher utility.
  *
  * Extracted from cli/management.ts so it can be used by input-layer commands
  * without creating an upward cli→input dependency. Lives in utils (Layer 0)
@@ -12,7 +12,7 @@ import { summarizeError } from '@pellux/goodvibes-sdk/platform/utils';
 /**
  * Open the given URL in the user's default browser.
  * Returns a status string (success or error description).
- * Does not throw — errors are returned as a descriptive string.
+ * Does not throw, errors are returned as a descriptive string.
  */
 export function openBrowser(url: string): string {
   const platform = process.platform;

package/src/utils/format-duration.ts

@@ -2,13 +2,13 @@
  * Shared ms-duration formatters for the panel layer.
  *
  * Two families are exported, each matching a distinct behavior cluster.
- * DO NOT replace these with formatElapsed from utils/format-elapsed.ts —
+ * DO NOT replace these with formatElapsed from utils/format-elapsed.ts,
  * formatElapsed floors to integer seconds and lacks the null/'?ms' guards
  * that the latency panels require.
  *
  * (the purge): formatShortDuration (used only by incident-review-panel
- * and eval-panel, both removed — RETIRE-INTO-FLEET and DELETE respectively)
- * was removed as a genuinely orphaned export — no remaining caller anywhere
+ * and eval-panel, both removed, RETIRE-INTO-FLEET and DELETE respectively)
+ * was removed as a genuinely orphaned export, no remaining caller anywhere
  * in src/.
  */
 

package/src/utils/format-number.ts

@@ -30,7 +30,7 @@ export interface AbbreviateCountOpts {
   mDecimals?: number;
   /**
    * 'fixed'  → value.toFixed(decimals)  (default)
-   * 'round'  → String(Math.round(value)) — always integer, decimals ignored
+   * 'round'  → String(Math.round(value)), always integer, decimals ignored
    */
   rounding?: 'fixed' | 'round';
   /**
@@ -39,7 +39,7 @@ export interface AbbreviateCountOpts {
    */
   bSuffix?: boolean;
   /**
-   * When true, never produce M/B/T — format as k even for millions.
+   * When true, never produce M/B/T, format as k even for millions.
    * Default: false
    */
   noM?: boolean;

package/src/utils/splash-lines.ts

@@ -23,7 +23,7 @@ const TAGLINE = '[ ｇｏｏｄ ｖｉｂｅｓ ・ Ａ Ｉ ・ いい雰囲気
 const versionLine = (version: string) =>
   `　✦　v${version}　█　terminal AI assistant　█　自動ｺｰﾄﾞ 　✦`;
 
-/** Fixed hint line — the three primary shell entry points. */
+/** Fixed hint line, the three primary shell entry points. */
 const HINT_LINE = 'Ctrl+P panels  /  ? help  /  F2 fleet';
 
 export interface SplashOptions {
@@ -38,7 +38,7 @@ export interface SplashOptions {
   lastSessionId?: string;
   /**
    * Version string rendered on the splash's version line. Defaults to the
-   * build VERSION; injectable so the golden-frame tests can pin a fixture —
+   * build VERSION; injectable so the golden-frame tests can pin a fixture,
    * the version's display width shifts the line's centering, so goldens tied
    * to the live VERSION break on every release bump (v1.0.0 release failure).
    */
@@ -66,7 +66,7 @@ export function getSplashLines(columns: number, opts: SplashOptions = {}): strin
     '',
   ];
 
-  // Live session context — real state pulled from SplashOptions rather than a
+  // Live session context, real state pulled from SplashOptions rather than a
   // static "/help" signpost: model (provider) and tool count on one line, cwd
   // on the next.
   const contextBits: string[] = [];
@@ -85,7 +85,7 @@ export function getSplashLines(columns: number, opts: SplashOptions = {}): strin
 
   // Last-session resume pointer (readLastSessionPointer result), when recorded.
   if (opts.lastSessionId) {
-    lines.push(metaLine(`↩ resume last session — /resume ${opts.lastSessionId}`, columns));
+    lines.push(metaLine(`↩ resume last session: /resume ${opts.lastSessionId}`, columns));
   }
 
   lines.push('');

package/src/utils/terminal-width.ts

@@ -52,7 +52,7 @@ function stripAnsi(text: string): string {
  * each component's width is summed and ZWJ/VS chars contribute zero width,
  * so the total is accurate. However, truncation (truncateDisplay) may split
  * a ZWJ family mid-sequence, leaving dangling ZWJ/VS characters. This is a
- * cosmetic degradation only — line widths remain correct.
+ * cosmetic degradation only, line widths remain correct.
  */
 export function getDisplayWidth(text: string): number {
   text = stripAnsi(text);
@@ -86,7 +86,7 @@ export function getDisplayWidth(text: string): number {
       code === 0x2717 ||
       code === 0x2714 ||
       code === 0x2718 ||
-      // ✕ (0x2715) and ✖ (0x2716) — the multiplication-X cross family used for
+      // ✕ (0x2715) and ✖ (0x2716), the multiplication-X cross family used for
       // the error-line prefix. Terminals draw them one cell wide, but they sit
       // inside the 0x2600–0x27bf emoji block below and would otherwise be counted
       // as width 2, desyncing the styled cell grid from the physical glyph and
@@ -176,12 +176,12 @@ export interface PrioritizedSegment {
 /**
  * Join segments left-to-right with `separator`, but when the joined line
  * would exceed `width`, drop whole low-priority segments (highest `priority`
- * number first) one at a time until it fits — rather than character-truncating
+ * number first) one at a time until it fits, rather than character-truncating
  * the joined string, which can mangle a high-value segment mid-word (e.g. a
  * `spine:online` daemon-liveness marker clipped to `spi…`).
  *
  * Only falls back to character truncation (via truncateDisplay) if the
- * remaining highest-priority segments still don't fit at width — a rare,
+ * remaining highest-priority segments still don't fit at width, a rare,
  * very-narrow-terminal case.
  */
 export function joinPrioritizedSegments(
@@ -192,11 +192,11 @@ export function joinPrioritizedSegments(
   const join = (list: readonly PrioritizedSegment[]) => list.map(s => s.text).join(separator);
   let kept = segments;
   // Keep dropping whole segments while more than one remains; once a single
-  // segment is left, stop — an empty result would be a worse outcome than
+  // segment is left, stop, an empty result would be a worse outcome than
   // falling through to character truncation on that last segment below.
   while (kept.length > 1 && getDisplayWidth(join(kept)) > width) {
     // Drop the single lowest-priority (highest `priority` number) segment;
-    // ties broken toward the leftmost (earlier-declared) segment surviving —
+    // ties broken toward the leftmost (earlier-declared) segment surviving,
     // `>=` (not `>`) so that on equal priority the LATER index keeps winning
     // as the drop candidate, leaving the earliest-declared segment of that
     // priority tier intact (e.g. cwd survives over model when both are

package/src/verification/verification-ledger.ts

@@ -43,21 +43,21 @@ const SETTINGS_BEHAVIOR_BASELINE = 184;
  * knobs"). Adding these 28 keys raised the settings inventory `total` without a
  * matching rise in behavior-verified coverage, which dropped
  * `localBehaviorPercent` below its floor. Each key is a local, non-external
- * setting whose full persistence behavior — schema default, set → write →
- * reload round-trip, and reset-to-default — is exercised by
+ * setting whose full persistence behavior, schema default, set → write →
+ * reload round-trip, and reset-to-default, is exercised by
  * `src/test/verification/feature-knob-settings-persistence.test.ts`. They are
  * therefore genuinely behavior-verified locally and counted as such here.
  */
 /**
  * The daemon's own mailbox and calendar keys (`surfaces.email.*`,
- * `surfaces.calendar.*`), promoted into CONFIG_SCHEMA so the settings modal —
- * which renders from that schema — can actually show them. Before, the
+ * `surfaces.calendar.*`), promoted into CONFIG_SCHEMA so the settings modal,
+ * which renders from that schema, can actually show them. Before, the
  * daemon's handlers named these keys in their own error messages
  * ("Set surfaces.calendar.caldavUrl and surfaces.calendar.caldavUser") while
  * the UI that told an operator to set them could not display one of them.
  *
  * Declaring 25 keys raised the settings inventory `total` with no matching
- * behavior coverage, which dropped `localBehaviorPercent` below its floor —
+ * behavior coverage, which dropped `localBehaviorPercent` below its floor,
  * the same arrival shape as the two lists above.
  *
  * THIS CONSTANT IS NOT A DIAL. Every key below is counted because
@@ -75,7 +75,7 @@ const SETTINGS_BEHAVIOR_BASELINE = 184;
  * (platform/email/surface-config.ts and platform/calendar/caldav-gateway-config.ts)
  * resolve every one of them when the daemon serves `email.*` and `calendar.*`,
  * and the `surfaces.email.inbound.*` block is read by the daemon's inbound mail
- * reader — the poller/IDLE path that makes incoming mail visible at all.
+ * reader, the poller/IDLE path that makes incoming mail visible at all.
  * The five password keys (email.password, email.imapPassword,
  * email.imap.password, email.smtp.password, calendar.caldavPassword)
  * additionally resolve through the daemon secret tier rather than from config,
@@ -106,7 +106,7 @@ export const DAEMON_MAILBOX_LOCAL_SETTINGS = [
   'surfaces.email.smtp.port',
   'surfaces.email.smtp.password',
   'surfaces.email.smtp.secure',
-  // Inbound mail — the poller/IDLE reader the daemon runs to actually READ mail,
+  // Inbound mail, the poller/IDLE reader the daemon runs to actually READ mail,
   // which arrived with the platform runtime. These are counted for exactly the
   // reason the keys above are: the same four-part persistence contract runs over
   // every one of them in
@@ -179,14 +179,14 @@ export const FEATURE_KNOB_LOCAL_SETTINGS = [
  * exercises it was actually written:
  * `src/test/verification/device-and-trigger-settings-persistence.test.ts` runs,
  * for every key in this list, the same four-part persistence contract the
- * feature-knob keys are counted for — schema default exposure, `set()` through
+ * feature-knob keys are counted for, schema default exposure, `set()` through
  * the key's own validator to disk, reload into a fresh ConfigManager with
  * read-back equality, and reset-to-default that also survives reload. That test
  * additionally asserts this list is exactly the non-enablement key set of both
  * domains and overlaps neither of the other counted sets, so nothing here is
  * double-counted and nothing drifts in uncounted.
  *
- * PER-KEY EVIDENCE — what is verified, and how live the key is TODAY.
+ * PER-KEY EVIDENCE, what is verified, and how live the key is TODAY.
  * "persistence" below means the four-part contract above; the second column
  * records whether anything in this tree (TUI or the pinned SDK) reads the key,
  * audited by tracing each key to its consumers.
@@ -221,7 +221,7 @@ export const FEATURE_KNOB_LOCAL_SETTINGS = [
  *   device.grants.auditRetentionDays         persistence · behavior (grant ledger retention)
  *   device.nodes.maxPaired                   persistence · live (enforced at the pairing path)
  *
- * On `device.nodes.maxPaired`: the SDK enforces it where a device pairs —
+ * On `device.nodes.maxPaired`: the SDK enforces it where a device pairs,
  * PairingTokenManager.mint refuses a NEW node at the cap with
  * PairingLimitReachedError (code DEVICE_NODES_MAX_PAIRED, carrying the setting
  * name, the cap and the live count), mapped to HTTP 409 by the pairing and
@@ -231,7 +231,7 @@ export const FEATURE_KNOB_LOCAL_SETTINGS = [
  * non-finite value reads as no cap so a broken setting cannot lock the owner
  * out. The cap is read per mint, so a change applies without a restart. What is
  * counted HERE is still only the persistence contract this repo's test
- * exercises — the enforcement itself is SDK-side and SDK-tested, and is recorded
+ * exercises, the enforcement itself is SDK-side and SDK-tested, and is recorded
  * in this column so the "how live is it" audit stays accurate, not to claim
  * coverage this repo did not write.
  *
@@ -239,7 +239,7 @@ export const FEATURE_KNOB_LOCAL_SETTINGS = [
  * platform's device posture runtime (src/runtime/device-posture-composition.ts)
  * on the same distributed runtime phones pair onto and the same approval broker
  * every other confirmation rides, registers the `phone` tool on it, and binds the
- * devices.* verbs to it — so each of these keys is read, per request and per
+ * devices.* verbs to it, so each of these keys is read, per request and per
  * sweep, by the DeviceCapabilityService / DeviceGrantStore /
  * DeviceCaptureArtifactStore / DeviceHousekeeper that serve them. They were dark
  * before that: the policy struct carried matching defaults and named the keys in
@@ -251,7 +251,7 @@ export const FEATURE_KNOB_LOCAL_SETTINGS = [
  * `src/test/verification/device-posture-behavior.test.ts`, which drives THIS
  * app's composition with a real ConfigManager over a temp home and the platform's
  * real stores, stubs only the peer transport and the approval bridge, and asserts
- * per key at two values that the observable outcome differs — refusal code, how
+ * per key at two values that the observable outcome differs, refusal code, how
  * authority was established, what reached the transport, what the prompt was
  * given, and what survives on disk after a sweep. It also invokes the `phone`
  * tool and the devices.* verbs off the composed runtime graph, because a mapping
@@ -259,7 +259,7 @@ export const FEATURE_KNOB_LOCAL_SETTINGS = [
  *
  * STILL NOT COUNTED, FOR A DIFFERENT REASON NOW: the 24 non-enablement
  * `voice.wake.*` keys from the same release. When this list was written they were
- * excluded because nothing captured audio at all. That is no longer true — this
+ * excluded because nothing captured audio at all. That is no longer true, this
  * terminal opens a recorder subprocess (src/audio/capture.ts), runs the SDK
  * detector over it through onnxruntime-web (src/audio/wake-inference.ts), and
  * hands both a confirmed wake's utterance and a push-to-talk recording to the
@@ -267,8 +267,8 @@ export const FEATURE_KNOB_LOCAL_SETTINGS = [
  * resolveWakeRuntimeSettings and reaches this surface's runtime.
  *
  * They stay out of the NUMERATOR here because this constant counts exactly one
- * thing — keys whose four-part PERSISTENCE contract is exercised by
- * device-and-trigger-settings-persistence.test.ts — and that test does not cover
+ * thing, keys whose four-part PERSISTENCE contract is exercised by
+ * device-and-trigger-settings-persistence.test.ts, and that test does not cover
  * them. The behaviour they now drive is covered instead by the capture and wake
  * tests under src/test/audio/, which assert the frame path, the detection
  * chain, the disabled-means-no-capture rule, the supervisor's restart and latch,
@@ -343,24 +343,24 @@ const ENABLEMENT_KEYS_BEHAVIOR_VERIFIED = new Set(
  * THIS CONSTANT IS NOT A DIAL, for the same reason as the two above. Each key
  * is counted because a test that exercises it was actually written:
  * `src/test/verification/cluster-group-settings-persistence.test.ts` runs the
- * same four-part persistence contract — schema default exposure, `set()`
+ * same four-part persistence contract, schema default exposure, `set()`
  * through the key's own validator to disk, reload into a fresh ConfigManager
  * with read-back equality, and reset-to-default that survives reload.
  *
  * PER-KEY EVIDENCE. All four have a LIVE consumer in this build, which is more
  * than several of the keys counted above can claim:
- *   cluster.keyRotationHours        — read by resolveClusterGroupSettings and
+ *   cluster.keyRotationHours       , read by resolveClusterGroupSettings and
  *                                     compared against the current key's age in
  *                                     the group runtime's rotation check.
- *   cluster.keyRotationGraceMinutes — sets how long the previous generation
+ *   cluster.keyRotationGraceMinutes, sets how long the previous generation
  *                                     stays accepted after a scheduled
  *                                     rotation; drives the keyring's accepted
  *                                     generation set.
- *   cluster.beaconSeconds           — the discovery beacon timer interval, and
+ *   cluster.beaconSeconds          , the discovery beacon timer interval, and
  *                                     the basis of the "recently heard from"
  *                                     window that decides which member mints a
  *                                     rotation.
- *   cluster.rosterGossipSeconds     — how often the member list is shared.
+ *   cluster.rosterGossipSeconds    , how often the member list is shared.
  *
  * `cluster.enabled` is NOT here: it is a feature enablement key and the ledger
  * counts those in its own set.
@@ -378,7 +378,7 @@ export const CLUSTER_GROUP_LOCAL_SETTINGS = [
  * budget knobs, the shipping-tier preference, the fourteen billing/shipping
  * address sub-fields, the two approval/veto windows, and the notify-channel
  * list. Card MATERIAL (number, expiry, CVV, cardholder name) is intentionally
- * NOT here — see the SDK's schema-domain-payments.ts header and this repo's
+ * NOT here, see the SDK's schema-domain-payments.ts header and this repo's
  * own input/payments-config.ts: that material lives write-only in the daemon
  * secret store, never in CONFIG_SCHEMA, so there is nothing for a persistence
  * contract to count.
@@ -439,7 +439,7 @@ export const PAYMENTS_LOCAL_SETTINGS = [
 ] as const;
 
 /**
- * Connector settings — the daemon's Google-backed mail/calendar keys plus the
+ * Connector settings, the daemon's Google-backed mail/calendar keys plus the
  * connected-host dial switch and the hosted-turn routing switch, declared as
  * described schema rows by platform runtime 2.0.8 (before that the connector
  * keys were cast onto the live config invisibly and the settings surface
@@ -451,7 +451,7 @@ export const PAYMENTS_LOCAL_SETTINGS = [
  * google prefixes
  * plus the two switches. Arriving uncounted is what dropped
  * localBehaviorPercent through its floor when the schema grew. The three
- * secret-valued keys hold goodvibes://secrets/ references, never values —
+ * secret-valued keys hold goodvibes://secrets/ references, never values,
  * same note as the mailbox passwords above.
  */
 export const CONNECTOR_LOCAL_SETTINGS = [
@@ -577,7 +577,7 @@ function countBuiltinPanels(root: string): number {
 
 /**
  * The CLI command words this terminal actually accepts, read off the catalog
- * the parser is driven by rather than scraped out of a source file — the
+ * the parser is driven by rather than scraped out of a source file, the
  * ledger counts what ships, and a catalog entry IS what ships.
  */
 function listCliCommands(): string[] {

package/src/version.ts

@@ -6,21 +6,21 @@ import { join } from 'node:path';
 // The prebuild script updates the fallback value before compilation.
 // Uses import.meta.dir (Bun) to locate package.json relative to this file,
 // which is correct regardless of the process working directory.
-let _version = '2.0.14';
+let _version = '2.0.15';
 try {
   const pkg = JSON.parse(readFileSync(join(import.meta.dir, '..', 'package.json'), 'utf-8'));
   // Only trust a version read from OUR OWN package.json. A Bun single-file
   // compiled binary resolves import.meta.dir to a virtual root where the
   // `../package.json` path can land on a DIFFERENT package.json (a bundled
   // dependency's, or the embedded runtime's) that reports a placeholder like
-  // "0.0.0" — exactly the wrong-version banner a bare daemon launch showed in
+  // "0.0.0", exactly the wrong-version banner a bare daemon launch showed in
   // the field. Guarding on the package name means the prebuild-baked fallback
   // above wins in that case instead of a stray version.
   if (pkg?.name === '@pellux/goodvibes-tui' && typeof pkg.version === 'string' && pkg.version.length > 0) {
     _version = pkg.version;
   }
 } catch {
-  // Compiled binary or missing package.json — use fallback
+  // Compiled binary or missing package.json, use fallback
 }
 
 export const VERSION = _version;