@pellux/goodvibes-agent 1.4.4 → 1.5.0

package/src/shell/startup-wiring.ts

@@ -0,0 +1,221 @@
+import { logger, summarizeError } from '@pellux/goodvibes-sdk/platform/utils';
+import type { HookDispatcher, HookEventPath, HookPhase, HookCategory } from '@pellux/goodvibes-sdk/platform/hooks';
+import type { SessionManager } from '@pellux/goodvibes-sdk/platform/sessions';
+import {
+  checkRecoveryFile,
+  formatReturnContextForDisplay,
+  persistConversation,
+  writeRecoveryFile,
+} from '@/runtime/index.ts';
+import type { SessionSnapshot } from '@/runtime/index.ts';
+import {
+  readOnboardingCheckMarker,
+  readOnboardingCompletionMarker,
+} from '../runtime/onboarding/index.ts';
+import { deriveOnboardingState } from '../runtime/onboarding/onboarding-state.ts';
+import { buildSetupIncompleteHint } from '../core/setup-incomplete-hint.ts';
+
+export interface SessionPersistenceAndRecoveryDeps {
+  readonly buildCurrentSessionSnapshot: () => SessionSnapshot;
+  readonly runtime: { readonly sessionId: string; readonly model: string; readonly provider: string };
+  readonly conversation: { readonly title?: string | null };
+  readonly workingDir: string;
+  readonly homeDirectory: string;
+  readonly systemMessageRouter: { high(message: string): void; low(message: string): void };
+  readonly render: () => void;
+  readonly unsubs: Array<() => void>;
+  readonly uiServicesTurns: {
+    on(event: 'TURN_COMPLETED' | 'STREAM_START' | 'STREAM_DELTA', handler: () => void): () => void;
+  };
+  readonly hookDispatcher: HookDispatcher;
+  readonly sessionManager: SessionManager;
+  /**
+   * Called whenever the computed stream-token speed changes so that main.ts
+   * (which owns the render closure) can keep the value up to date.
+   */
+  readonly onStreamSpeedUpdate: (tokensPerSecond: number) => void;
+}
+
+export interface SessionPersistenceAndRecoveryResult {
+  /** Interval handle for the periodic recovery-file writer; clear it on exit. */
+  recoveryInterval: ReturnType<typeof setInterval>;
+  /** True if an unsaved recovery session was found and the user prompt was shown. */
+  recoveryPending: boolean;
+}
+
+/**
+ * Wires streaming-speed event subscriptions, auto-save on turn completion,
+ * and the post-first-render recovery-file check + periodic writer.
+ *
+ * Call AFTER the first render() so that:
+ * - announceAwayDigest has already run
+ * - startHardwareProbe has already fired
+ * - The recovery prompt lands as ambient context, not a startup blocker
+ */
+export function wireSessionPersistenceAndRecovery(
+  deps: SessionPersistenceAndRecoveryDeps,
+): SessionPersistenceAndRecoveryResult {
+  const {
+    buildCurrentSessionSnapshot,
+    runtime,
+    conversation,
+    workingDir,
+    homeDirectory,
+    systemMessageRouter,
+    render,
+    unsubs,
+    uiServicesTurns,
+    hookDispatcher,
+    sessionManager,
+    onStreamSpeedUpdate,
+  } = deps;
+
+  // --- Streaming speed + tool preview wiring ---
+  let streamStartTime = 0;
+  let streamDeltaCount = 0;
+
+  unsubs.push(uiServicesTurns.on('TURN_COMPLETED', () => {
+    // Auto-save after every LLM turn so kills don't lose the session
+    try {
+      const snapshot = buildCurrentSessionSnapshot();
+      persistConversation(
+        runtime.sessionId,
+        snapshot,
+        runtime.model,
+        runtime.provider,
+        conversation.title || '',
+        { workingDirectory: workingDir, homeDirectory, sessionManager },
+      );
+      hookDispatcher.fire({ path: 'Lifecycle:session:save' as HookEventPath, phase: 'Lifecycle' as HookPhase, category: 'session' as HookCategory, specific: 'save', sessionId: runtime.sessionId, timestamp: Date.now(), payload: { sessionId: runtime.sessionId } }).catch((err: unknown) => logger.debug('hook fire error', { error: summarizeError(err) }));
+    } catch (e) { logger.debug('auto-save on turn:complete failed', { error: summarizeError(e) }); }
+  }));
+
+  unsubs.push(uiServicesTurns.on('STREAM_START', () => {
+    streamStartTime = Date.now();
+    streamDeltaCount = 0;
+    onStreamSpeedUpdate(0);
+  }));
+  unsubs.push(uiServicesTurns.on('STREAM_DELTA', () => {
+    streamDeltaCount++;
+    const elapsed = (Date.now() - streamStartTime) / 1000;
+    // Note: counts stream deltas, not actual tokens. ~1 delta per token for most providers.
+    onStreamSpeedUpdate(elapsed > 0 ? streamDeltaCount / elapsed : 0);
+  }));
+
+  // Recovery file check: display prompt if an unsaved session exists.
+  // Runs after the first render so the message lands as ambient context.
+  let recoveryPending = false;
+  const recoveryInfo = checkRecoveryFile({ workingDirectory: workingDir, homeDirectory });
+  if (recoveryInfo) {
+    systemMessageRouter.high(`[Recovery] Found unsaved session from ${new Date(recoveryInfo.timestamp).toLocaleString()}. Title: "${recoveryInfo.title}". Press Ctrl+R to restore, Esc to discard, or start typing to ignore it.`);
+    for (const line of formatReturnContextForDisplay(recoveryInfo.returnContext)) {
+      systemMessageRouter.low(`[Recovery] ${line}`);
+    }
+    render();
+    recoveryPending = true;
+  }
+
+  const recoveryInterval = setInterval(() => {
+    const snapshot = buildCurrentSessionSnapshot();
+    writeRecoveryFile(
+      snapshot,
+      runtime.sessionId,
+      conversation.title ?? '',
+      { workingDirectory: workingDir, homeDirectory },
+    );
+  }, 60_000);
+
+  return { recoveryInterval, recoveryPending };
+}
+
+/**
+ * Deps for the setup-incomplete hint, kept narrow so the function is easily
+ * testable and doesn't pull in the full bootstrap context.
+ */
+export interface SetupIncompleteHintDeps {
+  /**
+   * Shell path service — used to locate onboarding marker files.
+   * Must satisfy the minimal interface required by readOnboardingCheckMarker.
+   */
+  readonly shellPaths: Parameters<typeof readOnboardingCheckMarker>[0];
+  /**
+   * Whether a model route is currently configured and usable for chat.
+   * Pass true when the provider registry has a ready model, false otherwise.
+   */
+  readonly providerReady: boolean;
+  /**
+   * Whether a local model route has been detected and is ready for chat.
+   * Pass true when the local-model-cookbook status === 'detected-local-route'.
+   * Omit or pass false when unavailable. Best-effort: errors should produce false.
+   * This mirrors the 'local-model-readiness' plan item in the real buildSetupPlan.
+   */
+  readonly localReady?: boolean;
+  /**
+   * Whether the assistant service (connected host) is known to be running.
+   * Pass true/false for a confirmed live signal; omit or pass null/undefined
+   * when the signal is unreliable or unavailable at startup time.
+   */
+  readonly hostReady?: boolean | null;
+  /** Low-priority message channel — same interface as SystemMessageRouter.low(). */
+  readonly systemMessageRouter: { low(message: string): void };
+}
+
+/**
+ * Derives the current onboarding state from disk markers and, if setup is
+ * in-progress, pushes a plain-language hint to the conversation feed.
+ *
+ * Best-effort: any error is caught silently so startup is never blocked.
+ * Call after the first render, alongside announceAwayDigest.
+ */
+export function wireSetupIncompleteHint(deps: SetupIncompleteHintDeps): void {
+  try {
+    const { shellPaths, providerReady, localReady, hostReady, systemMessageRouter } = deps;
+
+    const checkMarker = readOnboardingCheckMarker(shellPaths, 'user');
+    const completionMarker = readOnboardingCompletionMarker(shellPaths, 'user');
+
+    // Build a minimal plan with BOTH provider-access AND local-model-readiness so
+    // that deriveReadyToChat mirrors the workspace's canonical definition:
+    //   readyToChat = providerItem.ready OR localModelItem.ready
+    //
+    // local-model-readiness uses blocksAutonomy:false so it never becomes a false
+    // blocker when only the local route is available (matching real plan behaviour).
+    // We intentionally skip the full buildSetupPlan (requires CommandContext +
+    // async collectSnapshot) because this is a best-effort ambient hint.
+    const minimalPlan = [
+      {
+        id: 'provider-access',
+        label: 'Model access',
+        status: providerReady ? ('ready' as const) : ('blocked' as const),
+        priority: 20,
+        blocksAutonomy: true,
+        reason: '',
+        nextAction: 'Choose a model to start chatting.',
+        userRoute: 'Agent Workspace -> Start -> Choose main model',
+        modelRoute: '',
+      },
+      {
+        id: 'local-model-readiness',
+        label: 'Local model',
+        status: localReady ? ('ready' as const) : ('recommended' as const),
+        priority: 25,
+        blocksAutonomy: false,
+        reason: '',
+        nextAction: 'Run /agent to set up a local model route.',
+        userRoute: 'Agent Workspace -> Models -> Local models',
+        modelRoute: '',
+      },
+    ];
+
+    const state = deriveOnboardingState({ plan: minimalPlan, checkMarker, completionMarker });
+
+    const hint = buildSetupIncompleteHint(state, hostReady);
+    if (hint === null) return;
+
+    for (const line of hint.lines) {
+      systemMessageRouter.low(`[Setup] ${line}`);
+    }
+  } catch {
+    // Never block startup on hint errors.
+  }
+}

