@pellux/goodvibes-tui 0.18.19 → 0.18.23

package/src/renderer/buffer.ts

@@ -2,17 +2,23 @@ import { type Line, type Cell, createEmptyLine, createEmptyCell } from '../types
 
 /**
  * TerminalBuffer - Represents a 2D grid of styled cells.
+ * Tracks a per-row dirty bitmap so the diff engine can skip rows that were
+ * never written in the current frame.
  */
 export class TerminalBuffer {
   public cells: Line[];
+  /** dirtyRows[y] is true if row y was written since the last reset(). */
+  public dirtyRows: boolean[];
 
   constructor(public width: number, public height: number) {
     this.cells = Array.from({ length: height }, () => createEmptyLine(width));
+    this.dirtyRows = new Array(height).fill(false);
   }
 
   public setCell(x: number, y: number, cell: Partial<Cell>): void {
     if (y >= 0 && y < this.height && x >= 0 && x < this.width) {
       this.cells[y][x] = { ...this.cells[y][x], ...cell };
+      this.dirtyRows[y] = true;
     }
   }
 
@@ -23,30 +29,35 @@ export class TerminalBuffer {
   public blitLine(row: number, line: Line): void {
     if (row >= 0 && row < this.height) {
       this.cells[row] = [...line];
+      this.dirtyRows[row] = true;
     }
   }
 
   public clone(): TerminalBuffer {
     const newBuf = new TerminalBuffer(this.width, this.height);
     newBuf.cells = this.cells.map(line => line.map(cell => ({ ...cell })));
+    newBuf.dirtyRows = [...this.dirtyRows];
     return newBuf;
   }
 
   /**
    * Reset all cells in-place to empty, reusing this buffer instance.
    * If dimensions changed, reallocates cells array.
+   * Always clears the dirty bitmap.
    */
   public reset(width: number, height: number): void {
     if (width !== this.width || height !== this.height) {
       this.width = width;
       this.height = height;
       this.cells = Array.from({ length: height }, () => createEmptyLine(width));
+      this.dirtyRows = new Array(height).fill(false);
     } else {
       for (let y = 0; y < this.height; y++) {
         const row = this.cells[y]!;
         for (let x = 0; x < this.width; x++) {
           row[x] = createEmptyCell();
         }
+        this.dirtyRows[y] = false;
       }
     }
   }

package/src/renderer/diff.ts

@@ -29,6 +29,14 @@ export class DiffEngine {
     let output = '';
     
     for (let y = 0; y < newBuffer.height; y++) {
+      // Skip rows that were not written in either the old or new buffer.
+      // If neither side touched the row, both must match the prior frame:
+      // old row was never written this frame (clean) and new row is also
+      // clean, so the on-screen content is still correct. No diff needed.
+      const newDirty = newBuffer.dirtyRows[y] ?? false;
+      const oldDirty = oldBuffer ? (oldBuffer.dirtyRows[y] ?? false) : true;
+      if (!newDirty && !oldDirty) continue;
+
       for (let x = 0; x < newBuffer.width; x++) {
         const oldCell = oldBuffer?.getCell(x, y);
         const newCell = newBuffer.cells[y]?.[x];

package/src/renderer/help-overlay.ts

@@ -64,36 +64,45 @@ export function renderHelpOverlay(
     '',
   ];
 
-  const commandRows: string[] = [
-    '  Quick Start',
-    '  ' + '\u2500'.repeat(40),
-    '  /setup onboarding     Guided first-run review and environment posture',
-    '  /cockpit              Unified runtime control room',
-    '  /settings             Settings and config browser',
-    '',
-    '  Build And Operate',
-    '  ' + '\u2500'.repeat(40),
-    '  /provider             Choose provider or model family',
-    '  /subscription         Review provider logins and subscriptions',
-    '  /marketplace open     Browse plugins, skills, and packs',
-    '  /remote setup         Review remote, bridge, and tunnel flows',
-    '  /sandbox review       Inspect secure execution posture',
-    '',
-    '  Review And Govern',
-    '  ' + '\u2500'.repeat(40),
-    '  /security             Security review workspace',
-    '  /policy               Simulation, lint, and preflight review',
-    '  /incident             Incident workspace and export flows',
-    '  /knowledge            Durable knowledge and review queue',
-    '',
-    '  Power Surfaces',
-    '  ' + '\u2500'.repeat(40),
-    '  /hooks                Hook workbench and runtime activity',
-    '  /orchestration        Graph and recursive-agent control room',
-    '  /communication        Structured agent communication workspace',
-    '  /tasks                Task surface for list/show/pause/resume/output',
+  // Featured commands shown in the Quick Start section.
+  // Each entry is [commandName, subcommandOrArgHint, description].
+  // Commands not registered in the live registry are omitted at render time.
+  const FEATURED_COMMANDS: Array<[name: string, argHint: string, desc: string]> = [
+    ['setup',        'onboarding', 'Guided first-run review and environment posture'],
+    ['cockpit',      '',           'Unified runtime control room'],
+    ['settings',     '',           'Settings and config browser'],
+    ['provider',     '',           'Choose provider or model family'],
+    ['subscription', '',           'Review provider logins and subscriptions'],
+    ['marketplace',  'open',       'Browse plugins, skills, and packs'],
+    ['remote',       'setup',      'Review remote, bridge, and tunnel flows'],
+    ['sandbox',      'review',     'Inspect secure execution posture'],
+    ['security',     '',           'Security review workspace'],
+    ['policy',       '',           'Simulation, lint, and preflight review'],
+    ['incident',     '',           'Incident workspace and export flows'],
+    ['knowledge',    '',           'Durable knowledge and review queue'],
+    ['hooks',        '',           'Hook workbench and runtime activity'],
+    ['orchestration','',           'Graph and recursive-agent control room'],
+    ['communication','',           'Structured agent communication workspace'],
+    ['tasks',        '',           'Task surface for list/show/pause/resume/output'],
   ];
 
+  // Build command rows from featured list, filtering out unregistered commands.
+  function featuredRow(name: string, argHint: string, desc: string): string {
+    const invocation = argHint ? `/${name} ${argHint}` : `/${name}`;
+    return `  ${invocation.padEnd(23)}  ${desc}`;
+  }
+
+  const quickStartRows: string[] = [];
+  for (const [name, argHint, desc] of FEATURED_COMMANDS) {
+    if (!hasCommand(name)) continue; // omit if not in live registry
+    quickStartRows.push(featuredRow(name, argHint, desc));
+  }
+
+  const commandRows: string[] = [];
+  if (quickStartRows.length > 0) {
+    commandRows.push('  Quick Start', '  ' + '\u2500'.repeat(40), ...quickStartRows, '');
+  }
+
   if (commands && commands.length > 0) {
     commandRows.push('', '  Available Slash Commands', '  ' + '\u2500'.repeat(40));
     const preferred = ['setup', 'cockpit', 'settings', 'provider', 'subscription', 'marketplace', 'remote', 'sandbox', 'security', 'policy', 'incident', 'knowledge', 'hooks', 'orchestration', 'communication', 'tasks'];

package/src/renderer/markdown.ts

@@ -30,153 +30,11 @@ function isLikelyTableHeaderRow(row: string): boolean {
 }
 
 /**
- * renderMarkdown - Parse markdown text into styled Line[] using a line-by-line state machine.
- * Supports headers, bold, italic, inline code, code blocks, lists, and links.
+ * renderMarkdown - Parse markdown text into styled Line[].
+ * Thin wrapper over renderMarkdownTracked for callers that don't need code-block metadata.
  */
 export function renderMarkdown(text: string, width: number, options: MarkdownRenderOptions = {}): Line[] {
-  const lines: Line[] = [];
-  const rawLines = text.split('\n');
-
-  let inCodeBlock = false;
-  let codeBlockLang = '';
-  let codeBlockLines: string[] = [];
-  const indent = LAYOUT.LEFT_MARGIN;
-  const contentWidth = LAYOUT.contentWidth(width);
-
-  for (let i = 0; i < rawLines.length; i++) {
-    const raw = rawLines[i];
-
-    // --- Code block fence ---
-    const fenceMatch = raw.match(/^```(\w*)/);
-    if (fenceMatch && !inCodeBlock) {
-      inCodeBlock = true;
-      codeBlockLang = fenceMatch[1] || '';
-      codeBlockLines = [];
-      continue;
-    }
-    if (inCodeBlock) {
-      if (raw.trimStart().startsWith('```')) {
-        // End of code block - delegate to code block renderer
-        const rendered = renderCodeBlock(codeBlockLines, codeBlockLang, width, {
-          showLineNumbers: options.codeBlockLineNumbers ?? true,
-        });
-        lines.push(...rendered);
-        inCodeBlock = false;
-        codeBlockLang = '';
-        codeBlockLines = [];
-      } else {
-        codeBlockLines.push(raw);
-      }
-      continue;
-    }
-
-    // --- Empty line ---
-    if (raw.trim() === '') {
-      lines.push(UIFactory.stringToLine('', width));
-      continue;
-    }
-
-    // --- Heading ---
-    const h3 = raw.match(/^### (.+)/);
-    const h2 = raw.match(/^## (.+)/);
-    const h1 = raw.match(/^# (.+)/);
-    if (h1) {
-      lines.push(UIFactory.stringToLine(' '.repeat(indent) + h1[1].toUpperCase(), width, { fg: '#00ffff', bold: true }));
-      lines.push(UIFactory.stringToLine(' '.repeat(indent) + '━'.repeat(Math.min(getDisplayWidth(h1[1]), contentWidth)), width, { fg: '244' }));
-      continue;
-    }
-    if (h2) {
-      lines.push(UIFactory.stringToLine(' '.repeat(indent) + h2[1], width, { fg: '#00ffff', bold: true }));
-      lines.push(UIFactory.stringToLine(' '.repeat(indent) + '─'.repeat(Math.min(getDisplayWidth(h2[1]), contentWidth)), width, { fg: '240' }));
-      continue;
-    }
-    if (h3) {
-      lines.push(UIFactory.stringToLine(' '.repeat(indent) + h3[1], width, { fg: '111', bold: true }));
-      continue;
-    }
-
-    // --- Task list ---
-    const taskMatch = raw.match(/^(\s*)[-*] \[([ xX])\] (.+)/);
-    if (taskMatch) {
-      const listIndent = Math.floor(taskMatch[1].length / 2);
-      const checked = taskMatch[2] !== ' ';
-      const bulletX = indent + listIndent * 2;
-      const textStartX = bulletX + 4;
-      const checkbox = checked ? '\u2611 ' : '\u2610 '; // ☑ or ☐
-      const rendered = renderInlineMarkdown(taskMatch[3]);
-      const prefix = ' '.repeat(bulletX) + checkbox;
-      const style = checked ? { fg: '244', strikethrough: true } : {};
-      lines.push(...compositeInlineLine(prefix, rendered, width, { fg: checked ? '#22c55e' : '252', ...style }, textStartX));
-      continue;
-    }
-
-    // --- Unordered list ---
-    const ulMatch = raw.match(/^(\s*)[-*] (.+)/);
-    if (ulMatch) {
-      const listIndent = Math.floor(ulMatch[1].length / 2);
-      const bulletX = indent + listIndent * 2;
-      const textStartX = bulletX + 2;
-      const rendered = renderInlineMarkdown(ulMatch[2]);
-      const prefix = ' '.repeat(bulletX) + '• ';
-      lines.push(...compositeInlineLine(prefix, rendered, width, { fg: '135', bold: false }, textStartX));
-      continue;
-    }
-
-    // --- Ordered list ---
-    const olMatch = raw.match(/^(\s*)(\d+)\. (.+)/);
-    if (olMatch) {
-      const listIndent = Math.floor(olMatch[1].length / 2);
-      const numStr = olMatch[2] + '. ';
-      const bulletX = indent + listIndent * 2;
-      const textStartX = bulletX + numStr.length;
-      const rendered = renderInlineMarkdown(olMatch[3]);
-      const prefix = ' '.repeat(bulletX) + numStr;
-      lines.push(...compositeInlineLine(prefix, rendered, width, { fg: '135', bold: false }, textStartX));
-      continue;
-    }
-
-    // --- Horizontal rule ---
-    if (/^[-*_]{3,}$/.test(raw.trim())) {
-      lines.push(UIFactory.stringToLine(' '.repeat(indent) + '─'.repeat(contentWidth), width, { fg: '240' }));
-      continue;
-    }
-
-    // --- Blockquote ---
-    const bqMatch = raw.match(/^> (.*)/);
-    if (bqMatch) {
-      const rendered = renderInlineMarkdown(bqMatch[1]);
-      const prefix = ' '.repeat(indent) + '┃ ';
-      lines.push(...compositeInlineLine(prefix, rendered, width, { fg: '244', italic: true }, indent + 3));
-      continue;
-    }
-
-    // --- Table ---
-    if (raw.includes('|') && i + 1 < rawLines.length && isLikelyTableHeaderRow(raw) && isLikelyTableSeparatorRow(rawLines[i + 1])) {
-      const tableRows: string[] = [];
-      let j = i;
-      while (j < rawLines.length && rawLines[j].includes('|')) {
-        tableRows.push(rawLines[j]);
-        j++;
-      }
-      i = j - 1;
-      lines.push(...renderTable(tableRows, width, indent));
-      continue;
-    }
-
-    // --- Normal paragraph ---
-    const rendered = renderInlineMarkdown(raw);
-    lines.push(...compositeInlineLine(' '.repeat(indent), rendered, width, {}, indent));
-  }
-
-  // Handle unclosed code block
-  if (inCodeBlock && codeBlockLines.length > 0) {
-    const rendered = renderCodeBlock(codeBlockLines, codeBlockLang, width, {
-      showLineNumbers: options.codeBlockLineNumbers ?? true,
-    });
-    lines.push(...rendered);
-  }
-
-  return lines;
+  return renderMarkdownTracked(text, width, options).lines;
 }
 
 export interface CodeBlockSpan {

package/src/renderer/status-token.ts

@@ -0,0 +1,71 @@
+// ---------------------------------------------------------------------------
+// buildStatusToken — always glyph + color, never color-only.
+//
+// Maps a semantic state to a Unicode glyph AND a palette color so that
+// colorblind users and screen readers can distinguish states without color.
+//
+// Glyphs:
+//   good  ✓  (CHECK MARK)
+//   warn  ⚠  (WARNING SIGN)
+//   bad   ✕  (MULTIPLICATION X)
+//   info  ○  (WHITE CIRCLE)
+// ---------------------------------------------------------------------------
+
+import type { Cell } from '../types/grid.ts';
+import { DEFAULT_PANEL_PALETTE } from '../panels/polish.ts';
+
+export type StatusState = 'good' | 'warn' | 'bad' | 'info';
+
+const STATE_GLYPHS: Record<StatusState, string> = {
+  good: '\u2713', // ✓
+  warn: '\u26a0', // ⚠
+  bad:  '\u2715', // ✕
+  info: '\u25cb', // ○
+};
+
+const STATE_COLORS: Record<StatusState, string> = {
+  good: DEFAULT_PANEL_PALETTE.good,
+  warn: DEFAULT_PANEL_PALETTE.warn,
+  bad:  DEFAULT_PANEL_PALETTE.bad,
+  info: DEFAULT_PANEL_PALETTE.info,
+};
+
+export interface StatusTokenOpts {
+  /** Append a numeric count after the label, e.g. "label (3)". */
+  count?: number;
+  /** Override the default glyph for this state. */
+  glyph?: string;
+}
+
+/**
+ * Build a small sequence of styled cells: [glyph, space, label, optional count]
+ *
+ * Always prepends the glyph so colorblind users can parse the state.
+ * The color is applied to both glyph and label as a redundant cue.
+ *
+ * @param state   Semantic state — controls default glyph and color.
+ * @param label   Human-readable label string.
+ * @param opts    Optional count suffix or glyph override.
+ * @returns       Array of Cell objects ready to embed in a Line.
+ */
+export function buildStatusToken(
+  state: StatusState,
+  label: string,
+  opts?: StatusTokenOpts,
+): Cell[] {
+  const glyph = opts?.glyph ?? STATE_GLYPHS[state];
+  const color = STATE_COLORS[state];
+  const suffix = opts?.count !== undefined ? ` (${opts.count})` : '';
+  const text = `${glyph} ${label}${suffix}`;
+
+  return text.split('').map((char): Cell => ({
+    char,
+    fg: color,
+    bg: '',
+    bold: false,
+    dim: false,
+    underline: false,
+    italic: false,
+    strikethrough: false,
+  }));
+}

package/src/version.ts

@@ -6,7 +6,7 @@ import { join } from 'node:path';
 // The prebuild script updates the fallback value before compilation.
 // Uses import.meta.dir (Bun) to locate package.json relative to this file,
 // which is correct regardless of the process working directory.
-let _version = '0.18.19';
+let _version = '0.18.23';
 try {
   const pkg = JSON.parse(readFileSync(join(import.meta.dir, '..', 'package.json'), 'utf-8'));
   _version = pkg.version ?? _version;