@pellux/goodvibes-tui 0.28.0 → 0.29.0

package/src/panels/communication-panel.ts

@@ -1,5 +1,6 @@
 import type { Line } from '../types/grid.ts';
 import { ScrollableListPanel } from './scrollable-list-panel.ts';
+import type { PanelIntegrationContext } from './types.ts';
 import type { UiCommunicationSnapshot, UiReadModel } from '../runtime/ui-read-models.ts';
 import { truncateDisplay } from '../utils/terminal-width.ts';
 import {
@@ -18,15 +19,9 @@ import {
 } from './polish.ts';
 import { createEmptyLine } from '../types/grid.ts';
 
-const C = {
-  ...DEFAULT_PANEL_PALETTE,
-  header: '#94a3b8',
-  headerBg: '#1e293b',
-  ok: '#22c55e',
-  warn: '#eab308',
-  error: '#ef4444',
-  selectBg: '#0f172a',
-} as const;
+// Base chrome only — title band, state colors, and text tokens all come
+// straight from DEFAULT_PANEL_PALETTE (WO-002).
+const C = DEFAULT_PANEL_PALETTE;
 
 type CommunicationRecord = UiCommunicationSnapshot['records'][number];
 
@@ -41,18 +36,31 @@ function fmtAgo(ts: number): string {
   return `${Math.floor(sec / 3600)}h`;
 }
 
+// Set by handleInput (enter/o) and consumed on the very next
+// handlePanelIntegrationAction dispatch of that same key — handleInput has
+// no access to the panelManager/openAgentDetail callback.
+type PendingCommunicationAction =
+  | { readonly kind: 'open-orchestration' }
+  | { readonly kind: 'open-agent-detail'; readonly agentId: string };
+
 export class CommunicationPanel extends ScrollableListPanel<CommunicationRecord> {
   private readonly readModel?: UiReadModel<UiCommunicationSnapshot>;
   private readonly unsub: (() => void) | null;
+  private readonly openAgentDetail?: (agentId: string) => void;
   /** When true, the list is narrowed to blocked messages only (b toggles). */
   private blockedOnly = false;
+  private pendingAction: PendingCommunicationAction | null = null;
 
-  public constructor(readModel?: UiReadModel<UiCommunicationSnapshot>) {
-    super('communication', 'Communication', 'Y', 'monitoring');
+  public constructor(
+    readModel?: UiReadModel<UiCommunicationSnapshot>,
+    openAgentDetail?: (agentId: string) => void,
+  ) {
+    super('communication', 'Communication', 'Y', 'runtime-ops');
     this.showSelectionGutter = true; // I5: non-color selection affordance
     this.filterEnabled = true;
     this.filterLabel = 'Filter messages';
     this.readModel = readModel;
+    this.openAgentDetail = openAgentDetail;
     this.unsub = readModel ? readModel.subscribe(() => this.markDirty()) : null;
   }
 
@@ -86,9 +94,38 @@ export class CommunicationPanel extends ScrollableListPanel<CommunicationRecord>
       this.markDirty();
       return true;
     }
+    // Enter jumps to the orchestration workspace that actually produces this
+    // traffic (a direct panel jump, not a printed "/orchestration" signpost).
+    // Guarded so it doesn't hijack the filter input's commit-on-enter.
+    if ((key === 'enter' || key === 'return') && !this.filterActive) {
+      this.pendingAction = { kind: 'open-orchestration' };
+      return true;
+    }
+    // `o` on a blocked record opens the sender's agent detail (falling back
+    // to the receiver when the sender id is missing) via deps.openAgentDetail.
+    if (key === 'o' && !this.filterActive) {
+      const record = this.getSelectedItem();
+      if (!record || record.status !== 'blocked') return false;
+      const agentId = record.fromId || record.toId;
+      if (!agentId) return false;
+      this.pendingAction = { kind: 'open-agent-detail', agentId };
+      return true;
+    }
     return super.handleInput(key);
   }
 
+  public handlePanelIntegrationAction(_key: string, ctx: PanelIntegrationContext): boolean {
+    if (!this.pendingAction) return false;
+    const action = this.pendingAction;
+    this.pendingAction = null;
+    if (action.kind === 'open-orchestration') {
+      ctx.panelManager.open('orchestration');
+      return true;
+    }
+    this.openAgentDetail?.(action.agentId);
+    return true;
+  }
+
   protected renderItem(record: CommunicationRecord, _index: number, selected: boolean, width: number): Line {
     const bg = selected ? C.selectBg : undefined;
     const badge = buildStatusBadge(statusBadgeKind(record.status), record.status)[0]!;
@@ -117,7 +154,6 @@ export class CommunicationPanel extends ScrollableListPanel<CommunicationRecord>
   protected override getEmptyStateActions(): Array<{ command: string; summary: string }> {
     return [
       { command: '/orchestration', summary: 'review graphs and recursive agent activity that emit messages' },
-      { command: '/communication', summary: 'reopen this workspace once the runtime emits message traffic' },
     ];
   }
 
@@ -133,7 +169,7 @@ export class CommunicationPanel extends ScrollableListPanel<CommunicationRecord>
             width,
             ' Runtime store not wired into this panel yet.',
             'This workspace needs the live runtime store before it can show communication history and policy outcomes.',
-            [{ command: '/communication', summary: 'reopen the workspace from the shell-owned runtime' }],
+            [{ command: '/orchestration', summary: 'inspect recursive routing and agent activity while this workspace is unwired' }],
             C,
           ),
         }],
