@pellux/goodvibes-tui 2.0.14 → 2.0.16

package/src/runtime/context-accounting-source.ts

@@ -6,7 +6,7 @@ import type { ContextAccountingSource } from '@pellux/goodvibes-sdk/platform/too
 /**
  * Minimal runtime-bus surface this module needs: subscribe by event type.
  * A narrow structural type (not the full RuntimeEventBus) so this module
- * doesn't pull in `runtime/index.ts` — bootstrap.ts imports this module
+ * doesn't pull in `runtime/index.ts`, bootstrap.ts imports this module
  * directly, and index.ts re-exports bootstrap.ts, so a barrel import here
  * would be a 3-file cycle (see check-architecture.ts's cycle detector).
  */
@@ -15,7 +15,7 @@ export interface CompactionActivityBus {
 }
 
 /**
- * Builds the interactive session's ContextAccountingSource — the honest
+ * Builds the interactive session's ContextAccountingSource, the honest
  * backing for the SDK's `context_accounting` tool. Bound onto
  * `RuntimeServices.contextAccountingHolder` once, at bootstrap (see
  * bootstrap.ts, right after the live Orchestrator is constructed).
@@ -29,7 +29,7 @@ export interface CompactionActivityBus {
  *  - getCompactionState()  → compactionCount from SessionLineageTracker (the
  *    same counter session-maintenance and /compact read); isCompacting is
  *    tracked from real 'compaction' domain lifecycle events on the runtime
- *    bus, since the Orchestrator has no public accessor for it — NOT
+ *    bus, since the Orchestrator has no public accessor for it, NOT
  *    fabricated. Without a runtime bus (headless/test callers), isCompacting
  *    honestly stays false rather than guessing.
  */
