@pellux/goodvibes-tui 0.19.6 → 0.19.9

package/CHANGELOG.md

@@ -4,6 +4,42 @@ All notable changes to GoodVibes TUI.
 
 ---
 
+## [0.19.9] - 2026-04-19
+
+### Fixed
+- Regenerated `docs/foundation-artifacts/operator-contract.json` after the 0.19.8 SDK bump; the foundation-artifacts-gate test failed on the 0.19.8 CI because the JSON was not regenerated. 0.19.8 tag has no npm artifact; 0.19.9 supersedes.
+## [0.19.8] - 2026-04-19
+
+### Changed
+- Upgraded `@pellux/goodvibes-sdk` from 0.21.10 to 0.21.15. Carries the companion main-chat routing fix: `POST /api/sessions/:id/messages` with `kind: 'message'` now short-circuits the daemon's WRFC engineer-chain fall-through. The TUI wires `orchestrator.handleUserInput()` into the `COMPANION_MESSAGE_RECEIVED` runtime bus subscriber, so companion main-chat sends fire a real LLM turn via the same entry point as the TUI input box. Turn `STREAM_DELTA` / `TURN_COMPLETED` events stream back to both TUI and companion SSE.
+
+### Fixed
+- Companion main-chat no longer produces engineer-agent acknowledgement boilerplate ("Update noted", "WRFC chain has passed all gates"). The message is received by the daemon, persisted, rendered in the TUI, and answered by a normal chat turn.
+## [0.19.7] - 2026-04-18
+
+### Changed
+- Bumped `@pellux/goodvibes-sdk` from 0.21.10 to 0.21.12.
+
+### Investigation: No TUI-side changes required
+
+SDK 0.21.12 ships three additive changes to the daemon:
+
+1. **`kind: 'message'` at `POST /api/sessions/:id/messages`** now calls `sessionBroker.submitMessage()` (previously called `appendCompanionMessage` + `publishConversationFollowup` only). A real turn is started and the full agent event chain (TURN_STARTED, STREAM_DELTA, TURN_COMPLETED) streams back.
+
+2. **`kind: 'followup'`** — new kind value that always spawns an agent. Full agent event chain streams to companion SSE.
+
+3. **`GET /api/sessions/:id/events`** — new session-scoped SSE endpoint for companion consumption.
+
+4. **`'turn'` added to `DEFAULT_DOMAINS`** in the control-plane gateway — turn events are now auto-delivered to all SSE subscribers.
+
+Investigation of the TUI's `COMPANION_MESSAGE_RECEIVED` subscription in `src/runtime/bootstrap-core.ts` found **no double-render risk**: SDK 0.21.12 emits `COMPANION_MESSAGE_RECEIVED` _before_ calling `submitMessage`, so the user message is shown once (via the existing subscription), and the AI response streams in via the existing `uiServices.events.turns` path. These are distinct content on distinct code paths. No deduplication needed.
+
+The `'turn'` addition to `DEFAULT_DOMAINS` affects external SSE subscribers (companion app). The TUI consumes turn events in-process via the runtime bus — unaffected.
+
+Updated `docs/foundation-artifacts/operator-contract.json` SDK version field to `0.21.12`.
+
+---
+
 ## [0.19.6] - 2026-04-18
 
 ### Fixed

package/README.md

