@pellux/goodvibes-tui 0.25.0 → 0.27.0

package/src/runtime/services.ts

@@ -7,15 +7,6 @@ import { AutomationDeliveryManager, AutomationManager, AutomationRouteStore } fr
 import { ChannelPluginRegistry, ChannelPolicyManager, RouteBindingManager, SurfaceRegistry } from '@pellux/goodvibes-sdk/platform/channels';
 import { ChannelDeliveryRouter } from '@pellux/goodvibes-sdk/platform/channels';
 import { ApprovalBroker, GatewayMethodCatalog, SharedSessionBroker } from '@pellux/goodvibes-sdk/platform/control-plane';
-import {
-  registerDaemonOperatorSurfaces,
-  type DaemonOperatorSurfaces,
-} from '../daemon/operator/surfaces.ts';
-import type {
-  OperatorContext,
-  OperatorLogger,
-} from '../daemon/operator/index.ts';
-import { attachRemoteInvokeRoute } from '../daemon/remote/index.ts';
 import { WatcherRegistry } from '@pellux/goodvibes-sdk/platform/watchers';
 import { ArtifactStore } from '@pellux/goodvibes-sdk/platform/artifacts';
 import {
@@ -72,6 +63,7 @@ import { CacheHitTracker } from '@pellux/goodvibes-sdk/platform/providers';
 import { FavoritesStore } from '@pellux/goodvibes-sdk/platform/providers';
 import { BenchmarkStore } from '@pellux/goodvibes-sdk/platform/providers';
 import { ModelLimitsService } from '@pellux/goodvibes-sdk/platform/providers';
+import { inferFallbackContextWindow } from '@pellux/goodvibes-sdk/platform/providers';
 import { KeybindingsManager } from '../input/keybindings.ts';
 import { SessionMemoryStore } from '@pellux/goodvibes-sdk/platform/core';
 import { SessionLineageTracker } from '@pellux/goodvibes-sdk/platform/core';
@@ -96,6 +88,19 @@ import {
   type WorkflowServices,
 } from '@pellux/goodvibes-sdk/platform/tools';
 import { WorkPlanStore } from '../work-plans/work-plan-store.ts';
+import {
+  registerDaemonHandlers,
+  type DaemonHandlerSurfaces,
+} from '../daemon/handlers/index.ts';
+import type { HandlerContext, HandlerLogger } from '../daemon/handlers/context.ts';
+import { createDaemonCredentialStore } from '../daemon/handlers/credentials.ts';
+import { registerRouting } from '../daemon/handlers/routing/index.ts';
+import { registerInboxMethods } from '../daemon/handlers/inbox/index.ts';
+import { registerTriagedInbox } from '../daemon/handlers/triage/index.ts';
+import { registerDraftMethods } from '../daemon/handlers/drafts/index.ts';
+import { registerCalendar } from '../daemon/handlers/calendar/index.ts';
+import { registerEmailMethods } from '../daemon/handlers/email/index.ts';
+import { registerRemoteSurface } from '../daemon/handlers/remote/index.ts';
 
 const REGULAR_KNOWLEDGE_DB_FILE = 'knowledge-wiki.sqlite';
 const HOME_GRAPH_KNOWLEDGE_DB_FILE = 'knowledge-home-graph.sqlite';
@@ -119,7 +124,9 @@ function buildFallbackModelDefinition(provider: string, modelId: string): ModelD
       reasoning: isReasoningProvider,
       multimodal: isReasoningProvider,
     },
-    contextWindow: isReasoningProvider ? 128_000 : 32_000,
+    // Pre-catalog fallback uses the SDK's family-aware inference (SDK 0.35.0+),
+    // matching the post-catalog window so the meter/compaction denominator agrees.
+    contextWindow: inferFallbackContextWindow(provider, modelId),
     contextWindowProvenance: 'fallback',
     selectable: true,
     tier: 'standard',
@@ -178,7 +185,6 @@ export interface RuntimeServices {
   readonly deliveryManager: AutomationDeliveryManager;
   readonly automationManager: AutomationManager;
   readonly gatewayMethods: GatewayMethodCatalog;
-  readonly operatorSurfaces: DaemonOperatorSurfaces;
   readonly artifactStore: ArtifactStore;
   readonly knowledgeService: KnowledgeService;
   readonly agentKnowledgeService: KnowledgeService;
@@ -225,6 +231,7 @@ export interface RuntimeServices {
   readonly providerRegistry: ProviderRegistry;
   readonly toolLLM: ToolLLM;
   readonly distributedRuntime: DistributedRuntimeManager;
+  readonly daemonHandlers: DaemonHandlerSurfaces;
   readonly remoteRunnerRegistry: RemoteRunnerRegistry;
   readonly remoteSupervisor: RemoteSupervisor;
   readonly sessionMemoryStore: SessionMemoryStore;
@@ -281,30 +288,6 @@ export function createRuntimeServices(options: RuntimeServicesOptions): RuntimeS
     globalHome: homeDirectory,
     configManager,
   });
