pi-blackhole 0.4.2 → 0.4.3

package/src/om/configure-overlay.ts

@@ -17,19 +17,19 @@ import { dirname } from "node:path";
 // ---------------------------------------------------------------------------
 
 interface FieldDef {
-	key: string;
-	label: string;
-	type: "number" | "boolean" | "enum";
-	section: string;
-	enumValues?: string[];
-	helpText?: string;
+  key: string;
+  label: string;
+  type: "number" | "boolean" | "enum";
+  section: string;
+  enumValues?: string[];
+  helpText?: string;
 }
 
 interface FieldState {
-	def: FieldDef;
-	value: string;
-	editing: boolean;
-	cursor: number;
+  def: FieldDef;
+  value: string;
+  editing: boolean;
+  cursor: number;
 }
 
 // ---------------------------------------------------------------------------
@@ -37,51 +37,169 @@ interface FieldState {
 // ---------------------------------------------------------------------------
 
 const FIELDS: FieldDef[] = [
-	// ── Compaction ──
-	{ key: "compaction", label: "Compaction mode", type: "enum", section: "Compaction", enumValues: ["auto", "manual", "off"],
-		helpText: "auto=trigger on threshold, manual=only /blackhole, off=auto:Pi handles, /blackhole:blackhole pipeline" },
-	{ key: "compactionEngine", label: "Compaction engine", type: "enum", section: "Compaction", enumValues: ["blackhole", "pi-default"],
-		helpText: "blackhole=structured summary+OM, pi-default=built-in Pi summarization" },
-	{ key: "tailBehavior", label: "Visible tail", type: "enum", section: "Compaction", enumValues: ["minimal", "pi-default"],
-		helpText: "minimal=keep last user message only (default), pi-default=keep Pi's preserved visible context" },
-	{ key: "midRunCompaction", label: "Mid-run compaction", type: "enum", section: "Compaction", enumValues: ["resume", "pause", "off"],
-		helpText: "resume=compact at threshold during tool loops and continue (default), pause=compact and stop, off=only check when run ends" },
-	{ key: "compactAfterTokens", label: "Auto-compact threshold", type: "number", section: "Compaction",
-		helpText: "Token count that triggers auto-compaction when reached" },
-
-	// ── Observational Memory ──
-	{ key: "memory", label: "Observational memory", type: "boolean", section: "Observational Memory",
-		helpText: "Enable OM workers (observer, reflector, dropper) and content injection" },
-	{ key: "sessionFallback", label: "Session model fallback", type: "boolean", section: "Observational Memory",
-		helpText: "off=skip stage when all OM models fail, instead of falling back to the main coding model" },
-	{ key: "observeAfterTokens", label: "Observer threshold", type: "number", section: "Observational Memory",
-		helpText: "Tokens accumulated since last observer run before triggering next observe" },
-	{ key: "reflectAfterTokens", label: "Reflect + dropper threshold", type: "number", section: "Observational Memory",
-		helpText: "Tokens accumulated since last reflect before triggering reflector and dropper" },
-	{ key: "observationsPoolMaxTokens", label: "Observation pool max", type: "number", section: "Observational Memory",
-		helpText: "Max tokens in observation pool before dropper prunes (fold pressure)" },
-	{ key: "observationsPoolTargetTokens", label: "Observation pool target", type: "number", section: "Observational Memory",
-		helpText: "Target tokens after dropper prunes (defaults to half of pool max)" },
-	{ key: "reflectorInputMaxTokens", label: "Reflector input max", type: "number", section: "Observational Memory",
-		helpText: "Max prompt tokens for reflector model input (rolling window cap)" },
-	{ key: "dropperInputMaxTokens", label: "Dropper input max", type: "number", section: "Observational Memory",
-		helpText: "Max prompt tokens for dropper model input (rolling window cap)" },
-	{ key: "observerChunkMaxTokens", label: "Observer chunk max", type: "number", section: "Observational Memory",
-		helpText: "Max source entry tokens sent to observer per chunk" },
-	{ key: "observerPreambleMaxTokens", label: "Observer preamble max", type: "number", section: "Observational Memory",
-		helpText: "Preamble budget in manual compaction mode (0=auto-compute 30% of chunk)" },
-	{ key: "dropperPressureThreshold", label: "Dropper pressure threshold", type: "number", section: "Observational Memory",
-		helpText: "Fraction of reflectorInputMaxTokens that triggers pressure-driven dropper (0-1, default 0.70)" },
-	{ key: "agentMaxTurns", label: "Max turns per agent", type: "number", section: "Observational Memory",
-		helpText: "Shared turn cap for background memory agents" },
-	{ key: "fullFoldAlways", label: "Preserve OM on first compaction", type: "boolean", section: "Observational Memory",
-		helpText: "When true, early reflections/drops survive the first compaction in a fresh session" },
-
-	// ── Debug ──
-	{ key: "debug", label: "Debug snapshots", type: "boolean", section: "Debug",
-		helpText: "Write detailed debug snapshots to /tmp/pi-blackhole-debug.json" },
-	{ key: "debugLog", label: "Debug JSONL logging", type: "boolean", section: "Debug",
-		helpText: "Write structured JSONL debug logs to agent directory" },
+  // ── Compaction ──
+  {
+    key: "compaction",
+    label: "Compaction mode",
+    type: "enum",
+    section: "Compaction",
+    enumValues: ["auto", "manual", "off"],
+    helpText:
+      "auto=trigger on threshold, manual=only /blackhole, off=auto:Pi handles, /blackhole:blackhole pipeline",
+  },
+  {
+    key: "compactionEngine",
+    label: "Compaction engine",
+    type: "enum",
+    section: "Compaction",
+    enumValues: ["blackhole", "pi-default"],
+    helpText:
+      "blackhole=structured summary+OM, pi-default=built-in Pi summarization",
+  },
+  {
+    key: "tailBehavior",
+    label: "Visible tail",
+    type: "enum",
+    section: "Compaction",
+    enumValues: ["minimal", "pi-default"],
+    helpText:
+      "minimal=keep last user message only (default), pi-default=keep Pi's preserved visible context",
+  },
+  {
+    key: "midRunCompaction",
+    label: "Mid-run compaction",
+    type: "enum",
+    section: "Compaction",
+    enumValues: ["resume", "pause", "off"],
+    helpText:
+      "resume=compact at threshold during tool loops and continue (default), pause=compact and stop, off=only check when run ends",
+  },
+  {
+    key: "compactAfterTokens",
+    label: "Auto-compact threshold",
+    type: "number",
+    section: "Compaction",
+    helpText: "Token count that triggers auto-compaction when reached",
+  },
+
+  // ── Observational Memory ──
+  {
+    key: "memory",
+    label: "Observational memory",
+    type: "boolean",
+    section: "Observational Memory",
+    helpText:
+      "Enable OM workers (observer, reflector, dropper) and content injection",
+  },
+  {
+    key: "sessionFallback",
+    label: "Session model fallback",
+    type: "boolean",
+    section: "Observational Memory",
+    helpText:
+      "off=skip stage when all OM models fail, instead of falling back to the main coding model",
+  },
+  {
+    key: "observeAfterTokens",
+    label: "Observer threshold",
+    type: "number",
+    section: "Observational Memory",
+    helpText:
+      "Tokens accumulated since last observer run before triggering next observe",
+  },
+  {
+    key: "reflectAfterTokens",
+    label: "Reflect + dropper threshold",
+    type: "number",
+    section: "Observational Memory",
+    helpText:
+      "Tokens accumulated since last reflect before triggering reflector and dropper",
+  },
+  {
+    key: "observationsPoolMaxTokens",
+    label: "Observation pool max",
+    type: "number",
+    section: "Observational Memory",
+    helpText:
+      "Max tokens in observation pool before dropper prunes (fold pressure)",
+  },
+  {
+    key: "observationsPoolTargetTokens",
+    label: "Observation pool target",
+    type: "number",
+    section: "Observational Memory",
+    helpText:
+      "Target tokens after dropper prunes (defaults to half of pool max)",
+  },
+  {
+    key: "reflectorInputMaxTokens",
+    label: "Reflector input max",
+    type: "number",
+    section: "Observational Memory",
+    helpText:
+      "Max prompt tokens for reflector model input (rolling window cap)",
+  },
+  {
+    key: "dropperInputMaxTokens",
+    label: "Dropper input max",
+    type: "number",
+    section: "Observational Memory",
+    helpText: "Max prompt tokens for dropper model input (rolling window cap)",
+  },
+  {
+    key: "observerChunkMaxTokens",
+    label: "Observer chunk max",
+    type: "number",
+    section: "Observational Memory",
+    helpText: "Max source entry tokens sent to observer per chunk",
+  },
+  {
+    key: "observerPreambleMaxTokens",
+    label: "Observer preamble max",
+    type: "number",
+    section: "Observational Memory",
+    helpText:
+      "Preamble budget in manual compaction mode (0=auto-compute 30% of chunk)",
+  },
+  {
+    key: "dropperPressureThreshold",
+    label: "Dropper pressure threshold",
+    type: "number",
+    section: "Observational Memory",
+    helpText:
+      "Fraction of reflectorInputMaxTokens that triggers pressure-driven dropper (0-1, default 0.70)",
+  },
+  {
+    key: "agentMaxTurns",
+    label: "Max turns per agent",
+    type: "number",
+    section: "Observational Memory",
+    helpText: "Shared turn cap for background memory agents",
+  },
+  {
+    key: "fullFoldAlways",
+    label: "Preserve OM on first compaction",
+    type: "boolean",
+    section: "Observational Memory",
+    helpText:
+      "When true, early reflections/drops survive the first compaction in a fresh session",
+  },
+
+  // ── Debug ──
+  {
+    key: "debug",
+    label: "Debug snapshots",
+    type: "boolean",
+    section: "Debug",
+    helpText: "Write detailed debug snapshots to /tmp/pi-blackhole-debug.json",
+  },
+  {
+    key: "debugLog",
+    label: "Debug JSONL logging",
+    type: "boolean",
+    section: "Debug",
+    helpText: "Write structured JSONL debug logs to agent directory",
+  },
 ];
 
 // ---------------------------------------------------------------------------
@@ -89,13 +207,13 @@ const FIELDS: FieldDef[] = [
 // ---------------------------------------------------------------------------
 
 function formatValue(def: FieldDef, rawValue: unknown): string {
-	if (rawValue === undefined || rawValue === null) return "";
-	switch (def.type) {
-		case "boolean":
-			return rawValue ? "on" : "off";
-		default:
-			return String(rawValue);
-	}
+  if (rawValue === undefined || rawValue === null) return "";
+  switch (def.type) {
+    case "boolean":
+      return rawValue ? "on" : "off";
+    default:
+      return String(rawValue);
+  }
 }
 
 // ---------------------------------------------------------------------------
@@ -103,11 +221,11 @@ function formatValue(def: FieldDef, rawValue: unknown): string {
 // ---------------------------------------------------------------------------
 
 type ThemeShim = {
-	fg: (style: string, text: string) => string;
+  fg: (style: string, text: string) => string;
 };
 
 const EMPTY_THEME: ThemeShim = {
-	fg: (_s: string, t: string) => t,
+  fg: (_s: string, t: string) => t,
 };
 
 // ---------------------------------------------------------------------------
@@ -115,9 +233,9 @@ const EMPTY_THEME: ThemeShim = {
 // ---------------------------------------------------------------------------
 
 export interface OverlayResult {
-	saved: boolean;
-	path: string;
-	error?: string;
+  saved: boolean;
+  path: string;
+  error?: string;
 }
 
 /**
@@ -132,292 +250,337 @@ export interface OverlayResult {
  * ```
  */
 export function createConfigureOverlay(
-	configPath: string,
-	theme: unknown,
-	tui: { requestRender: () => void },
-	done: (result: OverlayResult | undefined) => void,
-): { render(width: number): string[]; handleInput(data: string): void; invalidate(): void; dispose(): void } {
-	const th: ThemeShim = (theme && typeof (theme as Record<string, unknown>).fg === "function")
-		? theme as ThemeShim
-		: EMPTY_THEME;
-
-	// Parse config file
-	let raw: Record<string, unknown>;
-	let fileError: string | undefined;
-	try {
-		raw = JSON.parse(readFileSync(configPath, "utf8")) as Record<string, unknown>;
-	} catch (e) {
-		if (existsSync(configPath)) {
-			fileError = `Config file has invalid JSON: ${(e as Error).message}. Edit the file directly to fix it.`;
-		}
-		raw = {};
-	}
-
-	const defaults = DEFAULTS as unknown as Record<string, unknown>;
-	const fields: FieldState[] = FIELDS.map((def) => ({
-		def,
-		value: formatValue(def, def.key in raw ? raw[def.key] : defaults[def.key]),
-		editing: false,
-		cursor: 0,
-	}));
-
-	let selectedIndex = 0;
-	let cachedLines: string[] | undefined;
-
-	function invalidate(): void {
-		cachedLines = undefined;
-		try { tui.requestRender(); } catch { /* no-op */ }
-	}
-
-	function save(): boolean {
-		try {
-			const updated = { ...raw };
-			for (const f of fields) {
-				const val = f.value;
-				if (val === "" && !(f.def.key in raw)) continue;
-				switch (f.def.type) {
-					case "boolean":
-						updated[f.def.key] = val === "on";
-						break;
-					case "number": {
-						const num = Number(val);
-						if (val && !isNaN(num)) {
-							if (f.def.key === "dropperPressureThreshold") {
-								// Clamp to (0, 1] — runtime validates on reload
-								updated[f.def.key] = Math.min(1, Math.max(0.01, num));
-							} else {
-								updated[f.def.key] = num;
-							}
-						} else {
-							updated[f.def.key] = raw[f.def.key];
-						}
-						break;
-					}
-					case "enum":
-						updated[f.def.key] = val;
-						break;
-					default:
-						updated[f.def.key] = val;
-						break;
-				}
-			}
-			mkdirSync(dirname(configPath), { recursive: true });
-			writeFileSync(configPath, JSON.stringify(updated, null, 2) + "\n");
-			raw = updated;
-			return true;
-		} catch {
-			return false;
-		}
-	}
-
-	function handleInput(data: string): void {
-		const cur = fields[selectedIndex];
-		if (!cur) return;
-
-		// While editing a number field
-		if (cur.editing) {
-			if (matchesKey(data, "escape")) {
-				cur.value = formatValue(cur.def, raw[cur.def.key]);
-				cur.editing = false;
-				invalidate();
-				return;
-			}
-			if (matchesKey(data, "enter") || matchesKey(data, "tab")) {
-				cur.editing = false;
-				invalidate();
-				return;
-			}
-			if (matchesKey(data, "backspace")) {
-				if (cur.cursor > 0) {
-					cur.value = cur.value.slice(0, cur.cursor - 1) + cur.value.slice(cur.cursor);
-					cur.cursor--;
-					invalidate();
-				}
-				return;
-			}
-			if (matchesKey(data, "left")) {
-				cur.cursor = Math.max(0, cur.cursor - 1);
-				invalidate();
-				return;
-			}
-			if (matchesKey(data, "right")) {
-				cur.cursor = Math.min(cur.value.length, cur.cursor + 1);
-				invalidate();
-				return;
-			}
-			const digit = decodeKittyPrintable(data) ?? data;
-			if (digit.length === 1 && digit >= "0" && digit <= "9") {
-				cur.value = cur.value.slice(0, cur.cursor) + digit + cur.value.slice(cur.cursor);
-				cur.cursor++;
-				invalidate();
-			}
-			return;
-		}
-
-		// Not editing — global navigation
-
-		// Ctrl+S → save and close
-		if (matchesKey(data, "ctrl+s")) {
-			if (fileError) {
-				done({ saved: false, path: configPath, error: fileError });
-				return;
-			}
-			const saved = save();
-			done({ saved, path: configPath, error: saved ? undefined : "Failed to write config file" });
-			return;
-		}
-
-		// Esc → close without saving
-		if (matchesKey(data, "escape")) {
-			done(undefined);
-			return;
-		}
-
-		// Enter/space → edit/toggle
-		if (matchesKey(data, "enter") || matchesKey(data, "space")) {
-			switch (cur.def.type) {
-				case "boolean":
-					cur.value = cur.value === "on" ? "off" : "on";
-					invalidate();
-					break;
-				case "enum": {
-					const vals = cur.def.enumValues;
-					if (!vals || vals.length === 0) return;
-					const idx = vals.indexOf(cur.value);
-					cur.value = vals[(idx + 1) % vals.length];
-					invalidate();
-					break;
-				}
-				case "number":
-					cur.editing = true;
-					cur.cursor = cur.value.length;
-					invalidate();
-					break;
-			}
-			return;
-		}
-
-		// ↑↓ navigation
-		if (matchesKey(data, "up")) {
-			selectedIndex = Math.max(0, selectedIndex - 1);
-			invalidate();
-			return;
-		}
-		if (matchesKey(data, "down")) {
-			selectedIndex = Math.min(fields.length - 1, selectedIndex + 1);
-			invalidate();
-			return;
-		}
-	}
-
-	/** Compute minimum width to fit all content, plus borders. */
-	function contentWidth(): number {
-		const longestHelp = FIELDS.reduce((max, f) => {
-			if (!f.helpText) return max;
-			return Math.max(max, visibleWidth(f.helpText));
-		}, 0);
-		// Help line: │  ${help} ${│  → 4 chars overhead + at least 1 space
-		const helpOverhead = 4;
-		const longestLabel = FIELDS.reduce((max, f) => {
-			return Math.max(max, visibleWidth(f.label) + 3); // prefix + space + label
-		}, 0);
-		// Section header: ── N ──  → up to ~30, but help text dominates
-		// Hint line: ~54 visible
-		return Math.max(54 + 4, longestHelp + helpOverhead, longestLabel + 20);
-	}
-
-	function render(width: number): string[] {
-		if (cachedLines) return cachedLines;
-
-		const minW = contentWidth();
-		const w = Math.max(2, Math.min(width - 2, Math.max(minW, 50)));
-		const innerW = w - 4;
-		const fg = (style: string, text: string) => th.fg(style, text);
-
-		const lines: string[] = [];
-
-		// Top border + header
-		lines.push(fg("border", `╭${"─".repeat(w - 2)}╮`));
-		lines.push(fg("border", `│ ${fg("accent", "Blackhole Configuration")}${" ".repeat(Math.max(0, innerW + 1 - 24))}│`));
-		lines.push(fg("border", `├${"─".repeat(w - 2)}┤`));
-
-		// Error banner when config file has invalid JSON
-		if (fileError) {
-			lines.push(fg("border", `│${" ".repeat(w - 2)}│`));
-			const errorPrefix = fg("error", " ERROR:");
-			const errorText = fg("error", fileError);
-			const errorLine = `│ ${errorPrefix} ${errorText}${" ".repeat(Math.max(0, innerW - visibleWidth(errorPrefix) - 1 - visibleWidth(errorText)))}│`;
-			lines.push(errorLine);
-			lines.push(fg("border", `│${" ".repeat(w - 2)}│`));
-		}
-
-		let currentSection = "";
-
-		for (let i = 0; i < fields.length; i++) {
-			const f = fields[i];
-			const isSelected = i === selectedIndex;
-			const isEditing = isSelected && f.editing;
-
-			// Section header
-			if (f.def.section !== currentSection) {
-				currentSection = f.def.section;
-				if (i > 0) {
-					lines.push(fg("border", `│${" ".repeat(w - 2)}│`));
-				}
-				lines.push(fg("border", `│ ${fg("dim", `── ${currentSection} ──`)}${" ".repeat(Math.max(0, innerW + 1 - currentSection.length - 6))}│`));
-			}
-
-			// Field label
-			const prefix = isSelected ? fg("accent", isEditing ? ">>" : " >") : "  ";
-			const label = isSelected ? fg("accent", `${f.def.label}:`) : fg("text", `${f.def.label}:`);
-			const labelStr = `${prefix} ${label}`;
-			const labelVis = visibleWidth(labelStr);
-
-			// Value
-			let valueStr: string;
-			if (isEditing) {
-				const before = f.value.slice(0, f.cursor);
-				const cursorChar = f.cursor < f.value.length ? f.value[f.cursor] : " ";
-				const after = f.value.slice(f.cursor + 1);
-				valueStr = `${before}\x1b[7m${cursorChar}\x1b[27m${after}`;
-			} else {
-				switch (f.def.type) {
-					case "boolean":
-						valueStr = f.value === "on" ? fg("success", "on") : fg("muted", "off");
-						break;
-					default:
-						valueStr = f.value ? fg("text", f.value) : fg("dim", "(empty)");
-						break;
-				}
-			}
-
-			const valSpace = Math.max(10, innerW - labelVis - 1);
-			const truncated = visibleWidth(valueStr) > valSpace
-				? valueStr.slice(0, Math.max(0, valSpace - 3)) + "..."
-				: valueStr;
-			const remaining = innerW + 1 - labelVis - visibleWidth(truncated);
-
-			lines.push(fg("border", `│ ${labelStr}${truncated}${" ".repeat(Math.max(1, remaining))}│`));
-
-			// Help text for selected item
-			if (isSelected && f.def.helpText && !isEditing) {
-				const help = fg("dim", f.def.helpText);
-				const helpRemaining = innerW - visibleWidth(help);
-				lines.push(fg("border", `│  ${help}${" ".repeat(Math.max(1, helpRemaining))}│`));
-			}
-		}
-
-		// Bottom hints
-		lines.push(fg("border", `│${" ".repeat(w - 2)}│`));
-		const hintText = fileError
-			? " Ctrl+S blocked  ↑↓ navigate  Esc cancel (fix JSON first) "
-			: " Ctrl+S save  \u2191\u2193 navigate  Enter toggle  Esc cancel ";
-		lines.push(fg("border", `│${fg("accent", hintText)}${" ".repeat(Math.max(1, innerW + 2 - visibleWidth(hintText)))}│`));
-		lines.push(fg("border", `╰${"─".repeat(w - 2)}╯`));
-
-		cachedLines = lines;
-		return lines;
-	}
-
-	return { render, handleInput, invalidate, dispose: () => {} };
+  configPath: string,
+  theme: unknown,
+  tui: { requestRender: () => void },
+  done: (result: OverlayResult | undefined) => void,
+): {
+  render(width: number): string[];
+  handleInput(data: string): void;
+  invalidate(): void;
+  dispose(): void;
+} {
+  const th: ThemeShim =
+    theme && typeof (theme as Record<string, unknown>).fg === "function"
+      ? (theme as ThemeShim)
+      : EMPTY_THEME;
+
+  // Parse config file
+  let raw: Record<string, unknown>;
+  let fileError: string | undefined;
+  try {
+    raw = JSON.parse(readFileSync(configPath, "utf8")) as Record<
+      string,
+      unknown
+    >;
+  } catch (e) {
+    if (existsSync(configPath)) {
+      fileError = `Config file has invalid JSON: ${(e as Error).message}. Edit the file directly to fix it.`;
+    }
+    raw = {};
+  }
+
+  const defaults = DEFAULTS as unknown as Record<string, unknown>;
+  const fields: FieldState[] = FIELDS.map((def) => ({
+    def,
+    value: formatValue(def, def.key in raw ? raw[def.key] : defaults[def.key]),
+    editing: false,
+    cursor: 0,
+  }));
+
+  let selectedIndex = 0;
+  let cachedLines: string[] | undefined;
+
+  function invalidate(): void {
+    cachedLines = undefined;
+    try {
+      tui.requestRender();
+    } catch {
+      /* no-op */
+    }
+  }
+
+  function save(): boolean {
+    try {
+      const updated = { ...raw };
+      for (const f of fields) {
+        const val = f.value;
+        if (val === "" && !(f.def.key in raw)) continue;
+        switch (f.def.type) {
+          case "boolean":
+            updated[f.def.key] = val === "on";
+            break;
+          case "number": {
+            const num = Number(val);
+            if (val && !isNaN(num)) {
+              if (f.def.key === "dropperPressureThreshold") {
+                // Clamp to (0, 1] — runtime validates on reload
+                updated[f.def.key] = Math.min(1, Math.max(0.01, num));
+              } else {
+                updated[f.def.key] = num;
+              }
+            } else {
+              updated[f.def.key] = raw[f.def.key];
+            }
+            break;
+          }
+          case "enum":
+            updated[f.def.key] = val;
+            break;
+          default:
+            updated[f.def.key] = val;
+            break;
+        }
+      }
+      mkdirSync(dirname(configPath), { recursive: true });
+      writeFileSync(configPath, JSON.stringify(updated, null, 2) + "\n");
+      raw = updated;
+      return true;
+    } catch {
+      return false;
+    }
+  }
+
+  function handleInput(data: string): void {
+    const cur = fields[selectedIndex];
+    if (!cur) return;
+
+    // While editing a number field
+    if (cur.editing) {
+      if (matchesKey(data, "escape")) {
+        cur.value = formatValue(cur.def, raw[cur.def.key]);
+        cur.editing = false;
+        invalidate();
+        return;
+      }
+      if (matchesKey(data, "enter") || matchesKey(data, "tab")) {
+        cur.editing = false;
+        invalidate();
+        return;
+      }
+      if (matchesKey(data, "backspace")) {
+        if (cur.cursor > 0) {
+          cur.value =
+            cur.value.slice(0, cur.cursor - 1) + cur.value.slice(cur.cursor);
+          cur.cursor--;
+          invalidate();
+        }
+        return;
+      }
+      if (matchesKey(data, "left")) {
+        cur.cursor = Math.max(0, cur.cursor - 1);
+        invalidate();
+        return;
+      }
+      if (matchesKey(data, "right")) {
+        cur.cursor = Math.min(cur.value.length, cur.cursor + 1);
+        invalidate();
+        return;
+      }
+      const digit = decodeKittyPrintable(data) ?? data;
+      if (digit.length === 1 && digit >= "0" && digit <= "9") {
+        cur.value =
+          cur.value.slice(0, cur.cursor) + digit + cur.value.slice(cur.cursor);
+        cur.cursor++;
+        invalidate();
+      }
+      return;
+    }
+
+    // Not editing — global navigation
+
+    // Ctrl+S → save and close
+    if (matchesKey(data, "ctrl+s")) {
+      if (fileError) {
+        done({ saved: false, path: configPath, error: fileError });
+        return;
+      }
+      const saved = save();
+      done({
+        saved,
+        path: configPath,
+        error: saved ? undefined : "Failed to write config file",
+      });
+      return;
+    }
+
+    // Esc → close without saving
+    if (matchesKey(data, "escape")) {
+      done(undefined);
+      return;
+    }
+
+    // Enter/space → edit/toggle
+    if (matchesKey(data, "enter") || matchesKey(data, "space")) {
+      switch (cur.def.type) {
+        case "boolean":
+          cur.value = cur.value === "on" ? "off" : "on";
+          invalidate();
+          break;
+        case "enum": {
+          const vals = cur.def.enumValues;
+          if (!vals || vals.length === 0) return;
+          const idx = vals.indexOf(cur.value);
+          cur.value = vals[(idx + 1) % vals.length];
+          invalidate();
+          break;
+        }
+        case "number":
+          cur.editing = true;
+          cur.cursor = cur.value.length;
+          invalidate();
+          break;
+      }
+      return;
+    }
+
+    // ↑↓ navigation
+    if (matchesKey(data, "up")) {
+      selectedIndex = Math.max(0, selectedIndex - 1);
+      invalidate();
+      return;
+    }
+    if (matchesKey(data, "down")) {
+      selectedIndex = Math.min(fields.length - 1, selectedIndex + 1);
+      invalidate();
+      return;
+    }
+  }
+
+  /** Compute minimum width to fit all content, plus borders. */
+  function contentWidth(): number {
+    const longestHelp = FIELDS.reduce((max, f) => {
+      if (!f.helpText) return max;
+      return Math.max(max, visibleWidth(f.helpText));
+    }, 0);
+    // Help line: │  ${help} ${│  → 4 chars overhead + at least 1 space
+    const helpOverhead = 4;
+    const longestLabel = FIELDS.reduce((max, f) => {
+      return Math.max(max, visibleWidth(f.label) + 3); // prefix + space + label
+    }, 0);
+    // Section header: ── N ──  → up to ~30, but help text dominates
+    // Hint line: ~54 visible
+    return Math.max(54 + 4, longestHelp + helpOverhead, longestLabel + 20);
+  }
+
+  function render(width: number): string[] {
+    if (cachedLines) return cachedLines;
+
+    const minW = contentWidth();
+    const w = Math.max(2, Math.min(width - 2, Math.max(minW, 50)));
+    const innerW = w - 4;
+    const fg = (style: string, text: string) => th.fg(style, text);
+
+    const lines: string[] = [];
+
+    // Top border + header
+    lines.push(fg("border", `╭${"─".repeat(w - 2)}╮`));
+    lines.push(
+      fg(
+        "border",
+        `│ ${fg("accent", "Blackhole Configuration")}${" ".repeat(Math.max(0, innerW + 1 - 24))}│`,
+      ),
+    );
+    lines.push(fg("border", `├${"─".repeat(w - 2)}┤`));
+
+    // Error banner when config file has invalid JSON
+    if (fileError) {
+      lines.push(fg("border", `│${" ".repeat(w - 2)}│`));
+      const errorPrefix = fg("error", " ERROR:");
+      const errorText = fg("error", fileError);
+      const errorLine = `│ ${errorPrefix} ${errorText}${" ".repeat(Math.max(0, innerW - visibleWidth(errorPrefix) - 1 - visibleWidth(errorText)))}│`;
+      lines.push(errorLine);
+      lines.push(fg("border", `│${" ".repeat(w - 2)}│`));
+    }
+
+    let currentSection = "";
+
+    for (let i = 0; i < fields.length; i++) {
+      const f = fields[i];
+      const isSelected = i === selectedIndex;
+      const isEditing = isSelected && f.editing;
+
+      // Section header
+      if (f.def.section !== currentSection) {
+        currentSection = f.def.section;
+        if (i > 0) {
+          lines.push(fg("border", `│${" ".repeat(w - 2)}│`));
+        }
+        lines.push(
+          fg(
+            "border",
+            `│ ${fg("dim", `── ${currentSection} ──`)}${" ".repeat(Math.max(0, innerW + 1 - currentSection.length - 6))}│`,
+          ),
+        );
+      }
+
+      // Field label
+      const prefix = isSelected ? fg("accent", isEditing ? ">>" : " >") : "  ";
+      const label = isSelected
+        ? fg("accent", `${f.def.label}:`)
+        : fg("text", `${f.def.label}:`);
+      const labelStr = `${prefix} ${label}`;
+      const labelVis = visibleWidth(labelStr);
+
+      // Value
+      let valueStr: string;
+      if (isEditing) {
+        const before = f.value.slice(0, f.cursor);
+        const cursorChar = f.cursor < f.value.length ? f.value[f.cursor] : " ";
+        const after = f.value.slice(f.cursor + 1);
+        valueStr = `${before}\x1b[7m${cursorChar}\x1b[27m${after}`;
+      } else {
+        switch (f.def.type) {
+          case "boolean":
+            valueStr =
+              f.value === "on" ? fg("success", "on") : fg("muted", "off");
+            break;
+          default:
+            valueStr = f.value ? fg("text", f.value) : fg("dim", "(empty)");
+            break;
+        }
+      }
+
+      const valSpace = Math.max(10, innerW - labelVis - 1);
+      const truncated =
+        visibleWidth(valueStr) > valSpace
+          ? valueStr.slice(0, Math.max(0, valSpace - 3)) + "..."
+          : valueStr;
+      const remaining = innerW + 1 - labelVis - visibleWidth(truncated);
+
+      lines.push(
+        fg(
+          "border",
+          `│ ${labelStr}${truncated}${" ".repeat(Math.max(1, remaining))}│`,
+        ),
+      );
+
+      // Help text for selected item
+      if (isSelected && f.def.helpText && !isEditing) {
+        const help = fg("dim", f.def.helpText);
+        const helpRemaining = innerW - visibleWidth(help);
+        lines.push(
+          fg("border", `│  ${help}${" ".repeat(Math.max(1, helpRemaining))}│`),
+        );
+      }
+    }
+
+    // Bottom hints
+    lines.push(fg("border", `│${" ".repeat(w - 2)}│`));
+    const hintText = fileError
+      ? " Ctrl+S blocked  ↑↓ navigate  Esc cancel (fix JSON first) "
+      : " Ctrl+S save  \u2191\u2193 navigate  Enter toggle  Esc cancel ";
+    lines.push(
+      fg(
+        "border",
+        `│${fg("accent", hintText)}${" ".repeat(Math.max(1, innerW + 2 - visibleWidth(hintText)))}│`,
+      ),
+    );
+    lines.push(fg("border", `╰${"─".repeat(w - 2)}╯`));
+
+    cachedLines = lines;
+    return lines;
+  }
+
+  return { render, handleInput, invalidate, dispose: () => {} };
 }