@pellux/goodvibes-tui 0.28.0 → 1.0.0

package/src/panels/agent-inspector-shared.ts

@@ -1,4 +1,7 @@
 import { formatDuration } from '../utils/format-duration.ts';
+import { calcSessionCost } from '../export/cost-utils.ts';
+import type { AgentEvent } from '@/runtime/index.ts';
+import type { UiEventFeed } from '../runtime/ui-events.ts';
 
 export type AgentInspectorEntryKind = 'user' | 'assistant' | 'tool_call' | 'tool_result' | 'session' | 'error';
 
@@ -121,3 +124,144 @@ export function agentKindStyle(kind: AgentInspectorEntryKind, colors: Record<str
     default: return { fg: colors.dimmed ?? colors.system, prefix: '[?]        ' };
   }
 }
+
+// ---------------------------------------------------------------------------
+// Inspector filter modes (ported from the merged agent-logs console — WO-110)
+// ---------------------------------------------------------------------------
+
+export type AgentInspectorFilterType = 'all' | 'assistant' | 'tool' | 'error';
+
+export const AGENT_INSPECTOR_FILTER_CYCLE: readonly AgentInspectorFilterType[] = ['all', 'assistant', 'tool', 'error'];
+
+export const AGENT_INSPECTOR_FILTER_LABELS: Record<AgentInspectorFilterType, string> = {
+  all: 'All',
+  assistant: 'Assistant',
+  tool: 'Tool',
+  error: 'Error',
+};
+
+/**
+ * True when a timeline entry's kind should be visible under the given filter.
+ * Mapped against the REAL row kinds jsonlToTimeline() produces (tool_call /
+ * tool_result), not the legacy agent-logs-shared schema (assistant/tool_call/
+ * tool_result vs. the writer-emitted tool_execution/llm_response) that never
+ * matched anything.
+ */
+export function matchesInspectorFilter(kind: AgentInspectorEntryKind, filter: AgentInspectorFilterType): boolean {
+  switch (filter) {
+    case 'all': return true;
+    case 'assistant': return kind === 'assistant';
+    case 'tool': return kind === 'tool_call' || kind === 'tool_result';
+    case 'error': return kind === 'error';
+    default: return true;
+  }
+}
+
+// ---------------------------------------------------------------------------
+// Per-agent cost/token summary (calcSessionCost, cost-utils.ts)
+// ---------------------------------------------------------------------------
+
+export function formatAgentCost(usd: number): string {
+  if (usd === 0) return '$0.00';
+  if (usd < 0.0001) return '<$0.0001';
+  if (usd < 0.01) return `$${usd.toFixed(4)}`;
+  return `$${usd.toFixed(3)}`;
+}
+
+export interface AgentUsageSummary {
+  readonly tokens: number;
+  readonly cost: number;
+}
+
+interface AgentUsageLike {
+  readonly usage?: {
+    readonly inputTokens: number;
+    readonly outputTokens: number;
+    readonly cacheReadTokens?: number;
+    readonly cacheWriteTokens?: number;
+  } | null;
+  readonly model?: string | null;
+}
+
+/**
+ * Total token count + cost for an agent record's usage, or null when no usage
+ * data has landed yet (honest-UX: never fabricate a $0.00/0-token reading for
+ * an agent that simply hasn't reported usage).
+ */
+export function summarizeAgentUsage(rec: AgentUsageLike): AgentUsageSummary | null {
+  if (!rec.usage) return null;
+  const inputTokens = rec.usage.inputTokens + (rec.usage.cacheReadTokens ?? 0) + (rec.usage.cacheWriteTokens ?? 0);
+  const cost = calcSessionCost(
+    rec.usage.inputTokens,
+    rec.usage.outputTokens,
+    rec.usage.cacheReadTokens ?? 0,
+    rec.usage.cacheWriteTokens ?? 0,
+    rec.model ?? 'unknown',
+  );
+  return { tokens: inputTokens + rec.usage.outputTokens, cost };
+}
+
+// ---------------------------------------------------------------------------
+// WRFC badge segments (AgentRecord.wrfcId/wrfcRole)
+// ---------------------------------------------------------------------------
+
+interface WrfcLike {
+  readonly wrfcId?: string | null;
+  readonly wrfcRole?: string | null;
+}
+
+/** Segments for the WRFC chain badge + token/cost line, or null when neither applies. */
+export function buildWrfcCostSegments(
+  rec: WrfcLike & AgentUsageLike,
+  palette: { readonly label: string; readonly info: string },
+  formatTokens: (n: number) => string,
+): Array<[string, string]> | null {
+  const segments: Array<[string, string]> = [];
+  if (rec.wrfcId) {
+    segments.push([' WRFC ', palette.label]);
+    segments.push([`${rec.wrfcRole ?? 'agent'} · ${rec.wrfcId.slice(-8)}`, palette.info]);
+  }
+  const usage = summarizeAgentUsage(rec);
+  if (usage) {
+    segments.push([segments.length > 0 ? '   Tokens ' : ' Tokens ', palette.label]);
+    segments.push([formatTokens(usage.tokens), palette.info]);
+    segments.push(['   Cost ', palette.label]);
+    segments.push([formatAgentCost(usage.cost), palette.info]);
+  }
+  return segments.length > 0 ? segments : null;
+}
+
+// ---------------------------------------------------------------------------
+// Agent lifecycle event subscriptions (ported from agent-logs — WO-110)
+// ---------------------------------------------------------------------------
+
+export interface AgentActivityHooks {
+  /** Any AGENT_* activity event for this agent (running/progress/stream/awaiting/finalizing/spawning). */
+  readonly onActivity: (agentId: string) => void;
+  /** AGENT_SPAWNING only — used for auto-select-newest-agent. */
+  readonly onSpawn: (agentId: string) => void;
+  /** AGENT_COMPLETED/FAILED/CANCELLED. */
+  readonly onTerminal: (agentId: string) => void;
+}
+
+/** Wires the standard set of agent lifecycle listeners; returns their unsubscribe functions. */
+export function subscribeAgentActivity(agentEvents: UiEventFeed<AgentEvent>, hooks: AgentActivityHooks): Array<() => void> {
+  const unsubs: Array<() => void> = [];
+  const activityTypes = [
+    'AGENT_RUNNING', 'AGENT_PROGRESS', 'AGENT_STREAM_DELTA',
+    'AGENT_AWAITING_MESSAGE', 'AGENT_AWAITING_TOOL', 'AGENT_FINALIZING',
+  ] as const;
+  for (const type of activityTypes) {
+    unsubs.push(agentEvents.on(type, (payload: { agentId: string }) => hooks.onActivity(payload.agentId)));
+  }
+  unsubs.push(agentEvents.on('AGENT_SPAWNING', (payload) => {
+    hooks.onActivity(payload.agentId);
+    hooks.onSpawn(payload.agentId);
+  }));
+  unsubs.push(
+    agentEvents.on('AGENT_COMPLETED', (payload) => hooks.onTerminal(payload.agentId)),
+    agentEvents.on('AGENT_FAILED', (payload) => hooks.onTerminal(payload.agentId)),
+    agentEvents.on('AGENT_CANCELLED', (payload) => hooks.onTerminal(payload.agentId)),
+  );
+  return unsubs;
+}

