@pellux/goodvibes-agent 1.4.4 → 1.5.1

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,

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

@@ -11,9 +11,6 @@ type AgentWorkspaceToolAction =
   | 'surfaces'
   | 'surface'
   | 'open'
-  | 'panels'
-  | 'panel'
-  | 'open_panel'
   | 'shortcuts'
   | 'keybindings'
   | 'keybinding'
@@ -33,7 +30,6 @@ interface AgentWorkspaceToolArgs {
   readonly actionId?: unknown;
   readonly workspaceActionId?: unknown;
   readonly surfaceId?: unknown;
-  readonly panelId?: unknown;
   readonly command?: unknown;
   readonly commandName?: unknown;
   readonly args?: unknown;
@@ -47,7 +43,6 @@ interface AgentWorkspaceToolArgs {
   readonly combos?: unknown;
   readonly key?: unknown;
   readonly value?: unknown;
-  readonly pane?: unknown;
   readonly includeParameters?: unknown;
   readonly limit?: unknown;
   readonly confirm?: unknown;
@@ -79,9 +74,6 @@ function normalizeWorkspaceAction(value: unknown): AgentWorkspaceToolAction | nu
   if (action === 'surfaces' || action === 'ui_surfaces' || action === 'screens' || action === 'views') return 'surfaces';
   if (action === 'surface' || action === 'ui_surface' || action === 'screen' || action === 'view') return 'surface';
   if (action === 'open' || action === 'navigate' || action === 'open_surface' || action === 'open_ui_surface') return 'open';
-  if (action === 'panels' || action === 'panes') return 'panels';
-  if (action === 'panel' || action === 'pane') return 'panel';
-  if (action === 'open_panel' || action === 'open_pane') return 'open_panel';
   if (action === 'shortcuts' || action === 'shortcut_help' || action === 'help') return 'shortcuts';
   if (action === 'keybindings' || action === 'bindings' || action === 'keys') return 'keybindings';
   if (action === 'keybinding' || action === 'binding' || action === 'key') return 'keybinding';
@@ -100,7 +92,6 @@ function readAction(args: AgentWorkspaceToolArgs): AgentWorkspaceToolAction {
   const explicit = normalizeWorkspaceAction(args.action) ?? normalizeWorkspaceAction(args.mode);
   if (explicit) return explicit;
   if (readString(args.surfaceId)) return 'surface';
-  if (readString(args.panelId)) return 'panel';
   if (readString(args.actionId) || readString(args.workspaceActionId)) return 'action';
   if (readString(args.command) || readString(args.commandName)) return 'command';
   if (readString(args.key)) return 'keybinding';
@@ -120,10 +111,6 @@ function surfaceLookup(args: AgentWorkspaceToolArgs): string {
   return readString(args.surfaceId) || readString(args.id);
 }
 
-function panelLookup(args: AgentWorkspaceToolArgs): string {
-  return readString(args.panelId) || readString(args.id);
-}
-
 function confirmedArgs(args: AgentWorkspaceToolArgs): Record<string, unknown> {
   return compactArgs({
     confirm: args.confirm,
@@ -173,19 +160,6 @@ function surfaceArgs(mode: 'ui_surface' | 'open_ui_surface', args: AgentWorkspac
   });
 }
 
-function panelArgs(mode: 'panel' | 'open_panel', args: AgentWorkspaceToolArgs): Record<string, unknown> {
-  const panelId = panelLookup(args);
-  return compactArgs({
-    mode,
-    panelId,
-    target: panelId ? undefined : args.target,
-    query: panelId ? undefined : args.query,
-    pane: args.pane,
-    includeParameters: mode === 'panel' ? args.includeParameters : undefined,
-    ...(mode === 'open_panel' ? confirmedArgs(args) : {}),
-  });
-}
-
 function keybindingArgs(mode: 'keybinding' | 'run_keybinding' | 'set_keybinding' | 'reset_keybinding', args: AgentWorkspaceToolArgs): Record<string, unknown> {
   const actionId = lookupId(args);
   return compactArgs({
@@ -232,15 +206,14 @@ export function createAgentWorkspaceTool(deps: AgentWorkspaceToolDeps): Tool {
         properties: {
           action: {
             type: 'string',
-            enum: ['status', 'actions', 'action', 'run', 'surfaces', 'surface', 'open', 'panels', 'panel', 'open_panel', 'shortcuts', 'keybindings', 'keybinding', 'run_keybinding', 'set_keybinding', 'reset_keybinding', 'commands', 'command', 'run_command', 'cli_commands', 'cli_command'],
+            enum: ['status', 'actions', 'action', 'run', 'surfaces', 'surface', 'open', 'shortcuts', 'keybindings', 'keybinding', 'run_keybinding', 'set_keybinding', 'reset_keybinding', 'commands', 'command', 'run_command', 'cli_commands', 'cli_command'],
             description: 'Inspect catalogs, open UI, or run approved workspace actions.',
           },
           mode: { type: 'string', description: 'Alias for action.' },
-          id: { type: 'string', description: 'Generic action, surface, panel, or keybinding id.' },
+          id: { type: 'string', description: 'Generic action, surface, or keybinding id.' },
           actionId: { type: 'string', description: 'Workspace action or keybinding action id.' },
           workspaceActionId: { type: 'string', description: 'Workspace action id alias.' },
           surfaceId: { type: 'string', description: 'UI surface id.' },
-          panelId: { type: 'string', description: 'Panel id.' },
           command: { type: 'string', description: 'Slash or CLI command string.' },
           commandName: { type: 'string', description: 'Slash or CLI command name.' },
           args: { type: 'array', items: { type: 'string' }, description: 'Command arguments when commandName is used.' },
@@ -254,7 +227,6 @@ export function createAgentWorkspaceTool(deps: AgentWorkspaceToolDeps): Tool {
           combos: { type: 'array', items: { type: 'object' }, description: 'Multiple keybinding combos.' },
           key: { type: 'string', description: 'Keybinding lookup key.' },
           value: { anyOf: [{ type: 'string' }, { type: 'number' }, { type: 'boolean' }], description: 'Keybinding value such as Ctrl+G.' },
-          pane: { type: 'string', enum: ['top', 'bottom'], description: 'Preferred pane for panel open.' },
           includeParameters: { type: 'boolean', description: 'Include detailed schemas or route metadata.' },
           limit: { type: 'number', description: 'Maximum rows returned for catalog actions.' },
           confirm: { type: 'boolean', description: 'Required true for UI opens and side-effecting actions.' },
@@ -276,9 +248,6 @@ export function createAgentWorkspaceTool(deps: AgentWorkspaceToolDeps): Tool {
       if (action === 'surfaces') return harnessTool.execute(compactArgs({ mode: 'ui_surfaces', target: args.target, query: args.query, limit: args.limit, includeParameters: args.includeParameters }));
       if (action === 'surface') return harnessTool.execute(surfaceArgs('ui_surface', args));
       if (action === 'open') return harnessTool.execute(surfaceArgs('open_ui_surface', args));
-      if (action === 'panels') return harnessTool.execute(compactArgs({ mode: 'panels', target: args.target, query: args.query, limit: args.limit, includeParameters: args.includeParameters }));
-      if (action === 'panel') return harnessTool.execute(panelArgs('panel', args));
-      if (action === 'open_panel') return harnessTool.execute(panelArgs('open_panel', args));
       if (action === 'shortcuts') return harnessTool.execute(compactArgs({ mode: 'shortcuts', target: args.target, query: args.query, limit: args.limit, includeParameters: args.includeParameters }));
       if (action === 'keybindings') return harnessTool.execute(compactArgs({ mode: 'keybindings', target: args.target, query: args.query, limit: args.limit, includeParameters: args.includeParameters }));
       if (action === 'keybinding') return harnessTool.execute(keybindingArgs('keybinding', args));

package/src/utils/terminal-width.ts

@@ -139,14 +139,20 @@ export function wrapText(text: string, width: number): string[] {
         if (currentLine) lines.push(currentLine);
         let remaining = word;
         while (getDisplayWidth(remaining) > width) {
+          const codePoints = [...remaining];
           let splitIdx = 0;
           let currentWidth = 0;
-          for (let i = 0; i < remaining.length; i++) {
-            const charWidth = getDisplayWidth(remaining[i]!);
+          for (const cp of codePoints) {
+            const charWidth = getDisplayWidth(cp);
             if (currentWidth + charWidth > width) break;
             currentWidth += charWidth;
-            splitIdx = i + 1;
+            splitIdx += cp.length;
           }
+          // Safety: guarantee at least one code point of progress even if a
+          // single grapheme is wider than the available width (e.g. CJK/emoji
+          // at width===1). Without this guard splitIdx stays 0, remaining
+          // never shrinks, and the while-loop hangs.
+          if (splitIdx === 0) splitIdx = codePoints[0]!.length;
           lines.push(remaining.slice(0, splitIdx));
           remaining = remaining.slice(splitIdx);
         }

package/src/version.ts

@@ -6,7 +6,7 @@ import { join } from 'node:path';
 // The prebuild script updates the fallback value before compilation.
 // Uses import.meta.dir (Bun) to locate package.json relative to this file,
 // which is correct regardless of the process working directory.
-let _version = '1.4.4';
+let _version = '1.5.1';
 try {
   const pkg = JSON.parse(readFileSync(join(import.meta.dir, '..', 'package.json'), 'utf-8')) as {
     readonly version?: unknown;

package/src/input/agent-workspace-panel-route.ts

@@ -1,44 +0,0 @@
-import type { AgentWorkspaceCategoryId } from './agent-workspace-types.ts';
-
-const PANEL_CATEGORY_ROUTES = {
-  knowledge: 'knowledge',
-  memory: 'memory',
-  'work-plan': 'work',
-  'project-planning': 'work',
-  plan: 'work',
-  approval: 'work',
-  tasks: 'work',
-  'provider-health': 'setup',
-  automation: 'automation',
-  schedule: 'automation',
-  providers: 'setup',
-  accounts: 'setup',
-  subscription: 'setup',
-  cost: 'setup',
-  tokens: 'setup',
-  security: 'tools',
-  policy: 'tools',
-  tools: 'tools',
-  'qr-code': 'channels',
-  sessions: 'conversation',
-  context: 'conversation',
-  thinking: 'conversation',
-  'system-messages': 'conversation',
-  docs: 'home',
-  'panel-list': 'home',
-} as const satisfies Readonly<Record<string, AgentWorkspaceCategoryId>>;
-
-type AgentWorkspacePanelRouteId = keyof typeof PANEL_CATEGORY_ROUTES;
-
-function hasPanelCategoryRoute(panelId: string): panelId is AgentWorkspacePanelRouteId {
-  return Object.prototype.hasOwnProperty.call(PANEL_CATEGORY_ROUTES, panelId);
-}
-
-export function agentWorkspaceCategoryForPanel(panelId: string): AgentWorkspaceCategoryId {
-  return hasPanelCategoryRoute(panelId) ? PANEL_CATEGORY_ROUTES[panelId] : 'home';
-}
-
-export function agentWorkspaceCommandForPanel(panelId: string): string {
-  const category = agentWorkspaceCategoryForPanel(panelId);
-  return category === 'setup' ? '/agent' : `/agent ${category}`;
-}

package/src/input/panel-integration-actions.ts

@@ -1,26 +0,0 @@
-import type { CommandContext } from './command-registry.ts';
-import { logger } from '@pellux/goodvibes-sdk/platform/utils';
-import type { Panel } from '../panels/types.ts';
-import type { PanelManager } from '../panels/panel-manager.ts';
-import { ApprovalPanel } from '../panels/approval-panel.ts';
-
-export function handlePanelIntegrationAction(
-  panelManager: PanelManager,
-  activePanel: Panel | null,
-  key: string,
-  commandContext?: CommandContext,
-): boolean {
-  if (!activePanel) return false;
-
-  if ((key === 'enter' || key === 'return') && activePanel instanceof ApprovalPanel) {
-    const command = activePanel.getSelectedCommand();
-    if (!command || !commandContext?.executeCommand) return false;
-    const parts = command.replace(/^\//, '').split(/\s+/).filter(Boolean);
-    const [name, ...args] = parts;
-    if (!name) return false;
-    void commandContext.executeCommand(name, args).catch((err) => { logger.debug('approval panel command dispatch failed', { err }); });
-    return true;
-  }
-
-  return false;
-}

package/src/panels/approval-panel.ts

@@ -1,149 +0,0 @@
-import type { Line } from '../types/grid.ts';
-import { ScrollableListPanel } from './scrollable-list-panel.ts';
-import {
-  buildGuidanceLine,
-  buildKeyValueLine,
-  buildPanelLine,
-  DEFAULT_PANEL_PALETTE,
-} from './polish.ts';
-import type { PolicyRuntimeState } from '@/runtime/index.ts';
-import { buildPermissionRuleSuggestions } from '@/runtime/index.ts';
-
-const C = {
-  ...DEFAULT_PANEL_PALETTE,
-  headerBg: '#111827',
-} as const;
-
-const APPROVAL_ROWS = [
-  ['shell', 'why prompted: side effects, destructive ops, secret exposure, escalation', 'review via /approval review shell'],
-  ['file', 'why prompted: config mutation, notebook edits, secret-bearing paths', 'review via /approval review file'],
-  ['network', 'why prompted: external hosts, fetch scope, egress policy', 'review via /approval review network'],
-  ['delegate', 'why prompted: explicit TUI build handoff, write scope, result ownership', 'review via /approval review delegate'],
-  ['mcp', 'why prompted: trust escalation, host scope, path scope, coherence mismatch', 'review via /approval review mcp'],
-  ['remote', 'why prompted: build-host trust, remote write scope, artifact requirements', 'review via /approval review remote'],
-  ['hook', 'why prompted: deny/mutate authority, blocking behavior, execution provenance', 'review via /approval review hook'],
-  ['plugin', 'why prompted: install/update lifecycle, provenance, capability grants', 'review via /approval review plugin'],
-] as const;
-
-type ApprovalRow = (typeof APPROVAL_ROWS)[number];
-
-export class ApprovalPanel extends ScrollableListPanel<ApprovalRow> {
-  private readonly policyRuntimeState: Pick<PolicyRuntimeState, 'getSnapshot'>;
-
-  public constructor(policyRuntimeState: Pick<PolicyRuntimeState, 'getSnapshot'>) {
-    super('approval', 'Approval', 'A', 'monitoring');
-    this.showSelectionGutter = true; // I5: non-color selection affordance
-    this.policyRuntimeState = policyRuntimeState;
-  }
-
-  protected override getPalette() { return C; }
-  protected override getEmptyStateMessage() { return ' No approval lanes defined.'; }
-
-  protected getItems(): readonly ApprovalRow[] {
-    return APPROVAL_ROWS;
-  }
-
-  protected renderItem(row: ApprovalRow, index: number, selected: boolean, width: number): Line {
-    const bg = selected ? C.selectBg : undefined;
-    return buildPanelLine(width, [
-      ['  ', C.label],
-      [row[0].padEnd(10), C.info, bg],
-      [row[1].slice(0, Math.max(0, width - 18)), C.value, bg],
-    ]);
-  }
-
-  public handleInput(key: string): boolean {
-    if (key === 'home') {
-      this.selectedIndex = 0;
-      this.markDirty();
-      return true;
-    }
-    if (key === 'end') {
-      this.selectedIndex = APPROVAL_ROWS.length - 1;
-      this.markDirty();
-      return true;
-    }
-    if (key === 'enter' || key === 'return') {
-      return true;
-    }
-    return super.handleInput(key);
-  }
-
-  public getSelectedCommand(): string | null {
-    const selected = APPROVAL_ROWS[this.selectedIndex] ?? null;
-    return selected ? selected[2].replace('review via ', '').trim() : null;
-  }
-
-  public render(width: number, height: number): Line[] {
-    this.clampSelection();
-    const policySnapshot = this.policyRuntimeState.getSnapshot();
-    const approvalCount = policySnapshot.recentPermissionAudit.filter((e) => e.approved === true).length;
-    const denialCount = policySnapshot.recentPermissionAudit.filter((e) => e.approved === false).length;
-    const pendingCount = policySnapshot.recentPermissionAudit.filter((e) => e.approved === undefined).length;
-
-    const selected = APPROVAL_ROWS[this.selectedIndex] ?? null;
-    const detailLines: Line[] = [];
-    if (selected) {
-      detailLines.push(buildPanelLine(width, [['  Selected Lane', C.label]]));
-      detailLines.push(buildKeyValueLine(width, [
-        { label: 'lane', value: selected[0], valueColor: C.info },
-        { label: 'next review', value: selected[2], valueColor: C.dim },
-      ], C));
-      detailLines.push(buildPanelLine(width, [[` ${selected[1]}`, C.value]]));
-      detailLines.push(buildGuidanceLine(width, selected[2].replace('review via ', ''), `open the ${selected[0]} review path`, C));
-    }
-
-    const recentAuditLines: Line[] = [];
-    for (const entry of policySnapshot.recentPermissionAudit.slice(0, 5)) {
-      const decision = entry.approved === undefined ? 'pending' : entry.approved ? 'approved' : 'denied';
-      const decisionColor = entry.approved === undefined ? C.info : entry.approved ? C.good : C.bad;
-      recentAuditLines.push(buildPanelLine(width, [
-        [`  ${decision.padEnd(8)}`, decisionColor],
-        [`${entry.tool}`.padEnd(14), C.label],
-        [entry.summary.slice(0, Math.max(0, width - 28)), C.value],
-      ]));
-      if (entry.reasons[0]) {
-        recentAuditLines.push(buildPanelLine(width, [[`    ${entry.reasons[0]}`, C.dim]]));
-      }
-    }
-    if (recentAuditLines.length === 0) {
-      recentAuditLines.push(buildPanelLine(width, [[`  No recent approval pressure. Live requests and decisions will appear here.`, C.dim]]));
-    }
-
-    const ruleSuggestionLines: Line[] = [];
-    for (const suggestion of buildPermissionRuleSuggestions(policySnapshot.recentPermissionAudit).slice(0, 3)) {
-      ruleSuggestionLines.push(buildPanelLine(width, [[`  ${suggestion.summary}`, C.info]]));
-      ruleSuggestionLines.push(buildGuidanceLine(width, suggestion.command, suggestion.reason, C));
-    }
-    if (ruleSuggestionLines.length === 0) {
-      ruleSuggestionLines.push(buildPanelLine(width, [[`  No repeated denials currently suggest a durable rule.`, C.dim]]));
-    }
-
-    const headerLines: Line[] = [
-      buildPanelLine(width, [['  Approval posture', C.label]]),
-      buildKeyValueLine(width, [
-        { label: 'why prompted', value: 'risk summary', valueColor: C.value },
-        { label: 'what-if', value: '/security review', valueColor: C.info },
-        { label: 'operator', value: '/approval + /security', valueColor: C.good },
-      ], C),
-      buildPanelLine(width, [
-        ['  \u2713 ', C.good],
-        [`approvals (${approvalCount})  `, C.good],
-        ['\u2715 ', C.bad],
-        [`denials (${denialCount})  `, C.bad],
-        ['\u25cb ', C.info],
-        [`pending (${pendingCount})`, C.info],
-      ]),
-      buildGuidanceLine(width, '/approval review shell', 'inspect the highest-risk approval lane and refine scoped review posture', C),
-      ...detailLines,
-      ...recentAuditLines,
-      ...ruleSuggestionLines,
-    ];
-
-    return this.renderList(width, height, {
-      title: 'Approval Control Room',
-      header: headerLines,
-      footer: [buildPanelLine(width, [[`  Up/Down move  Home/End jump  selected lane opens the next command path`, C.dim]])],
-    });
-  }
-}