@pellux/goodvibes-agent 1.4.4 → 1.5.0

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

@@ -204,10 +204,6 @@ function describeCandidate(route: ExecutionRoute, context: CommandContext, toolR
   };
 }
 
-function toolInspectorAvailable(context: CommandContext): boolean {
-  return Boolean(context.workspace.panelManager?.getRegisteredTypes().some((panel) => panel.id === 'tools'));
-}
-
 function executionSupervisionRoutes(context: CommandContext, route: ExecutionRoute): readonly ExecutionSupervisionRoute[] {
   const routes: ExecutionSupervisionRoute[] = [];
   if (route.id === 'local-shell-command') {
@@ -228,15 +224,6 @@ function executionSupervisionRoutes(context: CommandContext, route: ExecutionRou
       },
     );
   }
-  if (route.id === 'local-shell-command' || route.id === 'local-edit-write') {
-    routes.push({
-      id: 'tool-inspector',
-      label: 'Tool Call Inspector',
-      available: toolInspectorAvailable(context),
-      modelRoute: 'workspace action:"open_panel" panelId:"tools"',
-      requiresConfirmation: true,
-    });
-  }
   return routes;
 }
 
@@ -244,11 +231,9 @@ function executionSupervisionSummary(context: CommandContext): Record<string, un
   return {
     processMonitorAvailable: typeof context.openProcessModal === 'function',
     liveTailAvailable: typeof context.openLiveTail === 'function',
-    toolInspectorAvailable: toolInspectorAvailable(context),
     routes: [
       'workspace action:"open" surfaceId:"process-monitor"',
       'workspace action:"open" surfaceId:"live-tail"',
-      'workspace action:"open_panel" panelId:"tools"',
     ],
   };
 }

package/src/tools/agent-harness-keybinding-metadata.ts

@@ -310,30 +310,37 @@ function keybindingOperationRoute(action: KeyAction): KeybindingOperationRoute {
         confirmation: 'agent_harness mode:"run_keybinding" requires confirm:true and explicitUserRequest.',
         note: 'Runs the available clear-screen route.',
       };
-    case 'panel-picker':
+    case 'sidebar-toggle':
+      return {
+        supported: false,
+        effect: 'visible-ui-navigation',
+        confirmation: 'Direct user interaction only.',
+        note: 'Shows or hides the activity sidebar. Layout preferences stay under user control.',
+      };
+    case 'workspace-picker':
       return {
         supported: true,
         effect: 'visible-ui-navigation',
         preferredMode: 'run_keybinding',
         surfaceId: 'agent-workspace',
         confirmation: 'agent_harness mode:"run_keybinding" requires confirm:true and explicitUserRequest.',
-        note: 'Opens the same Agent workspace home route as the panel-picker shortcut.',
+        note: 'Opens the Agent workspace home.',
       };
-    case 'panel-close':
+    case 'workspace-close':
       return {
         supported: true,
         effect: 'visible-ui-navigation',
         preferredMode: 'run_keybinding',
         confirmation: 'agent_harness mode:"run_keybinding" requires confirm:true and explicitUserRequest.',
-        note: 'Dismisses Agent workspace first. If it is not active, closes the active panel and focuses the prompt when focus is available.',
+        note: 'Dismisses the Agent workspace.',
       };
