@pellux/goodvibes-tui 2.0.20 → 2.1.0

package/src/panels/types.ts

@@ -1,191 +0,0 @@
-import type { Line } from '@pellux/goodvibes-sdk/platform/types';
-import type { StatusState } from '../renderer/status-glyphs.ts';
-import type { ComponentResourceContract, ComponentHealthState } from '../runtime/perf/panel-contracts.ts';
-// Routed through the `@/` alias (not `./panel-manager.ts`) so this foundational
-// types module stays a leaf in the relative-import graph the architecture
-// cycle-checker walks. Type-only, erased at runtime, no real dependency edge.
-import type { PanelManager } from '@/panels/panel-manager.ts';
-
-/**
- * Context passed to a panel's `handlePanelIntegrationAction` hook so it can
- * drive cross-panel behavior (e.g. file-explorer opening the preview panel)
- * without the input layer needing `instanceof` knowledge of each panel type.
- */
-export interface PanelIntegrationContext {
-  readonly panelManager: PanelManager;
-  readonly executeCommand?: (name: string, args: string[]) => Promise<unknown>;
-}
-
-/**
- * A cross-panel deep-link target (item 4): a specific node a panel
- * should select + reveal when opened via `PanelManager.open(id, pane, target)`,
- * e.g. a Fleet tree node keyed by its process id and (optionally) its
- * ProcessKind for disambiguation. Panel-agnostic on purpose, only panels that
- * have a "node" concept implement `Panel.receiveDeepLink`.
- */
-export interface PanelDeepLinkTarget {
-  readonly id: string;
-  readonly kind?: string;
-}
-
-/**
- * Named logical key identifiers emitted by the input tokenizer.
- * These are the ONLY key names that will appear in `handleInput` calls;
- * the tokenizer never emits DOM/browser-style names like 'ArrowUp' or 'Enter'.
- *
- * Printable single-character input is passed through as-is; the `string & {}`
- * fallback preserves that handling while making named-key completions discoverable
- * in editors and keeping non-null single-character values compatible.
- */
-export type NamedKey =
-  | 'up' | 'down' | 'left' | 'right'
-  | 'home' | 'end' | 'pageup' | 'pagedown'
-  | 'insert' | 'delete' | 'backspace'
-  | 'enter' | 'return' | 'escape' | 'space' | 'tab'
-  | 'f1' | 'f2' | 'f3' | 'f4' | 'f5' | 'f6'
-  | 'f7' | 'f8' | 'f9' | 'f10' | 'f11' | 'f12';
-
-/**
- * The full key type accepted by `Panel.handleInput`.
- *
- * Named keys (arrow keys, modifiers, function keys) are represented by
- * lowercase `NamedKey` members. Single printable characters are passed
- * verbatim via the `string & {}` escape hatch so panels can handle 'j', 'k',
- * 'r', etc. without losing type safety on the named members.
- */
-export type KeyName = NamedKey | (string & {});
-
-/**
- * the former single 'monitoring' bucket held 33 panels pre-merge,
- * too coarse for the picker to be useful and too large for any one operator
- * mental model. Split into named operator domains so each category holds a
- * bounded, coherent set (no category may exceed 10 registrations):
- *   - providers:              provider/model connectivity, cost, and token usage
- *   - security-policy:        auth, policy governance, and isolation posture
- *   - automation-control:     hooks, plugins, marketplace, automation jobs, worktrees
- *   - incidents-diagnostics:  failure review, eval gates, API call debugging
- *   - runtime-ops:            live operator consoles (cockpit, tasks, orchestration, comms)
- */
-export type PanelCategory =
-  | 'development'
-  | 'agent'
-  | 'session'
-  | 'ai'
-  | 'providers'
-  | 'security-policy'
-  | 'automation-control'
-  | 'incidents-diagnostics'
-  | 'runtime-ops';
-
-export interface Panel {
-  id: string;
-  name: string;
-  icon: string; // single char for tab bar
-  category: PanelCategory;
-
-  // Lifecycle
-  onActivate(): void;
-  onDeactivate(): void;
-  onDestroy(): void;
-
-  // Rendering
-  render(width: number, height: number): Line[];
-
-  // State
-  isTransient: boolean;
-  isPinned: boolean;
-  needsRender: boolean;
-
-  // Dirty-flag contract (activated panel render skipping)
-  /** Mark this panel as needing a re-render on the next frame. */
-  invalidate(): void;
-  /** Called by the compositor after a successful render to clear the dirty flag. */
-  markRendered(): void;
-
-  // Resource contract (optional, panels may declare resource requirements)
-  resourceContract?: Readonly<ComponentResourceContract>;
-
-  // Health state (optional, set by ComponentHealthMonitor when panel is registered)
-  healthState?: Readonly<ComponentHealthState>;
-
-  // Input (optional)
-  handleInput?(key: KeyName): boolean;
-
-  /**
-   * Optional: report `true` while this panel has an active inline text
-   * capture (e.g. a `/`-to-filter or search buffer) that deliberately wants
-   * every character of a burst, paste, or several keystrokes typed fast
-   * enough to land in one `input.feed()` call, delivered to `handleInput`
-   * one at a time, same as it always has. When this returns `false` or is
-   * not implemented, the input router treats a printable burst as never a
-   * deliberate single-key panel hotkey and routes it to the composer
-   * instead of exploding it into per-char `handleInput` calls (see
-   * `handlePanelFocusToken` in `src/input/handler-feed-routes.ts`).
-   */
-  isCapturingTextBurst?(): boolean;
-
-  // Scroll input (optional)
-  // Positive delta scrolls down; negative delta scrolls up.
-  handleScroll?(deltaRows: number): boolean;
-
-  /**
-   * Cross-panel integration hook (optional). Called before the panel's own
-   * `handleInput` when a navigation/confirm key is pressed, so a panel can
-   * drive another panel (e.g. open a file in the preview panel). Return `true`
-   * to consume the key. The legacy `handlePanelIntegrationAction` router in
-   * `src/input/panel-integration-actions.ts` consults this first, then falls
-   * back to its built-in `instanceof` routing.
-   */
-  handlePanelIntegrationAction?(key: string, ctx: PanelIntegrationContext): boolean;
-
-  /**
-   * Optional: called by the global Ctrl+X (`panel-close`) shortcut BEFORE it
-   * closes this panel (handler-shortcuts.ts). Return `true` to consume
-   * Ctrl+X for an in-panel action instead, e.g. FleetPanel (session
-   * tabs) detaches its active tab and leaves the panel open. Return `false`
-   * (or omit this hook) to fall through to the ordinary close behavior.
-   * Ctrl+X never reaches a panel's own `handleInput` (ctrl/meta combos are
-   * intercepted earlier, see handlePanelFocusToken in
-   * src/input/handler-feed-routes.ts), so this is the one seam a panel has
-   * to intervene before the global shortcut acts.
-   */
-  interceptPanelClose?(): boolean;
-
-  /**
-   * Optional: consume a deep-link target handed in by `PanelManager.open(id,
-   * pane, target)` (item 4), select + reveal the matching node on the
-   * panel's next snapshot. A panel without a "node" concept simply doesn't
-   * implement this (the call is a no-op via optional chaining in
-   * PanelManager). A panel that DOES implement it owns its own honest
-   * "not found" surface (e.g. FleetPanel.setError) when the target no longer
-   * resolves, PanelManager has no transcript/print seam to report on its
-   * behalf.
-   */
-  receiveDeepLink?(target: PanelDeepLinkTarget): void;
-}
-
-export interface PanelRegistration extends Pick<Panel, 'id' | 'name' | 'icon' | 'category'> {
-  factory: () => Panel;
-  description: string;
-  /**
-   * lifecycle flags. `preload` and `retainOnClose` are independent,
-   * a panel can eagerly instantiate without being kept alive on close, or be
-   * kept alive on close without eager bootstrap instantiation. They happened
-   * to be identical for all builtin panels before this split (both driven by
-   * a single `preload: true`); that existing 10-panel retained set is
-   * preserved unchanged by giving each of those 10 registrations both flags.
-   *
-   * - `preload`, instantiate this panel during `PanelManager.prewarmRegistered()`
-   *   (called once at bootstrap) so its factory runs and it starts
-   *   accumulating background data (subscriptions, timers) before the user
-   *   ever opens the workspace. Panels without `preload` are instantiated
-   *   lazily on first `open()`.
-   * - `retainOnClose`, when this panel is closed, keep the live instance in
-   *   `PanelManager`'s retained-panel map (background subscriptions/timers
-   *   keep running) instead of calling `onDestroy()`. Reopening returns the
-   *   same instance with its accumulated state intact. Panels without
-   *   `retainOnClose` are destroyed on close and rebuilt fresh next open.
-   */
-  preload?: boolean;
-  retainOnClose?: boolean;
-}