package/src/panels/approval-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 {
   buildBodyText,
   buildDetailBlock,
@@ -15,10 +16,18 @@ import { truncateDisplay } from '../utils/terminal-width.ts';
 import type { PolicyRuntimeState, PermissionAuditEntry } from '@/runtime/index.ts';
 import { buildPermissionRuleSuggestions } from '@/runtime/index.ts';
 
-const C = {
-  ...DEFAULT_PANEL_PALETTE,
-  headerBg: '#111827',
-} as const;
+// Splits a computed '/command arg1 arg2' string into the { name, args } shape
+// ctx.executeCommand expects — same leading-slash-stripping approach used by
+// remote-panel.ts / session-browser-panel.ts for their own dispatched commands.
+function parseCommand(command: string): { name: string; args: string[] } | null {
+  const parts = command.replace(/^\//, '').split(/\s+/).filter(Boolean);
+  const [name, ...args] = parts;
+  return name ? { name, args } : null;
+}
+
+// Base chrome only — title band comes straight from DEFAULT_PANEL_PALETTE
+// (WO-002).
+const C = DEFAULT_PANEL_PALETTE;
 
 // Reference catalog of approval lanes and where each one is reviewed. Used to
 // resolve the next-step command for a live request and as a fallback reference
@@ -71,13 +80,45 @@ function fmtAgo(ts: number): string {
 
 export class ApprovalPanel extends ScrollableListPanel<PermissionAuditEntry> {
   private readonly policyRuntimeState: Pick<PolicyRuntimeState, 'getSnapshot'>;
+  // p / 1-3 = dispatch '/policy simulate' or a rule-suggestion command via the
+  // handleInput -> handlePanelIntegrationAction bridge (handleInput has no
+  // ctx.executeCommand — same pattern as remote-panel.ts / session-browser-panel.ts).
+  private pendingCommand: { name: string; args: string[] } | null = null;
 
   public constructor(policyRuntimeState: Pick<PolicyRuntimeState, 'getSnapshot'>) {
-    super('approval', 'Approval', 'A', 'monitoring');
+    super('approval', 'Approval', 'A', 'security-policy');
     this.showSelectionGutter = true; // I5: non-color selection affordance
     this.policyRuntimeState = policyRuntimeState;
   }
 
+  private getRuleSuggestions() {
+    const audit = this.policyRuntimeState.getSnapshot().recentPermissionAudit;
+    return buildPermissionRuleSuggestions(audit).slice(0, 3);
+  }
+
+  override handleInput(key: string): boolean {
+    if (key === 'p') {
+      this.pendingCommand = parseCommand('/policy simulate');
+      return true;
+    }
+    if (key === '1' || key === '2' || key === '3') {
+      const suggestion = this.getRuleSuggestions()[Number(key) - 1];
+      if (!suggestion) return false;
+      this.pendingCommand = parseCommand(suggestion.command);
+      return true;
+    }
+    return super.handleInput(key);
+  }
+
+  /** Drains `pendingCommand` (set by 'p' or '1'/'2'/'3') via the ctx.executeCommand bridge. */
+  handlePanelIntegrationAction(_key: string, ctx: PanelIntegrationContext): boolean {
+    if (!this.pendingCommand) return false;
+    const { name, args } = this.pendingCommand;
+    this.pendingCommand = null;
+    void ctx.executeCommand?.(name, args);
+    return true;
+  }
+
   protected override getPalette() { return C; }
   protected override getEmptyStateMessage() { return ' No approval pressure right now.'; }
   protected override getEmptyStateActions() {
@@ -112,8 +153,7 @@ export class ApprovalPanel extends ScrollableListPanel<PermissionAuditEntry> {
 
   /** Next-step review command for the currently selected request (if any). */
   public getSelectedCommand(): string | null {
-    const items = this.getVisibleItems();
-    const selected = items[this.selectedIndex];
+    const selected = this.getSelectedItem();
     return selected ? reviewFor(selected).command : null;
   }
 
@@ -124,8 +164,7 @@ export class ApprovalPanel extends ScrollableListPanel<PermissionAuditEntry> {
     const denialCount = audit.filter((e) => e.approved === false).length;
     const pendingCount = audit.filter((e) => e.approved === undefined).length;
 
-    const items = this.getVisibleItems();
-    const selected = items[this.selectedIndex] ?? null;
+    const selected = this.getSelectedItem() ?? null;
 
     // ---- Posture summary (severity + counts first) ----
     const headerLines: Line[] = [
@@ -139,7 +178,10 @@ export class ApprovalPanel extends ScrollableListPanel<PermissionAuditEntry> {
       ]),
       pendingCount > 0
         ? buildPanelLine(width, [[`  ${pendingCount} request${pendingCount !== 1 ? 's' : ''} awaiting a decision — select one to see its review path.`, C.warn]])
-        : buildGuidanceLine(width, '/policy simulate', 'preview which requests a scoped rule change would auto-approve', C),
+        : buildPanelLine(width, [
+            ['  p ', C.info],
+            ['/policy simulate — preview which requests a scoped rule change would auto-approve', C.dim],
+          ]),
     ];
 
     // ---- Detail block for the selected request ----
@@ -162,18 +204,25 @@ export class ApprovalPanel extends ScrollableListPanel<PermissionAuditEntry> {
         ...(selected.reasons[0]
           ? buildBodyText(width, `why prompted: ${selected.reasons[0]}`, C, C.dim)
           : buildBodyText(width, `why prompted: ${review.why}`, C, C.dim)),
-        buildGuidanceLine(width, review.command, `review and decide the ${laneOf(selected)} request`, C),
+        // WO-160: dropped the printed `review.command` guidance line here —
+        // Enter is already wired by the shell to dispatch that exact command
+        // (see onSelect below and the "Enter: review (<command>)" footer
+        // hint), so restating it as a signpost was a pure action substitute.
       ], C));
     }
 
-    // ---- Durable-rule suggestions from repeated denials ----
+    // ---- Durable-rule suggestions from repeated denials (dispatchable via 1/2/3) ----
+    const suggestions = this.getRuleSuggestions();
     const ruleSuggestionLines: Line[] = [];
-    for (const suggestion of buildPermissionRuleSuggestions(audit).slice(0, 3)) {
-      ruleSuggestionLines.push(buildPanelLine(width, [[`  ${truncateDisplay(suggestion.summary, Math.max(0, width - 4))}`, C.info]]));
+    suggestions.forEach((suggestion, index) => {
+      ruleSuggestionLines.push(buildPanelLine(width, [
+        [`  ${index + 1}) `, C.info],
+        [truncateDisplay(suggestion.summary, Math.max(0, width - 8)), C.info],
+      ]));
       ruleSuggestionLines.push(buildGuidanceLine(width, suggestion.command, suggestion.reason, C));
-    }
+    });
     if (ruleSuggestionLines.length > 0) {
-      ruleSuggestionLines.unshift(buildPanelLine(width, [['  Suggested durable rules', C.label]]));
+      ruleSuggestionLines.unshift(buildPanelLine(width, [['  Suggested durable rules (press 1/2/3 to apply)', C.label]]));
     }
 
     // ---- Context-aware footer: only show review key when a request is selected ----
@@ -187,6 +236,8 @@ export class ApprovalPanel extends ScrollableListPanel<PermissionAuditEntry> {
           { keys: '↑/↓', label: 'select' },
           { keys: 'g/G', label: 'top/bottom' },
         ];
+    if (suggestions.length > 0) hints.push({ keys: '1-3', label: 'apply rule suggestion' });
+    if (pendingCount === 0) hints.push({ keys: 'p', label: 'policy simulate' });
 
     return this.renderList(width, height, {
       title: 'Approval Control Room',