@pellux/goodvibes-tui 1.19.3 → 1.19.4

package/CHANGELOG.md

@@ -4,6 +4,15 @@ All notable changes to GoodVibes TUI.
 
 ---
 
+## [1.19.4] - 2026-07-17
+
+### Changes
+
+Fixed: selecting a custom-provider model no longer crashes the app at the next startup — boot now waits for custom providers to load before resolving the current model, and falls back to a selectable model (with a logged warning) if the configured provider's file was removed.
+Fixed: startup failures now print a clear "goodvibes failed to start" message with the real error and exit, instead of hanging on a blank screen while the log recorded an empty error object.
+Fixed: after an automatic context compaction, the transcript shows a compact expandable "compaction handoff" block instead of re-printing the entire re-injected instruction text every time.
+Updated: @pellux/goodvibes-sdk and @pellux/goodvibes-terminal-shell to 1.11.3 (quieter startup when no hooks file exists; error summaries no longer mangle redacted paths).
+
 ## [1.19.3] - 2026-07-17
 
 ### Changes

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-1.19.3-blue.svg)](https://github.com/mgd34msu/goodvibes-tui)
+[![Version](https://img.shields.io/badge/version-1.19.4-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": "1.11.2"
+    "version": "1.11.3"
   },
   "auth": {
     "modes": [

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pellux/goodvibes-tui",
-  "version": "1.19.3",
+  "version": "1.19.4",
   "description": "Terminal-native GoodVibes product for coding, operations, automation, knowledge, channels, and daemon-backed control-plane workflows.",
   "type": "module",
   "main": "src/main.ts",
@@ -104,8 +104,8 @@
     "@anthropic-ai/vertex-sdk": "^0.16.0",
     "@ast-grep/napi": "^0.42.0",
     "@aws/bedrock-token-generator": "^1.1.0",
-    "@pellux/goodvibes-sdk": "1.11.2",
-    "@pellux/goodvibes-terminal-shell": "1.11.2",
+    "@pellux/goodvibes-sdk": "1.11.3",
+    "@pellux/goodvibes-terminal-shell": "1.11.3",
     "bash-language-server": "^5.6.0",
     "fuse.js": "^7.1.0",
     "graphql": "^16.13.2",
@@ -131,10 +131,10 @@
     "typescript": "^5.9.3"
   },
   "optionalDependencies": {
-    "@pellux/goodvibes-tui-linux-x64": "1.19.3",
-    "@pellux/goodvibes-tui-linux-arm64": "1.19.3",
-    "@pellux/goodvibes-tui-darwin-x64": "1.19.3",
-    "@pellux/goodvibes-tui-darwin-arm64": "1.19.3"
+    "@pellux/goodvibes-tui-linux-x64": "1.19.4",
+    "@pellux/goodvibes-tui-linux-arm64": "1.19.4",
+    "@pellux/goodvibes-tui-darwin-x64": "1.19.4",
+    "@pellux/goodvibes-tui-darwin-arm64": "1.19.4"
   },
   "trustedDependencies": [],
   "overrides": {

package/src/cli/tui-startup.ts

@@ -5,6 +5,7 @@ import type { WorkspaceRegistrationManager } from '../runtime/trust/workspace-re
 import { hasResumableWizardProgress, readOnboardingCheckMarker, readWizardProgress } from '../runtime/onboarding/index.ts';
 import { startOnboardingFastPath } from '../runtime/onboarding/fast-path.ts';
 import { readLastSessionPointer } from '@/runtime/index.ts';
+import { logger } from '@pellux/goodvibes-sdk/platform/utils';
 import type { GoodVibesCliParseResult } from './types.ts';
 
 export type TuiStartupShellPaths = Parameters<typeof readOnboardingCheckMarker>[0] & {
@@ -173,3 +174,28 @@ export function applyInitialTuiCliState(options: {
     // request, not here — see main.ts's trustPromptRef wiring.
   }
 }
+
+/**
+ * reportFatalStartupError — the main() catch handler. A bare Error
+ * JSON-serializes to {} in structured logs, and background timers keep the
+ * process alive after a boot failure — the historical result was a blank
+ * screen and a useless `Fatal error {"error": {}}` log line. Log real
+ * fields, tell the user what broke, and exit. Both writes are individually
+ * best-effort — a failing logger or torn-down stderr must never hide the
+ * original launch failure.
+ */
+export function reportFatalStartupError(err: unknown): void {
+  const message = err instanceof Error ? err.message : String(err);
+  const stack = err instanceof Error ? err.stack : undefined;
+  try {
+    logger.error('Fatal error', { message, ...(stack ? { stack } : {}) });
+  } catch {
+    // Startup diagnostics must never hide the original launch failure.
+  }
+  try {
+    process.stderr.write(`goodvibes failed to start: ${message}\n${stack ? `${stack}\n` : ''}`);
+  } catch {
+    // Ignore secondary stderr failures during process teardown.
+  }
+  process.exit(1);
+}

package/src/core/conversation-line-cache.ts

@@ -224,7 +224,7 @@ function renderOne(
   cfg: RenderConfig,
 ): void {
   if (message.role === 'user') {
-    renderConversationUserMessage(ctx, message, width);
+    renderConversationUserMessage(ctx, message, width, absoluteIdx);
   } else if (message.role === 'assistant') {
     renderConversationAssistantMessage(ctx, message, width, cfg.lineNumberMode, cfg.collapseThreshold, absoluteIdx);
   } else if (message.role === 'system') {

package/src/core/conversation-rendering.ts

@@ -17,7 +17,7 @@ import { GLYPHS } from '../renderer/ui-primitives.ts';
 import type { BlockMeta } from './conversation-types.ts';
 import type { ConversationMessageSnapshot } from '@pellux/goodvibes-sdk/platform/core';
 import { parseDiffForApply } from '@pellux/goodvibes-sdk/platform/core';
-import { extractUserDisplayText } from '@pellux/goodvibes-sdk/platform/core';
+import { extractUserDisplayText, COMPACTION_HANDOFF_HEADER } from '@pellux/goodvibes-sdk/platform/core';
 // SystemMessageKind imported from runtime directly to avoid cycle:
 //   conversation-rendering.ts → system-message-router.ts → conversation.ts → conversation-rendering.ts
 import type { SystemMessageKind } from '@/runtime/index.ts';
@@ -88,6 +88,7 @@ export function renderConversationUserMessage(
   context: ConversationRenderContext,
   message: Extract<Message, { role: 'user' }>,
   width: number,
+  msgIdx?: number,
 ): void {
   const T = activeTheme();
   const displayText = extractUserDisplayText(message.content);
@@ -95,9 +96,74 @@ export function renderConversationUserMessage(
     context.history.addLines(UIFactory.createMessageBar(width, displayText, T.errorBarBg, '196', ' x ', true));
     return;
   }
+  // Compaction-continuation handoff: a user-ROLE message the compactor
+  // authored, not something the user typed. Rendered in full it repeats the
+  // entire re-injected instruction block after every automatic compaction —
+  // a multi-kilobyte wall in the transcript. Fold it like a tool result; the
+  // full payload stays reachable through the normal expand toggle.
+  if (msgIdx !== undefined && displayText.startsWith(COMPACTION_HANDOFF_HEADER)) {
+    renderCompactionContinuationMessage(context, displayText, width, msgIdx);
+    return;
+  }
   context.history.addLines(UIFactory.createMessageBar(width, displayText));
 }
 
+/** Fold a compaction-continuation user message to one header + preview line. */
+function renderCompactionContinuationMessage(
+  context: ConversationRenderContext,
+  content: string,
+  width: number,
+  msgIdx: number,
+): void {
+  const T = activeTheme();
+  const collapseKey = `msg_${msgIdx}`;
+  const blockIdx = context.blockRegistry.length;
+  const startLine = context.history.getLineCount();
+  const lineCount = content.split('\n').length;
+  const isCollapsed = context.collapseState.has(collapseKey)
+    ? context.collapseState.get(collapseKey)!
+    : true;
+  if (!context.collapseState.has(collapseKey)) {
+    context.collapseState.set(collapseKey, true);
+  }
+
+  context.history.addLine(renderConversationEventLine(width, {
+    marker: GLYPHS.status.active,
+    markerFg: T.toolAccent,
+    label: 'compaction handoff',
+    labelFg: T.toolAccent,
+    detailFg: '244',
+  }, [
+    { text: ` ${isCollapsed ? GLYPHS.navigation.collapsed : GLYPHS.navigation.expanded} ${lineCount} line${lineCount === 1 ? '' : 's'} `, fg: '244', dim: true },
+  ]));
+
+  if (isCollapsed) {
+    const rendered = renderConversationCollapsedFragment(
+      'compacted-context handoff (re-injected instructions + session summary)',
+      width,
+      {
+        prefix: ` ${GLYPHS.navigation.collapsed} `,
+        prefixFg: T.toolAccent,
+        text: '244',
+        bodyBg: T.collapsedBodyBg,
+        dim: true,
+      },
+    );
+    context.history.addLines(rendered);
+  } else {
+    context.history.addLines(renderMarkdownTracked(content, width).lines);
+  }
+
+  context.blockRegistry.push({
+    blockIndex: blockIdx,
+    collapseKey,
+    type: 'tool',
+    startLine,
+    lineCount: context.history.getLineCount() - startLine,
+    rawContent: content,
+  });
+}
+
 export function renderConversationAssistantMessage(
   context: ConversationRenderContext,
   message: Extract<Message, { role: 'assistant' }>,
@@ -400,7 +466,7 @@ export function appendConversationMessages(
     const absoluteIdx = msgIndexOffset + msgIdx;
     messageLineRegistry[absoluteIdx] = renderContext.history.getLineCount();
     if (message.role === 'user') {
-      renderConversationUserMessage(renderContext, message, width);
+      renderConversationUserMessage(renderContext, message, width, absoluteIdx);
     } else if (message.role === 'assistant') {
       renderConversationAssistantMessage(renderContext, message, width, lineNumberMode, collapseThreshold, absoluteIdx);
     } else if (message.role === 'system') {

package/src/main.ts

@@ -45,7 +45,7 @@ import { deriveComposerState } from './core/composer-state.ts';
 import { buildPersistedSessionContext, getReturnContextMode, maybeAssistReturnContextSummary } from '@/runtime/index.ts';
 import { summarizeError } from '@pellux/goodvibes-sdk/platform/utils';
 import { prepareShellCliRuntime } from './cli/entrypoint.ts';
-import { applyInitialTuiCliState } from './cli/tui-startup.ts';
+import { applyInitialTuiCliState, reportFatalStartupError } from './cli/tui-startup.ts';
 import { applyConfiguredHitlMode, applyRuntimeConfigValue, applyTuiRuntimeConfigDefaults } from './cli/config-overrides.ts';
 import { renderToolCallBlock } from './renderer/tool-call.ts';
 import { wireSpokenTurnRuntime } from './audio/spoken-turn-wiring.ts';
@@ -797,4 +797,4 @@ async function main() {
 
 }
 
-main().catch(err => logger.error('Fatal error', { error: err }));
+main().catch(reportFatalStartupError);

package/src/runtime/bootstrap.ts

@@ -37,12 +37,13 @@ import { pruneStaleOperatorTokens } from '@pellux/goodvibes-sdk/platform/pairing
 import { resolveDaemonCompanionToken, workspaceOperatorTokenCandidates } from './operator-token-cleanup.ts';
 import type { UiRuntimeServices } from './ui-services.ts';
 import { initializeBootstrapCore } from './bootstrap-core.ts';
+import { ensureBootModelResolvable } from './provider-fallback.ts';
 import { createBootstrapShell } from './bootstrap-shell.ts';
 import { announceResumeState } from './resume-notice.ts';
 import { announceInstallSelfCheck } from './install-self-check-startup.ts';
 import { buildSharedOrchestratorCoreServices, refreshMemoryRecallSnapshot } from './orchestrator-core-services.ts';
 import { consumeDaemonAttachNotices, consumeExternalDaemonAttachNotices } from './daemon-attach-notices.ts';
-import { createContextAccountingSource } from './context-accounting-source.ts';
+import { wireContextAccountingSource } from './context-accounting-source.ts';
 import { createEmbeddedServiceFactories } from './embedded-service-factories.ts';
 import { buildRelayExternalServiceMethods } from './relay-reachability-bridge.ts';
 import { startMcpConfigAutoReload } from '../mcp/runtime-reload.ts';
@@ -194,7 +195,8 @@ export async function bootstrapRuntime(
     panelManager,
     pluginManager,
   } = services;
-
+  // A saved custom-provider model must never crash boot — see provider-fallback.ts.
+  await ensureBootModelResolvable(providerRegistry, configManager);
   // A liveness flip is only PAINTED once something calls requestRender(); without
   // this, the footer's spine segment sat correct-but-undrawn until incidental activity redrew it (minutes, during an idle stretch).
   sessionUnionCache.setOnTransition(() => requestRender());
@@ -248,11 +250,9 @@ export async function bootstrapRuntime(
   conversation.setSessionLineageTracker(services.sessionLineageTracker);
 
   // Bind context_accounting's session source to the live Orchestrator (see context-accounting-source.ts).
-  const contextAccountingSourceHandle = createContextAccountingSource({
+  wireContextAccountingSource({
     orchestrator, providerRegistry, sessionLineageTracker: services.sessionLineageTracker, runtimeBus, sessionId: runtime.sessionId,
-  });
-  services.contextAccountingHolder.setSource(contextAccountingSourceHandle.source);
-  bootstrapUnsubs.push(contextAccountingSourceHandle.dispose);
+  }, services.contextAccountingHolder, bootstrapUnsubs);
 
   const acpManager = new AcpManager({
     requestPermission: (request) => permissionPromptRef.requestPermission(request),

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

@@ -101,3 +101,17 @@ export function createContextAccountingSource(deps: ContextAccountingSourceDeps)
     dispose: () => { for (const unsub of unsubs) unsub(); },
   };
 }
+
+/**
+ * Create the source, install it on the holder, and register its dispose on
+ * the caller's unsubscribe list — the one-call bootstrap wiring.
+ */
+export function wireContextAccountingSource(
+  deps: ContextAccountingSourceDeps,
+  holder: { setSource: (source: ContextAccountingSourceHandle['source']) => void },
+  unsubs: Array<() => void>,
+): void {
+  const handle = createContextAccountingSource(deps);
+  holder.setSource(handle.source);
+  unsubs.push(handle.dispose);
+}

package/src/runtime/provider-fallback.ts

@@ -7,6 +7,7 @@
  */
 import type { ConfigManager } from '@pellux/goodvibes-sdk/platform/config';
 import { inferFallbackContextWindow, type ModelDefinition, type ProviderRegistry } from '@pellux/goodvibes-sdk/platform/providers';
+import { logger } from '@pellux/goodvibes-sdk/platform/utils';
 
 function buildFallbackModelDefinition(provider: string, modelId: string): ModelDefinition {
   const providerLower = provider.toLowerCase();
@@ -53,3 +54,33 @@ export function ensureConfiguredModelIsRoutable(providerRegistry: ProviderRegist
     models: [buildFallbackModelDefinition(providerId, modelId)],
   });
 }
+
+/**
+ * Boot-time custom-provider readiness. Custom providers register
+ * asynchronously (services.ts fires initCustomProviders() without awaiting),
+ * while the boot path resolves the current model synchronously — without
+ * waiting here, a saved provider.model that points at a custom provider
+ * throws "not in registry" before the first frame renders. The routability
+ * guard must re-run after ready(): its services-composition pass bails when
+ * the provider itself isn't registered yet, which is exactly the
+ * custom-provider case. As a last resort (the configured provider's file was
+ * deleted entirely), boot on a real selectable model with a warning instead
+ * of dying before the UI exists.
+ */
+export async function ensureBootModelResolvable(
+  providerRegistry: ProviderRegistry,
+  configManager: ConfigManager,
+): Promise<void> {
+  await providerRegistry.ready();
+  ensureConfiguredModelIsRoutable(providerRegistry, configManager);
+  try {
+    providerRegistry.getCurrentModel();
+  } catch (err) {
+    const configured = String(configManager.get('provider.model') ?? '');
+    const replacement = providerRegistry.getSelectableModels()[0]?.registryKey;
+    if (!replacement) throw err;
+    providerRegistry.setCurrentModel(replacement);
+    configManager.set('provider.model', replacement);
+    logger.warn(`[bootstrap] Configured model '${configured}' is not resolvable (its provider is not registered); switched to '${replacement}'.`);
+  }
+}

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.19.3';
+let _version = '1.19.4';
 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