package/src/panels/wrfc-panel-format.ts

@@ -1,133 +0,0 @@
-import { truncateDisplay } from '../utils/terminal-width.ts';
-import type { WrfcState, Constraint, ConstraintFinding } from '@pellux/goodvibes-sdk/platform/agents';
-import { DEFAULT_PANEL_PALETTE, extendPalette } from './polish.ts';
-
-// ---------------------------------------------------------------------------
-// Colour palette + formatting helpers for the WRFC panel.
-//
-// Extracted from wrfc-panel.ts to keep that module under the architecture
-// line-count cap. Leaf module (only polish + terminal-width + sdk types); the
-// panel re-exports the public helpers so ./wrfc-panel.ts stays their import site.
-// ---------------------------------------------------------------------------
-export const C = extendPalette(DEFAULT_PANEL_PALETTE, {
-  // WRFC state-machine colours (domain status -- no shared equivalent)
-  passed:     '#22c55e', // green
-  failed:     '#ef4444', // red
-  reviewing:  '#eab308', // yellow
-  engineering:'#22d3ee', // cyan
-  fixing:     '#f97316', // orange
-  pending:    '#6b7280', // grey
-  gating:     '#a78bfa', // violet
-  committing: '#38bdf8', // sky
-  integrating:'#818cf8', // indigo
-
-  // Issue-severity ramp (domain -- no shared equivalent)
-  issueCrit:  '#ef4444',
-  issueMaj:   '#f97316',
-  issueMin:   '#eab308',
-  issueSug:   '#6b7280',
-
-  // Selection + divider chrome with no shared equivalent
-  selected:   '#1e40af', // selection bg
-  selectedFg: '#f8fafc',
-  border:     '#334155',
-});
-
-// ---------------------------------------------------------------------------
-// Helpers
-// ---------------------------------------------------------------------------
-const SPARKLINE_CHARS = '._-:=+*#';
-
-export function sparkline(scores: number[], maxScore = 10): string {
-  if (scores.length === 0) return '';
-  return scores
-    .map(s => {
-      const ratio = Math.max(0, Math.min(1, s / maxScore));
-      const idx   = Math.round(ratio * (SPARKLINE_CHARS.length - 1));
-      return SPARKLINE_CHARS[idx];
-    })
-    .join('');
-}
-
-export function stateColor(state: WrfcState): string {
-  switch (state) {
-    case 'passed':          return C.passed;
-    case 'failed':          return C.failed;
-    case 'reviewing':       return C.reviewing;
-    case 'engineering':     return C.engineering;
-    case 'fixing':          return C.fixing;
-    case 'gating':
-    case 'awaiting_gates':  return C.gating;
-    case 'committing':      return C.committing;
-    case 'integrating':     return C.integrating;
-    default:                return C.pending;
-  }
-}
-
-export function stateLabel(state: WrfcState): string {
-  switch (state) {
-    case 'engineering':    return 'ENG';
-    case 'reviewing':      return 'REV';
-    case 'fixing':         return 'FIX';
-    case 'gating':         return 'GATE';
-    case 'awaiting_gates': return 'WAIT';
-    case 'committing':     return 'COMMIT';
-    case 'integrating':    return 'INTG';
-    case 'passed':         return 'PASS';
-    case 'failed':         return 'FAIL';
-    default:               return 'PEND';
-  }
-}
-
-export function issueColor(severity: string): string {
-  switch (severity) {
-    case 'critical': return C.issueCrit;
-    case 'major':    return C.issueMaj;
-    case 'minor':    return C.issueMin;
-    default:         return C.issueSug;
-  }
-}
-
-export function issuePrefix(severity: string): string {
-  switch (severity) {
-    case 'critical': return '[CRIT] ';
-    case 'major':    return '[MAJR] ';
-    case 'minor':    return '[MINR] ';
-    default:         return '[SUGG] ';
-  }
-}
-
-export function truncate(s: string, max: number): string {
-  return truncateDisplay(s, max);
-}
-
-// ---------------------------------------------------------------------------
-// Constraint helpers
-// ---------------------------------------------------------------------------
-
-/**
- * Returns display tag, foreground colour, and dim flag for a single constraint
- * based on whether a reviewer finding exists for it.
- */
-export function constraintStatusMarker(
-  constraint: Constraint,
-  findings: ConstraintFinding[] | undefined,
-): { tag: string; fg: string; dim: boolean } {
-  const finding = findings?.find(f => f.constraintId === constraint.id);
-  if (!finding) {
-    return { tag: '[UNV]', fg: C.dim, dim: true };
-  }
-  if (finding.satisfied) {
-    return { tag: '[SAT]', fg: C.good, dim: false };
-  }
-  // Unsatisfied, use severity to pick colour and tag text
-  const sev = finding.severity ?? 'major';
-  let sevTag: string;
-  let fg: string;
-  switch (sev) {
-    case 'critical': sevTag = '[UNS CRIT]';  fg = C.issueCrit; break;
-    case 'minor':    sevTag = '[UNS MINOR]'; fg = C.issueMin;  break;
-    default:         sevTag = '[UNS MAJOR]'; fg = C.issueMaj;  break;
-  }
-  return { tag: sevTag, fg, dim: false };
-}

