@pellux/goodvibes-tui 0.26.0 → 0.27.0

package/src/runtime/wrfc-persistence.ts

@@ -10,10 +10,11 @@
  *
  * 2. `WrfcPersistence.rehydrate(router)` is called once on boot (after the
  *    SystemMessageRouter is available). It reads the snapshot, identifies
- *    chains that were in a non-terminal state at last write, and emits a
- *    high-priority 'wrfc' system message per interrupted chain. The
- *    `interruptedChains` accessor makes the data available for panel reads
- *    without coupling this module to wrfc-panel.ts.
+ *    chains that were in a non-terminal state at last write, emits a
+ *    high-priority 'wrfc' system message per interrupted chain, and re-imports
+ *    each interrupted chain into the WrfcController so it reappears as a panel
+ *    row and can be resumed by the operator. The `interruptedChains` accessor
+ *    additionally exposes the recovered set for inspection.
  *
  * 3. Snapshot lifecycle:
  *    - Terminal chains ('passed' | 'failed') are pruned from the snapshot
@@ -57,6 +58,14 @@ interface WrfcSnapshot {
 /** Subset of WrfcController needed by this module. */
 export interface WrfcControllerReader {
   listChains(): WrfcChain[];
+  /**
+   * Re-import a chain recovered from a previous process so it reappears in the
+   * controller's in-memory map and becomes selectable/resumable from the panel.
+   * Optional so read-only test doubles can omit it; the real WrfcController
+   * provides it. `force` is left at its default — on a fresh start the map is
+   * empty so importing never clobbers a live chain.
+   */
+  importChain?(chain: WrfcChain, force?: boolean): boolean;
 }
 
 export interface WrfcPersistenceOptions {
@@ -64,7 +73,7 @@ export interface WrfcPersistenceOptions {
   readonly snapshotPath: string;
   /** Factory for the current SystemMessageRouter — may return null before it is wired. */
   readonly getSystemMessageRouter: () => SystemMessageRouter | null;
-  /** WrfcController reader — only listChains() is needed. */
+  /** WrfcController access — listChains() (read) plus optional importChain() (re-import on rehydrate). */
   readonly controller: WrfcControllerReader;
 }
 
@@ -140,6 +149,12 @@ class WrfcPersistenceImpl implements WrfcPersistence {
 
     const router = this.getSystemMessageRouter();
     for (const chain of interrupted) {
+      // Re-import so the chain reappears in the controller's in-memory map and
+      // becomes selectable/resumable from the panel. On a fresh process start
+      // the map is empty, so importChain (force=false) never clobbers a live
+      // chain. The accessor is optional for read-only test doubles.
+      this.controller.importChain?.(chain);
+
       const msg =
         `[WRFC] Chain ${chain.id.slice(0, 12)} (${chain.task.slice(0, 60).trim()}) ` +
         `was interrupted by a restart — state was '${chain.state}' ` +

package/src/shell/ui-openers.ts

@@ -13,6 +13,7 @@ import type { SecretsManager } from '@pellux/goodvibes-sdk/platform/config';
 import type { ServiceInspectionQuery } from '../runtime/ui-service-queries.ts';
 import type { ModelPickerTargetInfo } from '../input/model-picker.ts';
 import { syncServiceSettingToPlatform } from './service-settings-sync.ts';
+import { summarizeError } from '@pellux/goodvibes-sdk/platform/utils';
 
 type WireShellUiOpenersOptions = {
   commandContext: CommandContext;
@@ -203,7 +204,7 @@ export function wireShellUiOpeners(options: WireShellUiOpenersOptions): void {
       input.modelPicker.openAllModels(models, getCurrentModelForPickerTarget());
       render();
     })().catch((error: unknown) => {
-      commandContext.print?.(`Model picker failed to open: ${error instanceof Error ? error.message : String(error)}`);
+      commandContext.print?.(`Model picker failed to open: ${summarizeError(error)}`);
       render();
     });
   };
@@ -223,7 +224,7 @@ export function wireShellUiOpeners(options: WireShellUiOpenersOptions): void {
       input.modelPicker.openProviders(providers, getCurrentProviderForPickerTarget());
       render();
     })().catch((error: unknown) => {
-      commandContext.print?.(`Provider picker failed to open: ${error instanceof Error ? error.message : String(error)}`);
+      commandContext.print?.(`Provider picker failed to open: ${summarizeError(error)}`);
       render();
     });
   };

package/src/verification/live-verifier.ts

@@ -3,6 +3,7 @@ import { join, resolve } from 'node:path';
 import { spawn } from 'node:child_process';
 import { auditGoodVibesHome } from '../config/goodvibes-home-audit.ts';
 import { buildVerificationLedger } from './verification-ledger.ts';
+import { summarizeError } from '@pellux/goodvibes-sdk/platform/utils';
 
 export type LiveVerificationStatus = 'pass' | 'warn' | 'fail' | 'skip';
 
@@ -111,7 +112,7 @@ function runCommand(command: string, args: string[], cwd: string, timeoutMs = 15
       resolveCommand({
         exitCode: -1,
         stdout: '',
-        stderr: error instanceof Error ? error.message : String(error),
+        stderr: summarizeError(error),
         timedOut,
       });
     });
@@ -161,7 +162,7 @@ function commandCheck(
         title,
         status: 'fail',
         summary: 'Command succeeded but did not return valid JSON.',
-        detail: error instanceof Error ? error.message : String(error),
+        detail: summarizeError(error),
       };
     }
   }
