@pi-unipi/unipi 2.6.0 → 2.6.2

package/packages/unipi/index.ts

@@ -29,18 +29,22 @@ import compactor from "@pi-unipi/compactor";
 import footer from "@pi-unipi/footer";
 import updater from "@pi-unipi/updater";
 import inputShortcuts from "@pi-unipi/input-shortcuts";
-import cocoindex from "@pi-unipi/cocoindex";
 import image from "../image/src/index.js";
 
 export default function (pi: ExtensionAPI) {
   workflow(pi);
   ralph(pi);
   memory(pi);
+  // Utility loads BEFORE info-screen: the name badge overlay must be pushed
+  // to the BOTTOM of the overlay stack. hideOverlay() pops the topmost entry,
+  // and a capturing overlay's done() callback is one-shot — if the badge were
+  // stacked above the boot info-screen, the info-screen's auto-close would pop
+  // the badge (spending its done()) and strand the dashboard uncloseable.
+  utility(pi);
   infoScreen(pi);
   subagents(pi);
   btw(pi);
   webApi(pi);
-  utility(pi);
   askUser(pi);
   mcp(pi);
   notify(pi);
@@ -51,6 +55,5 @@ export default function (pi: ExtensionAPI) {
   footer(pi);
   updater(pi);
   inputShortcuts(pi);
-  cocoindex(pi);
   image(pi);
 }

package/packages/updater/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-unipi/updater",
-  "version": "2.6.0",
+  "version": "2.6.1",
   "description": "Auto-updater, changelog browser, and readme browser for Unipi — checks npm registry, renders CHANGELOG.md and README.md files in TUI overlays",
   "type": "module",
   "main": "index.ts",
@@ -34,7 +34,7 @@
     "access": "public"
   },
   "dependencies": {
-    "@pi-unipi/core": "2.6.0"
+    "@pi-unipi/core": "2.6.1"
   },
   "peerDependencies": {
     "@earendil-works/pi-coding-agent": "^0.80.0",

package/packages/updater/src/changelog.ts

@@ -8,7 +8,7 @@
 import { existsSync, readFileSync } from "fs";
 import { dirname, join } from "node:path";
 import { fileURLToPath } from "node:url";
-import { findPackageRoot } from "@pi-unipi/core";
+import { findPackageRoot, compareVersions } from "@pi-unipi/core";
 import { isNewerVersion } from "./version.js";
 import type { ChangelogEntry } from "../types.js";
 
@@ -155,18 +155,3 @@ export function getNewerVersions(
   }
   return result;
 }
-
-/**
- * Compare semver strings (simple lexicographic for x.y.z format).
- * Returns positive if a > b, negative if a < b, 0 if equal.
- */
-export function compareVersions(a: string, b: string): number {
-  const pa = a.split(".").map(Number);
-  const pb = b.split(".").map(Number);
-  for (let i = 0; i < Math.max(pa.length, pb.length); i++) {
-    const na = pa[i] ?? 0;
-    const nb = pb[i] ?? 0;
-    if (na !== nb) return na - nb;
-  }
-  return 0;
-}

package/packages/updater/src/index.ts

@@ -165,9 +165,4 @@ export default function updaterExtension(pi: ExtensionAPI): void {
       // Update check failure — silent, non-critical
     }
   });
-
-  // Cleanup on session shutdown
-  pi.on("session_shutdown", async () => {
-    // No cleanup needed
-  });
 }

package/packages/updater/src/installer.ts

@@ -2,12 +2,11 @@
  * @pi-unipi/updater — Update installer
  *
  * Wraps child_process.exec for installing updates via pi CLI.
- * Emits UPDATE_APPLIED or UPDATE_ERROR events.
  */
 
 import { exec } from "child_process";
 import { promisify } from "util";
-import { getInstalledPackageVersion, emitEvent, UNIPI_EVENTS } from "@pi-unipi/core";
+import { getInstalledPackageVersion } from "@pi-unipi/core";
 import type { InstallResult } from "../types.js";
 
 const execAsync = promisify(exec);
@@ -20,14 +19,11 @@ const INSTALL_TIMEOUT_MS = 60000;
  * Uses pi CLI: `pi install npm:@pi-unipi/unipi`
  * Returns structured result with success/failure info.
  */
