@pellux/goodvibes-agent 0.1.102 → 0.1.104

package/src/input/handler.ts

@@ -21,14 +21,12 @@ import type { BlockMeta, ConversationManager } from '../core/conversation';
 import { ProcessModal } from '../renderer/process-modal.ts';
 import { LiveTailModal } from '../renderer/live-tail-modal.ts';
 import { BlockActionsMenu } from '../renderer/block-actions.ts';
-import { AgentDetailModal } from '../renderer/agent-detail-modal.ts';
 import { ContextInspectorModal } from '../renderer/context-inspector.ts';
 import { BookmarkModal } from './bookmark-modal.ts';
 import { SettingsModal } from './settings-modal.ts';
 import { McpWorkspace } from './mcp-workspace.ts';
 import { AgentWorkspace } from './agent-workspace.ts';
 import { SessionPickerModal } from './session-picker-modal.ts';
-import { GOODVIBES_AGENT_SURFACE_ROOT } from '../config/surface.ts';
 import { ProfilePickerModal } from './profile-picker-modal.ts';
 import { OnboardingWizardController, type OnboardingWizardAction, type OnboardingWizardMode } from './onboarding/onboarding-wizard.ts';
 import {
@@ -155,7 +153,6 @@ export class InputHandler {
   public searchManager = new SearchManager();
   public processModal: ProcessModal;
   public liveTailModal: LiveTailModal;
-  public agentDetailModal: AgentDetailModal;
   public contextInspectorModal = new ContextInspectorModal();
   public bookmarkModal: BookmarkModal;
   public blockActionsMenu = new BlockActionsMenu();
@@ -220,8 +217,7 @@ export class InputHandler {
     public scroll: (delta: number) => void,
     public exitApp: () => void,
     public readonly uiServices: Pick<UiRuntimeServices,
-      'agents'
-      | 'environment'
+      'environment'
       | 'platform'
       | 'providers'
       | 'sessions'
@@ -235,22 +231,11 @@ export class InputHandler {
       uiServices.providers.providerRegistry,
     );
     this.processModal = new ProcessModal({
-      agentManager: uiServices.agents.agentManager,
       processManager: uiServices.shell.processManager,
-      wrfcController: uiServices.agents.wrfcController,
-      agentEntries: 'hidden',
     });
     this.liveTailModal = new LiveTailModal({
-      agentManager: uiServices.agents.agentManager,
       processManager: uiServices.shell.processManager,
     });
-    this.agentDetailModal = new AgentDetailModal({
-      agentManager: uiServices.agents.agentManager,
-      agentMessageBus: uiServices.agents.agentMessageBus,
-      sessionLogPathResolver: (agentId) => uiServices.environment.shellPaths.resolveProjectPath(GOODVIBES_AGENT_SURFACE_ROOT, 'sessions', `${agentId}.jsonl`),
-      // SDK 0.23.0: supply wrfcController so the modal can show constraint data
-      wrfcController: uiServices.agents.wrfcController,
-    });
     this.bookmarkModal = new BookmarkModal(uiServices.shell.bookmarkManager);
     this.sessionPickerModal = new SessionPickerModal(uiServices.sessions.sessionManager);
     this.profilePickerModal = new ProfilePickerModal(uiServices.shell.profileManager);
@@ -294,7 +279,6 @@ export class InputHandler {
         onboardingWizard: this.onboardingWizard,
         processModal: this.processModal,
         liveTailModal: this.liveTailModal,
-        agentDetailModal: this.agentDetailModal,
         contextInspectorModal: this.contextInspectorModal,
         blockActionsMenu: this.blockActionsMenu,
         searchManager: this.searchManager,

package/src/main.ts

@@ -398,11 +398,6 @@ async function main() {
     scroll,
     exitApp,
     {
-      agents: {
-        agentManager,
-        agentMessageBus: ctx.services.agentMessageBus,
-        wrfcController: ctx.services.wrfcController,
-      },
       providers: {
         benchmarkStore: ctx.services.benchmarkStore,
         favoritesStore: ctx.services.favoritesStore,
@@ -450,7 +445,6 @@ async function main() {
   input.setConversationManager(conversation);
   input.setContentWidth(getPromptContentWidth());
   input.filePicker.setOnUpdate(() => render());
-  input.agentDetailModal.setOnRefresh(() => render());
   input.processModal.setOnRefresh(() => render());
 
   // Model picker callback is handled in bootstrap.ts — do not duplicate here.

package/src/panels/builtin/agent.ts

@@ -1,5 +1,4 @@
 import type { PanelManager } from '../panel-manager.ts';
-import { AgentLogsPanel } from '../agent-logs-panel.ts';
 import { ContextVisualizerPanel } from '../context-visualizer-panel.ts';
 import { ThinkingPanel } from '../thinking-panel.ts';
 import { ToolInspectorPanel } from '../tool-inspector-panel.ts';
@@ -50,22 +49,6 @@ export function registerAgentPanels(manager: PanelManager, deps: ResolvedBuiltin
     ),
   });
 
-  manager.registerType({
-    id: 'agent-logs',
-    name: 'Agents',
-    icon: 'A',
-    category: 'agent',
-    description: 'View-only stream for explicit delegated build/review session records',
-    preload: true,
-    factory: () => {
-      const ui = requireUiServices(deps);
-      return new AgentLogsPanel(ui.events.agents, {
-        agentManager: ui.agents.agentManager,
-        workingDirectory: ui.environment.workingDirectory,
-      });
-    },
-  });
-
   manager.registerType({
     id: 'work-plan',
     name: 'Work Plan',

package/src/panels/index.ts

@@ -4,8 +4,6 @@ export { BasePanel } from './base-panel.ts';
 export { PanelManager } from './panel-manager.ts';
 export { TokenBudgetPanel } from './token-budget-panel.ts';
 export { CostTrackerPanel } from './cost-tracker-panel.ts';
-export { AgentInspectorPanel } from './agent-inspector-panel.ts';
-export { AgentLogsPanel } from './agent-logs-panel.ts';
 export { ProviderHealthPanel } from './provider-health-panel.ts';
 export { ProviderHealthTracker } from './provider-health-tracker.ts';
 export type { ProviderHealth, ProviderStatus } from './provider-health-tracker.ts';

package/src/renderer/agent-workspace.ts

@@ -123,6 +123,8 @@ function localLibraryLines(
       selected ? 'selected' : '',
       item.active ? 'active' : '',
       item.enabled === true ? 'enabled' : item.enabled === false ? 'disabled' : '',
+      item.scope && item.cls ? `${item.scope}/${item.cls}` : '',
+      item.confidence !== undefined ? `${item.confidence}%` : '',
       item.reviewState,
       item.startCount !== undefined ? `starts ${item.startCount}` : '',
     ].filter(Boolean).join(' / ');
@@ -194,8 +196,8 @@ function snapshotLines(workspace: AgentWorkspace, category: AgentWorkspaceCatego
     );
   } else if (category.id === 'setup') {
     base.push(
-      { text: `GoodVibes runtime: ${snapshot.daemonBaseUrl}`, fg: PALETTE.info },
-      { text: `Runtime owner: ${snapshot.daemonOwnership}; Agent connects but never starts or restarts it`, fg: PALETTE.good },
+      { text: `GoodVibes runtime: ${snapshot.runtimeBaseUrl}`, fg: PALETTE.info },
+      { text: `Runtime owner: ${snapshot.runtimeOwnership}; Agent connects but never starts or restarts it`, fg: PALETTE.good },
       ...setupChecklistLines(snapshot),
       { text: '' },
       { text: `Workspace: ${snapshot.workingDirectory}`, fg: PALETTE.muted },
@@ -216,7 +218,7 @@ function snapshotLines(workspace: AgentWorkspace, category: AgentWorkspaceCatego
       ...snapshot.channels.filter((channel) => !channel.enabled),
     ].slice(0, 6);
     base.push(
-      { text: `GoodVibes runtime: ${snapshot.daemonBaseUrl}`, fg: PALETTE.info },
+      { text: `GoodVibes runtime: ${snapshot.runtimeBaseUrl}`, fg: PALETTE.info },
       { text: `Readiness: ${readyCount}/${snapshot.channels.length} ready; ${enabledCount} enabled; ${configuredDefaults} default target(s) configured.`, fg: PALETTE.info },
       { text: `Ready channels: ${readyChannels.join(', ') || 'none'}.`, fg: readyChannels.length > 0 ? PALETTE.good : PALETTE.warn },
       { text: `Needs default target: ${needsTarget.map((channel) => `${channel.label} -> ${channel.defaultTargetKeys.join('|')}`).join(', ') || 'none'}.`, fg: needsTarget.length > 0 ? PALETTE.warn : PALETTE.muted },
@@ -301,11 +303,14 @@ function snapshotLines(workspace: AgentWorkspace, category: AgentWorkspaceCatego
   } else if (category.id === 'memory') {
     base.push(
       { text: `Session memories: ${snapshot.sessionMemoryCount}`, fg: PALETTE.info },
+      { text: `Agent memory: ${snapshot.localMemoryCount}; review queue: ${snapshot.localMemoryReviewQueueCount}`, fg: PALETTE.info },
       { text: `Local routines: ${snapshot.localRoutineCount}; enabled: ${snapshot.enabledRoutineCount}`, fg: PALETTE.info },
       { text: `Local skills: ${snapshot.localSkillCount}; enabled: ${snapshot.enabledSkillCount}; bundles: ${snapshot.localSkillBundleCount}; active skills: ${snapshot.activeSkillCount}`, fg: PALETTE.info },
       { text: `Local personas: ${snapshot.localPersonaCount}; active: ${snapshot.activePersonaName}`, fg: PALETTE.info },
       { text: 'Durable memory, routines, skills, and personas remain Agent-local until shared registry contracts exist.', fg: PALETTE.good },
       { text: 'Secrets are rejected/redacted; store secret references instead of secret values.', fg: PALETTE.warn },
+      { text: '' },
+      ...localLibraryLines('Agent Memory', snapshot.localMemories, 'No Agent memory yet. Create one here with Create memory.', workspace.selectedLocalLibraryItem('memory')?.id ?? null),
     );
   } else if (category.id === 'personas') {
     base.push(

package/src/renderer/conversation-overlays.ts

@@ -9,7 +9,6 @@ import { renderSelectionModalOverlay } from './selection-modal-overlay.ts';
 import { renderSearchOverlay } from './search-overlay.ts';
 import { renderHistorySearchOverlay } from './history-search-overlay.ts';
 import { renderProcessModal } from './process-modal.ts';
-import { renderAgentDetailModal } from './agent-detail-modal.ts';
 import { renderLiveTailModal } from './live-tail-modal.ts';
 import { renderContextInspector } from './context-inspector.ts';
 import { renderSettingsModal } from './settings-modal.ts';
@@ -74,11 +73,6 @@ export function applyConversationOverlays(
     next = overlayViewportBottom(next, lines, conversationWidth, viewportHeight, bottomDockInset);
   }
 
-  if (input.agentDetailModal.active) {
-    const lines = renderAgentDetailModal(input.agentDetailModal, conversationWidth);
-    next = overlayViewportBottom(next, lines, conversationWidth, viewportHeight, bottomDockInset);
-  }
-
   if (input.liveTailModal.active) {
     const lines = renderLiveTailModal(input.liveTailModal, conversationWidth, viewportHeight);
     next = overlayViewportBottom(next, lines, conversationWidth, viewportHeight, bottomDockInset);

package/src/renderer/live-tail-modal.ts

@@ -1,28 +1,16 @@
 import { type Line } from '../types/grid.ts';
 import { ModalFactory } from './modal-factory.ts';
 import type { ProcessManager } from '@pellux/goodvibes-sdk/platform/tools';
-import type { AgentManager } from '@pellux/goodvibes-sdk/platform/tools';
 import type { ProcessEntry } from './process-modal.ts';
 import { getOverlaySurfaceMetrics, getStableOverlayContentRows } from './overlay-viewport.ts';
 
 export interface LiveTailModalDeps {
-  readonly agentManager: Pick<AgentManager, 'getStatus'>;
   readonly processManager: Pick<ProcessManager, 'stop' | 'getOutput'>;
 }
 
-// ─── LiveTailModal ────────────────────────────────────────────────────────────
-
-/**
- * LiveTailModal — manages state for the live output peek modal.
- *
- * Shows streaming stdout/stderr from a selected shell process or agent
- * progress notes. Auto-scrolls to the bottom unless the user scrolled up.
- */
 export class LiveTailModal {
   public active = false;
   public entry: ProcessEntry | null = null;
-
-  /** Number of lines scrolled up from the bottom (0 = at bottom). */
   public scrollOffset = 0;
 
   constructor(private readonly deps: LiveTailModalDeps) {}
@@ -40,64 +28,27 @@ export class LiveTailModal {
   }
 
   scrollUp(): void {
-    this.scrollOffset++;
+    this.scrollOffset += 1;
   }
 
   scrollDown(): void {
     this.scrollOffset = Math.max(0, this.scrollOffset - 1);
   }
 
-  /**
-   * Stop the current shell process when it is an exec entry.
-   * Agent entries are read-only in GoodVibes Agent; build execution and
-   * cancellation belong to GoodVibes TUI/shared-session owners.
-   */
-  killProcess(): boolean {
+  stopProcess(): boolean {
     if (!this.entry) return false;
-
-    if (this.entry.type === 'exec') {
-      return this.deps.processManager.stop(this.entry.id);
-    }
-    return false;
+    return this.deps.processManager.stop(this.entry.id);
   }
 
-  /** Retrieve the current output text for the watched process. */
   getOutput(): string {
     if (!this.entry) return '';
-
-    if (this.entry.type === 'exec') {
-      const out = this.deps.processManager.getOutput(this.entry.id);
-      if (!out) return '';
-      const combined = [out.stdout, out.stderr].filter(Boolean).join('\n').trim();
-      return combined || '(no output yet)';
-    } else {
-      // For agents, show progress note and status
-      const rec = this.deps.agentManager.getStatus(this.entry.id);
-      if (!rec) return '(process not found)';
-      const parts: string[] = [
-        `Task: ${rec.task}`,
-        `Status: ${rec.status}`,
-        `Tool calls: ${rec.toolCallCount}`,
-      ];
-      if (rec.progress) parts.push(`Progress: ${rec.progress}`);
-      if (rec.error) parts.push(`Error: ${rec.error}`);
-      return parts.join('\n');
-    }
+    const output = this.deps.processManager.getOutput(this.entry.id);
+    if (!output) return '';
+    const combined = [output.stdout, output.stderr].filter(Boolean).join('\n').trim();
+    return combined || '(no output yet)';
   }
 }
 
-// ─── renderLiveTailModal ──────────────────────────────────────────────────────
-
-/**
- * Render the live-tail peek modal as Line[] for overlay in the viewport.
- *
- * Shows a scrollable view of the process output. scrollOffset=0 means the
- * bottom of the output is visible (auto-scroll behaviour).
- *
- * @param modal  LiveTailModal state
- * @param width  Terminal width
- * @param maxOutputLines  Maximum lines to show inside the box (default: 16)
- */
 export function renderLiveTailModal(
   modal: LiveTailModal,
   width: number,
@@ -115,35 +66,25 @@ export function renderLiveTailModal(
 
   const output = modal.getOutput();
   const allLines = output.split('\n');
-
-  // Apply scroll: scrollOffset=0 → show tail; larger → scroll toward head
   const totalLines = allLines.length;
-  // Clamp scrollOffset so we never scroll past the top of the content
   const maxScroll = Math.max(0, totalLines - maxOutputLines);
   const clampedOffset = Math.min(modal.scrollOffset, maxScroll);
   const endIdx = Math.max(maxOutputLines, totalLines - clampedOffset);
   const startIdx = Math.max(0, endIdx - maxOutputLines);
   const visibleLines = allLines.slice(startIdx, endIdx);
 
-  const typeTag = entry.type === 'agent' ? '[agent]' : '[exec]';
   const maxLabelW = Math.max(20, width - 30);
-  const title = `${typeTag} ${entry.label.slice(0, maxLabelW)}`;
-
-  // Build scroll indicator for text section header
+  const title = `[exec] ${entry.label.slice(0, maxLabelW)}`;
   const scrollInfo = totalLines > maxOutputLines
     ? `  Lines ${startIdx + 1}-${Math.min(endIdx, totalLines)} of ${totalLines}  [Up/Down] Scroll`
     : '';
 
-  const contentText = visibleLines.join('\n') || '(no output yet)';
-
   const sections: import('./modal-factory.ts').ModalSection[] = [];
-
   if (scrollInfo) {
     sections.push({ type: 'text', content: scrollInfo });
     sections.push({ type: 'separator' });
   }
-
-  sections.push({ type: 'text', content: contentText });
+  sections.push({ type: 'text', content: visibleLines.join('\n') || '(no output yet)' });
 
   return ModalFactory.createModal({
     title,
@@ -151,6 +92,6 @@ export function renderLiveTailModal(
     margin: 2,
     targetContentRows,
     sections,
-    hints: ['[Up/Down] Scroll', '[k] Stop exec only', '[Esc] Back'],
+    hints: ['[Up/Down] Scroll', '[k] Stop process', '[Esc] Back'],
   }, width);
 }