@@ -104,7 +104,7 @@ export function createContextAccountingSource(deps: ContextAccountingSourceDeps)
 
 /**
  * Create the source, install it on the holder, and register its dispose on
- * the caller's unsubscribe list — the one-call bootstrap wiring.
+ * the caller's unsubscribe list, the one-call bootstrap wiring.
  */
 export function wireContextAccountingSource(
   deps: ContextAccountingSourceDeps,

package/src/runtime/context.ts

@@ -1,5 +1,5 @@
 /**
- * RuntimeContext — the composition root object returned by bootstrapRuntime().
+ * RuntimeContext, the composition root object returned by bootstrapRuntime().
  *
  * main.ts receives this and uses it to drive the render loop, input handling,
  * and terminal lifecycle. The bootstrap owns initialization; main.ts owns
@@ -143,7 +143,7 @@ export interface RuntimeContext {
 
   /**
    * Logical shutdown: save session, fire hooks, stop background managers.
-   * Does NOT touch the terminal — main.ts owns terminal teardown.
+   * Does NOT touch the terminal, main.ts owns terminal teardown.
    *
    * @param sessionData - Latest conversation data to persist.
    */

package/src/runtime/conversation-rewind-port.ts

@@ -1,5 +1,5 @@
 // ---------------------------------------------------------------------------
-// conversation-rewind-port.ts — the TUI's ConversationManager as the SDK unified
+// conversation-rewind-port.ts, the TUI's ConversationManager as the SDK unified
 // rewind service's RewindConversationPort.
 //
 // The SDK's UnifiedRewindService (platform/rewind) joins files rewind (workspace
@@ -8,11 +8,11 @@
 // messages would truncate to a recorded turn boundary, and rewind() performs the
 // truncation and captures the pre-/post-truncation snapshots so the reversal can
 // be undone and re-applied. The truncation boundary is the message count recorded
-// for the anchor's turnId at TURN_COMPLETED (rewind-turn-anchors.ts) — the same
+// for the anchor's turnId at TURN_COMPLETED (rewind-turn-anchors.ts), the same
 // join key files rewind uses against the workspace checkpoint.
 //
 // This module is the single implementation of that port for the TUI, and it has
-// two consumers again — but the second one reaches it differently now.
+// two consumers again, but the second one reaches it differently now.
 //
 // It used to be the composed daemon's own rewind.plan/apply verbs, resolving a
 // conversation out of the registry below because the daemon was this process.
@@ -48,7 +48,7 @@ export interface ConversationRewindPort extends RewindConversationPort {
   restoreAfter(undoSnapshotId: string): boolean;
 }
 
-/** One truncation's captured state — the target conversation and its snapshots. */
+/** One truncation's captured state, the target conversation and its snapshots. */
 interface SnapshotPair {
   readonly conv: ConversationManager;
   readonly before: ConversationJson;
@@ -57,7 +57,7 @@ interface SnapshotPair {
 
 /**
  * Build a conversation rewind port. `resolveConversation` maps an anchor's
- * sessionId to the live ConversationManager — for the single-session /rewind
+ * sessionId to the live ConversationManager, for the single-session /rewind
  * command it returns the one bound conversation; for the daemon it looks the
  * session up in the live registry. A null resolution means no live conversation
  * for that session, reported as "nothing to drop" rather than a fabricated count.
@@ -116,7 +116,7 @@ export function createConversationRewindPort(
 //
 // This process's conversations, by session id. Two readers now: the local
 // /rewind command, and the host loop that answers the DAEMON's questions about
-// a session this surface is running (the SDK's client/conversation-rewind-host.ts) —
+// a session this surface is running (the SDK's client/conversation-rewind-host.ts),
 // which is what makes conversation rewind work from any surface again, since
 // only the process holding the messages can count or drop them.
 // ---------------------------------------------------------------------------

package/src/runtime/credential-composition.ts

@@ -7,7 +7,7 @@
  * These are grouped because they share one fact, and it is the fact this round
  * had to fix: WHERE the daemon's own credentials live. `daemonHome` is threaded
  * rather than defaulted, because without it a daemon told to run out of a temp
- * tree moved its identity directory and nothing else — the credential store
+ * tree moved its identity directory and nothing else, the credential store
  * stayed in the real home, so an "isolated" test daemon held live credentials
  * and long-polled a real account. A default here would silently restore that.
  *
@@ -25,7 +25,7 @@ interface CredentialCompositionInput {
   readonly homeDirectory: string;
   /**
    * The daemon's state root when the host was told one. Absent means "not
-   * overridden" and must stay absent rather than being defaulted — see above.
+   * overridden" and must stay absent rather than being defaulted, see above.
    */
   readonly daemonHomeDirectory?: string | undefined;
   readonly configManager: ConstructorParameters<typeof SecretsManager>[0]['configManager'];

package/src/runtime/daemon-attach-notices.ts

@@ -1,12 +1,12 @@
 /**
- * daemon-attach-notices — the attach-time consuming read.
+ * daemon-attach-notices, the attach-time consuming read.
  *
  * On daemon attach a surface reads the daemon's undelivered receipts
  * (update/crash/migration) AND the pending feature-announcement queue
  * (web-surface URL, the sandbox first-contained-run line, …) exactly once,
- * rendering each as a one-line notice. Both sources clear on read — the
+ * rendering each as a one-line notice. Both sources clear on read, the
  * facade's receipt queue marks its entries delivered, and drainPending() clears
- * the announcement queue — so a second attach with nothing new shows nothing.
+ * the announcement queue, so a second attach with nothing new shows nothing.
  * This mirrors the daemon's own /status?receipts=consume fold, done in-process
  * for the embedded daemon.
  */
@@ -35,7 +35,7 @@ export interface DaemonAttachNoticesDeps {
 }
 
 /**
- * Consume — exactly once — the daemon's receipts and the pending announcement
+ * Consume, exactly once, the daemon's receipts and the pending announcement
  * queue, returning the one-line notices to render. Receipts first (crash /
  * update / migration), then announcements (web-surface URL, sandbox line, …).
  */
@@ -62,7 +62,7 @@ export interface ExternalDaemonAttachReadDeps {
   readonly authToken: string;
   /**
    * Ask for the daemon's undelivered receipts on this read. Delivery at the
-   * daemon is destructive — served once, to the consuming reader — so only the
+   * daemon is destructive, served once, to the consuming reader, so only the
    * read that RENDERS them asks. A reconnect read that just refreshes the two
    * build floors leaves this off, and no receipt is eaten before a surface can
    * show it.
@@ -97,7 +97,7 @@ export interface ExternalDaemonAttachRead {
  *
  * The default deployment adopts such a daemon with no in-process handle
  * (daemonServer is null), so the embedded in-process fold sees nothing; this
- * hits the daemon's own endpoint instead — GET <baseUrl>/status with the shared
+ * hits the daemon's own endpoint instead, GET <baseUrl>/status with the shared
  * bearer, plus `?receipts=consume` when the caller is going to render them. The
  * daemon marks the served receipts delivered as it responds (its own
  * exactly-once store), so a second attach with nothing new returns no notices.

package/src/runtime/diagnostics/panels/index.ts

@@ -1,5 +1,5 @@
 /**
- * Diagnostics panels barrel — re-exports all panel data provider classes.
+ * Diagnostics panels barrel, re-exports all panel data provider classes.
  *
  * Import from this module to access the individual diagnostic panel providers.
  */

package/src/runtime/diagnostics/panels/ops.ts

@@ -1,5 +1,5 @@
 /**
- * OpsPanel — diagnostic data provider for the Operator Control Plane.
+ * OpsPanel, diagnostic data provider for the Operator Control Plane.
  *
  * Subscribes to OPS_AUDIT events from the UI-facing ops event feed and maintains a
  * bounded buffer of intervention records for display
@@ -149,7 +149,7 @@ export class OpsPanel {
       try {
         cb();
       } catch {
-        // Non-fatal — subscriber errors must not crash the panel
+        // Non-fatal, subscriber errors must not crash the panel
       }
     }
   }

package/src/runtime/disposal-wiring.ts

@@ -3,8 +3,8 @@
  *
  * Teardown for every poller this surface's runtime graph starts.
  *
- * The mechanics — the ordered, best-effort, idempotent scope and the
- * all-required owner list — live in the SDK (`platform/runtime/disposal`), so
+ * The mechanics, the ordered, best-effort, idempotent scope and the
+ * all-required owner list, live in the SDK (`platform/runtime/disposal`), so
  * this surface and the SDK's own composition root cannot drift into two
  * different ideas of what "stop the graph" means. What lives here is only the
  * mapping from this fork's assembled graph onto that list.
@@ -15,7 +15,7 @@
  * Ownership note, and the reason any of this exists: this surface builds the
  * graph and hands the SAME object to `DaemonServer`. The SDK facade disposes
  * only a graph it constructed itself, so it deliberately leaves this one alone
- * — nothing upstream will ever stop these pollers. The shutdown paths in
+ *, nothing upstream will ever stop these pollers. The shutdown paths in
  * daemon/cli.ts, main.ts's teardown registry and the one-shot CLI commands are
  * the only things that can.
  *
@@ -41,7 +41,7 @@ export interface SurfaceRuntimePollerOwners extends Omit<RuntimePollerOwners, 's
   /**
    * Fork-only: the wake-word recovery sweep and a pending boot provision
    * (voice-setup-services.ts). Started only when an entrypoint opted into boot
-   * provisioning, and a no-op otherwise — but it is on this list unconditionally,
+   * provisioning, and a no-op otherwise, but it is on this list unconditionally,
    * because "the graph did not start it this time" is not a reason for the
    * teardown path to have no way to stop it.
    */
@@ -50,7 +50,7 @@ export interface SurfaceRuntimePollerOwners extends Omit<RuntimePollerOwners, 's
 
 /**
  * The poller owners this fork holds that are NOT reachable from the assembled
- * graph — handles the factory keeps as locals.
+ * graph, handles the factory keeps as locals.
  */
 export interface RuntimeDisposalExtras {
   /** Handle returned by `ConfigManager.watchConfigFiles()` (durability-services.ts). */
@@ -61,7 +61,7 @@ export interface RuntimeDisposalExtras {
  * Register the stop call for every poller the graph started.
  *
  * `services` is the fully-assembled graph, which already exposes each poller
- * owner as a field — so a poller whose owner reaches the public surface is
+ * owner as a field, so a poller whose owner reaches the public surface is
  * wired by name rather than by threading another local out of the factory.
  */
 export function registerSurfaceRuntimePollers(

package/src/runtime/index.ts

@@ -9,7 +9,7 @@
 // No runtime namespace objects are imported as values here: an eager
 // `export const X = ns.X` compiles to a top-level property read off a lazy
 // namespace object, and Bun's single-file compiler orders module bodies
-// nondeterministically — on some builds the read lands before the defining
+// nondeterministically, on some builds the read lands before the defining
 // module and the compiled binary dies at load with a ReferenceError. Every
 // value below is a grouped live re-export from the SDK's registered runtime
 // subpaths instead (exactly how the operations block below fixed the first
@@ -73,7 +73,7 @@ export type {
   WorkflowEvent,
 } from '@pellux/goodvibes-sdk/events';
 
-// Bootstrap compatibility aliases — grouped live re-exports (see the header
+// Bootstrap compatibility aliases, grouped live re-exports (see the header
 // comment for why these must not be eager namespace reads).
 export {
   scheduleBackgroundMcpDiscovery,
@@ -135,7 +135,7 @@ export type OperatorClient = Bootstrap.OperatorClient;
 export type PeerClient = Bootstrap.PeerClient;
 export type OpsApi = Bootstrap.OpsApi;
 
-// Transport compatibility aliases — grouped live re-exports.
+// Transport compatibility aliases, grouped live re-exports.
 export {
   createDirectTransport,
   createDirectTransportFromServices,
@@ -173,8 +173,8 @@ export {
   GlobalNetworkTransportInstaller,
   inspectOutboundTls,
 } from '@pellux/goodvibes-sdk/platform/runtime/transport';
-// Re-exported here because bootstrap.ts — the one place an exit can await
-// anything — sits on the 800-line per-file gate, so it reaches this through the
+// Re-exported here because bootstrap.ts, the one place an exit can await
+// anything, sits on the 800-line per-file gate, so it reaches this through the
 // import block it already has rather than a second import line.
 export { leaveHostedSessionOnExit } from './client/hosted-exit.ts';
 
@@ -218,7 +218,7 @@ export type SerializedRuntimeEnvelope = Transport.SerializedRuntimeEnvelope;
 // `export const X = operations.X` module-scope reads off the `operations`
 // namespace object: those reads evaluated the namespace getter while the
 // compiled single-file bundle could still be mid-cycle, and the binding they
-// reached for was not defined yet — source execution hid this, the compiled
+// reached for was not defined yet, source execution hid this, the compiled
 // binary died on it at load. A grouped `export { ... } from '<subpath>'` is a
 // live binding resolved by the module system, not a module-scope value read,
 // so it is cycle-safe.
@@ -314,7 +314,7 @@ export {
 } from '@pellux/goodvibes-sdk/platform/runtime/operations';
 // Snapshot-retention symbols (SnapshotPruner, RetentionPolicy,
 // DEFAULT_RETENTION_CONFIG) are intentionally NOT re-exported here. No app code
-// consumes them — only the retention unit test does, and it now imports them
+// consumes them, only the retention unit test does, and it now imports them
 // straight from the SDK `operations` namespace. Re-exporting them created a
 // second top-level binding named `SnapshotPruner` that collided with the SDK's
 // own `class SnapshotPruner`, forcing the bundler to rename ours to
@@ -333,13 +333,13 @@ export {
 // OpsTargetNotFoundError, ToolContractVerifier, McpLifecycleManager,
 // McpPermissionManager, McpSchemaFreshnessTracker, PluginLifecycleManager,
 // PluginQuarantineEngine, PluginTrustStore, RemoteRunnerRegistry,
-// RemoteSupervisor, DistributedRuntimeManager) — their instance types are
+// RemoteSupervisor, DistributedRuntimeManager), their instance types are
 // forwarded automatically and are not redeclared below.
 // The declare-once storage handle (platform/runtime/session-surface.ts) plus
 // the two prompted-recovery primitives that only accept it. Every session
 // path in this app derives from one surface built in runtime/services.ts.
 export type SessionSurface = Operations.SessionSurface;
-// Probe for a NAMED session's live snapshot (per-session supersession — see
+// Probe for a NAMED session's live snapshot (per-session supersession, see
 // session-recovery.ts's header) without retiring anything. Backs the
 // --continue / --resume pre-resume check in cli/tui-startup.ts.
 export type RecoveryFileInfo = Operations.RecoveryFileInfo;
@@ -421,7 +421,7 @@ export type ToolExecutionPhase = Operations.ToolExecutionPhase;
 export type PhaseResult = Operations.PhaseResult;
 export type ToolExecutionRecord = Operations.ToolExecutionRecord;
 
-// Runtime shell compatibility aliases — grouped live re-exports.
+// Runtime shell compatibility aliases, grouped live re-exports.
 // WorktreeRegistry is a class: the value re-export carries its instance type,
 // so the old separate `export type WorktreeRegistry` alias is gone (TS2484).
 export {
@@ -486,7 +486,7 @@ export type EcosystemCatalogBundle = Shell.EcosystemCatalogBundle;
 export type EcosystemCatalogEntry = Shell.EcosystemCatalogEntry;
 export type EcosystemEntryKind = Shell.EcosystemEntryKind;
 
-// Runtime security compatibility aliases — grouped live re-exports. The class
+// Runtime security compatibility aliases, grouped live re-exports. The class
 // re-exports (DivergenceDashboard, PolicyRegistry, …) carry their instance
 // types, so the old separate type companions for those names are gone.
 export {

package/src/runtime/install-self-check-startup.ts

@@ -1,5 +1,5 @@
 /**
- * install-self-check-startup.ts — the boot-time surfacing of the install
+ * install-self-check-startup.ts, the boot-time surfacing of the install
  * self-check (install-self-check.ts). Called once from the runtime bootstrap,
  * it wires the real host inputs (process exec path, the packaged root, the
  * real daemon resolution, and existsSync) into the pure evaluator and prints
@@ -32,7 +32,7 @@ export function announceInstallSelfCheck(router: SystemMessageRouter): void {
       router.high(`[Install] ${finding.summary} ${finding.detail} Repair: ${finding.repairCommand}`);
     }
   } catch {
-    // Best-effort — an install self-check must never block or crash boot.
+    // Best-effort, an install self-check must never block or crash boot.
   }
 }
 
@@ -42,7 +42,7 @@ export function announceInstallSelfCheck(router: SystemMessageRouter): void {
  * current release (path-shadow-startup).
  *
  * They are announced together because they are the same question from the
- * user's side — "am I looking at the build I think I am" — and because the
+ * user's side, "am I looking at the build I think I am", and because the
  * three answers are useless apart: a complete install nobody can reach, or a
  * reachable install three versions old, both end with the product appearing
  * to have lost a capability it never had.
@@ -54,6 +54,6 @@ export function announceInstallSelfCheck(router: SystemMessageRouter): void {
 export function announceInstallHealth(router: SystemMessageRouter): void {
   announceInstallSelfCheck(router);
   void announceInstallReachability(router).catch(() => {
-    // Best-effort — a reachability check must never block or crash boot.
+    // Best-effort, a reachability check must never block or crash boot.
   });
 }

package/src/runtime/install-self-check.ts

@@ -1,5 +1,5 @@
 /**
- * install-self-check.ts — cheap, non-fatal detection of an incomplete
+ * install-self-check.ts, cheap, non-fatal detection of an incomplete
  * install, run at TUI startup and re-surfaced by `goodvibes doctor`.
  *
  * A packaged install (the standalone `install.sh` binary, or the
@@ -12,16 +12,16 @@
  *   2. the daemon launch path resolves to nothing runnable, so any
  *      daemon/control-plane/listener/web surface silently fails to start.
  *
- * This module decides those two facts from injected inputs only — an
+ * This module decides those two facts from injected inputs only, an
  * install kind, a package root, the resolved daemon executable, and a
- * `fileExists` predicate — so it is pure and testable with fake paths and
+ * `fileExists` predicate, so it is pure and testable with fake paths and
  * never touches the network. The production wiring (real existsSync + the
  * real daemon resolution) lives in install-self-check-startup.ts and in the
  * CLI status/doctor entrypoint; this file only makes the judgement.
  *
  * A `source` checkout run via `bun src/main.ts` is NOT an incomplete
- * install — there is no vendor directory to populate and the daemon runs
- * from source — so it is never flagged.
+ * install, there is no vendor directory to populate and the daemon runs
+ * from source, so it is never flagged.
  *
  * Install-kind detection and the per-kind repair one-liner are reused from
  * update-check.ts rather than re-derived here.
@@ -39,7 +39,7 @@ import { resolveArtifactNames } from './release-artifacts.ts';
  */
 export interface DaemonPathResolution {
   readonly command: string;
-  /** How the path was resolved. 'fallback' means no runnable file was found — a bare PATH command. */
+  /** How the path was resolved. 'fallback' means no runnable file was found, a bare PATH command. */
   readonly source: string;
   readonly absolute: boolean;
 }

package/src/runtime/interaction-seams.ts

@@ -1,5 +1,5 @@
 // ---------------------------------------------------------------------------
-// interaction-seams.ts — the command-context interaction seams and the memory-
+// interaction-seams.ts, the command-context interaction seams and the memory-
 // provenance UI controller, extracted from main.ts (file-size hygiene).
 //
 // The seams expose the in-process orchestrator's per-tool cancel and mid-turn
@@ -39,7 +39,7 @@ export interface InteractionSeamDeps {
 /** Wire the per-tool cancel, queue edit/delete, power, and memory-provenance seams onto the command context. */
 export function wireInteractionSeams(cc: CommandContext, deps: InteractionSeamDeps): void {
   cc.cancelToolCall = createCancelToolCall(deps.orchestrator, deps.getActiveToolCallId, () => {
-    deps.notify('[Tool] Cancelled the running tool call — the turn continues.');
+    deps.notify('[Tool] Cancelled the running tool call; the turn continues.');
     deps.render();
   });
   cc.listQueuedMessages = () => deps.orchestrator.listQueuedMessages();
@@ -56,7 +56,7 @@ export interface MemoryProvenanceUi {
   toggle(): void;
   /** Capture the memory record ids from a completed turn's payload (metadata.memory.recordIds). */
   onTurnCompleted(evt: unknown): void;
-  /** The chip lines for the current turn — empty when the setting is off or the turn used no memories. */
+  /** The chip lines for the current turn, empty when the setting is off or the turn used no memories. */
   renderChip(width: number, configManager: Pick<ConfigManager, 'get'>): Line[];
 }
 
@@ -67,7 +67,7 @@ type ResolvedRecord = { readonly summary: string; readonly cls: string } | null;
  * Create the memory-provenance chip controller (default OFF; reads nothing when
  * off). When the drill-in is expanded it RESOLVES each memory record id to its
  * summary through the memory spine (the same host-or-daemon access the recall
- * commands use, so an adopted daemon resolves over the wire) — per-id and
+ * commands use, so an adopted daemon resolves over the wire), per-id and
  * lazily, so one forgotten record never blanks the rest and a collapsed chip
  * costs nothing. Mirrors the webui's provenance detail fetch.
  */

package/src/runtime/knowledge-services.ts

@@ -1,11 +1,11 @@
 // ---------------------------------------------------------------------------
-// knowledge-services.ts — the knowledge/wiki + home-graph stack (TUI wiring)
+// knowledge-services.ts, the knowledge/wiki + home-graph stack (TUI wiring)
 //
 // Constructs the three KnowledgeStores (regular wiki, agent, home-graph), their
 // semantic services, the ingest services, the home-graph service, and the
 // project-planning + work-plan stores. Extracted into its own module rather
 // than built inline in services.ts, which sits at the architecture check's
-// 800-line cap (scripts/check-architecture.ts) — new/large construction blocks
+// 800-line cap (scripts/check-architecture.ts), new/large construction blocks
 // get their own module and a single wiring call there (mirrors
 // createWorkstreamServices / createDurabilityServices / createCodeIndexServices).
 //