-export async function installUpdate(
-  pi?: { events: { emit: (name: string, payload: unknown) => void } },
-): Promise<InstallResult> {
+export async function installUpdate(): Promise<InstallResult> {
   const thisDir = new URL("..", import.meta.url).pathname;
-  const installedBefore = getInstalledPackageVersion(thisDir, "@pi-unipi/unipi");
 
   try {
-    const { stdout, stderr } = await execAsync(
+    await execAsync(
       "pi install npm:@pi-unipi/unipi",
       {
         timeout: INSTALL_TIMEOUT_MS,
@@ -38,34 +34,16 @@ export async function installUpdate(
     // Get new version after install
     const installedAfter = getInstalledPackageVersion(thisDir, "@pi-unipi/unipi");
 
-    const result: InstallResult = {
+    return {
       success: true,
       version: installedAfter,
     };
-
-    // Emit success event
-    if (pi) {
-      emitEvent(pi, UNIPI_EVENTS.UPDATE_APPLIED, {
-        previousVersion: installedBefore,
-        newVersion: installedAfter,
-      });
-    }
-
-    return result;
   } catch (err: unknown) {
     const errorMessage = (err instanceof Error && 'stderr' in err ? String((err as Error & { stderr?: string }).stderr) : undefined)
       || (err instanceof Error ? err.message : undefined)
       || String(err)
       || "Unknown install error";
 
-    // Emit error event
-    if (pi) {
-      emitEvent(pi, UNIPI_EVENTS.UPDATE_ERROR, {
-        error: errorMessage,
-        phase: "install",
-      });
-    }
-
     return {
       success: false,
       error: errorMessage,

package/packages/updater/src/markdown.ts

@@ -1,173 +1,27 @@
 /**
  * @pi-unipi/updater — Markdown terminal renderer
  *
- * Renders markdown to terminal-formatted strings.
- * When a Theme is provided, uses the full Markdown component from pi-tui
- * with theme-aware styling. Falls back to simple ANSI rendering otherwise.
+ * Renders markdown to terminal-formatted strings using the full
+ * Markdown component from pi-tui with theme-aware styling.
  */
 
 import { Markdown } from "@earendil-works/pi-tui";
-import type { MarkdownTheme } from "@earendil-works/pi-tui";
 import { getMarkdownTheme } from "@earendil-works/pi-coding-agent";
 import type { Theme } from "@earendil-works/pi-coding-agent";
 
 /**
  * Render markdown text to terminal-formatted lines.
  *
- * When a Theme is provided, uses the full Markdown component from pi-tui
- * with syntax highlighting, proper list nesting, tables, etc.
+ * Uses pi-tui's Markdown component with syntax highlighting,
+ * proper list nesting, tables, etc.
  *
  * @param text - Markdown text to render
  * @param width - Available width for rendering
- * @param theme - Optional Theme for styled rendering
+ * @param theme - Theme for styled rendering
  * @returns Array of rendered terminal lines
  */
-export function renderMarkdown(text: string, width: number, theme?: Theme): string[] {
-  if (theme) {
-    return renderWithTheme(text, width, theme);
-  }
-  return renderSimple(text, width);
-}
-
-/**
- * Render using the full Markdown component with theme support.
- */
-function renderWithTheme(text: string, width: number, theme: Theme): string[] {
+export function renderMarkdown(text: string, width: number, theme: Theme): string[] {
   const mdTheme = getMarkdownTheme();
   const md = new Markdown(text, 1, 0, mdTheme);
   return md.render(width);
 }
-
-/**
- * Simple fallback renderer using basic ANSI codes.
- * Used when no Theme is available.
- */
-
-/** ANSI escape codes */
-const ESC = "\x1b";
-const BOLD = `${ESC}[1m`;
-const DIM = `${ESC}[2m`;
-const UNDERLINE = `${ESC}[4m`;
-const RESET = `${ESC}[0m`;
-
-/** Wrap text in ANSI formatting */
-function fmt(code: string, text: string): string {
-  return `${code}${text}${RESET}`;
-}
-
-/** Word-wrap a line to fit within width */
-function wordWrap(line: string, width: number): string[] {
-  // Strip ANSI for length calculation but preserve in output
-  const stripped = line.replace(/\x1b\[[0-9;]*m/g, "");
-  if (stripped.length <= width) return [line];
-
-  const words = line.split(/(\s+)/);
-  const result: string[] = [];
-  let currentLine = "";
-  let currentWidth = 0;
-
-  for (const word of words) {
-    const wordWidth = word.replace(/\x1b\[[0-9;]*m/g, "").length;
-    if (currentWidth + wordWidth > width && currentLine) {
-      result.push(currentLine);
-      currentLine = word.trimStart();
-      currentWidth = currentLine.replace(/\x1b\[[0-9;]*m/g, "").length;
-    } else {
-      currentLine += word;
-      currentWidth += wordWidth;
-    }
-  }
-  if (currentLine) result.push(currentLine);
-  return result.length > 0 ? result : [""];
-}
-
-/** Apply inline formatting: bold, italic, code, links */
-function formatInline(text: string): string {
-  // Inline code: `code`
-  text = text.replace(/`([^`]+)`/g, (_, code) => fmt(DIM, code));
-
-  // Bold: **text**
-  text = text.replace(/\*\*([^*]+)\*\*/g, (_, bold) => fmt(BOLD, bold));
-
-  // Italic: *text*
-  text = text.replace(/(?<!\*)\*([^*]+)\*(?!\*)/g, (_, italic) => fmt(UNDERLINE, italic));
-
-  // Links: [text](url) → underlined text
-  text = text.replace(/\[([^\]]+)\]\([^)]+\)/g, (_, linkText) => fmt(UNDERLINE, linkText));
-
-  return text;
-}
-
-/**
- * Simple markdown renderer (fallback).
- */
-function renderSimple(text: string, width: number): string[] {
-  const lines = text.split("\n");
-  const result: string[] = [];
-  let inCodeBlock = false;
-
-  for (const line of lines) {
-    const trimmed = line.trim();
-
-    // Code fence toggle
-    if (trimmed.startsWith("```")) {
-      inCodeBlock = !inCodeBlock;
-      continue;
-    }
-
-    // Inside code block — dim, no formatting
-    if (inCodeBlock) {
-      const formatted = fmt(DIM, line);
-      result.push(...wordWrap(formatted, width));
-      continue;
-    }
-
-    // Heading: #, ##, ###
-    if (trimmed.startsWith("### ")) {
-      const heading = trimmed.slice(4);
-      const formatted = fmt(BOLD + UNDERLINE, heading);
-      result.push(...wordWrap(formatted, width));
-      continue;
-    }
-    if (trimmed.startsWith("## ")) {
-      const heading = trimmed.slice(3);
-      const formatted = fmt(BOLD, heading);
-      result.push(...wordWrap(formatted, width));
-      continue;
-    }
-    if (trimmed.startsWith("# ")) {
-      const heading = trimmed.slice(2);
-      const formatted = fmt(BOLD, heading);
-      result.push(...wordWrap(formatted, width));
-      continue;
-    }
-
-    // Bullet list: - or *
-    if (trimmed.startsWith("- ") || trimmed.startsWith("* ")) {
-      const content = trimmed.slice(2);
-      const formatted = "  • " + formatInline(content);
-      result.push(...wordWrap(formatted, width));
-      continue;
-    }
-
-    // Numbered list: 1. 2. etc.
-    const numberMatch = trimmed.match(/^(\d+)\.\s+(.+)$/);
-    if (numberMatch) {
-      const formatted = `  ${numberMatch[1]}. ${formatInline(numberMatch[2])}`;
-      result.push(...wordWrap(formatted, width));
-      continue;
-    }
-
-    // Empty line — preserve spacing
-    if (!trimmed) {
-      result.push("");
-      continue;
-    }
-
-    // Regular paragraph — apply inline formatting
-    const formatted = formatInline(trimmed);
-    result.push(...wordWrap(formatted, width));
-  }
-
-  return result;
-}

package/packages/updater/src/settings.ts

@@ -56,21 +56,6 @@ export function saveConfig(config: UpdaterConfig): void {
   writeFileSync(configPath, JSON.stringify(config, null, 2) + "\n", "utf-8");
 }
 
-/** Validate config, returning list of error messages */
-export function validateConfig(config: UpdaterConfig): string[] {
-  const errors: string[] = [];
-
-  if (!VALID_MODES.includes(config.autoUpdate)) {
-    errors.push(`autoUpdate must be one of: ${VALID_MODES.join(", ")}`);
-  }
-
-  if (config.checkIntervalMs < 60000) {
-    errors.push("checkIntervalMs must be at least 60000 (1 minute)");
-  }
-
-  return errors;
-}
-
 /** Get human-readable label for an interval */
 export function getIntervalLabel(ms: number): string {
   for (const [label, value] of Object.entries(VALID_INTERVALS)) {

package/packages/updater/src/tui/changelog-overlay.ts

@@ -1,257 +1,65 @@
 /**
  * @pi-unipi/updater — Changelog TUI Overlay
  *
- * Version list with Current/New labels, Enter opens detail view, Esc/q back.
- * Uses ctx.ui.custom() overlay API with component return pattern.
+ * Version list with Current/New labels, Enter opens detail view.
  */
 
 import { existsSync } from "fs";
-import { join } from "path";
-import { Key, matchesKey, truncateToWidth, visibleWidth } from "@earendil-works/pi-tui";
 import type { Theme } from "@earendil-works/pi-coding-agent";
 import { parseChangelog, resolveChangelogPath } from "../changelog.js";
 import { renderMarkdown } from "../markdown.js";
-import { getInstalledPackageVersion, boxInnerWidth } from "@pi-unipi/core";
+import { getInstalledPackageVersion } from "@pi-unipi/core";
+import { createListDetailOverlay } from "./list-detail-overlay.js";
 import type { ChangelogEntry } from "../../types.js";
 
-type View = "list" | "detail";
-
-interface ChangelogState {
-  view: View;
-  entries: ChangelogEntry[];
-  listIndex: number;
-  listScroll: number;
-  detailScroll: number;
-  installedVersion: string;
-}
-
-/**
- * Pad content to exact visible width.
- */
-function padVisible(content: string, targetWidth: number): string {
-  const vw = visibleWidth(content);
-  const pad = Math.max(0, targetWidth - vw);
-  return content + " ".repeat(pad);
-}
-
 /**
  * Render the changelog overlay.
  */
 export function renderChangelogOverlay() {
-  return (
-    tui: import("@earendil-works/pi-tui").TUI,
-    theme: Theme,
-    _kb: import("@earendil-works/pi-coding-agent").KeybindingsManager,
-    done: (result: { viewed: boolean } | null) => void,
-  ) => {
-    const installedVersion = getInstalledPackageVersion(
-      new URL("..", import.meta.url).pathname,
-      "@pi-unipi/unipi",
-    );
-
-    const state: ChangelogState = {
-      view: "list",
-      entries: [],
-      listIndex: 0,
-      listScroll: 0,
-      detailScroll: 0,
-      installedVersion,
-    };
-
-    // Load changelog
-    let loaded = false;
-    const ensureLoaded = () => {
-      if (loaded) return;
+  const installedVersion = getInstalledPackageVersion(
+    new URL("..", import.meta.url).pathname,
+    "@pi-unipi/unipi",
+  );
+
+  return createListDetailOverlay<ChangelogEntry>({
+    title: " 📋 Changelog ",
+    emptyMessage: "No changelog available.",
+    listFooter: " j/k navigate  Enter view details  q/Esc close",
+    detailFooter: " j/k scroll  q/Esc back to list",
+
+    loadEntries: () => {
       const changelogPath = resolveChangelogPath();
-      if (existsSync(changelogPath)) {
-        state.entries = parseChangelog(changelogPath);
-      }
-      loaded = true;
-    };
-
-    const render = (width: number): string[] => {
-      ensureLoaded();
-
-      const innerWidth = boxInnerWidth(width);
-      const lines: string[] = [];
-
-      // ── Header ──────────────────────────────────────────────────────
-      lines.push(theme.fg("accent", `╭${"─".repeat(innerWidth)}╮`));
-      lines.push(
-        theme.fg("accent", "│") +
-        padVisible(theme.fg("accent", theme.bold(" 📋 Changelog ")), innerWidth) +
-        theme.fg("accent", "│"),
-      );
-      lines.push(theme.fg("accent", `├${"─".repeat(innerWidth)}┤`));
-
-      // ── Content ─────────────────────────────────────────────────────
-      if (state.view === "list") {
-        renderListView(lines, innerWidth);
+      return existsSync(changelogPath) ? parseChangelog(changelogPath) : [];
+    },
+
+    renderItem: (entry, selected, theme) => {
+      let label: string;
+      if (entry.version === "Unreleased") {
+        label = theme.fg("muted", "Unreleased");
+      } else if (entry.version === installedVersion) {
+        label = theme.fg("success", "✓ Current");
       } else {
-        renderDetailView(lines, innerWidth);
-      }
-
-      // ── Footer ──────────────────────────────────────────────────────
-      lines.push(theme.fg("accent", `├${"─".repeat(innerWidth)}┤`));
-      const footer =
-        state.view === "list"
-          ? ` ${theme.fg("accent", "j/k")} navigate  ${theme.fg("success", "Enter")} view details  ${theme.fg("error", "q/Esc")} close`
-          : ` ${theme.fg("accent", "j/k")} scroll  ${theme.fg("error", "q/Esc")} back to list`;
-      lines.push(
-        theme.fg("accent", "│") +
-        padVisible(truncateToWidth(footer, innerWidth), innerWidth) +
-        theme.fg("accent", "│"),
-      );
-      lines.push(theme.fg("accent", `╰${"─".repeat(innerWidth)}╯`));
-
-      return lines;
-    };
-
-    const renderListView = (lines: string[], innerWidth: number) => {
-      if (state.entries.length === 0) {
-        lines.push(
-          theme.fg("accent", "│") +
-          padVisible(theme.fg("muted", "  No changelog available."), innerWidth) +
-          theme.fg("accent", "│"),
-        );
-        return;
+        const pa = entry.version.split(".").map(Number);
+        const pb = installedVersion.split(".").map(Number);
+        const isNewer =
+          pa[0]! > pb[0]! ||
+          (pa[0] === pb[0] && pa[1]! > pb[1]!) ||
+          (pa[0] === pb[0] && pa[1] === pb[1] && pa[2]! > pb[2]!);
+        label = isNewer ? theme.fg("warning", "↑ New") : "";
       }
 
-      state.listIndex = Math.min(state.listIndex, state.entries.length - 1);
-      state.listIndex = Math.max(0, state.listIndex);
-
-      // Show visible entries
-      const maxLines = 20;
-      if (state.listIndex < state.listScroll) state.listScroll = state.listIndex;
-      if (state.listIndex >= state.listScroll + maxLines) {
-        state.listScroll = state.listIndex - maxLines + 1;
-      }
+      const version = selected ? theme.bold(entry.version) : theme.fg("text", entry.version);
+      const date = entry.date ? ` — ${theme.fg("muted", entry.date)}` : "";
+      const prefix = selected ? theme.fg("accent", "▸ ") : "  ";
+      return ` ${prefix}${version}${date}  ${label}`;
+    },
 
-      const visible = state.entries.slice(state.listScroll, state.listScroll + maxLines);
-
-      for (let i = 0; i < visible.length; i++) {
-        const entry = visible[i]!;
-        const globalIdx = state.listScroll + i;
-        const selected = globalIdx === state.listIndex;
-        const prefix = selected ? theme.fg("accent", "▸ ") : "  ";
-
-        let label: string;
-        if (entry.version === "Unreleased") {
-          label = theme.fg("muted", "Unreleased");
-        } else if (entry.version === state.installedVersion) {
-          label = theme.fg("success", "✓ Current");
-        } else {
-          const pa = entry.version.split(".").map(Number);
-          const pb = state.installedVersion.split(".").map(Number);
-          const isNewer =
-            pa[0]! > pb[0]! ||
-            (pa[0] === pb[0] && pa[1]! > pb[1]!) ||
-            (pa[0] === pb[0] && pa[1] === pb[1] && pa[2]! > pb[2]!);
-          label = isNewer ? theme.fg("warning", "↑ New") : "";
-        }
-
-        const version = selected ? theme.bold(entry.version) : theme.fg("text", entry.version);
-        const date = entry.date ? ` — ${theme.fg("muted", entry.date)}` : "";
-        const line = ` ${prefix}${version}${date}  ${label}`;
-        lines.push(
-          theme.fg("accent", "│") +
-          padVisible(
-            selected ? theme.bg("selectedBg", truncateToWidth(line, innerWidth)) : truncateToWidth(line, innerWidth),
-            innerWidth,
-          ) +
-          theme.fg("accent", "│"),
-        );
-      }
-    };
-
-    const renderDetailView = (lines: string[], innerWidth: number) => {
-      const entry = state.entries[state.listIndex];
-      if (!entry) {
-        lines.push(
-          theme.fg("accent", "│") +
-          padVisible(theme.fg("muted", "  No entry selected."), innerWidth) +
-          theme.fg("accent", "│"),
-        );
-        return;
-      }
-
-      const title = entry.date
+    renderDetailTitle: (entry, theme) =>
+      entry.date
         ? `${theme.bold(entry.version)} — ${theme.fg("muted", entry.date)}`
-        : `${theme.bold(entry.version)} — ${theme.fg("muted", "Unreleased")}`;
-      lines.push(
-        theme.fg("accent", "│") +
-        padVisible(truncateToWidth(`  ${title}`, innerWidth), innerWidth) +
-        theme.fg("accent", "│"),
-      );
-      lines.push(
-        theme.fg("accent", "│") +
-        padVisible("", innerWidth) +
-        theme.fg("accent", "│"),
-      );
-
-      const bodyLines = renderMarkdown(entry.body, innerWidth - 2, theme);
-      const maxScroll = Math.max(0, bodyLines.length - 15);
-      state.detailScroll = Math.min(state.detailScroll, maxScroll);
-      state.detailScroll = Math.max(0, state.detailScroll);
-
-      const visible = bodyLines.slice(state.detailScroll, state.detailScroll + 15);
-      for (const line of visible) {
-        lines.push(
-          theme.fg("accent", "│") +
-          padVisible(truncateToWidth(`  ${line}`, innerWidth), innerWidth) +
-          theme.fg("accent", "│"),
-        );
-      }
-    };
-
-    const handleInput = (data: string) => {
-      ensureLoaded();
-
-      // Close from list view
-      if ((matchesKey(data, Key.escape) || data === "q") && state.view === "list") {
-        done({ viewed: true });
-        return;
-      }
-
-      // Back from detail view
-      if ((matchesKey(data, Key.escape) || data === "q") && state.view === "detail") {
-        state.view = "list";
-        state.detailScroll = 0;
-        tui.requestRender();
-        return;
-      }
-
-      // Navigation
-      if (state.view === "list") {
-        if (matchesKey(data, Key.down) || data === "j") {
-          state.listIndex = Math.min(state.listIndex + 1, state.entries.length - 1);
-        } else if (matchesKey(data, Key.up) || data === "k") {
-          state.listIndex = Math.max(state.listIndex - 1, 0);
-        } else if (matchesKey(data, Key.enter)) {
-          if (state.entries.length > 0) {
-            state.view = "detail";
-            state.detailScroll = 0;
-          }
-        } else if (data === "g") {
-          state.listIndex = 0;
-        } else if (data === "G") {
-          state.listIndex = state.entries.length - 1;
-        }
-      } else {
-        if (matchesKey(data, Key.down) || data === "j") {
-          state.detailScroll++;
-        } else if (matchesKey(data, Key.up) || data === "k") {
-          state.detailScroll = Math.max(0, state.detailScroll - 1);
-        } else if (data === "g") {
-          state.detailScroll = 0;
-        } else if (data === "G") {
-          state.detailScroll = 999999;
-        }
-      }
-
-      tui.requestRender();
-    };
+        : `${theme.bold(entry.version)} — ${theme.fg("muted", "Unreleased")}`,
 
-    return { render, handleInput, invalidate: () => {}, focused: true };
-  };
+    renderDetailBody: (entry, innerWidth, theme) =>
+      renderMarkdown(entry.body, innerWidth - 2, theme),
+  });
 }