package/src/renderer/diff-view.ts

@@ -1,134 +0,0 @@
-import { type Line, type Cell, createStyledCell } from '@pellux/goodvibes-sdk/platform/types';
-import { UIFactory } from './ui-factory.ts';
-import { getDisplayWidth, padDisplayEnd } from '../utils/terminal-width.ts';
-import { DIFF_TONES } from './ui-primitives.ts';
-
-/**
- * renderDiffView - Render a unified diff string as styled Line[].
- * '+' lines in green, '-' lines in red, '@@' hunks in blue (DIFF_TONES.hunk,
- * shared with diff-panel.ts and git-panel.ts's inline diff).
- */
-export function renderDiffView(diffText: string, width: number, filename?: string): Line[] {
-  const lines: Line[] = [];
-  const BG = '#0a0a0a';
-
-  // Filename header
-  if (filename) {
-    const header = ` ≡ ${filename} `;
-    lines.push(UIFactory.stringToLine(padDisplayEnd(header, width), width, { fg: '#1a1a1a', bg: '#569cd6', bold: true }));
-  }
-
-  const diffLines = diffText.split('\n');
-  let oldLineNo = 0;
-  let newLineNo = 0;
-
-  for (const raw of diffLines) {
-    if (raw === '') {
-      const emptyLine = makeFilledLine(width, BG);
-      lines.push(emptyLine);
-      continue;
-    }
-
-    // Hunk header: @@ -old,count +new,count @@
-    if (raw.startsWith('@@')) {
-      const hunkMatch = raw.match(/@@ -(\d+)(?:,\d+)? \+(\d+)(?:,\d+)? @@/);
-      if (hunkMatch) {
-        oldLineNo = parseInt(hunkMatch[1], 10) - 1;
-        newLineNo = parseInt(hunkMatch[2], 10) - 1;
-      }
-      lines.push(makeStyledLine(raw, width, DIFF_TONES.hunk, '#0f1f1f', false));
-      continue;
-    }
-
-    // File headers: --- and +++
-    if (raw.startsWith('--- ') || raw.startsWith('+++ ')) {
-      lines.push(makeStyledLine(raw, width, '244', BG, false));
-      continue;
-    }
-
-    // Added line
-    if (raw.startsWith('+')) {
-      newLineNo++;
-      const lineLabel = `${String(newLineNo).padStart(4)} `;
-      const content = raw.slice(1);
-      lines.push(makeGutterLine('+', lineLabel, content, width, DIFF_TONES.add, '#0a1a0a'));
-      continue;
-    }
-
-    // Removed line
-    if (raw.startsWith('-')) {
-      oldLineNo++;
-      const lineLabel = `${String(oldLineNo).padStart(4)} `;
-      const content = raw.slice(1);
-      lines.push(makeGutterLine('-', lineLabel, content, width, DIFF_TONES.del, '#1a0a0a'));
-      continue;
-    }
-
-    // Context line
-    if (raw.startsWith(' ') || (!raw.startsWith('\\') && raw.length > 0)) {
-      oldLineNo++;
-      newLineNo++;
-      const lineLabel = `${String(oldLineNo).padStart(4)} `;
-      const content = raw.startsWith(' ') ? raw.slice(1) : raw;
-      lines.push(makeGutterLine(' ', lineLabel, content, width, '244', BG));
-    }
-  }
-
-  return lines;
-}
-
-/** Build a line with gutter indicator, line number, and content. */
-function makeGutterLine(
-  gutter: string,
-  lineLabel: string,
-  content: string,
-  width: number,
-  fg: string,
-  bg: string
-): Line {
-  const line = makeFilledLine(width, bg);
-  let cx = 0;
-
-  // Gutter char
-  line[cx++] = createStyledCell(gutter, { fg, bg, bold: gutter !== ' ' });
-
-  // Line number
-  for (const ch of lineLabel) {
-    if (cx >= width) break;
-    line[cx++] = createStyledCell(ch, { fg: '238', bg, dim: true });
-  }
-
-  // Content
-  for (const ch of content) {
-    if (cx >= width) break;
-    const cw = getDisplayWidth(ch);
-    const code = ch.charCodeAt(0);
-    if (code < 32) { cx++; continue; }
-    line[cx] = createStyledCell(ch, { fg, bg });
-    if (cw === 2 && cx + 1 < width) line[cx + 1] = { ...line[cx], char: '' };
-    cx += cw;
-  }
-
-  return line;
-}
-
-/** Build a simple styled line from text. */
-function makeStyledLine(text: string, width: number, fg: string, bg: string, bold: boolean): Line {
-  const line = makeFilledLine(width, bg);
-  let cx = 0;
-  for (const ch of text) {
-    if (cx >= width) break;
-    const cw = getDisplayWidth(ch);
-    const code = ch.charCodeAt(0);
-    if (code < 32) { cx++; continue; }
-    line[cx] = createStyledCell(ch, { fg, bg, bold });
-    if (cw === 2 && cx + 1 < width) line[cx + 1] = { ...line[cx], char: '' };
-    cx += cw;
-  }
-  return line;
-}
-
-/** Create a line filled with bg color. */
-function makeFilledLine(width: number, bg: string): Cell[] {
-  return Array.from({ length: width }, () => createStyledCell(' ', { bg }));
-}