-  // Wire the daemon operator surfaces (routing / triaged inbox / drafts /
-  // calendar / email / remote) against the shared gateway-method catalog. Each
-  // surface initializes its stores lazily, so this stays synchronous.
-  const operatorLogger: OperatorLogger = {
-    info: (msg, meta) => console.info(`[operator] ${msg}`, meta ?? ''),
-    warn: (msg, meta) => console.warn(`[operator] ${msg}`, meta ?? ''),
-    error: (msg, meta) => console.error(`[operator] ${msg}`, meta ?? ''),
-  };
-  const operatorContext: OperatorContext = {
-    catalog: gatewayMethods,
-    secrets: secretsManager,
-    configManager,
-    workingDirectory,
-    homeDirectory,
-    logger: operatorLogger,
-  };
-  const operatorSurfaces = registerDaemonOperatorSurfaces(operatorContext);
-  // Wire the remote.peers.invoke route to the remote dispatch adapter, replacing
-  // the SDK builtin stub. This is the single faithful integration point: the
-  // DaemonHttpRouter resolves POST /api/remote/peers/{peerId}/invoke to
-  // catalog.invoke('remote.peers.invoke', ...), which now reaches the
-  // Docker/SSH/cloud-terminal/local-process backends via the dispatch adapter
-  // (which enforces confirm:true + explicitUserRequest before dispatch).
-  attachRemoteInvokeRoute(gatewayMethods, operatorSurfaces.remoteDispatch);
   const subscriptionManager = new SubscriptionManager(
     shellPaths.resolveUserPath('tui', 'subscriptions.json'),
   );
@@ -551,6 +534,39 @@ export function createRuntimeServices(options: RuntimeServicesOptions): RuntimeS
     approvalBridge: approvalBroker,
     automationBridge: automationManager,
   });
+
+  // Daemon handler surfaces: attach HOST handlers to the SDK-auto-registered
+  // builtin gateway descriptors (channels.* / email.* / calendar.*). The SDK
+  // owns every id/descriptor/schema; this only fills the handler slot via
+  // catalog.register(descriptor, handler, { replace: true }). Routing returns
+  // the resolver the inbox surface consumes; triage decorates channels.inbox.list.
+  // The remote surface reuses the SAME DistributedRuntimeManager the SDK facade
+  // injects, so its peer/work methods share one persistent store; remote.peers.*
+  // stay SDK-published routes (not catalog methods).
+  const handlerLogger: HandlerLogger = {
+    info: (message, meta) => console.info(message, meta ?? ''),
+    warn: (message, meta) => console.warn(message, meta ?? ''),
+    error: (message, meta) => console.error(message, meta ?? ''),
+  };
+  const handlerContext: HandlerContext = {
+    catalog: gatewayMethods,
+    credentials: createDaemonCredentialStore(secretsManager),
+    configManager,
+    workingDirectory,
+    homeDirectory,
+    logger: handlerLogger,
+  };
+  const daemonHandlers = registerDaemonHandlers(handlerContext, {
+    registerRouting,
+    registerInbox: (ctx, routing) =>
+      registerTriagedInbox(ctx, (inboxCtx) => registerInboxMethods(inboxCtx, routing))
+        .unregister,
+    registerDrafts: (ctx) => registerDraftMethods(ctx),
+    registerCalendar,
+    registerEmail: (ctx) => registerEmailMethods(ctx),
+    registerRemote: (ctx) => registerRemoteSurface(ctx, { manager: distributedRuntime }),
+  });
+
   const remoteRunnerRegistry = new RemoteRunnerRegistry(agentManager);
   const remoteSupervisor = new RemoteSupervisor(remoteRunnerRegistry);
   const sandboxSessionRegistry = new SandboxSessionRegistry(workingDirectory);
@@ -657,7 +673,6 @@ export function createRuntimeServices(options: RuntimeServicesOptions): RuntimeS
     deliveryManager,
     automationManager,
     gatewayMethods,
-    operatorSurfaces,
     artifactStore,
     knowledgeService,
     agentKnowledgeService,
@@ -704,6 +719,7 @@ export function createRuntimeServices(options: RuntimeServicesOptions): RuntimeS
     providerRegistry,
     toolLLM,
     distributedRuntime,
+    daemonHandlers,
     remoteRunnerRegistry,
     remoteSupervisor,
     sessionMemoryStore,

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.25.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',
-    );
-  }
-}

package/src/daemon/calendar/index.ts

@@ -1,52 +0,0 @@
-// Public surface barrel for the CalDAV calendar connector.
-//
-// Integration registers the surface by calling registerCalendarMethods(ctx).
-// All other exports are types / helpers for consumers and tests.
-
-export { registerCalendarMethods, CALENDAR_METHOD_IDS } from './register.ts';
-export type {
-  RegisterCalendarOptions,
-  CalDavClientFactory,
-  CalendarEventDetail,
-  CalendarEventSummaryResult,
-} from './register.ts';
-
-export {
-  createCalDavClient,
-  resolveCalDavConfig,
-  toRelativeHref,
-  parseMultiStatus,
-} from './caldav-client.ts';
-export type {
-  CalDavClient,
-  CalDavConfig,
-  CalDavEvent,
-  CreateCalDavClientOptions,
-  CreateEventInput,
-  CreatedEvent,
-  ImportResult,
-  ExportResult,
-  ListEventsOptions,
-  FetchLike,
-} from './caldav-client.ts';
-
-export {
-  parseICS,
-  generateICS,
-  generateCalendar,
-  foldLine,
-  unfoldLines,
-  escapeText,
-  unescapeText,
-  formatICalDate,
-  formatICalDateTime,
-  parseICalDate,
-  attendeeDisplayName,
-  eventToVEvent,
-  extractUid,
-} from './ics.ts';
-export type {
-  ParsedICalEvent,
-  ICalAttendee,
-  GenerateICalInput,
-} from './ics.ts';