@@ -208,7 +209,7 @@ async function fetchCheck(
       title,
       status: 'fail',
       summary: 'Request failed.',
-      detail: error instanceof Error ? error.message : String(error),
+      detail: summarizeError(error),
     };
   }
 }

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

package/src/core/context-auto-compact.ts

@@ -1,110 +0,0 @@
-/**
- * Auto-compaction helper — TASK-058.
- *
- * Evaluates whether auto-compact should run after a turn completes and, if so,
- * triggers compaction and posts an honest transcript notice.
- *
- * behavior.autoCompactThreshold: SDK schema range [10, 100], default 80.
- * Auto-compact is active whenever the threshold is in its valid range (>0).
- * The display/suggestion path (hint + meter) is always active regardless.
- */
-
-import type { ConfigManager } from '@pellux/goodvibes-sdk/platform/config';
-import type { ConversationManager } from './conversation';
-import type { ProviderRegistry } from '@pellux/goodvibes-sdk/platform/providers';
-import type { SystemMessageRouter } from './system-message-router';
-import { logger } from '@pellux/goodvibes-sdk/platform/utils';
-import { summarizeError } from '@pellux/goodvibes-sdk/platform/utils';
-import { getLastCompactionEvent } from '@pellux/goodvibes-sdk/platform/core';
-import type { CompactionContext } from '@pellux/goodvibes-sdk/platform/core';
-import { buildCompactionPreview, buildCompactionAfterNotice } from '../renderer/compaction-preview.ts';
-
-export interface AutoCompactDeps {
-  readonly configManager: Pick<ConfigManager, 'get'>;
-  readonly conversation: ConversationManager;
-  readonly providerRegistry: ProviderRegistry;
-  readonly systemMessageRouter: SystemMessageRouter;
-  readonly model: string;
-  readonly provider: string;
-  readonly lastInputTokens: number;
-  readonly contextWindow: number;
-}
-
-/**
- * Run after each TURN_COMPLETED event.
- *
- * Reads behavior.autoCompactThreshold from config (SDK default: 80, range [10, 100]).
- * When usage is at or above the threshold, compacts the conversation and posts
- * an honest transcript notice so the user understands any summary discontinuity.
- *
- * This function is intentionally non-throwing; failures are logged and
- * surfaced via the system message router.
- */
-export async function maybeAutoCompact(deps: AutoCompactDeps): Promise<void> {
-  // SDK schema default is 80; valid range is [10, 100]. The ?? 0 fallback is a
-  // defensive guard for missing/null values only — not a normal operating state.
-  const rawThreshold = Number(deps.configManager.get('behavior.autoCompactThreshold') ?? 0);
-  const thresholdPct = Number.isFinite(rawThreshold) ? rawThreshold : 0;
-
-  // Defensive guard: skip only when threshold is missing/non-positive (real config defaults to 80).
-  if (thresholdPct <= 0 || deps.contextWindow <= 0) return;
-
-  const usagePct = Math.min(100, Math.round((Math.max(0, deps.lastInputTokens) / deps.contextWindow) * 100));
-  if (usagePct < thresholdPct) return;
-
-  try {
-    logger.debug('auto-compact triggered', { usagePct, thresholdPct });
-    // Pre-compact preview — uses buildCompactionPreview for an honest estimate.
-    const messages = deps.conversation.getMessagesForLLM();
-    const sessionMemoryStore = deps.conversation.getSessionMemoryStore();
-    const sessionMemories = sessionMemoryStore?.list() ?? [];
-    const pinnedMemoryCount = sessionMemories.length;
-    const preview = buildCompactionPreview({
-      messages,
-      contextWindow: deps.contextWindow,
-      pinnedMemoryCount,
-      trigger: 'auto',
-    });
-    deps.systemMessageRouter.routeSystemMessage(preview, 'high');
-    const eventBefore = getLastCompactionEvent();
-    const compactionCtx: CompactionContext = {
-      messages,
-      sessionMemories,
-      agents: [],
-      wrfcChains: [],
-      activePlan: null,
-      lineageEntries: [],
-      compactionCount: 0,
-      contextWindow: deps.contextWindow,
-      trigger: 'auto',
-      extractionModelId: deps.model,
-      extractionProvider: deps.provider,
-    };
-    await deps.conversation.compact(
-      deps.providerRegistry,
-      deps.model,
-      'auto',
-      deps.provider,
-      compactionCtx,
-    );
-    // Post-compact notice using real CompactionEvent figures.
-    const eventAfter = getLastCompactionEvent();
-    if (eventAfter !== null && eventAfter !== eventBefore) {
-      deps.systemMessageRouter.routeSystemMessage(
-        buildCompactionAfterNotice({ event: eventAfter, pinnedMemoryCount }),
-        'low',
-      );
-    } else {
-      deps.systemMessageRouter.routeSystemMessage(
-        '[Context] Auto-compact complete — older turns summarised. Use /compact to compact again manually.',
-        'low',
-      );
-    }
-  } catch (err) {
-    logger.error('auto-compact failed', { error: summarizeError(err) });
-    deps.systemMessageRouter.routeSystemMessage(
-      `[Context] Auto-compact failed: ${summarizeError(err)}. Use /compact to try manually.`,
-      'high',
-    );
-  }
-}