package/src/shell/ui-openers.ts

@@ -3,7 +3,6 @@ import { getProviderIdFromModel } from '../config/provider-model.ts';
 import type { ConversationManager } from '../core/conversation';
 import type { CommandContext } from '../input/command-registry.ts';
 import type { InputHandler } from '../input/handler.ts';
-import type { PanelManager } from '../panels/panel-manager.ts';
 import { EFFORT_DESCRIPTIONS } from '@pellux/goodvibes-sdk/platform/providers';
 import type { ProviderRegistry } from '@pellux/goodvibes-sdk/platform/providers';
 import type { MutableRuntimeState } from '@/runtime/index.ts';
@@ -13,13 +12,12 @@ import type { SubscriptionManager } from '@pellux/goodvibes-sdk/platform/config'
 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 { buildLocalFitRecommendations, buildSignInRow, LOCAL_REC_PROVIDER } from '../input/model-picker-local-fit.ts';
 import { syncServiceSettingToPlatform } from './service-settings-sync.ts';
-import { agentWorkspaceCategoryForPanel } from '../input/agent-workspace-panel-route.ts';
 
 type WireShellUiOpenersOptions = {
   commandContext: CommandContext;
   input: InputHandler;
-  panelManager: PanelManager;
   conversation: ConversationManager;
   configManager: ConfigManager;
   providerRegistry: ProviderRegistry;
@@ -84,7 +82,6 @@ export function wireShellUiOpeners(options: WireShellUiOpenersOptions): void {
   const {
     commandContext,
     input,
-    panelManager,
     conversation,
     configManager,
     providerRegistry,
@@ -190,10 +187,19 @@ export function wireShellUiOpeners(options: WireShellUiOpenersOptions): void {
 
   commandContext.openModelPicker = () => {
     void (async () => {
-      const models = providerRegistry.getSelectableModels();
+      const catalogModels = providerRegistry.getSelectableModels();
       const configuredIds = new Set(getConfiguredProviderIds());
       input.modelPicker.configuredProviders = configuredIds;
-      const providerIds = [...new Set(models.map((m) => m.provider))];
+
+      // CRITICAL GATE: inject synthetic local recommendations ONLY when zero
+      // providers are configured. When any real credential exists these must
+      // not be present in the list. The sign-in row is appended last so it
+      // is always reachable but does not displace hardware-fit entries.
+      const models = configuredIds.size === 0
+        ? [...buildLocalFitRecommendations(), buildSignInRow(), ...catalogModels]
+        : catalogModels;
+
+      const providerIds = [...new Set(models.map((m) => m.provider).filter((p) => p !== LOCAL_REC_PROVIDER))];
       const secretProviderIds = await resolveSecretProviderIds();
       input.modelPicker.configuredViaMap = buildConfiguredViaMap(providerIds, configuredIds, subscriptionManager, secretProviderIds);
       void getPinned().then((pinned) => {
@@ -202,7 +208,11 @@ export function wireShellUiOpeners(options: WireShellUiOpenersOptions): void {
       void input.modelPicker.loadRecentModels().catch(() => {}); // best-effort: prefetch for UI, failure is non-visible
       input.modalOpened('modelPicker');
       input.modelPicker.setTargetInfos(buildModelPickerTargets());
-      input.modelPicker.openAllModels(models, getCurrentModelForPickerTarget());
+      // Pre-select the best-fit local recommendation when no providers are configured.
+      const preSelectId = configuredIds.size === 0 && models.length > 0
+        ? (models[0]?.registryKey ?? models[0]?.id ?? getCurrentModelForPickerTarget())
+        : getCurrentModelForPickerTarget();
+      input.modelPicker.openAllModels(models, preSelectId);
       render();
     })().catch((error: unknown) => {
       commandContext.print?.(`Model picker failed to open: ${error instanceof Error ? error.message : String(error)}`);
@@ -439,36 +449,15 @@ export function wireShellUiOpeners(options: WireShellUiOpenersOptions): void {
     render();
   };
 
-  commandContext.openPanelPicker = () => {
-    panelManager.hide();
-    input.panelFocused = false;
+  commandContext.openWorkspacePicker = () => {
     conversation.setSplashSuppressed(false);
-    conversation.log('Panel picker is handled through Agent Workspace. Use /agent for current operator controls.', { fg: '214' });
     input.openAgentWorkspace(commandContext, 'home');
     conversation.rebuildHistory();
     render();
   };
 
-  commandContext.focusPanels = () => {
-    input.panelFocused = false;
-    input.openAgentWorkspace(commandContext, 'home');
-    render();
-  };
-
   commandContext.focusPrompt = () => {
-    input.panelFocused = false;
     input.indicatorFocused = false;
     render();
   };
-
-  commandContext.showPanel = (panelId, pane) => {
-    void pane;
-    panelManager.hide();
-    input.panelFocused = false;
-    conversation.setSplashSuppressed(false);
-    conversation.log(`Panel route "${panelId}" is handled through Agent Workspace. Opening the matching operator area.`, { fg: '214' });
-    input.openAgentWorkspace(commandContext, agentWorkspaceCategoryForPanel(panelId));
-    conversation.rebuildHistory();
-    render();
-  };
 }

package/src/tools/agent-context-policy.ts

@@ -160,7 +160,7 @@ function buildAgentCapabilitiesContract(registry?: ToolRegistry): Record<string,
       },
       {
         area: 'Harness operation',
-        can: 'Discover and use harness modes, slash commands, workspace actions, settings, panels, UI surfaces, keybindings, and model tools.',
+        can: 'Discover and use harness modes, slash commands, workspace actions, settings, UI surfaces, keybindings, and model tools.',
         tools: [optionalTool('agent_harness'), optionalTool('host'), optionalTool('settings'), 'goodvibes_context'],
         inspect: 'agent_harness mode:"modes" query:"capability"',
       },

package/src/tools/agent-harness-background-processes.ts

@@ -658,7 +658,7 @@ export async function runBackgroundProcessAction(context: CommandContext, args:
   const action = readProcessAction(args);
   if (action === 'write') {
     const confirmationError = requireConfirmed(args, 'Background process stdin write');
-    if (confirmationError) throw new Error(confirmationError);
+    if (confirmationError) return { status: 'needs_confirmation', reason: confirmationError, policy: 'Writing stdin to a background process requires confirm:true and explicitUserRequest.' };
     const writeInput = firstManagerFunction(manager, STDIN_WRITE_METHOD_NAMES);
     if (!writeInput) {
       return {
@@ -710,7 +710,7 @@ export async function runBackgroundProcessAction(context: CommandContext, args:
   }
   if (action === 'start' || action === 'spawn' || action === 'run') {
     const confirmationError = requireConfirmed(args, 'Background process start');
-    if (confirmationError) throw new Error(confirmationError);
+    if (confirmationError) return { status: 'needs_confirmation', reason: confirmationError, policy: 'Starting a background process requires confirm:true and explicitUserRequest.' };
     const command = readCommand(args);
     if (!command) throw new Error('Background process start requires command.');
     if (looksLikeSudo(command)) {
@@ -749,7 +749,7 @@ export async function runBackgroundProcessAction(context: CommandContext, args:
 
   if (action === 'stop' || action === 'kill' || action === 'cancel') {
     const confirmationError = requireConfirmed(args, 'Background process stop');
-    if (confirmationError) throw new Error(confirmationError);
+    if (confirmationError) return { status: 'needs_confirmation', reason: confirmationError, policy: 'Stopping a background process requires confirm:true and explicitUserRequest.' };
     const processId = readProcessId(args);
     if (!processId) throw new Error('Background process stop requires processId.');
     const stopped = manager.stop(processId);
@@ -767,7 +767,7 @@ export async function runBackgroundProcessAction(context: CommandContext, args:
 
   if (action === 'wait') {
     const confirmationError = requireConfirmed(args, 'Background process wait');
-    if (confirmationError) throw new Error(confirmationError);
+    if (confirmationError) return { status: 'needs_confirmation', reason: confirmationError, policy: 'Waiting on a background process requires confirm:true and explicitUserRequest.' };
     const processId = readProcessId(args);
     if (!processId) throw new Error('Background process wait requires processId.');
     const timeoutMs = clampTimeout(args.timeoutMs ?? readField(args, 'timeoutMs'), DEFAULT_WAIT_TIMEOUT_MS);

package/src/tools/agent-harness-browser-cockpit-route.ts

@@ -36,7 +36,7 @@ const BROWSER_COCKPIT_EXPECTED_LANES = [
   {
     id: 'mobile-and-channels',
     label: 'Mobile and channels',
-    categoryIds: ['onboarding-channels', 'onboarding-voice-media', 'channels', 'voice-media', 'personal-ops'],
+    categoryIds: ['onboarding-channels', 'onboarding-voice-media', 'personal-ops'],
     userOutcome: 'Configure mobile-friendly messaging, notification, voice, phone, and daily personal-operation routes from one cockpit.',
   },
   {
@@ -48,13 +48,13 @@ const BROWSER_COCKPIT_EXPECTED_LANES = [
   {
     id: 'work-and-automation',
     label: 'Work and automation',
-    categoryIds: ['work', 'automation', 'onboarding-automation', 'delegate'],
+    categoryIds: ['work', 'automation'],
     userOutcome: 'Approve, supervise, recover, schedule, and delegate autonomous work with touch-friendly controls and receipts.',
   },
   {
     id: 'research-and-artifacts',
     label: 'Research and artifacts',
-    categoryIds: ['research', 'documents', 'artifacts'],
+    categoryIds: ['research', 'documents'],
     userOutcome: 'Run research, draft documents, review packets, browse artifacts, and export/share deliverables from the browser.',
   },
   {

package/src/tools/agent-harness-command-runner.ts

@@ -34,9 +34,14 @@ export async function runCommand(deps: AgentHarnessToolDeps, args: AgentHarnessT
   const commandArgs = invocationArgsFromLookup(resolved.lookup);
   const handled = await deps.commandRegistry.execute(resolved.command.name, [...commandArgs], toolContext);
   if (!handled) return error(`Unknown slash command /${resolved.command.name}.`);
+  const MAX_PRINTED_CHARS = 6000;
+  const raw = printed.length > 0 ? printed.join('\n') : '(no text output)';
+  const printedText = raw.length > MAX_PRINTED_CHARS
+    ? `${raw.slice(0, MAX_PRINTED_CHARS)}\n... output truncated`
+    : raw;
   return output([
     `Command ${safeCommandDisplay(resolved.command.name)} completed.`,
     `Resolved by ${resolved.lookup.source} ${resolved.lookup.resolvedBy}.`,
-    printed.length > 0 ? printed.join('\n') : '(no text output)',
+    printedText,
   ].join('\n'));
 }

package/src/tools/agent-harness-document-ops.ts

@@ -222,19 +222,14 @@ function buildLanes(context: CommandContext): readonly DocumentOpsLane[] {
     'document-draft-chat',
   ], available);
   const uploadActions = existingActions([
-    'artifact-attach-image',
-    'artifact-paste',
-    'artifact-ingest-file',
-    'artifact-ingest-url-list',
-    'artifact-import-bookmarks',
-    'artifact-browser-history',
     'document-attach-image',
-    'document-paste',
     'document-ingest-file',
+    'document-ingest-url-list',
+    'knowledge-import-bookmarks',
+    'knowledge-import-browser-history',
+    'conversation-image',
   ], available);
   const exportActions = existingActions([
-    'artifact-export-conversation',
-    'artifact-session-export',
     'conversation-export-current',
     'conversation-session-export',
     'document-export-draft',
@@ -242,20 +237,14 @@ function buildLanes(context: CommandContext): readonly DocumentOpsLane[] {
     'document-export-artifact-package',
     'document-export-conversation',
     'document-export-session',
-    'artifact-export-file',
-    'artifact-export-package',
   ], available);
   const sourceActions = existingActions([
-    'artifact-source-library',
-    'artifact-show-source',
     'research-knowledge-search',
     'research-knowledge-ask',
     'document-sources',
     'document-show-source',
   ], available);
   const mediaActions = existingActions([
-    'artifact-media-providers',
-    'artifact-generate-media',
     'media-providers',
     'media-generate',
     'document-media-providers',
@@ -267,16 +256,8 @@ function buildLanes(context: CommandContext): readonly DocumentOpsLane[] {
     'document-export-artifact-file',
     'document-export-artifact-package',
     'document-promote-artifact',
-    'artifact-browse',
-    'artifact-show',
-    'artifact-export-file',
-    'artifact-export-package',
-    'artifact-promote-knowledge',
-    'artifact-insert-document',
-    'artifact-attach-document',
-    'document-artifacts',
-    'artifact-flow',
-    'artifact-show-source',
+    'document-insert-artifact',
+    'document-attach-artifact',
   ], available);
   const modelCompareActions = existingActions([
     'document-run-compare',
@@ -287,12 +268,6 @@ function buildLanes(context: CommandContext): readonly DocumentOpsLane[] {
     'document-apply-compare',
     'document-record-route-decision',
     'document-export-compare',
-    'artifact-review-compare',
-    'artifact-diff-handoffs',
-    'artifact-judge-compare',
-    'artifact-compare-analytics',
-    'artifact-apply-compare',
-    'artifact-export-compare',
     'document-model-routing',
     'account-main-model',
   ], available);
@@ -314,11 +289,11 @@ function buildLanes(context: CommandContext): readonly DocumentOpsLane[] {
     && hasTool(context, 'agent_knowledge_ingest')
     && Boolean(context.platform.artifactStore?.list)
     && Boolean(context.platform.artifactStore?.readContent)
-    && artifactBrowserActions.includes('artifact-browse')
-    && artifactBrowserActions.includes('artifact-show')
-    && artifactBrowserActions.includes('artifact-export-file')
-    && artifactBrowserActions.includes('artifact-export-package')
-    && artifactBrowserActions.includes('artifact-promote-knowledge');
+    && artifactBrowserActions.includes('document-browse-artifacts')
+    && artifactBrowserActions.includes('document-show-artifact')
+    && artifactBrowserActions.includes('document-export-artifact-file')
+    && artifactBrowserActions.includes('document-export-artifact-package')
+    && artifactBrowserActions.includes('document-promote-artifact');
   const reviewerReadiness = buildReviewerReadinessChecklist(context, {
     documentsReady,
     modelCompareReady,
@@ -337,7 +312,7 @@ function buildLanes(context: CommandContext): readonly DocumentOpsLane[] {
       next: documentsReady
         ? 'Use versioned drafts, comments, suggestions, artifact reuse, artifact packages or ZIP archives, and artifact-backed comparison as one document workflow.'
         : 'Wire agent_documents plus browse/show/create/revise/review/export workspace actions.',
-      userRoute: 'Agent Workspace -> Documents & Compare -> Create document draft',
+      userRoute: 'Agent Workspace -> Documents & Files -> Create document draft',
       modelRoute: documentsReady ? 'agent_documents' : 'agent_harness mode:"workspace_actions" categoryId:"documents"',
       signals: [
         `Chat route ${snapshot.provider} / ${snapshot.modelDisplayName}`,
@@ -363,7 +338,7 @@ function buildLanes(context: CommandContext): readonly DocumentOpsLane[] {
       next: uploadActions.length > 0
         ? 'Use attach, paste, or reviewed source ingest depending on whether the file is prompt context or durable Agent Knowledge.'
         : 'Wire upload, paste, and file ingest actions before exposing this as ready.',
-      userRoute: 'Agent Workspace -> Documents & Compare -> Attach or ingest',
+      userRoute: 'Agent Workspace -> Documents & Files -> Attach or ingest',
       modelRoute: 'agent_harness mode:"workspace_actions" query:"upload file ingest"',
       signals: [
         `${uploadActions.length} upload/ingest action(s)`,
@@ -378,7 +353,7 @@ function buildLanes(context: CommandContext): readonly DocumentOpsLane[] {
       outcome: 'Turn conversations, sessions, documents, and saved artifacts into local files the user can keep or reuse.',
       current: `${exportActions.length} export action(s) are reachable for conversation, session, document, comparison, and saved artifact output.`,
       next: 'Keep transcript, session, document, comparison, and artifact exports visible, explicit, and reusable from one place.',
-      userRoute: 'Agent Workspace -> Documents & Compare -> Export',
+      userRoute: 'Agent Workspace -> Documents & Files -> Export',
       modelRoute: 'agent_harness mode:"workspace_actions" query:"export artifact"',
       signals: [
         `${exportActions.length} export action(s)`,
@@ -393,7 +368,7 @@ function buildLanes(context: CommandContext): readonly DocumentOpsLane[] {
       outcome: 'Know what must be resolved before exporting, handing off, archiving, or applying comparison-backed route changes.',
       current: `${reviewerReadiness.summary.documents} document draft(s), ${reviewerReadiness.summary.openComments} open comment(s), ${reviewerReadiness.summary.proposedSuggestions} proposed suggestion(s), ${reviewerReadiness.summary.savedComparisons} saved comparison(s), ${reviewerReadiness.summary.revealedJudgments} revealed judgment(s).`,
       next: reviewerReadiness.next,
-      userRoute: 'Agent Workspace -> Documents & Compare -> Review readiness preflight',
+      userRoute: 'Agent Workspace -> Documents & Files -> Review readiness preflight',
       modelRoute: 'agent_harness mode:"document_ops_lane" laneId:"reviewer_readiness"',
       signals: [
         `Reviewer readiness ${reviewerReadiness.status}`,
@@ -423,9 +398,8 @@ function buildLanes(context: CommandContext): readonly DocumentOpsLane[] {
           'document-export-compare',
         ].includes(id)),
         ...artifactBrowserActions.filter((id) => [
-          'artifact-browse',
-          'artifact-show',
-          'artifact-export-package',
+          'document-browse-artifacts',
+          'document-show-artifact',
         ].includes(id)),
       ],
       reviewerReadiness,
@@ -445,7 +419,7 @@ function buildLanes(context: CommandContext): readonly DocumentOpsLane[] {
         ? `${snapshot.reviewPacketTimeline.count} packet event(s); latest ${snapshot.reviewPacketTimeline.items[0].label}.`
         : 'No document, artifact, comparison, judgment, handoff, archive, or route-decision packet events are available yet.',
       next: snapshot.reviewPacketTimeline.next,
-      userRoute: 'Agent Workspace -> Documents & Compare -> Review packet timeline',
+      userRoute: 'Agent Workspace -> Documents & Files -> Review packet timeline',
       modelRoute: 'agent_harness mode:"document_ops_lane" laneId:"review_packet_timeline"',
       signals: [
         `Timeline available ${snapshot.reviewPacketTimeline.available ? 'yes' : 'no'}`,
@@ -471,9 +445,8 @@ function buildLanes(context: CommandContext): readonly DocumentOpsLane[] {
           'document-export-compare',
         ].includes(id)),
         ...artifactBrowserActions.filter((id) => [
-          'artifact-browse',
-          'artifact-show',
-          'artifact-export-package',
+          'document-browse-artifacts',
+          'document-show-artifact',
         ].includes(id)),
       ],
     },
@@ -492,7 +465,7 @@ function buildLanes(context: CommandContext): readonly DocumentOpsLane[] {
         ? `${snapshot.reviewPacketWizard.completedSteps}/${snapshot.reviewPacketWizard.totalSteps} step(s) complete; current ${snapshot.reviewPacketWizard.currentStepLabel}.`
         : `${snapshot.reviewPacketWizard.completedSteps}/${snapshot.reviewPacketWizard.totalSteps} step(s) complete.`,
       next: snapshot.reviewPacketWizard.next,
-      userRoute: 'Agent Workspace -> Documents & Compare -> Review packet wizard',
+      userRoute: 'Agent Workspace -> Documents & Files -> Review packet wizard',
       modelRoute: 'agent_harness mode:"document_ops_lane" laneId:"review_packet_wizard"',
       signals: [
         `Wizard status ${snapshot.reviewPacketWizard.status}`,
@@ -516,7 +489,7 @@ function buildLanes(context: CommandContext): readonly DocumentOpsLane[] {
       outcome: 'Inspect source-backed records before citing, summarizing, or promoting knowledge.',
       current: `${sourceActions.length} source lookup/search action(s) are reachable through isolated Agent Knowledge.`,
       next: 'Use source search/show for citation checks; ingest reviewed files or URLs only through explicit source actions.',
-      userRoute: 'Agent Workspace -> Documents & Compare -> Sources',
+      userRoute: 'Agent Workspace -> Documents & Files -> Sources',
       modelRoute: 'agent_harness mode:"workspace_actions" query:"Agent Knowledge sources"',
       signals: [
         `Knowledge route ${snapshot.knowledgeRoute}`,
@@ -533,7 +506,7 @@ function buildLanes(context: CommandContext): readonly DocumentOpsLane[] {
       next: snapshot.mediaGenerationProviderCount > 0
         ? 'Use confirmed media generation when the user asks for a concrete artifact.'
         : 'Configure a media generation provider before claiming generated media is ready.',
-      userRoute: 'Agent Workspace -> Documents & Compare -> Generate media',
+      userRoute: 'Agent Workspace -> Documents & Files -> Generate media',
       modelRoute: 'agent_media_generate',
       signals: [
         `${mediaActions.length} media action(s)`,
@@ -559,8 +532,8 @@ function buildLanes(context: CommandContext): readonly DocumentOpsLane[] {
         `Artifact browser tool: ${hasTool(context, 'agent_artifacts') ? 'available' : 'gap'}`,
         `Knowledge ingest tool: ${hasTool(context, 'agent_knowledge_ingest') ? 'available' : 'gap'}`,
         `Artifact list/read store: ${context.platform.artifactStore?.list && context.platform.artifactStore?.readContent ? 'available' : 'gap'}`,
-        `Artifact export action: ${artifactBrowserActions.includes('artifact-export-file') ? 'available' : 'gap'}`,
-        `Artifact package export action: ${artifactBrowserActions.includes('artifact-export-package') ? 'available' : 'gap'}`,
+        `Artifact export action: ${artifactBrowserActions.includes('document-export-artifact-file') ? 'available' : 'gap'}`,
+        `Artifact package export action: ${artifactBrowserActions.includes('document-export-artifact-package') ? 'available' : 'gap'}`,
         `Knowledge promotion action: ${artifactBrowserActions.includes('artifact-promote-knowledge') ? 'available' : 'gap'}`,
         `Document attachment action: ${artifactBrowserActions.includes('artifact-attach-document') ? 'available' : 'gap'}`,
         `Document insertion action: ${artifactBrowserActions.includes('artifact-insert-document') ? 'available' : 'gap'}`,
@@ -578,7 +551,7 @@ function buildLanes(context: CommandContext): readonly DocumentOpsLane[] {
       next: modelCompareReady
         ? 'Use cross-session synthesis and reviewer handoff ZIP archives around saved comparison, judgment, export, route-update, route-decision receipt, and source-artifact reuse artifacts.'
         : 'Implement a blind compare runner with selectable candidate models, identical prompt/context, rubric capture, delayed reveal, export, and route update handoff.',
-      userRoute: 'Agent Workspace -> Documents & Compare -> Run blind compare',
+      userRoute: 'Agent Workspace -> Documents & Files -> Run blind compare',
       modelRoute: modelCompareReady ? 'agent_model_compare' : 'models action:"status"',
       signals: [
         `Current model ${snapshot.provider} / ${snapshot.modelDisplayName}`,

package/src/tools/agent-harness-execution-history.ts

@@ -78,20 +78,8 @@ function isRecoverableRoute(record: AgentExecutionRecord): boolean {
   return tool.includes('write') || tool.includes('edit') || tool.includes('patch') || tool.includes('delete');
 }
 
-function toolInspectorAvailable(context: CommandContext): boolean {
-  return Boolean(context.workspace.panelManager?.getRegisteredTypes().some((panel) => panel.id === 'tools'));
-}
-
 function supervisionRoutes(context: CommandContext, record: AgentExecutionRecord): readonly Record<string, unknown>[] {
   const routes: Record<string, unknown>[] = [];
-  if (toolInspectorAvailable(context)) {
-    routes.push({
-      id: 'tool-inspector',
-      label: 'Tool Call Inspector',
-      modelRoute: 'workspace action:"open_panel" panelId:"tools"',
-      requiresConfirmation: true,
-    });
-  }
   if (record.routeKind === 'shell') {
     if (typeof context.openProcessModal === 'function') {
       routes.push({
@@ -397,7 +385,7 @@ function describeRecord(
       modelAccess: {
         inspectHistory: 'execution action:"history"',
         inspectRecord: routeForRecord(record),
-        toolInspector: 'workspace action:"open_panel" panelId:"tools" confirm:true explicitUserRequest:"..."',
+        toolInspector: 'execution action:"status"',
         fileRecovery: 'execution action:"recovery"',
       },
     } : {}),