package/src/renderer/fleet-tab-strip.ts

@@ -1,56 +0,0 @@
-// ---------------------------------------------------------------------------
-// fleet-tab-strip.ts
-//
-// Renders FleetPanel's session-tab strip by reusing
-// renderTabStrip exactly as panel-workspace-bar.ts does for the workspace
-// tab bar. Deliberately a SEPARATE, visually distinct strip from the
-// workspace bar: workspace tabs switch PANELS, fleet session tabs switch
-// ATTACHED PROCESSES within one panel, conflating their styling would blur
-// that distinction for the operator.
-// ---------------------------------------------------------------------------
-
-import type { Line } from '@pellux/goodvibes-sdk/platform/types';
-import type { FleetTabsState } from '../panels/fleet-tabs.ts';
-import { renderTabStrip, type TabHitRegion } from './tab-strip.ts';
-import { UI_TONES } from './ui-primitives.ts';
-
-const ACTIVE_FG = UI_TONES.fg.primary;
-const ACTIVE_BG = UI_TONES.bg.selected;
-const INACTIVE_FG = UI_TONES.fg.muted;
-const SEPARATOR_FG = UI_TONES.fg.dim;
-const LABEL_FG = UI_TONES.fg.secondary;
-
-/**
- * Render the fleet session-tab strip, or `null` when there are no attached
- * tabs, the panel omits the strip entirely in that case (root-tab-only),
- * which is what keeps the pre-session-tab fleet-panel goldens byte-identical.
- */
-export function renderFleetTabStrip(
-  state: FleetTabsState,
-  width: number,
-  onLayout?: (regions: readonly TabHitRegion[]) => void,
-): Line | null {
-  if (state.tabs.length === 0) return null;
-  return renderTabStrip({
-    width,
-    onLayout,
-    tabs: [
-      { label: 'Tree', active: state.activeTabIndex === 0 },
-      ...state.tabs.map((tab, index) => ({
-        label: tab.label,
-        active: state.activeTabIndex === index + 1,
-      })),
-    ],
-    prefixLabel: ' SESSIONS ',
-    style: {
-      activeFg: ACTIVE_FG,
-      activeBg: ACTIVE_BG,
-      activeBold: true,
-      inactiveFg: INACTIVE_FG,
-      separatorFg: SEPARATOR_FG,
-      labelFg: LABEL_FG,
-      overflowFg: SEPARATOR_FG,
-      trailingFg: INACTIVE_FG,
-    },
-  });
-}