-    case 'panel-close-all':
+    case 'workspace-close-all':
       return {
         supported: true,
         effect: 'visible-ui-navigation',
         preferredMode: 'run_keybinding',
         confirmation: 'agent_harness mode:"run_keybinding" requires confirm:true and explicitUserRequest.',
-        note: 'Dismisses Agent workspace first. If it is not active, closes open panels and focuses the prompt when focus is available.',
+        note: 'Dismisses the Agent workspace (alias chord: Ctrl+Shift+X has the same effect as Ctrl+X).',
       };
     case 'history-search':
       return {
@@ -373,8 +380,8 @@ function keybindingOperationRoute(action: KeyAction): KeybindingOperationRoute {
         confirmation: 'agent_harness mode:"run_keybinding" requires confirm:true and explicitUserRequest.',
         note: 'Opens the visible nearest-block actions surface. The exact block action remains an interactive visible-shell selection because it depends on cursor/scroll position.',
       };
-    case 'panel-tab-next':
-    case 'panel-tab-prev':
+    case 'workspace-tab-next':
+    case 'workspace-tab-prev':
       return {
         supported: false,
         effect: 'visible-ui-navigation',
@@ -558,17 +565,17 @@ export function runHarnessKeybinding(context: CommandContext, args: HarnessKeybi
       if (!context.clearScreen) return runUnavailable(resolved.action, route, 'Clear-screen route is unavailable.');
       context.clearScreen();
       return { status: 'executed', action: resolved.action, effect: 'screen-clear', keybinding: descriptor };
-    case 'panel-picker':
-      if (context.openPanelPicker) {
-        context.openPanelPicker();
-        return { status: 'executed', action: resolved.action, effect: 'agent-workspace-opened', route: 'openPanelPicker', keybinding: descriptor };
+    case 'workspace-picker':
+      if (context.openWorkspacePicker) {
+        context.openWorkspacePicker();
+        return { status: 'executed', action: resolved.action, effect: 'agent-workspace-opened', route: 'openWorkspacePicker', keybinding: descriptor };
       }
       if (context.openAgentWorkspace) {
         context.openAgentWorkspace('home');
         return { status: 'executed', action: resolved.action, effect: 'agent-workspace-opened', route: 'openAgentWorkspace', categoryId: 'home', keybinding: descriptor };
       }
-      return runUnavailable(resolved.action, route, 'No panel picker or Agent workspace route is available.');
-    case 'panel-close': {
+      return runUnavailable(resolved.action, route, 'No workspace picker or Agent workspace route is available.');
+    case 'workspace-close': {
       const dismissedAgentWorkspace = context.dismissAgentWorkspace?.() ?? false;
       if (dismissedAgentWorkspace) {
         return {
@@ -579,20 +586,17 @@ export function runHarnessKeybinding(context: CommandContext, args: HarnessKeybi
           keybinding: descriptor,
         };
       }
-      const active = context.workspace.panelManager?.getActivePanel() ?? null;
-      if (active) context.workspace.panelManager?.close(active.id);
-      if (!active && !context.focusPrompt) return runUnavailable(resolved.action, route, 'No active Agent workspace, active panel, or prompt focus route is available.');
-      if (context.focusPrompt) context.focusPrompt();
+      if (!context.focusPrompt) return runUnavailable(resolved.action, route, 'No active Agent workspace or prompt focus route is available.');
+      context.focusPrompt();
       context.renderRequest();
       return {
         status: 'executed',
         action: resolved.action,
-        effect: active ? 'active-panel-closed' : 'prompt-focused',
-        ...(active ? { panelId: active.id } : {}),
+        effect: 'prompt-focused',
         keybinding: descriptor,
       };
     }
-    case 'panel-close-all': {
+    case 'workspace-close-all': {
       const dismissedAgentWorkspace = context.dismissAgentWorkspace?.() ?? false;
       if (dismissedAgentWorkspace) {
         return {
@@ -603,18 +607,13 @@ export function runHarnessKeybinding(context: CommandContext, args: HarnessKeybi
           keybinding: descriptor,
         };
       }
-      const managerPanel = context.workspace.panelManager;
-      const openPanels = managerPanel?.getAllOpen() ?? [];
-      for (const panel of openPanels) managerPanel?.close(panel.id);
-      managerPanel?.hide();
-      if (openPanels.length === 0 && !context.focusPrompt) return runUnavailable(resolved.action, route, 'No active Agent workspace, open panels, or prompt focus route is available.');
-      if (context.focusPrompt) context.focusPrompt();
+      if (!context.focusPrompt) return runUnavailable(resolved.action, route, 'No active Agent workspace or prompt focus route is available.');
+      context.focusPrompt();
       context.renderRequest();
       return {
         status: 'executed',
         action: resolved.action,
-        effect: 'all-panels-closed',
-        closedPanels: openPanels.map((panel) => panel.id),
+        effect: 'prompt-focused',
         keybinding: descriptor,
       };
     }

package/src/tools/agent-harness-local-model-cookbook.ts

@@ -1,4 +1,5 @@
 import type { CommandContext } from '../input/command-registry.ts';
+import { estimateModelBytes, fitAssessment, fitVerdictLabel, readHardwareProfileSync, REPRESENTATIVE_7B_PARAMS } from '../core/hardware-profile.ts';
 import { previewHarnessText } from './agent-harness-text.ts';
 import { localModelDetection, localModelServerHealthMap } from './agent-harness-local-model-endpoints.ts';
 import { localHardwareProfile, localRecipeReadinessScore, localRecipeStackId, scoreLocalModelRecipe } from './agent-harness-model-readiness.ts';
@@ -83,6 +84,21 @@ export function localModelSetupPlan(
   };
 }
 
+/**
+ * Representative Q4 parameter count for a recipe's default/starter model.
+ * Used to estimate memory footprint for hardware fit annotation.
+ * Returns null for stacks where size is too variable to estimate safely.
+ */
+function recipeRepresentativeParamCount(recipe: LocalModelRecipe): number | null {
+  // Uses the shared REPRESENTATIVE_7B_PARAMS constant so the renderer and cookbook
+  // always reference the same basis.
+  // The generic openai-compatible stack is too varied to estimate.
+  if (recipe.id === 'ollama' || recipe.id === 'llama-cpp' || recipe.id === 'vllm') {
+    return REPRESENTATIVE_7B_PARAMS;
+  }
+  return null;
+}
+
 export function localModelRecipes(): readonly LocalModelRecipe[] {
   return [
     {
@@ -175,7 +191,14 @@ export function describeLocalModelRecipe(
     hardware: previewHarnessText(recipe.hardware, includeParameters ? 180 : 96),
     hardwareMatched: fit.reasons.slice(0, includeParameters ? 6 : 3),
     detected,
-      modelRoute: 'models action:"status" or agent_harness mode:"open_ui_surface"',
+    modelRoute: 'models action:"status" or agent_harness mode:"open_ui_surface"',
+    ...(() => {
+      const paramCount = recipeRepresentativeParamCount(recipe);
+      if (paramCount === null) return {};
+      const sizeBytes = estimateModelBytes(paramCount);
+      const label = fitVerdictLabel(fitAssessment(sizeBytes, readHardwareProfileSync()));
+      return label ? { hardwareFit: label } : {};
+    })(),
     ...(includeParameters ? {
       setup: recipe.setup,
       modelExamples: recipe.modelExamples,

package/src/tools/agent-harness-metadata.ts

@@ -204,6 +204,22 @@ export function describeCommandPolicy(commandName: string): CommandExecutionPoli
       boundary: 'Connected schedules require an explicit user request and do not create hidden Agent jobs or local schedulers.',
     };
   }
+  if (root === 'email') {
+    return {
+      effect: 'external-network',
+      confirmation,
+      preferredModelTool: `${agentHarnessModes('run_command')} (use /email set to configure, /email config to view settings)`,
+      boundary: 'Email IMAP reads are read-only (EXAMINE); sends require explicit --yes confirmation and route only to the configured account. Use /email set email.<key> <value> to configure email settings; use /email config to view current settings. The generic settings action cannot set email.* keys — /email set is the only supported path.',
+    };
+  }
+  if (root === 'calendar' || root === 'cal') {
+    return {
+      effect: 'local-state',
+      confirmation,
+      preferredModelTool: agentHarnessModes('run_command'),
+      boundary: 'Calendar commands manage local ICS events only; import/export and mutations require explicit --yes confirmation. Calendar configuration is set via /calendar subcommands, not the generic settings action.',
+    };
+  }
   if (root === 'channels' || root === 'channel' || root === 'notify') {
     return {
       effect: 'external-network',

package/src/tools/agent-harness-mode-catalog.ts

@@ -1,4 +1,5 @@
 import { AGENT_HARNESS_MODES } from './agent-harness-tool-schema.ts';
+import { readLimit } from './agent-harness-tool-utils.ts';
 
 export type AgentHarnessMode = typeof AGENT_HARNESS_MODES[number];
 
@@ -35,9 +36,6 @@ export const HARNESS_MODE_DESCRIPTORS: readonly HarnessModeDescriptor[] = [
   { id: 'route_decision', kind: 'inspect', family: 'start', summary: 'Choose the best visible Agent route for a plain user task.', next: 'Prefer route action:"plan" for first-class access.', keywords: ['where should this go', 'which tool', 'route user task', 'one assistant', 'user-first route'], parameters: ['query', 'target', 'limit', 'includeParameters'] },
   { id: 'cli_commands', kind: 'discover', family: 'cli', summary: 'List top-level package CLI mirrors for discovery only.', next: 'Prefer workspace action:"cli_commands|cli_command".', parameters: ['query', 'limit', 'includeParameters'] },
   { id: 'cli_command', kind: 'inspect', family: 'cli', summary: 'Inspect one CLI command parser result, policy, aliases, and model route.', parameters: ['cliCommand', 'command', 'commandName', 'target', 'query'] },
-  { id: 'panels', kind: 'discover', family: 'ui', summary: 'List built-in panel catalog state and workspace routes.', next: 'Prefer workspace action:"panels|panel|open_panel".', parameters: ['query', 'category', 'limit', 'includeParameters'] },
-  { id: 'panel', kind: 'inspect', family: 'ui', summary: 'Inspect one built-in panel, open state, workspace route, and policy.', parameters: ['panelId', 'target', 'query'] },
-  { id: 'open_panel', kind: 'effect', family: 'ui', summary: 'Route the visible shell to one built-in panel.', requiresConfirmation: true, parameters: ['panelId', 'target', 'query', 'pane', 'confirm', 'explicitUserRequest'] },
   { id: 'ui_surfaces', kind: 'discover', family: 'ui', summary: 'List modal, picker, and visible UI surfaces the model can request.', next: 'Prefer workspace action:"surfaces|surface|open".', keywords: ['web dashboard', 'pwa', 'browser cockpit', 'connected browser', 'mobile web'], parameters: ['query', 'limit', 'includeParameters'] },
   { id: 'ui_surface', kind: 'inspect', family: 'ui', summary: 'Inspect one visible modal or picker surface and route contract.', keywords: ['web dashboard', 'pwa', 'browser cockpit', 'connected browser', 'mobile web'], parameters: ['surfaceId', 'target', 'query'] },
   { id: 'open_ui_surface', kind: 'effect', family: 'ui', summary: 'Open one visible modal, picker, or operator workspace route.', requiresConfirmation: true, keywords: ['web dashboard', 'pwa', 'browser cockpit', 'connected browser', 'mobile web'], parameters: ['surfaceId', 'target', 'query', 'confirm', 'explicitUserRequest'] },
@@ -147,18 +145,13 @@ export const HARNESS_MODE_DESCRIPTORS: readonly HarnessModeDescriptor[] = [
   { id: 'connected_host_capability', kind: 'inspect', family: 'connected-host', summary: 'Inspect one connected-host capability and blocked surfaces.', next: 'Prefer host action:"capability".', parameters: ['capabilityId', 'target', 'query'] },
   { id: 'daemon', kind: 'alias', family: 'connected-host', summary: 'GoodVibes daemon -> connected_host; mutations use confirmed methods.', next: 'Prefer host action:"capabilities".', aliases: ['connected_host'], parameters: ['includeParameters'] },
   { id: 'daemon_status', kind: 'alias', family: 'connected-host', summary: 'GoodVibes daemon status -> connected_host_status.', next: 'Prefer host action:"status".', aliases: ['connected_host_status'], parameters: ['includeParameters'] },
+  { id: 'propose_skill_drafts', kind: 'effect', family: 'personal-ops', summary: 'Draft up to 3 skills from recent learning candidates for review.', requiresConfirmation: true, keywords: ['skill draft', 'propose skill', 'auto-propose', 'draft skills', 'learning draft'], parameters: ['confirm', 'explicitUserRequest'] },
 ] as const;
 
 function readString(value: unknown): string {
   return typeof value === 'string' ? value.trim() : '';
 }
 
-function readLimit(value: unknown, fallback: number): number {
-  const parsed = typeof value === 'string' && value.trim() ? Number(value) : value;
-  if (typeof parsed !== 'number' || !Number.isFinite(parsed)) return fallback;
-  return Math.max(1, Math.min(500, Math.trunc(parsed)));
-}
-
 function describeHarnessModeDescriptor(
   descriptor: HarnessModeDescriptor,
   options: { readonly includeParameters?: boolean; readonly lookup?: Record<string, unknown> } = {},

package/src/tools/agent-harness-provider-account-metadata.ts

@@ -1,6 +1,6 @@
 import type { CommandContext } from '../input/command-registry.ts';
-import type { ProviderAccountRecord, ProviderAccountSnapshot } from '../panels/provider-account-snapshot.ts';
-import { buildProviderAccountSnapshot } from '../panels/provider-account-snapshot.ts';
+import type { ProviderAccountRecord, ProviderAccountSnapshot } from '../runtime/provider-account-snapshot.ts';
+import { buildProviderAccountSnapshot } from '../runtime/provider-account-snapshot.ts';
 import { requireProvider, requireServiceRegistry, requireSubscriptionManager } from '../input/commands/runtime-services.ts';
 import { previewHarnessText } from './agent-harness-text.ts';
 

package/src/tools/agent-harness-setup-posture-utils.ts

@@ -1,5 +1,5 @@
 import { statSync } from 'node:fs';
-import { buildProviderAccountSnapshot } from '../panels/provider-account-snapshot.ts';
+import { buildProviderAccountSnapshot } from '../runtime/provider-account-snapshot.ts';
 import { requireLocalUserAuthManager, requirePlatform, requireProvider, requireSecretsManager, requireServiceRegistry, requireShellPaths, requireSubscriptionManager } from '../input/commands/runtime-services.ts';
 import { collectOnboardingSnapshot } from '../runtime/onboarding/index.ts';
 import type { CommandContext } from '../input/command-registry.ts';

package/src/tools/agent-harness-tool-schema.ts

@@ -1,5 +1,5 @@
 export const AGENT_HARNESS_MODES = [
-  'summary', 'modes', 'mode', 'route_decision', 'cli_commands', 'cli_command', 'panels', 'panel', 'open_panel',
+  'summary', 'modes', 'mode', 'route_decision', 'cli_commands', 'cli_command',
   'ui_surfaces', 'ui_surface', 'open_ui_surface',
   'shortcuts', 'keybindings', 'keybinding', 'run_keybinding', 'set_keybinding', 'reset_keybinding',
   'commands', 'command', 'run_command', 'channels', 'channel', 'channel_setup_guide', 'channel_triage', 'channel_deliveries', 'notifications', 'notification_target',
@@ -32,6 +32,12 @@ export const AGENT_HARNESS_MODES = [
   'service_posture', 'service_endpoint',
   'connected_host', 'connected_host_status', 'connected_host_capability',
   'daemon', 'daemon_status',
+  'propose_skill_drafts',
+] as const;
+
+const BACKGROUND_PROCESS_ACTION_VALUES = [
+  'start', 'spawn', 'run', 'stop', 'kill', 'cancel', 'wait', 'list',
+  'status', 'poll', 'log', 'output', 'write', 'capabilities', 'doctor', 'parity',
 ] as const;
 
 const KEY_COMBO_PARAMETER_SCHEMA = {
@@ -42,10 +48,12 @@ const KEY_COMBO_PARAMETER_SCHEMA = {
 } as const;
 
 export const AGENT_HARNESS_PARAMETER_PROPERTIES = {
+  // The full mode catalog is intentionally NOT inlined as an enum:
+  // it would bloat every model prompt. Dispatch still validates against
+  // AGENT_HARNESS_MODES and unknown modes return suggestions.
   mode: {
     type: 'string',
-    enum: AGENT_HARNESS_MODES,
-    description: 'Harness operation. Start with summary or a plural catalog mode.',
+    description: 'Operation. Start with "summary"; mode:"modes" lists the catalog.',
   },
   query: {
     type: 'string',
@@ -131,12 +139,12 @@ export const AGENT_HARNESS_PARAMETER_PROPERTIES = {
   },
   processAction: {
     type: 'string',
-    enum: ['start', 'spawn', 'run', 'stop', 'kill', 'cancel', 'wait', 'list', 'status', 'poll', 'log', 'output', 'write', 'capabilities', 'doctor', 'parity'],
+    enum: BACKGROUND_PROCESS_ACTION_VALUES,
     description: 'Background process lifecycle action or process-style alias.',
   },
   action: {
     type: 'string',
-    enum: ['start', 'spawn', 'run', 'stop', 'kill', 'cancel', 'wait', 'list', 'status', 'poll', 'log', 'output', 'write', 'capabilities', 'doctor', 'parity'],
+    enum: BACKGROUND_PROCESS_ACTION_VALUES,
     description: 'Process-style action alias for run_background_process.',
   },
   cwd: {
@@ -216,10 +224,6 @@ export const AGENT_HARNESS_PARAMETER_PROPERTIES = {
     type: 'string',
     description: 'UI surface id for ui_surface or open_ui_surface modes.',
   },
-  panelId: {
-    type: 'string',
-    description: 'Built-in panel id for panel or open_panel modes.',
-  },
   actionId: {
     type: 'string',
     description: 'Workspace action id or keybinding action id.',
@@ -332,11 +336,6 @@ export const AGENT_HARNESS_PARAMETER_PROPERTIES = {
     type: 'number',
     description: 'Maximum catalog entries to return.',
   },
-  pane: {
-    type: 'string',
-    enum: ['top', 'bottom'],
-    description: 'Preferred pane for open_panel.',
-  },
   confirm: {
     type: 'boolean',
     description: 'Required true for confirmed harness effects.',

package/src/tools/agent-harness-tool.ts

@@ -8,11 +8,12 @@ import { channelReadinessCatalogStatus, describeHarnessChannel, describeHarnessC
 import { blockedHarnessCliCommandTokens, describeHarnessCliCommand, listHarnessCliCommands, totalHarnessCliCommands } from './agent-harness-cli-metadata.ts';
 import { describeHarnessCommand, listHarnessCommands } from './agent-harness-command-catalog.ts';
 import { describeLearningCandidate, learningCuratorCatalogStatus, learningCuratorSummary } from './agent-harness-learning-curator.ts';
+import { runSkillDraftProposer } from '../agent/skill-draft-runner.ts';
+import { AgentSkillRegistry } from '../agent/skill-registry.ts';
 import { delegationPostureCatalogStatus, delegationPostureSummary, describeHarnessDelegationRoute } from './agent-harness-delegation-posture.ts';
 import { describeHarnessKeybinding, listHarnessKeybindings, listHarnessShortcuts, resetHarnessKeybinding, runHarnessKeybinding, setHarnessKeybinding, totalHarnessKeybindings, totalHarnessShortcuts } from './agent-harness-keybinding-metadata.ts';
 import { describeHarnessMediaProvider, mediaPostureCatalogStatus, mediaPostureSummary } from './agent-harness-media-posture.ts';
 import { describeHarnessNotificationTarget, listHarnessNotificationTargets, notificationTargetCatalogStatus } from './agent-harness-notification-metadata.ts';
-import { describeHarnessPanel, listHarnessPanels, openHarnessPanel, totalHarnessPanels } from './agent-harness-panel-metadata.ts';
 import { connectedHostStatusSummary } from './agent-harness-connected-host-status.ts';
 import { backgroundProcessCatalogStatus, backgroundProcessSummary, describeBackgroundProcess, runBackgroundProcessAction } from './agent-harness-background-processes.ts';
 import { describeDocumentOpsLane, documentOpsCatalogStatus, documentOpsSummary } from './agent-harness-document-ops.ts';
@@ -81,7 +82,6 @@ function compactHarnessModeGuide(): Record<string, unknown> {
 function detailedHarnessModelAccessGuide(): Record<string, string> {
   return {
     cliCommands: 'Prefer workspace action:"cli_commands|cli_command" for CLI discovery. Lower-level cli command modes remain available.',
-    panels: 'Prefer workspace action:"panels|panel|open_panel"; visible navigation needs confirm:true and explicitUserRequest.',
     uiSurfaces: 'Prefer workspace action:"surfaces|surface|open" for visible UI and computer action:"browser|open_browser" for browser/PWA. Lower-level UI modes remain available.',
     shortcuts: 'Prefer workspace action:"shortcuts|keybindings|keybinding|run_keybinding|set_keybinding|reset_keybinding"; effects need confirmation.',
     slashCommands: 'Prefer workspace action:"commands|command|run_command"; slash-command execution needs confirmation.',
@@ -131,7 +131,7 @@ export function createAgentHarnessTool(deps: AgentHarnessToolDeps): Tool {
   return {
     definition: {
       name: 'agent_harness',
-      description: 'Inspect or operate GoodVibes Agent harness surfaces.',
+      description: 'Inspect or operate Agent harness surfaces; mode:"modes" lists all.',
       parameters: {
         type: 'object',
         properties: AGENT_HARNESS_PARAMETER_PROPERTIES,
@@ -143,7 +143,13 @@ export function createAgentHarnessTool(deps: AgentHarnessToolDeps): Tool {
     },
     execute: async (rawArgs) => {
       const args = rawArgs as AgentHarnessToolArgs;
-      if (!isMode(args.mode)) return error(`Unknown agent_harness mode: ${String(args.mode)}`);
+      if (!isMode(args.mode)) {
+        const requested = String(args.mode).toLowerCase();
+        const suggestions = AGENT_HARNESS_MODES
+          .filter((mode) => mode.includes(requested) || requested.includes(mode))
+          .slice(0, 5);
+        return error(`Unknown agent_harness mode: ${String(args.mode)}. ${suggestions.length > 0 ? `Closest modes: ${suggestions.join(', ')}. ` : ''}Use mode:"modes" to list the full catalog.`);
+      }
       try {
         if (args.mode === 'summary') {
           const channelReadiness = channelReadinessCatalogStatus(deps.commandContext);
@@ -214,7 +220,6 @@ export function createAgentHarnessTool(deps: AgentHarnessToolDeps): Tool {
             harnessModes: HARNESS_MODE_DESCRIPTORS.length,
             cliCommands: totalHarnessCliCommands(),
             blockedCliCommandTokens: blockedHarnessCliCommandTokens(),
-            panels: totalHarnessPanels(deps.commandContext),
             uiSurfaces: totalHarnessUiSurfaces(),
             shortcuts: totalHarnessShortcuts(deps.commandContext),
             keybindings: totalHarnessKeybindings(deps.commandContext),
@@ -282,24 +287,6 @@ export function createAgentHarnessTool(deps: AgentHarnessToolDeps): Tool {
         if (args.mode === 'cli_command') {
           return output(describeHarnessCliCommand(args));
         }
-        if (args.mode === 'panels') {
-          const panels = listHarnessPanels(deps.commandContext, args);
-          return output({
-            panels,
-            returned: panels.length,
-            total: totalHarnessPanels(deps.commandContext),
-            policy: 'Panel modes expose Agent/TUI operator view catalog and open state. open_panel is confirmation-gated and routes through the current Agent operator surface.',
-          });
-        }
-        if (args.mode === 'panel') {
-          const panel = describeHarnessPanel(deps.commandContext, args);
-          return panel ? output(panel) : error(`Unknown panel ${readString(args.panelId || args.target || args.query) || '<missing>'}.`);
-        }
-        if (args.mode === 'open_panel') {
-          const confirmationError = requireConfirmedAction(args, 'Panel routing');
-          if (confirmationError) return error(confirmationError);
-          return output(openHarnessPanel(deps.commandContext, args));
-        }
         if (args.mode === 'ui_surfaces') {
           const surfaces = listHarnessUiSurfaces(deps.commandContext, args);
           return output({ surfaces, returned: surfaces.length, total: totalHarnessUiSurfaces() });
@@ -717,6 +704,23 @@ export function createAgentHarnessTool(deps: AgentHarnessToolDeps): Tool {
             includeParameters: args.includeParameters === true,
           }));
         }
+        if (args.mode === 'propose_skill_drafts') {
+          const confirmationError = requireConfirmedAction(args, 'Skill draft proposal');
+          if (confirmationError) return error(confirmationError);
+          const shellPaths = deps.commandContext.workspace?.shellPaths;
+          if (!shellPaths) return error('Skill draft proposal requires an active workspace.');
+          const registry = AgentSkillRegistry.fromShellPaths(shellPaths);
+          const result = runSkillDraftProposer(deps.commandContext, registry);
+          return output({
+            proposed: result.proposed,
+            skipped: result.skipped,
+            skillIds: result.skillIds,
+            message: result.proposed > 0
+              ? `Drafted ${result.proposed} skill${result.proposed !== 1 ? 's' : ''} for review. Find them under Memory > Skills with enabled:false.`
+              : 'No new skill drafts this pass.',
+            policy: 'Drafted skills are disabled and require review before use. Enable them under Memory > Skills.',
+          });
+        }
         return error(`Unhandled agent_harness mode: ${args.mode}`);
       } catch (err) {
         return error(formatHarnessError(err));

package/src/tools/agent-harness-ui-surface-metadata.ts

@@ -101,16 +101,6 @@ function openPanelWorkspaceSurface(
     options.opener();
     return opened(surface, { categoryId: options.categoryId, panelId: options.panelId, route: 'named-opener' });
   }
-  if (context.showPanel) {
-    const pane = optionalPane(args);
-    context.showPanel(options.panelId, pane);
-    return opened(surface, {
-      categoryId: options.categoryId,
-      panelId: options.panelId,
-      pane: pane ?? 'default',
-      route: 'panel-route',
-    });
-  }
   return routeUnavailable(surface);
 }
 
@@ -183,17 +173,17 @@ const UI_SURFACES: readonly UiSurfaceDefinition[] = [
   },
   {
     id: 'panel-picker',
-    label: 'Panel Picker',
+    label: 'Workspace Picker',
     kind: 'picker',
-    summary: 'Operator panel route into Agent Workspace home.',
+    summary: 'Operator route into the Agent Workspace home.',
     command: 'Ctrl+P',
-    preferredModelRoute: `Use ${agentHarnessModes('panels', 'panel', 'open_panel')} for panel catalog and routing, or mode:"workspace_actions" for concrete model operation.`,
-    available: (context) => typeof context.openPanelPicker === 'function' || typeof context.openAgentWorkspace === 'function',
+    preferredModelRoute: 'Use mode:"workspace_actions" for concrete model operation, or mode:"open_ui_surface" for visible routing.',
+    available: (context) => typeof context.openWorkspacePicker === 'function' || typeof context.openAgentWorkspace === 'function',
     open: (context) => {
       const surface = findSurfaceById('panel-picker')!;
-      if (context.openPanelPicker) {
-        context.openPanelPicker();
-        return opened(surface, { categoryId: 'home', route: 'panel-picker' });
+      if (context.openWorkspacePicker) {
+        context.openWorkspacePicker();
+        return opened(surface, { categoryId: 'home', route: 'workspace-picker' });
       }
       return openAgentWorkspaceCategory(context, surface, 'home');
     },
@@ -206,7 +196,7 @@ const UI_SURFACES: readonly UiSurfaceDefinition[] = [
     command: '/security',
     preferredModelRoute: `Use mode:"workspace_actions" for security review actions or ${agentHarnessModes('run_command')} for confirmed /security review output.`,
     parameters: ['pane'],
-    available: (context) => typeof context.openAgentWorkspace === 'function' || typeof context.openSecurityPanel === 'function' || typeof context.showPanel === 'function',
+    available: (context) => typeof context.openAgentWorkspace === 'function' || typeof context.openSecurityPanel === 'function',
     open: (context, args) => {
       const surface = findSurfaceById('security-panel')!;
       return openPanelWorkspaceSurface(context, args, surface, {
@@ -224,7 +214,7 @@ const UI_SURFACES: readonly UiSurfaceDefinition[] = [
     command: '/knowledge',
     preferredModelRoute: `Use agent_knowledge, agent_knowledge_ingest, mode:"workspace_actions", or ${agentHarnessModes('run_command')} for confirmed /knowledge operation.`,
     parameters: ['pane'],
-    available: (context) => typeof context.openAgentWorkspace === 'function' || typeof context.openKnowledgePanel === 'function' || typeof context.showPanel === 'function',
+    available: (context) => typeof context.openAgentWorkspace === 'function' || typeof context.openKnowledgePanel === 'function',
     open: (context, args) => {
       const surface = findSurfaceById('knowledge-panel')!;
       return openPanelWorkspaceSurface(context, args, surface, {
@@ -242,7 +232,7 @@ const UI_SURFACES: readonly UiSurfaceDefinition[] = [
     command: '/subscription',
     preferredModelRoute: `Use mode:"workspace_actions" or ${agentHarnessModes('run_command')} for confirmed /subscription mirrors.`,
     parameters: ['pane'],
-    available: (context) => typeof context.openAgentWorkspace === 'function' || typeof context.openSubscriptionPanel === 'function' || typeof context.showPanel === 'function',
+    available: (context) => typeof context.openAgentWorkspace === 'function' || typeof context.openSubscriptionPanel === 'function',
     open: (context, args) => {
       const surface = findSurfaceById('subscription-panel')!;
       return openPanelWorkspaceSurface(context, args, surface, {

package/src/tools/agent-harness-workspace-action-runner.ts

@@ -49,7 +49,7 @@ export async function runWorkspaceAction(
   if (action.kind === 'settings-import') {
     const preview = previewAgentWorkspaceTuiSettingsImport(deps.commandContext);
     if (!preview) return error('GoodVibes settings import is unavailable in this runtime.');
-    if (args.confirm !== true) {
+    if (args.confirm !== true && !readString(args.explicitUserRequest)) {
       return output({
         status: 'confirmation_required',
         action: describeWorkspaceAction(category, action, { lookup }),
@@ -57,10 +57,9 @@ export async function runWorkspaceAction(
         next: 'Run with confirm:true and explicitUserRequest after the user asks to import these settings.',
       });
     }
+    const confirmationError = requireConfirmedAction(args, 'GoodVibes settings import');
+    if (confirmationError) return error(confirmationError);
     const explicitUserRequest = readString(args.explicitUserRequest);
-    if (!explicitUserRequest) {
-      return error('GoodVibes settings import requires explicitUserRequest when confirm is true.');
-    }
     const outcome = await importAgentWorkspaceTuiSettings(deps.commandContext);
     return output({
       status: outcome.status,