@pellux/goodvibes-tui 0.18.20 → 0.19.0

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/settings-modal-helpers.ts

@@ -78,6 +78,7 @@ export const CATEGORY_LABELS: Record<(typeof SETTINGS_CATEGORIES)[number], strin
   danger: 'Danger',
   tools: 'Tools',
   flags: 'Flags',
+  network: 'Network',
 };
 
 export const SETTING_LABELS: Partial<Record<string, string>> = {
@@ -102,6 +103,32 @@ export const SETTING_LABELS: Partial<Record<string, string>> = {
   'helper.enabled': 'Helper Enabled',
   'helper.globalProvider': 'Helper Provider',
   'helper.globalModel': 'Helper Model',
+  // Control Plane
+  'controlPlane.enabled': 'CP Enabled',
+  'controlPlane.hostMode': 'CP Host Mode',
+  'controlPlane.host': 'CP Host',
+  'controlPlane.port': 'CP Port',
+  'controlPlane.baseUrl': 'CP Base URL',
+  'controlPlane.streamMode': 'CP Stream Mode',
+  'controlPlane.allowRemote': 'CP Allow Remote',
+  'controlPlane.trustProxy': 'CP Trust Proxy',
+  'controlPlane.tls.mode': 'CP TLS Mode',
+  'controlPlane.tls.certFile': 'CP TLS Cert',
+  'controlPlane.tls.keyFile': 'CP TLS Key',
+  // HTTP Listener
+  'httpListener.hostMode': 'HTTP Host Mode',
+  'httpListener.host': 'HTTP Host',
+  'httpListener.port': 'HTTP Port',
+  'httpListener.trustProxy': 'HTTP Trust Proxy',
+  'httpListener.tls.mode': 'HTTP TLS Mode',
+  'httpListener.tls.certFile': 'HTTP TLS Cert',
+  // Web Server
+  'web.enabled': 'Web Enabled',
+  'web.hostMode': 'Web Host Mode',
+  'web.host': 'Web Host',
+  'web.port': 'Web Port',
+  'web.publicBaseUrl': 'Web Public Base URL',
+  'web.staticAssetsDir': 'Web Static Assets Dir',
 };
 
 export function getSettingLabel(entry: SettingEntry): string {

package/src/renderer/settings-modal.ts

@@ -63,6 +63,7 @@ export function renderSettingsModal(
   const isFlagsTab = SETTINGS_CATEGORIES[modal.categoryIndex] === 'flags';
   const isUiTab = SETTINGS_CATEGORIES[modal.categoryIndex] === 'ui';
   const isToolsTab = SETTINGS_CATEGORIES[modal.categoryIndex] === 'tools';
+  const isNetworkTab = SETTINGS_CATEGORIES[modal.categoryIndex] === 'network';
   let persistentHelpers: import('./modal-factory.ts').ModalHelperRow[] | undefined;
   sections.push({
     type: 'text',
@@ -78,12 +79,28 @@ export function renderSettingsModal(
             ? 'Feature flags control staged or experimental behavior. Some changes may require restart.'
             : isToolsTab
               ? 'Configure tool LLM routing and helper model. Provider and model fields are optional — empty means use the active provider.'
-              : 'Browse and adjust operator-facing runtime settings by category.',
+              : isNetworkTab
+                ? 'Configure control-plane and HTTP-listener binding. hostMode local/network use preset hosts; custom enables the host field. Changes trigger auto-restart.'
+                : 'Browse and adjust operator-facing runtime settings by category.',
     style: { fg: '246', dim: true },
   });
 
   sections.push({ type: 'separator' });
 
+  // ── Network tab restart notice ─────────────────────────────────
+  if (isNetworkTab && modal.lastSaveTriggeredRestart !== null) {
+    const restartTarget = modal.lastSaveTriggeredRestart === 'control-plane'
+      ? 'control-plane server'
+      : modal.lastSaveTriggeredRestart === 'http-listener'
+        ? 'HTTP listener'
+        : 'web server';
+    sections.push({
+      type: 'text',
+      content: truncateDisplay(`Restarting ${restartTarget}… server will reconnect momentarily.`, contentW),
+      style: { fg: '#38bdf8' },
+    });
+  }
+
   // ── Flags tab ──────────────────────────────────────────────────
   if (isFlagsTab) {
     const flagEntries: FlagEntry[] = modal.flagEntries;

package/src/renderer/status-glyphs.ts

@@ -0,0 +1,21 @@
+// ---------------------------------------------------------------------------
+// status-glyphs.ts — canonical glyph map for status states.
+//
+// Extracted as a neutral module so both status-token.ts and polish.ts can
+// import from here without creating a circular ESM dependency.
+//
+// Glyphs:
+//   good  ✓  (CHECK MARK U+2713)
+//   warn  ⚠  (WARNING SIGN U+26A0)
+//   bad   ✕  (MULTIPLICATION X U+2715)
+//   info  ○  (WHITE CIRCLE U+25CB)
+// ---------------------------------------------------------------------------
+
+export type StatusState = 'good' | 'warn' | 'bad' | 'info';
+
+export const STATE_GLYPHS: Record<StatusState, string> = {
+  good: '\u2713', // ✓
+  warn: '\u26a0', // ⚠
+  bad:  '\u2715', // ✕
+  info: '\u25cb', // ○
+};

package/src/renderer/status-token.ts

@@ -13,15 +13,11 @@
 
 import type { Cell } from '../types/grid.ts';
 import { DEFAULT_PANEL_PALETTE } from '../panels/polish.ts';
+import { type StatusState, STATE_GLYPHS } from './status-glyphs.ts';
 
-export type StatusState = 'good' | 'warn' | 'bad' | 'info';
-
-const STATE_GLYPHS: Record<StatusState, string> = {
-  good: '\u2713', // ✓
-  warn: '\u26a0', // ⚠
-  bad:  '\u2715', // ✕
-  info: '\u25cb', // ○
-};
+// Re-export for downstream consumers that import from this module.
+export type { StatusState } from './status-glyphs.ts';
+export { STATE_GLYPHS } from './status-glyphs.ts';
 
 const STATE_COLORS: Record<StatusState, string> = {
   good: DEFAULT_PANEL_PALETTE.good,

package/src/renderer/tool-call.ts

@@ -2,6 +2,7 @@ import { type Line, createStyledCell, createEmptyLine } from '../types/grid.ts';
 import { LAYOUT, TOOL_STATUS } from './layout.ts';
 import { getDisplayWidth, truncateDisplay } from '../utils/terminal-width.ts';
 import type { ToolCall } from '@pellux/goodvibes-sdk/platform/types/tools';
+import { stripDangerousAnsi } from './ansi-sanitize.ts';
 
 const TOOL_NAME_MIN_WIDTH = 8;
 const TOOL_NAME_MAX_WIDTH = 20;
@@ -194,11 +195,11 @@ export function renderToolCallBlock(
   const rawName = toolCall.name.includes('__')
     ? toolCall.name.split('__').pop()!
     : toolCall.name;
-  const keyArg = extractKeyArg(toolCall);
+  const keyArg = stripDangerousAnsi(extractKeyArg(toolCall));
   const suffixText = status === 'error' && errorMsg
-    ? `- ${errorMsg.slice(0, 40)}`
+    ? `- ${stripDangerousAnsi(errorMsg).slice(0, 40)}`
     : status === 'done' && resultSummary
-      ? `(${resultSummary})`
+      ? `(${stripDangerousAnsi(resultSummary)})`
       : '';
   const leftBudget = Math.max(0, leftEndExclusive - col);
   const leftSegments = buildLeftSegments(rawName, keyArg, suffixText, leftBudget);

package/src/runtime/bootstrap-core.ts

@@ -362,7 +362,7 @@ export async function initializeBootstrapCore(
       displayName: 'Terminal UI',
       lastSeenAt: Date.now(),
     },
-  }).catch(() => {});
+  }).catch((err) => { logger.debug('session broker create session failed at bootstrap', { err }); });
 
   domainDispatch.syncSessionState({
     id: userSessionId,

package/src/runtime/bootstrap-hook-bridge.ts

@@ -51,7 +51,7 @@ export function createResumeSessionHandler(options: ResumeSessionOptions): (sess
       if (meta?.model) options.runtime.model = meta.model;
       if (meta?.provider) options.runtime.provider = meta.provider;
       options.writeLastSessionPointer(sessionId);
-      void options.sharedSessionBroker.reopenSession(sessionId).catch(() => {});
+      void options.sharedSessionBroker.reopenSession(sessionId).catch((err) => { logger.debug('session broker reopen session failed', { err }); });
       options.conversation.log(`Resumed session: ${sessionId}`, { fg: '135' });
       const reopenedPanels: string[] = [];
       if (meta.returnContext?.openPanels?.length) {

package/src/runtime/bootstrap.ts

@@ -177,13 +177,13 @@ export async function bootstrapRuntime(
     requestRender: (): void => { requestRender(); },
   };
 
-  const orchestrator = new Orchestrator(
+  const orchestrator = new Orchestrator({
     conversation,
-    () => orchestratorRefs.getViewportHeight(),
-    (vHeight: number) => orchestratorRefs.scrollToEnd(vHeight),
+    getViewportHeight: () => orchestratorRefs.getViewportHeight(),
+    scrollToEnd: (vHeight: number) => orchestratorRefs.scrollToEnd(vHeight),
     toolRegistry,
     permissionManager,
-    () => {
+    getSystemPrompt: () => {
       const currentModel = providerRegistry.getCurrentModel();
       const contextWindow = providerRegistry.getContextWindowForModel(currentModel);
       const tier = getTierForContextWindow(contextWindow);
@@ -191,14 +191,13 @@ export async function bootstrapRuntime(
       return supplement ? runtime.systemPrompt + '\n\n' + supplement : runtime.systemPrompt;
     },
     hookDispatcher,
-    null,
-    () => orchestratorRefs.requestRender(),
+    requestRender: () => orchestratorRefs.requestRender(),
     runtimeBus,
-    {
+    services: {
       agentManager: services.agentManager,
       wrfcController: services.wrfcController,
     },
-  );
+  });
   conversationFollowUpRef.value = (item) => orchestrator.enqueueConversationFollowUp(item);
   orchestrator.setCoreServices({
     configManager,

package/src/runtime/diagnostics/panels/policy.ts

@@ -19,6 +19,7 @@ import type { PermissionAuditEntry } from '@pellux/goodvibes-sdk/platform/runtim
 import type { PolicyLintFinding } from '@pellux/goodvibes-sdk/platform/runtime/permissions/lint';
 import type { PolicySimulationSummary } from '@pellux/goodvibes-sdk/platform/runtime/permissions/simulation-scenarios';
 import type { PolicyPreflightReview } from '@pellux/goodvibes-sdk/platform/runtime/permissions/preflight';
+import { logger } from '@pellux/goodvibes-sdk/platform/utils/logger';
 
 /**
  * A point-in-time snapshot of policy state for diagnostics rendering.
@@ -169,7 +170,7 @@ export class PolicyPanel {
       try {
         cb();
       } catch (err) {
-        console.debug('[PolicyPanel] subscriber error:', err);
+        logger.warn(`[PolicyPanel] subscriber error: ${err}`);
       }
     }
   }

package/src/shell/ui-openers.ts

@@ -51,7 +51,7 @@ export function wireShellUiOpeners(options: WireShellUiOpenersOptions): void {
     void getPinned().then((pinned) => {
       input.modelPicker.pinnedIds = new Set(pinned);
     });
-    void input.modelPicker.loadRecentModels().catch(() => {});
+    void input.modelPicker.loadRecentModels().catch(() => {}); // best-effort: prefetch for UI, failure is non-visible
     input.modalOpened('modelPicker');
     input.modelPicker.openAllModels(models, runtime.model);
     render();

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.20';
+let _version = '0.19.0';
 try {
   const pkg = JSON.parse(readFileSync(join(import.meta.dir, '..', 'package.json'), 'utf-8'));
   _version = pkg.version ?? _version;