@@ -159,7 +195,7 @@ export class CommunicationPanel extends ScrollableListPanel<CommunicationRecord>
       ]),
       snapshot.totalBlocked > 0
         ? buildPanelLine(width, [[`  ${snapshot.totalBlocked} message${snapshot.totalBlocked !== 1 ? 's' : ''} blocked by routing policy — press b to isolate blocked lanes.`, C.warn]])
-        : buildGuidanceLine(width, '/orchestration', 'inspect recursive routing, message handoff, and broadcast posture', C),
+        : buildPanelLine(width, [['  No blocked lanes — press Enter on a record to inspect routing in the orchestration console.', C.dim]]),
     ];
 
     if (records.length === 0) {
@@ -171,14 +207,13 @@ export class CommunicationPanel extends ScrollableListPanel<CommunicationRecord>
     }
 
     this.clampSelection();
-    const visible = this.getVisibleItems();
-    const selected = visible[this.selectedIndex];
+    const selected = this.getSelectedItem();
 
     const detailLines: Line[] = [];
     if (selected) {
       detailLines.push(...buildDetailBlock(width, `Message · ${selected.kind}`, [
         buildKeyValueLine(width, [
-          { label: 'status', value: selected.status, valueColor: selected.status === 'blocked' ? C.error : C.ok },
+          { label: 'status', value: selected.status, valueColor: selected.status === 'blocked' ? C.bad : C.good },
           { label: 'scope', value: selected.scope, valueColor: C.value },
           { label: 'when', value: `${fmtAgo(selected.timestamp)} ago`, valueColor: C.dim },
         ], C),
@@ -196,15 +231,18 @@ export class CommunicationPanel extends ScrollableListPanel<CommunicationRecord>
     return this.renderList(width, height, {
       title: 'Communication Control Room',
       header: postureLines(),
-      footer: [...detailLines, buildKeyboardHints(width, this.footerHints(true), C)],
+      footer: [...detailLines, buildKeyboardHints(width, this.footerHints(true, selected?.status === 'blocked'), C)],
     });
   }
 
   // Context-aware hints: filter keys reflect filter state, `b` reflects the
-  // blocked-only toggle, and the inspect hint only appears when rows exist.
-  private footerHints(hasRows: boolean): Array<{ keys: string; label: string }> {
+  // blocked-only toggle, `o` only appears on a blocked selection, and the
+  // inspect hint only appears when rows exist.
+  private footerHints(hasRows: boolean, onBlockedSelection = false): Array<{ keys: string; label: string }> {
     const hints: Array<{ keys: string; label: string }> = [];
     if (hasRows) hints.push({ keys: '↑/↓', label: 'select' });
+    hints.push({ keys: 'Enter', label: 'orchestration' });
+    if (onBlockedSelection) hints.push({ keys: 'o', label: 'sender detail' });
     hints.push({ keys: 'b', label: this.blockedOnly ? 'show all lanes' : 'blocked only' });
     if (this.filterActive) {
       hints.push({ keys: 'Esc', label: 'clear filter' });

package/src/panels/confirm-state.ts

@@ -28,6 +28,12 @@ export interface ConfirmState<T = string> {
   readonly subject: T;
   /** Human-readable label for the item being destroyed. */
   readonly label: string;
+  /**
+   * Action verb shown in the confirmation prompt (e.g. 'Cancel', 'Regenerate',
+   * 'Promote'). Defaults to 'Delete' so existing destructive confirms are
+   * unaffected; non-destructive confirms should set this for honest copy.
+   */
+  readonly verb?: string;
 }
 
 /**
@@ -61,9 +67,10 @@ export function handleConfirmInput<T = string>(
  */
 export function renderConfirmLines<T = string>(width: number, state: ConfirmState<T>): Line[] {
   const palette = DEFAULT_PANEL_PALETTE;
+  const verb = state.verb ?? 'Delete';
   return [
     buildPanelLine(width, [[
-      ` Delete "${state.label}"?`,
+      ` ${verb} "${state.label}"?`,
       palette.warn,
     ]]),
     buildPanelLine(width, [