package/src/renderer/footer-tips.ts

@@ -1,53 +0,0 @@
-/**
- * footer-tips.ts, the persistent discoverability hint shown in the shell
- * footer's tip slot.
- *
- * instead of a single frozen "/help for commands" line, the footer
- * surfaces a rotating set of the highest-value affordances (panels, process
- * monitor, help, quit). Rotation is *contextual, not timed*: when an agent turn
- * is actively running, the process-monitor tip is promoted to the front so the
- * operator can jump to F2 while work is in flight. Selection is a pure function
- * of context so the footer renders deterministically (golden frames stay
- * stable), no wall-clock input.
- */
-
-export interface FooterTipContext {
-  /** True while an agent turn is actively running (streaming / tools / hooks). */
-  readonly agentActive: boolean;
-}
-
-/** Composer status labels that mean an agent turn is actively in flight. */
-const ACTIVE_TURN_STATUSES = new Set(['preflight', 'streaming', 'tools', 'post-hooks']);
-
-/** Derive agent-active state from the composer status label passed to the footer. */
-export function isAgentActive(composerStatus: string | undefined): boolean {
-  return composerStatus !== undefined && ACTIVE_TURN_STATUSES.has(composerStatus);
-}
-
-// Ctrl+P is a TOGGLE (open+focus when nothing is open or unfocused,
-// hide when the workspace already has focus, see openPanelPicker in
-// shell/ui-openers.ts). The bare noun 'panels' undersold that; naming the verb
-// keeps the tip honest about what the chord actually does.
-const TIP_PANELS = 'Ctrl+P toggle panels';
-// F2 now opens the Fleet panel (the process modal was retired), so the
-// tip names 'fleet', not 'processes'.
-const TIP_PROCESSES = 'F2 fleet';
-const TIP_HELP = '? help';
-// f: an empty-composer Ctrl+C does NOT quit on the first press, it arms a
-// ~1s "press again to exit" confirm, and only a SECOND Ctrl+C within that window
-// exits (an intentional accidental-exit guard). The old 'Ctrl+C quit' tip
-// implied a single press quits, so two presses seconds apart appeared to do
-// nothing. 'Ctrl+C x2 quit' matches the real double-press contract (and the
-// help overlay's "Ctrl+C x2  Exit" row).
-const TIP_QUIT = 'Ctrl+C x2 quit';
-
-/**
- * Build the footer discoverability tip. Segments join with the shared middle
- * dot. Agent-active state promotes the process-monitor tip to the front.
- */
-export function buildFooterTip(ctx: FooterTipContext): string {
-  const lead = ctx.agentActive
-    ? [TIP_PROCESSES, TIP_PANELS, TIP_HELP]
-    : [TIP_PANELS, TIP_PROCESSES, TIP_HELP];
-  return [...lead, TIP_QUIT].join(' · ');
-}