@pellux/goodvibes-agent 1.1.7 → 1.3.0

package/src/renderer/agent-workspace.ts

@@ -5,9 +5,8 @@ import type {
   AgentWorkspaceLocalEditor,
   AgentWorkspaceRuntimeSnapshot,
 } from '../input/agent-workspace.ts';
-import { formatAgentRecordReviewState } from '../agent/record-labels.ts';
 import type { Line } from '../types/grid.ts';
-import { truncateDisplay, wrapText } from '../utils/terminal-width.ts';
+import { wrapText } from '../utils/terminal-width.ts';
 import { GLYPHS } from './ui-primitives.ts';
 import {
   getFullscreenWorkspaceMetrics,
@@ -17,7 +16,8 @@ import {
   WORKSPACE_PALETTE as PALETTE,
   type WorkspaceRow,
 } from './fullscreen-workspace.ts';
-import { actionResultColor, setupStatusColor, type AgentWorkspaceContextLine as ContextLine } from './agent-workspace-style.ts';
+import { actionResultColor, type AgentWorkspaceContextLine as ContextLine } from './agent-workspace-style.ts';
+import { compactText, reviewerReadinessContextLines, snapshotLines } from './agent-workspace-context-lines.ts';
 
 function buildLeftRows(workspace: AgentWorkspace, height: number): WorkspaceRow[] {
   const rows: WorkspaceRow[] = [];
@@ -59,6 +59,7 @@ function actionCommand(action: AgentWorkspaceAction): string {
   if (action.kind === 'editor') return action.editorKind ? `edit ${action.editorKind}` : 'edit form';
   if (action.kind === 'setting') return action.settingKey ? `setting ${action.settingKey}` : 'setting';
   if (action.kind === 'settings-import') return 'import GoodVibes settings';
+  if (action.kind === 'setup-checkpoint') return action.setupCheckpointOperation ? `setup checkpoint ${action.setupCheckpointOperation}` : 'setup checkpoint';
   if (action.kind === 'model-picker') return action.modelPickerFlow === 'model' ? 'model picker' : 'provider/model picker';
   if (action.kind === 'settings-modal') return action.settingsTarget ? `settings ${action.settingsTarget}` : 'settings';
   if (action.kind === 'local-selection') return action.selectionDelta && action.selectionDelta < 0 ? 'select previous' : 'select next';
@@ -67,428 +68,46 @@ function actionCommand(action: AgentWorkspaceAction): string {
   return action.command ?? '(guidance)';
 }
 
-function setupCounts(snapshot: AgentWorkspaceRuntimeSnapshot): { ready: number; recommended: number; optional: number; blocked: number } {
-  return {
-    ready: snapshot.setupChecklist.filter((item) => item.status === 'ready').length,
-    recommended: snapshot.setupChecklist.filter((item) => item.status === 'recommended').length,
-    optional: snapshot.setupChecklist.filter((item) => item.status === 'optional').length,
-    blocked: snapshot.setupChecklist.filter((item) => item.status === 'blocked').length,
-  };
-}
-
-function setupStatusLabel(status: AgentWorkspaceRuntimeSnapshot['setupChecklist'][number]['status']): string {
-  return status === 'ready'
-    ? 'Ready'
-    : status === 'recommended'
-      ? 'Recommended'
-      : status === 'blocked'
-        ? 'Blocked'
-        : 'Optional';
-}
-
-function formatMegabytes(bytes: number): string {
-  if (!Number.isFinite(bytes) || bytes <= 0) return '0 MB';
-  return `${Math.round(bytes / (1024 * 1024))} MB`;
-}
-
-function compactText(text: string, maxWidth = 104): string {
-  const normalized = text.replace(/\s+/g, ' ').trim();
-  if (normalized.length === 0) return '';
-  const firstSentence = normalized.match(/^.*?[.!?](?:\s|$)/)?.[0]?.trim();
-  const source = firstSentence && firstSentence.length <= maxWidth ? firstSentence : normalized;
-  return truncateDisplay(source, maxWidth, '...');
-}
-
-function actionMetaLine(action: AgentWorkspaceAction): ContextLine {
-  return {
-    text: `Does: ${actionCommand(action)}`,
-    fg: action.safety === 'blocked' ? PALETTE.warn : action.kind === 'command' ? PALETTE.info : PALETTE.muted,
-  };
+function isOnboardingCategory(category: AgentWorkspaceCategory): boolean {
+  return category.group === 'ONBOARDING';
 }
 
-function setupAttentionItems(snapshot: AgentWorkspaceRuntimeSnapshot, limit: number): AgentWorkspaceRuntimeSnapshot['setupChecklist'] {
-  return [
-    ...snapshot.setupChecklist.filter((item) => item.status === 'blocked'),
-    ...snapshot.setupChecklist.filter((item) => item.status === 'recommended'),
-    ...snapshot.setupChecklist.filter((item) => item.status === 'optional'),
-  ].slice(0, limit);
-}
-
-function conciseSetupLine(snapshot: AgentWorkspaceRuntimeSnapshot): ContextLine {
-  const counts = setupCounts(snapshot);
-  return {
-    text: `Setup: ${counts.ready}/${snapshot.setupChecklist.length} ready; ${counts.recommended} recommended; ${counts.blocked} blocked.`,
-    fg: counts.blocked > 0 ? PALETTE.warn : PALETTE.info,
-  };
-}
-
-function setupOverviewLines(snapshot: AgentWorkspaceRuntimeSnapshot): ContextLine[] {
-  const counts = setupCounts(snapshot);
-  const nextItems = setupAttentionItems(snapshot, 3);
-  const lines: ContextLine[] = [
-    { text: 'Onboarding', fg: PALETTE.title, bold: true },
-    { text: `${counts.ready}/${snapshot.setupChecklist.length} ready; ${counts.recommended} recommended; ${counts.optional} optional; ${counts.blocked} blocked.`, fg: counts.blocked > 0 ? PALETTE.warn : PALETTE.info },
-    { text: `Chat: ${snapshot.provider} / ${snapshot.modelDisplayName}.`, fg: PALETTE.info },
-    { text: `Local: ${snapshot.localPersonaCount} personas, ${snapshot.localSkillCount} skills, ${snapshot.localRoutineCount} routines, ${snapshot.localMemoryCount} memories.`, fg: PALETTE.info },
-  ];
-  if (nextItems.length > 0) {
-    const item = nextItems[0]!;
-    lines.push({
-      text: `Next: ${item.label} (${setupStatusLabel(item.status).toLowerCase()})`,
-      fg: setupStatusColor(item.status),
-      bold: item.status === 'blocked',
-    });
+function onboardingActionChange(workspace: AgentWorkspace, action: AgentWorkspaceAction): string {
+  if (action.kind === 'setting') {
+    return workspace.settingActionPreview(action) ?? 'Update setting';
   }
-  return lines;
-}
-
-function homeNextActionLines(snapshot: AgentWorkspaceRuntimeSnapshot): ContextLine[] {
-  const candidates = setupAttentionItems(snapshot, 3);
-  if (candidates.length === 0) {
-    return [
-      { text: 'Next Actions', fg: PALETTE.title, bold: true },
-      { text: 'Core setup is ready. Continue normal assistant work, review Knowledge sources, or tune local skills/routines as needed.', fg: PALETTE.good },
-    ];
+  if (action.kind === 'settings-import') return 'Import compatible GoodVibes preferences';
+  if (action.kind === 'setup-checkpoint') {
+    if (action.setupCheckpointOperation === 'show') return 'Show saved setup progress';
+    if (action.setupCheckpointOperation === 'mark-current') return 'Save current setup progress';
+    if (action.setupCheckpointOperation === 'clear') return 'Clear saved setup progress';
+    return 'Review setup progress';
   }
-  const lines: ContextLine[] = [{ text: 'Next Actions', fg: PALETTE.title, bold: true }];
-  for (const item of candidates) {
-    lines.push({
-      text: `${setupStatusLabel(item.status)}: ${item.label}`,
-      fg: setupStatusColor(item.status),
-      bold: item.status === 'blocked',
-    });
-  }
-  return lines;
-}
-
-function companionAccessLine(snapshot: AgentWorkspaceRuntimeSnapshot): ContextLine {
-  const access = snapshot.companionAccess;
-  const tokenState = access.tokenReadable
-    ? `ready sha256:${access.tokenFingerprint ?? 'unknown'}`
-    : access.tokenPresent
-      ? 'present but unreadable'
-      : 'missing';
-  const error = access.tokenError ? `; token read error ${access.tokenError}` : '';
-  return {
-    text: `Companion: ${access.surface}; token ${tokenState}; QR ${access.qrCommand}${error}.`,
-    fg: access.pairingReady ? PALETTE.good : PALETTE.warn,
-  };
+  if (action.kind === 'model-picker') return action.modelPickerFlow === 'model' ? 'Choose a model' : 'Choose provider and model';
+  if (action.kind === 'settings-modal') return 'Open settings';
+  if (action.kind === 'workspace') return 'Open setup area';
+  if (action.kind === 'editor') return 'Open guided form';
+  if (action.kind === 'local-selection') return 'Move selection';
+  if (action.kind === 'local-operation') return 'Apply selected library action';
+  if (action.kind === 'onboarding-complete') return 'Save setup completion';
+  if (action.safety === 'read-only') return 'Review readiness';
+  if (action.safety === 'blocked') return 'Unavailable in this setup step';
+  return 'Open option';
 }
 
-function compactLocalLibraryLines(
-  title: string,
-  items: readonly AgentWorkspaceRuntimeSnapshot['localPersonas'][number][],
-  emptyText: string,
-  selectedId: string | null,
-): ContextLine[] {
-  const lines: ContextLine[] = [];
-  if (items.length === 0) {
-    lines.push({ text: `${title}: 0. ${emptyText}`, fg: PALETTE.warn });
-    return lines;
-  }
-  const selected = items.find((item) => item.id === selectedId) ?? items[0]!;
-  const status = [
-    selected.active ? 'active' : '',
-    selected.enabled === true ? 'enabled' : selected.enabled === false ? 'disabled' : '',
-    selected.scope && selected.cls ? `${selected.scope}/${selected.cls}` : '',
-    selected.confidence !== undefined ? `${selected.confidence}%` : '',
-    selected.requirementCount !== undefined && selected.requirementCount > 0
-      ? (selected.missingRequirementCount && selected.missingRequirementCount > 0 ? `needs ${selected.missingRequirementCount}/${selected.requirementCount}` : `ready ${selected.requirementCount}/${selected.requirementCount}`)
-      : '',
-    formatAgentRecordReviewState(selected.reviewState),
-    selected.startCount !== undefined ? `starts ${selected.startCount}` : '',
-  ].filter(Boolean).join(', ');
-  lines.push({
-    text: `${title}: ${items.length}; selected ${selected.name}${status ? ` (${status})` : ''}.`,
-    fg: selected.reviewState === 'stale' ? PALETTE.warn : PALETTE.info,
-    bold: selected.active === true,
-  });
-  if (selected.missingRequirements && selected.missingRequirements.length > 0) {
-    lines.push({ text: `Missing setup: ${selected.missingRequirements.join(', ')}`, fg: PALETTE.warn });
-  }
-  return lines;
-}
-
-type LocalRoutineItem = AgentWorkspaceRuntimeSnapshot['localRoutines'][number];
-
-function readyRoutineItems(snapshot: AgentWorkspaceRuntimeSnapshot): readonly LocalRoutineItem[] {
-  return snapshot.localRoutines.filter((routine) =>
-    routine.enabled === true
-    && routine.reviewState === 'reviewed'
-    && (routine.missingRequirementCount ?? 0) === 0
-  );
-}
-
-function routinesNeedingSetup(snapshot: AgentWorkspaceRuntimeSnapshot): readonly LocalRoutineItem[] {
-  return snapshot.localRoutines.filter((routine) => (routine.missingRequirementCount ?? 0) > 0);
-}
-
-function routinesNeedingReview(snapshot: AgentWorkspaceRuntimeSnapshot): readonly LocalRoutineItem[] {
-  return snapshot.localRoutines.filter((routine) => routine.reviewState !== 'reviewed');
-}
-
-function routineNextActionLine(snapshot: AgentWorkspaceRuntimeSnapshot): ContextLine {
-  const discovered = snapshot.discoveredBehavior.routines.count;
-  const ready = readyRoutineItems(snapshot);
-  const needsSetup = routinesNeedingSetup(snapshot);
-  const needsReview = routinesNeedingReview(snapshot);
-
-  if (snapshot.localRoutineCount === 0 && discovered > 0) {
-    return { text: 'Next routine action: Discover routine files, preview one, then import it from this workspace.', fg: PALETTE.warn, bold: true };
-  }
-  if (snapshot.localRoutineCount === 0) {
-    return { text: 'Next routine action: Create routine for a repeatable main-conversation workflow.', fg: PALETTE.warn, bold: true };
-  }
-  if (needsSetup.length > 0) {
-    return { text: `Next routine action: Needs setup for ${needsSetup[0]?.id ?? 'a routine'} before it can be trusted for schedule promotion.`, fg: PALETTE.warn, bold: true };
-  }
-  if (needsReview.length > 0) {
-    return { text: `Next routine action: Review selected or inspect ${needsReview[0]?.id ?? 'a routine'} before schedule promotion.`, fg: PALETTE.warn, bold: true };
-  }
-  if (ready.length > 0 && snapshot.routineScheduleReceiptCount === 0) {
-    return { text: `Next routine action: Promote ${ready[0]?.id ?? 'a reviewed routine'} to a connected schedule when you have a real cadence.`, fg: PALETTE.good, bold: true };
-  }
-  return { text: 'Next routine action: Start selected in the main conversation, inspect receipts, or reconcile connected schedules.', fg: PALETTE.info, bold: true };
+function actionChange(workspace: AgentWorkspace, category: AgentWorkspaceCategory, action: AgentWorkspaceAction): string {
+  return isOnboardingCategory(category) ? onboardingActionChange(workspace, action) : actionCommand(action);
 }
 
-function compactRoutineReceiptLine(snapshot: AgentWorkspaceRuntimeSnapshot): ContextLine {
-  const latest = snapshot.latestRoutineScheduleReceipt;
-  if (!latest) {
-    return {
-      text: `Promotion receipts: ${snapshot.routineScheduleReceiptCount}; none created yet.`,
-      fg: PALETTE.muted,
-    };
-  }
+function actionMetaLine(workspace: AgentWorkspace, category: AgentWorkspaceCategory, action: AgentWorkspaceAction): ContextLine {
+  const onboarding = isOnboardingCategory(category);
   return {
-    text: `Promotion receipts: ${snapshot.routineScheduleReceiptCount}; latest ${latest.status} ${latest.routineId}.`,
-    fg: latest.status === 'failed' ? PALETTE.warn : PALETTE.good,
+    text: `${onboarding ? 'Change' : 'Does'}: ${actionChange(workspace, category, action)}`,
+    fg: action.safety === 'blocked' ? PALETTE.warn : !onboarding && action.kind === 'command' ? PALETTE.info : PALETTE.muted,
   };
 }
 
-function automationNextActionLine(snapshot: AgentWorkspaceRuntimeSnapshot): ContextLine {
-  const ready = readyRoutineItems(snapshot);
-  const needsSetup = routinesNeedingSetup(snapshot);
-  const needsReview = routinesNeedingReview(snapshot);
-  if (snapshot.routineScheduleReceiptCount > 0) {
-    return { text: 'Next automation action: Reconcile schedules to compare local receipts with the connected host.', fg: PALETTE.info, bold: true };
-  }
-  if (ready.length > 0) {
-    return { text: `Next automation action: Promote ${ready[0]?.id ?? 'a reviewed routine'} or create a one-off reminder.`, fg: PALETTE.good, bold: true };
-  }
-  if (needsSetup.length > 0) {
-    return { text: `Next automation action: Resolve routine setup gaps in ${needsSetup[0]?.id ?? 'Routines'} before promotion, or create a reminder.`, fg: PALETTE.warn, bold: true };
-  }
-  if (needsReview.length > 0) {
-    return { text: `Next automation action: Review ${needsReview[0]?.id ?? 'a routine'} in Routines before schedule promotion, or create a reminder.`, fg: PALETTE.warn, bold: true };
-  }
-  return { text: 'Next automation action: Create a reminder, or create/import a routine before recurring workflow promotion.', fg: PALETTE.warn, bold: true };
-}
-
-function snapshotLines(workspace: AgentWorkspace, category: AgentWorkspaceCategory, snapshot: AgentWorkspaceRuntimeSnapshot | null): ContextLine[] {
-  if (!snapshot) return [{ text: 'Runtime context is not loaded yet.', fg: PALETTE.warn }];
-  const base: ContextLine[] = [];
-  if (category.id === 'home') {
-    base.push(
-      { text: `Chat route: ${snapshot.provider} / ${snapshot.modelDisplayName}`, fg: PALETTE.info },
-      { text: `Session: ${snapshot.sessionId}`, fg: PALETTE.muted },
-      { text: `Policy: ${snapshot.executionPolicy}; delegated review ${snapshot.delegatedReviewPolicy}`, fg: PALETTE.good },
-      conciseSetupLine(snapshot),
-      ...homeNextActionLines(snapshot),
-    );
-  } else if (category.id === 'setup') {
-    base.push(
-      ...setupOverviewLines(snapshot),
-    );
-  } else if (category.id === 'account-model') {
-    base.push(
-      { text: `Chat route: ${snapshot.provider} / ${snapshot.modelDisplayName}`, fg: PALETTE.info },
-      { text: `Subscriptions: ${snapshot.activeSubscriptionCount} active; ${snapshot.pendingSubscriptionCount} pending; ${snapshot.availableSubscriptionProviderCount} available.`, fg: snapshot.activeSubscriptionCount > 0 ? PALETTE.good : snapshot.pendingSubscriptionCount > 0 ? PALETTE.warn : PALETTE.muted },
-      { text: `Embedding: ${snapshot.embeddingProvider}; reasoning ${snapshot.reasoningEffort}.`, fg: PALETTE.info },
-      { text: `Helper: ${snapshot.helperEnabled ? 'enabled' : 'disabled'}; Tool LLM: ${snapshot.toolLlmEnabled ? 'enabled' : 'disabled'}.`, fg: snapshot.helperEnabled || snapshot.toolLlmEnabled ? PALETTE.good : PALETTE.muted },
-      { text: `Cache: ${snapshot.cacheEnabled ? snapshot.cacheStableTtl : 'off'}; monitor ${snapshot.cacheMonitorHitRate ? snapshot.cacheHitRateWarningThreshold : 'off'}; failure hints ${snapshot.providerFailureHints ? 'on' : 'off'}.`, fg: snapshot.cacheEnabled ? PALETTE.info : PALETTE.muted },
-    );
-  } else if (category.id === 'assistant-behavior') {
-    base.push(
-      { text: `Interaction: ${snapshot.hitlMode}; guidance ${snapshot.guidanceMode}; history ${snapshot.saveHistory ? 'saved' : 'off'}.`, fg: PALETTE.info },
-      { text: `Context: compact at ${snapshot.autoCompactThreshold}; stale warnings ${snapshot.staleContextWarnings ? 'on' : 'off'}.`, fg: PALETTE.info },
-      { text: `Reasoning display: thinking ${snapshot.showThinking ? 'on' : 'off'}; summaries ${snapshot.showReasoningSummary ? 'on' : 'off'}.`, fg: PALETTE.muted },
-    );
-  } else if (category.id === 'tools-permissions') {
-    base.push(
-      { text: `Permission mode: ${snapshot.permissionMode}.`, fg: snapshot.permissionMode === 'allow-all' ? PALETTE.warn : PALETTE.info },
-      { text: `Auto-approve ${snapshot.autoApprove ? 'on' : 'off'}; tool auto-heal ${snapshot.toolAutoHeal ? 'on' : 'off'}; token budget ${snapshot.toolsDefaultTokenBudget}.`, fg: snapshot.autoApprove ? PALETTE.warn : PALETTE.info },
-      { text: `Artifact limit ${formatMegabytes(snapshot.artifactMaxBytes)}; raw prompt telemetry ${snapshot.rawPromptTelemetry ? 'on' : 'off'}.`, fg: snapshot.rawPromptTelemetry ? PALETTE.warn : PALETTE.muted },
-      { text: `MCP servers: ${snapshot.mcpConnectedServerCount}/${snapshot.mcpServerCount} connected; quarantined ${snapshot.mcpQuarantinedServerCount}.`, fg: snapshot.mcpQuarantinedServerCount > 0 ? PALETTE.warn : PALETTE.info },
-      { text: 'MCP and secret setup use forms; selecting a row does not run arbitrary tools.', fg: PALETTE.good },
-    );
-  } else if (category.id === 'onboarding-display') {
-    base.push(
-      { text: `Theme: ${snapshot.theme}; streaming ${snapshot.stream ? 'on' : 'off'}; line numbers ${snapshot.lineNumbers}.`, fg: PALETTE.info },
-      { text: `Messages: operational ${snapshot.operationalMessages}; system ${snapshot.systemMessages}.`, fg: PALETTE.info },
-      { text: `Release channel: ${snapshot.releaseChannel}.`, fg: PALETTE.muted },
-    );
-  } else if (category.id === 'onboarding-channels') {
-    const enabledCount = snapshot.channels.filter((channel) => channel.enabled).length;
-    const readyCount = snapshot.channels.filter((channel) => channel.ready).length;
-    const needsConfig = snapshot.channels.filter((channel) => channel.setupState === 'needs-config');
-    const needsTarget = snapshot.channels.filter((channel) => channel.setupState === 'needs-target');
-    base.push(
-      { text: `Channels: ${readyCount}/${snapshot.channels.length} ready; ${enabledCount} enabled.`, fg: enabledCount > 0 ? PALETTE.info : PALETTE.muted },
-      { text: `Needs config: ${needsConfig.length}; needs target: ${needsTarget.length}.`, fg: needsConfig.length > 0 || needsTarget.length > 0 ? PALETTE.warn : PALETTE.good },
-      { text: 'Enable only the channels you want; hidden channel fields appear after the channel is enabled.', fg: PALETTE.good },
-    );
-  } else if (category.id === 'onboarding-voice-media') {
-    const readiness = snapshot.voiceMediaReadiness;
-    base.push(
-      { text: `Voice: ${snapshot.voiceSurfaceEnabled ? 'enabled' : 'disabled'}; TTS ${snapshot.ttsProvider}; voice ${snapshot.ttsVoice}.`, fg: snapshot.voiceSurfaceEnabled ? PALETTE.good : PALETTE.info },
-      { text: `Media readiness: ${readiness.readyMediaProviderCount}/${snapshot.mediaProviderCount}; generation providers ${snapshot.mediaGenerationProviderCount}.`, fg: readiness.readyMediaProviderCount > 0 ? PALETTE.good : PALETTE.muted },
-      { text: `Telephony channel: ${snapshot.channels.find((channel) => channel.id === 'telephony')?.setupState ?? 'disabled'}.`, fg: PALETTE.info },
-    );
-  } else if (category.id === 'onboarding-context') {
-    base.push(
-      { text: `Local context: ${snapshot.localMemoryCount} memories, ${snapshot.localNoteCount} notes, ${snapshot.localPersonaCount} personas.`, fg: PALETTE.info },
-      { text: `Skills: ${snapshot.enabledSkillCount}/${snapshot.localSkillCount} enabled; routines ${snapshot.enabledRoutineCount}/${snapshot.localRoutineCount} enabled.`, fg: PALETTE.info },
-      { text: `Discovered files: personas ${snapshot.discoveredBehavior.personas.count}, skills ${snapshot.discoveredBehavior.skills.count}, routines ${snapshot.discoveredBehavior.routines.count}.`, fg: PALETTE.muted },
-    );
-  } else if (category.id === 'onboarding-automation') {
-    base.push(
-      { text: `Automation: ${snapshot.automationEnabled ? 'enabled' : 'disabled'}; max ${snapshot.automationMaxConcurrentRuns} concurrent; history ${snapshot.automationRunHistoryLimit}.`, fg: snapshot.automationEnabled ? PALETTE.good : PALETTE.muted },
-      { text: `Timeout ${snapshot.automationDefaultTimeoutMs} ms; catch-up ${snapshot.automationCatchUpWindowMinutes} min; cooldown ${snapshot.automationFailureCooldownMs} ms.`, fg: PALETTE.info },
-      { text: `Delete one-shot jobs after success: ${snapshot.automationDeleteAfterRun ? 'yes' : 'no'}.`, fg: snapshot.automationDeleteAfterRun ? PALETTE.info : PALETTE.muted },
-    );
-  } else if (category.id === 'artifacts') {
-    const mediaReady = snapshot.voiceMediaReadiness.readyMediaProviderCount;
-    base.push(
-      { text: `Chat: ${snapshot.provider} / ${snapshot.modelDisplayName}; Knowledge: ${snapshot.knowledgeRoute}`, fg: PALETTE.info },
-      { text: `Media: ${mediaReady}/${snapshot.mediaProviderCount} ready; generation ${snapshot.mediaGenerationProviderCount}.`, fg: mediaReady > 0 ? PALETTE.good : PALETTE.warn },
-      { text: 'Files: attach, export, inspect, ingest reviewed sources, or generate media.', fg: PALETTE.good },
-      { text: 'Knowledge ingest and media generation require explicit actions.', fg: PALETTE.warn },
-    );
-  } else if (category.id === 'channels') {
-    const enabledCount = snapshot.channels.filter((channel) => channel.enabled).length;
-    const readyCount = snapshot.channels.filter((channel) => channel.ready).length;
-    const configuredDefaults = snapshot.channels.filter((channel) => channel.defaultTarget === 'configured').length;
-    const needsTarget = snapshot.channels.filter((channel) => channel.setupState === 'needs-target');
-    const needsConfig = snapshot.channels.filter((channel) => channel.setupState === 'needs-config');
-    const nextAttentionChannel = needsConfig[0] ?? needsTarget[0] ?? snapshot.channels.find((channel) => !channel.enabled);
-    base.push(
-      { text: `API: ${snapshot.runtimeBaseUrl}`, fg: PALETTE.info },
-      companionAccessLine(snapshot),
-      { text: `Channels: ${readyCount}/${snapshot.channels.length} ready; ${enabledCount} enabled; ${configuredDefaults} target(s).`, fg: PALETTE.info },
-      { text: `Next: ${nextAttentionChannel ? `${nextAttentionChannel.label} - ${compactText(nextAttentionChannel.nextStep)}` : 'All enabled channels ready.'}`, fg: nextAttentionChannel ? PALETTE.warn : PALETTE.good },
-      { text: 'Secrets hidden; sends require explicit action.', fg: PALETTE.warn },
-    );
-  } else if (category.id === 'knowledge') {
-    base.push(
-      { text: `Route: ${snapshot.knowledgeRoute}; isolation ${snapshot.knowledgeIsolation}.`, fg: PALETTE.info },
-      { text: 'Ask/search, ingest, review, reindex, and reports stay Agent-owned.', fg: PALETTE.good },
-      { text: 'Ingest requires explicit confirmation.', fg: PALETTE.warn },
-    );
-  } else if (category.id === 'research') {
-    base.push(
-      { text: `Chat: ${snapshot.provider} / ${snapshot.modelDisplayName}; Knowledge: ${snapshot.knowledgeRoute}`, fg: PALETTE.info },
-      { text: `Browser: ${snapshot.voiceMediaReadiness.browserToolState}; public URL ${snapshot.browserToolPublicBaseUrl}.`, fg: snapshot.browserToolExposureEnabled ? PALETTE.warn : PALETTE.muted },
-      { text: 'Research is read-only. Knowledge ingest is a separate confirmed action.', fg: PALETTE.good },
-      { text: compactText(snapshot.voiceMediaReadiness.browserToolNextStep), fg: PALETTE.muted },
-    );
-  } else if (category.id === 'tools') {
-    base.push(
-      { text: `MCP servers: ${snapshot.mcpConnectedServerCount}/${snapshot.mcpServerCount} connected; quarantined ${snapshot.mcpQuarantinedServerCount}; allow-all ${snapshot.mcpAllowAllServerCount}.`, fg: snapshot.mcpQuarantinedServerCount > 0 || snapshot.mcpAllowAllServerCount > 0 ? PALETTE.warn : PALETTE.info },
-      { text: 'Add/update/reload and trust changes require confirmation.', fg: PALETTE.good },
-      { text: 'Start: /mcp review, /mcp tools, /mcp config, Add MCP server.', fg: PALETTE.muted },
-    );
-  } else if (category.id === 'voice-media') {
-    const readiness = snapshot.voiceMediaReadiness;
-    base.push(
-      { text: `Voice: ${readiness.readyVoiceProviderCount}/${snapshot.voiceProviderCount} ready; TTS ${snapshot.ttsProvider}; voice ${snapshot.ttsVoice}.`, fg: readiness.readyVoiceProviderCount > 0 ? PALETTE.good : PALETTE.warn },
-      { text: `Media: ${readiness.readyMediaProviderCount}/${snapshot.mediaProviderCount} ready; generation ${snapshot.mediaGenerationProviderCount}.`, fg: readiness.readyMediaProviderCount > 0 ? PALETTE.good : PALETTE.warn },
-      { text: `Browser: ${readiness.browserToolState}; public URL ${snapshot.browserToolPublicBaseUrl}.`, fg: snapshot.browserToolExposureEnabled ? PALETTE.warn : PALETTE.muted },
-      { text: readiness.nextSteps[0] ? `Next: ${compactText(readiness.nextSteps[0])}` : 'Next: voice/media setup is ready.', fg: readiness.nextSteps.length > 0 ? PALETTE.info : PALETTE.good },
-      { text: 'Secrets hidden; voice, browser, and media side effects require explicit action.', fg: PALETTE.warn },
-    );
-  } else if (category.id === 'profiles') {
-    const defaultProfile = snapshot.selectedRuntimeProfile
-      ? `${snapshot.selectedRuntimeProfile}${snapshot.selectedRuntimeProfileExists ? '' : ' (missing)'}`
-      : '(base Agent home)';
-    base.push(
-      { text: `Profiles: active ${snapshot.activeRuntimeProfile}; default ${defaultProfile}.`, fg: snapshot.selectedRuntimeProfileExists || !snapshot.selectedRuntimeProfile ? PALETTE.info : PALETTE.warn },
-      { text: `Local profiles: ${snapshot.runtimeProfileCount}; starters ${snapshot.runtimeStarterTemplateCount}; custom ${snapshot.localStarterTemplateCount}.`, fg: PALETTE.info },
-      { text: `Starter ids: ${truncateDisplay(snapshot.runtimeStarterTemplates.map((template) => template.id).join(', ') || 'none', 96, '...')}`, fg: PALETTE.muted },
-      { text: 'Profiles isolate local Agent config, sessions, memory, personas, skills, routines, setup, and bundles.', fg: PALETTE.good },
-    );
-  } else if (category.id === 'memory') {
-    base.push(
-      { text: `Memory: ${snapshot.localMemoryCount}; prompt ${snapshot.localMemoryPromptActiveCount}; queue ${snapshot.localMemoryReviewQueueCount}; session ${snapshot.sessionMemoryCount}.`, fg: PALETTE.info },
-      { text: `Notes: ${snapshot.localNoteCount}; skills ${snapshot.localSkillCount}/${snapshot.enabledSkillCount}; routines ${snapshot.localRoutineCount}/${snapshot.enabledRoutineCount}; personas ${snapshot.localPersonaCount}.`, fg: PALETTE.info },
-      { text: `Active persona: ${snapshot.activePersonaName}.`, fg: PALETTE.info },
-      ...compactLocalLibraryLines('Agent Memory', snapshot.localMemories, 'Create one with Create memory.', workspace.selectedLocalLibraryItem('memory')?.id ?? null),
-      { text: 'Secrets are rejected or redacted; use secret references.', fg: PALETTE.warn },
-    );
-  } else if (category.id === 'notes') {
-    base.push(
-      { text: `Scratchpad notes: ${snapshot.localNoteCount}; review queue: ${snapshot.localNoteReviewQueueCount}`, fg: PALETTE.info },
-      ...compactLocalLibraryLines('Scratchpad Notes', snapshot.localNotes, 'Create one with Create note.', workspace.selectedLocalLibraryItem('note')?.id ?? null),
-      { text: 'Notes stay local unless promoted by explicit action.', fg: PALETTE.warn },
-    );
-  } else if (category.id === 'personas') {
-    base.push(
-      { text: `Personas: ${snapshot.localPersonaCount}; active: ${snapshot.activePersonaName}`, fg: PALETTE.info },
-      ...compactLocalLibraryLines('Persona Library', snapshot.localPersonas, 'Create one with Create persona.', workspace.selectedLocalLibraryItem('persona')?.id ?? null),
-      { text: 'Personas shape the serial main-conversation assistant.', fg: PALETTE.good },
-    );
-  } else if (category.id === 'skills') {
-    base.push(
-      { text: `Skills: ${snapshot.localSkillCount}; enabled: ${snapshot.enabledSkillCount}; bundles: ${snapshot.localSkillBundleCount}; enabled bundles: ${snapshot.enabledSkillBundleCount}; active skills: ${snapshot.activeSkillCount}`, fg: PALETTE.info },
-      ...compactLocalLibraryLines('Skill Library', snapshot.localSkills, 'Create one with Create skill.', workspace.selectedLocalLibraryItem('skill')?.id ?? null),
-      ...compactLocalLibraryLines('Skill Bundles', snapshot.localSkillBundles, 'Create one after adding skills.', null),
-      { text: 'Enabled skills/bundles become operating guidance; secrets are rejected.', fg: PALETTE.warn },
-    );
-  } else if (category.id === 'routines') {
-    const ready = readyRoutineItems(snapshot);
-    const needsSetup = routinesNeedingSetup(snapshot);
-    const needsReview = routinesNeedingReview(snapshot);
-    base.push(
-      { text: `Routines: ${snapshot.localRoutineCount}; enabled: ${snapshot.enabledRoutineCount}`, fg: PALETTE.info },
-      { text: `Schedule-ready routines: ${ready.length}; setup gaps: ${needsSetup.length}; review needed: ${needsReview.length}`, fg: needsSetup.length > 0 || needsReview.length > 0 ? PALETTE.warn : PALETTE.good },
-      routineNextActionLine(snapshot),
-      compactRoutineReceiptLine(snapshot),
-      ...compactLocalLibraryLines('Routine Library', snapshot.localRoutines, 'Create one with Create routine.', workspace.selectedLocalLibraryItem('routine')?.id ?? null),
-      { text: 'Scheduling requires a confirmed action.', fg: PALETTE.warn },
-    );
-  } else if (category.id === 'work') {
-    base.push(
-      { text: 'Work plans and approvals are read or explicitly confirmed.', fg: PALETTE.info },
-      { text: 'Selection alone does not approve, deny, cancel, or mutate requests.', fg: PALETTE.good },
-      { text: 'Approval actions require id plus typed confirmation.', fg: PALETTE.warn },
-    );
-  } else if (category.id === 'automation') {
-    const ready = readyRoutineItems(snapshot);
-    base.push(
-      { text: `Automation: ${ready.length} schedule-ready routine(s); receipts ${snapshot.routineScheduleReceiptCount}.`, fg: ready.length > 0 ? PALETTE.good : PALETTE.warn },
-      automationNextActionLine(snapshot),
-      compactRoutineReceiptLine(snapshot),
-      { text: 'Reminders and routine promotion require confirmation.', fg: PALETTE.warn },
-    );
-  } else if (category.id === 'delegate') {
-    base.push(
-      { text: 'Build/fix/review work is handed to GoodVibes TUI.', fg: PALETTE.info },
-      { text: `Delegated review policy: ${snapshot.delegatedReviewPolicy}`, fg: PALETTE.warn },
-      { text: 'No coding-role Agent jobs are created here.', fg: PALETTE.good },
-    );
-  } else if (category.id === 'finish') {
-    base.push(
-      { text: 'Apply & close marks onboarding finished for this user.', fg: PALETTE.good },
-      { text: 'Future normal launches start in the main conversation.', fg: PALETTE.info },
-      { text: 'Use /agent, /setup, or /onboarding to reopen this workspace later.', fg: PALETTE.muted },
-    );
-  }
-  if (snapshot.warnings.length > 0) {
-    base.push({ text: `Warnings: ${snapshot.warnings.map((warning) => compactText(warning, 60)).join('; ')}`, fg: PALETTE.warn });
-  }
-  return base;
-}
-
-function editorContextLines(editor: AgentWorkspaceLocalEditor): ContextLine[] {
+function editorContextLines(editor: AgentWorkspaceLocalEditor, snapshot: AgentWorkspaceRuntimeSnapshot | null): ContextLine[] {
   const selected = editor.fields[editor.selectedFieldIndex];
   const lines: ContextLine[] = [
     { text: editor.title, fg: PALETTE.title, bold: true },
@@ -501,6 +120,30 @@ function editorContextLines(editor: AgentWorkspaceLocalEditor): ContextLine[] {
       { text: compactText(selected.hint), fg: PALETTE.muted },
     );
   }
+  lines.push(...reviewerReadinessContextLines(editor, snapshot));
+  if (editor.kind === 'model-compare-handoff-diff') {
+    const left = editor.fields.find((field) => field.id === 'leftArtifactId')?.value.trim();
+    const right = editor.fields.find((field) => field.id === 'rightArtifactId')?.value.trim();
+    lines.push(
+      { text: 'Split view: compare left and right reviewer handoff artifacts without changing model routing.', fg: PALETTE.good },
+      { text: 'Section jumps: all, metadata, policy, related, comparison.', fg: PALETTE.info },
+      left && right
+        ? { text: `Current diff: ${left} -> ${right}.`, fg: PALETTE.good }
+        : { text: 'No complete handoff pair selected; submitting lists recent saved handoffs.', fg: PALETTE.info },
+    );
+    if (snapshot?.recentReviewerHandoffArtifacts.length) {
+      const choices = snapshot.recentReviewerHandoffArtifacts
+        .slice(0, 4)
+        .map((artifact) => `${artifact.id} (${artifact.sourceKind}; related ${artifact.relatedArtifactCount})`)
+        .join(', ');
+      lines.push({ text: `Recent choices: ${choices}.`, fg: PALETTE.muted });
+    }
+  } else if (editor.kind === 'document-reviewer-readiness') {
+    lines.push(
+      { text: 'Preflight checks: comments, suggestions, source artifacts, comparison reveal, route decisions, handoff evidence.', fg: PALETTE.info },
+      { text: 'Use before export, handoff archive, or applying a comparison winner.', fg: PALETTE.good },
+    );
+  }
   return lines;
 }
 
@@ -518,25 +161,26 @@ function buildContextRows(workspace: AgentWorkspace, category: AgentWorkspaceCat
       },
       { text: 'Enter opens; Esc clears.', fg: PALETTE.muted },
     ] satisfies ContextLine[] : []),
-    ...(workspace.localEditor ? editorContextLines(workspace.localEditor) : []),
+    ...(workspace.localEditor ? editorContextLines(workspace.localEditor, workspace.runtimeSnapshot) : []),
   ];
 
   const selectedActionLines: ContextLine[] = action
     ? [
       { text: `Selected: ${action.label}`, fg: PALETTE.title, bold: true },
-      actionMetaLine(action),
+      actionMetaLine(workspace, category, action),
     ]
     : [];
   const snapshotContextLines = snapshotLines(workspace, category, workspace.runtimeSnapshot);
   lines.push(...selectedActionLines, ...snapshotContextLines);
 
+  const onboarding = isOnboardingCategory(category);
   if (workspace.lastActionResult) {
     lines.push(
-      { text: 'Action Result', fg: PALETTE.title, bold: true },
+      { text: onboarding ? 'Result' : 'Action Result', fg: PALETTE.title, bold: true },
       { text: workspace.lastActionResult.title, fg: actionResultColor(workspace.lastActionResult), bold: true },
       { text: compactText(workspace.lastActionResult.detail), fg: PALETTE.text },
     );
-    if (workspace.lastActionResult.command) {
+    if (!onboarding && workspace.lastActionResult.command) {
       lines.push({ text: `Command: ${workspace.lastActionResult.command}`, fg: PALETTE.muted });
     }
   }
@@ -602,6 +246,8 @@ function buildEditorFieldRows(editor: AgentWorkspaceLocalEditor, index: number,
 
 function buildActionRows(workspace: AgentWorkspace, width: number, height: number): WorkspaceRow[] {
   if (workspace.localEditor) return buildEditorRows(workspace.localEditor, width, height);
+  const category = workspace.selectedActionCategory;
+  const onboarding = isOnboardingCategory(category);
   const rows: WorkspaceRow[] = [];
   const labelWidth = Math.min(34, Math.max(18, Math.floor(width * 0.38)));
   const commandWidth = Math.max(10, width - labelWidth - 6);
@@ -613,7 +259,7 @@ function buildActionRows(workspace: AgentWorkspace, width: number, height: numbe
     });
   }
   rows.push({
-    text: `  ${padDisplay(workspace.actionSearchActive ? 'Result' : 'Action', labelWidth)}  ${padDisplay('Does', commandWidth)}`,
+    text: `  ${padDisplay(workspace.actionSearchActive ? 'Result' : onboarding ? 'Option' : 'Action', labelWidth)}  ${padDisplay(onboarding ? 'Change' : 'Does', commandWidth)}`,
     fg: PALETTE.muted,
     bold: true,
   });
@@ -627,10 +273,11 @@ function buildActionRows(workspace: AgentWorkspace, width: number, height: numbe
     const action = actions[index]!;
     const selected = index === workspace.selectedActionIndex;
     const searchResult = workspace.actionSearchActive ? workspace.actionSearchResults[index] : null;
+    const actionCategory = searchResult?.category ?? category;
     const label = searchResult ? `${searchResult.category.label} / ${action.label}` : action.label;
     const marker = selected ? GLYPHS.navigation.selected : ' ';
     rows.push({
-      text: `${marker} ${padDisplay(label, labelWidth)}  ${padDisplay(actionCommand(action), commandWidth)}`,
+      text: `${marker} ${padDisplay(label, labelWidth)}  ${padDisplay(actionChange(workspace, actionCategory, action), commandWidth)}`,
       selected: selected && workspace.focusPane === 'actions',
       fg: action.safety === 'blocked' ? PALETTE.warn : selected ? PALETTE.text : PALETTE.info,
       bold: selected,
@@ -642,11 +289,11 @@ function buildActionRows(workspace: AgentWorkspace, width: number, height: numbe
   rows.push({ text: `Status: ${workspace.status}`, fg: PALETTE.muted });
   if (workspace.lastActionResult) {
     rows.push({ text: '' });
-    rows.push({ text: `Action Result: ${workspace.lastActionResult.title}`, fg: actionResultColor(workspace.lastActionResult), bold: true });
+    rows.push({ text: `${onboarding ? 'Result' : 'Action Result'}: ${workspace.lastActionResult.title}`, fg: actionResultColor(workspace.lastActionResult), bold: true });
     for (const line of wrapText(workspace.lastActionResult.detail, Math.max(1, width - 2))) {
       rows.push({ text: `  ${line}`, fg: PALETTE.text });
     }
-    if (workspace.lastActionResult.command) {
+    if (!onboarding && workspace.lastActionResult.command) {
       rows.push({ text: `  Command: ${workspace.lastActionResult.command}`, fg: PALETTE.muted });
     }
   }

package/src/renderer/settings-modal-helpers.ts

@@ -85,6 +85,18 @@ export const CATEGORY_LABELS: Record<(typeof SETTINGS_CATEGORIES)[number], strin
   helper: 'Helper',
   tts: 'TTS',
   automation: 'Automation',
+  service: 'Service',
+  controlPlane: 'Control Plane',
+  httpListener: 'HTTP Listener',
+  web: 'Web',
+  watchers: 'Watchers',
+  network: 'Network',
+  orchestration: 'Orchestration',
+  runtime: 'Runtime',
+  sandbox: 'Sandbox',
+  batch: 'Batch',
+  cloudflare: 'Cloudflare',
+  wrfc: 'WRFC',
   telemetry: 'Telemetry',
   cache: 'Cache',
   mcp: 'MCP',

package/src/renderer/settings-modal.ts

@@ -34,6 +34,18 @@ const CATEGORY_INFO: Record<SettingsCategory, string> = {
   helper: 'Helper model defaults used by helper subsystems when they do not use the main chat route.',
   tts: 'Text-to-speech provider, voice, and optional spoken-turn LLM overrides.',
   automation: 'Scheduled and automated run settings, concurrency, timeout, catch-up, cooldown, and retention behavior.',
+  service: 'GoodVibes daemon service posture and restart/autostart preferences.',
+  controlPlane: 'Control-plane endpoint, stream, remote access, and TLS settings used by daemon-backed operator routes.',
+  httpListener: 'HTTP listener binding, trust proxy, and TLS settings for inbound companion/channel routes.',
+  web: 'Web companion surface settings including host, port, public URL, and static asset path.',
+  watchers: 'Polling watcher and heartbeat behavior for runtime recovery and periodic checks.',
+  network: 'Outbound TLS and remote fetch network policy.',
+  orchestration: 'Visible agent orchestration limits such as recursion and active-agent caps.',
+  runtime: 'Runtime service limits and event bus settings.',
+  sandbox: 'Isolation settings for REPL, MCP, and VM-backed sessions.',
+  batch: 'Batch queue backend, limits, and provider batching behavior.',
+  cloudflare: 'Cloudflare worker, tunnel, queue, storage, and token-reference settings.',
+  wrfc: 'WRFC review/fix chain scoring, fix attempts, and commit preferences.',
   telemetry: 'Telemetry payload policy.',
   cache: 'Provider and model cache behavior, TTL, and hit-rate monitoring.',
   mcp: 'MCP server trust and scope review. Trust changes can expose local files, tools, databases, browsers, or remote automation depending on the server.',