@pellux/goodvibes-tui 2.0.21 → 2.1.1

package/src/input/commands/notify-runtime.ts

@@ -2,6 +2,15 @@ import type { CommandRegistry } from '../command-registry.ts';
 import { requireWebhookNotifier } from './runtime-services.ts';
 
 export function registerNotifyRuntimeCommands(registry: CommandRegistry): void {
+  registry.register({
+    name: 'notifications',
+    aliases: [],
+    description: 'Open the notification history: everything collected, newest first; Enter goes to its subject',
+    handler(_args, ctx) {
+      if (ctx.openNotifications) ctx.openNotifications();
+      else ctx.print('The Notifications view is not available in this session.');
+    },
+  });
   registry.register({
     name: 'notify',
     aliases: [],

package/src/input/commands/operator-runtime.ts

@@ -2,14 +2,14 @@ import type { CommandRegistry } from '../command-registry.ts';
 import { ToolContractVerifier } from '@/runtime/index.ts';
 import type { ReplaySnapshotInput } from '@/runtime/index.ts';
 import { logger } from '@pellux/goodvibes-sdk/platform/utils';
-import { registerOperatorPanelCommand } from './operator-panel-runtime.ts';
+import { registerLegacyPanelCommand } from './legacy-panel-command.ts';
 import { requireOpsApi, requireProfileManager, requireReplayEngine } from './runtime-services.ts';
 import { summarizeError } from '@pellux/goodvibes-sdk/platform/utils';
 import { estimateConversationTokens } from '@pellux/goodvibes-sdk/platform/core';
 import { handleContextWindowSubcommand } from './context-window.ts';
 
 export function registerOperatorRuntimeCommands(registry: CommandRegistry): void {
-  registerOperatorPanelCommand(registry);
+  registerLegacyPanelCommand(registry);
 
   registry.register({
     name: 'settings',
@@ -187,12 +187,10 @@ export function registerOperatorRuntimeCommands(registry: CommandRegistry): void
       const sub = args[0];
 
       if (sub === 'view' || sub === undefined) {
-        // Parity with the Ctrl+O keybind (handler-shortcuts.ts): prefer the
-        // flag-wired opener, else fall back to the always-registered panel,
-        // which renders its own honest not-configured state.
-        if (ctx.openOpsPanel) ctx.openOpsPanel();
-        else if (ctx.showPanel) ctx.showPanel('ops-control');
-        else ctx.print('Operator Control Plane panel is not available in this shell.');
+        // The flag-wired opener when the control plane is on, else the
+        // Agents view, which holds the live operations content either way.
+        if (ctx.openOpsView) ctx.openOpsView();
+        else if (!ctx.openView?.('ops-control')) ctx.print('The Agents view is not available in this shell.');
         return;
       }
 
@@ -242,7 +240,7 @@ export function registerOperatorRuntimeCommands(registry: CommandRegistry): void
 
       ctx.print(
         'Usage: /ops <subcommand>\n'
-        + '  /ops view                              — open the Ops Control panel (Ctrl+O)\n'
+        + '  /ops view                              — open Agents (F2 or Ctrl+O)\n'
         + '  /ops task cancel <id> [note]           — cancel a task\n'
         + '  /ops task pause  <id> [note]           — pause a task\n'
         + '  /ops task resume <id> [note]           — resume a blocked task\n'
@@ -310,8 +308,8 @@ export function registerOperatorRuntimeCommands(registry: CommandRegistry): void
     handler(args, ctx) {
       const sub = args[0];
       if (sub === undefined || sub === 'view') {
-        if (ctx.openForensicsPanel) ctx.openForensicsPanel();
-        else ctx.print('Forensics panel is not available.');
+        if (ctx.openForensicsView) ctx.openForensicsView();
+        else ctx.print('The Agents view is not available.');
         return;
       }
       if (sub === 'latest') {
@@ -391,7 +389,7 @@ export function registerOperatorRuntimeCommands(registry: CommandRegistry): void
       }
       ctx.print(
         'Usage: /forensics <subcommand>\n'
-        + '  /forensics             — open the Forensics panel\n'
+        + '  /forensics             — open Agents on forensics\n'
         + '  /forensics latest      — print the most recent failure report summary\n'
         + '  /forensics show <id>   — show full JSON for a specific report\n'
         + '  /forensics export <id> — export incident bundle JSON to the conversation'

package/src/input/commands/permissions-provenance.ts

@@ -4,7 +4,7 @@ import { CONFIG_SCHEMA } from '@pellux/goodvibes-sdk/platform/config';
 import { SHIPPED_CREDENTIAL_READ_RULES } from '@pellux/goodvibes-sdk/platform/permissions';
 
 // ---------------------------------------------------------------------------
-// /permissions provenance panel.
+// /permissions provenance report.
 //
 // Shows every permission-relevant setting in effect AND where each value came
 // from. Provenance is taken from the platform's own ConfigManager.
@@ -58,7 +58,7 @@ export interface PermissionProvenance {
   readonly shippedRules: readonly ShippedPolicyRuleRow[];
 }
 
-/** The permission-relevant settings, in the order the panel lists them. */
+/** The permission-relevant settings, in the order the report lists them. */
 const PERMISSION_KEYS: readonly { readonly key: string; readonly label: string; readonly note?: string }[] = [
   { key: 'permissions.mode', label: 'Permission mode', note: 'also the current session mode' },
   { key: 'permissions.backgroundAgents', label: 'Background-agent mode' },
@@ -218,7 +218,7 @@ function displayValue(value: unknown): string {
   return JSON.stringify(value);
 }
 
-/** Render the provenance panel as plain lines for the command surface. */
+/** Render the provenance report as plain lines for the command surface. */
 export function renderPermissionProvenance(provenance: PermissionProvenance): string {
   const lines: string[] = [
     'Permissions: settings in effect and where each came from',

package/src/input/commands/planning-runtime.ts

@@ -80,7 +80,7 @@ export function registerPlanningRuntimeCommands(registry: CommandRegistry): void
 
       const projectPlanningService = ctx.workspace.projectPlanningService;
       const projectId = ctx.workspace.projectPlanningProjectId;
-      const openProjectPlanningPanel = () => openModalCommand(ctx, 'planning-modal');
+      const openProjectPlanningModal = () => openModalCommand(ctx, 'planning-modal');
 
       if (args.length === 0) {
         if (projectPlanningService && projectId) {
@@ -95,7 +95,7 @@ export function registerPlanningRuntimeCommands(registry: CommandRegistry): void
           const { evaluation } = stateResult.state
             ? await persistEvaluatedNextQuestion(projectPlanningService, projectId, stateResult.state, initialEvaluation)
             : { evaluation: initialEvaluation };
-          openProjectPlanningPanel();
+          openProjectPlanningModal();
           ctx.print(
             `Project planning: ${evaluation.readiness}\n` +
             `Project: ${status.projectId}\n` +
@@ -116,8 +116,8 @@ export function registerPlanningRuntimeCommands(registry: CommandRegistry): void
       }
 
       if (args[0] === 'panel') {
-        openProjectPlanningPanel();
-        ctx.print('Opened project planning panel.');
+        openProjectPlanningModal();
+        ctx.print('Opened project planning.');
         return;
       }
 
@@ -144,7 +144,7 @@ export function registerPlanningRuntimeCommands(registry: CommandRegistry): void
           },
         });
         const evaluation = await projectPlanningService.evaluate({ projectId });
-        openProjectPlanningPanel();
+        openProjectPlanningModal();
         ctx.print(`Project planning approved. Readiness: ${evaluation.readiness}. State: ${result.state?.id ?? 'current'}.`);
         return;
       }
@@ -255,7 +255,7 @@ export function registerPlanningRuntimeCommands(registry: CommandRegistry): void
           }
           return;
         }
-        openProjectPlanningPanel();
+        openProjectPlanningModal();
         ctx.print(
           `Recorded answer to: ${answerResult.question?.prompt ?? 'question'}\n` +
           `Readiness: ${answerResult.evaluation.readiness}\n` +
@@ -307,7 +307,7 @@ export function registerPlanningRuntimeCommands(registry: CommandRegistry): void
         ? await persistEvaluatedNextQuestion(projectPlanningService, projectId, result.state, initialEvaluation)
         : { state: result.state, evaluation: initialEvaluation };
       sessionLineageTracker.setOriginalTask(taskDescription.slice(0, 200));
-      openProjectPlanningPanel();
+      openProjectPlanningModal();
 
       ctx.print(
         `Project planning seeded: "${state?.goal ?? taskDescription}"\n` +

package/src/input/commands/platform-sandbox-runtime.ts

@@ -1,7 +1,7 @@
 import { mkdirSync, readFileSync, rmSync, writeFileSync } from 'node:fs';
 import { dirname, resolve } from 'node:path';
 import type { CommandRegistry } from '../command-registry.ts';
-import { requirePanelManager, requireShellPaths } from './runtime-services.ts';
+import { requireShellPaths } from './runtime-services.ts';
 import {
   getSandboxPreset,
   inspectSandboxBundle,
@@ -98,7 +98,7 @@ export function registerPlatformSandboxRuntimeCommands(registry: CommandRegistry
       const shellPaths = requireShellPaths(ctx);
       const sub = args[0] ?? 'open';
       if (sub === 'open' || sub === 'panel') {
-        ctx.openModal?.('sandbox-modal'); // sandbox panel -> config modal
+        ctx.openModal?.('sandbox-modal'); // sandbox view -> config modal
         return;
       }
       if (sub === 'review') {

package/src/input/commands/platform-services-runtime.ts

@@ -147,7 +147,7 @@ export function registerPlatformServicesRuntimeCommands(registry: CommandRegistr
           `  tasks: ${review.tasks}`,
           `  pending approvals: ${review.pendingApprovals}`,
           `  remote contracts: ${review.remoteContracts}`,
-          `  registered panels: ${review.panels}`,
+          `  registered views: ${review.panels}`,
           '  api families:',
           ...review.apiFamilies.map((family) => `    - ${family}`),
           '  routes:',

package/src/input/commands/policy-dispatch.ts

@@ -289,7 +289,7 @@ async function handleStatus(_args: string[], context: CommandContext): Promise<v
 export function renderPolicyUsage(): string {
   return [
     'Usage: /policy <subcommand>',
-    '  /policy                        — open the policy/governance panel',
+    '  /policy                        — open the policy/governance modal',
     '  load <bundle-id> [rule-count]  — Load a candidate bundle',
     '  simulate [mode]               — Run simulation (silent|warn|enforce)',
     '  diff                          — Show rule diff (current vs candidate)',
@@ -302,7 +302,7 @@ export function renderPolicyUsage(): string {
 }
 
 // W6 command-path parity: the policy modal dropped its 'r' (record a
-// divergence-dashboard trend sample) action because the panel used to call
+// divergence-dashboard trend sample) action because the old view called
 // this._state.recordTrendEntry() directly and /policy had no equivalent verb.
 // This exposes it as a thin wrapper: recordTrendEntry() forwards to the attached
 // DivergencePanel, so it only captures a sample while a simulation dashboard is

package/src/input/commands/policy.ts

@@ -4,12 +4,12 @@ import { dispatchPolicyCommand } from './policy-dispatch.ts';
 export const policyCommand: SlashCommand = {
   name: 'policy',
   aliases: ['pol'],
-  description: 'Open the policy panel or manage versioned policy bundles (load, simulate, diff, promote, rollback)',
+  description: 'Open the policy modal or manage versioned policy bundles (load, simulate, diff, promote, rollback)',
   usage: '<subcommand> [args]',
   argsHint: 'load|simulate|diff|lint|preflight|promote|rollback|status',
   handler: async (args: string[], context: CommandContext): Promise<void> => {
-    if (args.length === 0 && context.openPolicyPanel) {
-      context.openPolicyPanel();
+    if (args.length === 0 && context.openPolicyView) {
+      context.openPolicyView();
       return;
     }
     await dispatchPolicyCommand(args, context);

package/src/input/commands/provider.ts

@@ -1,7 +1,7 @@
 /**
  * /provider command handler.
  *
- * Implements the Provider Optimizer panel commands:
+ * Implements the Provider Optimizer commands:
  *
  *   /provider optimizer on|off  , Enable or disable the provider optimizer
  *   /provider route auto|manual , Set optimizer routing mode

package/src/input/commands/remote-runtime.ts

@@ -48,7 +48,7 @@ export function registerRemoteRuntimeCommands(registry: CommandRegistry): void {
     async handler(args, ctx) {
       if (args.length === 0) {
         if (ctx.openModal) {
-          ctx.openModal('remote-modal'); // remote panel -> config modal
+          ctx.openModal('remote-modal'); // remote view -> config modal
           return;
         }
         ctx.print('Remote surface is not available in this runtime.');

package/src/input/commands/review-runtime.ts

@@ -1,8 +1,6 @@
 import type { CommandContext, CommandRegistry } from '../command-registry.ts';
-import type { DiffReviewPanel } from '../../panels/diff-review-panel.ts';
-import type { ReviewHunk } from '../../panels/diff-review-model.ts';
-import { hunkPatchText, buildHunkRevertReceiptBlock } from '../../panels/diff-review-model.ts';
-import { requirePanelManager } from './runtime-services.ts';
+import type { ReviewHunk } from '../../views/diff-review-model.ts';
+import { hunkPatchText, buildHunkRevertReceiptBlock } from '../../views/diff-review-model.ts';
 
 /** checkpoints.revertHunkPreview result, the read-only clean-or-conflict check plus a confirm token. */
 interface RevertHunkPreview {
@@ -50,17 +48,17 @@ function errorText(err: unknown): string {
 }
 
 /**
- * Reject one hunk: checkpoints.revertHunkPreview (read-only clean check + token)
- * → DiffPanel confirm of exactly what reverts → checkpoints.revertHunk with that
- * token. A stale hunk is an honest "changed since captured" with a refresh, never
- * a partial write; a success renders the HUNK_REVERTED receipt into the transcript
- * and refreshes the review panel so the reverted hunk disappears from the diff.
+ * Revert one hunk in the working tree: checkpoints.revertHunkPreview (a
+ * read-only clean check that returns a token), a confirm dialog naming exactly
+ * what reverts, then checkpoints.revertHunk with that token. A hunk that no
+ * longer applies is an honest "changed since captured", never a partial write;
+ * a success adds the [Revert] receipt to the transcript. Resolves to the line
+ * the Changes modal shows.
  */
-async function revertHunkFlow(ctx: CommandContext, panel: DiffReviewPanel, hunk: ReviewHunk): Promise<void> {
+export async function revertReviewHunk(ctx: CommandContext, hunk: ReviewHunk): Promise<string> {
   const gateway = ctx.workspace.gatewayMethods;
   if (!gateway || !ctx.workspace.workspaceCheckpointManager) {
-    panel.note('Hunk revert unavailable: the checkpoint/gateway surface is not wired in this session.');
-    return;
+    return 'Hunk revert unavailable: the checkpoint/gateway surface is not wired in this session.';
   }
   const path = hunk.filePath;
   const hunkText = hunkPatchText(hunk);
@@ -70,103 +68,50 @@ async function revertHunkFlow(ctx: CommandContext, panel: DiffReviewPanel, hunk:
   try {
     preview = (await gateway.invoke('checkpoints.revertHunkPreview', { ...INVOKE_CONTEXT, body: { path, hunk: hunkText } })) as RevertHunkPreview;
   } catch (err) {
-    panel.note(`Could not check this hunk: ${errorText(err)}`);
-    return;
+    return `Could not check this hunk: ${errorText(err)}`;
   }
-
   if (!preview.applies || !preview.token) {
-    panel.note(`Cannot revert: ${preview.conflict ?? 'this hunk no longer applies'}. The file changed since this diff was captured; press r again after /review reloads. Nothing was changed.`);
-    await panel.refresh();
-    return;
+    return `Cannot revert: ${preview.conflict ?? 'this hunk no longer applies'}. The file changed since this diff was captured; the view has been reloaded. Nothing was changed.`;
   }
 
-  const { DiffPanel } = await import('../../panels/diff-panel.ts');
-  const pm = requirePanelManager(ctx);
-  let diff = pm.getAllOpen().find((p) => p.id === 'diff');
-  if (!diff) {
-    try { diff = pm.open('diff'); } catch { panel.note('Could not open the diff panel to confirm the revert.'); return; }
-  }
-  pm.activateById('diff');
-  if (!pm.isVisible()) pm.show();
-  ctx.focusPanels?.();
-  const diffPanel = diff as InstanceType<typeof DiffPanel>;
-  diffPanel.showDiff(path, hunkText);
+  const summary = `Restores ${preview.removedLinesRestored} deleted and drops ${preview.addedLinesRemoved} added line(s) in ${path} (${hunk.header}).`;
+  const confirmed = ctx.confirm
+    ? await ctx.confirm({ title: 'Revert this hunk?', body: `${summary}\nThe receipt in the transcript says how to undo it.`, confirmLabel: 'Revert', tone: 'danger' })
+    : false;
+  if (!confirmed) return 'Revert cancelled: nothing changed.';
 
-  const summary = `restore ${preview.removedLinesRestored} deleted / drop ${preview.addedLinesRemoved} added line(s)`;
-  diffPanel.confirmOverlay.arm({
-    id: `${path}:${hunk.header}`,
-    label: `Revert hunk in ${path}: ${summary}`,
-    verb: 'Revert',
-    onConfirm: async () => {
-      let result: RevertHunkResult;
-      try {
-        result = (await gateway.invoke('checkpoints.revertHunk', { ...INVOKE_CONTEXT, body: { path, hunk: hunkText, confirmToken: preview.token, sessionId } })) as RevertHunkResult;
-      } catch (err) {
-        pm.close('diff');
-        ctx.focusPanels?.();
-        if (isConflict(err)) {
-          panel.note(`Not reverted: ${errorText(err)}. The file changed since captured; nothing was written. Reloading /review…`);
-          await panel.refresh();
-        } else {
-          panel.note(`Revert failed: ${errorText(err)}`);
-        }
-        return;
-      }
-      pm.close('diff');
-      ctx.focusPanels?.();
-      const receipt = result.receipt;
-      if (!receipt) {
-        panel.note(`Revert not applied: ${result.refusal?.reason ?? 'confirmation was refused.'}`);
-        return;
-      }
-      ctx.session.conversationManager.addTypedSystemMessage(buildHunkRevertReceiptBlock(receipt), 'operational');
-      await panel.refresh();
-    },
-    onCancel: () => {
-      pm.close('diff');
-      ctx.focusPanels?.();
-      panel.note('Revert cancelled: nothing changed.');
-    },
-  });
-  ctx.renderRequest();
+  let result: RevertHunkResult;
+  try {
+    result = (await gateway.invoke('checkpoints.revertHunk', { ...INVOKE_CONTEXT, body: { path, hunk: hunkText, confirmToken: preview.token, sessionId } })) as RevertHunkResult;
+  } catch (err) {
+    return isConflict(err)
+      ? `Not reverted: ${errorText(err)}. The file changed since captured; nothing was written.`
+      : `Revert failed: ${errorText(err)}`;
+  }
+  const receipt = result.receipt;
+  if (!receipt) return `Revert not applied: ${result.refusal?.reason ?? 'confirmation was refused.'}`;
+  ctx.session.conversationManager.addTypedSystemMessage(buildHunkRevertReceiptBlock(receipt), 'operational');
+  return `Reverted the hunk in ${receipt.path}; the receipt is in the transcript.`;
 }
 
 /**
- * `/review`, open the comment-on-hunk review loop. Loads this session's file
- * changes (the files the SDK SessionChangeTracker recorded) as a git working-
- * tree diff, hunk-boundaried, and wires the panel's steering submit path to the
- * session so an attached comment is sent to the model as a steering message with
- * structured context (file path, line range, patch excerpt). A hunk can also be
- * rejected: `r` reverse-applies exactly that one hunk via checkpoints.revertHunk
- * (preview + confirm), a working-tree change recorded as a [Revert] receipt.
+ * `/review`, the comment-on-hunk review loop, in the Changes modal: this
+ * session's file changes (the files the SDK SessionChangeTracker recorded) as a
+ * working-tree diff, hunk by hunk. c attaches a comment to a hunk and Enter
+ * sends the attached comments to the model as one steering message with each
+ * hunk's file, line range and patch excerpt; x reverts a hunk (asks first).
  */
 export function registerReviewRuntimeCommands(registry: CommandRegistry): void {
   registry.register({
     name: 'review',
     aliases: [],
     description: 'Review this session\'s diff hunk-by-hunk, steer comments, or revert a hunk',
-    async handler(_args, ctx) {
-      const pm = requirePanelManager(ctx);
-      let panel = pm.getAllOpen().find((p) => p.id === 'review');
-      if (!panel) {
-        try {
-          panel = pm.open('review');
-        } catch {
-          ctx.print('Could not open the review panel.');
-          return;
-        }
-      }
-      pm.activateById('review');
-      if (!pm.isVisible()) pm.show();
-
-      const reviewPanel = panel as DiffReviewPanel;
-      if (ctx.submitInput) {
-        reviewPanel.setSubmit((text) => ctx.submitInput!(text));
+    handler(_args, ctx) {
+      if (!ctx.openChanges) {
+        ctx.print('The Changes view is not available in this session.');
+        return;
       }
-      reviewPanel.setRevertHandler((hunk) => { void revertHunkFlow(ctx, reviewPanel, hunk); });
-      await reviewPanel.loadSessionReview();
-      ctx.focusPanels?.();
-      ctx.renderRequest();
+      ctx.openChanges({ source: 'session' });
     },
   });
 }