@pellux/goodvibes-tui 1.9.0 → 1.9.2

package/src/runtime/process-lifecycle.ts

@@ -57,7 +57,7 @@ export interface ProcessLifecycleDeps {
   readonly unsubs: ReadonlyArray<() => void>;
   readonly getRecoveryInterval: () => ReturnType<typeof setInterval> | null;
   readonly setRecoveryInterval: (value: ReturnType<typeof setInterval> | null) => void;
-  readonly getStopSpokenOutputForExit: () => (() => void) | null;
+  readonly getStopSpokenOutputForExit: () => (() => void | Promise<void>) | null;
 }
 
 export interface ProcessLifecycleHandlers {
@@ -153,7 +153,16 @@ export function installProcessLifecycle(deps: ProcessLifecycleDeps): ProcessLife
     // must not re-run teardown or double-fire ctx.shutdown.
     if (exiting) return;
     exiting = true;
-    getStopSpokenOutputForExit()?.();
+    // Exit lets the spoken audio the user is already hearing finish inside a
+    // short bounded window (capped inside stopForExit, under the 3s shutdown
+    // budget below) while the rest of teardown proceeds; queued-but-unplayed
+    // speech is dropped. Deliberate interrupts (Ctrl+C, /tts stop) still cut
+    // instantly through controller.stop() before this path is ever reached.
+    let spokenOutputDrain: Promise<void> = Promise.resolve();
+    try {
+      spokenOutputDrain = Promise.resolve(getStopSpokenOutputForExit()?.()).then(() => undefined);
+    } catch { /* non-fatal to exit */ }
+    spokenOutputDrain = spokenOutputDrain.catch(() => undefined);
     unsubs.forEach(fn => fn());
     const interval = getRecoveryInterval();
     if (interval !== null) { clearInterval(interval); setRecoveryInterval(null); }
@@ -169,9 +178,14 @@ export function installProcessLifecycle(deps: ProcessLifecycleDeps): ProcessLife
     try {
       // Race the graceful shutdown against a hard timeout — externalServices.stop() can hang
       // and we must still exit; deferredStartup.drain only budgets 100ms internally.
-      await Promise.race([
-        ctx.shutdown({ ...snapshot, ...buildPersistedSessionContext(snapshot.messages, ctx.conversation.getTitleSource(), buildSessionContinuityHints()) }).then(() => { shutdownOk = true; }),
-        new Promise<void>((resolve) => setTimeout(resolve, 3000)),
+      // The spoken-audio drain runs concurrently and is internally capped below
+      // this budget, so it never extends the exit beyond the hard timeout.
+      await Promise.all([
+        spokenOutputDrain,
+        Promise.race([
+          ctx.shutdown({ ...snapshot, ...buildPersistedSessionContext(snapshot.messages, ctx.conversation.getTitleSource(), buildSessionContinuityHints()) }).then(() => { shutdownOk = true; }),
+          new Promise<void>((resolve) => setTimeout(resolve, 3000)),
+        ]),
       ]);
     } catch (err) {
       logger.debug('ctx.shutdown error during exitApp (non-fatal)', { error: summarizeError(err) });

package/src/runtime/services.ts

@@ -222,12 +222,12 @@ export interface RuntimeServices {
   readonly agentOrchestrator: AgentOrchestrator;
   readonly wrfcController: WrfcController;
   readonly processManager: ProcessManager;
-  /** Wave 4 (wo703): the phase/work-item orchestration engine — see runtime/workstream-services.ts. */
+  /** The phase/work-item orchestration engine — see runtime/workstream-services.ts. */
   readonly orchestrationEngine: OrchestrationEngine;
   readonly workstreamCommands: WorkstreamCommandService;
-  /** Wave 5 (wo804): the repo source-tree code index — see runtime/code-index-services.ts. */
+  /** The repo source-tree code index — see runtime/code-index-services.ts. */
   readonly codeIndexStore: CodeIndexStore;
-  readonly codeIndexReindexScheduler: CodeIndexReindexScheduler; // Wave-5 Stage B tool-site reindex
+  readonly codeIndexReindexScheduler: CodeIndexReindexScheduler; // tool-site reindex
   /** W2.1/W2.2: unified live process registry (agents, WRFC chains, workflows, watchers, background processes) backing the Fleet panel. */
   readonly processRegistry: ProcessRegistry;
   readonly modeManager: ModeManager;
@@ -338,7 +338,7 @@ export function createRuntimeServices(options: RuntimeServicesOptions): RuntimeS
     executor: agentOrchestrator,
     configManager,
   });
-  agentOrchestrator.setConversationSink({ // Wave-3 Part C6 bridge (mirrors the SDK's own createRuntimeServices)
+  agentOrchestrator.setConversationSink({ // Conversation-snapshot bridge (mirrors the SDK's own createRuntimeServices)
     register: (agentId, source) => agentManager.registerConversationSource(agentId, source),
     release: (agentId) => agentManager.releaseConversationSource(agentId),
   });
@@ -387,7 +387,7 @@ export function createRuntimeServices(options: RuntimeServicesOptions): RuntimeS
   });
   const artifactStore = new ArtifactStore({ configManager });
   const memoryEmbeddingRegistry = new MemoryEmbeddingProviderRegistry({ configManager });
-  // W6-C2 (E6): open the ONE home-scoped canonical store; legacy per-project TUI memory folds in at boot (foldTuiLegacyMemory).
+  // Open the ONE home-scoped canonical store; legacy per-project TUI memory folds in at boot (foldTuiLegacyMemory).
   const memoryDbPath = resolveCanonicalMemoryDbPath(homeDirectory);
   const memoryStore = new MemoryStore(memoryDbPath, {
     embeddingRegistry: memoryEmbeddingRegistry,
@@ -595,18 +595,18 @@ export function createRuntimeServices(options: RuntimeServicesOptions): RuntimeS
     artifactStore,
   });
   const processManager = new ProcessManager();
-  // Wave 4 (wo703): the phase/work-item orchestration engine, constructed
+  // The phase/work-item orchestration engine, constructed
   // before the process registry so its fleet nodes (workstream/phase/
   // work-item) can be folded in below via the registry's optional
   // orchestrationEngine dep.
   const { orchestrationEngine, workstreamCommands } = createWorkstreamServices({
     agentManager, configManager, adaptivePlanner, runtimeBus: options.runtimeBus, projectRoot: workingDirectory,
   });
-  // Wave 5 (wo804): repo source-tree code index, sharing memoryEmbeddingRegistry
+  // Repo source-tree code index, sharing memoryEmbeddingRegistry
   // with MemoryStore above. Auto-build is config-gated (default off) — see
   // code-index-services.ts's header doc.
   const { codeIndexStore, codeIndexReindexScheduler } = createCodeIndexServices({ workingDirectory, configManager, memoryEmbeddingRegistry });
-  const codeInjectionOrchestratorDeps = { codeIndex: codeIndexStore, isCodeInjectionSettingEnabled: () => isCodeInjectionSettingEnabled(configManager), codeIndexReindexScheduler }; // Wave-5 Stage B seam (agent here; main via orchestrator-core-services.ts)
+  const codeInjectionOrchestratorDeps = { codeIndex: codeIndexStore, isCodeInjectionSettingEnabled: () => isCodeInjectionSettingEnabled(configManager), codeIndexReindexScheduler }; // Code-injection seam (agent here; main via orchestrator-core-services.ts)
   // W2.1/W2.2: one shared process registry aggregating the managers above —
   // the Fleet panel (panels/fleet-read-model.ts) is its first consumer.
   // Constructed once here (not per-consumer) so the coalesced tick and the
@@ -614,15 +614,15 @@ export function createRuntimeServices(options: RuntimeServicesOptions): RuntimeS
   const processRegistry = createProcessRegistry({
     agentManager,
     wrfcController,
-    orchestrationEngine, // Wave 4 (wo703): folds workstream/phase/work-item nodes into the fleet
-    codeIndexService: codeIndexStore, // Wave 5 (wo804): folds a single 'code-index' node into the fleet
+    orchestrationEngine, // Folds workstream/phase/work-item nodes into the fleet
+    codeIndexService: codeIndexStore, // Folds a single 'code-index' node into the fleet
     processManager,
     watcherRegistry,
     workflow,
     approvalBroker,
     sessionBroker,
-    messageBus: agentMessageBus, // Wave-3: backs steer()/`steerable` (wo612 builds the composer UI on top)
-    automationManager, // Wave 6 (wo-F item d4): folds /schedule AutomationJobs into the fleet as 'schedule' nodes
+    messageBus: agentMessageBus, // Backs steer()/`steerable` (the Fleet steer composer builds on top)
+    automationManager, // Folds /schedule AutomationJobs into the fleet as 'schedule' nodes
     runtimeBus: options.runtimeBus,
     // Honest pricing: never fabricate a cost for an unrecognized model.
     priceUsage: (model, usage) => {
@@ -681,7 +681,7 @@ export function createRuntimeServices(options: RuntimeServicesOptions): RuntimeS
     remoteRunnerRegistry,
     knowledgeService,
     memoryRegistry,
-    ...codeInjectionOrchestratorDeps, // Wave-5 Stage B: agent-run code injection + tool-site reindex
+    ...codeInjectionOrchestratorDeps, // Agent-run code injection + tool-site reindex
     archetypeLoader,
     configManager,
     providerRegistry,
@@ -790,7 +790,7 @@ export function createRuntimeServices(options: RuntimeServicesOptions): RuntimeS
     workspaceCheckpointManager,
     integrationHelpers,
     async rerootStores(newWorkingDir: string): Promise<void> {
-      // W6-C2 (E6): the memory store is the home-scoped canonical cross-surface store and
+      // The memory store is the home-scoped canonical cross-surface store and
       // deliberately does NOT reroot per-project (that would re-silo memory, the E6
       // regression). Only working-tree-bound stores (code index, project index) reroot.
       await codeIndexStore.reroot(newWorkingDir, codeIndexDbPath(newWorkingDir));

package/src/runtime/session-inbound-inputs.ts

@@ -1,7 +1,7 @@
 /**
  * session-inbound-inputs.ts
  *
- * D3 (One-Platform Wave 2, live-surface steer delivery — the inbound half of the
+ * Live-surface steer delivery (the inbound half of the
  * session spine): the TUI's INBOUND path for steer/follow-up inputs that another
  * live surface (e.g. the webui) queued against THIS TUI's session.
  *

package/src/runtime/session-spine-transport.ts

@@ -1,10 +1,10 @@
 /**
  * session-spine-transport.ts
  *
- * W3-T1 (One-Platform Wave 3, TUI session-spine cutover): the thin typed-client
+ * The TUI session-spine cutover: the thin typed-client
  * adapter that lets the TUI's bootstrap drive the SDK's
  * `@pellux/goodvibes-sdk/platform/runtime/session-spine` `SessionSpineClient`
- * (extracted in W3-S4) via its injected `SpineTransport` seam.
+ * via its injected `SpineTransport` seam.
  *
  * The TUI is SDK-clean here: `SpineSessionsClient` is exactly the narrow wire
  * surface the old TUI-local `session-spine-client.ts` declared (structurally

package/src/runtime/ui-services.ts

@@ -114,7 +114,7 @@ export interface UiRuntimeSharedServices {
     readonly remoteSupervisor: RuntimeServices['remoteSupervisor'] & RemoteSupervisor;
     /** W2.2: the shared live process registry backing the Fleet panel. */
     readonly processRegistry: RuntimeServices['processRegistry'];
-    /** Wave-3 (W3.2): the shared runtime event bus — the Fleet panel subscribes to its 'communication' domain for the honest steer-consumed signal. */
+    /** The shared runtime event bus — the Fleet panel subscribes to its 'communication' domain for the honest steer-consumed signal. */
     readonly runtimeBus: RuntimeServices['runtimeBus'];
   };
 }

package/src/runtime/workstream-services.ts

@@ -1,5 +1,5 @@
 // ---------------------------------------------------------------------------
-// workstream-services.ts — Wave 4 (wo703)
+// workstream-services.ts — phase/work-item orchestration engine
 //
 // Constructs the TUI's OrchestrationEngine instance (@pellux/goodvibes-sdk/
 // platform/orchestration, landed on SDK main as wo701/W4.1) and a thin

package/src/runtime/wrfc-persistence.ts

@@ -19,7 +19,7 @@
  *    additionally exposes the recovered set for inspection.
  *
  *    A chain that looked interrupted (non-terminal) in the old snapshot can be
- *    reaped to terminal IN PLACE by `WrfcController.importChain` (Wave 6, wo-F:
+ *    reaped to terminal IN PLACE by `WrfcController.importChain` (the zombie-reap pass:
  *    no member agent survived the restart — see wrfc-controller.ts). This
  *    module always re-checks `chain.state` AFTER the import call, never the
  *    pre-import classification, so a reaped chain is treated as history, not
@@ -209,7 +209,7 @@ class WrfcPersistenceImpl implements WrfcPersistence {
       // chain. The accessor is optional for read-only test doubles.
       //
       // NOTE: importChain may reap this chain to a terminal state IN PLACE
-      // (Wave 6, wo-F zombie-reap: no member agent survived the restart) —
+      // (zombie reap: no member agent survived the restart) —
       // always read chain.state AFTER this call below, never the pre-import
       // classification captured by candidateInterrupted.
       this.controller.importChain?.(chain);

package/src/shell/ui-openers.ts

@@ -231,7 +231,7 @@ export function wireShellUiOpeners(options: WireShellUiOpenersOptions): void {
 
   commandContext.openModelPicker = () => {
     void (async () => {
-      // D5: getSelectableModels() is catalog-driven and can list models whose
+      // getSelectableModels() is catalog-driven and can list models whose
       // `provider` id (e.g. 'google', sourced from the pricing catalog) was never
       // handed to providerRegistry.register()/registerRuntimeProvider(). Selecting
       // such a model fails hard at turn time with ProviderNotFoundError
@@ -264,7 +264,7 @@ export function wireShellUiOpeners(options: WireShellUiOpenersOptions): void {
 
   commandContext.openProviderPicker = () => {
     void (async () => {
-      // D5: listModels() surfaces every catalog provider id, not just the ones
+      // listModels() surfaces every catalog provider id, not just the ones
       // actually registered on this runtime (see openModelPicker above for the
       // same class of bug). Intersect against providerRegistry.has() so the
       // provider picker never offers a provider that will fail with

package/src/version.ts

@@ -6,7 +6,7 @@ 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 = '1.9.0';
+let _version = '1.9.2';
 try {
   const pkg = JSON.parse(readFileSync(join(import.meta.dir, '..', 'package.json'), 'utf-8'));
   _version = pkg.version ?? _version;