github-search-cli 9.2.1 → 9.3.0

package/README.md

@@ -203,11 +203,11 @@ npm run build    # build dist/
 ```
 
 ---
-### Unreleased
-- Fixed trending tabs (Today / This Week / This Month) returning the same repos — cache key now includes the `since` parameter so each period fetches independently
-- Added cyan underline indicator beneath the active trending tab for clearer visual selection
-- Added `ghfind --compare <repo1> <repo2> [repo3]` CLI command to compare repos side-by-side
-- Added `c` keybinding in TUI to toggle compare mode on selected repo
+### v0.9.3
+- Fixed `c` keybinding in TUI not adding repos to comparison — missing closing brace in space-key handler broke all subsequent key handlers (`c`, `t`, `?`)
+- Widened compare table columns (min 35 chars) and added Description + URL rows for richer side-by-side comparison
+- Compare table now shows up to 5 topics per repo (was 3)
+- README viewer now renders full markdown (headings, code blocks, lists, tables, blockquotes, rules) instead of stripping formatting
 ---
 ### v0.9.2
 

package/dist/cli.js

@@ -1144,9 +1144,10 @@ function buildComparisonTable(repos) {
   rows.push({ label: "Language", values: repos.map((r) => r.language ?? "\u2014") });
   rows.push({ label: "Created", values: repos.map((r) => r.createdAt?.slice(0, 7) ?? "\u2014") });
   rows.push({ label: "Updated", values: repos.map((r) => r.updatedAt?.slice(0, 10) ?? "\u2014") });
-  rows.push({ label: "Topics", values: repos.map((r) => (r.topics?.slice(0, 3).join(", ") ?? "\u2014") || "\u2014") });
-  const colW = Math.max(15, ...names.map((n) => n.length));
-  const valW = Math.max(8, ...rows.flatMap((r) => r.values.map((v) => v.length)));
+  rows.push({ label: "Topics", values: repos.map((r) => (r.topics?.slice(0, 5).join(", ") ?? "\u2014") || "\u2014") });
+  rows.push({ label: "Description", values: repos.map((r) => r.description?.slice(0, 50) ?? "\u2014") });
+  rows.push({ label: "URL", values: repos.map((r) => r.url || "\u2014") });
+  const colW = Math.max(35, ...names.map((n) => n.length));
   const header = ["".padEnd(14), ...names.map((n) => n.padEnd(colW))];
   const sep = ["".padEnd(14, "\u2500"), ...names.map(() => "".padEnd(colW, "\u2500"))];
   const lines = [
@@ -1575,6 +1576,177 @@ function buildDeepDiveText(data) {
   ].join("\n");
 }
 
+// src/markdown-render.ts
+function renderMarkdown(md, opts = {}) {
+  const width = opts.width ?? 80;
+  const lines = md.split("\n");
+  const out = [];
+  let inCodeBlock = false;
+  let codeLang = "";
+  let codeLines = [];
+  let inTable = false;
+  let tableRows = [];
+  function flushCode() {
+    if (codeLines.length === 0) return;
+    const maxLen = Math.max(...codeLines.map((l) => l.length), codeLang.length);
+    const barW = Math.min(maxLen + 4, width - 2);
+    out.push("\u250C" + "\u2500".repeat(barW) + "\u2510");
+    if (codeLang) out.push("\u2502 " + codeLang.padEnd(barW - 1) + "\u2502");
+    for (const cl of codeLines) {
+      out.push("\u2502 " + cl.padEnd(barW - 1) + "\u2502");
+    }
+    out.push("\u2514" + "\u2500".repeat(barW) + "\u2518");
+    out.push("");
+    codeLines = [];
+    codeLang = "";
+  }
+  function flushTable() {
+    if (tableRows.length === 0) return;
+    const dataRows = tableRows.filter(
+      (r) => !r.every((c) => /^[\s\-:|]+$/.test(c))
+    );
+    if (dataRows.length === 0) {
+      tableRows = [];
+      return;
+    }
+    const colCount = Math.max(...dataRows.map((r) => r.length));
+    const colW = [];
+    for (let ci = 0; ci < colCount; ci++) {
+      colW.push(
+        Math.max(
+          3,
+          ...dataRows.map((r) => (r[ci] ?? "").length)
+        )
+      );
+    }
+    const totalW = colW.reduce((a, b) => a + b, 0) + colCount * 3 - 1;
+    if (totalW > width) {
+      for (const row of dataRows) {
+        out.push("  " + row.map((c) => c.trim()).join(" \u2502 "));
+      }
+    } else {
+      const sep = "\u250C" + colW.map((w) => "\u2500".repeat(w + 2)).join("\u252C") + "\u2510";
+      const mid = "\u251C" + colW.map((w) => "\u2500".repeat(w + 2)).join("\u253C") + "\u2524";
+      const end = "\u2514" + colW.map((w) => "\u2500".repeat(w + 2)).join("\u2534") + "\u2518";
+      out.push(sep);
+      dataRows.forEach((row, i) => {
+        const cells = colW.map((w, ci) => (row[ci] ?? "").trim().padEnd(w));
+        out.push("\u2502 " + cells.map((c) => ` ${c} `).join("\u2502") + "\u2502");
+        if (i === 0) out.push(mid);
+      });
+      out.push(end);
+    }
+    out.push("");
+    tableRows = [];
+  }
+  function renderInline(line) {
+    line = line.replace(/\*\*(.+?)\*\*/g, "$1");
+    line = line.replace(/\*([^*]+?)\*/g, "$1");
+    line = line.replace(/`([^`]+)`/g, "\u27E8$1\u27E9");
+    line = line.replace(/\[([^\]]+)\]\(([^)]+)\)/g, "$1 ($2)");
+    line = line.replace(/!\[([^\]]*)\]\([^)]+\)/g, "\u{1F5BC} $1");
+    return line;
+  }
+  function wrapLine(line, maxW) {
+    if (line.length <= maxW) return [line];
+    const parts = [];
+    let remaining = line;
+    while (remaining.length > maxW) {
+      let breakAt = maxW;
+      const spaceIdx = remaining.lastIndexOf(" ", maxW);
+      if (spaceIdx > maxW / 2) breakAt = spaceIdx;
+      parts.push(remaining.slice(0, breakAt));
+      remaining = remaining.slice(breakAt).replace(/^ /, "");
+    }
+    if (remaining) parts.push(remaining);
+    return parts;
+  }
+  for (let i = 0; i < lines.length; i++) {
+    let line = lines[i];
+    if (/^```/.test(line)) {
+      if (inCodeBlock) {
+        flushCode();
+        inCodeBlock = false;
+      } else {
+        inCodeBlock = true;
+        codeLang = line.slice(3).trim();
+      }
+      continue;
+    }
+    if (inCodeBlock) {
+      codeLines.push(line);
+      continue;
+    }
+    if (line.includes("|")) {
+      const cells = line.split("|").map((c) => c.trim()).filter((_, idx, arr) => idx > 0 && idx < arr.length - 1);
+      if (cells.length >= 2) {
+        inTable = true;
+        tableRows.push(cells);
+        continue;
+      }
+    } else if (inTable) {
+      flushTable();
+      inTable = false;
+    }
+    if (/^(\s*[-*_]){3,}\s*$/.test(line)) {
+      out.push("\u2500".repeat(Math.min(width, 40)));
+      out.push("");
+      continue;
+    }
+    const headingMatch = line.match(/^(#{1,6})\s+(.+)/);
+    if (headingMatch) {
+      const level = headingMatch[1].length;
+      const text = renderInline(headingMatch[2]);
+      const prefix = "\u2588 ".repeat(level);
+      out.push("");
+      out.push(prefix + text);
+      out.push("");
+      continue;
+    }
+    const bqMatch = line.match(/^>\s?(.*)/);
+    if (bqMatch) {
+      const inner = renderInline(bqMatch[1]);
+      for (const wl of wrapLine(inner, width - 4)) {
+        out.push("\u2502 " + wl);
+      }
+      continue;
+    }
+    const ulMatch = line.match(/^(\s*)[-*+]\s+(.*)/);
+    if (ulMatch) {
+      const indent = ulMatch[1].length;
+      const depth = Math.floor(indent / 2);
+      const prefix = "  ".repeat(depth) + "\u2022 ";
+      const inner = renderInline(ulMatch[2]);
+      for (const wl of wrapLine(inner, width - prefix.length)) {
+        out.push(prefix + wl);
+      }
+      continue;
+    }
+    const olMatch = line.match(/^(\s*)\d+\.\s+(.*)/);
+    if (olMatch) {
+      const indent = olMatch[1].length;
+      const depth = Math.floor(indent / 2);
+      const prefix = "  ".repeat(depth) + "\u25E6 ";
+      const inner = renderInline(olMatch[2]);
+      for (const wl of wrapLine(inner, width - prefix.length)) {
+        out.push(prefix + wl);
+      }
+      continue;
+    }
+    if (line.trim() === "") {
+      out.push("");
+      continue;
+    }
+    const rendered = renderInline(line);
+    for (const wl of wrapLine(rendered, width)) {
+      out.push(wl);
+    }
+  }
+  if (inCodeBlock) flushCode();
+  if (inTable) flushTable();
+  return out.join("\n");
+}
+
 // src/explore.ts
 async function fetchTopics() {
   const res = await fetch("https://api.github.com/search/topics?q=popular&per_page=30", {
@@ -3265,8 +3437,8 @@ async function launchBrowser(theme_override) {
       if (!text) {
         readmeText.content = `  (no README found for ${repo.fullName})`;
       } else {
-        const stripped = text.replace(/```[\s\S]*?```/g, "[code block]").replace(/#{1,6}\s/g, "").replace(/\*\*(.+?)\*\*/g, "$1").replace(/\[(.+?)\]\(.+?\)/g, "$1").replace(/`([^`]+)`/g, "$1").split("\n").map((l) => l.trimEnd()).join("\n");
-        readmeText.content = stripped;
+        const termW = Math.max(40, (process.stdout.columns || 80) - 4);
+        readmeText.content = renderMarkdown(text, { width: termW });
       }
     } catch {
       readmeText.content = "  Failed to load README";
@@ -4050,18 +4222,18 @@ ${pack.description ?? ""}`;
         showLeaderMenu();
       }
       return;
-      if (key.name === "t" && !searchInput.focused) {
-        const current = config.theme;
-        const themes = listThemes();
-        const idx = themes.indexOf(current);
-        const next = themes[(idx + 1) % themes.length];
-        const nextTheme = loadTheme(next);
-        Object.assign(colors, nextTheme);
-        config.theme = next;
-        saveConfig(config);
-        renderer.requestRender();
-        return;
-      }
+    }
+    if (key.name === "t" && !searchInput.focused) {
+      const current = config.theme;
+      const themes = listThemes();
+      const idx = themes.indexOf(current);
+      const next = themes[(idx + 1) % themes.length];
+      const nextTheme = loadTheme(next);
+      Object.assign(colors, nextTheme);
+      config.theme = next;
+      saveConfig(config);
+      renderer.requestRender();
+      return;
     }
     if (key.name === "c" && !searchInput.focused) {
       if (currentOverlay === "compare") {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "github-search-cli",
-  "version": "9.2.1",
+  "version": "9.3.0",
   "description": "Terminal-native GitHub search — search, trending, bookmarks, history, deep-dive, compare, export, and more. All keyboard-driven, no browser needed.",
   "type": "module",
   "packageManager": "bun@1",

package/src/compare.ts

@@ -20,11 +20,12 @@ export function buildComparisonTable(repos: Repo[]): string {
   rows.push({ label: "Language", values: repos.map((r) => r.language ?? "—") });
   rows.push({ label: "Created", values: repos.map((r) => r.createdAt?.slice(0, 7) ?? "—") });
   rows.push({ label: "Updated", values: repos.map((r) => r.updatedAt?.slice(0, 10) ?? "—") });
-  rows.push({ label: "Topics", values: repos.map((r) => (r.topics?.slice(0, 3).join(", ") ?? "—") || "—") });
+  rows.push({ label: "Topics", values: repos.map((r) => (r.topics?.slice(0, 5).join(", ") ?? "—") || "—") });
+  rows.push({ label: "Description", values: repos.map((r) => (r.description?.slice(0, 50) ?? "—")) });
+  rows.push({ label: "URL", values: repos.map((r) => r.url || "—") });
 
   // Calculate the max width for each column
-  const colW = Math.max(15, ...names.map((n) => n.length));
-  const valW = Math.max(8, ...rows.flatMap((r) => r.values.map((v) => v.length)));
+  const colW = Math.max(35, ...names.map((n) => n.length));
 
   // Header row
   const header = ["".padEnd(14), ...names.map((n) => n.padEnd(colW))];

package/src/markdown-render.ts

@@ -0,0 +1,216 @@
+/**
+ * Minimal markdown-to-terminal-text renderer.
+ *
+ * Renders headings, code blocks, lists, tables, blockquotes, horizontal rules,
+ * bold/italic, and links into a plain-text representation suitable for a
+ * TextRenderable in the TUI.
+ */
+
+interface RenderOptions {
+  /** Terminal width for wrapping and table layout. Default: 80. */
+  width?: number;
+}
+
+/** Render markdown text to terminal-friendly plain text. */
+export function renderMarkdown(md: string, opts: RenderOptions = {}): string {
+  const width = opts.width ?? 80;
+  const lines = md.split("\n");
+  const out: string[] = [];
+  let inCodeBlock = false;
+  let codeLang = "";
+  let codeLines: string[] = [];
+  let inTable = false;
+  let tableRows: string[][] = [];
+
+  function flushCode() {
+    if (codeLines.length === 0) return;
+    const maxLen = Math.max(...codeLines.map((l) => l.length), codeLang.length);
+    const barW = Math.min(maxLen + 4, width - 2);
+    out.push("┌" + "─".repeat(barW) + "┐");
+    if (codeLang) out.push("│ " + codeLang.padEnd(barW - 1) + "│");
+    for (const cl of codeLines) {
+      out.push("│ " + cl.padEnd(barW - 1) + "│");
+    }
+    out.push("└" + "─".repeat(barW) + "┘");
+    out.push("");
+    codeLines = [];
+    codeLang = "";
+  }
+
+  function flushTable() {
+    if (tableRows.length === 0) return;
+    const dataRows = tableRows.filter(
+      (r) => !r.every((c) => /^[\s\-:|]+$/.test(c)),
+    );
+    if (dataRows.length === 0) {
+      tableRows = [];
+      return;
+    }
+    const colCount = Math.max(...dataRows.map((r) => r.length));
+    const colW: number[] = [];
+    for (let ci = 0; ci < colCount; ci++) {
+      colW.push(
+        Math.max(
+          3,
+          ...dataRows.map((r) => (r[ci] ?? "").length),
+        ),
+      );
+    }
+    const totalW = colW.reduce((a, b) => a + b, 0) + colCount * 3 - 1;
+    if (totalW > width) {
+      for (const row of dataRows) {
+        out.push("  " + row.map((c) => c.trim()).join(" │ "));
+      }
+    } else {
+      const sep = "┌" + colW.map((w) => "─".repeat(w + 2)).join("┬") + "┐";
+      const mid = "├" + colW.map((w) => "─".repeat(w + 2)).join("┼") + "┤";
+      const end = "└" + colW.map((w) => "─".repeat(w + 2)).join("┴") + "┘";
+      out.push(sep);
+      dataRows.forEach((row, i) => {
+        const cells = colW.map((w, ci) => (row[ci] ?? "").trim().padEnd(w));
+        out.push("│ " + cells.map((c) => ` ${c} `).join("│") + "│");
+        if (i === 0) out.push(mid);
+      });
+      out.push(end);
+    }
+    out.push("");
+    tableRows = [];
+  }
+
+  function renderInline(line: string): string {
+    // Bold
+    line = line.replace(/\*\*(.+?)\*\*/g, "$1");
+    // Italic (no asterisks)
+    line = line.replace(/\*([^*]+?)\*/g, "$1");
+    // Inline code
+    line = line.replace(/`([^`]+)`/g, "⟨$1⟩");
+    // Links [text](url)
+    line = line.replace(/\[([^\]]+)\]\(([^)]+)\)/g, "$1 ($2)");
+    // Images
+    line = line.replace(/!\[([^\]]*)\]\([^)]+\)/g, "🖼 $1");
+    return line;
+  }
+
+  function wrapLine(line: string, maxW: number): string[] {
+    if (line.length <= maxW) return [line];
+    const parts: string[] = [];
+    let remaining = line;
+    while (remaining.length > maxW) {
+      let breakAt = maxW;
+      const spaceIdx = remaining.lastIndexOf(" ", maxW);
+      if (spaceIdx > maxW / 2) breakAt = spaceIdx;
+      parts.push(remaining.slice(0, breakAt));
+      remaining = remaining.slice(breakAt).replace(/^ /, "");
+    }
+    if (remaining) parts.push(remaining);
+    return parts;
+  }
+
+  for (let i = 0; i < lines.length; i++) {
+    let line = lines[i];
+
+    // Code block boundaries
+    if (/^```/.test(line)) {
+      if (inCodeBlock) {
+        flushCode();
+        inCodeBlock = false;
+      } else {
+        inCodeBlock = true;
+        codeLang = line.slice(3).trim();
+      }
+      continue;
+    }
+    if (inCodeBlock) {
+      codeLines.push(line);
+      continue;
+    }
+
+    // Table row
+    if (line.includes("|")) {
+      const cells = line
+        .split("|")
+        .map((c) => c.trim())
+        .filter((_, idx, arr) => idx > 0 && idx < arr.length - 1);
+      if (cells.length >= 2) {
+        inTable = true;
+        tableRows.push(cells);
+        continue;
+      }
+    } else if (inTable) {
+      flushTable();
+      inTable = false;
+    }
+
+    // Horizontal rule
+    if (/^(\s*[-*_]){3,}\s*$/.test(line)) {
+      out.push("─".repeat(Math.min(width, 40)));
+      out.push("");
+      continue;
+    }
+
+    // Headings
+    const headingMatch = line.match(/^(#{1,6})\s+(.+)/);
+    if (headingMatch) {
+      const level = headingMatch[1].length;
+      const text = renderInline(headingMatch[2]);
+      const prefix = "█ ".repeat(level);
+      out.push("");
+      out.push(prefix + text);
+      out.push("");
+      continue;
+    }
+
+    // Blockquote
+    const bqMatch = line.match(/^>\s?(.*)/);
+    if (bqMatch) {
+      const inner = renderInline(bqMatch[1]);
+      for (const wl of wrapLine(inner, width - 4)) {
+        out.push("│ " + wl);
+      }
+      continue;
+    }
+
+    // Unordered list
+    const ulMatch = line.match(/^(\s*)[-*+]\s+(.*)/);
+    if (ulMatch) {
+      const indent = ulMatch[1].length;
+      const depth = Math.floor(indent / 2);
+      const prefix = "  ".repeat(depth) + "• ";
+      const inner = renderInline(ulMatch[2]);
+      for (const wl of wrapLine(inner, width - prefix.length)) {
+        out.push(prefix + wl);
+      }
+      continue;
+    }
+
+    // Ordered list
+    const olMatch = line.match(/^(\s*)\d+\.\s+(.*)/);
+    if (olMatch) {
+      const indent = olMatch[1].length;
+      const depth = Math.floor(indent / 2);
+      const prefix = "  ".repeat(depth) + "◦ ";
+      const inner = renderInline(olMatch[2]);
+      for (const wl of wrapLine(inner, width - prefix.length)) {
+        out.push(prefix + wl);
+      }
+      continue;
+    }
+
+    // Empty line
+    if (line.trim() === "") {
+      out.push("");
+      continue;
+    }
+
+    // Paragraph
+    const rendered = renderInline(line);
+    for (const wl of wrapLine(rendered, width)) {
+      out.push(wl);
+    }
+  }
+
+  if (inCodeBlock) flushCode();
+  if (inTable) flushTable();
+
+  return out.join("\n");
+}

package/src/tui.ts

@@ -79,6 +79,7 @@ import {
 import { saveSession, restoreSession } from "./session";
 import { fetchDeepDive, buildDeepDiveText } from "./deepdive";
 import { buildComparisonTable } from "./compare";
+import { renderMarkdown } from "./markdown-render";
 import { fetchTopics, type TopicItem } from "./explore";
 import { exportToFile, type ExportFormat } from "./output";
 import { listThemes, loadTheme } from "./themes";
@@ -1709,17 +1710,8 @@ export async function launchBrowser(theme_override?: string): Promise<void> {
       if (!text) {
         readmeText.content = `  (no README found for ${repo.fullName})`;
       } else {
-        // ponytail: just strip markdown formatting, show full text
-        const stripped = text
-          .replace(/```[\s\S]*?```/g, "[code block]")
-          .replace(/#{1,6}\s/g, "")
-          .replace(/\*\*(.+?)\*\*/g, "$1")
-          .replace(/\[(.+?)\]\(.+?\)/g, "$1")
-          .replace(/`([^`]+)`/g, "$1")
-          .split("\n")
-          .map((l) => l.trimEnd())
-          .join("\n");
-        readmeText.content = stripped;
+        const termW = Math.max(40, (process.stdout.columns || 80) - 4);
+        readmeText.content = renderMarkdown(text, { width: termW });
       }
     } catch {
       readmeText.content = "  Failed to load README";
@@ -2646,6 +2638,8 @@ ${pack.description ?? ""}`;
         showLeaderMenu();
       }
       return;
+    }
+
     // 't' toggles theme and persists to config
     if (key.name === "t" && !searchInput.focused) {
       const current = config.theme;
@@ -2659,7 +2653,6 @@ ${pack.description ?? ""}`;
       renderer.requestRender();
       return;
     }
-    }
 
     // 'c' toggles compare on selected repo (when not typing in search)
     if (key.name === "c" && !searchInput.focused) {