@@ -2,7 +2,7 @@
 
 [![CI](https://github.com/mgd34msu/goodvibes-tui/actions/workflows/ci.yml/badge.svg)](https://github.com/mgd34msu/goodvibes-tui/actions/workflows/ci.yml)
 [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://opensource.org/licenses/MIT)
-[![Version](https://img.shields.io/badge/version-0.19.6-blue.svg)](https://github.com/mgd34msu/goodvibes-tui)
+[![Version](https://img.shields.io/badge/version-0.19.9-blue.svg)](https://github.com/mgd34msu/goodvibes-tui)
 
 A terminal-native AI coding, operations, automation, knowledge, and integration console with a typed runtime, omnichannel surfaces, structured memory/knowledge, and a raw ANSI renderer.
 

package/docs/foundation-artifacts/operator-contract.json

@@ -3,7 +3,7 @@
   "product": {
     "id": "goodvibes",
     "surface": "operator",
-    "version": "0.21.10"
+    "version": "0.21.15"
   },
   "auth": {
     "modes": [

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pellux/goodvibes-tui",
-  "version": "0.19.6",
+  "version": "0.19.9",
   "description": "Terminal-native GoodVibes product for coding, operations, automation, knowledge, channels, and daemon-backed control-plane workflows.",
   "type": "module",
   "main": "src/main.ts",
@@ -89,7 +89,7 @@
     "@anthropic-ai/vertex-sdk": "^0.16.0",
     "@ast-grep/napi": "^0.42.0",
     "@aws/bedrock-token-generator": "^1.1.0",
-    "@pellux/goodvibes-sdk": "0.21.10",
+    "@pellux/goodvibes-sdk": "0.21.15",
     "bash-language-server": "^5.6.0",
     "fuse.js": "^7.1.0",
     "graphql": "^16.13.2",

package/src/runtime/bootstrap-core.ts

@@ -50,6 +50,12 @@ export interface BootstrapCoreState {
   readonly permissionPromptRef: { requestPermission: PermissionRequestHandler };
   readonly systemMessageRouterRef: { value: SystemMessageRouter | null };
   readonly conversationFollowUpRef: { value: ((item: ConversationFollowUpItem) => void) | null };
+  /**
+   * Mutable ref patched by bootstrap.ts after the Orchestrator is constructed.
+   * When non-null, COMPANION_MESSAGE_RECEIVED fires a real LLM turn via
+   * orchestrator.handleUserInput() instead of only appending the user message.
+   */
+  readonly orchestratorHandleUserInputRef: { value: ((text: string) => void) | null };
   readonly requestRender: () => void;
   readonly setRenderRequest: (fn: () => void) => void;
   readonly runtimeSessionIdRef: { value: string };
@@ -286,14 +292,30 @@ export async function initializeBootstrapCore(
     wrfcController: services.wrfcController,
   });
 
-  // Subscribe to companion follow-up messages received from the daemon's HTTP layer.
-  // The daemon emits COMPANION_MESSAGE_RECEIVED on the runtime bus (SDK 0.21.10+)
-  // after persisting the message, so the TUI conversation view can render it immediately.
+  // Subscribe to companion main-chat messages received from the daemon's HTTP layer.
+  // The daemon emits COMPANION_MESSAGE_RECEIVED on the runtime bus when a companion
+  // POST /api/sessions/:id/messages with kind='message' arrives.
+  //
+  // bootstrap.ts patches orchestratorHandleUserInputRef.value after the Orchestrator
+  // is constructed. When that ref is set, we delegate to orchestrator.handleUserInput()
+  // which (a) adds the user message to the conversation view and (b) fires a real LLM
+  // turn whose STREAM_DELTA / TURN_COMPLETED events flow to both TUI and companion SSE.
+  //
+  // The fallback (ref not yet set) adds the message to the conversation view only —
+  // this path is unreachable in practice because the event bus is not connected to
+  // any live HTTP traffic until after the orchestrator is wired in bootstrap.ts.
+  const orchestratorHandleUserInputRef: { value: ((text: string) => void) | null } = { value: null };
   runtimeUnsubs.push(runtimeBus.on<Extract<SessionEvent, { type: 'COMPANION_MESSAGE_RECEIVED' }>>(
     'COMPANION_MESSAGE_RECEIVED',
     ({ payload }) => {
-      conversation.addUserMessage(payload.body);
-      requestRender();
+      if (orchestratorHandleUserInputRef.value) {
+        // Delegate to the orchestrator: adds user message + fires a real LLM turn.
+        orchestratorHandleUserInputRef.value(payload.body);
+      } else {
+        // Fallback: render the user message immediately (orchestrator not yet ready).
+        conversation.addUserMessage(payload.body);
+        requestRender();
+      }
     },
   ));
 
@@ -418,6 +440,7 @@ export async function initializeBootstrapCore(
     permissionPromptRef,
     systemMessageRouterRef,
     conversationFollowUpRef,
+    orchestratorHandleUserInputRef,
     requestRender,
     setRenderRequest: (fn) => {
       renderRequestRef.value = fn;

package/src/runtime/bootstrap.ts

@@ -155,6 +155,7 @@ export async function bootstrapRuntime(
     permissionPromptRef,
     systemMessageRouterRef,
     conversationFollowUpRef,
+    orchestratorHandleUserInputRef,
     requestRender,
     setRenderRequest,
     runtimeSessionIdRef,
@@ -199,6 +200,12 @@ export async function bootstrapRuntime(
     },
   });
   conversationFollowUpRef.value = (item) => orchestrator.enqueueConversationFollowUp(item);
+  // Wire orchestratorHandleUserInputRef so COMPANION_MESSAGE_RECEIVED fires a real LLM turn.
+  orchestratorHandleUserInputRef.value = (text: string) => {
+    orchestrator.handleUserInput(text).catch((err: unknown) => {
+      logger.debug('companion handleUserInput safety catch', { error: String(err) });
+    });
+  };
   orchestrator.setCoreServices({
     configManager,
     providerRegistry,

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